Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2024-08-30
12:54:38 sean-k-mooney bauzas: gibi can we proceed with https://review.opendev.org/c/openstack/nova/+/922492
12:55:46 sean-k-mooney the depend on is already meged
12:58:39 opendevreview Fabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call https://review.opendev.org/c/openstack/nova/+/910627
12:59:13 gibi sean-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?
13:01:49 bauzas I left a +2 for saying I'm OK but I agree with gibi, we need to discuss that point before merging
13:02:24 sean-k-mooney https://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 ago
13:03:31 sean-k-mooney it will be a required testing runtime for 2025.1 so we will need to start moving all the jobs shortly
13:03:56 sean-k-mooney so i want to have this in dalmaiton too if we can
13:04:19 gibi sean-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:46 sean-k-mooney +1 to reverting if it proves unstable
13:05:16 gibi bauzas: any objection before I +A it?
13:06:04 sean-k-mooney i think the python 3.12 coverage is very valuable as its also a proxy for rhel 10
13:06:10 bauzas I don't find any
13:06:11 sean-k-mooney until we get centos 10 enabled
13:06:32 sean-k-mooney to od that i or someone need to update disk image built to supprot it but that not that hard
13:06:33 bauzas we could revert the bump if needed
13:07:15 gibi approved
13:07:46 sean-k-mooney thanks :) im going to grab a coffee brb
13:15:13 gibi have a nice one
13:16:20 opendevreview Fabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call https://review.opendev.org/c/openstack/nova/+/910627
13:20:44 sean-k-mooney its 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 morning
13:21:03 opendevreview Fabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call https://review.opendev.org/c/openstack/nova/+/910627
13:21:27 sean-k-mooney so this is my first coffe of the day with a big dash of salted carammel, i normlaly drink it black
13:22:05 sean-k-mooney idd 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:35 gibi sean-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 mokka
13:23:33 sean-k-mooney ya i got an expresso machien about 3-4 years ago so i went form 2-3 nesspreo pods a day to one ammericano
13:23:49 sean-k-mooney turns out to be about the same amount of cafeine
13:54:06 artom Where's our opensearch/elasticsearch instance again?
13:55:28 artom https://opensearch.logs.openstack.org/_dashboards/app/ needs a login...
14:01:23 dansmith artom: openstack/openstack
14:01:31 sean-k-mooney ya that
14:01:32 dansmith apparently the new one can't be unauthenticated
14:03:38 sean-k-mooney artom: are you looking into my comment https://review.opendev.org/c/openstack/nova/+/926083/comments/ad6713dc_e6542156
14:04:37 artom sean-k-mooney, the shelve thing? No, I got an openstacksdk error in the latest results
14:04:45 sean-k-mooney ah
14:04:55 sean-k-mooney ya i saw that job fail in one of my other patches
14:05:07 sean-k-mooney i have not reeckec but it passed on the next patch in my series
14:05:10 artom Really trivial fix, I was just looking how often it happened
14:05:13 sean-k-mooney so i assume there s an itermint failure
14:05:34 artom Yeah, cloud.get returns None, and we unconditionally try to [ ] that
14:06:00 artom Seems like just skipping the None case with an if and retrying the loop iteration would help...
14:07:14 sean-k-mooney manuvakery1: that feels like something i have heard fo before but its obviously a qemu bug
14:07:56 sean-k-mooney i guess it could be in libvirt but its defnintly not in nova
14:08:47 sean-k-mooney well cloud.get is presumable reading form the clouds.yaml which should nto be none
14:09:49 artom I think it's an API call... it's polling for stack events
14:10:17 sean-k-mooney i didnt look at the actual errror yet
14:10:19 artom Sorry, cloud.get_stack() :)
14:10:34 artom Which I assume calls Heat
14:10:44 sean-k-mooney presumeable
15:11:07 opendevreview Fabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call https://review.opendev.org/c/openstack/nova/+/910627
16:01:41 dansmith sean-k-mooney: +2 now ^
17:04:35 sean-k-mooney dansmith: 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:05:10 dansmith well, tbh, I wasn't even going to consider it a feature
17:05:30 dansmith it's tiny and it's just using the glance api the right way,
17:05:39 dansmith but if you think it needs to be considered as such, then that's fine
17:05:52 dansmith not sure what that really means with bauzas gone next week though
17:06:09 sean-k-mooney well more i was suggeting its small enouch that we may want to consier not procedurlly blocking it
17:06:25 sean-k-mooney if the tempet issue can be resolved by the next team meeting
17:06:38 sean-k-mooney well FFE dont requrie PTL input
17:06:50 sean-k-mooney historically it only requried 3 cores to say yes
17:07:23 sean-k-mooney i only consider https://review.opendev.org/c/openstack/nova/+/891036 a feature in that we would not back port it
17:08:04 sean-k-mooney mainly because of the requirements bump but i agree otherwise its just using glances api the secure way
17:08:04 dansmith there's no point in backporting it
17:08:22 sean-k-mooney right because there is no supprot in older glance
17:08:27 dansmith I 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 unclear
17:08:58 dansmith anyway, we'll keep pushing on the tempest patch and can revisit on monday maybe
17:09:24 sean-k-mooney ack, either way i thin kthis could merge after rc1 if we decied to wait
17:09:29 dansmith the 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 it
17:09:32 sean-k-mooney having it 2025.1 is the main thing
17:10:10 sean-k-mooney ah ok so its not strictly a dep but we would have some instablity without it
17:10:34 dansmith right
17:11:01 sean-k-mooney ok well im going to checkout for the weekend
17:11:07 sean-k-mooney catch you monday o/
17:11:18 dansmith o/
18:39:29 opendevreview Jay Faulkner proposed openstack/nova master: [ironic] Factor out metadata and send to ironic https://review.opendev.org/c/openstack/nova/+/927639
19:38:14 opendevreview Merged openstack/nova stable/2023.1: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/912511
19:39:16 bauzas I'm still there, I have some slides to work on
19:59:18 JayF I 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:29 JayF s/C/D/
20:53:41 opendevreview Brian Haley proposed openstack/nova master: Support creating servers with RBAC SGs https://review.opendev.org/c/openstack/nova/+/811521
20:57:27 opendevreview Merged openstack/nova master: Add hw_firmware_stateless image property https://review.opendev.org/c/openstack/nova/+/926590
21:00:45 opendevreview Merged openstack/nova master: Cleanup setup.py https://review.opendev.org/c/openstack/nova/+/905419
22:18:48 melwitt JayF: 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 instead
22:19:18 melwitt *a new review instead
22:19:18 JayF egad
22:21:53 opendevreview Jay Faulkner proposed openstack/nova master: [ironic] Factor out metadata and send to ironic https://review.opendev.org/c/openstack/nova/+/923910
22:22:27 JayF I had to completely blow away the commit hook to keep it from trying to add a new one everytime
22:23:16 JayF thanks melwitt :)
22:24:05 melwitt hm, I guess it really requires Change-Id to be the last line of the commit message. but apparently not all the time (?)
22:24:36 JayF yeah I don't really understand, but I'm just glad you noticed so I could fix it :)
23:52:00 opendevreview Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876
23:52:01 opendevreview Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877
23:52:01 opendevreview Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844
23:52:02 opendevreview Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126
23:52:02 opendevreview Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354
#openstack-nova - 2024-08-31
01:46:16 manuvakery1 sean-k-mooney: it was due to newly added ceph mon host with a different weight. Eventhough the new mon hosts are not added in to ceph.conf it was impacting all vms which has the ceph backend/or ceph volume attached to it.
01:47:02 sean-k-mooney[m] the ceph mons are encoded in the libvirt xml
01:47:19 sean-k-mooney[m] so you cant change the ips or expand the set if you have active vms
02:52:38 melwitt bauzas: I just +W the ironic metadata patch with the understanding that FF was extended to today (friday)
04:42:24 opendevreview Merged openstack/nova master: [ironic] Factor out metadata and send to ironic https://review.opendev.org/c/openstack/nova/+/923910

Earlier   Later