Earlier  
Posted Nick Remark
#openstack-nova - 2026-07-13
12:15:49 sean-k-mooney doing the release is cleaner
12:16:05 sean-k-mooney the factg we dont use tox-siblings is why depend on only works for devstack today
12:16:20 gibi I did not find an example of tox-siblings and release was faster :)
12:16:24 elodilles sean-k-mooney[m]: is it possible that this patch needs to be backported to all stable branches? https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/955714
12:17:08 elodilles sean-k-mooney[m]: as this test case is failing on nova's stable/2025.1 nova-ceph-multistore job: https://review.opendev.org/c/openstack/nova/+/995958
12:17:14 sean-k-mooney no it can be but its not required
12:17:23 elodilles (tempest is branchless)
12:17:36 sean-k-mooney there is a better change to backport i think
12:17:56 sean-k-mooney well so we have a "fix" for nova
12:18:04 sean-k-mooney backporting that may be better
12:18:39 sean-k-mooney the fix is adding a config option for current hardcode value so in addtion to that we woudl ahve to tweak the job to set that
12:19:10 sean-k-mooney elodilles: https://review.opendev.org/c/openstack/nova/+/969538
12:19:15 sean-k-mooney so if we backport that
12:19:40 sean-k-mooney oh mel set the defautl to 2
12:19:42 sean-k-mooney images_address_space_limit
12:19:52 sean-k-mooney so that should jsut work
12:20:06 sean-k-mooney sicne she alrady doubled the hardcoded value for the new config default
12:20:38 opendevreview sean mooney proposed openstack/nova stable/2026.1: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/997027
12:22:51 opendevreview sean mooney proposed openstack/nova stable/2025.2: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/997028
12:23:14 opendevreview sean mooney proposed openstack/nova stable/2025.1: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/997029
12:23:24 sean-k-mooney elodilles: so ^ in theory should work
12:23:39 elodilles sean-k-mooney: so to be clear, this patch is needed to make this test case stable: test_extend_attached_encrypted_volume_luksv1
12:23:42 elodilles right?
12:24:35 sean-k-mooney yes
12:24:59 elodilles ACK, sorry, i'm a bit slow today double checking things o:)
12:25:07 sean-k-mooney the luks hearder make 1g cinder voluem need more the 1G of aresss space for qemu img to map and or process them
12:25:33 sean-k-mooney the patch set the default for the new config option to 2g
12:25:39 elodilles though i'm not really fond of changing config value on stable branches only to make test cases pass :/
12:26:11 sean-k-mooney well the other option is no luks on ubuntu 24.04
12:26:26 sean-k-mooney because without the patch its hardcoded to 1
12:26:47 elodilles yes, that's what i'm saying o:)
12:26:55 elodilles s/what/why
12:27:10 sean-k-mooney so this was really cause by a chagne in qemu
12:27:33 sean-k-mooney and how qemu-img internally manges its concurancy
12:27:52 elodilles so we need 2G to make the test pass, but that changes the value for already deployed environments from 1G to 2G if i understand correctly
12:27:55 sean-k-mooney the current change mimics what cinder and ironic do
12:28:38 sean-k-mooney elodilles: so i think the real long tim fix is to remove the adress space based limit and move to RSS (resedet set size) or drop the limit entrily
12:29:08 sean-k-mooney it exits to prvent patalogical behvior in qemu-img form triggerign the OOM killer
12:30:13 elodilles i see
12:30:50 sean-k-mooney which form a defence in dept pont of view i get but adressspace was never the right thing to limit on in my view RSS was but even then we dont sandbox all the other clis we interact with so it feels kind of arbiaty that we did limit qemu-img
12:33:28 elodilles sean-k-mooney: still, i'm a bit hesitant to backport this patch. i'd perhaps tempest-regex-exclude for this single test case on stable branches (as this test case was disabled before the revert has landed on tempest)
12:34:09 sean-k-mooney well this test used to be sable on the stable branchs
12:34:20 sean-k-mooney it was broken by a qemu release
12:34:51 sean-k-mooney so sure we can disbale it but i think we expect to backport the bugfix regardless fo the ci
12:35:14 sean-k-mooney this sued to work on stable and it was regressed by a ubuntu package release
12:35:23 elodilles ah, i see
12:35:25 sean-k-mooney so we want to be abel to suprpot existing users
12:37:12 sean-k-mooney elodilles: this affect a very specific subset of deployemtn because it depens on teh qemu+librados/ceph clinet release in use. and you have to have encypted volumes as well.
12:37:37 sean-k-mooney anyway lets get some ci results adn see if the backprot even works
12:38:43 sean-k-mooney elodilles: we could also backport swaping the ceph jobs to debian 12 but chaning the disto seams more invasive to me
12:39:07 sean-k-mooney althouhg it would only be on 2025.1
12:39:25 sean-k-mooney 2025.2+ already were using debian for ceph
12:40:52 sean-k-mooney elodilles: https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/997035 we can proably dicusss that in the qa channel seperatly
12:44:00 elodilles sean-k-mooney: thanks for the explanation. i've commented on the patch my concern o:)
12:45:47 sean-k-mooney we could set it to the same value in the backport and tweak it in the job or as you said just exclude the test form the relevent jobs
12:48:36 elodilles i'm not fully against backporting the patch as is, if stable cores agree to do so o:) but maybe the above options are less invasive ^^^
13:12:50 opendevreview Joan Gilabert proposed openstack/nova master: Add libvirt driver MDEV ARQ support for Cyborg https://review.opendev.org/c/openstack/nova/+/995840
13:33:48 opendevreview Merged openstack/nova master: doc: Update cursive repository path https://review.opendev.org/c/openstack/nova/+/996880
15:27:29 gmaan gibi: thanks for placement release, I had to do same workaround of testing nova cross jobs with placement master when enforce_scope removal caused the issue. I tired to release it but then dropped the idea to let it releases during its own time.
15:27:46 gmaan gibi: on 2160224, I will check today bug and the change.
15:35:37 opendevreview Joan Gilabert proposed openstack/nova master: Report OWNER_NOVA trait on resource providers https://review.opendev.org/c/openstack/nova/+/994299
15:35:37 opendevreview Joan Gilabert proposed openstack/nova master: Add libvirt driver MDEV ARQ support for Cyborg https://review.opendev.org/c/openstack/nova/+/995840
15:35:56 gibi gmaan: thanks!
15:36:07 gibi it turned out that releasing a beta was easy
15:39:01 gibi FYI nova meeting starts in 20 mintes here. (I will be your guide today)
16:00:49 gibi #startmeeting nova
16:00:49 opendevmeet Meeting started Mon Jul 13 16:00:49 2026 UTC and is due to finish in 60 minutes. The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:49 opendevmeet Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:49 opendevmeet The meeting name has been set to 'nova'
16:00:59 gibi o/
16:01:01 tkajinam o/
16:01:03 elodilles o/
16:02:00 gibi lets wait a bit to see if others show up
16:02:17 gibi I know that Uggla and bauzas are off today
16:03:26 gibi #topic Bugs (stuck/critical)
16:03:32 gibi any bugs to talk about?
16:04:08 gibi #topic Gate status
16:04:21 gibi anything worth mentioning about the gate?
16:04:31 tkajinam nothing I'm aware of, though we might need to be careful about gate status for a while after mass m-2 releases of libraries
16:04:40 gibi good heads up
16:05:08 tkajinam removal of enforce_scope is one of the upcoming big changes.
16:05:41 gibi we also landed major functional test changes to support threading, the threading job is non voting at the moment and hopefully the change does not affect the eventlet job
16:06:03 gibi but if you see strange things in functional test then let me know
16:06:10 elodilles oh, btw, there was a cross-nova job failure in oslo.concurrency release
16:06:11 sean-k-mooney o/
16:06:21 gibi elodilles: do you have a link?
16:06:33 elodilles this is the patch: https://review.opendev.org/c/openstack/requirements/+/996809
16:06:46 elodilles and this is the job: https://zuul.opendev.org/t/openstack/build/9c3d2302c2dc453092804e0db538deb3
16:07:03 elodilles i've already pinged the oslo team
16:07:08 elodilles on IRC
16:07:19 gibi the failing test called test_concurrent_server_delete so I feel relavance here :)
16:07:28 sean-k-mooney hum
16:07:30 elodilles just popped into my mind now that tkajinam mentioned the mass m-2 releases o:)
16:07:31 gibi anyhow I can take a deper look tomorrow
16:08:09 tkajinam I wasn't aware of any functional changes we recently merged into oslo.concurrecy so that looks a bit strange
16:08:34 tkajinam ping me in case you need hands from oslo side (and I'm aware of some discussions about slow futurist between gibi and sean-k-mooney but am not following it fully)
16:08:39 sean-k-mooney that test hevially depens on lock machanics
16:08:55 gibi tkajinam: thanks.
16:09:09 gibi anyhow moving on, we can handle that async
16:09:12 gibi #topic Release Planning

Earlier   Later