spread: retry grepping docker logs
Metadata
Current evaluation
Merged a change adding a retry helper for grepping Docker logs in spread tests. Resolves test flakiness from container startup delays. Approved by two reviewers, passed CI, and modified three files.
Suggested action: —
No scores available.
Issue body
One possible source of flakiness in spread tests is the common idiom of starting a container and then immediately grepping the logs. Since our containers' entrypoint is Pebble, it might take a short amount of time to startup, create the services plan, and start whatever it is we want it to start.
Therefore, add a helper command to 'retry' this 'docker logs | grep' pipe.
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a change adding a retry helper for grepping Docker logs in spread tests. Resolves test flakiness from container startup delays. Approved by two reviewers, passed CI, and modified three files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for spread test flakiness by introducing a retry helper for docker logs grep. This resolves timing delays during Pebble container startup, ensuring reliable log extraction after services initialize. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a change adding a retry helper for the docker logs grep pipeline in spread tests. This resolves test flakiness by accounting for container startup delays before Pebble services initialize. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #96 spread: add log output when docker fails to run | rockcraft | closed | Superseded by pull request #94. Debug logging added to investigate flaky Docker daemon connection errors in spread tests was closed without merging after the underlying issue was resolved in the referenced PR. | |
| #94 spread: add 'global' clean-up calls | rockcraft | merged | Merged. Introduces global cleanup calls in spread tests to automatically remove Docker and LXC resources after each task. Approved by one reviewer and passed CI. A separate craft-providers configuration failure noted in comments did not block the merge. | |
| #534 test(spread): workaround for flaky test | fetch-service | merged | Merged a workaround for a flaky spread test (issue #533). The change adjusts one file (+2/-1 lines) to skip a no-op name query check while retaining other validations. Approved by two reviewers and passed all CI checks. |