-
Notifications
You must be signed in to change notification settings - Fork 183
Inject oci-image in release metadata and oci-images in release index
#3919
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
Conversation
|
Skipping CI for Draft Pull Request. |
|
Requires: coreos/stream-metadata-go#75 |
dustymabe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
To pick up the stream-metadata-go bump in the previous commit. But also this cleans up a bunch of other unrelated stuff.
No functional change. Prep for next patch.
Take the digest pullspec for the base OS bootable container and put it in the new `oci-image` field in the release metadata. Part of coreos/fedora-coreos-tracker#1823.
Currently, that function operates directly on an S3 bucket. This makes it harder to test locally. Add a new `--local-mode` option in which we expect to be in a cosa workdir with release metadata present and a release index. This is mostly peppering conditionals in the code. Patch best viewed with whitespace ignored.
When updating the release index, gather the OCI pullspecs across all arches for a given release into a single list and inject it into the new `oci-images` key, the same way we do for OSTree commits and `commits`. Part of coreos/fedora-coreos-tracker#1823.
321cccf to
195ef6e
Compare
|
Updated and out of draft! Now with the stream-metadata-go bump to pull in coreos/stream-metadata-go#75. |
dustymabe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
This broke SCOS for OKD.specifically the assert in 5900ba1 fails |
See individual commits.
Part of coreos/fedora-coreos-tracker#1823.