| damani | hi | 09:27 |
|---|---|---|
| damani | stephenfin, i have updated this patch https://review.opendev.org/c/openstack/oslo.utils/+/816544 | 09:28 |
| damani | stephenfin, if you can take a look please it will be awesome :) | 09:28 |
| damani | hberaud, https://review.opendev.org/c/openstack/oslo.cache/+/822671 | 09:33 |
| damani | hberaud, looks good to merge :) | 09:33 |
| damani | hberaud, what is the status of https://review.opendev.org/c/openstack/oslo.cache/+/803716 ? | 09:34 |
| damani | hberaud, i added a comment here https://review.opendev.org/c/openstack/oslo.cache/+/803747 | 09:38 |
| damani | only a question | 09:38 |
| damani | hberaud, looks good to merge https://review.opendev.org/c/openstack/oslo.utils/+/814264 | 09:49 |
| damani | gred | 09:49 |
| hberaud | damani: done (https://review.opendev.org/c/openstack/oslo.cache/+/822671) | 09:49 |
| hberaud | Concerning 803716 and 803747 I'm waiting for reviews | 09:49 |
| damani | hberaud++ | 09:50 |
| damani | thanks a lot | 09:50 |
| damani | about 803747 i asked you something on the patch | 09:50 |
| hberaud | concerning 822671 I'm the author of the backport, so, I'm waiting for reviews from other core reviewers | 09:51 |
| damani | hberaud, you mean 814264 i think | 09:56 |
| damani | stephenfin, can you please check https://review.opendev.org/c/openstack/oslo.utils/+/814264 | 09:56 |
| hberaud | ah yes sorry | 09:56 |
| hberaud | 814264 | 09:56 |
| damani | hberaud, no problem :) | 09:57 |
| hberaud | tkajinam: do we want to backport https://review.opendev.org/c/openstack/oslo.utils/+/814264 further than xena? I'd argue that yes | 09:59 |
| damani | hberaud, thanks a lot for asking | 09:59 |
| hberaud | tkajinam: which version of OSP are impacted by this problem? | 10:01 |
| damani | hberaud, i think we can merge it https://review.opendev.org/c/openstack/oslo.context/+/742487 | 10:12 |
| damani | stephenfin, i think we can merge it https://review.opendev.org/c/openstack/oslo.context/+/815938 | 10:13 |
| opendevreview | Merged openstack/oslo.cache stable/wallaby: Don't import module "etcd3gw" if etcd3gw backend is not used. https://review.opendev.org/c/openstack/oslo.cache/+/822671 | 11:19 |
| opendevreview | Balazs Gibizer proposed openstack/oslo.messaging master: [rabbit] use retry parameters during notification sending https://review.opendev.org/c/openstack/oslo.messaging/+/819142 | 11:23 |
| opendevreview | Thomas Goirand proposed openstack/oslo.config master: WIP: Add a new max_sample_default https://review.opendev.org/c/openstack/oslo.config/+/823710 | 12:29 |
| opendevreview | Thomas Goirand proposed openstack/oslo.config master: WIP: Add a new max_sample_default https://review.opendev.org/c/openstack/oslo.config/+/823710 | 12:33 |
| opendevreview | Merged openstack/oslo.utils master: Fix regex used to mask password https://review.opendev.org/c/openstack/oslo.utils/+/816544 | 12:45 |
| opendevreview | Merged openstack/oslo.utils stable/xena: QemuImgInfo: Fix inconsistent value format of encrypted https://review.opendev.org/c/openstack/oslo.utils/+/814264 | 12:55 |
| damani | stephenfin, thanks a lot for the code review | 12:58 |
| damani | hberaud, https://review.opendev.org/c/openstack/oslo.utils/+/816544 | 12:59 |
| damani | need to be backport ? | 12:59 |
| tkajinam | hberaud, we don't have actual impact caused by that issue. I found that issue while implementing https://review.opendev.org/c/openstack/cinder/+/799663 in cinder, to replace deprecated human format. | 13:06 |
| hberaud | tkajinam: ack. Thanks for your confirmation | 13:07 |
| hberaud | damani: FYI ^ | 13:07 |
| damani | tkajinam, hberaud, thanks a lot | 13:08 |
| opendevreview | Merged openstack/oslo.context master: Remove the deprecated argument tenant from RequestContext https://review.opendev.org/c/openstack/oslo.context/+/815938 | 13:23 |
| opendevreview | Merged openstack/oslo.messaging master: [rabbit] use retry parameters during notification sending https://review.opendev.org/c/openstack/oslo.messaging/+/819142 | 15:29 |
| opendevreview | Thomas Goirand proposed openstack/oslo.config master: WIP: Add a new max_sample_default https://review.opendev.org/c/openstack/oslo.config/+/823710 | 22:40 |
| opendevreview | Thomas Goirand proposed openstack/oslo.config master: WIP: Add a new max_sample_default https://review.opendev.org/c/openstack/oslo.config/+/823710 | 22:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!