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

Remove all unused images on cleanup, not just dangling ones #150

Merged
merged 1 commit into from
Oct 22, 2023

Conversation

pal03377
Copy link
Contributor

Motivation and Context

The test server disk space got full because non-dangling Docker images were not removed.

Description

Change the existing Docker cleanup (after TS deployment) to also remove non-dangling images.

Steps for Testing

Deploy to the test server. Then, observe how old images are removed and there won't be a disk space problem for a longer time span.

Screenshots

I manually executed the cleanup command to test:
screenshot-2023-10-22_002031

@pal03377 pal03377 self-assigned this Oct 22, 2023
@pal03377 pal03377 added the deploy:athena-test1 Athena Test Server 1 label Oct 22, 2023
@github-actions github-actions bot removed the deploy:athena-test1 Athena Test Server 1 label Oct 22, 2023
@github-actions

This comment was marked as outdated.

@github-actions github-actions bot added the deployment-error Added by deployment workflows if an error occured label Oct 22, 2023
@pal03377 pal03377 added deploy:athena-test1 Athena Test Server 1 and removed deployment-error Added by deployment workflows if an error occured labels Oct 22, 2023
@github-actions github-actions bot added lock:athena-test1 Is currently deployed to Athena Test Server 1 and removed deploy:athena-test1 Athena Test Server 1 labels Oct 22, 2023
@pal03377 pal03377 temporarily deployed to Athena Test Server 3 October 22, 2023 12:49 — with GitHub Actions Inactive
@pal03377 pal03377 removed the lock:athena-test1 Is currently deployed to Athena Test Server 1 label Oct 22, 2023
@pal03377
Copy link
Contributor Author

I'm merging this directly because it works and it's very valuable to have right away

@pal03377 pal03377 merged commit 4016dd9 into develop Oct 22, 2023
21 of 22 checks passed
@FelixTJDietrich
Copy link
Collaborator

Thanks @pal03377! :)

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.

2 participants