Skip to content

Conversation

@jcantrill
Copy link
Contributor

This PR adds support for ImageStreamImport to fix the referenced issue. It allows image metadata to be retrieved via openshift without having a connection to a docker daemon

cc @adietish @fbricon
[test]

@adietish
Copy link
Member

looking...

}

@Override
public DockerImageURI getDockerImageRepositorySpec() {
Copy link
Member

Choose a reason for hiding this comment

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

why not call this #getDockerImageURI to match what you get back when you call it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

was trying to explicitly distinguish it being in the spec section. The other is actually depricated i blieve

@jcantrill jcantrill force-pushed the jbide_21626_pod_not_created branch from 96d5992 to 65d443a Compare February 26, 2016 20:24

DockerImageURI getDockerImageRepositorySpec();

void setDockerImageRepositorySpec(String imageUri);
Copy link
Member

Choose a reason for hiding this comment

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

is this replacing deprecated #setDockerImageRepository? What is the difference? having this explained in javadoc would help a lot

@jcantrill jcantrill force-pushed the jbide_21626_pod_not_created branch 2 times, most recently from 927b984 to 026342f Compare March 1, 2016 15:58
@jcantrill jcantrill force-pushed the jbide_21626_pod_not_created branch from 026342f to b2d6786 Compare March 1, 2016 18:08
@openshift-bot
Copy link
Contributor

Evaluated for javaclient test up to b2d6786

@openshift-bot
Copy link
Contributor

Java Client Test Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test-openshift-restclient-java/78/)

@jcantrill
Copy link
Contributor Author

[merge]

@openshift-bot
Copy link
Contributor

Java Client Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge-openshift-restclient-java/73/)

@openshift-bot
Copy link
Contributor

Evaluated for javaclient merge up to b2d6786

openshift-bot pushed a commit that referenced this pull request Mar 1, 2016
@openshift-bot openshift-bot merged commit 49c740d into openshift:master Mar 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants