Skip to content

Conversation

@dfitzmau
Copy link
Contributor

@dfitzmau dfitzmau commented Aug 11, 2025

Version(s):
4.14+

Drop localnet reference for 4.13 and 4.12.

Issue:
OCPBUGS-36674

Link to docs preview:

  • SME has approved this change (Pablo Alonso Rodriguez, Ramesh Sahoo).
  • QE has approved this change (Arti Sood).

Additional resources:

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Aug 11, 2025
@dfitzmau dfitzmau force-pushed the OCPBUGS-36674-re branch 2 times, most recently from 908e3f7 to 5bd689f Compare August 11, 2025 15:47
@dfitzmau dfitzmau changed the title Ocpbugs 36674 re OCPBUGS-36674 re: Adding DAY 2 limitations for configuring the primary interface by using the OVNK plug-in Aug 11, 2025
@rameshsahoo111
Copy link
Contributor

rameshsahoo111 commented Aug 11, 2025

Please include this additional suggestions
If a node's primary network interface (e.g., eth0 or bond0) was configured during cluster installation, it cannot be reused to create additional networks. In such cases, you can use the OVNK localnet topology to create additional networks that connect to the underlying physical network, with or without VLAN tagging.

@palonsoro
Copy link
Contributor

@dfitzmau This would LGTM as it is currently written.

@rameshsahoo111 your suggestion would only apply to versions where localnet topology is available and only for pod secondary networks, not for the case where you need to make some networks accessible to the nodes (e.g. to access some storage backend). If we are to include your suggestion, we would need to explain these limitations.

@openshift-ci openshift-ci bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Aug 12, 2025
@dfitzmau dfitzmau force-pushed the OCPBUGS-36674-re branch 4 times, most recently from 8bf1e92 to 8a5cf58 Compare August 12, 2025 09:01
@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 12, 2025
@openshift-ci openshift-ci bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Aug 12, 2025
@openshift-ci
Copy link

openshift-ci bot commented Aug 14, 2025

New changes are detected. LGTM label has been removed.

…gs of the primary network

OCPBUGS-57928: Added forbidden reserved interface names to various Nm
@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 14, 2025
@openshift-ci
Copy link

openshift-ci bot commented Aug 14, 2025

@dfitzmau: 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.

@dfitzmau dfitzmau merged commit 557f019 into openshift:main Aug 14, 2025
2 checks passed
@dfitzmau
Copy link
Contributor Author

/cherrypick enterprise-4.20

@dfitzmau
Copy link
Contributor Author

/cherrypick enterprise-4.19

@dfitzmau
Copy link
Contributor Author

/cherrypick enterprise-4.18

@dfitzmau
Copy link
Contributor Author

/cherrypick enterprise-4.17

@dfitzmau
Copy link
Contributor Author

/cherrypick enterprise-4.16

@dfitzmau
Copy link
Contributor Author

/cherrypick enterprise-4.15

@dfitzmau
Copy link
Contributor Author

/cherrypick enterprise-4.14

@openshift-cherrypick-robot

@dfitzmau: new pull request created: #97559

In response to this:

/cherrypick enterprise-4.20

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.

@openshift-cherrypick-robot

@dfitzmau: new pull request created: #97560

In response to this:

/cherrypick enterprise-4.19

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.

@openshift-cherrypick-robot

@dfitzmau: new pull request created: #97561

In response to this:

/cherrypick enterprise-4.18

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.

@openshift-cherrypick-robot

@dfitzmau: #97387 failed to apply on top of branch "enterprise-4.17":

Patch is empty.
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To record the empty patch as an empty commit, run "git am --allow-empty".
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"

In response to this:

/cherrypick enterprise-4.17

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.

@openshift-cherrypick-robot

@dfitzmau: #97387 failed to apply on top of branch "enterprise-4.16":

