| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-16 | |||
| 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? | |
| 11:50:13 | lyarwood | bauzas: https://review.opendev.org/#/c/702021/ & https://review.opendev.org/#/c/711679/ for example | |
| 11:52:46 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: Avoid spurious error logging in _get_compute_nodes_in_db https://review.opendev.org/702902 | |
| 11:53:03 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: Reject boot request for unsupported images https://review.opendev.org/708577 | |
| 12:46:32 | bauzas | lyarwood: sorry, was at lunch, but indeed, everything can be accepted, I'm not the kind of person who tell you at the door 'sorry, but we can't accept you because of your shoes' | |
| 12:47:03 | stephenfin | lyarwood: those are dodgy shoes though | |
| 13:09:01 | lyarwood | stephenfin: white trainers ftw | |
| 13:11:21 | bauzas | lyarwood: get off my lawn ! | |
| 13:11:32 | bauzas | :p | |
| 13:12:44 | bauzas | heh, s/barf/bark of course | |
| 13:12:57 | bauzas | (pardon my French (c) ) | |
| 13:21:09 | kashyap | lyarwood: Zuul (grenade-py3) failure on this: https://review.opendev.org/#/c/702021/ | |
| 13:21:23 | kashyap | If you've already seen it, disregard me. | |
| 13:24:58 | lyarwood | kashyap: yeah unrelated | |
| 13:25:07 | lyarwood | thanks for looking | |
| 13:26:57 | kashyap | bauzas: FWIW, this is straightforward to just merge this: https://review.opendev.org/#/c/702021/ (libvirt: Remove VIR_DOMAIN [...]) | |
| 13:28:43 | kashyap | (And the 'qemu-img' one, too.) | |
| 14:47:49 | gibi | lyarwood, bauzas, stephenfin: Am I missing something here https://review.opendev.org/#/c/711679/5/nova/virt/images.py@42 ? | |
| 14:49:31 | stephenfin | Oh, it looks like it | |
| 14:50:02 | lyarwood | gibi: nope it's unused | |
| 14:50:25 | sean-k-mooney | well its used here https://review.opendev.org/#/c/711679/5/nova/virt/images.py@48 | |
| 14:50:37 | sean-k-mooney | but i dont know if its everset to anything other then json | |
| 14:50:54 | bauzas | gibi: lyarwood: wait, it can be | |
| 14:51:01 | bauzas | sec, finding the github link | |
| 14:51:49 | bauzas | gibi: lyarwood: stephenfin: https://github.com/openstack/nova/blob/e1359567e4985e9a671359d4c0d53404a8ba64ab/nova/virt/libvirt/utils.py#L224 | |
| 14:52:14 | stephenfin | bauzas: that's format, not output_format | |