Skip to content
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

doc: Fix minimal OVN configuration in preseed #605

Merged
merged 2 commits into from
Feb 4, 2025

Conversation

roosterfish
Copy link
Contributor

No description provided.

Signed-off-by: Julian Pelizäus <[email protected]>
@roosterfish roosterfish requested a review from minaelee February 3, 2025 12:19
@github-actions github-actions bot added the Documentation Documentation needs updating label Feb 3, 2025
@@ -174,6 +174,9 @@ In this case `micro01` takes over the role of the initiator.
Multicast discovery is used to find the other machines on the network.

On each of the machines `eth1` is used as uplink for the OVN network.
The network `192.0.2.0/24` is configured on the uplink with `192.0.2.1` as the gateway.
Copy link
Member

Choose a reason for hiding this comment

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

what does "network" mean in this case, the uplink network or the ovn network?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In this case it's the uplink network. I'll rephrase it to make it more clear.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

At least the IPv4 uplinks gateway and range have to be set to have outside connectivity from the OVN overlay networks.

Signed-off-by: Julian Pelizäus <[email protected]>
@roosterfish roosterfish merged commit d379dde into canonical:main Feb 4, 2025
14 checks passed
@roosterfish roosterfish deleted the fix_minimal_ovn branch February 4, 2025 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Documentation needs updating
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants