openstack/neutron
Code Issues Proposed changes

Properly clean up ovn-northd in functional tests

Browse Source 
ovn-northd.ctl is a socket, so os.path.isfile() returns false and
ovn-northd is not properly killed. Use os.path.exists() instead.

Closes-Bug: #1952023
Change-Id: I00fba2dc4395c0a8cd4631d4c2c71d9c3dc429e9
This commit is contained in:
Terry Wilson
2021-11-23 21:21:27 -06:00
parent 4b1bf15f7b
commit 74aa86a976
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
neutron/tests/functional/resources/process.py
View File

@@ -82,7 +82,7 @@ class OvnNorthd(DaemonProcessFixture):
def stop(self):
try:
if os.path.isfile(self.unixctl_path):
if os.path.exists(self.unixctl_path):
stop_cmd = ['ovs-appctl', '-t', self.unixctl_path, 'exit']
utils.execute(stop_cmd)
except Exception: