| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-05-29 | |||
| 16:13:47 | mriedem | https://docs.openstack.org/nova/latest/admin/pci-passthrough.html for the latter | |
| 16:13:56 | mriedem | *er pcpu pci passthrough | |
| 16:14:01 | mriedem | *pgpu | |
| 16:14:14 | mriedem | that doc mostly talks about sriov ports though | |
| 16:14:58 | sean-k-mooney | you sould also be about to use amds sriov based vgpu "MxGpu" or something like that via pci passthrough from juno or whenever we added that | |
| 16:15:44 | sean-k-mooney | i.e. if your gpu support virtualisation via sriov then that will jsut work like any other sriov vf with pci passhtough | |
| 16:15:57 | sean-k-mooney | the same we we supprot qat virtualistion via sriov | |
| 16:17:13 | phillw | I'll go and have a read up. A colleague of mine wants to use the gpu power for AI learning and I'm at the start of re-learning openstack after a break since 2014 :) | |
| 16:18:09 | mriedem | scanning the cern article it looks pretty comprehensive | |
| 16:18:14 | mriedem | and says what we just told you anyway :) | |
| 16:18:31 | mriedem | i'm not aware of who is using vgpus yet | |
| 16:18:32 | sean-k-mooney | the current state is we have dedicated support for the nvida data center gpus( which need a license) and untested support for sriov stuff which shoudl jsut work via pcie passthough | |
| 16:18:36 | mriedem | bauzas might know | |
| 16:18:38 | phillw | it missed out about deployment :) | |
| 16:18:49 | mriedem | phillw: yeah but it linked to those same docs | |
| 16:18:56 | mriedem | that's your homework | |
| 16:20:11 | sean-k-mooney | phillw: so you your colleage wants to use a consomer gpu they are out of luck unless they do a complte passthough which technicall violates nvidas eula | |
| 16:20:31 | phillw | mriedem: it's Tims home work, he wants to add gpu's :) As both our instances are private we'll be using packstack. | |
| 16:20:53 | sean-k-mooney | phillw: is packstack still a thing? | |
| 16:20:55 | phillw | sean-k-mooney: so not possible with sriov without breaching EULA? | |
| 16:21:05 | sean-k-mooney | on the nvida side | |
| 16:21:11 | phillw | sean-k-mooney: yeah, it's still hanging on in there :P | |
| 16:21:19 | sean-k-mooney | technically if you dont use the binary driver then your fine | |
| 16:21:31 | sean-k-mooney | but if you use the opensocrce driver you dont have gpu compute | |
| 16:21:38 | sean-k-mooney | so its kind of a catch 22 | |
| 16:21:49 | phillw | ah, understood. | |
| 16:22:20 | sean-k-mooney | nvida really want people bying the datacenter gpus insteadk of ther gaming ones for viutalisation | |
| 16:23:08 | mriedem | there is a workaround i believe | |
| 16:23:08 | mriedem | https://review.opendev.org/#/c/579897/ | |
| 16:23:13 | sean-k-mooney | yes there is | |
| 16:23:22 | phillw | yeah, i can understand that. I recall playstation upgrade being done which would prevent USAF using a cluster of them as cloud :) | |
| 16:23:25 | sean-k-mooney | you set the hypervior to hidden | |
| 16:23:29 | mriedem | ^ is from CERN btw | |
| 16:23:56 | sean-k-mooney | yes that is for hyperv i think | |
| 16:23:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop handling 'InstanceUnknownCell' exception https://review.opendev.org/651307 | |
| 16:23:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove unnecessary wrapper https://review.opendev.org/651308 | |
| 16:23:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Remove cell APIs https://review.opendev.org/651309 | |
| 16:23:59 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove cells v1 options, group https://review.opendev.org/651310 | |
| 16:23:59 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'ComputeManager._reschedule' https://review.opendev.org/660774 | |
| 16:24:00 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove cells v1 parameter from 'ComputeTaskAPI.resize_instance' https://review.opendev.org/651311 | |
| 16:24:00 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop passing 'kwargs' to 'rebuild_instance' https://review.opendev.org/651312 | |
| 16:24:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop passing 'delete_type' to 'terminate_instance' https://review.opendev.org/651313 | |
| 16:24:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: filters: Stop handling cells v1 https://review.opendev.org/651314 | |
| 16:24:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove nova.compute.*API() shims https://review.opendev.org/660527 | |
| 16:24:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Ensure controllers all call super https://review.opendev.org/660950 | |
| 16:24:04 | sean-k-mooney | for libvirt it already works | |
| 16:24:41 | phillw | that's cool, I'm on nodding terms with libvirt as I use it for my VMs :) | |
| 16:25:13 | mriedem | sean-k-mooney: that patch is for making nvidia work with windows guests in libvirt | |
| 16:25:19 | mriedem | the existing support is for libvirt guests only | |
| 16:25:33 | sean-k-mooney | * linux guests | |
| 16:26:05 | mriedem | i never really understood the opposition to that patch since it's just doing the same thing for windows guests on kvm that we allow for linux guests | |
| 16:26:12 | mriedem | i.e. we've already opened pandora's box there | |
| 16:26:23 | sean-k-mooney | we nack it however as we did not want to add a feature to specically workaound a driver check that is there to prevent you breaching nvidas eula | |
| 16:26:38 | mriedem | *we already do* | |
| 16:26:54 | mriedem | https://blueprints.launchpad.net/nova/+spec/hide-hypervisor-id-flavor-extra-spec | |
| 16:26:54 | sean-k-mooney | the hypervior hidden flag was not added for that use however | |
| 16:27:00 | sean-k-mooney | it was already there | |
| 16:27:06 | mriedem | yes it was | |
| 16:27:14 | mriedem | https://blueprints.launchpad.net/nova/+spec/add-kvm-hidden-feature | |
| 16:27:40 | mriedem | https://review.opendev.org/#/c/459753/18//COMMIT_MSG@14 | |
| 16:28:03 | mriedem | so jay was +W on that change, but not this other one to allow the same thing for windows guests | |
| 16:28:06 | mriedem | which i don't understand | |
| 16:28:14 | sean-k-mooney | huh in that case i guess i dont know why we have not made it work for windows guests | |
| 16:28:42 | sean-k-mooney | i had under stood this was not sepcifcally for the nvida case but i guess it was | |
| 16:28:45 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add reno for removed cells v1 policies https://review.opendev.org/662031 | |
| 16:30:48 | mriedem | that said i don't see any cern people around to rev that patch | |
| 16:30:49 | mriedem | or ovh people | |
| 16:31:07 | mriedem | amorin: you're ovh right? do you guys want to revive this? https://review.opendev.org/#/c/579897/ | |
| 16:31:27 | mriedem | looks like Maciej and Piotr cared about it at one point | |
| 16:32:44 | phillw | on a not totally dissimilar topic, I was actually paying attention on my last C7 installation and it offers to install a compute instance. Am I correct in my reading that this is a nova instance? | |
| 16:33:14 | mriedem | C7 = centos7? | |
| 16:33:23 | phillw | yes, sorry | |
| 16:33:25 | mriedem | "it offers to install a compute instance. Am I correct in my reading that this is a nova instance?" no idea | |
| 16:33:34 | jcosmao | hi mriedem, i'll forward info to my teammate (i work at ovh also) | |
| 16:33:43 | mriedem | jcosmao: thanks | |
| 16:34:05 | phillw | mriedem: home work for me, then :) | |
| 16:34:14 | mriedem | just make tim do it | |
| 16:35:20 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Refresh instance network info on deletion https://review.opendev.org/660761 | |
| 16:39:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hacking: Resolve E731 (do not assign a lambda expression, use a def) https://review.opendev.org/651554 | |
| 16:39:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hacking: Resolve W503 (line break occurred before a binary operator) https://review.opendev.org/651555 | |
| 16:39:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hacking: Resolve E741 (ambiguous variable name) https://review.opendev.org/652103 | |
| 16:39:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hacking: Resolve W605 (invalid escape sequence) https://review.opendev.org/652104 | |
| 16:40:20 | aspiers | mriedem: does this look familiar to you? b'/home/zuul/src/opendev.org/openstack/nova/.tox/lower-constraints/lib/python3.6/site-packages/pymysql/cursors.py:167: Warning: (1831, "Duplicate index \'block_device_mapping_instance_uuid_virtual_name_device_name_idx\' defined on the table \'fjisjekoia.block_device_mapping\'. This is deprecated and will be disallowed in a future release.")' | |
| 16:40:40 | aspiers | seems completely unrelated to my patch which is failing in the gate | |
| 16:40:50 | aspiers | and I think I've seen it a few times in the last 24 hours | |
| 16:43:00 | aspiers | oh, that's just a warning | |
| 16:43:34 | aspiers | still trying to find the real issue in http://logs.openstack.org/66/661266/4/gate/openstack-tox-lower-constraints/7af4be7/job-output.txt.gz | |
| 16:43:42 | aspiers | I see a bunch of timeouts | |
| 16:45:26 | openstackgerrit | Merged openstack/os-vif stable/stein: Prevent "qbr" Linux Bridge from replying to ARP messages https://review.opendev.org/655678 | |
| 16:47:06 | kashyap | aspiers: Remind me again, in 'os-traits' repo, what to do when 'tox' complains too long lines for a URL? | |
| 16:47:23 | aspiers | kashyap: isn't that just a warning? | |
| 16:47:36 | aspiers | if not, I guess # noqa or similar | |
| 16:48:05 | kashyap | aspiers: It moans an ERROR -- "E501 line too long (108 > 79 characters)" | |
| 16:48:28 | kashyap | aspiers: Before I go off shortly, I'm pushing the change. For convenience, I'm including a short "changelog" noting explicitly what I addressed | |
| 16:48:55 | kashyap | ... so that if you want to rebase on it tomm. | |
| 16:49:01 | aspiers | kashyap: http://flake8.pycqa.org/en/3.1.1/user/ignoring-errors.html#in-line-ignoring-errors | |
| 16:49:09 | aspiers | # noqa: E501 | |
| 16:49:42 | kashyap | Right, thanks! | |
| 16:53:52 | mriedem | aspiers: that duplicate index thing is super old yes | |
| 17:05:10 | aspiers | mriedem: the failures are very similar to https://bugs.launchpad.net/nova/+bug/1823251, but without the InterfaceError | |