Yes, thank you Ben.

On 2/28/20 2:22 PM, Ben Pfaff wrote:
Does this need a backport to OVS 2.12?

On Thu, Feb 27, 2020 at 04:27:33PM -0500, Mark Michelson wrote:
I've merged the change to master and branch-20.03.

On 2/27/20 2:37 PM, Numan Siddique wrote:
On Thu, Feb 27, 2020 at 4:04 PM Lorenzo Bianconi
<[email protected]> wrote:

ovn-controller is currently running as non-root so it is not allowed to
configure system networking breaking ovn QoS support. Fix the issue
granting CAP_NET_ADMIN capability to ovn-controller process

Tested-by: Ying Xu <[email protected]>
Signed-off-by: Lorenzo Bianconi <[email protected]>

Acked-by: Numan Siddique <[email protected]>

Thanks
Numan

---
   controller/ovn-controller.c | 2 +-
   1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/controller/ovn-controller.c b/controller/ovn-controller.c
index cacaaa578..53d80806f 100644
--- a/controller/ovn-controller.c
+++ b/controller/ovn-controller.c
@@ -1733,7 +1733,7 @@ main(int argc, char *argv[])
       char *ovs_remote = parse_options(argc, argv);
       fatal_ignore_sigpipe();

-    daemonize_start(false);
+    daemonize_start(true);

       char *abs_unixctl_path = get_abs_unix_ctl_path();
       retval = unixctl_server_create(abs_unixctl_path, &unixctl);
--
2.24.1

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to