← Back to issue list

tests: disable broken magma test

View original Github issue

Metadata

Project
charmcraft
Number
#1216
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #1216 (commit bcb648c7) temporarily disabled the broken magma spread test in2 files; approved by 2 reviewers, pending umbrella fix issue #1193.

Resolution: closed resolved

Reason: The PR itself was merged to main as commit bcb648c7, disabling the failing magma test as a temporary measure. The follow-up work tracked in #1193 later landed via #1223 (35993996), #1226 (f4dd212d) and #1234 (2f184582).

Evaluated by: mr-cal-bot

No scores available.

Issue body

Temporary until we can complete #1193

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #1216 (commit bcb648c7) temporarily disabled the broken magma spread test in2 files; approved by 2 reviewers, pending umbrella fix issue #1193.
stealth/space-bunny-alpha — closed resolved Merged PR #1216 (bcb648c7) temporarily disabled the broken magma test pending completion of charmcraft#1193; approved by two reviewers.
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged a temporary fix to disable a broken magma test, preventing CI failures while awaiting completion of issue #1193. Integrated by maintainer lengau.

Update history

No update history recorded yet.

Related work

  • Blocked By: canonical/charmcraft#1193 (confidence 92%)

    PR body states it is temporary until #1193 (Fix broken spread tests) is completed; #1193 was later closed by #1223/#1226/#1234.

  • Superseded By: canonical/charmcraft#1223 (confidence 78%)

    Merged 35993996 fixing the resource spread test, part of the #1193 remediation that the temporary disable anticipated.

  • Related To: canonical/charmcraft#1234 (confidence 70%)

    Merged 2f184582 allowing publish-lib without metadata.yaml, another #1193 follow-up.

  • Related To: canonical/charmcraft#1226 (confidence 65%)

    Merged f4dd212d removing k8s info from machine charm template, fixing store-task spread tests under #1193.

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.
83%
#1193 Fix broken spread tests charmcraft closed Broken spread tests were handled in #1197/#1216 and #1223/#1226/#1234; the magma test remains disabled, not genuinely fixed.
78%