Merge "Don't use root to list namespaces"

diff --cc neutron/agent/linux/ip_lib.py
index a140be290e8bee3f7482075710edaabd6608092f,80302aea2d19463ff81230667f9b1fe96bf304b9..62ba2a7bb2d6ef8c728f25738d6b4e3d3f25d74e
--- 1/neutron/agent/linux/ip_lib.py
--- 2/neutron/agent/linux/ip_lib.py
+++ b/neutron/agent/linux/ip_lib.py
@@@ -452,21 -452,21 +452,21 @@@ class IpNetnsCommand(IpCommandBase)
      def execute(self, cmds, addl_env={}, check_exit_code=True):
          if not self._parent.root_helper:
              raise exceptions.SudoRequired()
 -        elif not self._parent.namespace:
 -            raise Exception(_('No namespace defined for parent'))
 -        else:
 -            env_params = []
 -            if addl_env:
 -                env_params = (['env'] +
 -                              ['%s=%s' % pair for pair in addl_env.items()])
 -            return utils.execute(
 -                ['ip', 'netns', 'exec', self._parent.namespace] +
 -                env_params + list(cmds),
 -                root_helper=self._parent.root_helper,
 -                check_exit_code=check_exit_code)
 +        ns_params = []
 +        if self._parent.namespace:
 +            ns_params = ['ip', 'netns', 'exec', self._parent.namespace]
 +
 +        env_params = []
 +        if addl_env:
 +            env_params = (['env'] +
 +                          ['%s=%s' % pair for pair in addl_env.items()])
 +        return utils.execute(
 +            ns_params + env_params + list(cmds),
 +            root_helper=self._parent.root_helper,
 +            check_exit_code=check_exit_code)
  
      def exists(self, name):
-         output = self._as_root('list', options='o', use_root_namespace=True)
+         output = self._run('list', options='o')
  
          for line in output.split('\n'):
              if name == line.strip():