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

vmware sample does not work without disableComponents and 'cloudController' #227

Closed
donovanmuller opened this issue Dec 27, 2023 · 2 comments
Labels
kind/bug Something isn't working needs-priority Indicates an issue or PR needs a priority assigning to it needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done.

Comments

@donovanmuller
Copy link

What happened:

When using https://github.com/rancher-sandbox/cluster-api-provider-rke2/tree/main/samples/vmware, the first control plane Node would never pass the preflight checks due to the Node ProviderID not matching the vsphere ProviderID (see kubernetes-sigs/cluster-api-provider-vsphere#2400).

This is because the RKE cloud controller starts before the vsphere CPI is installed and sets the ProviderID to the RKE value, which won't be overwritten by the vsphere CPI once it's running.

Adding the following to https://github.com/rancher-sandbox/cluster-api-provider-rke2/blob/main/samples/vmware/rke2controlplane.yaml disabled the RKE cloud controller and allowed the vsphere CPI to set the ProviderID properly:

  serverConfig:
    disableComponents:
      kubernetesComponents:
        - cloudController

What did you expect to happen:

The first control plane Node to pass the preflight checks.

How to reproduce it:

Use the vpshere example as-is.

Anything else you would like to add:

If this is not expected, the sample should be updated to disable the cloudController.

Environment:

  • rke provider version: v0.2.2
  • RKE2 version: v1.26.11+rke2r1
@donovanmuller donovanmuller added kind/bug Something isn't working needs-priority Indicates an issue or PR needs a priority assigning to it needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Dec 27, 2023
@donovanmuller
Copy link
Author

See #268 (comment)

@furkatgofurov7
Copy link
Contributor

@donovanmuller Hey, the new provider version (v0.2.5) has been released, can you verify if it solves this issue?

@furkatgofurov7 furkatgofurov7 added the priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done. label Feb 23, 2024
@alexander-demicev alexander-demicev closed this as not planned Won't fix, can't repro, duplicate, stale Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working needs-priority Indicates an issue or PR needs a priority assigning to it needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done.
Projects
None yet
Development

No branches or pull requests

3 participants