-
Notifications
You must be signed in to change notification settings - Fork 38
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 upstream patch for --cloud-config option #180
Add upstream patch for --cloud-config option #180
Conversation
Skipping CI for Draft Pull Request. |
opening this as draft because it needs the chain ending with #179 before it can merge, and there are 2 big commits that need to be merged. |
dde0e46
to
a3bf8d1
Compare
rebased |
this is ready for review, but still needs to wait for #179 |
a3bf8d1
to
d45f4f8
Compare
this should go before #184 |
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.
/approve
The additional commit here is relatively straight forward, no issues from my side
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JoelSpeed 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 |
…api provider - Leverage --cloud-config to allow for providing a separate kubeconfig for Cluster API management and workload cluster resources - Allow for fallback to previous behavior when --cloud-config is not specified for backward compatibility - Provides a --clusterapi-cloud-config-authoritative flag to disable the above fallback behavior and allow for both the management and workload cluster clients to use the in-cluster config
d45f4f8
to
61b8048
Compare
rebased after 179 merged |
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
This change brings in the upstream patch kubernetes#3203