| opendevreview | Ghanshyam Maan proposed openstack/nova-specs master: Spec for graceful shutdown part3: safe termination https://review.opendev.org/c/openstack/nova-specs/+/1010320 | 03:35 |
|---|---|---|
| opendevreview | Ghanshyam Maan proposed openstack/nova-specs master: Spec for graceful shutdown part3: safe termination https://review.opendev.org/c/openstack/nova-specs/+/1010320 | 03:50 |
| *** JanWasilewski is now known as janwasilewski | 09:03 | |
| opendevreview | Max proposed openstack/nova master: revert: remove populate_missing_availability_zones https://review.opendev.org/c/openstack/nova/+/971894 | 09:21 |
| *** bauzas4 is now known as bauzas | 09:41 | |
| opendevreview | Merged openstack/nova master: Fix wrong Neutron install guide link for RDO controller https://review.opendev.org/c/openstack/nova/+/992035 | 11:23 |
| opendevreview | Merged openstack/nova master: Delete the suse reference in the documentation. https://review.opendev.org/c/openstack/nova/+/927705 | 11:23 |
| opendevreview | Maximilian Stinsky proposed openstack/nova stable/2026.1: Fix KeyError: 'body' in os-volumes_boot API proxy https://review.opendev.org/c/openstack/nova/+/1010162 | 11:24 |
| opendevreview | Alexandre Engloner proposed openstack/nova master: Replace ServerUpdateResponse server_groups array property https://review.opendev.org/c/openstack/nova/+/995470 | 11:26 |
| opendevreview | Alexandre Engloner proposed openstack/nova master: Replace ServerUpdateResponse server_groups array property https://review.opendev.org/c/openstack/nova/+/995470 | 11:28 |
| opendevreview | Merged openstack/nova master: Re-enable "test_tagged_attachment" test. https://review.opendev.org/c/openstack/nova/+/876699 | 13:02 |
| opendevreview | Merged openstack/nova master: Replace outdated usage of egrep by grep https://review.opendev.org/c/openstack/nova/+/1002754 | 13:02 |
| opendevreview | Merged openstack/nova master: Fix the libvirt.images_type value in KVM docs https://review.opendev.org/c/openstack/nova/+/896651 | 13:03 |
| tkajinam | I think we can abandon https://review.opendev.org/c/openstack/nova/+/853786 now, because the related changes for cinder and os-brick have been abandoned due to inactivity. https://review.opendev.org/q/topic:%22bp/add-lustre-driver%22 | 13:33 |
| tkajinam | could anyone from core spend a few minutes to abandon it (to reduce our review queue length) | 13:33 |
| gibi | tkajinam: ohh that is the exact mentally non challenging task I need as a disconnect before the PTG :) On it... | 13:33 |
| tkajinam | gibi, thx ! | 13:34 |
| gibi | is this a single patch? | 13:34 |
| gibi | (I prepared for a long list of patches to mechanically close and gain lot of free dopamine) | 13:35 |
| tkajinam | that's a single one, sorry | 13:35 |
| gibi | :) | 13:35 |
| tkajinam | I'm just going through patches appearing in my patches as conflicts | 13:36 |
| tkajinam | that's a not big win but a easy small step | 13:36 |
| gibi | I abandoned the snp series from your colleague the other day | 13:36 |
| gibi | as we landed yours | 13:36 |
| tkajinam | gibi, yeah I saw that. thanks | 13:37 |
| gibi | that was at least 5 patches to get rid off | 13:37 |
| * gibi is a junkie | 13:37 | |
| tkajinam | by the way I noticed we didn't discuss if nova-reviewers (or whatever it is named) should have right to abandon changes or -2. -2 might be too powerful but I personally feel like having more people to sweep the queue may be helpful | 13:37 |
| tkajinam | gibi, let's go into lfx insight to find how we get "number of reviews closed by the person" | 13:38 |
| tkajinam | and abandon 1000 reviews before you bring that stat number to your mgmt :-P | 13:38 |
| gmaan | Uggla: I might be away from 15 - 15.15 UTC ish to drop kids to school. I think cross cell migration and graceful shutdown is schedules after that 15 UTC? | 13:39 |
| gibi | tkajinam: I'm totally OK for nova-reviewers to got abadon rights. | 13:39 |
| gibi | to get | 13:39 |
| tkajinam | :-) | 13:40 |
| gibi | -2 is something that needs a dicussion for sure | 13:40 |
| gibi | I think -2 is sticky across patchsets so yeah that might be too much | 13:40 |
| Uggla | gmaan true for graceful. But x cell LM is starting at 15UTC | 13:41 |
| tkajinam | gmaan, 1500-1600UTC : Cross-cell live migration | 13:41 |
| tkajinam | gmaan, 1700-1800UTC : Nova graceful shutdown part-3: operations self-termination point | 13:41 |
| gmaan | yeah, i will be away for about 15 min for the cross-cell one | 13:41 |
| tkajinam | (just copy-pasting agenda lines) | 13:41 |
| gmaan | gibi: as you are here, or maybe dansmith or elodilles knows, do you know why nova-tox-validate-backport is non voting in check pipeline and voting in gate pipeline? catching invalid backport in check pipeline is worth right? | 13:42 |
| gibi | I do remember it was intentional but I cannot come up with a why | 13:43 |
| gibi | I think we need elodilles' memories | 13:43 |
| dansmith | gmaan: I think it might be because we wanted to be able to line up backports before master merged without having to recheck them again after master merged, but still prevent them landing out of order | 13:43 |
| gibi | ohh that sounds reasonable | 13:43 |
| gibi | one less recheck | 13:43 |
| gmaan | ohk, yeah, prevent recheck is good reason | 13:44 |
| dansmith | right, else we'd sort of serialize the whole thing | 13:44 |
| gmaan | let me add it reason in zuul file. i was wondering why we made it like this. thanks | 13:45 |
| gibi | thanks gmaan for documenting it | 13:53 |
| sean-k-mooney[m] | gmaan: its very intentional that its non-voting n check | 14:01 |
| sean-k-mooney[m] | and yes we anted it to not make the job red when we were proposing the backports | 14:01 |
| sean-k-mooney[m] | to avoid having to do a recheck alwasy | 14:02 |
| gmaan | yeah | 14:02 |
| Leo[m] | Hello, my PTG topic "Support for Hostname-based Ceph Mon References" is scheduled for now. Is this the right chat for Nova PTG discussions? Thank you | 14:02 |
| dansmith | Leo[m]: https://meetpad.opendev.org/nova-2027.1-ptg | 14:03 |
| dansmith | Leo[m]: we reordered you after another session since you weren't on, so just hold up for a bit and you'll be next | 14:03 |
| Leo[m] | Ok, thank you | 14:05 |
| dansmith | abhishekk: we have a question for you about image trusted certificate verification, maybe we can circle back to it when you're in the nova room later | 14:19 |
| abhishekk | dansmith: ack, you can post the question now as well | 14:35 |
| * tkajinam has been hearing some Zelda games sounds ... :-) | 15:25 | |
| gibi | Uggla: gmaan: heads up at some point between 17:00-18:00 UTC, which is potentially in the graceful shutdown discussion, I have to switch to mobile and that will put me in a more listening mode than speaking mode. | 16:32 |
| Uggla | gibi: ok thanks to let us know | 16:33 |
| Uggla | tkajinam, you have good taste in music! ;) | 16:35 |
| tkajinam | ;-) | 16:36 |
| dansmith | Uggla: you're not locally recording the nova sessions are you? | 17:52 |
| gmaan | added the notes in 'AGREE' section https://etherpad.opendev.org/p/nova-2027.1-ptg#L670 | 17:59 |
| Uggla | dansmith, no record. | 17:59 |
| dansmith | Uggla: ack, figured.. we had at least two cores not able to attend sessions this week or for part so just wondering.. that's one down side to meetpad for this stuff | 18:00 |
| elodilles | gmaan: sorry, missed your ping. but dansmith and sean-k-mooney[m] answered your question o:) (this was the patch that introduced and explained the non-voting+voting setup -> https://review.opendev.org/c/openstack/nova/+/796626 ) | 18:04 |
| gmaan | elodilles: yeah, thanks | 18:13 |
| gmaan | no notification about new change here. eems opendevreview bot not working? | 18:15 |
| gmaan | dansmith: gibi but i added note about backport validation job in this https://review.opendev.org/c/openstack/nova/+/1010507 | 18:15 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: [Trivial] Add a notes for nova-tox-validate-backport being non-voting in check https://review.opendev.org/c/openstack/nova/+/1010507 | 18:15 |
| gmaan | oh, late but working :) | 18:15 |
| dansmith | gmaan: I stephenfin'd it .. he would be very proud :) | 18:17 |
| gmaan | thanks :) | 18:18 |
| opendevreview | Merged openstack/nova master: [Trivial] Add a notes for nova-tox-validate-backport being non-voting in check https://review.opendev.org/c/openstack/nova/+/1010507 | 22:07 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!