*** tongl has quit IRC | 00:09 | |
*** yamamoto has quit IRC | 00:10 | |
*** yamamoto has joined #openstack-neutron-ovn | 00:11 | |
*** deep-book-gk_ has joined #openstack-neutron-ovn | 00:14 | |
*** deep-book-gk_ has left #openstack-neutron-ovn | 00:16 | |
*** saphi has joined #openstack-neutron-ovn | 01:56 | |
*** mestery_ has joined #openstack-neutron-ovn | 02:19 | |
*** mestery has quit IRC | 02:19 | |
*** mestery_ is now known as mestery | 02:20 | |
*** gongysh has joined #openstack-neutron-ovn | 04:34 | |
*** gongysh has quit IRC | 04:40 | |
*** janki has joined #openstack-neutron-ovn | 04:48 | |
*** gongysh has joined #openstack-neutron-ovn | 05:40 | |
*** yamamoto has quit IRC | 06:54 | |
*** mmirecki has joined #openstack-neutron-ovn | 07:06 | |
*** mmirecki has quit IRC | 07:23 | |
*** mmirecki has joined #openstack-neutron-ovn | 07:38 | |
*** yamamoto has joined #openstack-neutron-ovn | 07:55 | |
*** fzdarsky has joined #openstack-neutron-ovn | 07:57 | |
*** yamamoto has quit IRC | 08:02 | |
*** gongysh has quit IRC | 08:12 | |
*** lucas-afk is now known as lucasagomes | 08:15 | |
*** dasanind has quit IRC | 09:14 | |
*** gongysh has joined #openstack-neutron-ovn | 09:21 | |
*** dasanind has joined #openstack-neutron-ovn | 09:21 | |
*** dasanind has quit IRC | 09:21 | |
*** dasanind has joined #openstack-neutron-ovn | 09:21 | |
*** yamamoto has joined #openstack-neutron-ovn | 09:24 | |
*** yamamoto has quit IRC | 09:35 | |
*** yamamoto has joined #openstack-neutron-ovn | 09:45 | |
*** mmirecki is now known as mmirecki_lunch | 09:54 | |
*** mmirecki_lunch is now known as mmirecki | 11:18 | |
*** saphi has quit IRC | 11:28 | |
*** gongysh has quit IRC | 11:47 | |
*** lucasagomes is now known as lucas-hungry | 11:52 | |
dalvarez | russellb, ping re. https://github.com/openstack/networking-ovn/commit/e9f430135ce83bc98301739f40a8bdea7862f55a does it still make sense after the trunk_driver was introduced in networking-ovn? | 11:55 |
---|---|---|
*** yamamoto has quit IRC | 11:57 | |
*** yamamoto has joined #openstack-neutron-ovn | 12:13 | |
russellb | dalvarez: no | 12:31 |
russellb | dalvarez: that was pre vlan-aware-vms | 12:31 |
russellb | kept for backwards compat i suppose ... | 12:32 |
russellb | in fact, i think there's still something that uses it let me check | 12:32 |
dalvarez | russellb, ack | 12:32 |
russellb | yes ... there is a Docker networking plugin shipped with OVN that uses it still | 12:33 |
russellb | ovn/utilities/ovn-docker-underlay-driver | 12:33 |
*** yamamoto has quit IRC | 12:54 | |
dalvarez | russellb, ack got it thanks! | 12:55 |
dalvarez | russellb, i thought of removing it from networking-ovn tho | 12:55 |
* russellb nods | 12:56 | |
russellb | i would support that if it wasn't used | 12:56 |
russellb | so i'd want to migrate the existing code that uses it first | 12:56 |
russellb | and even then, you've got the past releases of OVS/OVN that still use it | 12:57 |
russellb | we're kinda stuck for a while | 12:57 |
russellb | at least it's small | 12:57 |
*** mmirecki is now known as mmirecki_netcont | 12:58 | |
*** mmichelson has joined #openstack-neutron-ovn | 13:01 | |
*** lucas-hungry is now known as lucasagomes | 13:05 | |
*** yamamoto has joined #openstack-neutron-ovn | 13:09 | |
dalvarez | russellb, right it's small... i'll be sending a patch to networking-ovn to support the trunk ports, right now we're not transitioning them to ACTIVE since they don't have host_id and plugin doesn't try to bind them | 13:26 |
dalvarez | for normal ports, nova sets the binding:host so my plan is, whenever 'up' in Port_Binding table goes from False to True, check if that's a subport and if so, inherit the binding:host from parent | 13:27 |
dalvarez | russellb, how does that sound? | 13:27 |
*** saphi has joined #openstack-neutron-ovn | 13:29 | |
*** mlavalle has joined #openstack-neutron-ovn | 13:53 | |
russellb | dalvarez: sounds reasonable to me | 14:18 |
dalvarez | russellb, ack thanks | 14:18 |
dalvarez | that should make subports go ACTIVE | 14:18 |
dalvarez | wich is needed by kuryr and probably someone else | 14:19 |
*** janki has quit IRC | 14:23 | |
openstackgerrit | Terry Wilson proposed openstack/networking-ovn master: Prepare for using ovsdbapp > 0.4.0 https://review.openstack.org/487442 | 14:24 |
*** yamamoto has quit IRC | 14:24 | |
openstackgerrit | Terry Wilson proposed openstack/networking-ovn master: Prepare for using ovsdbapp > 0.4.0 https://review.openstack.org/487442 | 15:02 |
otherwiseguy | russellb, dalvarez numans If you have a chance to look at https://review.openstack.org/#/c/484964/, it's just copying over RowEvent from networking-ovn to ovsdbapp, but it does change from using eventlet to native thread. I run a test with it in the ovn southbound patch and seems to work well. | 15:07 |
otherwiseguy | dalvarez, I seem to remember you using some RowEvents lately. :D | 15:08 |
russellb | dalvarez: yep, just an oversight really | 15:08 |
otherwiseguy | also, I would like to profess my undying love for git bisect. that is all. | 15:10 |
russellb | ha, indeed handy | 15:10 |
otherwiseguy | WHY U NO WORK WITH NEW OVSDBAPP?! git bisect start;git bisect bad;git bisect good 0.4.0;git bisect run ./test.sh and go work on something else for 10 minutes. :p It's like magic. | 15:12 |
* otherwiseguy waits for the machines to replace him | 15:13 | |
*** yamamoto has joined #openstack-neutron-ovn | 15:24 | |
*** yamamoto has quit IRC | 15:36 | |
*** mmirecki_netcont has quit IRC | 15:37 | |
dalvarez | otherwiseguy, lol | 16:09 |
dalvarez | looking | 16:09 |
*** lucasagomes is now known as lucas-afk | 16:23 | |
openstackgerrit | Russell Bryant proposed openstack/networking-ovn master: doc: Add link to tutorial. https://review.openstack.org/487506 | 16:28 |
openstackgerrit | Russell Bryant proposed openstack/networking-ovn master: doc: Fix list and link formatting. https://review.openstack.org/487507 | 16:28 |
dalvarez | otherwiseguy, dumb question... that patch is a part of the "getting rid of eventlet" effort in ovsdbapp right? | 16:34 |
dalvarez | does it enhance performance with native threading? | 16:34 |
otherwiseguy | dalvarez, no idea on performance, just eventlet isn't a dependency of ovsdbapp and didn't want it to be one. figure if someone is monkeypatching eventlet, queues/threads/etc. get patched and will work. | 16:35 |
dalvarez | otherwiseguy, gotcha | 16:35 |
otherwiseguy | similar to how the Connection spawns a thread instead of picking an event loop. | 16:35 |
dalvarez | rowevent is handy :) | 16:35 |
* dalvarez remembers that we dont want to pass in a Connection because it might be not used... | 16:36 | |
dalvarez | but cant remember really where :p | 16:36 |
otherwiseguy | dalvarez, I use it here https://review.openstack.org/#/c/481779/6/ovsdbapp/tests/functional/schema/ovn_southbound/test_impl_idl.py to implement a RowEvent that waits for northd to create a Port_Binding. seemed to work pretty well. (line 106) | 16:37 |
otherwiseguy | (and 25 I suppose) | 16:37 |
dalvarez | otherwiseguy, yeah i've used it as well | 16:38 |
dalvarez | otherwiseguy, ah to pick the event loop | 16:39 |
dalvarez | that's cool! and with a def timeout of 5s, neat :) | 16:40 |
otherwiseguy | I still had to stare at the RowEvent stuff a bit before it came back to me how to use it. | 16:40 |
*** saphi has quit IRC | 16:51 | |
openstackgerrit | Merged openstack/ovsdbapp master: Add RowEvent mechanism from networking-ovn https://review.openstack.org/484964 | 17:35 |
openstackgerrit | boden proposed openstack/networking-ovn master: use qos constants from neutron-lib https://review.openstack.org/487542 | 17:57 |
*** yamamoto has joined #openstack-neutron-ovn | 18:09 | |
*** yamamoto has quit IRC | 18:13 | |
openstackgerrit | Terry Wilson proposed openstack/ovsdbapp master: Return False when comparing a RowEvent to something else https://review.openstack.org/487547 | 18:23 |
openstackgerrit | Merged openstack/ovsdbapp master: Return False when comparing a RowEvent to something else https://review.openstack.org/487547 | 19:00 |
*** mmirecki_netcont has joined #openstack-neutron-ovn | 20:17 | |
openstackgerrit | Terry Wilson proposed openstack/ovsdbapp master: Add OVN_Southbound API support https://review.openstack.org/481779 | 20:17 |
*** mmirecki_netcont has quit IRC | 20:35 | |
*** yamamoto_ has joined #openstack-neutron-ovn | 21:19 | |
*** yamamoto_ has quit IRC | 21:21 | |
*** ezobn has joined #openstack-neutron-ovn | 21:27 | |
*** ezobn has quit IRC | 21:45 | |
*** mmichelson has quit IRC | 22:19 | |
*** mlavalle has quit IRC | 22:31 | |
*** yamamoto has joined #openstack-neutron-ovn | 23:10 | |
*** yamamoto has quit IRC | 23:37 | |
*** yamamoto has joined #openstack-neutron-ovn | 23:51 | |
*** yamamoto_ has joined #openstack-neutron-ovn | 23:55 | |
*** yamamoto has quit IRC | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!