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

[ci] remove Docker volumes during Azure cleanup #6760

Merged
merged 1 commit into from
Dec 15, 2024
Merged

Conversation

StrikerRUS
Copy link
Collaborator

I believe we don't need preserving anonymous volumes during cleanup.

Refer to https://depot.dev/blog/docker-clear-cache#removing-everything-with-docker-system-prune and https://docs.docker.com/reference/cli/docker/system/prune/.

By default, volumes aren't removed to prevent important data from being deleted if there is currently no container using the volume. Use the --volumes flag when running the command to prune anonymous volumes as well:

Copy link
Collaborator

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

I agree with you, the way we use this box, those volumes can all be safely removed. And that will free up a bit of disk space!

Thanks for thinking of this.

@jameslamb jameslamb merged commit 31205fc into master Dec 15, 2024
48 checks passed
@jameslamb jameslamb deleted the ci/docker-cleanup branch December 15, 2024 18:24
@StrikerRUS
Copy link
Collaborator Author

Thanks for thinking of this.

Thanks to this warning, I started looking into this 😄

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants