| opendevreview | Ashish Gupta proposed openstack/nova master: tests: file-backed SQLite with WAL in threading mode for Database and CellDatabases Fixtures https://review.opendev.org/c/openstack/nova/+/988583 | 00:11 |
|---|---|---|
| opendevreview | Ashish Gupta proposed openstack/nova master: tests: use file-backed Placement SQLite in functional threading https://review.opendev.org/c/openstack/nova/+/992581 | 00:13 |
| opendevreview | Ashish Gupta proposed openstack/nova master: tests: Use per-database write locks instead of global lock https://review.opendev.org/c/openstack/nova/+/992862 | 00:15 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Use domain capabilities to detect SEV-SNP support https://review.opendev.org/c/openstack/nova/+/996316 | 00:27 |
| sean-k-mooney | gibi: https://review.opendev.org/c/openstack/futurist/+/996567 and the followups may be of interst. | 00:52 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove redundant version check for virtio-fs https://review.opendev.org/c/openstack/nova/+/996496 | 02:24 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Call manager graceful_shutdown while RPC server waits https://review.opendev.org/c/openstack/nova/+/996583 | 02:41 |
| frickler | anyone around for stable reviews of the OSSN-0101 fix? https://review.opendev.org/c/openstack/nova/+/995956 etc. | 07:15 |
| opendevreview | Bhavana proposed openstack/nova master: Add Minimal Displacement Solver for MIG GPU Repartitioning https://review.opendev.org/c/openstack/nova/+/996058 | 07:25 |
| *** elodilles_pto is now known as elodilles | 07:52 | |
| gibi | frickler: +2 | 07:54 |
| gibi | elodilles: could you look at these backports? https://review.opendev.org/c/openstack/nova/+/995956 | 07:54 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: HACK:Speed up functional test in threading mode https://review.opendev.org/c/openstack/nova/+/996500 | 07:57 |
| opendevreview | Thibaut Démaret proposed openstack/nova master: libvirt: add disk rotation_rate support for local disks https://review.opendev.org/c/openstack/nova/+/979693 | 08:14 |
| elodilles | gibi: +2'd it. will follow up with the backports to older branches when this merges | 08:39 |
| gibi | elodilles: thanks | 09:30 |
| ralonsoh | hi folks, a quick review for https://review.opendev.org/c/openstack/os-vif/+/995933 will be appreciated | 10:43 |
| ralonsoh | thanks! | 10:43 |
| opendevreview | Merged openstack/nova stable/2026.1: Fix mutating global config in websocket proxy https://review.opendev.org/c/openstack/nova/+/995956 | 11:00 |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | 11:01 |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | 11:01 |
| opendevreview | Ashish Gupta proposed openstack/placement master: tests: Support file-backed SQLite URL in placement fixtures https://review.opendev.org/c/openstack/placement/+/993106 | 11:53 |
| opendevreview | Ashish Gupta proposed openstack/nova master: tests: use file-backed Placement SQLite in functional threading https://review.opendev.org/c/openstack/nova/+/992581 | 12:31 |
| opendevreview | Ashish Gupta proposed openstack/nova master: tests: Use per-database write locks instead of global lock https://review.opendev.org/c/openstack/nova/+/992862 | 12:34 |
| gibi | gmaan: sean-k-mooney: the threading functional test reproduces a seemingly valid API race condition between volume attach and attachment show https://bugs.launchpad.net/nova/+bug/2160224 | 13:14 |
| sean-k-mooney | :) thats fun | 13:15 |
| gibi | we create the BDM in one transaction and populate the attachment_id in second one | 13:15 |
| gibi | in between the two there is a BDM but no attachment_id in it | 13:15 |
| sean-k-mooney | i need to refine the futurist patches not only based on your feed back but to have them relibaliy pass the unit tests :) but there are a lot of optimisation opterutnities there | 13:15 |
| gibi | but the API response schema on showing the attachment insists on having a not non attachment_id in the response | 13:16 |
| gibi | ~not None | 13:16 |
| sean-k-mooney | we proably should weaken it slightly | 13:16 |
| gibi | I will push a fix to return 404 until the BDM has a valid attachement_id | 13:16 |
| sean-k-mooney | if the servier has been built it shoudl be non non | 13:16 |
| gibi | that does not need an response schema chang | 13:17 |
| sean-k-mooney | oh for just GET servers/<server_id>/os-volume_attachments/<volume_id> | 13:17 |
| gibi | yepp | 13:17 |
| sean-k-mooney | so this isnt an issue in server show | 13:17 |
| gibi | nope | 13:17 |
| sean-k-mooney | its an isssue int he aggachemts proxy api | 13:17 |
| sean-k-mooney | ok then ya a 404 makes sense before its finished attaching | 13:18 |
| gibi | but I'm not sure what nova does if creating an attachment in cinder fails | 13:18 |
| gibi | if we cannot show the attachment then I guess we cannot delete it either | 13:18 |
| gibi | i.e. the user cannot show the attachment as we will return 404 | 13:18 |
| gibi | hm | 13:19 |
| gibi | but deleting an attachment only needs the volume_id not the attachement_id | 13:19 |
| gibi | then the user can delete a half baked attachemtn | 13:19 |
| gibi | the it is OK | 13:19 |
| gibi | I can return 404 in show until the attachment_id is None | 13:20 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: HACK:Speed up functional test in threading mode https://review.opendev.org/c/openstack/nova/+/996500 | 14:26 |
| opendevreview | Merged openstack/nova stable/2025.2: Fix mutating global config in websocket proxy https://review.opendev.org/c/openstack/nova/+/995957 | 14:34 |
| sean-k-mooney | gibi: melwitt do we want to proceed with https://review.opendev.org/c/openstack/nova/+/969538 for now and we can loop back to adress_space vs rss vs both later? | 14:59 |
| sean-k-mooney | i am not sure i have the capsity to track that and rather then forget about it i think im ok to upgrade to +2w if ye are both happy with the current appoch | 15:02 |
| gibi | lets land it and get back to it later if needed | 15:11 |
| melwitt | thanks gibi and sean-k-mooney, if you prefer rss I can quickly change it to that, I am good either way | 16:01 |
| sean-k-mooney | melwitt: while i do in general i dont know htat i want to have to try and validate that does not break anythign else. it shoudl not but lets leave it for a follow up | 16:05 |
| melwitt | sean-k-mooney: yeah that's kinda where I was at too. thanks | 16:06 |
| sean-k-mooney | cool noted in the review +2w | 16:07 |
| melwitt | super thanks :) | 16:08 |
| sean-k-mooney | one thing to note is this test is not failign because of debian 12 and now 13 | 16:10 |
| sean-k-mooney | so unless we try this on ubuntu 24.04 we wont know if its fully "fixed" i.e. if tweadkin the config option in the ci job is enough | 16:11 |
| melwitt | yes that is true | 16:16 |
| melwitt | IIRC the error was pretty clear it had hit the process limit, that combined with how ironic and cinder also have to configure it made me reasonably confident this would work | 16:17 |
| sean-k-mooney | it shoudl but the defautl value might need to be higher, thats why i mentioned the job config. so one thing that might be interesting to try is how does ubuntu 26.04 and ceph intereact | 16:18 |
| sean-k-mooney | although since debian 13 is fine | 16:19 |
| sean-k-mooney | i think 26.04 will work as well | 16:19 |
| melwitt | yeah, it was just the encryption overhead makes a 1G image slightly larger than 1G so the failure makes sense | 16:19 |
| opendevreview | Ashish Gupta proposed openstack/placement master: tests: Support file-backed SQLite URL in placement fixtures https://review.opendev.org/c/openstack/placement/+/993106 | 16:32 |
| gouthamr | melwitt: bauzas gibi: could you folks review this OSSN for me: https://review.opendev.org/c/openstack/security-doc/+/996187 | 16:59 |
| melwitt | gouthamr: done | 17:11 |
| gouthamr | thank you melwitt! | 17:11 |
| gmaan | gibi: sean-k-mooney: on volume attachment id as None, agree on fixing the API and not schema. if atachment_id is not available we cannot guarantee that attachment is done in cinder side or not so returning 404 make sense . | 17:11 |
| gmaan | gibi: sean-k-mooney now I am wondering that we used to return 200 before schema restriction which converted it to 400. now if I consider case where schema restriction is not enabled (actual production env) when by fixing this bug we will be converting 200 to 404 so we need microversion bump. | 17:13 |
| gmaan | even tough it is bug fix, but need to take care of interoperability with version bump | 17:13 |
| sean-k-mooney | gmaan: well if the attchment id is not avaibel the attchment is defnitly not don in nova yet | 17:14 |
| sean-k-mooney | so regardless of cidner we have not compelted the attachment | 17:15 |
| sean-k-mooney | so a 404 until that is fully compelt makes sense i think | 17:15 |
| sean-k-mooney | the 200->404 change is worht consdiering but i agree that it should not need a microverion | 17:17 |
| gmaan | think about users who used to call this API to monitor if attachment is ready until attachment_id is not None they will be broken from 200 to 404 so IMO it definitely need microversion | 17:19 |
| gmaan | like server show to keep calling until server is active or ip addresses are pupulated in that | 17:19 |
| sean-k-mooney | gmaan: right so that is alreayd broken | 17:24 |
| sean-k-mooney | becuase it will be listed there before the attchment in nova is completed | 17:24 |
| sean-k-mooney | gmaan: the only way to do that and not ahve a race today is via server show | 17:25 |
| sean-k-mooney | gmaan: becasue of https://bugs.launchpad.net/nova/+bug/2058928 | 17:26 |
| gmaan | yes, that action finish time could be helpful | 17:28 |
| sean-k-mooney | the api gibi is trying to fix just tell you that we have complete attachign the device in cinder but not to the vm or nova when the id is populated | 17:29 |
| sean-k-mooney | tehre are actuly 2 more step to completing the volume attachment on the nova side although populating the attachment id in the bdms is near the end | 17:30 |
| -Guest12850- NOTICE: The Gerrit service on review.opendev.org will be offline briefly one hour from now, at 21:00 UTC, while we rename a project. | 20:02 | |
| opendevreview | Merged openstack/nova master: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/969538 | 20:49 |
| -Guest12850- NOTICE: The Gerrit service on review.opendev.org is going offline momentarily at 21:00 UTC while we rename a project, but will return within a few minutes. | 20:59 | |
| opendevreview | Merged openstack/nova master: tests: file-backed SQLite with WAL in threading mode for Database and CellDatabases Fixtures https://review.opendev.org/c/openstack/nova/+/988583 | 21:09 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!