Applying: OCPBUGS-36674-re: Documented support limitations for day 2 OVNK configs of the primary network
.git/rebase-apply/patch:103: trailing whitespace.
OVN-Kubernetes runs a daemon on each node. There are daemon sets for the databases and for the OVN controller that run on every node. The OVN controller programs the Open vSwitch daemon on the nodes to support the following network provider features: 
.git/rebase-apply/patch:84: new blank line at EOF.
+
warning: 2 lines add whitespace errors.
Using index info to reconstruct a base tree...
M	modules/nw-ovn-kuberentes-limitations.adoc
M	modules/nw-ovn-kubernetes-features.adoc
M	networking/ovn_kubernetes_network_provider/about-ovn-kubernetes.adoc
Falling back to patching base and 3-way merge...
Auto-merging networking/ovn_kubernetes_network_provider/about-ovn-kubernetes.adoc
Auto-merging modules/nw-ovn-kubernetes-features.adoc
CONFLICT (content): Merge conflict in modules/nw-ovn-kubernetes-features.adoc
Auto-merging modules/nw-ovn-kuberentes-limitations.adoc
CONFLICT (content): Merge conflict in modules/nw-ovn-kuberentes-limitations.adoc
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0001 OCPBUGS-36674-re: Documented support limitations for day 2 OVNK configs of the primary network

In response to this:

/cherrypick enterprise-4.16

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.

@openshift-cherrypick-robot

@dfitzmau: #97387 failed to apply on top of branch "enterprise-4.15":

Applying: OCPBUGS-36674-re: Documented support limitations for day 2 OVNK configs of the primary network
.git/rebase-apply/patch:103: trailing whitespace.
OVN-Kubernetes runs a daemon on each node. There are daemon sets for the databases and for the OVN controller that run on every node. The OVN controller programs the Open vSwitch daemon on the nodes to support the following network provider features: 
.git/rebase-apply/patch:84: new blank line at EOF.
+
warning: 2 lines add whitespace errors.
Using index info to reconstruct a base tree...
M	modules/nw-ovn-kuberentes-limitations.adoc
M	modules/nw-ovn-kubernetes-features.adoc
M	networking/ovn_kubernetes_network_provider/about-ovn-kubernetes.adoc
Falling back to patching base and 3-way merge...
Auto-merging networking/ovn_kubernetes_network_provider/about-ovn-kubernetes.adoc
CONFLICT (content): Merge conflict in networking/ovn_kubernetes_network_provider/about-ovn-kubernetes.adoc
Auto-merging modules/nw-ovn-kubernetes-features.adoc
CONFLICT (content): Merge conflict in modules/nw-ovn-kubernetes-features.adoc
Auto-merging modules/nw-ovn-kuberentes-limitations.adoc
CONFLICT (content): Merge conflict in modules/nw-ovn-kuberentes-limitations.adoc
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0001 OCPBUGS-36674-re: Documented support limitations for day 2 OVNK configs of the primary network

In response to this:

/cherrypick enterprise-4.15

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.

@openshift-cherrypick-robot

@dfitzmau: #97387 failed to apply on top of branch "enterprise-4.14":

Applying: OCPBUGS-36674-re: Documented support limitations for day 2 OVNK configs of the primary network
.git/rebase-apply/patch:103: trailing whitespace.
OVN-Kubernetes runs a daemon on each node. There are daemon sets for the databases and for the OVN controller that run on every node. The OVN controller programs the Open vSwitch daemon on the nodes to support the following network provider features: 
.git/rebase-apply/patch:84: new blank line at EOF.
+
warning: 2 lines add whitespace errors.
Using index info to reconstruct a base tree...
M	modules/nw-ovn-kuberentes-limitations.adoc
M	modules/nw-ovn-kubernetes-features.adoc
M	networking/ovn_kubernetes_network_provider/about-ovn-kubernetes.adoc
Falling back to patching base and 3-way merge...
Auto-merging networking/ovn_kubernetes_network_provider/about-ovn-kubernetes.adoc
CONFLICT (content): Merge conflict in networking/ovn_kubernetes_network_provider/about-ovn-kubernetes.adoc
Auto-merging modules/nw-ovn-kubernetes-features.adoc
CONFLICT (content): Merge conflict in modules/nw-ovn-kubernetes-features.adoc
Auto-merging modules/nw-ovn-kuberentes-limitations.adoc
CONFLICT (content): Merge conflict in modules/nw-ovn-kuberentes-limitations.adoc
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0001 OCPBUGS-36674-re: Documented support limitations for day 2 OVNK configs of the primary network

In response to this:

/cherrypick enterprise-4.14

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.

@dfitzmau
Copy link
Contributor Author

/cherrypick enterprise-4.17

@openshift-cherrypick-robot

@dfitzmau: new pull request created: #97562

In response to this:

/cherrypick enterprise-4.17

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants