-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Add test/clean.sh script #639
Conversation
OMG when were wildcards implemented in |
No idea, but I figured I'd test it to see if it worked. It's kind of surprising given that |
LGTM, but I'm still reeling from the fact that I somehow missed such a great feature that's clearly working in Docker 1.5.0! |
So far, my digging shows it's been there since at least as far back as 1.2.0; still digging to see how deep this rabbit hole goes. |
0.10.0 😳 |
This feature goes all the way back to Docker 0.6.2, and was added in moby/moby#1817. |
BTW, the "trail" I followed to find that PR was |
#!/bin/bash | ||
set -e | ||
|
||
docker images -q 'librarytest/*' | xargs docker rmi -f |
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.
Can we do a little changing here, to get rid of tags instead of images? Images will go away, but only if nothing else is using it.
docker images 'librarytest/*' | awk 'NR>1 { print $1":"$2 }' | xargs -r docker rmi
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.
Updated to only remove tags instead of images themselves. Too bad there isn't just a |
Of course, this introduces another GNU-ism ( |
OS X should just adopt the GNU 😺 |
(I'm OK with skipping the |
I love GNU, I really do, but I don't see OS X doing that. I should probably just go back to Linux. I'll leave the |
Ok, fair enough. ❤️ LGTM |
LGTM |
I just realized this is going to choke if someone creates an image in the |
Or perhaps Docker just doesn't allow what I'm saying, since it defaults to tagging as |
That's the one 👍
|
(Sans the new v2 digests thing)
|
Add 7.0 release
Fixes #638