Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

composefs: Bump composefs max version to 1 #3149

Merged
merged 2 commits into from
Jan 31, 2024

Conversation

alexlarsson
Copy link
Member

This generates the new format for whiteout markers which was added in 6.8 (and which will be backported to 6.7). Without this whiteouts will not work anymore.

This is a slight format change, but will only affect ostree commits that already were broken (i.e that had whiteouts), and since the composefs code is still marked experimental I think it is fine to do this without introducing another format version on the ostree side.

@alexlarsson alexlarsson requested a review from cgwalters January 31, 2024 11:08
This updates composefs to 1.0.3 which has support for the
new overlay nesting format.
This generates the new format for whiteout markers which was added in
6.8 (and which will be backported to 6.7). Without this whiteouts
will not work anymore.

This is a slight format change, but will only affect ostree commits
that already were broken (i.e that had whiteouts), and since the
composefs code is still marked experimental I think it is fine to
do this without introducing another format version on the ostree
side.

Signed-off-by: Alexander Larsson <[email protected]>
@cgwalters
Copy link
Member

/override continuous-integration/jenkins/pr-merge

@cgwalters cgwalters enabled auto-merge January 31, 2024 13:01
@alexlarsson
Copy link
Member Author

@cgwalters Is this supposed to be auto-merged with the override? It doesn't seem to be.

@cgwalters
Copy link
Member

/override continuous-integration/jenkins/pr-merge

@cgwalters cgwalters merged commit 0b0485c into ostreedev:main Jan 31, 2024
25 checks passed
Copy link

openshift-ci bot commented Jan 31, 2024

@cgwalters: Overrode contexts on behalf of cgwalters: continuous-integration/jenkins/pr-merge

In response to this:

/override continuous-integration/jenkins/pr-merge

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

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