ai-workspace-infra/playbooks
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(litellm): repair macOS dependency version probe one-liner

Browse Source 
The "Inspect installed LiteLLM dependency versions" probe was written as a
multi-line Python program under YAML `>-` folding, which collapses every
newline into a space. The resulting single logical line contained a
`for ... : try: ... except:` block, which is a SyntaxError. With
`failed_when: false` the failure was swallowed, leaving stdout empty, and
the subsequent `set_fact` crashed in `from_json('')` with
"Expecting value: line 1 column 1 (char 0)".

Rewrite the probe as a genuinely single-line program (dict/list
comprehensions over importlib.metadata.distributions(), joined by `;`),
and harden the decision `set_fact` with `default('{}', true)` so an empty
or malformed probe degrades to "install required" instead of aborting the
play.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Haitao Pan 2026-06-22 12:16:57 +08:00
parent f4a30b9e01
commit ce2070e779
1 changed files with 14 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
roles/vhosts/litellm/tasks/main.yml
View File

@ -141,19 +141,19 @@
become_user: "{{ litellm_service_user }}"
when: litellm_install_marker.stat.exists
# The probe must stay a valid single-line Python program: the `>-` folding
# collapses every newline into a space, so a `for ... : try: ... except:` block
# would become one illegal logical line, crash with SyntaxError, and (with
# failed_when:false) leave stdout empty -> the from_json below then explodes.
# Build the version map from importlib.metadata.distributions() with dict/list
# comprehensions, which are valid single statements joined by semicolons.
- name: Inspect installed LiteLLM dependency versions
ansible.builtin.command:
cmd: >-
{{ litellm_python_executable }} -c
"import importlib.metadata as m, json;
packages = ['litellm', 'prisma', 'psycopg2-binary'];
data = {};
for package in packages:
try:
data[package] = m.version(package)
except Exception:
data[package] = 'missing';
print(json.dumps(data))"
dists = {d.metadata['Name'].lower(): d.version for d in m.distributions()};
print(json.dumps({p: dists.get(p, 'missing') for p in ['litellm', 'prisma', 'psycopg2-binary']}))"
register: litellm_dependency_versions
changed_when: false
failed_when: false
@ -162,13 +162,16 @@
- name: Decide whether LiteLLM dependencies need installation
ansible.builtin.set_fact:
# default('{}', true) also substitutes when stdout is an empty string (not
# just undefined), so a failed/empty probe degrades to "install required"
# instead of crashing from_json with "Expecting value: line 1 column 1".
litellm_dependency_install_required: >-
{{
not litellm_install_marker.stat.exists
or (litellm_install_marker_content.stdout | default('') | trim != litellm_package_spec)
or (litellm_dependency_versions.stdout | default('{}') | from_json).litellm == 'missing'
or (litellm_dependency_versions.stdout | default('{}') | from_json).prisma == 'missing'
or (litellm_dependency_versions.stdout | default('{}') | from_json)['psycopg2-binary'] == 'missing'
or (litellm_dependency_versions.stdout | default('{}', true) | from_json).litellm == 'missing'
or (litellm_dependency_versions.stdout | default('{}', true) | from_json).prisma == 'missing'
or (litellm_dependency_versions.stdout | default('{}', true) | from_json)['psycopg2-binary'] == 'missing'
}}
- name: Ensure LiteLLM and DB dependencies are installed