← 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

Broken spread tests were handled in #1197/#1216 and #1223/#1226/#1234; the magma test remains disabled, not genuinely fixed.

Resolution: closed superseded

Reason: The umbrella tracker was replaced by follow-up work, including magma disabling commit bcb648c7 and store/resource fixes 35993996, f4dd212d, and 2f184582. The magma test is still manual, so this was not a full repair.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed superseded Broken spread tests were handled in #1197/#1216 and #1223/#1226/#1234; the magma test remains disabled, not genuinely fixed.
stealth/space-bunny-alpha — closed superseded Magma spread testing was disabled in #1216 (bcb648c7); store/resource fixes landed in #1223, #1226 and #1234, superseding this umbrella tracker.
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.

Update history

No update history recorded yet.

Related work

  • Likely Fixed By: charmcraft#1216 (confidence 99%)

    Merged as commit bcb648c7 to disable the magma spread test.

  • Likely Fixed By: charmcraft#1197 (confidence 99%)

    Merged as commit 4b34df89 to disable externally broken spread tests.

  • Duplicate Of: charmcraft#1215 (confidence 98%)

    The magma spread-test follow-up was recorded as a duplicate of the umbrella issue.

  • Likely Fixed By: charmcraft#1223 (confidence 96%)

    Merged as commit 35993996 to fix the flaky resource spread test.

  • Likely Fixed By: charmcraft#1226 (confidence 90%)

    Merged as commit f4dd212d to fix related store-task tests.

  • Likely Fixed By: charmcraft#1234 (confidence 88%)

    Merged as commit 2f184582 to fix publish-lib behavior without metadata.yaml.

Related issues

Issue Project State Summary Similarity
#1215 tests: magma pack spread test is broken charmcraft closed Closed by maintainer lengau as a duplicate of #1193. The broken magma spread test was disabled in PR #1216 (commit bcb648c7); spread fixes followed via #1223, #1226 and #1234.
86%
#1216 tests: disable broken magma test charmcraft merged Merged PR #1216 (commit bcb648c7) temporarily disabled the broken magma spread test in2 files; approved by 2 reviewers, pending umbrella fix issue #1193.
78%
#1197 ci: temporarily disable externally broken spread tests charmcraft merged Merged PR #1197 as commit 4b34df89, temporarily disabling spread tests broken by third-party dependency issues; re-enablement is tracked in #1193.
71%