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

Update opencontainers/image-spec to v1.1.0-rc5 #4334

Conversation

TBBle
Copy link
Collaborator

@TBBle TBBle commented Oct 14, 2023

Broken out of #4331 since the actual change there may not be the right solution, and landing this lets me rebase that PR down to the essentials before possibly archiving it.

Changes per opencontainers/image-spec@v1.1.0-rc3...v1.1.0-rc5. None of the changes required corresponding changes in the BuildKit source.

I took the opportunity to replace a bunch of existing strings with newly-added constants from image-spec. (If this is a concern, it'd be trivial to drop or split into a different commit)

The original motivation for updating image-spec was to get access to DescriptorEmptyJSON, which was renamed from ScratchDescriptor and had its media type changed. BuildKit had no existing users of this value, or the related media types.

Also, some other small fixes in-passing:

  • Fix docs-upstream GitHub Action
  • Correct package name for inline remote-cache source

This brings in new string constants ImageIndexFile and ImagesBlobsDir.

Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
@TBBle TBBle force-pushed the random_small_fixes_broken_out_of_empty_cache_layer_work branch from 26d3ec8 to f4d44fd Compare October 17, 2023 08:35
@jedevc jedevc merged commit 70c988a into moby:master Oct 17, 2023
52 of 54 checks passed
@jedevc
Copy link
Member

jedevc commented Oct 17, 2023

Thanks @TBBle ❤️

@TBBle TBBle deleted the random_small_fixes_broken_out_of_empty_cache_layer_work branch October 17, 2023 10:00
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.

3 participants