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: add cleanup images plugin #934

Merged
merged 5 commits into from
Dec 7, 2021
Merged

ci: add cleanup images plugin #934

merged 5 commits into from
Dec 7, 2021

Conversation

mudler
Copy link
Contributor

@mudler mudler commented Dec 6, 2021

Attempt to free up space during build (related to #917 )

@mudler mudler force-pushed the cleanup_images branch 2 times, most recently from ae14fc7 to ef7505b Compare December 6, 2021 13:48
@mudler
Copy link
Contributor Author

mudler commented Dec 6, 2021

Rebased on top of #935

Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
This breaks validation as ver-1 is lower than ver

See: #923

Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
@mudler mudler marked this pull request as ready for review December 6, 2021 17:28
Copy link
Contributor

@Itxaka Itxaka left a comment

Choose a reason for hiding this comment

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

Yikes. Can't we simply docker system prune -a and call it a day? That should clean up everything not being used no?

Also, what do we expect to clean up from this? Aren't ci nodes spin up from scratch on each job?

@mudler
Copy link
Contributor Author

mudler commented Dec 6, 2021

Yikes. Can't we simply docker system prune -a and call it a day? That should clean up everything not being used no?

Yes and no, I mean we definitely can delete all images after we finish building one package, the plugin is just a safeguard to avoid going to look at this again

Also, what do we expect to clean up from this? Aren't ci nodes spin up from scratch on each job?

Nope, we run 1 job building all images, rationale is explained #917 . we can split build into multiple, but that has other problems as well (starvation)

@mudler
Copy link
Contributor Author

mudler commented Dec 7, 2021

I think with #278 we can disable the plugin and the cleanup, and just run it after the whole build.. but keeping it for now until we have self-hosted runners

@mudler mudler merged commit 76d26fa into master Dec 7, 2021
@mudler mudler deleted the cleanup_images branch December 7, 2021 19:56
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