-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Force external cloud controller manager on OS #9942
Force external cloud controller manager on OS #9942
Conversation
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.
otherwise looks good but this feature is still behind featureflag?
export KOPS_FEATURE_FLAGS=EnableExternalCloudController
edit I might be wrong:
key := "openstack.addons.k8s.io" |
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
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: olemarkus, zetaab 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 |
@olemarkus try to rebase this against master if the test will be fixed (after that another PR #9943 is merged) |
dbf2b97
to
5279047
Compare
@olemarkus can you try rebasing one more time against master |
5279047
to
b8bc6d3
Compare
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 test is sort of redundant when we have integration tests etc covering rendering of these templates
e5a3693
to
d3ae52c
Compare
/retest |
/lgtm |
Fixes #9619
Supersedes #9625
/cc @zetaab