vexxhost/openstack-tools
Code Issues Proposed changes

Close NetNS for cleanup

Browse Source 
Change-Id: I89ecefb686cd6e7612b852809ccccb876379a5b8
This commit is contained in:
Mohammed Naser 2020-09-07 09:20:58 -04:00
parent 00453b2fe4
commit 5b90bec7de
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
openstack_tools/cmd/prometheus/routers_l3_ha.py
View File

@ -41,6 +41,8 @@ def get_pid_files(state_path):
def verify_router_configured(router_id, master): def verify_router_configured(router_id, master):
"""Verify is the router is properly configured on the system.""" """Verify is the router is properly configured on the system."""
configured = 1
try: try:
namespace = NetNS('qrouter-%s' % router_id) namespace = NetNS('qrouter-%s' % router_id)
except OSError: except OSError:
@ -55,13 +57,15 @@ def verify_router_configured(router_id, master):
# Router is not master but addresses are configured. # Router is not master but addresses are configured.
if master == 0 and len(addr) != 0: if master == 0 and len(addr) != 0:
return 0 configured = 0
# Router is master but addresses are not configured. # Router is master but addresses are not configured.
if master == 1 and len(addr) == 0: if master == 1 and len(addr) == 0:
return 0 configured = 0
return 1 namespace.close()
return configured
def main(): def main():