Skip to content
This repository has been archived by the owner on Feb 2, 2021. It is now read-only.

Mixed use of docker and $DOCKER #7

Closed
remohammadi opened this issue Dec 5, 2014 · 1 comment · Fixed by #8
Closed

Mixed use of docker and $DOCKER #7

remohammadi opened this issue Dec 5, 2014 · 1 comment · Fixed by #8

Comments

@remohammadi
Copy link
Contributor

Example of docker usage in the script:

cat containers.keep |
xargs -n 1 docker inspect -f '{{.Config.Image}}' 2>/dev/null |
sort | uniq |
xargs -n 1 docker inspect -f '{{.Id}}' 2>/dev/null |
sort | uniq > images.used

Is this by design? Or can I make a pull request to unify the method of calling docker?

@mbruggmann
Copy link
Member

Hey @remohammadi . Pretty sure this is just an oversight - would be great if you could submit a PR, otherwise I'll try to pick it up in the coming days.

remohammadi added a commit to remohammadi/docker-gc that referenced this issue Dec 8, 2014
paulRbr pushed a commit to trainline-eu/docker-gc that referenced this issue Sep 15, 2017
davidxia pushed a commit that referenced this issue May 23, 2018
* bump docker version and fix binary mv
* add volume cleanup
* mention volumes in readme
* keep docker-gc backward-compatibility
* improve remove volumes logic
* this would probably be a feature
* change back to original docker version
* exclude volumes removal
* add VOLUME_DELETE_ONLY_DRIVER
* fix typos (#6)
* exclude volumes file defaults to null (#7)
* it should be local, not locale (#8)
* Add dangling images to list of all images. (#9)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants