stackit ccm use new stackit client#1201
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/test pull-cloud-provider-stackit-verify |
|
/cherrypick release-v1.33 |
|
@breuerfelix: once the present PR merges, I will cherry-pick it on top of DetailsIn response to this:
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-sigs/prow repository. |
Signed-off-by: Felix Breuer <f.breuer94@gmail.com>
Signed-off-by: Felix Breuer <f.breuer94@gmail.com>
Signed-off-by: Felix Breuer <f.breuer94@gmail.com>
Signed-off-by: Felix Breuer <f.breuer94@gmail.com>
34463e5 to
765a187
Compare
How to categorize this PR?
/kind enhancement
What this PR does / why we need it:
This PR is a part of breaking down #1060 PR into smaller pieces, where we are having ccm using new stackit client.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
IaaS functions being added
LoadBalancer functions being added
In-terms of client code, there is no major changes, just it follows new way of using the stackit client.
NOTE:
In respective of testing the ccm uses the new client and the unit tests are being changed but we need to test it on ondemand.
Breaking changes: