[ https://issues.apache.org/jira/browse/JCLOUDS-1158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15448577#comment-15448577 ]
ASF subversion and git services commented on JCLOUDS-1158: ---------------------------------------------------------- Commit a4e2110a6307f00e10f1f2193b5f609bc185b628 in jclouds's branch refs/heads/master from [~kwart] [ https://git-wip-us.apache.org/repos/asf?p=jclouds.git;h=a4e2110 ] JCLOUDS-1158 fix Docker find image Predicate for cases where repoTags field contains 'docker.io/' registry host prefix > DockerComputeServiceAdapter.getImage(String) doesn't work correctly for > Docker on RHEL Atomic host > -------------------------------------------------------------------------------------------------- > > Key: JCLOUDS-1158 > URL: https://issues.apache.org/jira/browse/JCLOUDS-1158 > Project: jclouds > Issue Type: Bug > Components: jclouds-labs > Reporter: Josef Cacek > Labels: docker > Fix For: 2.0.0 > > > The > {{org.jclouds.docker.compute.strategy.DockerComputeServiceAdapter.getImage(String)}} > method doesn't find images when Docker from RHEL (Atomic) is used. The > reason is the returned Image {{repoTags}} field from the Docker server > contains also registry hostname. E.g. > {code} > "RepoTags":["docker.io/kwart/alpine-ext:3.3-ssh"] > {code} > The problem is in the condition used in this piece of code: > {code} > for (String tag : input.repoTags()) { > if (tag.equals(imageIdOrName) || tag.equals(imageIdOrName + ":latest")) { > return true; > } > } > {code} > IMO, we should add also 2 more options to statement: > {code} > final String imageNameWithHostPrefix = "docker.io/" + imageIdOrName; > for (String tag : input.repoTags()) { > if (tag.equals(imageIdOrName) || tag.equals(imageIdOrName + ":latest") > || tag.equals(imageNameWithHostPrefix) || > tag.equals(imageNameWithHostPrefix + ":latest")) { > ... > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332)