ci: temporarily disable externally broken spread tests
Metadata
Current evaluation
Merged to temporarily disable broken spread tests, restoring CI. Breakage stems from third-party dependency issues. Tests will be re-enabled once upstream fixes land. Approved by one reviewer, modified 3 files.
Suggested action: —
No scores available.
Issue body
Related to #1193, this should make our CI succeed. We will need to re-enable these tests once the issues with those charms are resolved.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to temporarily disable broken spread tests, restoring CI. Breakage stems from third-party dependency issues. Tests will be re-enabled once upstream fixes land. Approved by one reviewer, modified 3 files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a change to temporarily disable externally broken spread tests, resolving CI failures caused by third-party dependency issues. Tests are tracked under #1193 and will be re-enabled once upstream build problems are resolved. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to temporarily disable externally broken spread tests and restore CI. Related to #1193; tests will be re-enabled once upstream dependency and build issues are resolved. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1761 tests(spread): temporarily disable failing test | charmcraft | merged | Merged to temporarily disable a failing upload-resource test in the spread suite due to bug #1760. Approved by two reviewers, passed CI, and modified one file. | |
| #307 tests: disable broken integration test | craft-application | merged | Merged a temporary workaround disabling a broken integration test to restore CI functionality, pending a permanent fix in issue #306. Approved by two reviewers with passing checks. | |
| #2476 tests: disable spread colcon tests | snapcraft | merged | Merged a pull request temporarily disabling spread colcon tests after upstream changes broke the plugin. Approved and merged to maintain CI stability while the upstream issue is resolved. | |
| #1036 ci: temporarily disable external fetch service spread test | craft-application | merged | Merged a CI change to temporarily disable a failing external fetch service spread test. The failure stems from an unrelated upstream issue (#1035). Disabling it prevents CI noise and false negatives on subsequent pull requests. | |
| #2417 fix: fix a series of breaking spread tests | charmcraft | merged | Merged after fixing multiple breaking spread tests. Commits resolve individual failures, while remaining broken tests are marked non-blocking for core24 migration. Approved by reviewers and merged following CI validation. | |
| #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. | |
| #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. | |
| #534 test(spread): workaround for flaky test | fetch-service | merged | Merged a workaround for a flaky spread test (issue #533). The change adjusts one file (+2/-1 lines) to skip a no-op name query check while retaining other validations. Approved by two reviewers and passed all CI checks. | |
| #2032 ci: fixes to several weekly spread tests | charmcraft | merged | Merged CI fixes for several weekly spread tests. Approved by two reviewers and passed all checks with minimal changes across three files. Addresses initial test failures as a starting point for broader improvements. | |
| #127 Fix spread tests | imagecraft | merged | Merged after receiving approval from two reviewers and passing all CI checks. The changes fix spread tests across 11 files (+49/-38 lines). |