diff options
author | Valerij Fredriksen <valerijf@oath.com> | 2018-09-20 15:45:08 +0200 |
---|---|---|
committer | Valerij Fredriksen <valerijf@oath.com> | 2018-09-20 15:45:08 +0200 |
commit | c54edd1617c1e6bf4cecf3d95e3f1387e1a7d0fa (patch) | |
tree | e52fd3cdfb9c6ecdb8e9fcfdbbdb298c3db8a939 /docker-api | |
parent | de17850fb3d36e93495cbe27f8979e23c179474e (diff) |
Code review fixes
Diffstat (limited to 'docker-api')
-rw-r--r-- | docker-api/src/main/java/com/yahoo/vespa/hosted/dockerapi/DockerImageGarbageCollector.java | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/docker-api/src/main/java/com/yahoo/vespa/hosted/dockerapi/DockerImageGarbageCollector.java b/docker-api/src/main/java/com/yahoo/vespa/hosted/dockerapi/DockerImageGarbageCollector.java index e4edcf91811..0ae6004be12 100644 --- a/docker-api/src/main/java/com/yahoo/vespa/hosted/dockerapi/DockerImageGarbageCollector.java +++ b/docker-api/src/main/java/com/yahoo/vespa/hosted/dockerapi/DockerImageGarbageCollector.java @@ -27,7 +27,8 @@ import java.util.stream.Stream; * <p>Definitions: * <ul> * <li>Every image has exactly 1 id</li> - * <li>Every image has between 0..n tags</li> + * <li>Every image has between 0..n tags, see + * <a href="https://docs.docker.com/engine/reference/commandline/tag/">docker tag</a> for more</li> * <li>Every image has 0..1 parent ids</li> * </ul> * @@ -74,6 +75,8 @@ class DockerImageGarbageCollector { List<Container> containers = docker.listAllContainers(); Map<String, Image> imageByImageId = images.stream().collect(Collectors.toMap(Image::getId, Function.identity())); + + // Find all the ancestors for every local image id, this includes the image id itself Map<String, Set<String>> ancestorsByImageId = images.stream() .map(Image::getId) .collect(Collectors.toMap( |