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

Skip to content

Remove ~/repo before moving Cloud SDK components there #18840

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

Merged
merged 1 commit into from
Dec 18, 2015
Merged

Remove ~/repo before moving Cloud SDK components there #18840

merged 1 commit into from
Dec 18, 2015

Conversation

ikehz
Copy link
Contributor

@ikehz ikehz commented Dec 17, 2015

~/repo might have existed from a prior run, if we're doing jobs in sequence and not cleaning up in between.

Not sure if this is the right way to fix this problem; I'm open to suggestions.

Failure e.g. http://kubekins.dls.corp.google.com/view/Upgrade%20Test%20-%20GKE/job/kubernetes-upgrade-gke-1.0-master-step5-upgrade-cluster/21/console

@ikehz ikehz added priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. area/test-infra labels Dec 17, 2015
@k8s-github-robot
Copy link

Labelling this PR as size/XS

@k8s-github-robot k8s-github-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Dec 17, 2015
@j3ffml
Copy link
Contributor

j3ffml commented Dec 17, 2015

This should work fine. Haven't run into it yet because we normally wipe the workspace between runs.

@j3ffml j3ffml added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 17, 2015
@k8s-bot
Copy link

k8s-bot commented Dec 17, 2015

GCE e2e test build/test passed for commit 375f4df.

@j3ffml
Copy link
Contributor

j3ffml commented Dec 18, 2015

cc @k8s-oncall. Manually merging to unbreak kubernetes-soak-continuous-e2e-gke

j3ffml added a commit that referenced this pull request Dec 18, 2015
Remove ~/repo before moving Cloud SDK components there
@j3ffml j3ffml merged commit a2f076b into kubernetes:master Dec 18, 2015
k8s-github-robot referenced this pull request Dec 18, 2015
…8840-upstream-release-1.1

Auto commit by PR queue bot
ikehz pushed a commit that referenced this pull request Dec 21, 2015
…8840-#18903-upstream-release-1.0

Automated cherry pick of #18840 #18903 upstream release 1.0
shyamjvs referenced this pull request in shyamjvs/kubernetes Dec 1, 2016
…pick-of-#18840-upstream-release-1.1

Auto commit by PR queue bot
shouhong referenced this pull request in shouhong/kubernetes Feb 14, 2017
…pick-of-#18840-upstream-release-1.1

Auto commit by PR queue bot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/test-infra lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants