| openstackgerrit | Gal Sagie proposed stackforge/networking-ovn: Extract ovn_name method to utils file https://review.openstack.org/178507 | 05:08 |
|---|---|---|
| *** ajo has quit IRC | 07:01 | |
| *** ajo has joined #openstack-neutron-ovn | 07:51 | |
| openstackgerrit | Gal Sagie proposed stackforge/networking-ovn: Reduce the number of OVSDB transactions https://review.openstack.org/178540 | 07:52 |
| *** gsagie has joined #openstack-neutron-ovn | 08:05 | |
| *** openstackgerrit has quit IRC | 12:07 | |
| *** openstackgerrit has joined #openstack-neutron-ovn | 12:07 | |
| openstackgerrit | Gal Sagie proposed stackforge/networking-ovn: Security Groups Data Model https://review.openstack.org/175149 | 13:28 |
| openstackgerrit | Merged stackforge/networking-ovn: Extract ovn_name method to utils file https://review.openstack.org/178507 | 14:11 |
| openstackgerrit | Merged stackforge/networking-ovn: Security Groups Data Model https://review.openstack.org/175149 | 14:12 |
| russellb | good progress on the OVN side too | 14:14 |
| russellb | 2 patch series being reviewed now - one that generates flows and the other that creates tunnels | 14:15 |
| russellb | close to being able to do real testing :) | 14:15 |
| mestery | russellb: Sweet! | 14:28 |
| *** shettyg has joined #openstack-neutron-ovn | 14:36 | |
| *** wznoinsk has joined #openstack-neutron-ovn | 14:43 | |
| *** markmcclain has joined #openstack-neutron-ovn | 15:06 | |
| *** wznoinsk_ has joined #openstack-neutron-ovn | 15:14 | |
| *** wznoinsk__ has joined #openstack-neutron-ovn | 15:18 | |
| *** wznoinsk has quit IRC | 15:23 | |
| *** wznoinsk__ has quit IRC | 15:24 | |
| *** wznoinsk_ is now known as wznoinsk | 15:24 | |
| *** blp has joined #openstack-neutron-ovn | 19:05 | |
| otherwiseguy | russellb: doh, rebased and the lport_status_up command I was abusing to test my update notification is gone. :) | 19:25 |
| russellb | how did you lose it exactly? | 19:25 |
| russellb | rebasing shouldn't have done that | 19:26 |
| otherwiseguy | russellb: I was using it in a script: api.set_lport_up_status(sys.argv[1], True).execute(), but it doesn't exist anymore. | 19:30 |
| otherwiseguy | I was just doing neutron port-creates and then calling my 'up' script. Because I'm lazy. | 19:31 |
| russellb | otherwiseguy: still not sure how a rebase could make code disappear :) | 19:31 |
| russellb | also not sure if you're asking for help, or just chatting :) | 19:31 |
| otherwiseguy | This coffee shop has a horrible ISP. | 19:34 |
| otherwiseguy | russellb: not sure if my response went through or if you replied. | 19:35 |
| otherwiseguy | russellb: Was just testing by doing neutron port-create, then calling script that did api.set_lport_up_status($port_id, True).execute(), but since set_lport_up_status was removed, that stopped working. So I'll have to start testing the 'real' way I guess. :D | 19:36 |
| russellb | ohhhh, got it | 19:37 |
| russellb | yeah, that's not something we'd ever call :-) | 19:37 |
| * otherwiseguy looks up what the real way to test is | 19:38 | |
| russellb | ha | 19:38 |
| russellb | well, you have 2 options | 19:38 |
| russellb | well not really | 19:38 |
| russellb | because ovn-northd and ovn-controller will step on you if you try to override it | 19:38 |
| russellb | basically ... you need to manually create a port on the integration bridge with an iface-id equal to the logical port name | 19:39 |
| russellb | and that should do it | 19:39 |
| russellb | using ovs-vsctl | 19:39 |
| * russellb wonders if he made sense there | 19:39 | |
| otherwiseguy | Sounds reasonable anyway. I'll try. :) | 19:39 |
| russellb | ok | 19:39 |
| russellb | ovn-controller is monitoring changes to the local switch config | 19:40 |
| russellb | it looks for ports on the integration bridge (br-int by default) and checks its external_ids | 19:40 |
| russellb | if the iface-id external_id == a logical port name, it says "AH HA! A logical port has appeared locally" | 19:40 |
| russellb | and so it updates the Bindings table in OVN_Southbound to say that the logical port exists on that system | 19:41 |
| russellb | and then ovn-northd sees that update, and updates the logical port to 'up' in OVN_Northbound | 19:41 |
| russellb | normally it'd be nova creating that port | 19:42 |
| russellb | but we're not there yet :) | 19:42 |
| russellb | CLOSE THOUGH | 19:42 |
| russellb | sooooooooooo close. | 19:42 |
| blp | With Justin's and my outstanding patches, there's only one little piece left. I'm working on that piece. | 19:44 |
| blp | If this conference call would finish up, then I'd actually make some progress on it. | 19:44 |
| russellb | blp: gah, work getting in the way of work | 19:45 |
| russellb | blp: i'm excited though :) | 19:45 |
| otherwiseguy | russellb: huh, it worked. :) | 19:54 |
| russellb | otherwiseguy: how 'bout that | 19:54 |
| openstackgerrit | Terry Wilson proposed stackforge/networking-ovn: Add port 'up' notification https://review.openstack.org/178826 | 20:00 |
| russellb | woooo | 20:00 |
| otherwiseguy | russellb: It's a bit...thready. | 20:01 |
| russellb | Python and threads, that always ends well | 20:01 |
| otherwiseguy | russellb: was already using them in the neutron transaction handling code...and just figured I'd keep going. :P Not hard to convert to anything else. | 20:02 |
| russellb | cool, will take a look | 20:02 |
| russellb | had one patch i wanted to do real quick before i forgot ... | 20:02 |
| otherwiseguy | pretty much just s/threading._start_new_thread/eventlet.spawn_n/ | 20:02 |
| otherwiseguy | It's just a little crazy because calling plugin.update_port_status() will end up calling update_port_postcommit() which runs ovn db transactions of its own, but its already in the middle of running a transaction when the notification happens. | 20:04 |
| russellb | oh dear | 20:04 |
| otherwiseguy | Anyway, I was starting to get really obsessy over all of the details, which is usually the time that I should just throw it out into the wild and get comments. :p | 20:05 |
| russellb | will review once i get this other patch tested and posted :) | 20:05 |
| russellb | should be quick | 20:05 |
| * otherwiseguy goes back to looking at cloud-init ipv6 | 20:05 | |
| russellb | i'm sure i jinxed myself | 20:05 |
| * otherwiseguy cries a little | 20:05 | |
| russellb | heh | 20:05 |
| * russellb grumbles about having to wait for stack.sh to run | 20:06 | |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: Configure OVN port security https://review.openstack.org/178830 | 20:23 |
| blp | I am *so* neglecting code reviews and email this week. Trying to get OVN pinging. | 20:39 |
| otherwiseguy | blp: good. :p | 20:42 |
| russellb | blp: GOGOGOGOGOGO | 20:45 |
| russellb | blp: and as soon as it does, we'll try to get it pinging with VMs set up by OpenStack ASAP | 20:46 |
| russellb | and THEN | 20:46 |
| russellb | ... there will be cake. | 20:46 |
| russellb | maybe. | 20:46 |
| russellb | otherwiseguy: i'm tired and medicated on cold meds ... i think reviewing your patch for real might have to wait until tomorrow | 20:46 |
| blp | It takes an OVN to bake a cake. | 20:46 |
| russellb | blp: zing | 20:46 |
| russellb | blp: http://openvswitch.org/pipermail/dev/2015-April/054594.html ... and search for "preheat" | 20:47 |
| russellb | I made myself giggle. | 20:47 |
| * otherwiseguy chuckles | 20:48 | |
| russellb | (no need to actually review that, needs to be reworked) | 20:48 |
| blp | russellb: haha | 20:48 |
| otherwiseguy | russellb: hope you get to feeling better. | 20:49 |
| russellb | thanks, been week and a half now, ugh | 20:49 |
| blp | russellb: same from me | 20:49 |
| otherwiseguy | ick. | 20:49 |
| otherwiseguy | For me it's just allergies. Every year. | 20:49 |
| * otherwiseguy glares at Spring | 20:49 | |
| blp | I'm excited for summer because we just booked a trip to see our friends in Bosnia for 2 weeks in August. | 20:50 |
| russellb | nice! never been, but travel is fun | 20:51 |
| *** ajo has quit IRC | 20:51 | |
| russellb | i've got a couple of Caribbean summer trips planned though :) | 20:51 |
| otherwiseguy | russellb: a *couple* Caribbean trips? | 20:52 |
| otherwiseguy | nice. :) | 20:52 |
| russellb | yeahhhhh | 20:52 |
| russellb | we planned one ... and then a friend is getting married the next month down there, and clearly we should go | 20:52 |
| russellb | so ... now two! | 20:52 |
| otherwiseguy | I keep meaning to plan a "just sit back and relax on an island somewhere where people bring us drinks" kind of vacation. | 20:53 |
| russellb | ++ | 20:53 |
| russellb | that's what both of these are | 20:54 |
| otherwiseguy | Our previous vacations have been of the "rush around and make sure you see everything" type vacations. | 20:54 |
| russellb | i like those too sometimes | 20:55 |
| blp | I hope that 2 weeks in one place will be long enough to relax for a while. | 20:55 |
| russellb | blp: sounds like enough to do some relaxing, some exploring, and even some relaxing exploring | 20:55 |
| * russellb out ... talk to you all tomorrow | 21:02 | |
| blp | later! | 21:03 |
| * otherwiseguy waves | 21:10 | |
| *** shettyg has quit IRC | 22:38 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!