-
Notifications
You must be signed in to change notification settings - Fork 95
feat: document for hotplug nic #917
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Tim Liou <[email protected]>
|
Signed-off-by: Tim Liou <[email protected]>
jillian-maroket
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review done
Co-authored-by: Jillian Maroket <[email protected]> Signed-off-by: Tim Liou <[email protected]>
Co-authored-by: Jillian Maroket <[email protected]> Signed-off-by: Tim Liou <[email protected]>
Co-authored-by: Jillian Maroket <[email protected]> Signed-off-by: Tim Liou <[email protected]>
wheatdog
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jillian-maroket Thanks for your previous review! I have checked the revised version. For hotunplug, I think VMs should have at least two interfaces.
Co-authored-by: Tim Liou <[email protected]> Signed-off-by: Jillian Maroket <[email protected]>
innobead
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one question.
P.S. It seems we rely only on migration-based hotplug: https://kubevirt.io/user-guide/network/hotplug_interfaces/#migration-based-hotplug.
|
|
||
| You cannot perform NIC hotplugging on virtual machines created in clusters running Harvester versions earlier than v1.7.0 _if the network interface's MAC address was not explicitly configured_. To enable this feature, you must restart those virtual machines. Restarting allows the system to automatically save the observed MAC addresses to the virtual machine's configuration. | ||
|
|
||
| In addition, virtual machines created by the [Harvester Node Driver](../rancher/node/node-driver.md) do not support NIC hotplugging and hotunplugging. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the Harvester Terraform provider support this? If not, we may also mention it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good question. I think it's not supported.
Yes, currently only for migration-based hotplug. The reason for not supporting non-migration-based hotplug is documented in harvester/harvester#9498. |
Signed-off-by: Tim Liou <[email protected]>
Problem:
Introduce the new Hotplug NIC feature and its limitation.
Solution:
Provide a dedicate documentation page and related images.
Related Issue(s):
harvester/harvester#7042
Test plan:
N/A
Additional documentation or context