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

feat(docker): Support dangling Docker images #816

Merged
merged 1 commit into from
Mar 27, 2024
Merged

feat(docker): Support dangling Docker images #816

merged 1 commit into from
Mar 27, 2024

Conversation

Kurt-von-Laven
Copy link
Contributor

Closes #572.

Docker images that were overwritten by a new one with the same name and tag do not have a name or tag. Docker images that were pulled by digest do not have a tag. Support the former by falling back on their ID and the latter by using their name.

These changes prevent the action from crashing on such images. Continue using both the name and tag for images that have them so that they are retained upon restoring the cache.

Docker images that were overwritten by a new one
with the same name and tag do not have a name or
tag. Docker images that were pulled by digest do
not have a tag. Support the former by falling back
on their ID and the latter by using their name.

These changes prevent the action from crashing on
such images. Continue using both the name and tag
for images that have them so that they are
retained upon restoring the cache.
@Kurt-von-Laven Kurt-von-Laven added the enhancement New feature or request label Mar 26, 2024
@Kurt-von-Laven Kurt-von-Laven requested a review from mwarres March 26, 2024 06:14
@Kurt-von-Laven Kurt-von-Laven self-assigned this Mar 26, 2024
Copy link
Contributor

@mwarres mwarres left a comment

Choose a reason for hiding this comment

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

LGTM

@Kurt-von-Laven Kurt-von-Laven merged commit 890a6fb into main Mar 27, 2024
9 checks passed
@Kurt-von-Laven Kurt-von-Laven deleted the docker branch March 27, 2024 03:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

Use Image IDs For Dangling Images
2 participants