*** manand has quit IRC | 00:03 | |
*** salv-orl_ has quit IRC | 00:08 | |
*** gangil has quit IRC | 00:08 | |
*** dlundquist1 has joined #openstack-neutron-ovn | 00:08 | |
*** dlundquist has quit IRC | 00:11 | |
*** roeyc has joined #openstack-neutron-ovn | 00:14 | |
*** gangil has joined #openstack-neutron-ovn | 00:23 | |
*** gangil has joined #openstack-neutron-ovn | 00:23 | |
*** dlundquist1 has quit IRC | 00:26 | |
openstackgerrit | Matthew Kassawara proposed openstack/networking-ovn: Docs: Implement doc8 linter https://review.openstack.org/274948 | 00:27 |
---|---|---|
*** azbiswas has joined #openstack-neutron-ovn | 00:56 | |
*** gangil has quit IRC | 01:15 | |
*** gangil has joined #openstack-neutron-ovn | 01:15 | |
*** gangil has joined #openstack-neutron-ovn | 01:15 | |
*** chandrav has quit IRC | 01:15 | |
*** azbiswas has quit IRC | 01:54 | |
*** fzdarsky_ has joined #openstack-neutron-ovn | 02:03 | |
*** azbiswas has joined #openstack-neutron-ovn | 02:07 | |
*** fzdarsky|afk has quit IRC | 02:07 | |
*** roeyc has quit IRC | 02:09 | |
*** chandrav has joined #openstack-neutron-ovn | 02:14 | |
openstackgerrit | Merged openstack/networking-ovn: Add msec resolution to ovn-northd console logs. https://review.openstack.org/274876 | 02:22 |
*** s3wong has quit IRC | 02:27 | |
*** gangil has quit IRC | 02:28 | |
*** chandrav has quit IRC | 02:39 | |
*** azbiswas has quit IRC | 02:40 | |
*** chandrav has joined #openstack-neutron-ovn | 02:49 | |
*** thumpba has quit IRC | 02:51 | |
*** chandrav has quit IRC | 03:00 | |
*** yamamoto has joined #openstack-neutron-ovn | 03:09 | |
*** jckasper has joined #openstack-neutron-ovn | 03:14 | |
*** allan_h has quit IRC | 03:15 | |
*** gangil has joined #openstack-neutron-ovn | 03:16 | |
*** gangil has quit IRC | 03:16 | |
*** gangil has joined #openstack-neutron-ovn | 03:16 | |
*** azbiswas has joined #openstack-neutron-ovn | 03:18 | |
*** yamamoto has quit IRC | 03:20 | |
*** manand has joined #openstack-neutron-ovn | 03:32 | |
*** azbiswas_ has joined #openstack-neutron-ovn | 03:48 | |
*** azbiswas has quit IRC | 03:52 | |
*** armax has quit IRC | 04:02 | |
*** armax has joined #openstack-neutron-ovn | 04:06 | |
*** yamamoto_ has joined #openstack-neutron-ovn | 04:08 | |
*** azbiswas_ has quit IRC | 04:13 | |
*** manand has quit IRC | 04:47 | |
*** chandrav has joined #openstack-neutron-ovn | 04:51 | |
*** azbiswas has joined #openstack-neutron-ovn | 04:53 | |
*** azbiswas has quit IRC | 04:54 | |
*** gongysh has joined #openstack-neutron-ovn | 05:00 | |
*** allan_h has joined #openstack-neutron-ovn | 05:00 | |
*** allan_h has quit IRC | 05:32 | |
*** gongysh has quit IRC | 06:04 | |
*** chandrav has quit IRC | 06:22 | |
*** armax has quit IRC | 07:03 | |
*** palexster has quit IRC | 08:22 | |
*** palexster has joined #openstack-neutron-ovn | 08:26 | |
openstackgerrit | Numan Siddique proposed openstack/networking-ovn: Create periodic status task to check dhcp agents in Ovn Worker https://review.openstack.org/274864 | 08:56 |
*** numans has joined #openstack-neutron-ovn | 08:57 | |
*** gangil has quit IRC | 09:00 | |
*** roeyc has joined #openstack-neutron-ovn | 09:29 | |
*** gongysh has joined #openstack-neutron-ovn | 09:56 | |
*** openstackgerrit has quit IRC | 10:17 | |
*** openstackgerrit has joined #openstack-neutron-ovn | 10:17 | |
openstackgerrit | Babu Shanmugam proposed openstack/networking-ovn: DPDK support for OVN https://review.openstack.org/275103 | 10:20 |
*** gongysh has quit IRC | 10:25 | |
*** yamamoto_ has quit IRC | 10:37 | |
openstackgerrit | Babu Shanmugam proposed openstack/networking-ovn: Removed unnecessary code from the plugin https://review.openstack.org/275125 | 11:07 |
*** roeyc has quit IRC | 11:13 | |
*** fzdarsky_ has quit IRC | 11:47 | |
*** fzdarsky has joined #openstack-neutron-ovn | 11:47 | |
*** roeyc has joined #openstack-neutron-ovn | 12:08 | |
*** yamamoto has joined #openstack-neutron-ovn | 12:15 | |
*** yamamoto has quit IRC | 12:22 | |
*** yamamoto has joined #openstack-neutron-ovn | 12:23 | |
*** yamamoto has quit IRC | 12:27 | |
*** numan_ has joined #openstack-neutron-ovn | 12:35 | |
*** numans has quit IRC | 12:36 | |
*** roeyc has quit IRC | 12:39 | |
*** yamamoto has joined #openstack-neutron-ovn | 12:46 | |
*** roeyc has joined #openstack-neutron-ovn | 12:49 | |
*** yamamoto has quit IRC | 12:56 | |
*** gongysh has joined #openstack-neutron-ovn | 13:25 | |
*** yamamoto has joined #openstack-neutron-ovn | 13:27 | |
*** yamamoto has quit IRC | 13:27 | |
*** yamamoto has joined #openstack-neutron-ovn | 13:28 | |
*** thumpba has joined #openstack-neutron-ovn | 13:42 | |
*** regXboi has joined #openstack-neutron-ovn | 13:45 | |
*** salv-orlando has joined #openstack-neutron-ovn | 14:01 | |
*** thumpba has quit IRC | 14:06 | |
*** jckasper has quit IRC | 14:18 | |
*** jckasper has joined #openstack-neutron-ovn | 14:18 | |
*** jckasper has quit IRC | 14:20 | |
*** jckasper has joined #openstack-neutron-ovn | 14:21 | |
openstackgerrit | Merged openstack/networking-ovn: Vagrant: Add docs for instance external net access https://review.openstack.org/274856 | 14:23 |
openstackgerrit | Matthew Kassawara proposed openstack/networking-ovn: Vagrant: Modify OpenStack services https://review.openstack.org/274795 | 14:31 |
Sam-I-Am | russellb: moo. | 14:43 |
russellb | quack | 14:44 |
Sam-I-Am | russellb: can you look at https://review.openstack.org/#/c/274795/ ? | 14:44 |
russellb | I can. | 14:44 |
russellb | but will I?! | 14:44 |
Sam-I-Am | if you're nice. | 14:44 |
Sam-I-Am | once that merges... and possibly this next patch i'm working on, i'm trying to make the mtu handed out by dhcp vary based on the mtu chosen for the vagrant vm net interfaces | 14:46 |
russellb | fancy | 14:50 |
russellb | Sam-I-Am: does metadata work? | 14:50 |
Sam-I-Am | russellb: yes | 14:50 |
Sam-I-Am | on both network types | 14:50 |
russellb | yay! | 14:51 |
russellb | that's the first successful test report of metadata with OVN btw | 14:51 |
russellb | does our base devstack setup need to get fixed to make that work too? | 14:51 |
russellb | maybe for your TODO :) | 14:52 |
Sam-I-Am | russellb: my guess is the dhcp option made it work, not something specific with md itself. | 14:53 |
* russellb nods | 14:53 | |
Sam-I-Am | if you dont do enable_isolated_metadata, it relies on the router for doing some iptables magic to proxy requests | 14:53 |
Sam-I-Am | so, interesting note about enable_isolated_metadata | 14:53 |
Sam-I-Am | dhcp is enabled (or not) at subnet creation. routers always come later. so that option always assumes your network as no router and enables the dhcp option. | 14:54 |
Sam-I-Am | the router md magic only comes into play if you disable that option and always have routers | 14:55 |
russellb | Sam-I-Am: commented on your patch | 14:58 |
Sam-I-Am | russellb: hmmmmmmmmmm | 15:00 |
Sam-I-Am | russellb: i guess that depends on the chances of the proxy magic ending up in the native l3 implementation | 15:00 |
russellb | not actively worked on | 15:00 |
russellb | and likely low priority | 15:00 |
Sam-I-Am | and/or if the native dhcp agent (if that is such a thing) would handle enable_isolated_metadata | 15:01 |
russellb | yeah, the case of no l3 agent and no dhcp agent is a bit up in the air | 15:01 |
* russellb punts that down the field | 15:02 | |
Sam-I-Am | ok, so moving my dhcp thinger to the devstacks is probably better than having it in the vagrants | 15:02 |
Sam-I-Am | i had it in the vagrants to test this specific case, but it does apply to a larger audience | 15:02 |
russellb | k :) | 15:03 |
Sam-I-Am | let me fixerate, test, then beg for +2s | 15:03 |
russellb | k | 15:03 |
*** thumpba has joined #openstack-neutron-ovn | 15:13 | |
regXboi | russellb: ping | 15:18 |
regXboi | russellb: FYI, I'm trying to test out http://openvswitch.org/pipermail/dev/2016-January/064854.html and get some solid numbers on how it improves things in my tests - it looks like I'm going to have to add some instrumentation to measure loop times and loop counts as a simple view of cpu usage isn't enough to draw a conclusion :( | 15:20 |
regXboi | russellb: thus I'm wondering - does a restack recompile ovn-controllerd even if I'm not recloning? | 15:21 |
*** chandrav has joined #openstack-neutron-ovn | 15:23 | |
*** azbiswas_ has joined #openstack-neutron-ovn | 15:25 | |
*** azbiswas_ has quit IRC | 15:28 | |
russellb | regXboi: it should recompile/reinstall yes | 15:29 |
regXboi | russellb: thx | 15:29 |
russellb | regXboi: my patch is only going to make a different in particular scenarios | 15:29 |
regXboi | russellb: ack, I want to see if the one I'm testing is one of those scenarios | 15:30 |
russellb | specifically a large # of networks, where a given hypervisor only has ports on a smaller subset of those networks | 15:30 |
regXboi | well, my test should be in that mode | 15:30 |
russellb | it cuts out some processing in that case, otherwise it won't make a difference | 15:30 |
regXboi | er, wait ... no I'm not | 15:30 |
regXboi | no wait, I'm confusing myself - I am :) | 15:30 |
russellb | heh | 15:30 |
regXboi | I think :) | 15:30 |
russellb | ¯\_(ツ)_/¯ | 15:31 |
regXboi | well, I guess the easiest way is to just put in the instrumentation and run the darn thing | 15:31 |
regXboi | and then I'll know | 15:32 |
russellb | OR maybe you'll come out with more questions | 15:34 |
*** chandrav has quit IRC | 15:39 | |
*** numan_ has quit IRC | 15:43 | |
*** roeyc has quit IRC | 16:03 | |
*** yamamoto has quit IRC | 16:05 | |
*** yamamoto has joined #openstack-neutron-ovn | 16:07 | |
*** yamamoto has quit IRC | 16:08 | |
openstackgerrit | Russell Bryant proposed openstack/networking-ovn: Add more thoughts to the HA section of the FAQ. https://review.openstack.org/274844 | 16:10 |
*** gongysh has quit IRC | 16:18 | |
*** roeyc has joined #openstack-neutron-ovn | 16:19 | |
*** armax has joined #openstack-neutron-ovn | 16:52 | |
openstackgerrit | Russell Bryant proposed openstack/networking-ovn: Update the HA section of the FAQ. https://review.openstack.org/274844 | 17:06 |
*** Guest75006 has joined #openstack-neutron-ovn | 17:07 | |
*** Guest75006 has quit IRC | 17:07 | |
*** Guest75006 has joined #openstack-neutron-ovn | 17:07 | |
*** Guest75006 is now known as arosen | 17:07 | |
openstackgerrit | Matthew Kassawara proposed openstack/networking-ovn: Docs: Implement doc8 linter https://review.openstack.org/274948 | 17:07 |
Sam-I-Am | russellb: i'll be pondering your patch | 17:08 |
regXboi | and I'm restacking on the other patch :) | 17:08 |
Sam-I-Am | i'm working on elevendyseven patches at once | 17:09 |
russellb | join the clubu | 17:10 |
openstackgerrit | Matthew Kassawara proposed openstack/networking-ovn: Vagrant: Share local directories https://review.openstack.org/275331 | 17:14 |
Sam-I-Am | ^ should be an easy one | 17:14 |
*** salv-orlando has quit IRC | 17:14 | |
* russellb usually lets mestery review the vagrant changes first :) | 17:15 | |
Sam-I-Am | its slowly becoming more robust | 17:15 |
russellb | really great work | 17:16 |
Sam-I-Am | more or less simulating what we should test for prod environments... just on vms. | 17:16 |
russellb | ++++ | 17:16 |
russellb | i really like having people that understand production environments helping :) | 17:16 |
Sam-I-Am | once those patches merge, and i fix that patch you commented on, i'm trying to add automatic mtu adjustment | 17:17 |
russellb | if you need more for your todo list, figuring out a multi-node env we could run in the gate would be cool too :-D | 17:17 |
Sam-I-Am | i just had too many dependent patches locally and knew i was reaching git-destruction level | 17:17 |
Sam-I-Am | russellb: yeah that and gating with native l3 | 17:17 |
Sam-I-Am | perhaps non-voting for a while | 17:18 |
russellb | ++ | 17:18 |
russellb | i'm all for making it voting ASAP | 17:18 |
russellb | just need to have a new devstackgaterc file that disables tests as needed | 17:18 |
russellb | it'll be a lot at first | 17:18 |
russellb | but have to start somewhere | 17:19 |
arosen | russellb: did you figure what what was causing the test failure? | 17:19 |
russellb | arosen: no :( | 17:19 |
Sam-I-Am | theres some other potential problems with devstack itself that might need fixin | 17:19 |
arosen | it seemed like the failures were always in tempest.api.networking :( | 17:19 |
russellb | arosen: it's still happening occasionally, and my last lead turned out to not work out | 17:20 |
arosen | i ran it like 100 times locally and reproduced the failure twice but wasn't able to track down what is causing it yet :( | 17:20 |
russellb | well, nice that you can reproduce | 17:20 |
russellb | perhaps realted, we're getting some errors in our log even on successful runs | 17:20 |
arosen | yea but not very often | 17:20 |
russellb | so maybe that's a good place to start | 17:20 |
arosen | hard to find what's giong on in the logs. | 17:20 |
russellb | yes.. | 17:20 |
arosen | I tried putting some sleeps in the code to see if i could make it happen more easily but nothing yet. | 17:21 |
Sam-I-Am | the good news, devstack doesn't run the 'bad things' when you service_disable q-l3 ... so if you dont mind fixing that stuff in ovn's plugin, we can do that. ideally devstack needs a lot of help so plugins dont have to bandaid it. | 17:21 |
arosen | yea i definitely wanna understand whats causing this seems really weird. | 17:21 |
russellb | this is from a successful test run: http://logs.openstack.org/05/269705/6/check/gate-tempest-dsvm-networking-ovn/98526a3/logs/screen-q-svc.txt.gz?level=TRACE | 17:21 |
russellb | some weird stuff | 17:22 |
russellb | first, lots of 2016-02-01 20:23:29.031 27514 WARNING neutron.notifiers.nova [-] Nova returned NotFound for event: [{'tag': u'17bbce2d-8b02-470f-b7a6-e193c68cdfa0', 'name': 'network-vif-deleted', 'server_uuid': u'e897b7b1-dfef-4f77-ac39-60a8b4d1b9cb'}] | 17:22 |
russellb | which seems quite odd. | 17:22 |
russellb | there's a NetworkInUse failure in there, though it didn't cause the test to fail | 17:22 |
arosen | I can explain that warning :) | 17:22 |
arosen | When you delete a port | 17:22 |
russellb | ooh, please do | 17:22 |
arosen | we tell nova that the port has been deleted so it can update it's cache to remove it from the instance. | 17:23 |
arosen | so in this case the port is usually deleted by nova. | 17:23 |
arosen | nova delete vm1 | 17:23 |
arosen | so it's a race that nova doesn't have the instance running anymore. | 17:23 |
russellb | ah, maybe shouldn't be a WARNING then | 17:23 |
russellb | sounds more like DEBUG | 17:23 |
arosen | it's kinda silly. | 17:23 |
arosen | yea i agree. | 17:24 |
* russellb patches that | 17:25 | |
arosen | cool :) | 17:25 |
russellb | we also get a db deadlock traceback on port insert sometimes | 17:25 |
*** salv-orlando has joined #openstack-neutron-ovn | 17:26 | |
arosen | Yea, the dhcp-agent's rpc interface uses transactions over the plugins :( | 17:27 |
arosen | i think that's the cause of that. | 17:27 |
arosen | has there been anymore progress on the ovn dhcp service? | 17:28 |
russellb | no, it's a bit stalled | 17:29 |
russellb | there was discussion about another way to do it, which depends on a new feature ben was working on, that is under review now | 17:29 |
russellb | so progress in a sense, but we're not close | 17:29 |
*** thumpba has quit IRC | 17:29 | |
arosen | nice | 17:30 |
russellb | arosen: https://review.openstack.org/275336 | 17:31 |
*** openstackgerrit has quit IRC | 17:32 | |
russellb | bbl lunch | 17:32 |
*** openstackgerrit has joined #openstack-neutron-ovn | 17:32 | |
*** allan_h has joined #openstack-neutron-ovn | 17:36 | |
arosen | LGTM | 17:42 |
arosen | alright, going to dig back into random test failure. | 17:43 |
*** salv-orl_ has joined #openstack-neutron-ovn | 17:50 | |
*** salv-orlando has quit IRC | 17:54 | |
Sam-I-Am | russellb: pssst - https://review.openstack.org/#/c/275331/ | 18:14 |
*** roeyc has quit IRC | 18:17 | |
*** fzdarsky is now known as fzdarsky|afk | 18:19 | |
*** thumpba has joined #openstack-neutron-ovn | 18:21 | |
*** thumpba has quit IRC | 18:27 | |
*** salv-orl_ has quit IRC | 18:28 | |
*** salv-orlando has joined #openstack-neutron-ovn | 18:28 | |
*** salv-orlando has quit IRC | 18:31 | |
*** salv-orlando has joined #openstack-neutron-ovn | 18:32 | |
*** dlundquist has joined #openstack-neutron-ovn | 18:33 | |
Sam-I-Am | well this is frustrating... not vagrant isnt picking up on my plugin.sh changes | 18:37 |
*** azbiswas has joined #openstack-neutron-ovn | 18:39 | |
*** thumpba has joined #openstack-neutron-ovn | 18:42 | |
arosen | do you guys all use vagrant because you do development on a mac? | 18:43 |
*** salv-orlando has quit IRC | 18:43 | |
Sam-I-Am | arosen: i'm running vagrant on a linux box | 18:43 |
*** salv-orlando has joined #openstack-neutron-ovn | 18:45 | |
*** numan_ has joined #openstack-neutron-ovn | 18:47 | |
arosen | Does that use virtualbox on linux or i guess you can configure it to use w/e? | 18:48 |
mestery | I use Vagrant because I can quickly spin-up repeatable environments for testing arosen | 18:52 |
mestery | Sam-I-Am: I'd really like https://review.openstack.org/#/c/274795/ to land | 18:52 |
mestery | arosen: ^^^^ | 18:52 |
mestery | Sam-I-Am: Do you plan to respin the above to address russellb comment soon? | 18:53 |
Sam-I-Am | mestery: yeah, working on the underlying patch now | 18:56 |
Sam-I-Am | for some reason its not picking up my changes to plugin.sh | 18:56 |
Sam-I-Am | well, vagrant isnt... and i have it looking at my local copy of networking-ovn | 18:57 |
Sam-I-Am | mestery: its basically one thing that needs to move to plugin.sh - enable_isolated_metadata | 18:58 |
Sam-I-Am | the other setting in neutron.conf only applies for deployments with > 1 compute node... so not stock devstack. that will stay in the vagrant patch. | 18:58 |
mestery | Sam-I-Am: Ack, thanks | 18:59 |
Sam-I-Am | if this doesnt work, there must be something else preventing that iniset from being called | 18:59 |
Sam-I-Am | i ran across this problem yesterday when i try trying to make mtu-via-dhcp configurable | 18:59 |
Sam-I-Am | but i thought that was just because vagrant kept recloning networking-ovn from master | 19:00 |
arosen | boom | 19:01 |
arosen | mestery: ^ -> https://review.openstack.org/#/c/274795/ to land | 19:01 |
mestery | Sam-I-Am: Is that ok that arosen just boomed that one in? | 19:01 |
arosen | I can unboom | 19:02 |
Sam-I-Am | mestery: yep | 19:02 |
mestery | Coolio | 19:02 |
mestery | arosen: Boom away my friend | 19:02 |
mestery | :) | 19:02 |
Sam-I-Am | that one makes my other bits easier | 19:02 |
mestery | Sam-I-Am: I like to make things easier | 19:02 |
Sam-I-Am | all the dependent-but-not-really patches yesterday almost got me into trouble with git | 19:02 |
mestery | lol | 19:03 |
Sam-I-Am | i still dont know why my stuff didnt work, but fewer git hacks makes it easier to troubleshoot | 19:03 |
*** numan_ has quit IRC | 19:03 | |
openstackgerrit | Merged openstack/networking-ovn: Vagrant: Modify OpenStack services https://review.openstack.org/274795 | 19:07 |
*** jamespd_ has joined #openstack-neutron-ovn | 19:07 | |
*** dlundquist has quit IRC | 19:10 | |
*** jamespd has quit IRC | 19:12 | |
*** dlundquist has joined #openstack-neutron-ovn | 19:12 | |
azbiswas | russellb: would love your thoughts on https://review.openstack.org/#/c/274274/ when you get some time. | 19:14 |
*** gangil has joined #openstack-neutron-ovn | 19:16 | |
*** gangil has joined #openstack-neutron-ovn | 19:16 | |
Sam-I-Am | sigh... i dont know why this isnt working | 19:17 |
Sam-I-Am | my changes to plugin.sh are being ignored | 19:18 |
* mestery hands Sam-I-Am a tissue | 19:18 | |
Sam-I-Am | i even commented out the thing that i thought was working, and its still doing that thing | 19:18 |
Sam-I-Am | y u no work | 19:19 |
arosen | azbiswas: I just did a read through of your doc. | 19:21 |
arosen | in this model there is no ovn gateway? | 19:21 |
arosen | nat is done directly on the host? | 19:21 |
azbiswas | Yes nat is done directly on the host | 19:21 |
azbiswas | nat can be done either using conntrack NAT support or 1-1 translation | 19:22 |
arosen | it also assumes that the hosts are all connected to the external network? | 19:22 |
azbiswas | right | 19:22 |
arosen | in this case how does it work for a 1 to many nat? | 19:22 |
azbiswas | Floating IP is 1-1 | 19:23 |
azbiswas | We are not solving 1-many (SNAT) yet | 19:23 |
arosen | hrm, I just wonder if not solving that problem at this stage makes things more awkard. | 19:24 |
arosen | you need one - many nat to access the internet. | 19:24 |
Sam-I-Am | arosen: can you review https://review.openstack.org/#/c/274795/ ? | 19:25 |
arosen | I wonder if it makes more sense to take that into consideration of the design since this will only be something in the interim until 1 to many is there right? | 19:25 |
Sam-I-Am | errr | 19:25 |
Sam-I-Am | not that one | 19:25 |
Sam-I-Am | https://review.openstack.org/#/c/275331/ | 19:25 |
*** salv-orlando has quit IRC | 19:25 | |
azbiswas | True, in this case we are assuming that the VM needs a Floating IP to access the internet and vice-versa. We will get to SNAT (1-many) once the OVN community figures out a strategy for that. | 19:26 |
arosen | Sam-I-Am: +2 | 19:26 |
Sam-I-Am | thx | 19:26 |
Sam-I-Am | one less thing i need to mangle locally | 19:27 |
arosen | azbiswas: yea i understand. Though I wonder if implementing this in the interim makes this more complex long term? Do you think we'll be able to easily make it one to many after we implement this/ | 19:27 |
arosen | azbiswas: are the lrouter flow additions something that is going to be down out side of the current northbound interfaec? | 19:28 |
arosen | or are you saying in ovn those flows will be added? | 19:29 |
azbiswas | We've had some thoughts on how SNAT might affect this design. If you add your SNAT concerns in the comments - I can try and address them in this design as well i.e. make sure we are at least not 2 far off from SNAT changes needed. | 19:29 |
azbiswas | Yes - to flows will be added. | 19:29 |
arosen | azbiswas: sounds good i'll just follow up on the review it self. | 19:30 |
arosen | thanks! | 19:30 |
azbiswas | arosen: The new flows need to be OKed by the OVN community as well. | 19:31 |
russellb | armax: azbiswas based on the last OVN IRC meeting, we're expecting a proposed OVN gateway design that includes NAT in the next few weeks | 19:33 |
russellb | i imagine we'll need to merge these proposals somehow | 19:33 |
armax | russellb: I supposed you wanted to tag arosen instead, didn’t you? | 19:34 |
russellb | correct. | 19:34 |
russellb | sorry. | 19:34 |
azbiswas | That's good to know - will the gateway reside on the host? | 19:35 |
mestery | armax: We just want you to know we're always thinking of you over here | 19:35 |
russellb | i think it's a proposal for a central gateway (or set of gateays) i don't know yet, not my proposal | 19:35 |
armax | mestery: oh | 19:36 |
russellb | was hoping we could make the 2 work together | 19:36 |
mestery | I bet it follows this: https://github.com/openvswitch/ovs/blob/master/ovn/OVN-GW-HA.md | 19:36 |
russellb | armax: it's true | 19:36 |
mestery | :) | 19:36 |
armax | russellb: that you thinking of me constantly? | 19:36 |
armax | russellb: or that you wanted to ping arosen? | 19:36 |
russellb | let's not get too personal | 19:36 |
mestery | armax:#neverforget | 19:37 |
armax | mestery: that’s eerily scary | 19:37 |
* armax goes and touches wood | 19:37 | |
mestery | armax: Don't read too much into it | 19:37 |
mestery | armax: That's just wrong | 19:37 |
* azbiswas reading https://github.com/openvswitch/ovs/blob/master/ovn/OVN-GW-HA.md | 19:40 | |
russellb | it won't necessraily follow that | 19:40 |
russellb | that was kind of a brain dump from someone before they left vmware, heh | 19:40 |
mestery | heh :) | 19:41 |
azbiswas | to where? | 19:41 |
arosen | armax: | 19:41 |
arosen | whoops sorry i mean't russellb | 19:41 |
arosen | ;P | 19:42 |
armax | lame | 19:42 |
armax | I am busy here you know? | 19:42 |
russellb | grad school | 19:42 |
mestery | armax: You can't fool us | 19:42 |
* armax put OVN in the his blacklist | 19:42 | |
mestery | lol | 19:43 |
russellb | :( | 19:43 |
russellb | armax: we try so hard to play nice though! | 19:43 |
armax | I am muting notifications from this channel…just so you know! | 19:44 |
russellb | :((( | 19:44 |
armax | russellb, mestery, arosen ^^^^ | 19:44 |
russellb | alright guys, time to pack up this room and move to #openstack-neutron | 19:44 |
mestery | lol | 19:44 |
mestery | :) | 19:44 |
armax | oh boy | 19:44 |
mestery | russellb armax: We can just bug him on google hangout chat :) | 19:45 |
armax | I long for simpler times | 19:45 |
Sam-I-Am | this looks transient? https://review.openstack.org/#/c/274948/ | 19:48 |
Sam-I-Am | russellb: ^ | 19:50 |
russellb | yes | 19:53 |
russellb | i'm going to offer a bounty of a large basket of cookies for fixing that soon | 19:53 |
russellb | i'm trying not to think about it this minute ... i don't want to lose the peace I found during my lunchtime walk on the beach | 19:56 |
mestery | Nice russellb :) | 19:57 |
Sam-I-Am | wait, what is this beach thing? | 19:59 |
Sam-I-Am | cookies? | 19:59 |
russellb | you heard me! | 19:59 |
*** thumpba has quit IRC | 20:00 | |
Sam-I-Am | crap, well i may just let the gate check this devstack patch | 20:01 |
russellb | that's not what everyone does for every patch? | 20:01 |
Sam-I-Am | i like to test things locally when i can | 20:02 |
Sam-I-Am | i suspect something is weird with vagrant though | 20:02 |
Sam-I-Am | just cant put my finger on it | 20:02 |
regXboi | russellb: is there anybody working on incremental processing within lflow_run/ | 20:04 |
regXboi | er ? | 20:04 |
russellb | ben said he was going to work on it | 20:04 |
regXboi | ok, I may ask if there is anything to test on Thursday | 20:05 |
russellb | ok, i don't think that work has started yet | 20:05 |
openstackgerrit | Matthew Kassawara proposed openstack/networking-ovn: Devstack: Provide metadata route via DHCP https://review.openstack.org/275392 | 20:06 |
openstackgerrit | Merged openstack/networking-ovn: Removed unnecessary code from the plugin https://review.openstack.org/275125 | 20:07 |
openstackgerrit | Merged openstack/networking-ovn: Vagrant: Share local directories https://review.openstack.org/275331 | 20:07 |
russellb | regXboi: there's some similar talk happening around ovn-northd | 20:09 |
russellb | but no code yet | 20:09 |
regXboi | russellb: for a test of stamping out 800 copies of n1 -- r1 -- x1 (each in separate projects), your patches in the ovn-controller-perf branch result in a 2% improvement in the number of flows processed, a 10% improvement in the number of matches processed, and a 16% improvement in the time spent in lflow_run after the test completes | 20:09 |
regXboi | I'm going to find your patch in ovs-dev and reply with that information | 20:09 |
mestery | Sam-I-Am: With 275331 merged, I'm seeing htis error on a fresh vagrant: http://paste.openstack.org/show/485769/ | 20:09 |
regXboi | and say this is a good first step, but things are still supra linear and so incremental processing is a must | 20:10 |
russellb | yes, it's just a short term low hanging fruit optimization | 20:10 |
Sam-I-Am | mestery: yeah, you gotta make those. i thought i put that in the docs. | 20:10 |
russellb | incremental processing is absolutely the answer for real performance improvements here | 20:10 |
* regXboi goes to look for the patch | 20:10 | |
mestery | Sam-I-Am: Who reads docs? | 20:10 |
* mestery ducks | 20:10 | |
Sam-I-Am | rather, i think the docs say to clone those repos prior to the vagrants | 20:10 |
Sam-I-Am | vagrant lacks a way to handle missing shared dirs, so there's a hack i put into setup-base to check if they're at least a git repo | 20:11 |
Sam-I-Am | but still requires that the dirs exist in some shape or form | 20:11 |
Sam-I-Am | mestery: sigh @ docs ... :/ | 20:11 |
mestery | Sam-I-Am: How does it work sharing hte same ~/devstack across all the VMs in that setup if htey all need different local.conf files? | 20:12 |
mestery | Does that work? | 20:12 |
Sam-I-Am | ok, i'm going to be reaaaaally happy when my new keyboard and mouse arrive | 20:12 |
Sam-I-Am | mestery: best i could tell, it builds vms in series, so it overwrites the config file before deploying the next one | 20:14 |
Sam-I-Am | or it should... i could have missed something | 20:14 |
mestery | Sam-I-Am: I'm slightly nervous about that, I'd almost prefer to *not* share devstack and only share networking-ovn for this reason | 20:14 |
Sam-I-Am | ok. my primary concern was sharing networking-ovn to test local changes. devstack was just another attempt at reducing build time. | 20:15 |
mestery | Sam-I-Am: Patch inbound | 20:15 |
Sam-I-Am | mestery: maybe just comment it out by default? | 20:15 |
mestery | Sam-I-Am: Exactly | 20:15 |
Sam-I-Am | might need some docs around it | 20:15 |
Sam-I-Am | you know, those things. | 20:16 |
openstackgerrit | Kyle Mestery proposed openstack/networking-ovn: Vagrant: Stop sharing the ~/devstack directory https://review.openstack.org/275397 | 20:18 |
mestery | Sam-I-Am: ^^^ | 20:18 |
Sam-I-Am | mestery: lgtm | 20:21 |
* Sam-I-Am finds +2 thing | 20:21 | |
mestery | Nice! | 20:22 |
*** salv-orlando has joined #openstack-neutron-ovn | 20:28 | |
Sam-I-Am | russellb: instances attached to provider nets shouldnt hit any tunnels, right? | 20:51 |
*** thumpba has joined #openstack-neutron-ovn | 20:51 | |
russellb | correct | 20:52 |
russellb | unless you've set up the network the provider net is mapped to with tunnels | 20:53 |
Sam-I-Am | yeah | 20:53 |
Sam-I-Am | so this mtu stuff will be more of a hack | 20:53 |
russellb | ovn doesn't know what's on the other side of the bridge | 20:53 |
Sam-I-Am | reality is, we dont need to pass option 26 to provider nets, just self-service/private nets | 20:53 |
* regXboi ponders how to cheaply implement some sort of incremental processing in lflow_run | 20:54 | |
russellb | ok | 20:54 |
Sam-I-Am | but theres no clean way to do that | 20:54 |
Sam-I-Am | its all or nothing with neutron right now | 20:54 |
Sam-I-Am | its broken now and i think my patch makes it less broken, but we're not going to solve this now | 20:54 |
*** fzdarsky|afk has quit IRC | 20:55 | |
*** thumpba has quit IRC | 20:56 | |
*** thumpba has joined #openstack-neutron-ovn | 20:59 | |
*** s3wong has joined #openstack-neutron-ovn | 21:02 | |
Sam-I-Am | russellb: should i use the somewhat-broken path_mtu option or the broken-in-different ways dnsmasq.conf option? | 21:03 |
Sam-I-Am | path mtu might provide the correct mtu for provider nets | 21:04 |
Sam-I-Am | thats the only benefit there | 21:04 |
*** salv-orl_ has joined #openstack-neutron-ovn | 21:04 | |
*** salv-orlando has quit IRC | 21:04 | |
russellb | Sam-I-Am: you tell me | 21:05 |
Sam-I-Am | russellb: sigh, nevermind. path_mtu is ML2 | 21:05 |
Sam-I-Am | yet somehow, advertise_mtu is neutron itself :/ | 21:06 |
* Sam-I-Am facepalm | 21:06 | |
Sam-I-Am | being that we're not ml2, we probably need our own bits for handing mtu values to instances after subtracting any overlay overhead | 21:07 |
*** salv-orl_ has quit IRC | 21:08 | |
*** salv-orlando has joined #openstack-neutron-ovn | 21:08 | |
Sam-I-Am | russellb: if your head hasnt exploded yet, give it time. | 21:10 |
russellb | i think i have to be able to absorb any of it to get to the exploding part | 21:10 |
Sam-I-Am | but long term, just using dnsmasq.conf to set option 26 is not going to work in production. | 21:10 |
openstackgerrit | Richard Theis proposed openstack/networking-ovn: WIP: Add network availability zone support https://review.openstack.org/275415 | 21:30 |
openstackgerrit | Matthew Kassawara proposed openstack/networking-ovn: Devstack: Add native MTU option https://review.openstack.org/275418 | 21:33 |
Sam-I-Am | russellb: ^ its hacky, but the best we can do right now... i think. | 21:35 |
russellb | thanks! | 21:35 |
Sam-I-Am | this is one of those problems where i really dont know where to start | 21:35 |
Sam-I-Am | the discussion for fixing mtu is all around the ml2 plug-in | 21:35 |
Sam-I-Am | so we'll probably need to implement something different for ovn that makes use of the mtu extension (which is semi-useless) | 21:36 |
Sam-I-Am | even if ovs doesnt have mtu problems, instances will | 21:36 |
*** flaviof has joined #openstack-neutron-ovn | 21:40 | |
*** azbiswas has quit IRC | 21:45 | |
*** flaviof has quit IRC | 21:51 | |
*** flaviof has joined #openstack-neutron-ovn | 21:51 | |
*** azbiswas has joined #openstack-neutron-ovn | 21:53 | |
*** thumpba has quit IRC | 21:54 | |
Sam-I-Am | bbiab - venturing away from screen | 22:00 |
openstackgerrit | Richard Theis proposed openstack/networking-ovn: WIP: Add network availability zone support https://review.openstack.org/275415 | 22:12 |
*** markmcclain has quit IRC | 22:25 | |
*** markmcclain has joined #openstack-neutron-ovn | 22:26 | |
regXboi | russellb: two part question for you if you are around: (1) do you know where networking-ovn defines its OVS Transaction Timeout from and (2) can that be changed via a configuration setting? | 22:31 |
russellb | 1) off hand no | 22:31 |
russellb | and so 2) don't know | 22:32 |
russellb | :) | 22:32 |
regXboi | ok I'll go spend some time tomorrow and look | 22:32 |
regXboi | because I think in the short term we are going to want to change (2) | 22:33 |
russellb | ok | 22:33 |
russellb | yeah i'd have to dig myself | 22:33 |
regXboi | no worries - I asked, I'll pick up the shovel | 22:33 |
russellb | heh, ok | 22:34 |
arosen | regXboi: it's in the config file for ovn | 22:34 |
arosen | ovsdb_connection_timeout | 22:34 |
* arosen *the plugin not ovn* | 22:34 | |
arosen | https://github.com/openstack/networking-ovn/blob/master/networking_ovn/common/config.py#L22 | 22:34 |
regXboi | arosen: thx - that was simple enough :) | 22:34 |
* russellb out for the night | 22:37 | |
russellb | ttyl | 22:37 |
arosen | later! | 22:38 |
regXboi | o/ | 22:38 |
regXboi | o\ | 22:38 |
*** mamulsow has joined #openstack-neutron-ovn | 22:38 | |
*** rtheis has quit IRC | 22:38 | |
*** armax_ has joined #openstack-neutron-ovn | 22:44 | |
*** armax has quit IRC | 22:45 | |
*** armax_ is now known as armax | 22:45 | |
*** azbiswas has quit IRC | 23:00 | |
*** azbiswas has joined #openstack-neutron-ovn | 23:01 | |
*** azbiswas_ has joined #openstack-neutron-ovn | 23:02 | |
*** gangil has quit IRC | 23:04 | |
*** azbiswas has quit IRC | 23:05 | |
*** gangil has joined #openstack-neutron-ovn | 23:06 | |
*** gangil has joined #openstack-neutron-ovn | 23:06 | |
*** salv-orlando has quit IRC | 23:13 | |
*** salv-orlando has joined #openstack-neutron-ovn | 23:13 | |
openstackgerrit | Aaron Rosen proposed openstack/networking-ovn: test https://review.openstack.org/275439 | 23:18 |
*** regXboi has quit IRC | 23:21 | |
*** thumpba has joined #openstack-neutron-ovn | 23:34 | |
openstackgerrit | Matthew Kassawara proposed openstack/networking-ovn: Devstack: Provide metadata route via DHCP https://review.openstack.org/275392 | 23:40 |
*** gangil has quit IRC | 23:48 | |
*** gangil has joined #openstack-neutron-ovn | 23:54 | |
*** gangil has joined #openstack-neutron-ovn | 23:54 | |
openstackgerrit | Matthew Kassawara proposed openstack/networking-ovn: Floating IP support design (ready for review). https://review.openstack.org/274274 | 23:58 |
*** gangil has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!