Wednesday, 2017-07-26

*** tongl has quit IRC00:09
*** yamamoto has quit IRC00:10
*** yamamoto has joined #openstack-neutron-ovn00:11
*** deep-book-gk_ has joined #openstack-neutron-ovn00:14
*** deep-book-gk_ has left #openstack-neutron-ovn00:16
*** saphi has joined #openstack-neutron-ovn01:56
*** mestery_ has joined #openstack-neutron-ovn02:19
*** mestery has quit IRC02:19
*** mestery_ is now known as mestery02:20
*** gongysh has joined #openstack-neutron-ovn04:34
*** gongysh has quit IRC04:40
*** janki has joined #openstack-neutron-ovn04:48
*** gongysh has joined #openstack-neutron-ovn05:40
*** yamamoto has quit IRC06:54
*** mmirecki has joined #openstack-neutron-ovn07:06
*** mmirecki has quit IRC07:23
*** mmirecki has joined #openstack-neutron-ovn07:38
*** yamamoto has joined #openstack-neutron-ovn07:55
*** fzdarsky has joined #openstack-neutron-ovn07:57
*** yamamoto has quit IRC08:02
*** gongysh has quit IRC08:12
*** lucas-afk is now known as lucasagomes08:15
*** dasanind has quit IRC09:14
*** gongysh has joined #openstack-neutron-ovn09:21
*** dasanind has joined #openstack-neutron-ovn09:21
*** dasanind has quit IRC09:21
*** dasanind has joined #openstack-neutron-ovn09:21
*** yamamoto has joined #openstack-neutron-ovn09:24
*** yamamoto has quit IRC09:35
*** yamamoto has joined #openstack-neutron-ovn09:45
*** mmirecki is now known as mmirecki_lunch09:54
*** mmirecki_lunch is now known as mmirecki11:18
*** saphi has quit IRC11:28
*** gongysh has quit IRC11:47
*** lucasagomes is now known as lucas-hungry11:52
dalvarezrussellb, 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 IRC11:57
*** yamamoto has joined #openstack-neutron-ovn12:13
russellbdalvarez: no12:31
russellbdalvarez: that was pre vlan-aware-vms12:31
russellbkept for backwards compat i suppose ...12:32
russellbin fact, i think there's still something that uses it let me check12:32
dalvarezrussellb, ack12:32
russellbyes ... there is a Docker networking plugin shipped with OVN that uses it still12:33
russellbovn/utilities/ovn-docker-underlay-driver12:33
*** yamamoto has quit IRC12:54
dalvarezrussellb, ack got it thanks!12:55
dalvarezrussellb, i thought of removing it from networking-ovn tho12:55
* russellb nods12:56
russellbi would support that if it wasn't used12:56
russellbso i'd want to migrate the existing code that uses it first12:56
russellband even then, you've got the past releases of OVS/OVN that still use it12:57
russellbwe're kinda stuck for a while12:57
russellbat least it's small12:57
*** mmirecki is now known as mmirecki_netcont12:58
*** mmichelson has joined #openstack-neutron-ovn13:01
*** lucas-hungry is now known as lucasagomes13:05
*** yamamoto has joined #openstack-neutron-ovn13:09
dalvarezrussellb, 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 them13:26
dalvarezfor 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 parent13:27
dalvarezrussellb, how does that sound?13:27
*** saphi has joined #openstack-neutron-ovn13:29
*** mlavalle has joined #openstack-neutron-ovn13:53
russellbdalvarez: sounds reasonable to me14:18
dalvarezrussellb, ack thanks14:18
dalvarezthat should make subports go ACTIVE14:18
dalvarezwich is needed by kuryr and probably someone else14:19
*** janki has quit IRC14:23
openstackgerritTerry Wilson proposed openstack/networking-ovn master: Prepare for using ovsdbapp > 0.4.0  https://review.openstack.org/48744214:24
*** yamamoto has quit IRC14:24
openstackgerritTerry Wilson proposed openstack/networking-ovn master: Prepare for using ovsdbapp > 0.4.0  https://review.openstack.org/48744215:02
otherwiseguyrussellb, 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
otherwiseguydalvarez, I seem to remember you using some RowEvents lately. :D15:08
russellbdalvarez: yep, just an oversight really15:08
otherwiseguyalso, I would like to profess my undying love for git bisect. that is all.15:10
russellbha, indeed handy15:10
otherwiseguyWHY 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 him15:13
*** yamamoto has joined #openstack-neutron-ovn15:24
*** yamamoto has quit IRC15:36
*** mmirecki_netcont has quit IRC15:37
dalvarezotherwiseguy, lol16:09
dalvarezlooking16:09
*** lucasagomes is now known as lucas-afk16:23
openstackgerritRussell Bryant proposed openstack/networking-ovn master: doc: Add link to tutorial.  https://review.openstack.org/48750616:28
openstackgerritRussell Bryant proposed openstack/networking-ovn master: doc: Fix list and link formatting.  https://review.openstack.org/48750716:28
dalvarezotherwiseguy, dumb question... that patch is a part of the "getting rid of eventlet" effort in ovsdbapp right?16:34
dalvarezdoes it enhance performance with native threading?16:34
otherwiseguydalvarez, 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
dalvarezotherwiseguy, gotcha16:35
otherwiseguysimilar to how the Connection spawns a thread instead of picking an event loop.16:35
dalvarezrowevent is handy :)16:35
* dalvarez remembers that we dont want to pass in a Connection because it might be not used...16:36
dalvarezbut cant remember really where :p16:36
otherwiseguydalvarez, 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
dalvarezotherwiseguy, yeah i've used it as well16:38
dalvarezotherwiseguy, ah to pick the event loop16:39
dalvarezthat's cool! and with a def timeout of 5s, neat :)16:40
otherwiseguyI 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 IRC16:51
openstackgerritMerged openstack/ovsdbapp master: Add RowEvent mechanism from networking-ovn  https://review.openstack.org/48496417:35
openstackgerritboden proposed openstack/networking-ovn master: use qos constants from neutron-lib  https://review.openstack.org/48754217:57
*** yamamoto has joined #openstack-neutron-ovn18:09
*** yamamoto has quit IRC18:13
openstackgerritTerry Wilson proposed openstack/ovsdbapp master: Return False when comparing a RowEvent to something else  https://review.openstack.org/48754718:23
openstackgerritMerged openstack/ovsdbapp master: Return False when comparing a RowEvent to something else  https://review.openstack.org/48754719:00
*** mmirecki_netcont has joined #openstack-neutron-ovn20:17
openstackgerritTerry Wilson proposed openstack/ovsdbapp master: Add OVN_Southbound API support  https://review.openstack.org/48177920:17
*** mmirecki_netcont has quit IRC20:35
*** yamamoto_ has joined #openstack-neutron-ovn21:19
*** yamamoto_ has quit IRC21:21
*** ezobn has joined #openstack-neutron-ovn21:27
*** ezobn has quit IRC21:45
*** mmichelson has quit IRC22:19
*** mlavalle has quit IRC22:31
*** yamamoto has joined #openstack-neutron-ovn23:10
*** yamamoto has quit IRC23:37
*** yamamoto has joined #openstack-neutron-ovn23:51
*** yamamoto_ has joined #openstack-neutron-ovn23:55
*** yamamoto has quit IRC23:56

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!