ci: fix TICS workflow failing on integration tests
Metadata
Current evaluation
Merged fix for TICS CI workflow failing on integration tests. The runner lacked LXD and Multipass. Added PYTEST_ADDOPTS to exclude lxd_instance and multipass_instance markers during coverage runs. Approved and merged after passing all CI checks.
Suggested action: —
No scores available.
Issue body
The TICS workflow (`.github/workflows/tics.yaml`) runs `make test-coverage` on a self-hosted runner that doesn't have LXD or Multipass installed. Since `test-coverage` runs the full pytest suite with no marker filtering, it picks up integration tests marked `lxd_instance`/`multipass_instance`, which fail with `FileNotFoundError: multipass` etc.
Fixes: https://github.com/canonical/craft-providers/actions/runs/29362672909/job/87186433705
## Fix
Default `PYTEST_ADDOPTS` to exclude these markers for the `test-coverage` target (only when not already set), mirroring how `qa.yaml`'s test jobs already filter markers via `PYTEST_ADDOPTS`.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged fix for TICS CI workflow failing on integration tests. The runner lacked LXD and Multipass. Added PYTEST_ADDOPTS to exclude lxd_instance and multipass_instance markers during coverage runs. Approved and merged after passing all CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
5
Complexity:
15
Confidence:
90
|
needs review | Fixes TICS CI workflow by excluding LXD/Multipass integration tests from coverage runs via PYTEST_ADDOPTS, matching existing qa.yaml behavior. Currently unreviewed and awaiting CI validation. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #52 fix: use self hosted runner for tics | starflow | merged | Merged a CI fix switching to self-hosted runners. This resolves failing LXD-dependent tests by leveraging runners preconfigured for LXD. All CI checks passed prior to merge. |