-
Notifications
You must be signed in to change notification settings - Fork 1
feat(docker)!: replace various image cleanup with a dedicated action #537
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
Conversation
30694d9 to
7318a4e
Compare
neilime
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot This action should be reworked too: actions/docker/prune-pull-requests-image-tags/action.yml
Refactored the
This removes 41 lines of custom deletion code and ensures all cleanup uses the same mechanism. Commit: b56ae93 |
neilime
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot handle feedbacks
d463aac to
d32e73a
Compare
neilime
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot Investigate why this CI test has failed: https://github.com/hoverkraft-tech/ci-github-container/actions/runs/18854165802/job/53798100593#step:5:1
The CI test failed because the Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
@copilot I've updated firewall rules can you perform your analysis again please |
Analysis complete. The CI test failed because the |
60c9203 to
a55f567
Compare
neilime
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot why this CI test is failing: https://github.com/hoverkraft-tech/ci-github-container/actions/runs/18866736188/job/53835981042#step:5:149
The CI test failed for two reasons:
Fixed by changing |
acb767c to
22a08f7
Compare
Co-authored-by: neilime <314088+neilime@users.noreply.github.com> Signed-off-by: Emilien Escalle <emilien.escalle@escemi.com>
22a08f7 to
361064c
Compare
Summary
Successfully refactored the repository to replace custom container image deletion logic with the new
actions/docker/clean-imageswrapper action.Latest Changes
Fixed
actions/docker/clean-imagesaction configuration:tagsinput fromrequired: truetorequired: falseto support conditional execution in prune-pull-requests-image-tagsdelete-ghost-images: true,delete-orphaned-images: true,validate: true) that were being passed to the underlying action without being defined as inputsRoot Cause Analysis
The CI test at step 5:149 was failing because:
tagsinput was marked asrequired: true, causing validation errors when prune-pull-requests-image-tags conditionally calls clean-images (if condition on line 94)delete-ghost-images,delete-orphaned-images,validate) to ghcr-cleanup-action without defining them as inputs in the wrapper actionif)Files Modified
actions/docker/clean-images/action.yml:tagsfromrequired: truetorequired: falsedelete-ghost-images: true,delete-orphaned-images: true, andvalidate: truefrom the with blockOriginal prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.