| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-07-08 | |||
| 20:29:54 | melwitt | looks like no one is using it, huh. https://codesearch.openstack.org/?q=resident_set_size&i=nope&literal=nope&files=&excludeFiles=&repos= | |
| 20:31:25 | sean-k-mooney | so i moved the ceph jobs to debian to work around it https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/955714 and proposed a reviert of the tempest skip https://review.opendev.org/c/openstack/tempest/+/955177 | |
| 20:31:54 | sean-k-mooney | but we never actully merge the tempest skip reviert | |
| 20:32:11 | melwitt | yeah, I remembered that. I had wondered what the "real fix" should be | |
| 20:33:48 | sean-k-mooney | i +1 your patch becasue we coudl proceed with it, i think movign to rss instead fo adresss space woudl make sense btu let see if gibi has or somoen else has an opion | |
| 20:34:20 | sean-k-mooney | melwitt: im not agaisnt makign it configurabl i just dont think that the correct limit to use in this case in general | |
| 20:34:31 | sean-k-mooney | but we coudl change that in a followup | |
| 20:35:14 | melwitt | sure. I'm also happy to just change it to resident_set_size if there is no potential bad thing about that haha | |
| 20:35:31 | melwitt | I am a bit wondering why no one has made use of it before | |
| 20:36:36 | sean-k-mooney | well almost nothign uses this in general https://codesearch.openstack.org/?q=ProcessLimits&i=nope&literal=nope&files=&excludeFiles=&repos= | |
| 20:36:45 | sean-k-mooney | and i think the code was cargo culted form nova to the rest | |
| 20:37:31 | sean-k-mooney | tweaking the adress space limit will allow more meory to be used like rss would | |
| 20:37:44 | melwitt | nova trend setter | |
| 20:37:47 | sean-k-mooney | but you may have to set it higher then otherwise | |
| 20:38:26 | melwitt | yeah. I mostly saw how cinder and ironic both doing it | |
| 20:38:44 | sean-k-mooney | you know what ill appove your current patch but lets chat about this more | |
| 20:38:53 | sean-k-mooney | your currrent patch matches cinder and ironics approch | |
| 20:39:06 | sean-k-mooney | so if it works for them it will proably be fine | |
| 20:39:11 | melwitt | it's ok, I don't mind chatting for rss | |
| 20:39:32 | sean-k-mooney | ok but lets check back tomorrow so we dont forget :) | |
| 20:40:12 | melwitt | I feel like dan would have a good input for this but I think he's away until next week | |
| 20:40:20 | sean-k-mooney | gmaan: related to ^ i rebased https://review.opendev.org/c/openstack/tempest/+/955177 to get new results | |
| 20:41:06 | sean-k-mooney | ok im goign to go eat dinner o/ | |
| 20:41:16 | melwitt | ok seeya o/ | |
| 20:48:49 | gmaan | sean-k-mooney: ack | |
| 21:38:07 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Task tracking mechanism for graceful shutdown https://review.opendev.org/c/openstack/nova/+/996299 | |
| 21:57:03 | opendevreview | Merged openstack/nova master: Rename cyborg-tempest to cyborg-tempest-py3 https://review.opendev.org/c/openstack/nova/+/996297 | |
| 23:18:48 | 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 | |
| 23:24:43 | opendevreview | Ashish Gupta proposed openstack/nova master: tests: use file-backed Placement SQLite in functional threading https://review.opendev.org/c/openstack/nova/+/992581 | |
| 23:27:11 | 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 | |
| #openstack-nova - 2026-07-09 | |||
| 00:11:18 | 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:13:14 | 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:15:14 | 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:27:58 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use domain capabilities to detect SEV-SNP support https://review.opendev.org/c/openstack/nova/+/996316 | |
| 00:52:40 | sean-k-mooney | gibi: https://review.opendev.org/c/openstack/futurist/+/996567 and the followups may be of interst. | |
| 02:24:28 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove redundant version check for virtio-fs https://review.opendev.org/c/openstack/nova/+/996496 | |
| 02:41:05 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Call manager graceful_shutdown while RPC server waits https://review.opendev.org/c/openstack/nova/+/996583 | |
| 07:15:43 | frickler | anyone around for stable reviews of the OSSN-0101 fix? https://review.opendev.org/c/openstack/nova/+/995956 etc. | |
| 07:25:33 | opendevreview | Bhavana proposed openstack/nova master: Add Minimal Displacement Solver for MIG GPU Repartitioning https://review.opendev.org/c/openstack/nova/+/996058 | |
| 07:54:35 | gibi | frickler: +2 | |
| 07:54:47 | gibi | elodilles: could you look at these backports? https://review.opendev.org/c/openstack/nova/+/995956 | |
| 07:57:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: HACK:Speed up functional test in threading mode https://review.opendev.org/c/openstack/nova/+/996500 | |
| 08:14:26 | 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:39:04 | elodilles | gibi: +2'd it. will follow up with the backports to older branches when this merges | |
| 09:30:16 | gibi | elodilles: thanks | |
| 10:43:54 | ralonsoh | hi folks, a quick review for https://review.opendev.org/c/openstack/os-vif/+/995933 will be appreciated | |
| 10:43:55 | ralonsoh | thanks! | |
| 11:00:03 | opendevreview | Merged openstack/nova stable/2026.1: Fix mutating global config in websocket proxy https://review.opendev.org/c/openstack/nova/+/995956 | |
| 11:01:35 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | |
| 11:01:35 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 11:53:00 | opendevreview | Ashish Gupta proposed openstack/placement master: tests: Support file-backed SQLite URL in placement fixtures https://review.opendev.org/c/openstack/placement/+/993106 | |
| 12:31:57 | 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:34:54 | 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 | |
| 13:14:19 | 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:15:05 | sean-k-mooney | :) thats fun | |
| 13:15:42 | gibi | we create the BDM in one transaction and populate the attachment_id in second one | |
| 13:15:53 | gibi | in between the two there is a BDM but no attachment_id in it | |
| 13:15:59 | 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:16:19 | gibi | but the API response schema on showing the attachment insists on having a not non attachment_id in the response | |
| 13:16:29 | gibi | ~not None | |
| 13:16:46 | sean-k-mooney | we proably should weaken it slightly | |
| 13:16:50 | gibi | I will push a fix to return 404 until the BDM has a valid attachement_id | |
| 13:16:58 | sean-k-mooney | if the servier has been built it shoudl be non non | |
| 13:17:00 | gibi | that does not need an response schema chang | |
| 13:17:40 | sean-k-mooney | oh for just GET servers/<server_id>/os-volume_attachments/<volume_id> | |
| 13:17:44 | gibi | yepp | |
| 13:17:48 | sean-k-mooney | so this isnt an issue in server show | |
| 13:17:51 | gibi | nope | |
| 13:17:59 | sean-k-mooney | its an isssue int he aggachemts proxy api | |
| 13:18:14 | sean-k-mooney | ok then ya a 404 makes sense before its finished attaching | |
| 13:18:23 | gibi | but I'm not sure what nova does if creating an attachment in cinder fails | |
| 13:18:38 | gibi | if we cannot show the attachment then I guess we cannot delete it either | |
| 13:18:57 | gibi | i.e. the user cannot show the attachment as we will return 404 | |
| 13:19:05 | gibi | hm | |
| 13:19:39 | gibi | but deleting an attachment only needs the volume_id not the attachement_id | |
| 13:19:46 | gibi | then the user can delete a half baked attachemtn | |
| 13:19:51 | gibi | the it is OK | |
| 13:20:05 | gibi | I can return 404 in show until the attachment_id is None | |
| 14:26:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: HACK:Speed up functional test in threading mode https://review.opendev.org/c/openstack/nova/+/996500 | |
| 14:34:58 | opendevreview | Merged openstack/nova stable/2025.2: Fix mutating global config in websocket proxy https://review.opendev.org/c/openstack/nova/+/995957 | |
| 14:59:51 | 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? | |
| 15:02:10 | 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:11:55 | gibi | lets land it and get back to it later if needed | |
| 16:01:37 | melwitt | thanks gibi and sean-k-mooney, if you prefer rss I can quickly change it to that, I am good either way | |
| 16:05:53 | 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:06:19 | melwitt | sean-k-mooney: yeah that's kinda where I was at too. thanks | |
| 16:07:22 | sean-k-mooney | cool noted in the review +2w | |
| 16:08:41 | melwitt | super thanks :) | |
| 16:10:17 | sean-k-mooney | one thing to note is this test is not failign because of debian 12 and now 13 | |
| 16:11:07 | 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:16:16 | melwitt | yes that is true | |
| 16:17:52 | 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:18:32 | 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:19:00 | sean-k-mooney | although since debian 13 is fine | |
| 16:19:08 | sean-k-mooney | i think 26.04 will work as well | |
| 16:19:11 | melwitt | yeah, it was just the encryption overhead makes a 1G image slightly larger than 1G so the failure makes sense | |
| 16:32:14 | 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:59:48 | gouthamr | melwitt: bauzas gibi: could you folks review this OSSN for me: https://review.opendev.org/c/openstack/security-doc/+/996187 | |
| 17:11:01 | melwitt | gouthamr: done | |
| 17:11:20 | gouthamr | thank you melwitt! | |