Skip to content

Conversation

@minglumlu
Copy link
Member

The issue is the "MANAGEMENT_INTERFACE" in /etc/xensource-inventory is not cleared in a networking reset. While the networkd is still trying to read the value of "MANAGEMENT_INTERFACE" as the bridge name. This caused the new network interface was added into the bridge before reset.

@minglumlu minglumlu requested review from liulinC and robhoes August 20, 2025 01:47
@last-genius
Copy link
Contributor

Looking at the xe-network-reset script in xapi on the parallel feature branch, the logic here looks somewhat different: https://github.com/xapi-project/xen-api/blob/feature/host-network-device-ordering/python3/bin/xe-reset-networking (I'm not even talking about the weird if not else construct in bridge = '' if not rename_script_exists else get_bridge_name(device, vlan)). On master, the scripts are practically identical and really easy to compare

Could we bring these scripts in sync? Otherwise it'd be difficult to spot differences from a quick look, and since xsconsole is practically untested, this will slip by

@minglumlu
Copy link
Member Author

Could we bring these scripts in sync?

The fix is to get them in sync in function. Unfortunately both xsconsole and xapi are using own code, and both need to be used for different releases. This introduces many if-else. Once they are forked, these could be eliminated.

If you are referring the condition name "rename_script_exists" vs "interface_rename_dir", I could get them same in this PR.

@minglumlu minglumlu marked this pull request as draft August 20, 2025 10:34
@minglumlu minglumlu force-pushed the private/mingl/CA-415864 branch from 7f5c233 to 547393c Compare August 20, 2025 12:23
@minglumlu minglumlu marked this pull request as ready for review August 20, 2025 12:28
@changlei-li changlei-li force-pushed the feature/host-network-device-ordering branch from bc8629d to 07739ba Compare September 1, 2025 02:36
@changlei-li
Copy link
Contributor

The feature branch is rebased on top of master. You may need also rebase your commits.

The issue is the "MANAGEMENT_INTERFACE" in /etc/xensource-inventory is
not cleared in a networking reset. While the networkd is still trying to
read the value of "MANAGEMENT_INTERFACE" as the bridge name. This caused
the new network interface was added into the bridge before reset.

Signed-off-by: Ming Lu <[email protected]>
@minglumlu minglumlu force-pushed the private/mingl/CA-415864 branch from 547393c to d8baa93 Compare September 1, 2025 06:30
@minglumlu minglumlu merged commit 5eabd53 into xapi-project:feature/host-network-device-ordering Sep 2, 2025
1 check passed
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.

5 participants