| opendevreview | Takashi Kajinami proposed openstack/oslo.messaging master: Remove scenario 03/04 tests from CI https://review.opendev.org/c/openstack/oslo.messaging/+/906334 | 04:01 |
|---|---|---|
| opendevreview | Merged openstack/oslo.messaging master: Drop unused function from scenario test script https://review.opendev.org/c/openstack/oslo.messaging/+/906274 | 05:14 |
| *** elodilles_pto is now known as elodilles | 06:54 | |
| -opendevstatus- NOTICE: all new logins to https://review.opendev.org are currently failing. investigation is ongoing, please be patient | 08:55 | |
| damani | hi | 09:23 |
| damani | i propose to abandon that one https://review.opendev.org/c/openstack/oslo.messaging/+/808550 | 09:23 |
| damani | what do you think? | 09:23 |
| opendevreview | Daniel Bengtsson proposed openstack/oslo.messaging master: Display the reply queue's name in timeout logs https://review.opendev.org/c/openstack/oslo.messaging/+/754049 | 09:46 |
| damani | i think we can merge it https://review.opendev.org/c/openstack/oslo.messaging/+/898309 | 09:48 |
| damani | what do you think? | 09:48 |
| damani | what we want to do with this backport https://review.opendev.org/c/openstack/oslo.messaging/+/875342 | 09:50 |
| damani | i think we can abandon it | 09:50 |
| opendevreview | Daniel Bengtsson proposed openstack/oslo.cache master: Add a new option to enforce the OpenSSL FIPS mode https://review.opendev.org/c/openstack/oslo.cache/+/797897 | 09:55 |
| tkajinam | o/ | 10:38 |
| tkajinam | I agree with abandoning 808550 and did it. I left a comment in 898309 . | 10:39 |
| tkajinam | I tend to abandon 875342 but if others think it's worth backporting then I'd not strongly block it | 10:39 |
| damani | tkajinam, i agree with you to abandon 875342 | 11:49 |
| damani | hberaud, what do you think? | 11:49 |
| damani | on about the requirements :) | 11:52 |
| damani | tkajinam, i think you will also disagree with this one https://review.opendev.org/c/openstack/oslo.messaging/+/836679 | 11:53 |
| tkajinam | damani, I agree with 836679. bump is actually needed to use the newer version of bandit which may have better checks | 11:53 |
| damani | tkajinam, can you take a look on that one please https://review.opendev.org/c/openstack/oslo.cache/+/797897 | 11:54 |
| damani | tkajinam, ok :) | 11:54 |
| damani | yes true | 11:54 |
| tkajinam | we have the upper constraint so we do need bump. and I totally agree with having that upper cap, to avoid mess caused in stable branches | 11:54 |
| tkajinam | the core problem is that bandit is excluded from u-c but I don't know the full history behind it so would be stick with the current way | 11:55 |
| damani | yes true | 11:55 |
| damani | hberaud, if you agree can you abandon or if you disagree can you put a comment please https://review.opendev.org/c/openstack/oslo.messaging/+/898309 | 11:56 |
| tkajinam | nudged 797897 | 11:59 |
| damani | tkajinam, i think we can merge it https://review.opendev.org/c/openstack/taskflow/+/816799 | 12:02 |
| opendevreview | Merged openstack/oslo.messaging master: Bump bandit https://review.opendev.org/c/openstack/oslo.messaging/+/836679 | 13:23 |
| opendevreview | Merged openstack/oslo.cache master: Add a new option to enforce the OpenSSL FIPS mode https://review.opendev.org/c/openstack/oslo.cache/+/797897 | 13:40 |
| opendevreview | Merged openstack/taskflow master: Avoid endless backtraces on StorageFailure https://review.opendev.org/c/openstack/taskflow/+/896232 | 13:49 |
| tkajinam | damani, I'll have a look tomorrow but am wondering if johnsom can check the reply to his old comment | 13:53 |
| damani | tkajinam, yes it will be great if he can check | 14:00 |
| damani | johnsom, can you take a look please | 14:00 |
| opendevreview | Merged openstack/taskflow master: Replace deprecated perl-style regex https://review.opendev.org/c/openstack/taskflow/+/906240 | 14:48 |
| *** sean-k-mooney1 is now known as sean-k-mooney | 16:13 | |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily for a restart, in order to attempt to restore OpenID login functionality | 19:37 | |
| -opendevstatus- NOTICE: OpenID logins for the Gerrit WebUI on review.opendev.org should be working normally again since the recent service restart | 20:03 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!