-
Notifications
You must be signed in to change notification settings - Fork 918
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(vmware): Support network events #6063
base: main
Are you sure you want to change the base?
Conversation
cc @PengpengSun |
ae2d751
to
429877d
Compare
429877d
to
e9917c1
Compare
Do you think I should remove HOTPLUG if the transport is IMC then? |
e9917c1
to
7351b91
Compare
@PengpengSun I updated the PR so that |
7351b91
to
d35e4a3
Compare
This patch updates the DataSource for VMware to support network reconfiguration when the BOOT, BOOT_LEGACY, BOOT_NEW_INSTANCE, and HOTPLUG events are received. Previously the datasource could only reconfigure the network if a new instance ID was detected. However, due to features like backup/restore, migrating VMs, etc., it was determined that it is valuable to support reconfiguring the network without changing the instance ID. This is because changing the instance ID also means running the per-instance configuration modules, such as regenerating the system's SSH host keys, which could lock out automation. This patch also indicates the datasource supports network reconfig for HOTPLUG events, such as when a new NIC is added to a VM. Please note that HOTPLUG is only supported when the transport type is not IMC.
d35e4a3
to
9be437d
Compare
Proposed Commit Message
Additional Context
NA
Test Steps
Merge type
Fixes #5729