| opendevreview | Takashi Kajinami proposed openstack/oslo.service master: Eliminate usage of oslo_config_glue https://review.opendev.org/c/openstack/oslo.service/+/975357 | 01:04 |
|---|---|---|
| *** mhen_ is now known as mhen | 02:32 | |
| opendevreview | Merged openstack/oslo.service master: Refactor threading launchers to prefer spawn-safe multiprocessing https://review.opendev.org/c/openstack/oslo.service/+/965208 | 03:16 |
| opendevreview | Ghanshyam proposed openstack/oslo.service master: Fix the followup comment from signalhander change https://review.opendev.org/c/openstack/oslo.service/+/975486 | 03:24 |
| gmaan | tkajinam: thanks for the review and approval, I added the signal handler releasenote in followup ^^ | 03:38 |
| tkajinam | gmaan, I wonder if we want to classify it as a bugfix, because this is an unexpected feature gap between threading version and eventlet version | 05:44 |
| tkajinam | (the main point is whether anyone aims to backport this) | 05:44 |
| opendevreview | Takashi Kajinami proposed openstack/castellan master: barbican: Drop redundant full url composition https://review.opendev.org/c/openstack/castellan/+/975497 | 10:22 |
| opendevreview | Takashi Kajinami proposed openstack/castellan master: barbican: Drop redundant full url composition https://review.opendev.org/c/openstack/castellan/+/975497 | 11:21 |
| opendevreview | Takashi Kajinami proposed openstack/castellan master: barbican: Drop redundant full url composition https://review.opendev.org/c/openstack/castellan/+/975497 | 11:59 |
| opendevreview | Takashi Kajinami proposed openstack/castellan master: barbican: Drop redundant full url composition https://review.opendev.org/c/openstack/castellan/+/975497 | 12:00 |
| amoralej | Hello, I'd appreciate reviews on https://review.opendev.org/c/openstack/taskflow/+/974999 which fixes an issue we are hitting in watcher while using TaskFlow lib, thanks! | 13:31 |
| opendevreview | Balazs Gibizer proposed openstack/futurist master: Add DelayedExecutorMixin variants https://review.opendev.org/c/openstack/futurist/+/975508 | 13:38 |
| opendevreview | Gregory Thiemonge proposed openstack/taskflow master: Avoid iterating over the same atoms in a graph https://review.opendev.org/c/openstack/taskflow/+/974999 | 14:53 |
| opendevreview | Merged openstack/oslo.service master: Add signal handler for ProcessLauncher with no_fork https://review.opendev.org/c/openstack/oslo.service/+/974024 | 16:19 |
| opendevreview | Merged openstack/oslo.service master: Fix the followup comment from signalhander change https://review.opendev.org/c/openstack/oslo.service/+/975486 | 16:19 |
| opendevreview | Merged openstack/oslo.utils master: Remove tzdata from requirements https://review.opendev.org/c/openstack/oslo.utils/+/975021 | 17:36 |
| gmaan | tkajinam: yeah, i am not sure we need backport it unless any project want it for threading mode in previous releases but yes it is backportable. I did not know if threading gaps are fixed as bug but do not disagree on that. | 18:15 |
| gmaan | thanks all for review though | 18:16 |
| opendevreview | Ghanshyam proposed openstack/oslo.service master: Add a set method to override the default value for service opts https://review.opendev.org/c/openstack/oslo.service/+/975574 | 20:47 |
| johnsom | Hi olso neighbors. I'm helping with the Designate migration from eventlet to threading. Can someone point me to the mechanism that calls thread_done with the thread backend? I'm not seeing it fire. Ref: https://opendev.org/openstack/oslo.service/src/branch/master/oslo_service/backend/_threading/threadgroup.py#L88 | 21:45 |
| johnsom | amoralej I have looked at that taskflow patch from Greg. I have asked him some questions about it (testing and deciders). Once I hear back I will comment on the patch or workflow it. | 21:47 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!