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.

Sign up for GitHub

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

Jump to bottom

containers/ws: Clean /var/cache/libdnf5 #21456

Merged
merged 1 commit into from
Dec 20, 2024
Merged

containers/ws: Clean /var/cache/libdnf5 #21456

merged 1 commit into from
Dec 20, 2024

Conversation

martinpitt
Copy link
Member

This currently contains 84 MB of dead weight (repository indexes). That made the container balloon from 57 to 116 MB.

Not super important, but for me it's a matter of pride! See https://quay.io/repository/cockpit/ws?tab=tags

Verified with podman build -t localhost/ws containers/ws/, and

quay.io/cockpit/ws                 329.1       496397eface4  2 weeks ago         147 MB
quay.io/cockpit/ws                 331         bba2e19b4d7b  54 minutes ago      233 MB
localhost/ws                       latest      4e1c0fdfff34  About a minute ago  145 MB

Note that podman images shows uncompressed size, whereas podman pull and the quay.io page show compressed.

@martinpitt
containers/ws: Clean /var/cache/libdnf5
529f172 
This currently contains 84 MB of dead weight (repository indexes).
That made the container balloon from 57 to 116 MB.
@martinpitt martinpitt added the no-test For doc/workflow changes, or experiments which don't need a full CI run, label Dec 19, 2024
@martinpitt martinpitt requested review from jelly and mvollmer December 19, 2024 16:22
mvollmer
mvollmer approved these changes Dec 20, 2024
View reviewed changes
Copy link
Member

@mvollmer mvollmer left a comment

Choose a reason for hiding this comment

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

Thanks!

@mvollmer mvollmer merged commit eb0d8d1 into cockpit-project:main Dec 20, 2024
32 checks passed
@martinpitt martinpitt deleted the shrink-container branch December 20, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mvollmer mvollmer mvollmer approved these changes

@jelly jelly Awaiting requested review from jelly

Assignees
No one assigned
Labels
no-test For doc/workflow changes, or experiments which don't need a full CI run,
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@martinpitt @mvollmer