Skip to content
This repository has been archived by the owner on Nov 7, 2024. It is now read-only.

Add convenient API/CLI to prune all content #571

Merged
merged 3 commits into from
Dec 11, 2023

Conversation

cgwalters
Copy link
Member

Add convenient API/CLI to prune all content

This is kind of a usability regression from
979b93a
"deploy: Add an API to prune undeployed images"

This new API and CLI make it convenient to prune the images,
the layers, and the underlying objects.

cc coreos/rpm-ostree#4720

This API in particular is what bootc wants. For rpm-ostree
we do still want to separate pruning refs from objects, because
rpm-ostree may have layered packages, and we only want to traverse
the ostree repo to prune objects once.


deploy: (style) Move external crates before internal

This is preferred style.


This is kind of a usability regression from
ostreedev@979b93a
"deploy: Add an API to prune undeployed images"

This new API and CLI make it convenient to prune the images,
the layers, *and* the underlying objects.

cc coreos/rpm-ostree#4720

This API in particular is what bootc wants.  For rpm-ostree
we do still want to separate pruning refs from objects, because
rpm-ostree may have layered packages, and we only want to traverse
the ostree repo to prune objects once.
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

@jmarrero jmarrero merged commit 4ab03f0 into ostreedev:main Dec 11, 2023
8 checks passed
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