| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-16 | |||
| 08:13:10 | bauzas | gibi: -EINTR | |
| 08:16:07 | bauzas | sem_wait(*bugs) | |
| 08:19:06 | openstackgerrit | Merged openstack/nova master: Fix server actions to be system and project scoped https://review.opendev.org/720123 | |
| 08:28:56 | gibi | bauzas: before I lost the lock I was looking at https://bugs.launchpad.net/nova/+bug/1861493 | |
| 08:28:56 | openstack | Launchpad bug 1861493 in OpenStack Compute (nova) "Nova sends an "X-Service-Token" header when "send_service_user_token" is disabled" [Undecided,New] | |
| 08:29:13 | openstack | Launchpad bug 1863844 in OpenStack Compute (nova) " The virtual machine with the config_driver device cannot properly mount the disk using the virtio-scsi driver" [Undecided,New] | |
| 08:29:34 | bauzas | sorry, I'm really not a volume expert :( | |
| 08:29:41 | bauzas | gibi: ack ok | |
| 08:30:08 | gibi | me neither | |
| 08:30:45 | bauzas | lyarwood: not sure you're up yet but could you tell me whether https://bugs.launchpad.net/nova/+bug/1863844 is legit ? | |
| 08:30:45 | openstack | Launchpad bug 1863844 in OpenStack Compute (nova) " The virtual machine with the config_driver device cannot properly mount the disk using the virtio-scsi driver" [Undecided,New] | |
| 08:33:04 | openstackgerrit | Ivaylo Mitev proposed openstack/nova master: Faults from cell DB missing in GET /servers/detail https://review.opendev.org/699176 | |
| 08:37:34 | lyarwood | bauzas: morning, yeah it's valid and I think I know why | |
| 08:37:51 | bauzas | lyarwood: k, no regression hopefully ? | |
| 08:38:34 | lyarwood | bauzas: actually they set --property hw_scsi_model='virtio-scsi' that I assumed they had missed | |
| 08:38:45 | lyarwood | bauzas: I've reproduced it FWIW with q35 | |
| 08:38:58 | lyarwood | bauzas: -> coffee and I'll look closer | |
| 08:39:18 | bauzas | ack, gtk | |
| 08:40:12 | bauzas | and thanks | |
| 09:01:51 | openstackgerrit | Ivaylo Mitev proposed openstack/nova master: Faults from cell DB missing in GET /servers/detail https://review.opendev.org/699176 | |
| 09:38:01 | bauzas | argh, I'm surprised to see the virtual PTG planned for only one single week... | |
| 09:38:16 | bauzas | I thought we had an agreement on having more than just one week | |
| 09:38:19 | bauzas | :/ | |
| 09:38:58 | hrw | lyarwood: I am waiting for x86 world to migrate to q35/uefi... | |
| 09:40:24 | lyarwood | hrw: trying to for V | |
| 09:40:35 | lyarwood | hrw: it's on my plate at least | |
| 09:41:14 | openstackgerrit | Dincer Celik proposed openstack/nova stable/train: Abort live-migration during instance_init https://review.opendev.org/720414 | |
| 09:46:53 | hrw | :) | |
| 09:57:04 | nightmare_unreal | can someone follow up on this : https://review.opendev.org/#/c/715395/ | |
| 09:57:08 | nightmare_unreal | Thanks :) | |
| 10:08:11 | bauzas | lyarwood: fwiw, i'm about to classify https://bugs.launchpad.net/nova/+bug/1862375 as invalid/wishlist since it seems an improvement | |
| 10:08:11 | openstack | Launchpad bug 1862375 in OpenStack Compute (nova) "Subsequent nova-api volume attach request waiting for previous one to complete" [Undecided,Invalid] | |
| 10:08:32 | bauzas | (for the timeout, i'll explain the option) | |
| 10:10:30 | gibi | bauzas: I agree and raced with you on that. sorry. I forget to take the lock. I'm stopping now | |
| 10:11:28 | bauzas | gibi: all cool | |
| 10:13:21 | lyarwood | bauzas / gibi ; meh, the whole device naming thing is pointless and in an ideal world should be removed | |
| 10:13:52 | gibi | lyarwood: removing that would need an API change as nova returns the device name in the POST rsp | |
| 10:13:53 | lyarwood | bauzas / gibi ; the computes actually serialise these requests anyway so we could remove this in the future | |
| 10:15:09 | lyarwood | gibi: right, we could remove it for virt drivers that can't guarantee it with a microversion right? | |
| 10:15:17 | lyarwood | gibi: such as libvirt | |
| 10:15:26 | lyarwood | actually can any virt drivers guarantee it | |
| 10:17:37 | gibi | I'm fine removing that in a microversion but that needs spec | |
| 10:18:09 | gibi | and I cannot really comment on the virt driver side so I trust you | |
| 10:20:15 | bauzas | huh, unless I misunderstand something, a snapshot is not intended to be written again by the instance that was snapshoted ? https://bugs.launchpad.net/nova/+bug/1860913 | |
| 10:20:15 | openstack | Launchpad bug 1860913 in OpenStack Compute (nova) "Instance uses base image file when it is rebooted after snapshot creation if cinder nfs backend is used" [Undecided,New] | |
| 10:20:22 | bauzas | so, Invalid, right ? ^ | |
| 10:20:48 | bauzas | a snapshot is by definition immutable | |
| 10:23:10 | lyarwood | the bug suggests we are writing to the snapshot and that would be a bug | |
| 10:23:21 | lyarwood | it might be that they are just looking at the QEMU command line that isn't updated | |
| 10:27:26 | bauzas | I'll ask them more then => Incomplete | |
| 10:59:52 | hrw | is there a way to set cpu_mode/cpu_model in CI jobs? | |
| 11:16:56 | sean-k-mooney | hrw: you can just set it in the devstack local.conf | |
| 11:17:47 | hrw | sean-k-mooney: thanks | |
| 11:17:58 | sean-k-mooney | hrw: like this https://opendev.org/x/networking-ovs-dpdk/src/branch/master/.zuul.yaml#L72-L86 | |
| 11:18:26 | hrw | great! | |
| 11:18:45 | sean-k-mooney | set cpu_mode: custome then cpu_model: whatever | |
| 11:18:55 | hrw | yay | |
| 11:19:15 | sean-k-mooney | also spelling custom correctly helps :) | |
| 11:19:39 | hrw | nova-next has it too ;d | |
| 11:20:16 | sean-k-mooney | im guessing its using an old model | |
| 11:21:22 | sean-k-mooney | oh you just ment setting config values https://github.com/openstack/nova/blob/master/.zuul.yaml#L184-L195 | |
| 11:22:19 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/714439 | |
| 11:22:20 | hrw | let's check ;d | |
| 11:22:57 | hrw | now it should be able to run VM instances | |
| 11:23:07 | hrw | ~curse aarch64 for lack of nested virt | |
| 11:23:10 | openstackgerrit | Wenping Song proposed openstack/nova master: Accurately clean up ARQs resources during build_instances() in conductor https://review.opendev.org/720439 | |
| 11:23:43 | sean-k-mooney | oh nova next has bandwith aware schulding configured resource_provider_bandwidths: br-ex:1000000:1000000 | |
| 11:24:01 | sean-k-mooney | i did not know we were running tempest test for that in nova next | |
| 11:24:12 | gibi | sean-k-mooney: we have qos tempet tests | |
| 11:24:20 | gibi | tempest | |
| 11:24:58 | sean-k-mooney | gibi: i just did not know we were runing them in the gate. is there a reason to only run those in nova-next instead of all nova tempest jobs | |
| 11:26:19 | sean-k-mooney | i gues it does not matter it should noe be affaced by say using ceph image backend or qcow | |
| 11:26:25 | sean-k-mooney | so nova next is fine | |
| 11:26:31 | gibi | sean-k-mooney: I think the original reason was to see if they are stable enoug | |
| 11:26:58 | sean-k-mooney | well nova next is voting so if they would fail it would fail the ci run anyway | |
| 11:27:14 | gibi | good point | |
| 11:27:56 | sean-k-mooney | we dont really have a nova base job anymore that is based on zull v3 so without that it would be annoying to configure this in all the job anyway so its fine | |
| 11:28:13 | gibi | yeah, it needs some zuul config to enable the tests | |
| 11:28:24 | gibi | I can play with it after Ussuri is done | |
| 11:28:49 | sean-k-mooney | this move operation support for this land this cycle or is that still pending | |
| 11:28:55 | sean-k-mooney | i kind of lost track of that | |
| 11:30:38 | sean-k-mooney | did you also need the allocation like we do for vgpus https://review.opendev.org/#/q/topic:bug/1778563+(status:open+OR+status:merged) | |
| 11:38:28 | bauzas | gibi: I'm done for today with bug triage, we're down to 69 | |
| 11:38:46 | bauzas | nothing really urgent afaics | |
| 11:39:06 | bauzas | any other reviews needed ? should I say | |
| 11:39:55 | bauzas | sean-k-mooney: hopefully, only a very few server actions miss allocations I think | |
| 11:40:04 | bauzas | and that being said... | |
| 11:40:07 | gibi | sean-k-mooney: support for move operations landed in Ussuri (tempest test is open) | |
| 11:40:20 | bauzas | gibi: sean-k-mooney: stephenfin: I'd indeed appreciate reviews of https://review.opendev.org/#/q/topic:bug/1778563+(status:open+OR+status:merged) | |
| 11:40:32 | gibi | sean-k-mooney: I don't think we need any extra support from the virt dirver side for qos | |
| 11:40:50 | gibi | bauzas: thanks for the triage. | |
| 11:41:00 | gibi | bauzas: and yes, I still have your patch series open | |
| 11:41:05 | bauzas | gibi: I'll continue tomorrow morning | |
| 11:41:09 | gibi | bauzas: thanks | |
| 11:41:32 | bauzas | but I think I'll open my review dashboard for bugs and see what to chime in | |
| 11:41:51 | bauzas | folks, a nova-core is looking for bugfixes to review, please hassle him <= | |
| 11:42:00 | bauzas | err, | |
| 11:42:06 | bauzas | <= please hassle him | |
| 11:43:02 | gibi | :) | |
| 11:50:01 | lyarwood | bauzas: do you also take trivial cleanups and fups? | |