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
hack/local-up-cluster.sh: Remove old dynamic certs #116385
Conversation
This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/retest |
2 similar comments
/retest |
/retest |
The failure seem completely unrelated, but quite persistent :-( |
/retest |
f1a5126
to
7355364
Compare
Ok, so I did a revert of my commit and pushed again. The diff on the PR is absolutely zero, but tests still fail.... :-/ |
7355364
to
52131a4
Compare
I've changed it to only run plain sudo if the command fails, so it should leave the CI path completely untouched too. |
/retest |
52131a4
to
c16d08d
Compare
It seems all are green now, but |
c16d08d
to
9d9a6a3
Compare
/retest |
a9f8d7b
to
0cecd4e
Compare
When running this script more than once on Debian and Ubuntu, we fail to chown -R the CERT_DIR due to this file owned by root and the CERT_DIR owned by the unprivileged user running the script. Let's remove the file, that is something we can always do, before generating the certs. This fixes the problem on Debian and Ubuntu local setups. Signed-off-by: Rodrigo Campos <rodrigoca@microsoft.com>
0cecd4e
to
d212041
Compare
/retest |
/retest |
@rata it's not your change breaking this CI job. it's been broken for a bit of time. will try to carve out time to fix it. /skip |
LGTM label has been added. Git tree hash: 6d8dc1c44ba60ba248af9948941ba0f66c4ad380
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dims, rata The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@rata: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/sig node /retest |
Thanks! Just updated the PR title and description to match what was merged (the initial PR was slightly different, I tried this alternative after opening the PR, and the alternative was better as it matches what we already do for other components in the script) |
hack/local-up-cluster.sh: Remove old dynamic certs
When running this script more than once on Debian and Ubuntu, we fail to
chown -R the CERT_DIR due to this file owned by root and the CERT_DIR
owned by the unprivileged user running the script.
Let's remove the file, that is something we can always do, before
generating the certs. This fixes the problem on Debian and Ubuntu local
setups.
Signed-off-by: Rodrigo Campos rodrigoca@microsoft.com
What type of PR is this?
/kind bug
What this PR does / why we need it:
Fixes local setups with local-up-cluster.sh
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Please note this is already done for the kubelet, it was just missing in the api server:
kubernetes/hack/local-up-cluster.sh
Line 740 in aab9a7c
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: