| itamarst | looks like nova is still having issues | 12:21 |
|---|---|---|
| itamarst | so I guess I'll look at the diff | 12:21 |
| frickler | note that the currently pinned version is 0.36.1, my hypothesis is that https://github.com/eventlet/eventlet/pull/975 is triggering some race conditions | 13:05 |
| hberaud[m] | #startmeeting openstack-eventlet-removal | 16:00 |
| opendevmeet | Meeting started Tue Jan 7 16:00:14 2025 UTC and is due to finish in 60 minutes. The chair is hberaud[m]. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| opendevmeet | The meeting name has been set to 'openstack_eventlet_removal' | 16:00 |
| hberaud[m] | Ping list: amorin, JayF, gouthamr, ralonsoh | 16:00 |
| hberaud[m] | #link https://etherpad.opendev.org/p/epoxy-eventlet-tracking | 16:00 |
| amorin | o/ | 16:01 |
| hberaud[m] | Sorry I'm late I made a mistake with timezones | 16:01 |
| amorin | no problem | 16:02 |
| hberaud[m] | #topic deprecation reminder | 16:02 |
| hberaud[m] | we are now at milestone 2, so, if not yet done, please prioritize your deprecations | 16:02 |
| hberaud[m] | #topic Review task completion | 16:03 |
| hberaud[m] | there is no task from previous weeks, so let's move on | 16:03 |
| hberaud[m] | and no updates from working groups | 16:04 |
| JayF | o/ | 16:04 |
| hberaud[m] | I propose to speak a bit the status of mistral | 16:04 |
| hberaud[m] | #topic mistral | 16:04 |
| amorin | yup, good news here, I was able to fix some tests / code that was failing with the last eventlet release | 16:05 |
| hberaud[m] | I seen that almost all the mistral patches are now merged | 16:05 |
| amorin | it's all green now | 16:05 |
| hberaud[m] | good job amorin | 16:05 |
| amorin | deprecation notice is also done | 16:05 |
| hberaud[m] | excellent | 16:05 |
| amorin | I was trying to totally get rid of eventlet, but that not yet possible until oslo* get rid of it | 16:06 |
| amorin | (oslo.service IIRC) | 16:06 |
| hberaud[m] | that was my coming question | 16:06 |
| amorin | https://review.opendev.org/c/openstack/mistral/+/938155 | 16:07 |
| amorin | #link https://review.opendev.org/c/openstack/mistral/+/938155 | 16:07 |
| amorin | it's here | 16:07 |
| hberaud[m] | If I correctly understand the status of mistral, we still have some parts that depends on eventlet, right? | 16:07 |
| amorin | true | 16:07 |
| amorin | I can maybe try to get rid of it partially now | 16:07 |
| amorin | what I wanted to achieve first was the compat with the last eventlet release to avoid the CI to be blocked by the bump | 16:08 |
| amorin | my next plan is to split this WIP change into multiple patchset, and try to limit the usage of eventlet to strict minimum | 16:08 |
| hberaud[m] | excellent | 16:09 |
| hberaud[m] | concerning oslo.service, Daniel just had a new baby, so I think he won't be available before 1 or 2 weeks | 16:10 |
| amorin | nice :) | 16:11 |
| hberaud[m] | and for now he volunteered for implementing the refactor of oslo.service | 16:11 |
| hberaud[m] | so I won't expect much activity than that on this side | 16:11 |
| hberaud[m] | at least for the coming weeks | 16:11 |
| damani[m] | no | 16:12 |
| damani[m] | i will finish the work about oslo.service | 16:12 |
| damani[m] | i will work this week | 16:12 |
| hberaud[m] | Oh Daniel, welcome | 16:12 |
| hberaud[m] | oh ok, good news | 16:12 |
| damani[m] | and then i will take my parental leave | 16:12 |
| hberaud[m] | I was thinking you will be AFK for one or two weeks | 16:13 |
| hberaud[m] | then, that's excellent | 16:13 |
| damani[m] | i want to merge first | 16:13 |
| hberaud[m] | damani: takashi sent you a couple of comments | 16:14 |
| damani[m] | i know | 16:14 |
| hberaud[m] | at least on https://review.opendev.org/c/openstack/oslo.service/+/937010 | 16:14 |
| damani[m] | i will check tomorrow | 16:14 |
| damani[m] | yes | 16:14 |
| hberaud[m] | damani: do not hesitate to ping Mike too | 16:15 |
| hberaud[m] | damani: Congrats for your baby | 16:15 |
| hberaud[m] | #topic requirements bump | 16:16 |
| hberaud[m] | Concerning the requirements bump https://review.opendev.org/c/openstack/requirements/+/933257, as I said yesterday, I tried to rerun the failing job, but the nova error seems legit | 16:17 |
| hberaud[m] | Itamar & Jens had a tiny discussion about that early today | 16:18 |
| hberaud[m] | Either nova need more adjustments or eventlet need a fix, I don't yet know | 16:18 |
| amorin | frickler had concerned about this commit: https://github.com/eventlet/eventlet/pull/975 | 16:19 |
| damani[m] | hberaud, thanks a lot | 16:19 |
| amorin | I endup on exactly the same when doing some debug on mistral in the past | 16:19 |
| hberaud[m] | with which version of eventlet? | 16:20 |
| hberaud[m] | the one officially in our upper-constraints or a local preview of the next version? | 16:20 |
| hberaud[m] | because if you faced the same issue with 0.36.1 https://opendev.org/openstack/requirements/src/branch/master/upper-constraints.txt#L162 that mean that this commit is not the root cause, because it was released with 0.37.0 which is not yet into our stack | 16:22 |
| hberaud[m] | when you say "I endup on exactly the same when doing some debug " you tell about the error described on the github issue you linked, or about the nova error we recently seen in the cross-job? | 16:24 |
| hberaud[m] | amorin: ^ | 16:24 |
| amorin | it was few weeks ago, while trying to fix eventlet bump on mistral, not on nova | 16:25 |
| hberaud[m] | amorin: ack. But what was your error? The same than nova (a race condition `Unexpected attempt to cancel events during shutdown.`), or the same that the one described on the github issue? | 16:26 |
| hberaud[m] | (just try to eliminate unrelated eventlet versions) | 16:27 |
| amorin | hum | 16:28 |
| amorin | it was a race, but I dont remember the details, I think I logged this stuff somewhere | 16:29 |
| amorin | I dont have more info | 16:29 |
| amorin | https://review.opendev.org/c/openstack/requirements/+/933257/comments/b6840804_5d86d740 | 16:30 |
| amorin | this is where I said this | 16:30 |
| hberaud[m] | ack | 16:31 |
| hberaud[m] | thanks | 16:31 |
| hberaud[m] | will double check later | 16:31 |
| hberaud[m] | in all the question I keep an eye on this bump | 16:32 |
| hberaud[m] | #topic Open Discussion | 16:32 |
| hberaud[m] | Do you have anything else that you want to discuss today? | 16:32 |
| hberaud[m] | then I think we are done | 16:35 |
| hberaud[m] | thanks everyone | 16:36 |
| hberaud[m] | #endmeeting | 16:36 |
| opendevmeet | Meeting ended Tue Jan 7 16:36:04 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:36 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/openstack_eventlet_removal/2025/openstack_eventlet_removal.2025-01-07-16.00.html | 16:36 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/openstack_eventlet_removal/2025/openstack_eventlet_removal.2025-01-07-16.00.txt | 16:36 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/openstack_eventlet_removal/2025/openstack_eventlet_removal.2025-01-07-16.00.log.html | 16:36 |
| amorin | thanks! | 16:36 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!