8000
Skip to content
/ incus Public

incusd/network: Exclude physical networks with parent=none from OVN chassis enablement#2442

Merged
stgraber merged 1 commit intolxc:mainfrom
ViniRodrig:feat/uplink-parent=none
Sep 15, 2025
Merged

incusd/network: Exclude physical networks with parent=none from OVN chassis enablement#2442
stgraber merged 1 commit intolxc:mainfrom
ViniRodrig:feat/uplink-parent=none

Conversation

@ViniRodrig
Copy link
Copy Markdown
Contributor
@ViniRodrig ViniRodrig commented Aug 29, 2025 •
edited
Loading
  • Updated OVN uplink and chassis detection logic to skip networks of type "physical" with parent set to "none".
  • Ensured physical network driver setup permits parent="none" and checks interface existence accordingly.
  • Nodes with physical uplinks that specify a valid parent (not "none") will participate in the OVN HA chassis group; nodes with parent="none" will be excluded.
  • Behaviour tested by creating network with both parent=enp6s0 and parent=none on multiple nodes using Incus deploy.
  • Only nodes with a valid physical uplink parent are added to the HA group, matching expected OVN behaviour.

@ViniRodrig ViniRodrig requested a review from stgraber as a code owner August 29, 2025 19:05
@ViniRodrig ViniRodrig mentioned this pull request Sep 1, 2025
Closed
1 task
@ViniRodrig ViniRodrig marked this pull request as draft September 1, 2025 16:05
@ViniRodrig
network/incusd: Allow parent=none physical networks, bypass chassis e…
fb473d7 
…nablement

Signed-off-by: Vinicius Rodrigues <vmarquesrodrigues25@gmail.com>
@ViniRodrig ViniRodrig force-pushed the feat/uplink-parent=none branch from 133f455 to fb473d7 Compare September 1, 2025 16:10
@ViniRodrig ViniRodrig marked this pull request as ready for review September 3, 2025 13:02
@stgraber
Copy link
Copy Markdown
Member
stgraber commented Sep 15, 2025

Sorry for the delay, had a few busy weeks here, reviewing this one now!

@stgraber stgraber merged commit 16f4bc8 into lxc:main Sep 15, 2025
66 of 68 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stgraber stgraber Awaiting requested review from stgraber stgraber is a code owner

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ViniRodrig @stgraber
0