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

Skip to content

Conversation

ikehz
Copy link
Contributor

@ikehz ikehz commented Sep 4, 2015

Round two of #13449.

@ikehz
Copy link
Contributor Author

ikehz commented Sep 4, 2015

@cjcullen I'm attempting to guard this on the environment variables passed in, so that we don't have the breakage in CI that we were expecting/got last night.

@k8s-bot
Copy link

k8s-bot commented Sep 4, 2015

GCE e2e build/test passed for commit 219a1fa.

@k8s-github-robot k8s-github-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 4, 2015
@k8s-github-robot
Copy link

Labelling this PR as size/S

@cjcullen
Copy link
Member

cjcullen commented Sep 4, 2015

I don't think we need to version guard this. That introduces extra cruft that we will immediately want to take out. Let's just manage the coordination a little better :).

@cjcullen cjcullen assigned cjcullen and unassigned davidopp Sep 4, 2015
@cjcullen
Copy link
Member

cjcullen commented Sep 4, 2015

(And by let's, I mostly mean that I shouldn't have left the LGTM on overnight)

@ikehz
Copy link
Contributor Author

ikehz commented Sep 4, 2015

@cjcullen Ack.

@k8s-bot
Copy link

k8s-bot commented Sep 4, 2015

GCE e2e build/test passed for commit 5936d0fb181048596cea0967e53efa12711fe971.

@k8s-bot
Copy link

k8s-bot commented Sep 4, 2015

GCE e2e build/test passed for commit 219a1fa.

@cjcullen cjcullen added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 4, 2015
@cjcullen
Copy link
Member

cjcullen commented Sep 4, 2015

LGTM.

ghost pushed a commit that referenced this pull request Sep 4, 2015
Second attempt at GCE tokens behavior to new format
@ghost ghost merged commit 059d762 into kubernetes:master Sep 4, 2015
zmerlynn added a commit that referenced this pull request Sep 10, 2015
…3594-upstream-release-1.0

Automated cherry pick of #13594
@brendandburns brendandburns mentioned this pull request Sep 10, 2015
@ikehz ikehz deleted the tokens branch October 14, 2015 18:17
This pull request was closed.
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. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants