-
Notifications
You must be signed in to change notification settings - Fork 409
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
mcd: remove unnecessary checks and simplify updateSSHKeys #415
mcd: remove unnecessary checks and simplify updateSSHKeys #415
Conversation
13f12cc
to
b7bd112
Compare
/test unit |
/approve |
/retest |
/test e2e-aws |
/retest |
3 similar comments
/retest |
/retest |
/retest |
/lgtm |
/retest Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest Please review the full test history for this PR and help us cut down flakes. |
Rebase needed |
c5d9d48
to
e0d4601
Compare
e0d4601
to
4b1ea26
Compare
4b1ea26
to
54f1504
Compare
Had some rebase woes (no actual changes on my end). Should be fine now. |
/retest |
I'm going to just hold off on retesting since AWS seems to be going through something today. |
/test e2e-aws |
1 similar comment
/test e2e-aws |
/retest |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jlebon, kikisdeliveryservice, runcom 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 |
/retest |
2 similar comments
/retest |
/retest |
Seeing intermittent bug #414 in the last e2e-aws run. |
/test e2e-aws |
/retest |
Minor updates that I wanted to fix/cleanup since my last ssh pr: