Skip to content

Conversation

cardoe
Copy link
Contributor

@cardoe cardoe commented Jun 11, 2025

When booting these images in an OpenStack environment Neutron and OVN are configured with the MAC addresses that the Neutron port. This means that when we use the IPs associated with these ports for sending and receiving traffic the interface that's used needs to use this MAC address as well. This configures the logical VMkernel interface with the MAC address that comes from the config drive which is coming from Neutron.

Additionally this series adds some extra logging to assist with debugging and troubleshooting. Renamed a few functions for clarity and added a final reboot to ensure that the machine is on the network configs as expected.

cardoe added 9 commits June 10, 2025 10:02
Move the test data into their own files to allow for easier testing and
adding data.
Add logging support which will go to the console and to syslog to help
troubleshoot issues with this script.
This function says its for assigning the portgroups but its really for
creating portgroups which can be used for tagged network traffic.
This helper method will create our management network portgroup for
traffic.
Correct how we parse and look for an associated link for a given
network.
VMware wants the VMkernel interfaces, which are the logical interfaces,
that get the IP address assigned to them to follow the vmkX pattern
where X is an integer starting from 0.
The network configuration that this script aims to bring up is the
management network so that someone can log into the hypervisor and then
finish configuring it. This makes adjustments to how the IP interfaces
are assigned to include their MAC address and the MTU for the network to
bring up the networks. Updates the configuration of the interfaces to
support DHCP as well as static IPv4 addresses.
Once the host is configured for all of our settings to take effect we
need to reboot the machine.
@cardoe cardoe requested a review from skrobul June 11, 2025 15:07
Copy link
Collaborator

@skrobul skrobul left a comment

Choose a reason for hiding this comment

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

LGTM, consider renaming inf -> interface or interface_name to be more descriptive

@cardoe cardoe merged commit 21e0b71 into main Jun 11, 2025
2 checks passed
@cardoe cardoe deleted the iterative-updates branch June 11, 2025 15:36
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.

3 participants