Skip to content

Comments

ImageStreamImage returns incorrect image info#6945

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
smarterclayton:imagestreamimage_wrong
Feb 2, 2016
Merged

ImageStreamImage returns incorrect image info#6945
openshift-bot merged 1 commit intoopenshift:masterfrom
smarterclayton:imagestreamimage_wrong

Conversation

@smarterclayton
Copy link
Contributor

Go range variable can't be the value we take a reference to.

Fixes #6919

@brenton this is a 3.1.1.1 candidate fix as well as 1.1.1.2

Go range variable can't be the value we take a reference to.
@smarterclayton
Copy link
Contributor Author

@miminar @soltysh review please

Copy link
Contributor

Choose a reason for hiding this comment

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

do you need to do the same thing to avoid sharing history?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

History has a pointer to a slice of items, so no.

@pweil-
Copy link

pweil- commented Feb 1, 2016

LGTM

@smarterclayton
Copy link
Contributor Author

[merge]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/4820/) (Image: devenv-rhel7_3307)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to ba19b8e

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@smarterclayton
Copy link
Contributor Author

[test]

1 similar comment
@smarterclayton
Copy link
Contributor Author

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to ba19b8e

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test ABORTED (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/685/)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI not reflecting correct deployment

4 participants