| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-04 | |||
| 14:03:23 | alex_xu | bauzas: but there is something strange in the gate http://logs.openstack.org/69/474869/5/check/gate-nova-tox-functional-ubuntu-xenial/308184e/testr_results.html.gz | |
| 14:03:37 | bauzas | alex_xu: so, 3.0.1 provides the trait https://github.com/openstack/os-traits/blob/0.3.1/os_traits/misc/__init__.py | |
| 14:03:39 | jangutter | sean-k-mooney: yep, it's always been in the back of my mind. I had thought of a support service that would persist VF settings across reboots. | |
| 14:04:09 | bauzas | and nova uses it https://github.com/openstack/nova/blob/master/requirements.txt#L56 | |
| 14:04:09 | alex_xu | bauzas: edleafe have a fix https://review.openstack.org/478564, but looks like still have problem in the gate | |
| 14:04:11 | jangutter | sean-k-mooney: maybe NetworkManager is the correct way to do it :-p | |
| 14:04:36 | sean-k-mooney | jangutter: except redhat dont use it... | |
| 14:05:09 | jangutter | sean-k-mooney: wha? RH wrote it! | |
| 14:05:53 | sean-k-mooney | jangutter: when you said networkmanager you were refering to /etc/network/interfeces and /etc/network/interfeces.d | |
| 14:06:10 | sean-k-mooney | jangutter: and the deamon that reads those files? | |
| 14:06:21 | bauzas | alex_xu: I can try to debut | |
| 14:06:23 | bauzas | debug | |
| 14:06:41 | bauzas | alex_xu: that's a transient bug, right? | |
| 14:07:15 | bauzas | ie. the gate is not blocked every time, but just having hits ? | |
| 14:07:29 | jangutter | sean-k-mooney: Yeah, the sysv initscripts and the thing that spawned from systemd. The problem with keeping state is proliferating yet another thing with another interface. | |
| 14:08:04 | sean-k-mooney | jangutter: but in either case no i think the configuration should be handled by os vif. eventurly this will get ported to the ovs windows datapath too so i would like to minimis external dependecies on linus | |
| 14:08:08 | sean-k-mooney | *linux | |
| 14:09:26 | alex_xu | bauzas: yes, looks like not very time | |
| 14:09:33 | jangutter | sean-k-mooney: aaah, yep. That's a very valid point. But my fever dream is that linux_net would call to nmcli or something if it exists, and that would persist VF settings, meaning that everything could come up asynchronously. | |
| 14:09:51 | sean-k-mooney | jangutter: for the intel intersect im actullly hoping to extend ovs to advertises the representor mapping in a table in the ovsdb that would be populated when you add the pf to the bridge | |
| 14:10:16 | jangutter | sean-k-mooney: yeah, that would be very useful. | |
| 14:10:19 | bauzas | alex_xu: I'm trying to reproduce the bug locally | |
| 14:10:37 | alex_xu | bauzas: if you lucky, you can see that locally :) | |
| 14:10:47 | bauzas | my guts is that this fixture doesn't correctly sync the lib | |
| 14:10:55 | sean-k-mooney | jangutter: i then want to have neutron read that table and populate a resouce provider for that in the placement api and have nova assign one of the representors as part of vm placement | |
| 14:10:55 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Correct the description of 'disable-log-reason' api-ref https://review.openstack.org/480209 | |
| 14:11:13 | bauzas | and that, if you run other test before that functional test, you could be lucky because then you have that lib sync'd | |
| 14:11:33 | jangutter | sean-k-mooney: yep, that makes sense. Then ovs persists the VF settings. | |
| 14:11:35 | bauzas | alex_xu: so, that's probably just us being explicit in the fixture to load the lib | |
| 14:12:34 | jangutter | sean-k-mooney: on a completely tangential topic, is this the place to ask some really, really newbie questions regarding third-party os-vif plugins? | |
| 14:13:13 | sean-k-mooney | jangutter: sure as long as we are not boaring the rest of the nova guys too much | |
| 14:14:03 | sean-k-mooney | jangutter: what did you want to ask? | |
| 14:14:59 | jangutter | sean-k-mooney: so our legacy solution (non TC) will need a third-party OS-VIF and neutron plugin. Are there any docs to follow for publication and submission? | |
| 14:16:48 | sean-k-mooney | jangutter: well the simplest way to get started is ask infra to create a new repo, then clone clone the ovs plugin subtree and use that as the base for your plugin | |
| 14:17:44 | stephenfin | and it has. Wonderful. | |
| 14:18:13 | sean-k-mooney | stephenfin: is if permenet or a backup nick? | |
| 14:18:33 | stephenfin | sean-k-mooney: Probably permanent. I've been meaning to do it for ages | |
| 14:18:36 | jangutter | sean-k-mooney: so I've built some stuff in devstack using a local repo, that wouldn't impede our publishing? | |
| 14:19:02 | stephenfin | sean-k-mooney: Got an RedHat email alias set up, so I can move things to stephenfin@redhat.com too | |
| 14:19:49 | sean-k-mooney | jangutter: no it should not when you ask infra to create the repo you can use what you already have as a seed for the repo. alternitivly you just need to host is somewhare publicly yourself. | |
| 14:20:36 | sean-k-mooney | jangutter: the process to add a plugin is simply pip install it | |
| 14:20:58 | sean-k-mooney | jangutter: you nova patch has already merged correct? | |
| 14:21:52 | sean-k-mooney | stephenfin: i thaugh you were planning to move to stephen@that.guru instead? | |
| 14:22:01 | jangutter | sean-k-mooney: nope, but the Mellanox patch does 99% of what we need for the legacy ml2. | |
| 14:22:20 | stephenfin | sean-k-mooney: Possibly, but I think I've got my Gmail filters working now so it might not be necessary | |
| 14:22:23 | stephenfin | We'll see, sure | |
| 14:23:33 | jangutter | sean-k-mooney: we just need the VNIC_VIRTIO_FORWARDER added to neutron-lib | |
| 14:23:49 | sean-k-mooney | jangutter: currently because of how the nova ovs_utils wrapper works you can use external plugins in os-vif without modifying it. | |
| 14:24:35 | sean-k-mooney | jangutter: in your case you would be better off createing a fork of os-vif with your patch applied and pip installing that for your legacy ml2 dirver | |
| 14:25:02 | sean-k-mooney | jangutter: or if we can make it compatible and land it upstream even better | |
| 14:25:34 | gmann_ | stephenfin: its you :) | |
| 14:25:57 | openstackgerrit | Merged openstack/nova master: trivial: Use valid UUIDs in test_admin_password https://review.openstack.org/393711 | |
| 14:25:58 | stephenfin | gmann_: Yeah, I decided to start using the nick I use everywhere else | |
| 14:26:02 | jangutter | sean-k-mooney: The kinky blocker is that switchdev support is needed for the upstream ovs driver to work out-of-the-box with our legacy. | |
| 14:26:13 | gmann_ | stephenfin: with ref we can only show complete api file, but i want to provide the exact API link - https://review.openstack.org/#/c/467938/16/api-ref/source/os-quota-class-sets.inc | |
| 14:26:16 | stephenfin | I should probably keep the other one running with an /away message, but eh :) | |
| 14:26:35 | gmann_ | stephenfin: nice | |
| 14:26:35 | jangutter | sean-k-mooney: So for older kernels we need a fallback mechanism to do the VF-to-repr mapping. | |
| 14:27:39 | sean-k-mooney | jangutter: you would not happen to be useing vlan for that would you? | |
| 14:28:24 | openstackgerrit | Moshe Levi proposed openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265 | |
| 14:28:27 | sean-k-mooney | jangutter: https://review.openstack.org/#/c/456481/1 this is the nove patch that is out standing for the legacy version correct | |
| 14:28:40 | stephenfin | gmann_: Could you just put an anchor into https://raw.githubusercontent.com/openstack/nova/master/api-ref/source/os-quota-sets.inc ? | |
| 14:28:48 | stephenfin | Assuming one isn't automatically inserted for you | |
| 14:29:26 | jangutter | sean-k-mooney: yep, that's the old stuff I'm refactoring now. | |
| 14:29:32 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | |
| 14:29:39 | stephenfin | e.g. above the 'List Default Quotas For Tenant' title, add '.. _list-default-quotas-for-tenant:' | |
| 14:29:56 | stephenfin | again, assuming ':ref:`list-default-quotas-for-tenant`' doesn't work already | |
| 14:29:59 | jangutter | sean-k-mooney: the external os-vif plugin will make that much cleaner. | |
| 14:30:25 | stephenfin | I know Sphinx can automatically insert references for headers, but I'm not sure if it's enabled for that build | |
| 14:30:56 | gmann_ | stephenfin: humm, ok. let me check otherwise ll add anchor | |
| 14:31:26 | stephenfin | gmann_: Do please. The advantage of doing this is that it always works while the link is valid, and the build fails if not | |
| 14:32:09 | jangutter | sean-k-mooney: https://review.openstack.org/#/c/456481/1/nova/network/neutronv2/api.py <--- this bit is obsoleted by https://review.openstack.org/#/c/398265/14/nova/network/neutronv2/api.py | |
| 14:32:40 | jangutter | sean-k-mooney: I think... | |
| 14:33:25 | openstackgerrit | Moshe Levi proposed openstack/os-vif master: fix read the representor phys_port_name https://review.openstack.org/480213 | |
| 14:33:43 | jangutter | sean-k-mooney: and this is the final piece we need: https://review.openstack.org/#/c/398265/14/nova/virt/libvirt/vif.py | |
| 14:34:53 | sean-k-mooney | jangutter: well in terms of setting the ovs-bridge to attach the representor to yes | |
| 14:35:38 | sean-k-mooney | the code you referenced though in the original patch appears to be constructing a pci request | |
| 14:40:06 | jangutter | sean-k-mooney: agh, I forgot. VNIC_VIRTIO_FORWARDER also needs to be added to VNIC_TYPES_SRIOV. | |
| 14:40:31 | jangutter | sean-k-mooney: that triggers the pci_request for it. | |
| 14:40:35 | sean-k-mooney | jangutter: well yes and no | |
| 14:40:49 | bauzas | alex_xu: looks like I'm fortunate, I hitted the race | |
| 14:41:11 | sean-k-mooney | it does not have to be pci back it will be in your case but for intel i was hoping to use that vnic type without sriov vf | |
| 14:41:34 | bauzas | alex_xu: the good thing is that I had a working run, and then a failed one, so I just need to see how to bisect that | |
| 14:42:09 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938 | |
| 14:42:16 | sean-k-mooney | at lest in some cases. i had not fully mapped out that but yes if we limit our selves to the netronome usecase you will need a pci device allocated | |
| 14:42:36 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214 | |
| 14:44:17 | sean-k-mooney | jangutter: ignore the intel stuff for now that si still in flux and i cant going into detail anyway at this time | |
| 14:44:18 | jangutter | sean-k-mooney: yep, it's not necessarily that all forwarders will have a 1:1 mapping either. | |
| 14:44:51 | jangutter | sean-k-mooney: maybe this means a new os-vnic library.... | |
| 14:46:08 | sean-k-mooney | sean-k-mooney: this is why i want ovs to track this and feed that info from neutron into the placement api so that we can handel these details in the ml2 driver and os-vif dirver generically | |
| 14:47:15 | bauzas | alex_xu: lots of fun, the working test has 309 tests, the failing test 310 :p | |
| 14:47:37 | jangutter | sean-k-mooney: yep, don't worry, you're not the only one thinking in that direction. | |
| 14:50:04 | jangutter | sean-k-mooney: ok, thanks very much. I'll get back on our TC interfaces post-haste and our refactored legacy driver should be done pretty soon. | |
| 14:50:43 | bauzas | alex_xu: I'm surprised, the test which fails wasn't run by the working test | |
| 14:53:33 | bauzas | actually nevermind | |
| 14:58:33 | moshele | sean-k-mooney: hi | |
| 14:59:21 | moshele | sean-k-mooney: is my comment https://review.openstack.org/#/c/398265/ answer your question ? | |
| 15:00:02 | moshele | sean-k-mooney: I have to go let talk tomorrow | |
| 15:21:29 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214 | |
| 16:41:28 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: Remove custom autodoc implementation https://review.openstack.org/480262 | |