| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-12-17 | |||
| 08:57:39 | gibi | that is your present from Zuul :D | |
| 08:59:13 | bauzas | well, given how I battled with Zuul for this change, I'm more like this https://www.youtube.com/watch?v=YfpDSNNgYhI | |
| 09:01:28 | gibi | hehe, that is very vacation specific movie at least in my country :) | |
| 09:01:36 | gibi | almost like home alone | |
| 09:06:31 | bauzas | agreed, same here | |
| 09:07:15 | bauzas | I don't expect Christmas periods without Die Hard 2 or 1 movies being broadcasted on at least one of them | |
| 09:07:35 | bauzas | the 2 gets more broadcasted tho | |
| 09:09:31 | bauzas | wow, TIL the reasoning behind https://www.youtube.com/watch?v=KAYIgEpds_Y | |
| 09:09:54 | bauzas | the French dubbing totally trampled this phrase | |
| 09:20:10 | gibi | bauzas: I will get you the hungarian dub after my morning call | |
| 09:20:11 | gibi | :) | |
| 10:03:55 | gibi | bauzas: https://www.youtube.com/watch?v=BeH-ODa-41o | |
| 10:04:19 | gibi | interestingly there is two different hun dub for die hard 1 | |
| 10:08:36 | gibi | gmann, sean-k-mooney, melwitt, bauzas, brinzhang: I'm OK with the proposal in the API etherpad. I'm glad we found an agreeable way forward | |
| 10:08:46 | bauzas | me too | |
| 10:08:55 | bauzas | fwiw, I just modified my change | |
| 10:09:16 | bauzas | https://review.opendev.org/c/openstack/nova/+/749068/5/nova/tests/functional/integrated_helpers.py | |
| 10:09:34 | bauzas | but I'll split this hairy change into multiple pieces now that everything works | |
| 10:09:56 | bauzas | gibi: sharpen your pen, hopefully will upload the series later the day | |
| 10:10:14 | gibi | bauzas: sure, I will try to review it before I go | |
| 10:10:31 | bauzas | no rushes, we're half the cycle | |
| 10:10:42 | bauzas | but I'm glad this works, as proven by the functests | |
| 10:12:19 | gibi | I'm happy too, it was something I promised making happen in the past so I'm happy to sink time on reviewing it | |
| 10:15:49 | stephenfin | gibi: sorry, was out yesterday. Looking now | |
| 10:16:04 | gibi | stephenfin: ack, no worries | |
| 10:16:18 | stephenfin | in the meantime, bauzas/gibi: any chance you'd look at https://review.opendev.org/c/openstack/nova-specs/+/765796 to sync spec and code? | |
| 10:16:19 | brinzhang | gibi: yeah, me too, happy to see the ethpad for clean multi apis ^^ | |
| 10:16:44 | gibi | stephenfin: on it | |
| 10:16:52 | bauzas | stephenfin: kids taxi, but I'll look | |
| 10:17:32 | gibi | bauzas: you don't have to, gmann and I took care of it now | |
| 10:17:42 | bauzas | kk, ttyl | |
| 10:18:56 | gibi | lyarwood: I'm look at another possible race in tempest about volume handling https://zuul.opendev.org/t/openstack/build/fce0115deae94e38aaef4a349e271e20/log/job-output.txt#69070 | |
| 10:19:03 | stephenfin | Thanks! There's also a part 3, but that's got an open question on it from sean-k-mooney. I wonder if we should discuss in the team meeting today? https://review.opendev.org/c/openstack/nova-specs/+/765797/1/specs/wallaby/approved/modernize-os-hypervisors-api.rst | |
| 10:21:07 | lyarwood | gibi: oh fun | |
| 10:21:15 | lyarwood | gibi: let me take a look | |
| 10:25:51 | elod | lyarwood, similarly like yesterday: can you hit now the bandit patch in queens: https://review.opendev.org/c/openstack/nova/+/767364 o:) | |
| 10:28:40 | lyarwood | elod: ack will do | |
| 10:28:50 | openstackgerrit | Merged openstack/nova-specs master: Update modernize-os-hypervisors-api spec, pt. 2 https://review.opendev.org/c/openstack/nova-specs/+/765796 | |
| 10:30:46 | openstackgerrit | Alexandre arents proposed openstack/nova master: Revert "nova-multi-cell: Skip test_cold_migrate_unshelved_instance" https://review.opendev.org/c/openstack/nova/+/765933 | |
| 10:31:58 | zigo | bauzas: Hi man! | |
| 10:31:58 | zigo | One quick question about GPU in passthrough mode: what's the reason for moving from "type-PCI" to "type-PF" ? | |
| 10:32:18 | zigo | We had the issue when we upgraded from Rocky to Train ... | |
| 10:37:10 | bauzas | zigo: that depends on which GPU card you have | |
| 10:37:34 | bauzas | if it supports SR-IOV, you need to modify the config because Nova will see the PF | |
| 10:37:35 | zigo | bauzas: If it has SR-IOV support, then type-PF ? | |
| 10:37:40 | bauzas | yes | |
| 10:37:43 | zigo | Ah ok... | |
| 10:37:44 | bauzas | nvidia T4s | |
| 10:37:52 | zigo | bauzas: That's the model we have, indeed. | |
| 10:38:06 | bauzas | yeah, we had a problem with nvidia | |
| 10:38:08 | zigo | bauzas: Is this for both the alias in the scheduler and in the compute ? | |
| 10:38:36 | bauzas | they only say that SR-IOV is experimental for T4, but given this, you actuallly need to change nova for it | |
| 10:38:46 | bauzas | zigo: hem, yes AFAIK | |
| 10:38:49 | bauzas | stephenfin: right ? ^ | |
| 10:40:09 | lyarwood | gibi: odd, so one of the tests makes two attempts to detach the same volume | |
| 10:40:30 | stephenfin | zigo, bauzas: correct - if the device reports SR-IOV capabilities, you need to set the device type | |
| 10:42:14 | zigo | bauzas: Thanks a lot. I'll add a per-compute option to select PCI / PF then. | |
| 10:42:32 | stephenfin | On that note, I still have two patches open to improve SR-IOV testing, if anyone has a few minutes. Happy to review other stuff in exchange https://review.opendev.org/q/topic:%2522sriov-functional-tests%2522+status:open | |
| 10:43:22 | lyarwood | gibi: https://review.opendev.org/c/openstack/tempest/+/760357 - this added a cleanup to detach the volume that these tests already detaches | |
| 10:44:17 | lyarwood | I'll write up a bug shortly | |
| 10:49:15 | zigo | bauzas: IMO, it'd be really nice to automate this, with placement reporting the correct type and the scheduler being auto-magic. It took us 2 days to figure out what was going on... | |
| 10:50:18 | elod | thx lyarwood ! | |
| 10:51:32 | gibi | stephenfin: ack, I can add that to today's agenda | |
| 10:53:28 | gibi | lyarwood: thanks! (sorry I was pulled after I pinged you) | |
| 10:53:41 | sean-k-mooney | zigo: we do not report pci devices in placement yet | |
| 10:54:00 | zigo | sean-k-mooney: But that would be a nice improvement ! :) | |
| 10:54:04 | sean-k-mooney | zigo: also this is because of a upgrade in your t4 device firmware | |
| 10:54:16 | sean-k-mooney | zigo: yes its on our todolist for a few years | |
| 10:54:32 | sean-k-mooney | there is a series that does most of it that was abandoned 2 years ago | |
| 10:54:37 | sean-k-mooney | we just need to revive it | |
| 10:54:56 | sean-k-mooney | it was blocked because placmenet did not support nested resouce providers proerly at the time | |
| 10:55:10 | sean-k-mooney | that has now meen fixed on the placment side | |
| 10:55:11 | gibi | lyarwood: i don't see the double detach here: tempest.scenario.test_minimum_basic.TestMinimumBasicScenario.test_minimum_basic_scenario | |
| 10:55:42 | zigo | sean-k-mooney: Is type-PCI and type-PF the only possible values? | |
| 10:55:52 | gibi | lyarwood: nvm, I see it now | |
| 10:55:56 | sean-k-mooney | no also type-VF | |
| 10:55:59 | zigo | (I'm implementing this in my deployment software and need to validate input) | |
| 10:56:04 | zigo | Ok, thanks. | |
| 10:56:12 | gibi | nova_volume_attach adds a cleanup and the test case itself adds the same cleanup | |
| 10:56:21 | sean-k-mooney | we are adding type-VDPA but i plan to not allow that in the alias | |
| 10:56:53 | sean-k-mooney | VDPA devices will be VF that cant be use with alias based passthough as they require an additonal contolplain to function | |
| 10:57:33 | sean-k-mooney | zigo: by the way are you currently supporting devname in the pci whitelist? | |
| 10:57:43 | sean-k-mooney | zigo: we woudl really like to remove that in the future | |
| 10:57:57 | zigo | sean-k-mooney: Yeah, I do ! | |
| 10:58:34 | sean-k-mooney | ok what we would prefer is for installer tools to convert that to the pci adress | |
| 10:58:39 | zigo | For each compute node, I can set: use_gpu, gpu_name, gpu_vendor_id, gpu_product_id and gpu_device_type. | |
| 10:58:52 | sean-k-mooney | gpu_name? | |
| 10:58:53 | zigo | I just need to expose gpu_device_type in my API which I didn't do... | |
| 10:59:02 | zigo | Yeah, a free name which goes in the alias. | |
| 10:59:13 | sean-k-mooney | oh that is differnt | |
| 10:59:21 | sean-k-mooney | im not talking about the alias | |
| 10:59:38 | sean-k-mooney | im talking about the whitelist where we support devname for nics e.g. eht1 | |
| 10:59:43 | sean-k-mooney | eth1 | |
| 10:59:50 | zigo | We don't do SR-IOV yet. | |
| 11:00:01 | zigo | Just GPUs... | |
| 11:00:47 | sean-k-mooney | vGPU or pci passthough | |
| 11:01:22 | sean-k-mooney | if your using pci passthough via the alias and pci whitelist that is most of the config for neutron sriov | |
| 11:01:50 | sean-k-mooney | well sriov is jsut a tech its not related to networkign some gpus supprot it too | |
| 11:02:22 | sean-k-mooney | e.g. you can create VF on some gpus and passthough those to the guest | |
| 11:02:29 | sean-k-mooney | instead of the PF | |