Project

General

Profile

Actions

action #32293

closed

action #30388: [qam] openqaworker10:4 - worker fail all tests - qemu instances are left behind

action #32263: [tools] Implement sub-reaper support in workers when starting isotovideo

[tools] Implement sub-reaper support in isotovideo

Added by EDiGiacinto about 6 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2018-02-26
Due date:
% Done:

0%

Estimated time:

Description

From http://e122.suse.de/tests/7790/file/autoinst-log.txt it can be seen that on sigterm, isotovideo is leaking childs, and the worker is reaping them.

To better handle this, we should do the same change in isotovideo and enable sub-reaping, and (try to) propagate the sigterm on the childs.


Checklist

  • Messages with: "Collected unknown process with pid 14306 and exit status: 0" are not shown anymore
  • Use rwp in a similar fashion as poo#32263 on os-autoinst

Related issues 1 (0 open1 closed)

Related to openQA Project - action #33037: dead children uncaught (again)ResolvedEDiGiacinto2018-03-10

Actions
Actions #1

Updated by szarate about 6 years ago

  • Checklist item changed from to [ ] Messages with: "Collected unknown process with pid 14306 and exit status: 0" are not shown anymore
  • Target version set to Ready

I think we have enough to pick this one soonish.

An approach similar to pr#1590 in openQA

Actions #2

Updated by szarate about 6 years ago

  • Checklist item changed from [ ] Messages with: "Collected unknown process with pid 14306 and exit status: 0" are not shown anymore to [ ] Messages with: "Collected unknown process with pid 14306 and exit status: 0" are not shown anymore, [ ] Use rwp in a similar fashion as poo#32263 on os-autoinst
Actions #3

Updated by coolo about 6 years ago

  • Target version changed from Ready to future

I don't care - for production it's perfectly fine if openQA worker cleans up

Actions #4

Updated by EDiGiacinto about 6 years ago

Actions #5

Updated by okurz almost 6 years ago

  • Target version changed from future to future
Actions #6

Updated by EDiGiacinto over 5 years ago

  • Checklist item changed from [ ] Messages with: "Collected unknown process with pid 14306 and exit status: 0" are not shown anymore, [ ] Use rwp in a similar fashion as poo#32263 on os-autoinst to [ ] Messages with: "Collected unknown process with pid 14306 and exit status: 0" are not shown anymore, [x] Use rwp in a similar fashion as poo#32263 on os-autoinst
Actions #7

Updated by okurz over 4 years ago

  • Category set to Feature requests
Actions #8

Updated by okurz almost 4 years ago

  • Checklist item changed from [ ] Messages with: "Collected unknown process with pid 14306 and exit status: 0" are not shown anymore, [x] Use rwp in a similar fashion as poo#32263 on os-autoinst to [x] Messages with: "Collected unknown process with pid 14306 and exit status: 0" are not shown anymore, [x] Use rwp in a similar fashion as poo#32263 on os-autoinst
  • Status changed from New to Resolved
  • Assignee set to okurz

The problem is gone. Also with https://github.com/os-autoinst/os-autoinst/pull/1277 and e.g. https://github.com/os-autoinst/os-autoinst/pull/1302 we have a cleaner and faster shutdown.

Actions

Also available in: Atom PDF