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

core: Add composefs digest for local commits too #4594

Merged
merged 2 commits into from
Sep 25, 2023

Conversation

cgwalters
Copy link
Member

This pairs with dbe7821 where we generate composefs metadata for base images.

It's going to be useful to do it for client side builds too, so that we can treat them symmetrically. It doesn't have a very high cost, just a metadata traversal.

@cgwalters
Copy link
Member Author

Needed for ostreedev/ostree#3021

@cgwalters cgwalters added needs-work/failing-ci Legitimate CI failure client-layering Issues related to `rpm-ostree install/override` client side labels Sep 12, 2023
@cgwalters cgwalters removed the needs-work/failing-ci Legitimate CI failure label Sep 12, 2023
@cgwalters
Copy link
Member Author

Always nice when the CI legitimately fails and you know your CI is working.

@cgwalters cgwalters added the needs-work/failing-ci Legitimate CI failure label Sep 13, 2023
@cgwalters
Copy link
Member Author

OK at some point I'll actually debug this test locally...

@cgwalters
Copy link
Member Author

Oh god of course the problem is that somewhere in the giant manual process of building ostree for 3 active Fedora releases I got side tracked and only did rawhide...

@cgwalters
Copy link
Member Author

This will now just need to sit here, blocked while we wait ~a week for no one to add karma to https://bodhi.fedoraproject.org/updates/FEDORA-2023-13b41d2e37 until it times out and ships...

@jmarrero
Copy link
Member

why is Stable by Karma stable by karma disabled?

@cgwalters
Copy link
Member Author

why is Stable by Karma stable by karma disabled?

I stopped adding it because sometimes it got 3 karma from people who weren't really testing things, although that seems to have stopped thankfully. (This is a complex topic; more ideally we gate on some FCOS tests and really upstream tests, but at least there is the silverblue test there now...)

For the composefs metadata API.
This pairs with coreos@dbe7821
where we generate composefs metadata for base images.

It's going to be useful to do it for client side builds too, so
that we can treat them symmetrically.  It doesn't have
a very high cost, just a metadata traversal.
@cgwalters
Copy link
Member Author

OK, this should now work since the new ostree went stable.

@cgwalters cgwalters enabled auto-merge (rebase) September 19, 2023 20:17
Copy link
Member

@jmarrero jmarrero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@cgwalters
Copy link
Member Author

/retest

@cgwalters
Copy link
Member Author

Blocking on coreos/fedora-coreos-tracker#1579

@cgwalters
Copy link
Member Author

/retest

@cgwalters cgwalters merged commit 4b1e956 into coreos:main Sep 25, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client-layering Issues related to `rpm-ostree install/override` client side
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants