Skip to content

network: fix route for own subnet#441

Merged
dupondje merged 1 commit intomasterfrom
fix/route
Oct 21, 2025
Merged

network: fix route for own subnet#441
dupondje merged 1 commit intomasterfrom
fix/route

Conversation

@dupondje
Copy link
Member

Before the fix, the following routes were created for traffic to ovirtmgmt:
default via 10.0.10.1 dev ovirtmgmt proto static
10.0.10.0/24 via 10.0.10.5 dev ovirtmgmt proto static

Where 10.0.10.5 is the IP address of the server itself.

While this seems to work fine on CentOS 9, this breaks traffic from the own subnet on CentOS 10.
Therefor we replace it with the proper route:
10.0.10.0/24 dev ovirtmgmt proto kernel scope link src 10.0.10.5 metric 425

A question for this was also opened on the nmstate repository [1].

1: nmstate/nmstate#3003

Before the fix, the following routes were created for traffic to
ovirtmgmt:
default via 10.0.10.1 dev ovirtmgmt proto static
10.0.10.0/24 via 10.0.10.5 dev ovirtmgmt proto static

Where 10.0.10.5 is the IP address of the server itself.

While this seems to work fine on CentOS 9, this breaks traffic from the
own subnet on CentOS 10.
Therefor we replace it with the proper route:
10.0.10.0/24 dev ovirtmgmt proto kernel scope link src 10.0.10.5 metric 425

A question for this was also opened on the nmstate repository [1].

1: nmstate/nmstate#3003

Signed-off-by: Jean-Louis Dupond <jean-louis@dupond.be>
@dupondje dupondje requested a review from almusil as a code owner October 13, 2025 06:54
@dupondje
Copy link
Member Author

/ost

@github-actions
Copy link

⏳ Running ost suite 'basic-suite-master' on distro 'el9stream'.

Follow the progress here.

@github-actions
Copy link

😎💪 ost suite 'basic-suite-master' on distro 'el9stream' finished successfully. (details)

@dupondje dupondje merged commit d3e5d98 into master Oct 21, 2025
34 checks passed
@dupondje dupondje deleted the fix/route branch October 21, 2025 12:16
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.

2 participants