Skip to content

Conversation

@sibucan
Copy link
Contributor

@sibucan sibucan commented Apr 14, 2022

General:

  • Have you removed all sensitive information, including but not limited to access keys and passwords?
  • Have you checked to ensure there aren't other open or closed Pull Requests for the same bug/feature/question?

Pull Request Guidelines:

  1. Does your submission pass tests?
  2. Have you added tests?
  3. Are you addressing a single feature in this PR?
  4. Are your commits atomic, addressing one change per commit?
  5. Are you following the conventions of the language?
  6. Have you saved your large formatting changes for a different PR, so we can focus on your work?
  7. Have you explained your rationale for why this feature is needed?
  8. Have you linked your PR to an open issue

This PR upgrades all Kubernetes-related dependencies from v1.19.2 to v1.21.0. This includes some changes on how the CCM command is initialized, which is explained in https://github.com/kubernetes/kubernetes/blob/v1.21.0/cmd/cloud-controller-manager/main.go and https://github.com/kubernetes/cloud-provider/blob/master/sample/README.md

Now that the Linode CCM is using v1.21.0 k8s dependencies, it should fix #107

Copy link
Contributor

@thorn3r thorn3r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested and working 👍

Copy link
Contributor

@0xch4z 0xch4z left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thorn3r thorn3r merged commit 1428580 into linode:master Apr 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CCM needs agent/worker nodes to create the automatic loadbalancer for a service/ingress

3 participants