Bug 1823560: backport fixes to support upstream->downstream merges #590
Conversation
Completes the switch from ovn2.11 to ovn2.12 by removing support for the now obsolete ovn2.11 SDN-643 https://issues.redhat.com/browse/SDN-643 Signed-off-by: Phil Cameron <pcameron@redhat.com> (cherry picked from commit f479a53)
For OVN NB DB interaction with the ovn-nbctl command, we use ovn-nbctl daemon mode, where we do not need to use SSL arguments to directly interact iwth the nbctl socket in the pod. However, SSL arguments are needed for non-nbctl commands like ovsdb client. This patch adds those arguments. In addition, the nbctl-daemon-mode argument was missing as well. This argument indicates to ovn-kubernetes that it needs to use nbctl daemon mode. Even though we were previously not setting this, we were exporting the proper bash env var to allow the process to automatically use daemon mode. However, we should pass the correct arguments to be consistent. Due to an OVN bug, we need to temporarily symlink the ovn-nbctl control socket to /var/run/ovn as it is accidentally placed in /var/run/openvswitch today. Signed-off-by: Tim Rozet <trozet@redhat.com> (cherry picked from commit 3a85ce8)
openshift-ci-robot
added
the
bugzilla/invalid-bug
|
@dcbw: This pull request references Bugzilla bug 1823560, which is invalid:
Comment In response to this:
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/test-infra repository. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dcbw The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci-robot
added
the
approved
|
/test e2e-aws-ovn |
|
/test e2e-gcp-ovn |
|
No longer required for 4.3 |
4.3 version of #589
Backports of #466, #467, and #572 . These are needed to make nbctl daemon mode work while at the same time being able to check local OVS database to figure out if SCTP is supported by OVN.
@trozet @knobunc @fepan @openshift/networking