Skip to content

[19.0] checklog: ignore chrome zombie teardown warning - #1577

Open
alan196 wants to merge 1 commit into
OCA:19.0from
Jarsa-dev:19.0-checklog-ignore-chrome-zombie
Open

[19.0] checklog: ignore chrome zombie teardown warning#1577
alan196 wants to merge 1 commit into
OCA:19.0from
Jarsa-dev:19.0-checklog-ignore-chrome-zombie

Conversation

@alan196

@alan196 alan196 commented Jun 27, 2026

Copy link
Copy Markdown

Problem

Every PR in this repo that runs an HttpCase tour (PoS / web frontend tests) currently fails CI even though the tests themselves pass (0 failed, 0 error(s)). The build is turned red by checklog-odoo, which flags this teardown warning:

WARNING ... Killing chrome descendants-or-self of N: M remaining
- chrome (zombie)

The headless Chrome used by the tour leaves defunct (zombie) child processes that Odoo force-kills at the end of the test. They are already dead; the warning is purely environmental (CI runner process reaping) and not a defect in any addon.

Effect

Any migration/feature PR that adds or touches a tour is blocked (e.g. #1576, #1570, #1574 — all green tests, red build on this exact warning), while PRs without tours pass.

Fix

Add an ignore pattern for this teardown warning to checklog-odoo.cfg, consistent with the existing 0 failed, 0 error(s) ignore entry.

HttpCase tour tests emit a teardown WARNING when the headless Chrome
leaves defunct (zombie) child processes that Odoo has to force-kill:

  WARNING ... Killing chrome descendants-or-self of N: M remaining
  - chrome (zombie)

These processes are already dead and the warning is purely environmental
(CI runner process reaping), but checklog-odoo flags it and turns every
PR that runs a PoS/web tour red. Ignore it.
@cifuentesmx

Copy link
Copy Markdown

@alan196 is this ready to merge?

@ByteMeAsap ByteMeAsap left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

@pedrobaeza

Copy link
Copy Markdown
Member

Please update copier template instead.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants