Skip to content

[18.0][FIX] checklog-odoo.cfg: ignore Killing chrome descendants-or-self warning#3507

Merged
pedrobaeza merged 1 commit intoOCA:18.0from
StefanRijnhart:18.0-checklog-odoo.cfg-ignore_warning_killing_chrome
Apr 13, 2026
Merged

[18.0][FIX] checklog-odoo.cfg: ignore Killing chrome descendants-or-self warning#3507
pedrobaeza merged 1 commit intoOCA:18.0from
StefanRijnhart:18.0-checklog-odoo.cfg-ignore_warning_killing_chrome

Conversation

@StefanRijnhart
Copy link
Copy Markdown
Member

Warning introduced in odoo/odoo@d22e0b4. The previous warning apparently did not occur, as it was not featured in checklog-odoo.cfg.

2026-04-13 03:20:41,702 367 WARNING odoo odoo.addons.web_disable_export_group.tests.test_tour.TestTour.test_admin: Killing chrome descendants-or-self of 382: 6 remaining
- chrome (zombie)
- chrome (zombie)
- chrome (zombie)
- chrome (zombie)
- chrome (zombie)
- chrome (zombie)

…rning

Warning introduced in odoo/odoo@d22e0b4. The previous warning apparently did not occur, as it was not featured in checklog-odoo.cfg.

```
2026-04-13 03:20:41,702 367 WARNING odoo odoo.addons.web_disable_export_group.tests.test_tour.TestTour.test_admin: Killing chrome descendants-or-self of 382: 6 remaining
- chrome (zombie)
- chrome (zombie)
- chrome (zombie)
- chrome (zombie)
- chrome (zombie)
- chrome (zombie)
```
@StefanRijnhart StefanRijnhart added this to the 18.0 milestone Apr 13, 2026
@pedrobaeza
Copy link
Copy Markdown
Member

Shouldn't this be added to the copier template, as it can happen in other repos?

@StefanRijnhart
Copy link
Copy Markdown
Member Author

@pedrobaeza There aren't that many repositories with javascript tests, so the change is only required on a small number of repos. Also, it sounds like a proper solution is still envisioned upstream (odoo/odoo#258062) so hopefully this is a temporary requirement.

@pedrobaeza
Copy link
Copy Markdown
Member

OK, I understand this will happen if you have JS tours, so that's more frequent than thought initially. Anyway, if it's a temporary one, let's see if this scales.

@pedrobaeza
Copy link
Copy Markdown
Member

We have another case here: OCA/rma#575

@pedrobaeza pedrobaeza merged commit 123101f into OCA:18.0 Apr 13, 2026
9 checks passed
@pedrobaeza
Copy link
Copy Markdown
Member

And another in OCA/mail: OCA/social#1851

@etobella
Copy link
Copy Markdown
Member

I will add it to the copier template. With Hoot, a lot of extra tests with OWL are appearing (Sign, account reconcile, automation, web...). We should be prepared.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants