| *** ralonsoh_ is now known as ralonsoh | 08:08 | |
| priteau | Hello. If someone has privileges on oslo.utils Launchpad, could they please mark cloudkitty as "Fix Released" here: https://bugs.launchpad.net/oslo.utils/+bug/1715325 | 08:33 |
|---|---|---|
| hberaud | done by Takashi | 09:00 |
| tkajinam | priteau, we can't. that's an unfortunate sideeffect of transition to storyboard | 11:13 |
| tkajinam | priteau, I've seen similar situations with heat, but once you disable bug management in launchpad, then you lose control about all existing bugs | 11:14 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.cache master: Make documentation and error about tls_enabled more friendly https://review.opendev.org/c/openstack/oslo.cache/+/908770 | 11:29 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.cache master: config: Document backends supporting specific options https://review.opendev.org/c/openstack/oslo.cache/+/908770 | 11:35 |
| damani | hi | 11:39 |
| tkajinam | tobias-urdin, hi. do you have a min to approve https://github.com/jd/pifpaf/pull/169 again to trigger CI ? | 12:16 |
| tobias-urdin | tkajinam: ack | 14:16 |
| tobias-urdin | sorry for delay, done | 14:17 |
| tkajinam | tobias-urdin, np, and thanks ! | 14:21 |
| *** jph3 is now known as jph | 15:06 | |
| opendevreview | Merged openstack/oslo.cache master: Fail if tls_enabled is True but backend does not support it https://review.opendev.org/c/openstack/oslo.cache/+/907330 | 15:24 |
| opendevreview | Merged openstack/oslo.cache master: Automate TLS certificates settings for redis backend https://review.opendev.org/c/openstack/oslo.cache/+/907334 | 15:24 |
| hberaud | tkajinam: o/ I reviewed all your redis patches. I wanted to let know that all these patches LGTM, I +2ed them, however, I made some suggestions, this is why I didn't +W'ed, just to let you see my suggestions. | 15:31 |
| hberaud | Feel free or not to address them. | 15:31 |
| hberaud | Else, I think we are ready to merge them | 15:32 |
| tkajinam | hberaud, thanks ! | 15:33 |
| hberaud | Would be awesome to see them merged ASAP before FF | 15:33 |
| tkajinam | I'll prepare follow-up change to update parameter descriptions and warning message to make these more friendly | 15:33 |
| hberaud | no problem, thanks | 15:34 |
| hberaud | those are just suggestions | 15:34 |
| tkajinam | I wonder if we can also merge the patches I submitted to tooz and taskflow, as well. https://etherpad.opendev.org/p/oslo-redis-sentinel-caracal | 15:35 |
| tkajinam | the patches for tooz already have one +2. the series of taskflow patches do not yet review scores, though | 15:35 |
| hberaud | let me check | 15:36 |
| tkajinam | these are based on my learning during digging into dogpile and redis-py to complete that oslo.cache changes | 15:36 |
| tkajinam | thanks ! | 15:36 |
| tkajinam | unfortunately it's a bit tricky to set up jobs with ssl using pifpaf, but I'll set up some jobs in puppet once these are merged | 15:37 |
| tkajinam | so we may be able to catch any problems during this cycle, hopefully. | 15:37 |
| hberaud | nice | 15:37 |
| opendevreview | Arnaud Morin proposed openstack/oslo.messaging master: Restore read stream queues from last known offset https://review.opendev.org/c/openstack/oslo.messaging/+/908840 | 16:56 |
| opendevreview | Merged openstack/tooz master: Fix handling of timeout and blocking https://review.opendev.org/c/openstack/tooz/+/904260 | 17:57 |
| opendevreview | Merged openstack/tooz master: redis: Add username https://review.opendev.org/c/openstack/tooz/+/907656 | 20:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!