← Back to issue list

Fix broken spread tests

View original Github issue

Metadata

Project
charmcraft
Number
#1193
Type
issue
State
closed
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Closed after the broken spread tests were resolved. Store tests were fixed in PR #1195, the magma pack test tracked in issue #1215, and other failures were addressed separately. The tracking issue was superseded by these targeted resolutions.

Suggested action: —

No scores available.

Issue body

### What needs to get done The following spread tests are broken and need fixing: * ~~Notebook_operators bundle does not build (Probably related to a dependency update, as it fails on stable versions too)~~ * ~~Prometheus k8s operator fails due to a dependency issue~~ * ~~Store tests need to be decoupled from an account~~ These are okay for now with #1195 * magma pack test (see #1215 ) ### Why it needs to get done CI always looks like it's failing because of these and we're spending a lot of time reviewing the failures and determining that they're just the same thing.

Evaluation history

Date Model Scores Action Summary
qwen3.6-35b-a3b-mtp-q6 — — Closed after the broken spread tests were resolved. Store tests were fixed in PR #1195, the magma pack test tracked in issue #1215, and other failures were addressed separately. The tracking issue was superseded by these targeted resolutions.
qwen/qwen3.6-35b-a3b — — Closed as resolved. The maintainer updated the checklist to mark the broken spread tests as fixed or deferred, clearing CI pipeline noise.
qwen/qwen3.6-35b-a3b — — Resolved and closed after individual test failures were fixed in separate PRs and issues. Store tests were addressed in #1195, the magma pack test in #1215, and dependency failures were patched independently.
qwen3.6-35b-a3b-mtp-q6 — — Closed as superseded. Broken spread tests were resolved via referenced fixes in #1195 and #1215, with remaining items deprioritized or addressed separately.
qwen3.6-35b-a3b-mtp-q6 — — Broken spread tests causing CI noise were resolved. Items were marked as fixed or deferred via related issues and PRs. The tracking issue was closed after test failures were mitigated.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
74%
#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.
74%
#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.
74%
#1215 tests: magma pack spread test is broken charmcraft closed Superseded by #1193. The magma spread test requires coordination with the telco team to re-integrate the charm into the CI suite.
73%
#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.
72%
#3966 Spread multipass tests snapcraft merged Merged a minor refactor moving manual multipass tests out of the legacy directory. An unrelated spread test failure was noted and resolved by a separate pull request.
72%
#1216 tests: disable broken magma test charmcraft merged Merged a temporary fix disabling a broken magma test pending issue #1193. Approved by two reviewers, the pull request modifies two files with minor configuration adjustments to bypass the failing test.
71%
#1223 tests: fix resource spread test charmcraft merged Merged to fix the resource spread test, advancing progress toward issue #1193. The update modified two files (+44/-23 lines) and was approved by two reviewers. Related library and store test corrections are addressed in separate pull requests.
71%
#2763 fix(tests/spread): fix reactive spread tests charmcraft merged Merged after resolving failing reactive spread tests. The changes split the test suite into legacy and resolute variants, switched to binary wheels to fix dependency resolution, and removed obsolete flags. Approved by reviewers and integrated.
70%
#6183 test(spread): debug spread failure snapcraft closed Closed without merging. The reported spread panic was reproduced and resolved in pull request #6198, superseding this debug branch.
70%