Skip to content
This repository has been archived by the owner on Jan 15, 2025. It is now read-only.

container: Add history struct to oci writing, tweak history description #183

Merged
merged 1 commit into from
Dec 10, 2021

Conversation

cgwalters
Copy link
Member

Prep for #69
where we'll split up the input ostree commit into content-addressed
blobs.

We want to inject something useful into the the history in the
config git that describes each chunk, so add support for that
into our OCI writer.

Change the default description for the (currently single) layer
to include the commit subject, if present; otherwise the
commit hash.

The description of the layer shouldn't change as this tool changes,
just as the input changes.

(Side note; today rpm-ostree isn't adding a subject description,
but hey, maybe someone else is)

Prep for ostreedev#69
where we'll split up the input ostree commit into content-addressed
blobs.

We want to inject something useful into the the `history` in the
config git that describes each chunk, so add support for that
into our OCI writer.

Change the default description for the (currently single) layer
to include the commit subject, if present; otherwise the
commit hash.

The description of the layer shouldn't change as this tool changes,
just as the input changes.

(Side note; today rpm-ostree isn't adding a subject description,
 but hey, maybe someone else is)
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

Thanks for the review!

@cgwalters cgwalters merged commit 6fa7fb9 into ostreedev:main Dec 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants