]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Run radvd as root
authorHenry Gessau <gessau@cisco.com>
Mon, 27 Apr 2015 13:59:21 +0000 (09:59 -0400)
committerHenry Gessau <gessau@cisco.com>
Tue, 28 Apr 2015 02:49:11 +0000 (02:49 +0000)
During the refactoring of external process management radvd lost
its root privileges.

Closes-bug: 1448813

Change-Id: I84883fe81684afafac9b024282a03f447c8f825a
(cherry picked from commit a5e54338770fc074e01fa88dbf909ee1af1b66b2)

neutron/agent/linux/ra.py
neutron/tests/unit/agent/l3/test_agent.py

index 7f800c269615c0f5f0d6842957a59d95865b7dce..d9eca8d6475c48bf2312c0a76501b21f720d4a07 100644 (file)
@@ -103,7 +103,8 @@ class DaemonMonitor(object):
             default_cmd_callback=callback,
             namespace=self._router_ns,
             service=RADVD_SERVICE_NAME,
-            conf=cfg.CONF)
+            conf=cfg.CONF,
+            run_as_root=True)
 
     def _spawn_radvd(self, radvd_conf):
         def callback(pid_file):
index 4c6682bd8aacec7c92f8f7b1dfe038862017f4e5..a5016f815f916908ab1a4e179b795600acd953eb 100644 (file)
@@ -1399,7 +1399,8 @@ class TestBasicRouterOperations(BasicRouterOperationsFramework):
                           service=process,
                           default_cmd_callback=mock.ANY,
                           namespace=ri.ns_name,
-                          conf=mock.ANY)]
+                          conf=mock.ANY,
+                          run_as_root=True)]
 
     def _process_router_ipv6_subnet_added(
             self, router, ipv6_subnet_modes=None):