osmith submitted this change.

View Change



1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Approvals: fixeria: Looks good to me, approved Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve
testenv: stop daemons in reverse order

The PyHSS daemons throw a lot of errors when the redis server is stopped
before the PyHSS daemons are stopped. Fix this by stopping the daemons
in the reverse order they were started.

Example:

start:
1) redis
2) pyhss_hss
3) pyhss_diameter
4) pyhss_api

stop:
1) pyhss_api
2) pyhss_diameter
3) pyhss_hss
4) redis

Change-Id: Ia4fa25998d81a9a3a27c5f3d341b9a02e5536e17
---
M _testenv/testenv/daemons.py
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/_testenv/testenv/daemons.py b/_testenv/testenv/daemons.py
index f57b4b4..2abe8c8 100644
--- a/_testenv/testenv/daemons.py
+++ b/_testenv/testenv/daemons.py
@@ -140,7 +140,7 @@

run_shell_on_stop = False

- for daemon in daemons:
+ for daemon in reversed(daemons):
pid = daemons[daemon].pid
logging.info(f"Stopping {daemon} ({pid})")
kill(pid)

To view, visit change 41421. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ia4fa25998d81a9a3a27c5f3d341b9a02e5536e17
Gerrit-Change-Number: 41421
Gerrit-PatchSet: 3
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>