| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-12-10 | |||
| 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 | melwitt | oh you squashed it. sorry /facepalm | |
| 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: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 | |
| 12:59:49 | gibi | ^^ \o/ | |
| 13:02:47 | openstackgerrit | Alexandre arents proposed openstack/nova master: libvirt: make cross cell resize spawn from snapshot image https://review.opendev.org/c/openstack/nova/+/765561 | |
| 13:08:15 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: nova-grenade-multinode: Skip test_live_block_migration_paused https://review.opendev.org/c/openstack/nova/+/765749 | |
| 13:08:50 | lyarwood | whops | |
| 13:17:33 | brinzhang | gibi, stephenfin, sean-k-mooney: hope you can review cyborg shelve/unshelve support https://review.opendev.org/c/openstack/nova/+/729563 | |
| 13:20:26 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: lower-constraints: Bump packaging to 20.4 https://review.opendev.org/c/openstack/nova/+/765749 | |
| 13:25:13 | lyarwood | brb in 30 | |