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
OCPCLOUD-2573: Enables kubelet cloud credential providers #1786
OCPCLOUD-2573: Enables kubelet cloud credential providers #1786
Conversation
Skipping CI for Draft Pull Request. |
Hello @theobarberbany! Some important instructions when contributing to openshift/api: |
333da64
to
d3a560f
Compare
d3a560f
to
0a68071
Compare
/test e2e-gcp |
Need an e2e-gcp to prove this is working as expected |
GCP e2es are a bit sad at the moment, but it's being worked on :) I don't think the failures are related to this PR! |
0a68071
to
4365cdc
Compare
da5de8f
to
75b5ca3
Compare
Changes LGTM but I need to see a pass on GCP before we can merge this |
GCP is failing due to a configuration error. openshift/machine-config-operator#4260 fixes that. |
75b5ca3
to
afa1c1f
Compare
/test e2e-gcp Cluster failed to bootstrap |
Private registry tests appear to be passing, among other inconsistent and spurious failures |
/hold Revision 2d24f30 was retested 3 times: holding |
Enables the DisableKubeletCloudCredentialProviders feature gate Cloud credential providers for AWS, GCP and Azure are now functional. We can disable the in tree kubelet providers.
2d24f30
to
fd10d8f
Compare
/test e2e-azure |
/test e2e-aws-serial |
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: damdo, JoelSpeed, theobarberbany 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 |
/unhold |
@theobarberbany: all tests passed! Full PR test history. Your PR dashboard. 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. |
[ART PR BUILD NOTIFIER] This PR has been included in build ose-cluster-config-api-container-v4.16.0-202403280418.p0.gf8aa75d.assembly.stream.el9 for distgit ose-cluster-config-api. |
Enables the DisableKubeletCloudCredentialProviders feature gate
Cloud credential providers for AWS, GCP and Azure are now functional. We can disable the in tree kubelet providers.