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

UPSTREAM: 39842: Remove duplicate calls to DescribeInstance #12740

Merged
merged 1 commit into from
Feb 7, 2017

Conversation

gnufied
Copy link
Member

@gnufied gnufied commented Jan 31, 2017

This removes duplicate DescribeInstance calls from AWS volume code path.

@gnufied gnufied added this to the 1.5.0 milestone Jan 31, 2017
@gnufied
Copy link
Member Author

gnufied commented Jan 31, 2017

cc @eparis @childsb

@gnufied
Copy link
Member Author

gnufied commented Jan 31, 2017

I wouldn't cherry pick this change in 3.4 because 3.4 doesn't have this describeInstance polling bug - kubernetes/kubernetes#31858

@gnufied
Copy link
Member Author

gnufied commented Feb 6, 2017

cc @eparis @childsb can we merge this?

@childsb
Copy link
Contributor

childsb commented Feb 6, 2017

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 55b8d3e

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/13618/) (Base Commit: bb51aa5)

@childsb
Copy link
Contributor

childsb commented Feb 6, 2017

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 55b8d3e

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/13637/) (Base Commit: 751d0ba)

@childsb childsb merged commit 2bf65d4 into openshift:master Feb 7, 2017
@smarterclayton
Copy link
Contributor

This was openshift/origin-gce#12

@pweil- pweil- mentioned this pull request Feb 23, 2017
20 tasks
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.

4 participants