test: add a spread test for pebble pruning
Metadata
Current evaluation
Merged PR #1199 as commit 591fea81, adding a spread test for Pebble pruning and detection of mismatched file metadata.
Resolution: closed resolved
Reason: PR #1199 was approved and merged as commit 591fea81. It verifies that an identical Pebble binary is pruned from the lifecycle layer while surfacing differing filesystem metadata.
Evaluated by: mr-cal-bot
No scores available.
Issue body
The existing pruning mechanism is enough to remove the pebble binary from the lifecycle layer if an identical binary already exists in the base layer, which is the case as of 26.04. Add a more specific version of the pruning test because if both files are present but with different stats we want to know about it (why did it happen?)
---
- [ ] I've followed the [contribution guidelines](https://github.com/canonical/rockcraft/blob/main/CONTRIBUTING.md).
- [ ] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [ ] I've successfully run `make lint && make test`.
- [ ] I've added or updated any relevant documentation.
- [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing.
- [ ] I've updated the relevant release notes.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1199 as commit 591fea81, adding a spread test for Pebble pruning and detection of mismatched file metadata. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as 591fea81, adding a spread test that checks Pebble is pruned when an identical base-layer binary exists and exposes copies with different filesystem metadata. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1199, commit 591fea81, adding a spread test covering pebble pruning, including the case where identical binaries remain with different filesystem metadata. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged test coverage for pebble pruning to detect stat discrepancies when identical binaries exist in base and lifecycle layers. Approved by two reviewers and passed CI checks prior to integration. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a change adding a spread test for pebble pruning. The test validates behavior when identical binaries exist in base and lifecycle layers with differing file stats, ensuring unexpected discrepancies are caught. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #304 Pebble doc and test updates | rockcraft | merged | Merged PR #304 as commit 31700eaa, correcting Pebble documentation formatting and updating docs and spread tests for recent additions such as --args. | |
| #645 fix(spread): Pebble no longer supports -v with exec | rockcraft | merged | Merged PR #645 (commit 513ff8b4) removed the unsupported Pebble -v option from a spread test, fixing the failure caused by Pebble dropping that option. | |
| #2400 tests: fix maven spread test | snapcraft | merged | Merged PR #2400 as commit 155306683, correcting the Maven spread test with a two-line change. | |
| #13 test(spread): add more spread tests | snapcraft-rocks | merged | Merged PR #13 added Spread tests for architectures, Chisel slices, craftctl, package repositories, and patchelf (+206/-5 across 15 files). | |
| #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. | |
| #520 fix(spread): more robust check of non-pruned files | rockcraft | merged | Merged PR #520 improved the layer-pruning spread test to compare file contents when metadata matched, preventing false failures after a libuuid1 update. | |
| #534 test(spread): workaround for flaky test | fetch-service | merged | Merged PR #534 (commit 4f89bb7) worked around #533 by skipping the flaky snap names-query assertion while retaining the other spread-test checks. |