]> code.ossystems Code Review - openembedded-core.git/commitdiff
selftest/context: Avoid tracebacks from tests using multiprocessing
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Wed, 19 Feb 2020 17:27:26 +0000 (17:27 +0000)
committerAnuj Mittal <anuj.mittal@intel.com>
Tue, 28 Jul 2020 08:15:51 +0000 (16:15 +0800)
We can see tracebacks where the SIGTERM handler catches things
it shouldn't. Avoid exit(1) unless we're the process that
it was intended for.

[YOCTO #13664]

(From OE-Core rev: d9c62ffac611310efd47ed6397d31dccb72fe868)

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit dba8c1d5ef0b574b7772d59e5992bfad8b7cca13)
Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
meta/lib/oeqa/selftest/context.py

index c4eb5d614eba8281b29d68eeb520b92a2c0f58ea..3d3b19c6e80205b9260a86fd09d85dafe45d569d 100644 (file)
@@ -280,11 +280,15 @@ class OESelftestTestContextExecutor(OETestContextExecutor):
         return rc
 
     def _signal_clean_handler(self, signum, frame):
-        sys.exit(1)
+        if self.ourpid == os.getpid():
+            sys.exit(1)
     
     def run(self, logger, args):
         self._process_args(logger, args)
 
+        # Setup a SIGTERM handler to allow restoration of files like local.conf and bblayers.conf
+        # but don't interfer with other processes
+        self.ourpid = os.getpid()
         signal.signal(signal.SIGTERM, self._signal_clean_handler)
 
         rc = None