tests(spread): don't use microk8s for dependencies tests
Metadata
Current evaluation
Merged to remove microk8s from spread dependency tests, deleting seven lines. Approved by two reviewers and dependent on PR #1521.
Suggested action: —
No scores available.
Issue body
Requires: #1521
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to remove microk8s from spread dependency tests, deleting seven lines. Approved by two reviewers and dependent on PR #1521. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to remove microk8s from the dependencies spread tests. Implemented after prerequisite #1521, the update switches the test environment to an alternative dependency verification method. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged update removes microk8s as a dependency for spread tests, relying on #1521. The change streamlines test execution by eliminating the microk8s requirement. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2978 tests: add microk8s spread test | snapcraft | merged | Merged after single reviewer approval and passing CI. Adds a spread test to verify microk8s builds and installs correctly, deferring instrumentation to a future update. Changes affect 2 files (+35/-0 lines). | |
| #3432 spread tests: remove legacy plugin tests | snapcraft | merged | Merged removal of legacy plugin tests from the spread test suite. Deleted 665 lines across 29 files. Approved by reviewer, passed CI, and maintained coverage. Unrelated test failures were addressed in a separate PR. | |
| #1290 test(spread): clean up store tests | charmcraft | merged | Merged cleanup of spread store tests, removing 28 lines across four files. The underlying library test failure is fixed in PR #1292. Approved by two reviewers. | |
| #1366 tests(spread): make release test less flaky | charmcraft | merged | Merged changes that reduce flakiness in the spread release test, resolving issue #1362. Approved by two reviewers, the update modifies one file (+17/-7) after addressing feedback. | |
| #1503 test(spread): disable centos spread test | charmcraft | merged | Merged to disable the CentOS spread test. Approved by two reviewers, the change modifies one file with three lines added and is now part of the main branch. | |
| #1502 test(spread): disable centos spread test | charmcraft | merged | Merged a change disabling the CentOS spread test. The update modifies one file with three added lines, received approval from two reviewers, and was integrated without CI checks or comments. | |
| #1084 tests: check for submodules before running spread | charmcraft | merged | Merged changes verifying the snapd-testing-tools submodule before running Spread tests. Adds a clear error if missing. Approved by a reviewer, passed CI, and merged with minimal modifications. | |
| #4105 tests: check for submodules before running spread | snapcraft | merged | Merged. Adds a pre-run check for the snapd-testing-tools submodule in spread tests to provide a clear error if missing. Approved by two reviewers, passed CI, and introduced no coverage changes. | |
| #240 tests: check for submodules before running spread | rockcraft | merged | Merged. Adds a pre-check for the snapd-testing-tools submodule before running spread tests, providing a clear error if missing. Approved by two reviewers, passed CI, and merged successfully. | |
| #6197 test(spread): debug spread failures 2 | snapcraft | closed | Closed without merging. The author reproduced the failing spread tests in pull request #6198, making this debugging effort redundant and abandoned. |