| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-01-13 | |||
| 14:41:11 | sean-k-mooney | dansmith: paperwork wise i would just tack it however you find simplest unless bauzas has a specific request of for a dedicate bug | |
| 14:41:22 | dansmith | yeah, so I was just going to say: | |
| 14:41:50 | dansmith | bauzas: this fix needs to get in: https://review.opendev.org/c/openstack/nova/+/931833 we noticed it because it caused a problem for the fix of another bug, but we never filed a bug for this specifically, | |
| 14:42:01 | dansmith | since it was just on the heels of landing the initial gpt detection support | |
| 14:42:17 | bauzas | dansmith: context ? | |
| 14:42:20 | dansmith | so if we need paperwork, please say, but if not, could you review? | |
| 14:42:35 | bauzas | want me to review ? | |
| 14:42:37 | bauzas | ack, ok | |
| 14:42:47 | bauzas | I already added myself for reviewing it indeed | |
| 14:42:57 | dansmith | thanks | |
| 14:43:22 | dansmith | I've got a tangled web of reviews for disk inspection across several projects and I need to start reducing that before I lose it :) | |
| 14:53:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: [CI][nova-next]test with placement ac breadth-first https://review.opendev.org/c/openstack/nova/+/937275 | |
| 15:02:29 | sean-k-mooney | gibi: ^ fyi see comment inline, im pretty sure peers is not the correct group | |
| 15:04:05 | sean-k-mooney | im gong to see if i can quickly update https://review.opendev.org/c/openstack/nova/+/933365 to adress the review feedback. if i cant get that to a mergable state today however im goign to have to ask someone else to take it over to unblock the requirements patch | |
| 15:04:45 | gibi | sean-k-mooney: thanksh | |
| 15:04:48 | gibi | sean-k-mooney: thank | |
| 15:04:51 | gibi | s | |
| 15:06:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: [CI][nova-next]test with placement ac breadth-first https://review.opendev.org/c/openstack/nova/+/937275 | |
| 15:34:37 | opendevreview | sean mooney proposed openstack/nova master: [eventlet] update nova tests for eventlet 0.37.0 https://review.opendev.org/c/openstack/nova/+/933365 | |
| 15:35:57 | opendevreview | sean mooney proposed openstack/nova master: [eventlet] update nova tests for eventlet 0.38.2 https://review.opendev.org/c/openstack/nova/+/933365 | |
| 15:36:50 | sean-k-mooney | hberaud ^ i think that might be enough. the issue we had with the funcitoal test seam to have been resolved with the oslo.log release as well | |
| 15:38:37 | opendevreview | Dr. Jens Harbott proposed openstack/nova master: DNM: Test eventlet bump https://review.opendev.org/c/openstack/nova/+/938879 | |
| 15:47:09 | sean-k-mooney | frickler: i rebased ^ to confirm if my other change is enogh but it works locally so i suspect it is | |
| 15:48:13 | sean-k-mooney | locally im emulating the test env by doing " .tox/functional/bin/python3 -m pip install -U -c https://opendev.org/openstack/requirements/raw/commit/27ded6c22e5f1ea9a16c8d3dabbd7fad72775f00/upper-constraints.txt -r requirements.txt -r test-requirements.txt eventlet" | |
| 15:48:25 | sean-k-mooney | that the url to the uc bump patch | |
| 16:43:50 | MengyangZhang[m] | sean-k-mooney: Hey sean, are you planning to talk about this https://review.opendev.org/c/openstack/nova-specs/+/932653 on Thur? I see it listed on the agenda of Thu's cinder-nova meeting. Just want to give a quick update about. I already brought it up to cinder last week and they gave the green light since there was no cinder changes needed. | |
| 16:43:51 | MengyangZhang[m] | I was wondering what's next step regarding submitting the proposed code changes. Since the spec freeze has passed I heard, would it be possible to follow the specless workflow? Happy to discuss it in the next nova meeting. | |
| 17:35:06 | sean-k-mooney | MengyangZhang[m]: so technially we are past the spec approval deadline for this release. it was last thrusday | |
| 17:36:21 | sean-k-mooney | so procedurely this would need a spec freeze excption. it might be better however to plan this for next cycle but start on the impementation in parallel | |
| 18:08:49 | sean-k-mooney | hi folks https://review.opendev.org/c/openstack/nova/+/933365 is green and so is https://review.opendev.org/c/openstack/nova/+/938879/2 so i think we are ok to proceed with merging that | |
| 18:09:20 | sean-k-mooney | gibi: melwitt: would ye be able to review https://review.opendev.org/c/openstack/nova/+/933365/3 | |
| 18:11:56 | melwitt | sean-k-mooney: did you ever find some reason that the new eventlet cause the waitall() calls to change? | |
| 18:12:37 | sean-k-mooney | no | |
| 18:13:06 | sean-k-mooney | i dont think its particalarly relevent to the behavior of the test however | |
| 18:13:21 | sean-k-mooney | so i didnt spend much time trying to find out | |
| 18:13:47 | JayF | New eventlet fixed a few locking issues around os.read()/os.write() patched interactions aiui; the unit test change made sense to me in that context | |
| 18:14:02 | JayF | but I'm obviously not a nova or eventlet expert, just someone who has read all the involved code :) | |
| 18:28:35 | sean-k-mooney | im going to quickly test without the change to make sure it still fails | |
| 18:28:48 | sean-k-mooney | its possibel that the oslo change make it not required | |
| 18:29:02 | sean-k-mooney | but i suspect that the extra call to waitall is related to that | |
| 18:29:44 | sean-k-mooney | with the new oslo release i was able to remove the funcitonal test change | |
| 18:30:00 | sean-k-mooney | but i didnt test master with the new versions which im doing now | |
| 18:30:28 | JayF | nice, that'll build even more confidence that the oslo.log pipemutex stuff fixed something \o/ | |
| 18:33:04 | sean-k-mooney | hum ok master apprelty passed. but i need to verify that it has the correct versions fo eventlet. changing branch might have cause the venv to be recreateded | |
| 18:34:28 | sean-k-mooney | https://paste.opendev.org/show/bUq1UNJTP1NHybsQJHPN/ | |
| 18:34:40 | sean-k-mooney | so ya i actully think the nova change is not requried any more | |
| 18:35:09 | sean-k-mooney | frickler: do we want to try droping the depends-on form the requirements change? | |
| 18:36:15 | sean-k-mooney | oslo.log==7.0.0 might be all tha twas requried to fix nova with eventlet 0.38.2 | |
| 19:04:45 | opendevreview | melanie witt proposed openstack/nova master: Bump requirement to PrettyTable>=2.4.0 https://review.opendev.org/c/openstack/nova/+/939157 | |
| 19:13:30 | frickler | sean-k-mooney: ack, confirmed locally, updating the change now | |
| 19:32:36 | sean-k-mooney | cool im goign to call it a night, ill leave my nova change open but ill abandon it if the requiremetn patch passes | |
| 19:38:05 | opendevreview | melanie witt proposed openstack/nova stable/2024.2: libvirt: Wrap un-proxied listDevices() and listAllDevices() https://review.opendev.org/c/openstack/nova/+/939158 | |
| 20:19:19 | opendevreview | Dr. Jens Harbott proposed openstack/nova master: DNM: Test eventlet bump https://review.opendev.org/c/openstack/nova/+/938879 | |
| 20:33:19 | opendevreview | melanie witt proposed openstack/nova stable/2024.1: libvirt: Wrap un-proxied listDevices() and listAllDevices() https://review.opendev.org/c/openstack/nova/+/939162 | |
| 20:34:25 | opendevreview | melanie witt proposed openstack/nova stable/2023.2: libvirt: Wrap un-proxied listDevices() and listAllDevices() https://review.opendev.org/c/openstack/nova/+/939163 | |
| #openstack-nova - 2025-01-14 | |||
| 00:58:44 | opendevreview | Merged openstack/nova master: Handle iso+gpt detections https://review.opendev.org/c/openstack/nova/+/931833 | |
| 01:55:56 | opendevreview | Tatsuya Hayashino proposed openstack/nova-specs master: Repropose instance metadata protection https://review.opendev.org/c/openstack/nova-specs/+/939190 | |
| 01:59:46 | opendevreview | Tatsuya Hayashino proposed openstack/nova-specs master: Repropose instance metadata protection https://review.opendev.org/c/openstack/nova-specs/+/939190 | |
| 08:52:43 | opendevreview | Merged openstack/nova-specs master: Create specs directory for 2025.2 Flamingo https://review.opendev.org/c/openstack/nova-specs/+/939091 | |
| 14:29:48 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Handle GPT image detection https://review.opendev.org/c/openstack/nova/+/933928 | |
| 14:29:48 | opendevreview | Dan Smith proposed openstack/nova master: DNM: Test with latest oslo.utils https://review.opendev.org/c/openstack/nova/+/933444 | |
| 16:01:45 | bauzas | #startmeeting nova | |
| 16:01:45 | opendevmeet | Meeting started Tue Jan 14 16:01:45 2025 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:01:45 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:45 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:01:49 | bauzas | heyho | |
| 16:01:52 | fwiesel | \o | |
| 16:02:01 | masahito | o/ | |
| 16:02:02 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:02:13 | s3rj1k | hi all | |
| 16:03:14 | elodilles | o/ | |
| 16:03:15 | Uggla | o/ | |
| 16:03:26 | bauzas | (on a meeting but I can run) | |
| 16:05:06 | bauzas | #topic Bugs (stuck/critical) | |
| 16:05:44 | bauzas | #info No Critical bug | |
| 16:05:50 | bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | |
| 16:05:57 | bauzas | any bug report to discuss ? | |
| 16:06:39 | bauzas | looks not | |
| 16:06:49 | masahito | If there is a time, I want to talk this bug in the review priority topic. https://bugs.launchpad.net/nova/+bug/2085709 | |
| 16:06:50 | bauzas | #topic Gate status | |
| 16:07:02 | bauzas | masahito: sure, I'll ping you at that time | |
| 16:07:12 | masahito | thank you. | |
| 16:07:37 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:07:44 | bauzas | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | |
| 16:07:57 | bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status | |
| 16:08:03 | bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | |
| 16:08:10 | bauzas | #info Please try to provide meaningful comment when you recheck | |
| 16:08:46 | opendevreview | Lajos Katona proposed openstack/nova master: blueprint: iothreads-for-instances https://review.opendev.org/c/openstack/nova/+/939254 | |
| 16:09:54 | bauzas | periodic jobs are all green | |
| 16:10:01 | bauzas | any gate failure to report ? | |
| 16:12:19 | bauzas | okay moving on | |
| 16:14:02 | bauzas | #topic Release Planning | |
| 16:14:09 | bauzas | #link https://releases.openstack.org/epoxy/schedule.html | |
| 16:14:16 | bauzas | #info Nova deadlines are set in the above schedule | |
| 16:14:53 | bauzas | #info 6 weeks before Feature Freeze | |
| 16:14:59 | bauzas | #info we'll try to create as os-traits release by the next 2 weeks, please provide your os-traits patches sooner than later. | |
| 16:15:02 | bauzas | you know this | |
| 16:16:09 | bauzas | #topic Review priorities | |
| 16:16:16 | bauzas | #link https://etherpad.opendev.org/p/nova-2025.1-status | |
| 16:16:29 | bauzas | #action bauzas to update the etherpad with the latest changes | |