-
Notifications
You must be signed in to change notification settings - Fork 17
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
feat: find node by uuid #142
Conversation
Hi, @fibbs can you check this PR. I've run test in my lab already. It can find the VM by UUID. Can you check the rancher setup... git clone ...
make images USERNAME=$GITHUB-ACC PLATFORM=linux/amd64 TAG=uuid PUSH=true |
In some setups, the Proxmox VM name may differ from the Linux hostname. To reliably identify a VM within a Proxmox cluster, we can use the system's UUID Signed-off-by: Serge Logvinov <[email protected]>
I've run some test, and i hope it solves the rancher case. You can check the |
I will test this within the next days, now that I have got the variant running that has node-name and vm name equal. Very much appreciated that you consider to support such a more dynamic configuration. |
I did successfully test now: Installed cluster in the same way as described in the other issue, but with the difference that my three nodes are called
So, there is no way to get the reference to the VMID by matching the K8S node name to the VM name. I have used the following
CCM starts and does its thing:
First node correctly gets taint removed, labels and providerID set:
After this first node being all up and running, Rancher continues to set the other nodes up, which also get their labels etc. correctly:
All fine! Great stuff! |
In some setups, the Proxmox VM name may differ from the Linux hostname. To reliably identify a VM within a Proxmox cluster, we can use the system's UUID
refs #140
Pull Request
What? (description)
Why? (reasoning)
Acceptance
Please use the following checklist:
make lint
)make unit
)