fix(setup-python-runner): honour use-lxd input to skip LXD installation
Metadata
Current evaluation
Merged fix: setup-python-runner declares a use-lxd input, gates the Setup LXD step on it, forwards it from test-python.yaml (defaults true), plus regression self-tests. Focal LXD 6 check failure deemed external.
Resolution: closed resolved
Reason: PR canonical/starflow#166 was approved by 2 reviewers and merged into main, adding the use-lxd input to setup-python-runner, gating the 'Setup LXD' step on it (previously only runner.os == 'Linux'), passing the input through test-python.yaml, and adding regression self-tests; the sole failing check (LXD 6 focal) was attributed by the maintainer to the upstream canonical/setup-lxd#41.
Evaluated by: mr-cal-bot
No scores available.
Issue body
`use-lxd: false` had no effect: `setup-python-runner` never declared a `use-lxd` input (its "Setup LXD" step only checked `runner.os == 'Linux'`), and `test-python.yaml` never passed its `use-lxd` input through to the action.
Adds the input to the action, gates the LXD setup step on it, wires it through from `test-python.yaml`, and defaults it to `true` everywhere to preserve current behaviour. Adds regression self-tests.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged fix: setup-python-runner declares a use-lxd input, gates the Setup LXD step on it, forwards it from test-python.yaml (defaults true), plus regression self-tests. Focal LXD 6 check failure deemed external. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/starflow#166: setup-python-runner now honours use-lxd, conditionally skips LXD setup, forwards the input through test-python.yaml, defaults to true, and adds regression tests. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged fix to make setup-python-runner respect the use-lxd input, gating LXD installation accordingly. Defaults to true, wires the input through test-python.yaml, and adds regression tests. CI passed with one noted external failure. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
0
Complexity:
15
Confidence:
95
|
needs review | Adds use-lxd input to setup-python-runner to skip LXD installation, wires it through test-python.yaml, and defaults to true. Currently approved by two reviewers with one unresolved comment; CI passes except a known external focal failure. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
5
Complexity:
15
Confidence:
90
|
needs review | Fixes setup-python-runner to respect the use-lxd input by adding the missing input declaration, gating the LXD setup step, and wiring it through test-python.yaml. Currently approved with one unresolved comment and mostly passing CI. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
5
Complexity:
15
Confidence:
90
|
needs review | Adds use-lxd input to skip LXD installation, gates setup step, wires through workflow, defaults to true. Approved by 1 reviewer, 1 unresolved comment, CI mostly passing with 1 external focal failure. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
5
Complexity:
15
Confidence:
90
|
needs review | Adds missing use-lxd input to setup-python-runner action, gates LXD setup on it, and wires it through from test workflow. Currently pending maintainer review with one unresolved comment and mostly passing CI. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
5
Complexity:
15
Confidence:
90
|
needs review | Fixes setup-python-runner to respect the use-lxd input for skipping LXD installation. Wires input through workflow, defaults to true, and adds regression tests. Currently pending maintainer review with CI passing. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
5
Complexity:
15
Confidence:
90
|
needs review | Fixes a bug where the use-lxd input was ignored by adding the input declaration, gating the LXD setup step, wiring it through test workflows, and adding regression tests. Currently awaiting review. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
5
Complexity:
15
Confidence:
85
|
needs review | Fixes setup-python-runner to respect the use-lxd input for skipping LXD installation. Wires input through test workflow, defaults to true, and adds regression tests. Currently pending maintainer review with one failing CI job. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
5
Complexity:
20
Confidence:
85
|
needs review | Fixes setup-python-runner to respect the use-lxd input by adding the input declaration, gating the LXD setup step, and wiring it through test-python.yaml. Currently pending review with 4 unresolved comments and 2 failing CI jobs. |
Update history
| Date | Change |
|---|---|
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| created |
Related work
-
Related To:
setup-lxd#41
(confidence 88%)
Upstream issue cited by lengau as the home for the external Focal LXD 6 CI failure that left one check red.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #56 feat(test-python): optionally setup lxd | starflow | closed | Merged PR #56 added optional LXD setup to test-python, enabling craft-providers testing; all CI checks passed. | |
| #42 feat(test-python): install lxd | starflow | merged | Merged #42 as commit 2c6a4a5, adding opt-in LXD setup for test-python hosted runners so use-lxd works with craft-providers. |