← Back to issue list

spread: don't cut dotnet slices unnecessarily

View original Github issue

Metadata

Project
rockcraft
Number
#530
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

Merged a one-line change replacing fragile dotnet slices with coreutils_bins in spread tests to avoid instability from ongoing time_t changes. Approved by three reviewers and passed CI.

Suggested action:

No scores available.

Issue body

Those slices are currently fragile due to the ongoing time_t changes. Instead, use coreutils_bins because it has fewer dependencies (but still some) and is more stable. The purpose of the test is just to check that we can cut any slices at all for base devel anyway. - [ ] 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 one-line change replacing fragile dotnet slices with coreutils_bins in spread tests to avoid instability from ongoing time_t changes. Approved by three reviewers and passed CI.
qwen3.6-35b-a3b-mtp-q6 Merged changes replacing fragile dotnet slices with coreutils_bins in spread tests. This avoids time_t dependency issues, improves stability, and preserves slice cutting functionality for base devel.
qwen3.6-35b-a3b-mtp-q6 Merged. Replaced fragile dotnet slice cutting with coreutils_bins to avoid instability from ongoing time_t changes. The update maintains slice cutting functionality for base devel tests.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#13 test(spread): add more spread tests snapcraft-rocks merged Successfully merged spread tests covering architectures, chisel slices, craftctl, package-repositories, and patchelf. Approved by two reviewers across 15 files (+206/-5 lines).
74%
#155 spread: fix project with dotnet chisel slices rockcraft merged Merged to remove an obsolete prime override and symlink in the spread configuration, as dotnet packages no longer create version-specific subdirectories. The change resolved CI failures blocking other pull requests.
73%
#62 spread: add a "big" spread test rockcraft merged Merged a reusable spread test framework to house multiple checks without repeated setup. The initial test validates bug #44 regarding directory permissions. Approved by one reviewer and passed CI checks.
73%
#2216 spread tests: keep sources local snapcraft merged Merged PR #2216 to replace remote FTP sources in spread tests with in-tree tar archives, preventing test failures from server downtime. Approved by one reviewer, passed CI checks, and slightly increased code coverage.
73%
#573 ci: run spread tests on 'spread-installed' runners rockcraft merged Merged a two-file, four-line CI configuration change to run spread tests on spread-installed runners. Approved by three reviewers and integrated.
73%
#3491 spread: run cross compile tests on amd64 only snapcraft merged Merged into master. Configures spread to run cross-compile tests exclusively on amd64. Approved by one reviewer, passed CI, and applied a minimal three-line change with no coverage impact.
71%
#107 spread: restrict the files we package rockcraft merged Merged to restrict spread packaging to test/ and tools/ directories only. This reduces local run overhead by excluding hundreds of megabytes of unnecessary files. Approved by one reviewer, passed CI, and modified one file.
71%
#3662 tests: update spread url snapcraft merged Merged update to spread tarball URLs in test files. Approved by a reviewer, passed CI checks, modified two files with minimal changes, and maintained existing code coverage.
70%
#127 Fix spread tests imagecraft merged Merged after receiving approval from two reviewers and passing all CI checks. The changes fix spread tests across 11 files (+49/-38 lines).
70%
#2417 fix: fix a series of breaking spread tests charmcraft merged Merged after fixing multiple breaking spread tests. Commits resolve individual failures, while remaining broken tests are marked non-blocking for core24 migration. Approved by reviewers and merged following CI validation.
70%