Skip to content
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

Up cloud-provider version #67

Merged
merged 2 commits into from
Nov 2, 2024

Conversation

Hitych
Copy link
Contributor

@Hitych Hitych commented Nov 1, 2024

Cloud-provider actual tag now is 0.31.2. There are few impovements in comparison with 0.27.3. Most interesting to me are metrics for node-controller.
It requires minor change in app.NewCloudControllerManagerCommand call related to new controllers naming convention.
I tested base scenario and it works as expected:

  1. Create/delete VM
  2. Create/delete LoadBalancer service

@nabokihms
Copy link
Member

@Hitych thanks for the PR. Could you please rebase the branch? I've just updated images in the Dockerfile.

@Hitych Hitych force-pushed the up-cloud-provider-version branch from 789e535 to 9ad2f4c Compare November 2, 2024 03:42
@Hitych
Copy link
Contributor Author

Hitych commented Nov 2, 2024

@Hitych thanks for the PR. Could you please rebase the branch? I've just updated images in the Dockerfile.

Done

go.mod Outdated Show resolved Hide resolved
Signed-off-by: Maksim Nabokikh <[email protected]>
@nabokihms nabokihms merged commit 15852d2 into deckhouse:master Nov 2, 2024
3 checks passed
@nabokihms
Copy link
Member

@Hitych thank you for your contribution.

@nabokihms nabokihms added the dependencies Pull requests that update a dependency file label Nov 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants