fix(test): fix lifecycle spread tests for tox-uv
Metadata
Current evaluation
Merged PR #2416 as 4fbe5e18, restoring tox-uv setup omitted during init-template updates so lifecycle spread tests pass on Ubuntu 24.04.
Resolution: closed resolved
Reason: The PR was approved and merged into main as commit 4fbe5e18. Its lifecycle spread-test changes resolved CHARMCRAFT-636, and the Ubuntu 24.04 snap-tests check passed.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This was left out when updating init templates. This PR should make the Ubuntu 24.04 spread tests succeed.
CHARMCRAFT-636
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2416 as 4fbe5e18, restoring tox-uv setup omitted during init-template updates so lifecycle spread tests pass on Ubuntu 24.04. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #2416 for CHARMCRAFT-636, correcting lifecycle spread-test setup left out of init-template updates so Ubuntu 24.04 tests pass with tox-uv. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix for lifecycle spread tests involving tox-uv, resolving CHARMCRAFT-636. Updates init templates to ensure Ubuntu 24.04 test success. Approved and merged after CI checks, with expected failures on older Ubuntu versions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for lifecycle spread tests related to tox-uv. Updates init templates to restore Ubuntu 24.04 spread test functionality, resolving CHARMCRAFT-636. |
Update history
No update history recorded yet.
Related work
-
Caused By:
canonical/charmcraft#2312
(confidence 98%)
That template migration adopted uv for Kubernetes and machine charms but omitted the corresponding lifecycle spread-test setup.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1484 fix(spread): run tox in lifecycle spread test | charmcraft | merged | Merged #1484, which adds tox execution to the lifecycle spread test and fixes #1471. | |
| #215 test: add tox environments for snapping and running spread tests | rockcraft | closed | Approved but never merged; the tox convenience change was closed unmerged after prolonged inactivity, with README updates and four review comments unresolved. | |
| #1674 fix: make tox tests succeed | charmcraft | merged | Merged in commit b24ccd5b: fixed the tox environment variable and added a workaround for canonical/operator#1212, allowing tox tests to succeed. |