From 9779a91a2eb0ae7745552d30af1accbfe7a70ca0 Mon Sep 17 00:00:00 2001 From: Tomi Juvonen Date: Wed, 28 Feb 2018 12:05:26 +0200 Subject: Removing temporary process group solution. Introducing root cause fix for sample inspector shutdown JIRA: DOCTOR-123 Change-Id: Ibc95d482f476b60584c224d8e2757d071149d611 Signed-off-by: Tomi Juvonen --- doctor_tests/inspector/sample.py | 2 +- doctor_tests/main.py | 7 ------- 2 files changed, 1 insertion(+), 8 deletions(-) (limited to 'doctor_tests') diff --git a/doctor_tests/inspector/sample.py b/doctor_tests/inspector/sample.py index fadfd3d4..abc4fc6d 100644 --- a/doctor_tests/inspector/sample.py +++ b/doctor_tests/inspector/sample.py @@ -168,7 +168,7 @@ class InspectorApp(Thread): self.inspector.handle_events(events) return "OK" - @app.route('/shutdown', methods=['POST']) + @app.route('/events/shutdown', methods=['POST']) def shutdown(): self.log.info('shutdown inspector app server at %s' % time.time()) func = request.environ.get('werkzeug.server.shutdown') diff --git a/doctor_tests/main.py b/doctor_tests/main.py index 8051ad6e..3f54fc32 100644 --- a/doctor_tests/main.py +++ b/doctor_tests/main.py @@ -9,7 +9,6 @@ import os from os.path import isfile, join import random -import signal import sys import time @@ -258,16 +257,10 @@ class DoctorTest(object): self.installer.cleanup() self.image.delete() self.user.delete() - # Kill possible hanging subprocess - os.killpg(0, signal.SIGKILL) def main(): """doctor main""" - # TODO (tojuvone): JIRA DOCTOR-123: Test cases have some issue to always - # kill all subprocesses. To ensure they are killed this group is done so - # all processes can be killed without knowing what they are. - os.setpgrp() test_dir = os.path.split(os.path.realpath(__file__))[0] doctor_root_dir = os.path.dirname(test_dir) -- cgit 1.2.3-korg