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

Add release note for OVN-Kubernetes secondary networks #87995

Open
wants to merge 4 commits into
base: enterprise-4.18
Choose a base branch
from

Conversation

@jab-rh jab-rh added this to the Planned for 4.18 GA milestone Feb 4, 2025
@jab-rh jab-rh self-assigned this Feb 4, 2025
@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 4, 2025
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Feb 4, 2025

🤖 Sat Feb 08 00:23:59 - Prow CI generated the docs preview:

https://87995--ocpdocs-pr.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-18-release-notes.html

@openshift-ci openshift-ci bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Feb 5, 2025
@maiqueb
Copy link

maiqueb commented Feb 7, 2025

cc @maiqueb @RamLavi

Copy link

@maiqueb maiqueb left a comment

Choose a reason for hiding this comment

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

We need to explain the network configuration sync concept to properly explain the localnet configuration update IMHO.

[id="ocp-release-notes-networking-network-attachment-definition_{context}"]
==== Network attachments support dynamic reconfiguration

Previously, the `NetworkAttachmentDefinition` CR was immutable. With this release, you can edit an existing `NetworkAttachmentDefinition` CR. Support for editing makes it easier to accommodate changes in the underlying network infrastructure, such as adjusting the MTU of a network interface. You still must restart any workloads that use the network attachment definition for the network changes to take effect for those pods.
Copy link

Choose a reason for hiding this comment

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

It's imperative that we mention the concept of the network being in-sync: let's say you have two NADs that both grant access to the same network (i.e. same network name).

If the configurations on these networks are not in sync, when you restart the workloads, they will not get the updated values. Only once all NADs pointing to the same network are sync'ed - i.e. share the same configuration - will the updated configuration trickle down to the workloads (after a restart).

While the network is not in sync, the behavior is undefined - let's say the control planes restarts while the network is not in sync: we do not have any guarantees over which configuration will be picked up.

@jcaamano FYI ; in case you want to say something further.

Copy link

openshift-ci bot commented Feb 8, 2025

@jab-rh: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.18 size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants