tests: fix resource spread test
Metadata
Current evaluation
Merged #1223 (commit 35993996), fixing the flaky store-resources spread test and advancing #1193; related store fixes landed in #1226 and #1234.
Resolution: closed resolved
Reason: PR #1223 was approved and merged as commit 35993996, directly fixing the resource spread test associated with #1218.
Evaluated by: mr-cal-bot
No scores available.
Issue body
progress toward #1193
Libraries are fixed in #1234
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #1223 (commit 35993996), fixing the flaky store-resources spread test and advancing #1193; related store fixes landed in #1226 and #1234. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #1223 as commit 35993996, fixing the resource spread test and advancing #1193; related library and store-test fixes landed in #1234 and #1226. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #1223 (commit 35993996) fixing the flaky/broken resource spread test, progress toward umbrella issue #1193; remaining store-test and library fixes continued in #1226 and #1234. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #1223 (commit 35993996) with a two-file fix to the resource spread test, advancing #1193; library and remaining store-test fixes continued in #1234 and #1226. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as commit 35993996, fixing the resource spread test and advancing #1193. Related store and library failures were handled in separate PRs. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to resolve the resource spread test failure. The change advances progress toward issue #1193, with related library and store test fixes handled in separate pull requests #1234 and #1226. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/charmcraft#1218
(confidence 100%)
PR #1223 fixed the flaky store-resources spread test tracked by #1218.
-
Related To:
canonical/charmcraft#1193
(confidence 100%)
The PR identifies itself as progress toward the broader spread-test fix effort.
-
Related To:
canonical/charmcraft#1234
(confidence 95%)
The author cited this merged PR as the fix for the library-related tests.
-
Related To:
canonical/charmcraft#1226
(confidence 95%)
The author cited this PR as fixing remaining broken store tests.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1218 tests: store resources spread test is flaky | charmcraft | closed | Flaky store resources spread test was fixed by merged PR canonical/charmcraft#1223 in commit 35993996. | |
| #1460 fix(tests): make store spread tests less flaky | charmcraft | merged | Merged PR #1460 as fd5bd6ba, making store spread tests less timing-sensitive and fixing flaky upload/release checks in #1429. | |
| #5954 tests: fix store tests | snapcraft | merged | PR #5954 merged as commit 0be1e384d: store spread tests switched to new staging credentials/key test-staging-key-2 and to wheel-based automated tests (as in #5885). | |
| #1290 test(spread): clean up store tests | charmcraft | merged | Merged in commit 34ab4bdd, cleaning up store spread tests by deleting 28 lines across four files; the remaining library-test failure was fixed separately in #1292. | |
| #1712 Store resources spread test is broken | charmcraft | closed | CI's store-resources spread test returned an empty value; PR #1761 temporarily disabled the related upload-resource test, but no permanent fix was recorded. | |
| #1366 tests(spread): make release test less flaky | charmcraft | merged | Merged #1366 at commit 05b18a03, reducing flakiness in the Spread release test and resolving #1362. | |
| #631 Fix the store integration tests | snapcraft | merged | Merged PR #631 (commit 351d4aaac) fixed store integration tests by pausing after rapid snap registrations so later tests did not inherit the rate-limit error. | |
| #3089 ci: move staging store tests to spread | snapcraft | closed | Superseded by merged PR #3090, which moved staging store tests to Spread in commit 3a037fb11. | |
| #1984 tests: update store tests user | snapcraft | merged | Merged PR #1984 (3c313dab1), updating the Store test account, removing two obsolete tests, and revising related test data, documentation, and warnings. | |
| #1736 fix: fix some edge cases with resource uploads | charmcraft | merged | Merged PR #1736 as commit 97603da4, fixing resource-upload edge cases and resolving charmcraft#1712's empty-resource spread-test failure. |