tests/spread: fix reactive spread tests on older channels
Metadata
Current evaluation
Closed unmerged after 86 days; no resolving commit or replacement PR was identified, and the Conventional Commits policy check remained failing.
Resolution: closed stale
Reason: The proposed two-line test-template change was closed without a merge, resolving commit, maintainer comment, or identified superseding work after 86 days of inactivity.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Removes `--upgrade-buildvenv-core-deps` and `--ignore-requires-python` from the base reactive charmcraft.yaml template, as they are not supported by charm-build in older snap channels (e.g. stable). They are still injected dynamically for 26.04 where the candidate channel of charm-tools supports them.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed stale | Closed unmerged after 86 days; no resolving commit or replacement PR was identified, and the Conventional Commits policy check remained failing. | |
| stealth/space-bunny-alpha | — | closed stale | Proposed removing two unsupported reactive-charm flags on older channels while retaining dynamic 26.04 injection; the PR was closed with no identified merge, commit, or maintainer resolution. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without merging due to CI failures, after proposing to remove unsupported flags from the reactive charmcraft.yaml template to fix older channel tests. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2763 fix(tests/spread): fix reactive spread tests | charmcraft | merged | Merged #2763 as 6e775f4a: split reactive spread coverage by legacy/Resolute, enabled binary wheels, and removed obsolete build flags. | |
| #415 test: fix typo in spread extension | fetch-service | closed | PR #415 was closed after approval was retracted; its typo fix was incorporated into merged PR #417 via commit 8363f15. | |
| #926 test: address a couple of non-flaky spread failures | rockcraft | merged | Merged PR #926 updated three spread-test configuration files to address persistent failures; all CI checks passed and two reviewers approved. | |
| #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. | |
| #12 Craft 2382 more spread tests | snapcraft-rocks | closed | Spread-test expansion closed unmerged after nearly 1,000 days; no review, CI, comments, or resolving commit was recorded. | |
| #1555 refactor(tests/spread): stop use init in most spread tests | charmcraft | closed | Closed unmerged and replaced by PR #1556, merged as commit 4682b32b, which updated Spread tests to use explicit charmcraft.yaml files instead of charmcraft init. | |
| #6350 test: fix spread tests | snapcraft | merged | Merged PR #6350 as a18c7f485, fixing all non-store Spread tests and OSV scanning, including pinning Black 26.3.1 to avoid patchelf failures. | |
| #2417 fix: fix a series of breaking spread tests | charmcraft | merged | Merged PR #2417 at commit 3bee888e, fixing multiple spread-test failures and unblocking the core24 migration; three listed tests remained broken but non-blocking. | |
| #3480 ci: spread testing | snapcraft | closed | Closed unmerged: cjp256's experimental CI spread-testing PR, described as 'Just a test, ignore me for now', drew no reviews, comments or CI checks and was abandoned. No commit references #3480. |