Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-12-10
21:56:27 melwitt you can say that again
21:56:30 lyarwood there's so many fires at the moment, I wouldn't worry about it :)
21:56:49 melwitt last night I almost tweeted "openstack ci right now is" with a mushroom cloud gif
21:56:58 melwitt hah xD
21:57:48 melwitt or the gif of elmo with the fire behind him
21:59:57 lyarwood hehe yeah both would still work right now :D
22:00:16 lyarwood ah cool the stable/victoria LC change made it to the gate
22:00:55 melwitt hot dog!
22:06:29 melwitt lyarwood: hm something I don't get though is how the LC change is having nova-multi-cell as non-voting if the disable voting change isn't in the gate?
22:08:31 lyarwood melwitt: different branches, I wrote the LC change switching nova-multi-cell to non-voting before gibi pushed his master change
22:09:03 lyarwood melwitt: I was going to pull it out and backport squash his into the stable/victoria LC fix but he said to just go on with the non-voting change
22:10:09 lyarwood melwitt: either way it's only to allow the LC change to land in stable/victoria, we will need to follow up once it's in
22:10:24 melwitt lyarwood: yeah but https://review.opendev.org/c/openstack/nova/+/766440 isn't approved? maybe I'm just missing something dumb
22:11:02 lyarwood melwitt: oh I need to close that, I squashed that into the actual LC change, that's an old PS that isn't used anymore
22:11:02 melwitt oh you squashed it. sorry /facepalm
22:11:10 melwitt yeah I see now. guhhhhh
22:11:19 lyarwood yeah sorry I've got dead changes all over the place from the last few days
22:11:32 melwitt no it's not your fault that I can't see things :)
22:12:57 melwitt ok, so iiuc we'll do this and then after gibi's change lands, we need to backport it to stable/victoria, then we re-enable voting
22:14:01 lyarwood yeah I think so
22:14:40 lyarwood okay bed time for me now, good luck with the gate :D
22:15:55 melwitt ok cool, gnight o/
22:17:45 melwitt argh nova-ceph-multistore failed in the gate queue. of course it did
#openstack-nova - 2020-12-11
00:40:41 brinzhang gmann: ack, that's what I did, just change the List Security Groups By Server API, dont change the deprecated apis.
00:40:44 brinzhang gmann: thanks
00:41:07 gmann brinzhang: +1, I will check your patches tomorrow.
00:42:32 brinzhang gmann: yeah, thanks, I have to add the needed unit tests, then submit it, and add you as a reviewer
00:42:57 gmann cool
06:59:02 openstackgerrit zhulipeng proposed openstack/nova stable/pike: Fixed multipath residue when evacuating the VM https://review.opendev.org/c/openstack/nova/+/766675
08:05:13 gibi good morning
08:06:53 gibi melwitt: thanks for the babysitt
08:08:59 gibi I see we were not lucky so far
08:27:19 bauzas gibi: good morning
08:29:16 gibi bauzas: o/
09:02:33 gibi https://review.opendev.org/c/openstack/nova/+/765749 is in the gate queue (again), fingers crossed
09:52:04 elod lyarwood: thanks for those backports \o/ we'll see whether everything is happy with it :) so there are now 3 possible solution for the bandit issue, so hopefully it'll be sorted out soon!
09:53:28 elod lyarwood: btw, I'm kinda @ PTO today and monday, but I'll be here time to time and try to look at things
11:22:58 hemanth_n Hi Nova team, can i get reviews on ussuri backport https://review.opendev.org/c/openstack/nova/+/761701 (Had initial review from elod with +2)
11:25:44 lyarwood hemanth_n: ack will look shortly
11:26:46 hemanth_n lyarwood: thanks
11:28:45 gibi lyarwood: https://review.opendev.org/c/openstack/nova/+/765749 bounced from the gate again
11:29:23 gibi with the grande job failing on test_live_block_migration_paused
11:37:50 openstackgerrit Kashyap Chamarthy proposed openstack/nova-specs master: Re-propose "Secure Boot support for KVM & QEMU guests" for Wallaby https://review.opendev.org/c/openstack/nova-specs/+/759731
11:38:25 kashyap gibi: stephenfin: --^ Addressed the small comments from PTG and review (adding the bit about traits; and adjusting version info, and couple of minor bits)
11:39:30 kashyap (Cc: Sean, Bauzas. Previous reviewers :-))
11:39:38 lyarwood gibi: gah, that's still on bionic iirc,
11:39:58 lyarwood gibi: iirc I had see quite a few failures recently like that on master, I'll write up an ER query shortly and check
11:45:27 sean-k-mooney yep test_live_block_migration_paused has been failing on master
11:46:01 lyarwood just in the grenade job however right?
11:46:07 lyarwood I don't think the focal job is hitting this
11:46:12 sean-k-mooney yes i think so
11:46:48 lyarwood hemanth_n: https://review.opendev.org/c/openstack/nova/+/761701 LGTM but the gate is blocked at the moment, I'm going to +2 and follow up with a +W once it's working again
11:47:03 sean-k-mooney lyarwood: did you see my review comments
11:47:16 lyarwood sean-k-mooney: nope
11:47:31 sean-k-mooney lyarwood: it looks fine but i think it would be cleaner to pull in the two missing trivial refactor patches
11:47:45 lyarwood sean-k-mooney: ack, I can help with that
11:48:06 hemanth_n sean-k-mooney: do you want them backport separately or as part of this patch
11:48:33 sean-k-mooney hemanth_n: seperatly
11:48:43 sean-k-mooney https://github.com/openstack/nova/commit/17ba10b75961ef667e45b9dedbac34212a7e7e9b and https://github.com/openstack/nova/commit/0ab8a03c782851d4019c90e6159bd1f83c0658e6
11:48:53 sean-k-mooney well as in as seperate patches
11:48:58 sean-k-mooney but backprot the 3 together
11:50:09 hemanth_n sean-k-mooney: ack, lyardwood: i am my eod already, if you want me to backport other patches i can do them on monday
11:50:10 sean-k-mooney the order shoudl be 0ab8a03c782851d4019c90e6159bd1f83c0658e6 then 17ba10b75961ef667e45b9dedbac34212a7e7e9b then your patch
11:51:15 lyarwood hemanth_n: ack, I can try to do this later today for you
11:51:31 hemanth_n sean-k-mooney, lyardwood: thanks for the quick responses once again
12:06:33 openstackgerrit Lee Yarwood proposed openstack/nova master: nova-grenade-multinode: Skip test_live_block_migration_paused https://review.opendev.org/c/openstack/nova/+/766720
12:07:04 lyarwood gibi: ^ lets skip that test until multinode grenade moves to zuulv3 and Focal
12:08:59 brinzhang hi team, when do we support GET /allocation_candidates?resources=CUSTOM_PMEM_NAMESPACE_xxGB? in nova and placement?
12:09:52 brinzhang I know support this when we accept the PMEM support patch in Nova, but Icannot find the placement changes, anyone know?
12:11:49 sean-k-mooney brinzhang: there was no placment change required
12:11:51 brinzhang we hit an error: GET /allocation_candidates?limit=1000&resources=CUSTOM_PMEM_NAMESPACE_10GB%3A1%2CDISK_GB%3A30%2CMEMORY_MB%3A2048%2CVCPU%3A2 HTTP/1.1" 400
12:12:49 sean-k-mooney brinzhang: what was the actul error in the placement logs
12:13:55 sean-k-mooney its asking for CUSTOM_PMEM_NAMESPACE_10GB:1,DISK_GB:30,MEMORY_MB:2048,VCPU:24
12:14:15 sean-k-mooney that looks like a perfectly normal request to me
12:14:24 stephenfin brinzhang: what's configured in '[libvirt] pmem_namespaces' in nova.conf? You should have an entry like 'pmem_namespaces = 10GB:ns0,...
12:14:33 brinzhang sean-k-mooney: please see: Invalid resource class in resources parameter: No such resource class CUSTOM_PMEM_NAMESPACE_30GB
12:14:52 stephenfin then nova isn't reporting the inventory
12:15:14 sean-k-mooney yep nova should create the inventory and register teh resouce class
12:15:37 stephenfin either you don't have it configured, your configuration is wrong, or you're using nova < Train
12:15:45 brinzhang stephenfin: pmem_namaspaces = 10GB:ns10_0|ns10_1|ns10_2|ns10_3,30GB:ns30_0|ns30_1|ns30_2|ns30_3
12:15:54 brinzhang this is our config in nova.conf
12:16:31 brinzhang stephenfin: does the config is right?
12:17:13 stephenfin it looks okay
12:17:16 stephenfin and what does 'openstack resource provider inventory list $COMPUTE_ID' show?
12:17:18 sean-k-mooney based on the docs yes
12:18:05 sean-k-mooney brinzhang: you precreated the namespaces right?
12:18:15 sean-k-mooney you have to do that manually nova will not do that for you
12:18:34 brinzhang yes, we precreated the namespaces manully
12:19:00 brinzhang s/manully/manually
12:19:01 sean-k-mooney ok then ya can you provide the output of the command stephenfin asked for
12:19:11 brinzhang let my try
12:20:37 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: DNM - Testing https://review.opendev.org/c/openstack/devstack/+/766622 https://review.opendev.org/c/openstack/nova/+/766723
12:21:00 sean-k-mooney if the inventories are not present you need to check the nova comptue agent to see if the update_avaiable_resoue periodic task is failing and if so why
12:22:11 gibi lyarwood: +2 on the skipping the failing live migration test
12:37:53 brinzhang sean-k-mooney, stephenfin: yes, it's empty, Ishould check the update_avaiable_resoue periodic task, thanks.
12:38:11 brinzhang I will check it later ^
12:44:44 openstackgerrit Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List SG API https://review.opendev.org/c/openstack/nova/+/766726
12:45:16 openstackgerrit Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List SG API https://review.opendev.org/c/openstack/nova/+/766726
12:50:44 openstackgerrit Alexandre arents proposed openstack/nova master: libvirt: make cross cell resize spawn from snapshot image https://review.opendev.org/c/openstack/nova/+/765561
12:53:18 openstackgerrit Merged openstack/nova master: Disable qos resize tempest test for nova-multi-cell job https://review.opendev.org/c/openstack/nova/+/766471

Earlier   Later