Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

justinsb
Copy link
Member

Fix #11201

@thockin thockin added lgtm "Looks good to me", indicates that a PR is ready to be merged. ok-to-merge labels Jul 14, 2015
@thockin
Copy link
Member

thockin commented Jul 14, 2015

Required for AWS support. Justin, I presume this would need to be cherry-picked into release-1.0?

@k8s-bot
Copy link

k8s-bot commented Jul 14, 2015

GCE e2e build/test passed for commit 355e922.

erictune added a commit that referenced this pull request Jul 14, 2015
AWS: Restore (erroneously) removed get_instance_private_ip
@erictune erictune merged commit 758913c into kubernetes:master Jul 14, 2015
@thockin
Copy link
Member

thockin commented Jul 14, 2015

@justinsb cherry pick for 1.0 or not?

On Tue, Jul 14, 2015 at 1:04 PM, Eric Tune [email protected] wrote:

Merged #11241
#11241.


Reply to this email directly or view it on GitHub
#11241 (comment)
.

@justinsb
Copy link
Member Author

It's only for KUBE_ENABLE_MINION_PUBLIC_IP=false, which is not the common configuration, but we do document it. The patch is low risk (adds a function to a bash file), so I think we should probably merge it, but I'm not sure exactly how problematic it is to cherry-pick things to 1.0 at this point in the cycle. If we have a 1.0.1 backlog this might be a candidate for that.

@thockin
Copy link
Member

thockin commented Jul 14, 2015

Cherry picking IS the 1.0.1 queue-up :)

Can you try running ./hack/cherry_pick_pull.sh (so someone outside of us
can try it :) ?

On Tue, Jul 14, 2015 at 2:03 PM, Justin Santa Barbara <
[email protected]> wrote:

It's only for KUBE_ENABLE_MINION_PUBLIC_IP=false, which is not the common
configuration, but we do document it. The patch is low risk (adds a
function to a bash file), so I think we should probably merge it, but I'm
not sure exactly how problematic it is to cherry-pick things to 1.0 at this
point in the cycle. If we have a 1.0.1 backlog this might be a candidate
for that.


Reply to this email directly or view it on GitHub
#11241 (comment)
.

zmerlynn added a commit that referenced this pull request Jul 14, 2015
…41-upstream-release-1.0

Automated cherry pick of #11241
@zmerlynn zmerlynn mentioned this pull request Jul 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants