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

Automate Local Image Pruning #3587

Merged
merged 8 commits into from
Jun 6, 2024

Conversation

safwank97
Copy link
Contributor

@safwank97 safwank97 commented May 18, 2024

Which issue this PR addresses:

Fixes https://issues.redhat.com/browse/ARO-7189

What this PR does / why we need it:

This PR helps us automatically prune local container images generated by make ci, so that we as developers do not run out of local disk space in our machines. Additionally it also does labeling for all the intermediate images in a multistage Docker file which could help us at the time of pruning.

Test plan for issue:

Run make ci-rp to produce aro-* images and later run make ci-clean to verify and validate whether it is pruning those images to free up our local disk space.

Is there any documentation that needs to be updated for this PR?

None that I'm aware of.

Copy link
Collaborator

@SudoBrendan SudoBrendan left a comment

Choose a reason for hiding this comment

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

The direction of the PR is really good. Let's align on image tags, and document/remove --force.

Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Dockerfile.ci-rp Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Copy link

Please rebase pull request.

@github-actions github-actions bot added the needs-rebase branch needs a rebase label May 22, 2024
@safwank97 safwank97 force-pushed the local-image-pruning-for-makeci-images branch from 6f5bb98 to 14e0b5f Compare May 28, 2024 18:59
@github-actions github-actions bot removed the needs-rebase branch needs a rebase label May 28, 2024
@safwank97 safwank97 closed this May 29, 2024
@safwank97 safwank97 reopened this May 29, 2024
@safwank97 safwank97 marked this pull request as ready for review May 29, 2024 14:58
Copy link
Collaborator

@shubhadapaithankar shubhadapaithankar left a comment

Choose a reason for hiding this comment

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

Looks good to me

tsatam
tsatam previously approved these changes May 30, 2024
@SudoBrendan
Copy link
Collaborator

/azp run ci

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Collaborator

@SudoBrendan SudoBrendan left a comment

Choose a reason for hiding this comment

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

Just a quick fix - we need to mark the new target as .PHONY for makefile reasons, since we aren't compiling a C program with this target.

Makefile Show resolved Hide resolved
@safwank97 safwank97 dismissed stale reviews from tsatam and shubhadapaithankar via acd60ed June 3, 2024 18:41
@safwank97
Copy link
Contributor Author

/azp run ci

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@github-actions github-actions bot added the needs-rebase branch needs a rebase label Jun 4, 2024
Copy link

github-actions bot commented Jun 4, 2024

Please rebase pull request.

@safwank97 safwank97 force-pushed the local-image-pruning-for-makeci-images branch from acd60ed to 546faa1 Compare June 6, 2024 17:02
@github-actions github-actions bot removed the needs-rebase branch needs a rebase label Jun 6, 2024
@safwank97
Copy link
Contributor Author

/azp run ci

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Makefile Show resolved Hide resolved
@SudoBrendan SudoBrendan merged commit 470884a into master Jun 6, 2024
19 checks passed
@SudoBrendan SudoBrendan deleted the local-image-pruning-for-makeci-images branch July 24, 2024 15:52
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.

5 participants