Project

General

Profile

Actions
Copy link

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 over 6 years ago. Updated over 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
okurz
Category:
Feature requests
Target version:
QA - future
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.

Hide closed

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 open — 1 closed)

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

Actions
Actions
Copy link
 #1

Updated by szarate over 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
Copy link
 #2

Updated by szarate over 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
Copy link
 #3

Updated by coolo over 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
Copy link
 #4

Updated by EDiGiacinto over 6 years ago

Actions
Copy link
 #5

Updated by okurz about 6 years ago

  • Target version changed from future to future
Actions
Copy link
 #6

Updated by EDiGiacinto about 6 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
Copy link
 #7

Updated by okurz about 5 years ago

  • Category set to Feature requests
Actions
Copy link
 #8

Updated by okurz over 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
Copy link

Also available in: Atom PDF