| *** Callum0276 is now known as Callum027 | 06:20 | |
| opendevreview | Johannes Beisiegel proposed openstack/nova master: feat: compute created weigher https://review.opendev.org/c/openstack/nova/+/947503 | 06:21 |
|---|---|---|
| whoami-rajat_ | hi #openstack-nova , can i get a review on this rebuild fix? the cinder counterpart is merged and the bug only depends on this fix https://review.opendev.org/c/openstack/nova/+/916409 | 08:43 |
| andrewbonney | Hi. Would any other cores mind reviewing the small bug fix in https://review.opendev.org/c/openstack/nova/+/919961 - sean has already added +2 | 08:57 |
| opendevreview | Yaguang Tang proposed openstack/nova stable/2024.2: Handle iso+gpt detections https://review.opendev.org/c/openstack/nova/+/943940 | 09:21 |
| opendevreview | Yaguang Tang proposed openstack/nova stable/2024.2: Handle iso+gpt detections https://review.opendev.org/c/openstack/nova/+/943940 | 10:00 |
| sean-k-mooney | whoami-rajat_: ill add it to my list | 10:19 |
| sean-k-mooney | andrewbonney: ah that ya that still a good enhanchment to land | 10:19 |
| whoami-rajat_ | sean-k-mooney, thanks Sean! | 10:47 |
| sean-k-mooney | oh wrong tab i was like why is that touching neutorn | 10:49 |
| sean-k-mooney | whoami-rajat_: so i need to find the linke but i know dansmith_pto wanted to stop creating the 0 byte snapshot in glance for bfv snapshots and and some patches up to change it | 10:52 |
| sean-k-mooney | oh we already merged that https://review.opendev.org/c/openstack/nova/+/930754 | 10:53 |
| sean-k-mooney | ah i see | 10:55 |
| sean-k-mooney | the fix is to use the volume size | 10:56 |
| sean-k-mooney | whoami-rajat_: +2 with a request to update the release note | 11:17 |
| opendevreview | Merged openstack/os-resource-classes master: Fix outdated envlist https://review.opendev.org/c/openstack/os-resource-classes/+/939018 | 11:45 |
| opendevreview | Rajat Dhasmana proposed openstack/nova master: Fix: Rebuild volume backed server with snapshot image https://review.opendev.org/c/openstack/nova/+/916409 | 11:46 |
| opendevreview | Merged openstack/os-traits master: Replace UPPER_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/os-traits/+/947487 | 11:47 |
| whoami-rajat_ | sean-k-mooney, thanks Sean, I've addressed the releasenote concern | 11:48 |
| opendevreview | Merged openstack/nova master: doc: Drop deprecated [api] auth_strategy https://review.opendev.org/c/openstack/nova/+/944028 | 11:49 |
| opendevreview | Merged openstack/nova master: doc: Remove non-existent [service_user] auth_strategy https://review.opendev.org/c/openstack/nova/+/944029 | 11:49 |
| opendevreview | Merged openstack/osc-placement master: Drop unnecessary 'x' bit from doc config file https://review.opendev.org/c/openstack/osc-placement/+/932881 | 11:49 |
| sean-k-mooney | whoami-rajat_: +2 thanks | 11:52 |
| elodilles | sean-k-mooney: thanks for all the stable reviews you did in the past days, that helped a lot and a lot of patches were merged thanks to you o:) | 12:02 |
| elodilles | sean-k-mooney: and sorry for asking more... o:) ... but maybe these 2 patches could also land: https://review.opendev.org/c/openstack/nova/+/932534 | 12:03 |
| sean-k-mooney | elodilles: im happy to take a look at anything you think we should merge | 12:18 |
| sean-k-mooney | ah the architecture traits ones | 12:19 |
| elodilles | yepp, that merged to 2024.2 and 2024.1 as well | 12:25 |
| elodilles | maybe we are good with the patches that already merged, but these two and maybe https://review.opendev.org/c/openstack/nova/+/947214 could also merge. I'll update the nova 2023.2 release patch either way | 12:28 |
| opendevreview | Merged openstack/placement master: Skip functional tests on pre-commit config update https://review.opendev.org/c/openstack/placement/+/933191 | 13:23 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Remove WSGIService and WSGIServer classes https://review.opendev.org/c/openstack/nova/+/947245 | 13:30 |
| sean-k-mooney | elodilles: i have approved all 3. i need ot take a look at watcher next. liekly that wont be ready until next week | 13:30 |
| gibi | sean-k-mooney: melwitt: ^^ Thanks for the reviews. I fixed up the things. | 13:31 |
| opendevreview | Merged openstack/nova stable/2023.2: Reproducer test for image property hw_architecture https://review.opendev.org/c/openstack/nova/+/932533 | 13:54 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [doc]Remove eventlet based API endpoints https://review.opendev.org/c/openstack/nova/+/947507 | 14:28 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [doc]Describe file based GMR triggering https://review.opendev.org/c/openstack/nova/+/947508 | 14:28 |
| gibi | and fixed up the GMR doc patch as well ^^ | 14:28 |
| elodilles | sean-k-mooney: ACK, thanks o/ as I see there is only 1 patch currently for watcher that can be reviewed, but i don't know whether there are more that were not backported yet | 14:32 |
| elodilles | so watcher might be good without a release for bobcat EOL | 14:33 |
| sean-k-mooney | elodilles: ack i had not found time to look yet :) | 14:34 |
| sean-k-mooney | but that soudn promising at least | 14:34 |
| elodilles | :] | 14:35 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Remove WSGIServer related config options https://review.opendev.org/c/openstack/nova/+/947595 | 14:48 |
| opendevreview | Merged openstack/nova stable/2023.2: Libvirt: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/932534 | 15:13 |
| opendevreview | Merged openstack/nova stable/2023.2: [doc]Add `socket` option to [pci]alias numa_policy https://review.opendev.org/c/openstack/nova/+/947214 | 15:38 |
| elodilles | patches have merged to stable/2023.2 and I think we don't need to wait anymore, I've updated the release patch, so it should be good to go: https://review.opendev.org/c/openstack/releases/+/941420 | 17:46 |
| elodilles | Uggla auniyal bauzas ^^^ | 17:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!