| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-04-01 | |||
| 20:29:53 | sean-k-mooney | you technially have extra () | |
| 20:29:59 | sean-k-mooney | that you dont need | |
| 20:30:15 | sean-k-mooney | equest = next((r for r in instance.pci_requests.requests | |
| 20:30:17 | sean-k-mooney | if r.requester_id == port['id'])) | |
| 20:30:37 | artom | No, that's needed | |
| 20:30:40 | artom | Wait, is it? | |
| 20:30:44 | artom | Maybe it isn't... | |
| 20:30:54 | artom | What you pass to next() has to be an interator | |
| 20:30:55 | sean-k-mooney | you are creating a touple | |
| 20:31:04 | artom | *iterator, not a list/tuple | |
| 20:31:07 | sean-k-mooney | but you can pass the generator expression driectly to next | |
| 20:31:38 | sean-k-mooney | iterator or generator | |
| 20:31:58 | sean-k-mooney | "r for r in instance.pci_requests.requests" is a generator expression | |
| 20:32:08 | artom | Oh yeah, works without the extra ( ) | |
| 20:32:11 | artom | TIL | |
| 20:32:36 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Update SRIOV port pci_slot when unshelving https://review.opendev.org/c/openstack/nova/+/784168 | |
| 20:32:36 | sean-k-mooney | with the extra () its doing a touple comprehention and then it taking the first element | |
| 20:32:53 | sean-k-mooney | without it its iterating the generator untile the first value is yeilded | |
| 20:33:20 | sean-k-mooney | so its faster too since it will early exit | |
| 20:35:03 | sean-k-mooney | artom: thats a python 3 thing by the way | |
| 20:35:20 | artom | Oh | |
| 20:35:21 | artom | Hrmm | |
| 20:35:34 | artom | Well, no, it's fine, Train is Python3 | |
| 20:35:36 | sean-k-mooney | its easy to add them back when you get to train | |
| 20:35:44 | sean-k-mooney | its both | |
| 20:35:52 | artom | Ah, it's OSP16 that's py3-only | |
| 20:35:58 | artom | Because RHEL8 | |
| 20:36:03 | sean-k-mooney | ussuri was the first one that is pyton 3 only | |
| 20:36:07 | sean-k-mooney | yes | |
| 20:36:21 | sean-k-mooney | which is proably the only time rhel has done someting first :) | |
| 20:37:16 | artom | *snerk* | |
| 20:40:19 | openstackgerrit | sean mooney proposed openstack/nova stable/train: add functional regression test for bug #1888395 https://review.opendev.org/c/openstack/nova/+/759533 | |
| 20:40:21 | openstack | bug 1888395 in OpenStack Compute (nova) train "live migration of a vm using the single port binding work flow is broken in train as a result of the introduction of sriov live migration" [High,In progress] https://launchpad.net/bugs/1888395 - Assigned to Billy Olsen (billy-olsen) | |
| 20:40:22 | openstackgerrit | sean mooney proposed openstack/nova stable/train: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/c/openstack/nova/+/759153 | |
| #openstack-nova - 2021-04-02 | |||
| 03:00:08 | openstackgerrit | melanie witt proposed openstack/nova master: Remove unused DeleteFromSelect db api method https://review.opendev.org/c/openstack/nova/+/784528 | |
| 03:07:49 | yonglihe | alex_xu, we reproposal smart nic spec to this release, needs your bandwidth for that, and whole refactored codes plus fucntional test: https://review.opendev.org/c/openstack/nova-specs/+/783632 | |
| 05:39:36 | alex_xu | yonglihe: yea, that is in my list | |
| 05:39:52 | yonglihe | thanks, | |
| 06:08:19 | openstackgerrit | Wenping Song proposed openstack/nova-specs master: Trival change: correct some nits https://review.opendev.org/c/openstack/nova-specs/+/780443 | |
| 06:10:56 | openstackgerrit | Wenping Song proposed openstack/nova-specs master: Trival change: correct some nits https://review.opendev.org/c/openstack/nova-specs/+/780443 | |
| 21:55:54 | openstackgerrit | sean mooney proposed openstack/nova stable/train: add functional regression test for bug #1888395 https://review.opendev.org/c/openstack/nova/+/759533 | |
| 21:55:56 | openstack | bug 1888395 in OpenStack Compute (nova) train "live migration of a vm using the single port binding work flow is broken in train as a result of the introduction of sriov live migration" [High,In progress] https://launchpad.net/bugs/1888395 - Assigned to Billy Olsen (billy-olsen) | |
| 22:00:22 | openstackgerrit | sean mooney proposed openstack/nova stable/train: add functional regression test for bug #1888395 https://review.opendev.org/c/openstack/nova/+/759533 | |
| 22:00:23 | openstack | bug 1888395 in OpenStack Compute (nova) train "live migration of a vm using the single port binding work flow is broken in train as a result of the introduction of sriov live migration" [High,In progress] https://launchpad.net/bugs/1888395 - Assigned to Billy Olsen (billy-olsen) | |
| 22:02:41 | openstackgerrit | sean mooney proposed openstack/nova stable/train: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/c/openstack/nova/+/759153 | |
| #openstack-nova - 2021-04-03 | |||
| 11:23:21 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Update contributor guide for Xena https://review.opendev.org/c/openstack/nova/+/783534 | |
| 13:27:14 | openstackgerrit | Vlad Gusev proposed openstack/nova stable/stein: Correctly disable greendns https://review.opendev.org/c/openstack/nova/+/784581 | |
| #openstack-nova - 2021-04-06 | |||
| 01:34:09 | openstackgerrit | Kevin Zhao proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/c/openstack/nova/+/784870 | |
| 01:37:20 | openstackgerrit | Kevin Zhao proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/c/openstack/nova/+/714439 | |
| 01:41:33 | openstackgerrit | Kevin Zhao proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/c/openstack/nova/+/714439 | |
| 03:28:14 | openstackgerrit | Nobuhiro MIKI proposed openstack/nova master: libvirt: update the backing file path during hard_reboot https://review.opendev.org/c/openstack/nova/+/784875 | |
| 07:03:30 | gibi | good morning | |
| 07:20:08 | bauzas | good morning folks | |
| 07:20:30 | bauzas | my kids are at home as the school is closed, so today I'll be a bit off this morning | |
| 07:21:23 | gibi | bauzas: ack | |
| 08:10:34 | gibi | yonglihe: brinzhang_: hi! do you happen to know if the cyborg team has any topics for the PTG that would like to discuss with the nova team? If there is any then we can try to find a timeslot in the schedule to meet | |
| 08:13:19 | brinzhang_ | gibi: as I know, we have not any topic need to be talk at this PTG, for example, the cybrog vGPU support we have an AGGREMENT but less of implementation at Wallaby, just need to re-propose | |
| 08:17:34 | brinzhang_ | gibi: I will check with yonglihe and xinran, is there something need to be discuss with smartNIC, later ping you again^ | |
| 08:23:34 | brinzhang_ | gibi: I have checked with xinranwang, we dont have any topics with smarNIC for the PTG, thanks | |
| 08:23:47 | gibi | brinzhang_: I looked through the current nova PTG etherpad and got the same feeling we have previous agreements with cyborg around features | |
| 08:24:03 | gibi | brinzhang_: thanks for checking | |
| 08:24:34 | brinzhang_ | gibi: yes, thanks too | |
| 08:24:46 | gibi | then I will not dedicate a slot for cyborg in the nova PTG, if anything comes up then we can take them in the normal nova sessions | |
| 08:27:11 | brinzhang_ | gibi: yeah, if there is anything we will take them here ^ | |
| 08:28:25 | gibi | cool | |
| 09:03:26 | openstackgerrit | Kevin Zhao proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/c/openstack/nova/+/714439 | |
| 09:04:51 | gibi | bauzas: when you are back, there is a fresh vgpu bugrepot in LP https://bugs.launchpad.net/nova/+bug/1922264 I think you have better context for that than me | |
| 09:04:52 | openstack | Launchpad bug 1922264 in OpenStack Compute (nova) "On a compute node with 3 GPUs and 2 vgpu groups, nova fails to load second group" [Undecided,New] | |
| 09:05:02 | bauzas | gibi: I'm here | |
| 09:05:09 | bauzas | looking | |
| 09:06:53 | gibi | thanks | |
| 09:07:52 | bauzas | gibi: argh, looks like we don't create the conf groups | |
| 09:08:30 | gibi | dansmith: when you are up, there is a fresh glance multistore related bug in LP https://bugs.launchpad.net/nova/+bug/1922251 I guess you have better context around this feature than me so If you have time please check the bug | |
| 09:08:39 | openstack | Launchpad bug 1922251 in OpenStack Compute (nova) "nova instance snapshots fail when using rbd" [Undecided,New] | |
| 09:10:15 | gibi | bauzas: the warning message in the bugreport seems confusing | |
| 09:10:45 | bauzas | gibi: that's because we create the opt groups before | |
| 09:10:59 | bauzas | I'm looking at the source code | |
| 09:13:32 | lyarwood | gibi: I can take a look at the rbd issue | |
| 09:18:41 | gibi | lyarwood: thanks appreciate it | |
| 09:40:50 | gibi | bauzas: I tried to reproduce the vgpu config bug in wallaby devstack by simply adding the config from the bugreport to my nova-cpu.conf but for me the config groups are created properly | |
| 09:45:10 | bauzas | gibi: that's normal, this is fixed in Wallaby | |
| 09:45:10 | bauzas | https://bugs.launchpad.net/nova/+bug/1922264 | |
| 09:45:13 | openstack | Launchpad bug 1900006 in OpenStack Compute (nova) "duplicate for #1922264 Asking for different vGPU types is racey" [Medium,Fix released] - Assigned to Sylvain Bauza (sylvain-bauza) | |
| 09:45:21 | bauzas | I closed it as duplicate of ^ | |
| 09:45:48 | bauzas | and I'll backport https://review.opendev.org/c/openstack/nova/+/758470 to stable/victoria later this afternoon | |
| 09:46:10 | bauzas | looks like we forgot (at least me) to backport it | |
| 09:46:30 | gibi | bauzas: I've just found the same :) | |
| 09:46:53 | gibi | bauzas: thanks for mark it as duplicated | |
| 09:47:47 | gibi | stops even | |
| 09:47:55 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Replace grenade and nova-grenade-multinode with grenade-multinode https://review.opendev.org/c/openstack/nova/+/778885 | |
| 09:47:55 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Remove nova-dsvm-multinode-base https://review.opendev.org/c/openstack/nova/+/778908 | |
| 09:47:56 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Create and use nova-libvirt-irrelevant-files https://review.opendev.org/c/openstack/nova/+/778909 | |
| 09:49:19 | openstackgerrit | Sylvain Bauza proposed openstack/nova stable/victoria: Fix the vGPU dynamic options race https://review.opendev.org/c/openstack/nova/+/784907 | |
| 09:49:23 | bauzas | gibi: ^ | |
| 09:49:36 | bauzas | (gym) | |
| 09:49:46 | gibi | bauzas: thanks | |
| 10:07:32 | kashyap | Does anyone here know if Vexxhost enables nested KVM on _all_ their hardware that is provided to OpenStack CI? | |
| 10:07:37 | kashyap | Probably a question for -infra | |
| 10:27:14 | lyarwood | gibi: https://review.opendev.org/c/openstack/nova/+/775575 - if you're in the mood for reviewing some trivial zuul changes | |
| 10:29:41 | gibi | lyarwood: I'm :) | |