-
Notifications
You must be signed in to change notification settings - Fork 182
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
Add credentials request for AWS. #116
Conversation
Relocated from cloud-credentials-operator repo per recent discussions. Ingress team will have control over your own creds going forward.
/lgtm |
/retest |
1 similar comment
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest |
/retest |
/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. |
/retest |
2 similar comments
/retest |
/retest |
/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. |
/retest |
1 similar comment
/retest |
@dgoodwin something's consistently failing here... |
Yeah I think it's far before we could break anything though if it's a routing table timeout. |
Does it seem to be only affecting this PR? |
/test e2e-aws |
It's not just this PR (I've seen similar errors on other PRs today), but it seems like this PR has not had any successful runs, which is suspicious. |
Similar one over in installer just now: https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/openshift_installer/1157/pull-ci-openshift-installer-master-e2e-aws/3516 |
/test e2e-aws-operator |
/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. |
This is becoming a regular story: operator log and clusterversion have cluster id |
/retest Please review the full test history for this PR and help us cut down flakes. |
Relocated from cloud-credentials-operator repo per recent discussions.
Ingress team will have control over your own creds going forward.