Skip to content
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

JBIDE-22627 Docker image tag is ignored in Deploy Image to OpenShift wizard #1248

Merged
merged 1 commit into from Jul 19, 2016

Conversation

scabanovich
Copy link
Contributor

No description provided.

}
model.setStartedWithActiveConnection(isAuthorized);
model.initModel(connection, project, isAuthorized);

setNeedsProgressMonitor(true);
}

private String getImageNameWithTag(IDockerImage image) {
Copy link
Contributor

Choose a reason for hiding this comment

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

How does this get the tag we selected? Doesn't this just get the first sorted one, which may or may not be the one i desire? Based on the issue, it looks like we should be getting a specific tag.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Content provider for Docker explorer for each original image creates a set of 'explorer' images based on repoTag, so that each 'explorer' image mostly has 1 tag + occasionally 'latest'. That can be seen on the screenshot to the issue. When user selects an image in Docker explorer, handler gets image object not with all tags for the repo, but most often 1. At least I see it this way. @jcantrill could you please run this PR?

Copy link
Contributor

Choose a reason for hiding this comment

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

I am unable to test. My deploy menu throws exceptions that I have not been able to resolve.

@fbricon
Copy link
Member

fbricon commented Jul 19, 2016

I tested the PR and it works as advertised

@fbricon fbricon merged commit f50daad into jbosstools:master Jul 19, 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.

None yet

3 participants