spread: retry grepping docker logs
Metadata
Current evaluation
Merged PR #282 added a Spread helper that retries Docker log-grep checks, reducing flakes while Pebble-backed containers initialize (commit e6e45976).
Resolution: closed resolved
Reason: PR #282 was approved and merged as commit e6e45976. It added tools/spread/grep_docker_log to retry log checks three times, one second apart, while waiting for container startup.
Evaluated by: mr-cal-bot
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #282 added a Spread helper that retries Docker log-grep checks, reducing flakes while Pebble-backed containers initialize (commit e6e45976). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged “spread: retry grepping docker logs,” adding a retry helper for Docker log-grep commands to reduce flakes while Pebble starts. | |
| 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. |
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 | Closed unmerged because its Docker-related spread diagnostics were addressed by merged PR #94 in commit ad384842. |