| opendevreview | Merged openstack/oslo.log master: Imported Translations from Zanata https://review.opendev.org/c/openstack/oslo.log/+/912705 | 01:03 |
|---|---|---|
| opendevreview | Adam McArthur proposed openstack/oslo.service master: Removing oslo_wsgi eventlet dependency https://review.opendev.org/c/openstack/oslo.service/+/914927 | 01:24 |
| opendevreview | Xiang Wang proposed openstack/oslo.messaging master: Add HTTP driver to oslo.messaging.drivers https://review.opendev.org/c/openstack/oslo.messaging/+/914840 | 04:12 |
| opendevreview | Xiang Wang proposed openstack/oslo.messaging master: Add HTTP driver to oslo.messaging.drivers https://review.opendev.org/c/openstack/oslo.messaging/+/914840 | 05:59 |
| opendevreview | Merged openstack/oslo.messaging stable/2024.1: Fix incorrect desc of rabbit_stream_fanout option https://review.opendev.org/c/openstack/oslo.messaging/+/914583 | 08:46 |
| opendevreview | Xiang Wang proposed openstack/oslo.messaging master: Implement HTTPListener and HTTPClient and add OpenAPI specs https://review.opendev.org/c/openstack/oslo.messaging/+/915056 | 11:13 |
| opendevreview | Merged openstack/oslo.messaging stable/2023.1: kafka: Fix invalid hostaddr format for IPv6 address https://review.opendev.org/c/openstack/oslo.messaging/+/914450 | 11:13 |
| *** sfinucan is now known as stephenfin | 12:25 | |
| hberaud | elodilles: o/ would be good to have your thinking here https://review.opendev.org/c/openstack/oslo.log/+/914880 | 13:17 |
| elodilles | hberaud: looking | 13:22 |
| elodilles | nice :/ in the first place, deprecations should happen on master branch. furthermore, there were some changes around deprecation (of features) due to SLURP releases (something like: things should be deprecated in a SLURP release, and can only be removed on another SLURP release, or something like that)... given though that this is needed for py312 compatibility, it's quite hard situation | 13:26 |
| hberaud | yep | 13:33 |
| hberaud | that's why I requete | 13:33 |
| hberaud | requested your expertize$ | 13:33 |
| hberaud | esp | 13:33 |
| hberaud | expertise | 13:34 |
| * hberaud should boughts new fingers | 13:34 | |
| elodilles | :) | 13:36 |
| elodilles | i've commented on the patch, but further ideas are welcome | 13:37 |
| hberaud | thanks | 13:37 |
| elodilles | (i don't think deprecation warning should not be backported, but if the feature needs to be removed as that is 1) causing errors 2) not even functional, then just simply remove it maybe?) | 13:40 |
| elodilles | *i don't think deprecation warning should be backported | 13:40 |
| elodilles | so let's NOT backport it o:) | 13:40 |
| tkajinam | My plan was following what was done in nova for hyper-v removal. they did backport deprecation to stable/2023.1 and removed the feature in stable/2023.2 after creating a new 2023.1 release afair | 13:54 |
| tkajinam | Making it noop is another option and probably a bit simpler approach | 13:59 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.log master: Disable implementation for watch_log_file https://review.opendev.org/c/openstack/oslo.log/+/915068 | 14:10 |
| tkajinam | elodilles hberaud, I'll send an email to the list (replying to https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/AQ7HH5KNLMLY66NY5TXQST6EXSUYB6VM/ ) I thought to do so but was pulled by other stuffs during my day | 14:14 |
| hberaud | ack thanks | 14:16 |
| elodilles | tkajinam: thanks! | 14:35 |
| tkajinam | I wish I could have found it early... but I didn't notice the situation until I looked into py312 job which was added recently... | 14:40 |
| elodilles | we just started to test py312 as non-voting job, so i'm sure we'll see more similar issues in the coming months :/ | 14:51 |
| tkajinam | yeah | 14:52 |
| tkajinam | I spent some time during 2024.1 cycle to review items in upper constraints and global requirements. hope I could caught some of such problems early by that work. | 14:52 |
| elodilles | tkajinam: thanks for mentioning nova's case about deprecations. i see that there was a similar discussion in the comments ( https://review.opendev.org/c/openstack/nova/+/899159 ), so yes, there we agreed to backport the deprecation warning :S | 14:53 |
| elodilles | though it was to mark a *driver* to be deprecated and due to the new rule about SLURP releases | 14:54 |
| tkajinam | yeah | 14:54 |
| elodilles | (so that it was marked as deprecated in Antelope and could be removed in Caracal) | 14:55 |
| tkajinam | I think we can take a simpler approach for this case because we are talking about a single optional feature which has small impact | 14:55 |
| elodilles | here if i understand correctly we can't wait with the removal for E release anyway... :/ | 14:56 |
| elodilles | tkajinam: yepp, i agree ++ | 14:56 |
| tkajinam | I'll see if anyone else brings any better idea, but if not then I'll update the mocking patch I proposed so that we can move it forward | 15:02 |
| opendevreview | Merged openstack/oslo.messaging stable/2023.2: kafka: Fix invalid hostaddr format for IPv6 address https://review.opendev.org/c/openstack/oslo.messaging/+/914448 | 16:00 |
| tkajinam | hberaud, fyi. I found a new interesting eventlet problem https://bugs.launchpad.net/oslo.cache/+bug/2060215 | 16:17 |
| tkajinam | hberaud, this may probably need to be reported in eventlet but created one in oslo.cache first to quickly capture my findings... | 16:17 |
| tkajinam | hmm. ignore it. it seems the issue should be fixed in newer eventlet by https://github.com/eventlet/eventlet/commit/caf9f9983a43c537efff5c4c9ff1d543af6e1220 | 16:20 |
| tkajinam | ok it turned out eventlet in rdo is a bit old | 16:31 |
| amorin | hey team, is there any room/talk on PTG next week for olso topics? Like oslo.messaging? | 21:23 |
| opendevreview | Xiang Wang proposed openstack/oslo.messaging master: Implement HTTPListener and HTTPClient and add OpenAPI specs https://review.opendev.org/c/openstack/oslo.messaging/+/915056 | 23:08 |
| opendevreview | Xiang Wang proposed openstack/oslo.messaging master: Implement HTTPListener and HTTPClient and add OpenAPI specs https://review.opendev.org/c/openstack/oslo.messaging/+/915056 | 23:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!