| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-14 | |||
| 15:18:30 | feeblepipes | jangutter: what about having "iommu_passthrough" as the feature, then? | |
| 15:19:44 | jangutter | feeblepipes: well, iommu_passthrough won't tell us anything of the device type that's going to appear inside the VM. | |
| 15:19:46 | feeblepipes | jangutter: would an end-user say "I want a vNIC that is associated to a VF that has IOMMU passthrough features" or would the end user say "I want a vNIC that is associated with a VF that is a Netronome Agilio SmartNIC PF"? | |
| 15:23:10 | jangutter | feeblepipes: my view of one usecase is this: an end-user says "I want a vNIC, associated with a VF, passed through via IOMMU, with <insert-hw-driver-here> PCI driver, additionally pin everything for me optimally, pretty please!" | |
| 15:23:29 | feeblepipes | :) | |
| 15:23:42 | feeblepipes | jangutter: ok, cool. we're getting somewhere here :) | |
| 15:23:56 | jangutter | feeblepipes: with DPDK applications being specifically compiled for specific PCI drivers, it's sometimes a _must_. | |
| 15:24:11 | feeblepipes | jangutter: the <insert-hw-driver-here> goes in the *image metadata*, though. | |
| 15:25:04 | feeblepipes | jangutter: because the image metadata is what describes the workload to be launched. that image metadata can describe the HW driver's installed on the image, and that information can/should be used by os-vif in setting up the VIF plug on the host. | |
| 15:25:07 | jangutter | feeblepipes: yeah, but that leads to the other bit of fun. Give me one vNIC on control, standard TAP, and vNIC 2 for my DPDK NFV application. | |
| 15:25:14 | feeblepipes | jangutter: so I think we're *almost* there. | |
| 15:25:46 | jangutter | feeblepipes: is it possible to spec stuff that fine-grained? | |
| 15:26:02 | feeblepipes | jangutter: ah, indeed. so perhaps that information (hw driver) does need to be in the port binding profile. | |
| 15:26:18 | feeblepipes | jangutter: yes, it is possible. via the port profile information. | |
| 15:26:34 | feeblepipes | jangutter: I still don't think we need to have vendor_id/product_id in there, though. :) | |
| 15:26:54 | feeblepipes | jangutter: in any case... I think this is also something we can hash out in the code rather than the spec. | |
| 15:27:33 | jangutter | feeblepipes: I am definitely not averse to having a "driver list". That's actually a pretty good idea from a user's perspective. | |
| 15:27:34 | feeblepipes | jangutter: I'm willing to +2 the spec on condition that you just place a note in the spec that we've discussed on IRC how to make the information passed between Neutron and Nova via os-vif as generic as possible. | |
| 15:27:50 | feeblepipes | jangutter: if you can do that quick, I'll +2 it and mriedem can as well. | |
| 15:30:53 | mriedem | sfinucan: how do i reference another link within the same docs using rst again? | |
| 15:31:08 | mriedem | like i want to link to a section in the nova-manage man page from the nova-cells man page | |
| 15:31:43 | jangutter | I shall definitely do so: to summarize: It is undesirable to send the VF selection parameters to Nova as PCI vendor/product ID, further work should be done to develop a capability or driver list that would abstract this away from users. | |
| 15:32:01 | mriedem | oh maybe :ref: | |
| 15:32:11 | efried | mriedem Looks like that kolla thing got sorted. Can you throw me a link to whatever dashboard/view I should be using, so I can maybe pick off some lhf? | |
| 15:32:42 | mriedem | efried: not sorted yet, docs change coming soon | |
| 15:32:55 | efried | I meant triaged | |
| 15:33:05 | efried | That's the goal of the current sweep, right? | |
| 15:33:23 | efried | Not to resolve, but to get 'em out of 'New'? | |
| 15:33:44 | mriedem | efried: yeah | |
| 15:33:54 | mriedem | like 'unexpected api error' is usually always invalid | |
| 15:33:58 | mriedem | and a misconfiguration | |
| 15:34:05 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note and update cell install guide for multi-cell limitations https://review.openstack.org/456923 | |
| 15:34:12 | mriedem | melwitt: ^ | |
| 15:35:40 | mriedem | EmilienM: owalsh: ^ | |
| 15:38:26 | efried | This thing about having to run a new discovery command to make stuff work... Could it not just be made to work out of the box if you only have one cell? | |
| 15:47:49 | efried | [irony: when --help doesn't automatically page, and you have to say --help|less] | |
| 15:50:14 | mriedem | efried: we have a config option if you want things discovered automatically, | |
| 15:50:20 | mriedem | but it's a bit heavy for large deployments | |
| 15:50:27 | mriedem | dan has a series up to improve that | |
| 15:50:32 | jangutter | Is there a way to refer to today's IRC chat in a spec? | |
| 15:50:34 | efried | mriedem Okay, neat. | |
| 15:50:40 | mriedem | jangutter: yeah, esc | |
| 15:50:41 | mriedem | *Sec | |
| 15:50:43 | efried | http://eavesdrop.openstack.org/irclogs/%23openstack-nova/ | |
| 15:50:47 | efried | jangutter ^^ | |
| 15:50:48 | mriedem | efried: https://blueprints.launchpad.net/nova/+spec/discover-hosts-faster | |
| 15:51:03 | mriedem | jangutter: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-04-14.log.html#t2017-04-14T14:44:06 | |
| 15:52:24 | jangutter | Whatever you write is now memorialized forever. Or until cats reclaim their rightful place on top of the food chain. | |
| 15:54:10 | mriedem | our cat can easily be defeated with a bag of chips | |
| 15:54:47 | openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | |
| 15:57:26 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Tell people that the nova-cells man page is for cells v1 https://review.openstack.org/456925 | |
| 15:57:29 | jangutter | Another newbie question. I've replied to a bunch of comments on an older review. They're in "draft" in gerrit now, am I too late? | |
| 15:57:41 | mriedem | jangutter: no, you can still publish those | |
| 15:57:43 | efried | jangutter No - go to that patch set and they'll show up. | |
| 15:58:15 | efried | Still not entirely understanding why a compute node can't automatically 'discover' itself into a cell if there's only one cell... | |
| 15:58:31 | mriedem | efried: it would require an upcall from the compute to the api layer, which we don't want | |
| 15:58:35 | mriedem | upcalls are the devil | |
| 15:58:43 | mriedem | s/the/a/ | |
| 15:58:53 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder v3 detach call to _terminate_volume_connections https://review.openstack.org/456896 | |
| 15:59:12 | mriedem | efried: also, that would require either an upcall or putting the nova api db credentials in the compute config | |
| 15:59:16 | mriedem | another thing we don't want | |
| 16:00:20 | efried | mriedem And all of that is different from how a compute node registered itself B.C. (before cells)? | |
| 16:00:52 | mriedem | yes | |
| 16:00:58 | mriedem | in the before times, | |
| 16:01:01 | mriedem | there was no api db | |
| 16:01:08 | mriedem | there were no host mappings in the api db | |
| 16:01:36 | mriedem | when you start nova-compute, it creates it's own compute_nodes table record for that host (happens in ResourceTracker.update_available_resource eventually) | |
| 16:01:57 | mriedem | when you start the service it also creates a record in the services table for the nova-compute binary | |
| 16:02:05 | mriedem | those all get mangled up in the api somehow | |
| 16:02:08 | mriedem | feeblepipes loves it | |
| 16:02:32 | mriedem | but with no cells, i.e. regular old stock nova, there was just a single db and everything was in there | |
| 16:03:05 | mriedem | efried: are you going to be in boston? | |
| 16:03:10 | efried | mriedem Yessir. | |
| 16:03:45 | mriedem | then put https://www.openstack.org/summit/boston-2017/summit-schedule/events/18727/ on your schedule | |
| 16:04:18 | efried | mriedem Roger wilco, and thanks. | |
| 16:12:46 | EmilienM | mriedem: ack | |
| 16:33:37 | feeblepipes | jangutter: yes, your summary of the requested changes above is good. | |
| 16:36:09 | jangutter | feeblepipes: if all else fails there's also that list of emulated vNICs that qemu uses: confusingly it's called "hw_vif_model" | |
| 16:36:23 | feeblepipes | jangutter: yeah, I know :( | |
| 16:43:06 | feeblepipes | peter-hamilton: approved. | |
| 16:43:30 | peter-hamilton | feeblepipes: awesome, thanks! | |
| 16:43:46 | feeblepipes | peter-hamilton: np. sorry again for the delay. | |
| 16:44:02 | peter-hamilton | feeblepipes: no problem, i hope you recover quickly | |
| 16:44:24 | feeblepipes | peter-hamilton: about 50% today. which is better than yesterday's 10% :) | |
| 16:44:52 | peter-hamilton | feeblepipes: yep, i've been there | |
| 16:45:05 | peter-hamilton | feeblepipes: thanks for the comment on the history btw | |
| 16:45:14 | peter-hamilton | feeblepipes: i find it helps to summarize decisions and the back-and-forth over the months | |
| 16:45:30 | peter-hamilton | feeblepipes: no one else seems to use it for much | |
| 16:45:30 | feeblepipes | yeah, it actually makes it a bit easier for those of us kinda coming into a spec late in its lifetime. | |
| 16:45:42 | feeblepipes | ++ | |
| 16:52:44 | jangutter | Any more actions on https://review.openstack.org/456386 I need to take? | |
| 16:56:52 | feeblepipes | jangutter: did you push a new revision with wording around the "genericness" of parameters passed via os-vif/Neutron port binding profile? | |
| 16:57:49 | jangutter | feeblepipes: yeah, that's been updated, plus a link to the chat. | |
| 16:58:56 | feeblepipes | jangutter: awesome. reviewing again. | |
| 17:06:54 | feeblepipes | jangutter: k, +2 from me. mriedem, feel free to give 456386 another spin. | |
| 17:07:37 | jangutter | feeblepipes: thanks! | |
| 17:07:44 | feeblepipes | jangutter: np | |
| 17:08:03 | feeblepipes | dansmith: did you see johnthetubaguy's request for your review on https://review.openstack.org/#/c/407237/? | |
| 17:37:54 | owalsh | mriedem: yea, the related change in tripleo was to handle this | |
| 18:08:02 | mriedem | feeblepipes: jangutter: done | |