Skip to content

feat: support rechunking with build-chunked-oci#610

Merged
gmpinder merged 4 commits intoblue-build:mainfrom
HastD:build-chunked-oci
Nov 21, 2025
Merged

feat: support rechunking with build-chunked-oci#610
gmpinder merged 4 commits intoblue-build:mainfrom
HastD:build-chunked-oci

Conversation

@HastD
Copy link
Copy Markdown
Collaborator

@HastD HastD commented Nov 17, 2025

Add support for rechunking images with rpm-ostree compose build-chunked-oci. This is available in the CLI with the --build-chunked-oci option, with an optional --max-layers argument to specify the maximum number of layers in the rechunked image.

@HastD HastD requested a review from gmpinder as a code owner November 17, 2025 17:49
@gmpinder
Copy link
Copy Markdown
Member

The code looks really good! Now if you can add a test case for a single arch and multi arch build using this? You can crib from the justfile and .github/workflows/test.yml. It would help to also create new recipe files in integration-tests/test-repo/recipes/ to prevent images pushing on top of one another.

@HastD HastD force-pushed the build-chunked-oci branch from 5d6540b to adf3f88 Compare November 17, 2025 18:55
@HastD
Copy link
Copy Markdown
Collaborator Author

HastD commented Nov 17, 2025

@gmpinder Added integration test cases as requested.

@gmpinder
Copy link
Copy Markdown
Member

Some of the tests might be failing cause of permission issues. The chunked oci ones I think are failing cause of the Driver::init function I mentioned in the previous comment.

@HastD
Copy link
Copy Markdown
Collaborator Author

HastD commented Nov 18, 2025

@gmpinder I don't think that "manifest unknown" error happening in some of the tests is due to my changes; I saw the same thing yesterday in a PR build for secureblue that doesn't change anything except updating the CLI to v0.9.25 (from v0.9.24) and adding the cosign-version: none option: https://github.com/secureblue/secureblue/actions/runs/19450325721/job/55653616076?pr=1605

@HastD HastD force-pushed the build-chunked-oci branch from adf3f88 to 160d508 Compare November 18, 2025 17:19
@HastD HastD requested a review from gmpinder November 18, 2025 17:20
@gmpinder
Copy link
Copy Markdown
Member

@gmpinder I don't think that "manifest unknown" error happening in some of the tests is due to my changes; I saw the same thing yesterday in a PR build for secureblue that doesn't change anything except updating the CLI to v0.9.25 (from v0.9.24) and adding the cosign-version: none option: https://github.com/secureblue/secureblue/actions/runs/19450325721/job/55653616076?pr=1605

Interesting. The integration tests didn't catch that.

@HastD
Copy link
Copy Markdown
Collaborator Author

HastD commented Nov 18, 2025

Yeah so going from the most recent test logs for this PR, it looks like rechunking with build-chunked-oci is working fine (which makes sense, I tested it locally), but then something afterward (in code I didn't change) isn't working.

Add support for rechunking images with `rpm-ostree compose
build-chunked-oci`. This is available in the CLI with the
`--build-chunked-oci` option, with an optional `--max-layers` argument
to specify the maximum number of layers in the rechunked image.
@HastD HastD force-pushed the build-chunked-oci branch from 160d508 to 2805f4a Compare November 19, 2025 21:55
@gmpinder
Copy link
Copy Markdown
Member

gmpinder commented Nov 21, 2025

I'm working on trying to fix the manifest issue in #615. It appears this issue occurs outside of your changes too.

@gmpinder
Copy link
Copy Markdown
Member

Ok that change seemed to work. I'm going to update your branch real quick.

@gmpinder gmpinder merged commit f4ea145 into blue-build:main Nov 21, 2025
25 of 28 checks passed
@HastD HastD deleted the build-chunked-oci branch November 21, 2025 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants