Yes, the current code (since 2015) states: return ip_wrapper.netns.execute( [ns_wrapper, '--mount_paths=/etc:%s/etc,%s:%s/var/run' % ( self.config_dir, self._strongswan_piddir, self.config_dir), ('--rootwrap_config=%s' % self._rootwrap_cfg if self._rootwrap_cfg else ''), '--cmd=%s' % ','.join(cmd)], --
But it should be /run not /var/run but due to legacy we can't say if /var/run should be removed...
Would something like this work: --- a/neutron_vpnaas/services/vpn/device_drivers/strongswan_ipsec.py +++ b/neutron_vpnaas/services/vpn/device_drivers/strongswan_ipsec.py @@ -113,8 +113,8 @@ class StrongSwanProcess(ipsec.BaseSwanProcess): ns_wrapper = self.get_ns_wrapper() return ip_wrapper.netns.execute( [ns_wrapper, - '--mount_paths=/etc:%s/etc,%s:%s/var/run' % ( - self.config_dir, self._strongswan_piddir, self.config_dir), + '--mount_paths=/etc:%s/etc,%s:%s/var/run,%s:%s/run' % ( + self.config_dir, self._strongswan_piddir, self.config_dir, self._strongswan_piddir, self.config_dir), ('--rootwrap_config=%s' % self._rootwrap_cfg if self._rootwrap_cfg else ''), '--cmd=%s' % ','.join(cmd)],
---
Completely untested.
Yes, the current code (since 2015) states: netns.execute(
[ns_ wrapper,
' --mount_ paths=/ etc:%s/ etc,%s: %s/var/ run' % (
self. config_ dir, self._strongswa n_piddir, self.config_dir),
( '--rootwrap_ config= %s' % self._rootwrap_cfg
' --cmd=% s' % ','.join(cmd)],
return ip_wrapper.
if self._rootwrap_cfg else ''),
--
But it should be /run not /var/run but due to legacy we can't say if /var/run should be removed...
Would something like this work: vpnaas/ services/ vpn/device_ drivers/ strongswan_ ipsec.py vpnaas/ services/ vpn/device_ drivers/ strongswan_ ipsec.py ss(ipsec. BaseSwanProcess ): ns_wrapper( ) netns.execute(
[ ns_wrapper, paths=/ etc:%s/ etc,%s: %s/var/ run' % ( n_piddir, self.config_dir), paths=/ etc:%s/ etc,%s: %s/var/ run,%s: %s/run' % ( n_piddir, self.config_dir, self._strongswa n_piddir, self.config_dir),
('--rootwrap_ config= %s' % self._rootwrap_cfg
if self._rootwrap_cfg else ''),
'--cmd= %s' % ','.join(cmd)],
--- a/neutron_
+++ b/neutron_
@@ -113,8 +113,8 @@ class StrongSwanProce
ns_wrapper = self.get_
return ip_wrapper.
- '--mount_
- self.config_dir, self._strongswa
+ '--mount_
+ self.config_dir, self._strongswa
---
Completely untested.