[cloud-provider-vcd] Add legacy mode support for VCD cloud provider CCM#21261
Draft
ptrvsrg wants to merge 1 commit into
Draft
[cloud-provider-vcd] Add legacy mode support for VCD cloud provider CCM#21261ptrvsrg wants to merge 1 commit into
ptrvsrg wants to merge 1 commit into
Conversation
Signed-off-by: Sergey Petrov <sergey.petrov@flant.com>
Collaborator
|
⚪ Workflow details |
Collaborator
|
🟢 Workflow detailsVCD-Standard-Containerd-1.34 - Connection string: 🟢 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fixes the CCM controller list rendering for
cloud-provider-vcd. The--controllersflag was built with a brokenprintfformat string ("%s,$s"instead of"%s,%s"), producing an invalid value like--controllers=cloud-node,$s%!(EXTRA string=cloud-node-lifecycle).Also restores support for legacy CCM controller names for VCD Legacy mode:
legacyMode: true):cloud-node,cloud-node-lifecycle, andservice(when the load balancer is enabled);cloud-node-controller,cloud-node-lifecycle-controller, andservice-lb-controller.Why do we need it, and what problem does it solve?
On clusters installed from the
mainbranch, the VCD CCM was inCrashLoopbecause the--controllersargument was malformed and old CCM controller names were no longer supported for VCD Legacy. This change fixes the malformed argument and returns the legacy controller names, so CCM works forboth legacy and current VCD API versions.
Why do we need it in the patch release (if we do)?
Not necessarily
Checklist
Changelog entries