Friday, 2024-08-30

opendevreviewMerged openstack/nova master: libvirt: Add config option to require secure SPICE.  https://review.opendev.org/c/openstack/nova/+/92254400:28
opendevreviewMerged openstack/nova stable/2023.1: Functional test test_boot_reschedule_with_proper_pci_device_count  https://review.opendev.org/c/openstack/nova/+/92655800:28
opendevreviewMichael Still proposed openstack/nova master: libvirt: direct SPICE console object changes  https://review.opendev.org/c/openstack/nova/+/92687601:07
opendevreviewMichael Still proposed openstack/nova master: libvirt: direct SPICE console database changes  https://review.opendev.org/c/openstack/nova/+/92687701:07
opendevreviewMichael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu  https://review.opendev.org/c/openstack/nova/+/92484401:07
opendevreviewMichael Still proposed openstack/nova master: libvirt: Add extra spec for sound device.  https://review.opendev.org/c/openstack/nova/+/92612601:07
opendevreviewMichael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection.  https://review.opendev.org/c/openstack/nova/+/92735401:07
opendevreviewMerged openstack/nova stable/2023.1: Fix PCI passthrough cleanup on reschedule  https://review.opendev.org/c/openstack/nova/+/92655901:16
opendevreviewMerged openstack/nova master: Reproducer test for image property hw_architecture  https://review.opendev.org/c/openstack/nova/+/92612801:16
opendevreviewMerged openstack/nova master: Add a functest for verifying multiple VGPU allocations  https://review.opendev.org/c/openstack/nova/+/84574701:16
opendevreviewMerged openstack/nova master: Correct info about volume-backed server rebuild  https://review.opendev.org/c/openstack/nova/+/92457401:16
opendevreviewsean mooney proposed openstack/nova master: update nova-next to use ubuntu 24.04  https://review.opendev.org/c/openstack/nova/+/92249201:29
opendevreviewsean mooney proposed openstack/nova master: Test live migration between hosts with differnet cpu_shared_sets  https://review.opendev.org/c/openstack/nova/+/91374401:31
opendevreviewsean mooney proposed openstack/nova master: enable numa live migration in the ceph job  https://review.opendev.org/c/openstack/nova/+/91384201:32
opendevreviewArtom Lifshitz proposed openstack/nova stable/2024.1: libvirt: call get_capabilities() with all CPUs online  https://review.opendev.org/c/openstack/nova/+/92755602:00
opendevreviewArtom Lifshitz proposed openstack/nova stable/2023.2: libvirt: call get_capabilities() with all CPUs online  https://review.opendev.org/c/openstack/nova/+/92755702:03
opendevreviewArtom Lifshitz proposed openstack/nova stable/2023.1: libvirt: call get_capabilities() with all CPUs online  https://review.opendev.org/c/openstack/nova/+/92755802:03
opendevreviewMerged openstack/python-novaclient master: tox: Add Python-specific functional envs  https://review.opendev.org/c/openstack/python-novaclient/+/92725002:52
*** bauzas_ is now known as bauzas02:54
opendevreviewMerged openstack/python-novaclient master: Fix Python 3.12 compatibility  https://review.opendev.org/c/openstack/python-novaclient/+/92725103:03
opendevreviewMerged openstack/nova master: Update [scheduler] limit_tenants_to_placement_aggregate config help  https://review.opendev.org/c/openstack/nova/+/84368403:20
opendevreviewzhou zhong proposed openstack/nova master: nova-manage: modify image properties in request_spec  https://review.opendev.org/c/openstack/nova/+/92431903:23
opendevreviewMichael Still proposed openstack/nova master: libvirt: direct SPICE console object changes  https://review.opendev.org/c/openstack/nova/+/92687606:31
opendevreviewMichael Still proposed openstack/nova master: libvirt: direct SPICE console database changes  https://review.opendev.org/c/openstack/nova/+/92687706:31
opendevreviewMichael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu  https://review.opendev.org/c/openstack/nova/+/92484406:31
opendevreviewMichael Still proposed openstack/nova master: libvirt: Add extra spec for sound device.  https://review.opendev.org/c/openstack/nova/+/92612606:31
opendevreviewMichael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection.  https://review.opendev.org/c/openstack/nova/+/92735406:31
opendevreviewMerged openstack/nova master: libvirt: Report available TPM models  https://review.opendev.org/c/openstack/nova/+/90918307:28
opendevreviewMichael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu  https://review.opendev.org/c/openstack/nova/+/92484407:35
opendevreviewMichael Still proposed openstack/nova master: libvirt: Add extra spec for sound device.  https://review.opendev.org/c/openstack/nova/+/92612607:35
opendevreviewMichael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection.  https://review.opendev.org/c/openstack/nova/+/92735407:35
*** bauzas_ is now known as bauzas08:30
*** __ministry is now known as Guest187508:34
gibiUggla: I tried to test the sdk and openstackclient patches for the virtiofs series but they werent bumped to use the new 2.97 microversion so all things fails using 2.96 and getting rightful 404s from nova11:09
opendevreviewMerged openstack/python-novaclient master: support get and list server's metadata  https://review.opendev.org/c/openstack/python-novaclient/+/91597711:12
gibiUggla: on the other hand I found the reason why the instance.update notification had shares=null and it was a config issue on my side as that notification coming from other than the compute service and I only enabled shares in the notification in the compute config. So that works as expected and shares are filled when I set the config in the relevant / all services11:18
gibielodilles: could you check this backport https://review.opendev.org/q/I9a2a7614313297f11a55d99fb94916d3583a9504 , please? :)11:26
manuvakery1Hey.. All of sudden lot of vms shutoff automatically12:04
manuvakery1"libvirt.libvirtError: internal error: qemu unexpectedly closed the monitor: /usr/include/c++/11/bits/random.tcc:2667: void std::discrete_distribution<_IntType>::param_type::_M_initialize() [with _IntType = int]: Assertion '__sum > 0' failed"12:04
manuvakery1I see this error in the logs 12:04
manuvakery1qemu instance log also has same error12:05
manuvakery1https://www.irccloud.com/pastebin/edKaN6J0/12:05
manuvakery1running openstack wallaby12:05
manuvakery1any hints on the root cause12:06
auniyalit says reason=crashed, so nova may be not aware of it !12:06
auniyalanything in nova-compute logs ?12:06
manuvakery1compute log also has same error 12:07
manuvakery12024-08-30 11:56:08.942 22938 ERROR nova.compute.manager [instance: 9dea6796-4103-4f55-8caa-16dc6848bbf9] libvirt.libvirtError: internal error: qemu unexpectedly closed the monitor: /usr/include/c++/11/bits/random.tcc:2667: void std::discrete_distribution<_IntType>::param_type::_M_initialize() [with _IntType = int]: Assertion '__sum > 0' failed.12:07
manuvakery1https://www.irccloud.com/pastebin/5H4M8DvL/12:08
auniyalso this is during new VM creation12:09
manuvakery1yup12:09
elodillesgibi: ACK, will do o/12:14
elodillesbauzas auniyal : if you have some time to review the python-novaclient release patch, that would be awesome: https://review.opendev.org/c/openstack/releases/+/92713612:15
Ugglagibi, sorry I have just noticed you comments. Good that the instance.update was due to a misconfiguration because I was not confident on that one.12:25
Ugglagibi, I need to fix some stuff on the client patches, so I will bump to 2.97. I will tell you when it will be ready.12:26
auniyalelodilles, done12:28
elodillesauniyal: thanks \o/12:30
sean-k-mooneybauzas: gibi  can we proceed with https://review.opendev.org/c/openstack/nova/+/92249212:54
sean-k-mooneythe depend on is already meged 12:55
opendevreviewFabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call  https://review.opendev.org/c/openstack/nova/+/91062712:58
gibisean-k-mooney: so 3.12 is a non mandatory test target for 2024.2 and ubuntu 24.04 not on the tested runtimes list. How stable this job is with 3.12? Do we risk a gate block?12:59
bauzasI left a +2 for saying I'm OK but I agree with gibi, we need to discuss that point before merging13:01
sean-k-mooneyhttps://zuul.openstack.org/builds?job_name=devstack-platform-ubuntu-noble&skip=0 it looks resanobale to me on the devstack side https://zuul.openstack.org/builds?job_name=devstack-platform-ubuntu-noble&skip=0 we also moved whitebox to it a while ago13:02
sean-k-mooneyit will be a required testing runtime for 2025.1 so we will need to start moving all the jobs shortly13:03
sean-k-mooneyso i want to have this in dalmaiton too if we can13:03
gibisean-k-mooney: OK those job results looks acceptable to me. I think we can merge it and if we see an uptick of nova-next failures we can always revert it.13:04
sean-k-mooney+1 to reverting if it proves unstable13:04
gibibauzas: any objection before I +A it?13:05
sean-k-mooneyi think the python 3.12 coverage is very valuable as its also a proxy for rhel 1013:06
bauzasI don't find any 13:06
sean-k-mooneyuntil we get centos 10 enabled13:06
sean-k-mooneyto od that i or someone need to update disk image built to supprot it but that not that hard13:06
bauzaswe could revert the bump if needed13:06
gibiapproved13:07
*** haleyb|out is now known as haleyb13:07
sean-k-mooneythanks :) im going to grab a coffee brb13:07
gibihave a nice one13:15
opendevreviewFabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call  https://review.opendev.org/c/openstack/nova/+/91062713:16
sean-k-mooneyits some cold brew i made at the start of the week. i have not been sleeping weel the last few days so (4-5) horus most night beign awake until after 5 in the morning13:20
opendevreviewFabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call  https://review.opendev.org/c/openstack/nova/+/91062713:21
sean-k-mooneyso this is my first coffe of the day with a big dash of salted carammel, i normlaly drink it black13:21
sean-k-mooneyidd blame the lack fo sleep on the cofee but i normally only have one cup a day and i skip it 2 days thsi week 13:22
gibisean-k-mooney: this year I did not brew cold. I try to limit my coffein intake as it disturbs my sleep and the cold one has more kick than a lightly packed mokka13:22
sean-k-mooneyya i got an expresso machien about 3-4 years ago so i went form 2-3 nesspreo pods a day to one ammericano13:23
sean-k-mooneyturns out to be about the same amount of cafeine13:23
artomWhere's our opensearch/elasticsearch instance again?13:54
artomhttps://opensearch.logs.openstack.org/_dashboards/app/ needs a login...13:55
dansmithartom: openstack/openstack14:01
sean-k-mooneyya that14:01
dansmithapparently the new one can't be unauthenticated14:01
sean-k-mooneyartom: are you looking into my comment https://review.opendev.org/c/openstack/nova/+/926083/comments/ad6713dc_e654215614:03
artomsean-k-mooney, the shelve thing? No, I got an openstacksdk error in the latest results14:04
sean-k-mooneyah14:04
sean-k-mooneyya i saw that job fail in one of my other patches14:04
sean-k-mooneyi have not reeckec but it passed on the next patch in my series14:05
artomReally trivial fix, I was just looking how often it happened14:05
sean-k-mooneyso i assume there s an itermint failure14:05
artomYeah, cloud.get returns None, and we unconditionally try to [ ] that14:05
artomSeems like just skipping the None case with an if and retrying the loop iteration would help...14:06
sean-k-mooneymanuvakery1: that feels like something i have heard fo before but its obviously a qemu bug14:07
sean-k-mooneyi guess it could be in libvirt but its defnintly not in nova14:07
sean-k-mooneywell cloud.get is presumable reading form the clouds.yaml which should nto be none14:08
artomI think it's an API call... it's polling for stack events14:09
sean-k-mooneyi didnt look at the actual errror yet14:10
artomSorry, cloud.get_stack() :)14:10
artomWhich I assume calls Heat14:10
sean-k-mooneypresumeable14:10
*** bauzas_ is now known as bauzas14:15
*** bauzas_ is now known as bauzas14:48
opendevreviewFabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call  https://review.opendev.org/c/openstack/nova/+/91062715:11
*** ykarel__ is now known as ykarel15:25
dansmithsean-k-mooney: +2 now ^16:01
sean-k-mooneydansmith: im out of energy now but i reveiwed that and approved. if you are going to request a FFE for the nova enablement of multiple locations support, given its size i think i woudl support that but if just want to leave that to next cycle it think thats also fine. it really depends n when the tempest stuff get resolved i guess so ill deffer to your judgement there.17:04
dansmithwell, tbh, I wasn't even going to consider it a feature17:05
dansmithit's tiny and it's just using the glance api the right way,17:05
dansmithbut if you think it needs to be considered as such, then that's fine17:05
dansmithnot sure what that really means with bauzas gone next week though17:05
sean-k-mooneywell more i was suggeting its small enouch that we may want to consier not procedurlly blocking it 17:06
sean-k-mooneyif the tempet issue can be resolved by the next team meeting17:06
sean-k-mooneywell FFE dont requrie PTL input17:06
sean-k-mooneyhistorically it only requried 3 cores to say yes17:06
sean-k-mooneyi only consider https://review.opendev.org/c/openstack/nova/+/891036 a feature in that we would not back port it17:07
sean-k-mooneymainly because of the requirements bump but i agree otherwise its just using glances api the secure way17:08
dansmiththere's no point in backporting it17:08
sean-k-mooneyright because there is no supprot in older glance17:08
dansmithI meant determining whether it was a feature or not, for the PTL buy-in.. I think that's bauzas' decision to make if it's unclear17:08
dansmithanyway, we'll keep pushing on the tempest patch and can revisit on monday maybe17:08
sean-k-mooneyack, either way i thin kthis could merge after rc1 if we decied to wait17:09
dansmiththe tempest patch isn't required, AFAIK, it just reduces a race that is already there just not hit because we don't actually use it yet, but it has passed without it17:09
sean-k-mooneyhaving it 2025.1 is the main thing17:09
sean-k-mooneyah ok so its not strictly a dep but we would have some instablity without it17:10
dansmithright17:10
sean-k-mooneyok well im going to checkout for the weekend17:11
sean-k-mooneycatch you monday o/17:11
dansmitho/17:11
opendevreviewJay Faulkner proposed openstack/nova master: [ironic] Factor out metadata and  send to ironic  https://review.opendev.org/c/openstack/nova/+/92763918:39
opendevreviewMerged openstack/nova stable/2023.1: Added context manager for instance lock  https://review.opendev.org/c/openstack/nova/+/91251119:38
bauzasI'm still there, I have some slides to work on19:39
*** bauzas_ is now known as bauzas19:47
JayFI think I'm going to need an FFE for 927639 if it's merging for C; I've been blocked on reviews and got some today and revised. I'd imagine it'll be ready to land very early next week. TBH, I'd like it in so it's off my plate, but it won't cause significant hardship if we have to push it.19:59
JayFs/C/D/19:59
opendevreviewBrian Haley proposed openstack/nova master: Support creating servers with RBAC SGs  https://review.opendev.org/c/openstack/nova/+/81152120:53
opendevreviewMerged openstack/nova master: Add hw_firmware_stateless image property  https://review.opendev.org/c/openstack/nova/+/92659020:57
opendevreviewMerged openstack/nova master: Cleanup setup.py  https://review.opendev.org/c/openstack/nova/+/90541921:00
*** bauzas_ is now known as bauzas22:15
melwittJayF: I think you accidentally added a new Change-Id to your next PS, so it did not go onto the review we've been doing. it made a new patch instead22:18
melwitt*a new review instead22:19
JayFegad22:19
opendevreviewJay Faulkner proposed openstack/nova master: [ironic] Factor out metadata and  send to ironic  https://review.opendev.org/c/openstack/nova/+/92391022:21
JayFI had to completely blow away the commit hook to keep it from trying to add a new one everytime22:22
JayFthanks melwitt :)22:23
melwitthm, I guess it really requires Change-Id to be the last line of the commit message. but apparently not all the time (?)22:24
JayFyeah I don't really understand, but I'm just glad you noticed so I could fix it :)22:24
*** bauzas_ is now known as bauzas22:39
opendevreviewMichael Still proposed openstack/nova master: libvirt: direct SPICE console object changes  https://review.opendev.org/c/openstack/nova/+/92687623:52
opendevreviewMichael Still proposed openstack/nova master: libvirt: direct SPICE console database changes  https://review.opendev.org/c/openstack/nova/+/92687723:52
opendevreviewMichael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu  https://review.opendev.org/c/openstack/nova/+/92484423:52
opendevreviewMichael Still proposed openstack/nova master: libvirt: Add extra spec for sound device.  https://review.opendev.org/c/openstack/nova/+/92612623:52
opendevreviewMichael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection.  https://review.opendev.org/c/openstack/nova/+/92735423:52

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!