| zigo | amorin: I think I've finally found my "unit test stuck" mistery !!! :) | 08:29 |
|---|---|---|
| zigo | I'll open a launchpad bug + patch soon. | 08:29 |
| zigo | Basically, in mistral/engine/post_tx_queue.py, we need to: | 08:31 |
| zigo | - t = threading.Thread(target=_within_new_thread) | 08:31 |
| zigo | + t = threading.Thread(target=_within_new_thread, daemon=True) | 08:31 |
| zigo | Otherwise, the thread cannot be shutdown. | 08:32 |
| amorin | nice! | 08:36 |
| zigo | Oh, just found another issue ... :) | 09:07 |
| zigo | (leaking tests) | 09:07 |
| opendevreview | Thomas Goirand proposed openstack/mistral master: Make the post-transaction engine thread a daemon https://review.opendev.org/c/openstack/mistral/+/1010620 | 09:11 |
| opendevreview | Thomas Goirand proposed openstack/mistral master: Fix leakage of auth_enable state in unit tests https://review.opendev.org/c/openstack/mistral/+/1010627 | 09:41 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!