fix(test): mark chisel integration tests as flaky
Metadata
Current evaluation
Merged changes marking chisel integration tests as flaky to accommodate intermittent network failures. Approved by two reviewers, passed CI checks, and resolves issue #1306.
Suggested action: —
No scores available.
Issue body
These sometimes fail due to network issues, so they're marked as flaky so we can retry them if necessary.
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint && make test`?
- [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)?
---
Fixes #1306
CRAFT-4771
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged changes marking chisel integration tests as flaky to accommodate intermittent network failures. Approved by two reviewers, passed CI checks, and resolves issue #1306. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged update marking chisel integration tests as flaky to handle intermittent network failures and enable retries. Resolves #1306 and CRAFT-4771. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1306 Mark chisel tests as flaky | craft-parts | closed | Closed without implementation. The enhancement to retry flaky chisel tests via pytest-rerunfailures was not applied. | |
| #466 fix(spread): tests must also use newer Chisel (#456) | rockcraft | merged | Merged a cherry-picked commit from main to align tests with the newer Chisel version, facilitating the 1.1.2 release. Approved by one reviewer. | |
| #625 ci: update chisel (#622) | craft-parts | merged | Merged a cherry-pick PR to update the chisel dependency and fix CI in a feature branch. Approved by two reviewers with passing checks, the change modified one file (+3/-1 lines) to resolve continuous integration failures. | |
| #456 fix(spread): tests must also use newer Chisel | rockcraft | merged | Merged. Resolves failing spread tests by switching Chisel installation from a hardcoded commit to the snap store. Addressed review feedback and re-triggered CI after a timeout. | |
| #1316 fix(test): mark complex script piping test as flaky | craft-parts | merged | Merged to mark a complex script piping test as flaky, resolving intermittent failures on ppc64el and s390x runners caused by pipe ordering. Approved by reviewers, passed all CI checks, and successfully merged. | |
| #467 fix(spread): tests must also use newer Chisel (#456) (#466) | rockcraft | closed | Merged and closed as a cherry-pick from main to update test dependencies to a newer Chisel version for the 1.1.2 release. | |
| #307 tests: disable broken integration test | craft-application | merged | Merged a temporary workaround disabling a broken integration test to restore CI functionality, pending a permanent fix in issue #306. Approved by two reviewers with passing checks. | |
| #4441 feat: enable chisel | snapcraft | merged | Merged into main after passing CI and receiving approval from two reviewers. The change enables chisel, adding 45 lines across three files with no test coverage impact. |