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.

Sign up for GitHub

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

Jump to bottom

rkt: Update the directory path for saving auth config. #23667

Merged
merged 1 commit into from
Apr 15, 2016
Merged

rkt: Update the directory path for saving auth config. #23667

merged 1 commit into from
Apr 15, 2016

Conversation

yifan-gu
Copy link
Contributor

Since #23308 is merged, now we have more stable way to determine where to store the auth configs.

cc @yujuhong @sjpotter

@yifan-gu yifan-gu added sig/node Categorizes an issue or PR as relevant to SIG Node. area/rkt labels Mar 31, 2016
@k8s-github-robot k8s-github-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Mar 31, 2016
@k8s-bot
Copy link

k8s-bot commented Mar 31, 2016

GCE e2e build/test passed for commit 00fd95caa8855b341417df1a5153a986084e7df0.

@yifan-gu yifan-gu added release-note-none Denotes a PR that doesn't merit a release note. and removed release-note-label-needed labels Apr 5, 2016
@yifan-gu yifan-gu added this to the rktnetes-v1.0 milestone Apr 5, 2016
@k8s-bot
Copy link

k8s-bot commented Apr 7, 2016

GCE e2e build/test passed for commit 143f4df7134d3b6a6c86b61de3511e329732aa5e.

@yifan-gu
Copy link
Contributor Author

yifan-gu commented Apr 8, 2016

ping @yujuhong ?

@yujuhong
Copy link
Contributor

yujuhong commented Apr 8, 2016

Will review today.

@yujuhong yujuhong added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 8, 2016
@yifan-gu
Copy link
Contributor Author

yifan-gu commented Apr 8, 2016

@yujuhong Thank you

@k8s-bot
Copy link

k8s-bot commented Apr 9, 2016

GCE e2e build/test passed for commit 143f4df7134d3b6a6c86b61de3511e329732aa5e.

@k8s-bot
Copy link

k8s-bot commented Apr 11, 2016

GCE e2e build/test passed for commit 143f4df7134d3b6a6c86b61de3511e329732aa5e.

@k8s-bot
Copy link

k8s-bot commented Apr 13, 2016

GCE e2e build/test passed for commit 143f4df7134d3b6a6c86b61de3511e329732aa5e.

@pwittrock pwittrock mentioned this pull request Apr 14, 2016
Closed
@yujuhong
Copy link
Contributor

yujuhong commented Apr 14, 2016 •
edited
Loading

The node e2e suite failed at cleaning up the test:

E0413 13:18:44.382200    4904 e2e_remote.go:132] Failed to cleanup tmp directory /tmp/gcloud-e2e-169525228 on host Command [ssh -i /home/jenkins/.ssh/google_compute_engine -o UserKnownHostsFile=/dev/null -o IdentitiesOnly=yes -o CheckHostIP=no -o StrictHostKeyChecking=no tmp-node-e2e-433d8995-e2e-node-coreos-stable20160218-image -- rm -rf /tmp/gcloud-e2e-169525228] failed with error: exit status 255 and output:
ssh_exchange_identification: Connection closed by remote host
.  Output:
ssh_exchange_identification: Connection closed by remote host
================================================================
Failure Finished Host tmp-node-e2e-433d8995-e2e-node-coreos-stable20160218-image Test Suite

According to @pwittrock, this should be an infra issue.

@yujuhong
Copy link
Contributor

@k8s-bot test node e2e experimental

@yifan-gu
Copy link
Contributor Author

Rebased the master to include #24303

@k8s-github-robot
Copy link

PR changed after LGTM, removing LGTM.

@k8s-github-robot k8s-github-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 15, 2016
@yifan-gu yifan-gu added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 15, 2016
@k8s-bot
Copy link

k8s-bot commented Apr 15, 2016

GCE e2e build/test passed for commit e803f55.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit d56ec66 into kubernetes:master Apr 15, 2016
@yifan-gu yifan-gu deleted the config branch April 15, 2016 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@yujuhong yujuhong

Labels
area/rkt lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. sig/node Categorizes an issue or PR as relevant to SIG Node. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
rktnetes-v1.0
Development

Successfully merging this pull request may close these issues.
7 participants
@yifan-gu @k8s-bot @yujuhong @k8s-github-robot @k8s-teamcity-mesosphere @googlebot @wojtek-t