flaviof | to all the Asian (Chinese calendar) folks in this channel: happy new year!!! | 00:02 |
---|---|---|
*** salv-orl_ has quit IRC | 00:19 | |
*** allan_h has quit IRC | 00:22 | |
*** allan_h has joined #openstack-neutron-ovn | 00:28 | |
*** salv-orlando has joined #openstack-neutron-ovn | 01:19 | |
*** brad_behle has quit IRC | 01:25 | |
*** chandrav has joined #openstack-neutron-ovn | 01:26 | |
*** yamamoto_ has joined #openstack-neutron-ovn | 01:48 | |
*** salv-orlando has quit IRC | 01:50 | |
*** chandrav has quit IRC | 01:52 | |
*** yamamoto_ has quit IRC | 02:07 | |
*** yamamoto_ has joined #openstack-neutron-ovn | 02:08 | |
*** yamamoto_ has quit IRC | 02:10 | |
*** yamamoto_ has joined #openstack-neutron-ovn | 02:10 | |
*** yamamoto_ has quit IRC | 02:10 | |
*** yamamoto_ has joined #openstack-neutron-ovn | 02:58 | |
*** salv-orlando has joined #openstack-neutron-ovn | 03:10 | |
*** salv-orlando has quit IRC | 03:13 | |
*** jckasper has joined #openstack-neutron-ovn | 03:25 | |
*** jckasper has quit IRC | 04:05 | |
*** jckasper has joined #openstack-neutron-ovn | 04:06 | |
*** jckasper has quit IRC | 04:07 | |
*** jckasper has joined #openstack-neutron-ovn | 04:07 | |
*** chandrav has joined #openstack-neutron-ovn | 04:14 | |
*** chandrav has quit IRC | 04:34 | |
*** salv-orlando has joined #openstack-neutron-ovn | 04:44 | |
*** salv-orlando has quit IRC | 04:56 | |
*** chandrav has joined #openstack-neutron-ovn | 05:05 | |
*** chandrav has quit IRC | 05:07 | |
*** chandrav has joined #openstack-neutron-ovn | 05:07 | |
*** chandrav has quit IRC | 05:12 | |
*** numans has joined #openstack-neutron-ovn | 06:34 | |
*** allan_h has quit IRC | 06:39 | |
*** salv-orlando has joined #openstack-neutron-ovn | 06:48 | |
*** salv-orlando has quit IRC | 06:55 | |
*** gsagie has joined #openstack-neutron-ovn | 07:12 | |
*** salv-orlando has joined #openstack-neutron-ovn | 08:07 | |
*** salv-orlando has quit IRC | 08:12 | |
*** openstackgerrit has quit IRC | 09:02 | |
*** openstackgerrit has joined #openstack-neutron-ovn | 09:02 | |
*** salv-orlando has joined #openstack-neutron-ovn | 09:31 | |
*** salv-orl_ has joined #openstack-neutron-ovn | 10:40 | |
*** salv-orlando has quit IRC | 10:43 | |
*** yamamoto_ has quit IRC | 11:27 | |
*** numans has quit IRC | 12:03 | |
*** chandrav has joined #openstack-neutron-ovn | 12:07 | |
*** salv-orl_ has quit IRC | 12:18 | |
*** yamamoto has joined #openstack-neutron-ovn | 12:21 | |
*** roeyc has joined #openstack-neutron-ovn | 12:27 | |
*** rtheis has joined #openstack-neutron-ovn | 12:30 | |
*** salv-orlando has joined #openstack-neutron-ovn | 12:42 | |
*** yamamoto has quit IRC | 12:42 | |
*** yamamoto has joined #openstack-neutron-ovn | 12:43 | |
*** yamamoto has quit IRC | 12:48 | |
*** yamamoto has joined #openstack-neutron-ovn | 12:58 | |
*** jckasper has quit IRC | 13:05 | |
*** jckasper has joined #openstack-neutron-ovn | 13:05 | |
*** gal_ has joined #openstack-neutron-ovn | 13:06 | |
*** jckasper has quit IRC | 13:10 | |
*** gsagie has quit IRC | 13:10 | |
*** gal_ is now known as gsagie | 13:12 | |
*** yamamoto has quit IRC | 13:27 | |
*** tongli has joined #openstack-neutron-ovn | 13:38 | |
*** jckasper has joined #openstack-neutron-ovn | 13:42 | |
*** yamamoto has joined #openstack-neutron-ovn | 13:58 | |
*** brad_behle has joined #openstack-neutron-ovn | 14:01 | |
Sam-I-Am | mornings | 14:03 |
*** gsagie has quit IRC | 14:05 | |
russellb | morning | 14:06 |
Sam-I-Am | today should be a holiday for denverites | 14:08 |
*** gsagie has joined #openstack-neutron-ovn | 14:14 | |
mestery | Sam-I-Am: Congrats on that amazing defensive win | 14:15 |
mestery | What a good game | 14:15 |
russellb | defensive win is a good way to put it | 14:16 |
russellb | wow that defense .. | 14:17 |
russellb | suffocating | 14:17 |
mestery | I know, right? | 14:17 |
mestery | Amazing | 14:17 |
Sam-I-Am | yeah it was pretty intense | 14:17 |
tongli | guys, come on. people in carolinas hurt pretty bad! | 14:17 |
Sam-I-Am | my theory was if we could stop cam's offense early, we'd have a much better chance | 14:17 |
russellb | tongli: i live in the carolinas :) | 14:17 |
Sam-I-Am | cuz the denver offense is terrible | 14:17 |
russellb | your defense stepped up in a big way | 14:18 |
russellb | impressive. | 14:18 |
Sam-I-Am | i was stuck in traffic for an hour and a half at 10 last night just trying to get back home... which is downtown. | 14:18 |
Sam-I-Am | the police pretty much closed off downtown | 14:18 |
russellb | you live in denver? | 14:18 |
Sam-I-Am | yeah | 14:18 |
Sam-I-Am | this city was WILD last night | 14:18 |
russellb | i bet :) | 14:18 |
Sam-I-Am | horns blaring, people screaming, people just... everywhere. | 14:19 |
Sam-I-Am | i've lived in a small college town during a national championship win, but not a big city like this | 14:19 |
tongli | I went to bed after the third period. | 14:19 |
russellb | i thought you lived in texas, but i probably just made that up since you worked for rax... | 14:19 |
Sam-I-Am | every time i tried to sleep, more fireworks went off | 14:19 |
Sam-I-Am | russellb: rax moved me from denver to texas. survived a year, barely. moved back to denver as a remote worker. | 14:20 |
russellb | tongli: i probably should have | 14:20 |
russellb | Sam-I-Am: ah, got it | 14:20 |
Sam-I-Am | i dont seem to function well outside of denver | 14:20 |
Sam-I-Am | that was my fourth time moving back here | 14:20 |
russellb | i can relate to wanting to just stay "home" | 14:20 |
russellb | i'm back to where i grew up working remote | 14:20 |
russellb | seems unlikely that i'll ever move out of the area | 14:21 |
Sam-I-Am | i'm trying not to move again. the other moves were not my choice. | 14:21 |
* russellb nods | 14:21 | |
Sam-I-Am | the economy was ummmm fun between 08 and 11, so i got to bounce around a bit. | 14:21 |
russellb | my moves were school and first big boy job | 14:21 |
russellb | yeah those were rough years.. | 14:22 |
Sam-I-Am | definitely some ptsd from those times | 14:23 |
Sam-I-Am | you dont forget watching good people with families get laid off | 14:23 |
russellb | me too | 14:23 |
russellb | yep, i had that | 14:23 |
*** yamamoto has quit IRC | 14:28 | |
Sam-I-Am | hmm @ trying to get devstack to use a local repo and branch for enable_plugin | 14:32 |
Sam-I-Am | its just not a happy camper | 14:32 |
*** regXboi has joined #openstack-neutron-ovn | 14:33 | |
russellb | i believe in you :-p | 14:37 |
* Sam-I-Am looks for the magic switch | 14:38 | |
*** salv-orlando has quit IRC | 14:41 | |
Sam-I-Am | i think that worked | 14:41 |
russellb | Mic22: hi! | 14:47 |
Mic22 | Hi! | 14:48 |
Mic22 | the patch is here: http://pastebin.com/BD0TBD7e | 14:48 |
russellb | so you're working on splitting ovn nb and sb dbs? | 14:48 |
Mic22 | I splitted ovn dbs from the master ovs db | 14:48 |
Mic22 | because in HA if you have those tight it's painfull | 14:48 |
Mic22 | when you move OVN you don't usually need to move ovs | 14:49 |
Mic22 | especially if you deploy compute nodes tight to controller nodes | 14:49 |
* regXboi reads patch | 14:49 | |
Mic22 | so I updated the ctl script to run his ovn db and the nbctl and sbctl to point to the proper db | 14:49 |
Mic22 | it's a small patch I tested on my env and it works but I would like some advices | 14:50 |
* russellb reading | 14:50 | |
mestery | Mic22: Nice! regXboi: ^^^^ | 14:50 |
* regXboi reads patch | 14:51 | |
regXboi | well hold on... this patch doesn't actually split NB and SB yet | 14:51 |
russellb | sort of a step 1 of that | 14:51 |
russellb | Mic22: this looks like a good start | 14:51 |
regXboi | ah ok... I was thinking this was further along... | 14:52 |
russellb | Mic22: ovn-northd should probably get a --ovnsb-db arg passed as well | 14:52 |
russellb | Mic22: and it would be nice to extend this a little more to run ovsdb-server twice (once for NB and once for SB) | 14:53 |
russellb | and in fact, make it possible to run only one or the other, or both | 14:53 |
regXboi | russellb: the ptcp port needs to be an enviro variable as well | 14:53 |
russellb | i imagine the most common case is running both, but separate processes | 14:53 |
russellb | so i'd be ok with just that (and not support for *only* running one of them) | 14:53 |
regXboi | so that you can have the NB listen on one port and the SB on a different port | 14:53 |
russellb | yes | 14:53 |
regXboi | pretty much, just copy what is in 276364 and 276454 and then we can abandon those | 14:54 |
russellb | Mic22: the "proper" way to post this for feedback is to post the patch to the ovs dev mailing list | 14:54 |
russellb | Check out the "git-send-email" tool | 14:54 |
*** salv-orlando has joined #openstack-neutron-ovn | 14:55 | |
russellb | $ git send-email --to=dev@openvswitch.org -1 | 14:55 |
russellb | and then i can provide complete feedback there | 14:55 |
Mic22 | I will post this with the separation of dbs | 14:56 |
Mic22 | it's a little step forward to be done | 14:56 |
russellb | OK great | 14:56 |
russellb | you'll also need to add a "Signed-off-by" header to the commit message | 14:57 |
Mic22 | so to recap ptcp ort should be env | 14:57 |
russellb | there's a bit more about that in https://github.com/openvswitch/ovs/blob/master/CONTRIBUTING.md | 14:57 |
regXboi | yes, the vconsole and vsyslogs need to be enviros as well | 14:58 |
regXboi | and vfile for that matter | 14:58 |
regXboi | because we'll need to be able to set the logging from inside the devstack plugin to work with screen (where we can) | 14:58 |
Mic22 | you mean this part -vconsole:emer -vsyslog:err -vfile:info | 14:58 |
regXboi | Mic22: yes, and the -vconsole:off on L50 | 14:59 |
Mic22 | but in start_northd this is: set "$@" -vconsole:emer -vsyslog:err -vfile:info | 14:59 |
regXboi | yes... I'm talking about both | 15:00 |
Mic22 | so step by step for now I will focus on port and divide dbs | 15:00 |
Mic22 | later on we can put this stuff in proper variables | 15:00 |
regXboi | Mic22: ack - sounds good, we can tune the rest in later revisions | 15:00 |
russellb | progress!@ | 15:07 |
gsagie | regXboi: one thing that probably needs to be addressed later when separating the DB's are various consistency problems (in addition to the various problems we have now with Neutron-OVN NB DB) | 15:10 |
regXboi | gsagie: ack | 15:11 |
gsagie | or atleast i dont know how ovn-northd behaves in all these scenarios | 15:11 |
gsagie | its using monitor or its constantly polling NB? because with the monitor you might lose some events under load and never sync them | 15:12 |
* regXboi mutters, well this is ... annoying | 15:12 | |
gsagie | I have a nice solution for that in Dragonflow, we can iterate on it later on, i think at least the Neutron-OVN NB part should work, i dont remember, does OVSDB support compare and swap? | 15:14 |
russellb | gsagie: using monitor | 15:14 |
russellb | and it would sync things even if it missed events | 15:14 |
russellb | vim | 15:15 |
regXboi | emacs | 15:16 |
Mic22 | http://pastebin.com/3k8Dr5UF | 15:16 |
Mic22 | this should have the separation properly done | 15:16 |
regXboi | nit: should we be able to start/stop the NB and SB dbs independently? | 15:17 |
Mic22 | i thought about that but the question is why? | 15:18 |
regXboi | That's why I marked it as a nit | 15:18 |
Mic22 | when you have northd running you need both | 15:18 |
regXboi | but yeah, I'd say submit this to the ML | 15:18 |
Mic22 | probably better | 15:18 |
regXboi | russellb: ok, can you tell me where struct sbrec_logical_flow is defined? | 15:19 |
russellb | regXboi: what editor do you use? | 15:19 |
regXboi | russellb: I bounce around :) | 15:20 |
russellb | heh | 15:20 |
regXboi | I'm looking at some files in emacs and some in vim as we speak | 15:20 |
russellb | well as a side note, look up "ctags" and integration with your editor of choice | 15:20 |
russellb | it will help with things like this | 15:20 |
russellb | but | 15:20 |
russellb | ovn/lib/ovn-sb-idl.h | 15:20 |
russellb | IIRC | 15:20 |
russellb | and it's a trick, because that header is auto generated, not in git | 15:20 |
regXboi | yeah - I've been figuring that out | 15:21 |
regXboi | I looked through the *.in files | 15:21 |
russellb | it's automatically generated from ovn/ovn-sb.ovsschema | 15:21 |
regXboi | I'm trying to fix this incremental flow patch to be more correct | 15:21 |
regXboi | russellb: is it acceptable to inject a non-column variable into an autodefined structure? | 15:23 |
russellb | probably not | 15:24 |
russellb | well | 15:24 |
russellb | no | 15:24 |
russellb | :) | 15:24 |
Mic22 | russellb: practically i have to add signed-off-by and send an email in the dev list? | 15:24 |
regXboi | hmm, then that means I can't carry a "dirty" working variable around without changing the db schema and that strikes me as overkill | 15:24 |
russellb | Mic22: yes | 15:24 |
Mic22 | is there an automatic way or do I have to manually write this in the commit message? | 15:25 |
russellb | Mic22: git commit --amend -s | 15:26 |
russellb | Mic22: and make sure you look at what this means (that you agree with the Developer Certificate of Origin in CONTRIBUTING.md) | 15:27 |
russellb | if you haven't seen it before | 15:27 |
tongli | can someone take a look at this patch? https://review.openstack.org/#/c/276475/ it makes vagrant uses the configuration file more, and can configure the provider network any size | 15:29 |
tongli | or any address block. | 15:29 |
lrichard | "git format-patch" with --signoff works too | 15:30 |
Mic22 | http://pastebin.com/xMiqqGTm | 15:31 |
Mic22 | can you guys see if that's ok | 15:31 |
Mic22 | if it is I will send it | 15:31 |
openstackgerrit | litong01 proposed openstack/networking-ovn: Vagrant: Generify post-config options for DevStack https://review.openstack.org/276866 | 15:34 |
Mic22 | russelb: ping | 15:36 |
Mic22 | russellb: ping | 15:36 |
*** chandrav has quit IRC | 15:40 | |
Mic22 | sent | 15:41 |
russellb | thanks! i'll take a look and give it a try today | 15:41 |
Mic22 | Tomorrow I will start documenting HA | 15:42 |
Mic22 | just one question ATM I'm using haproxy to have a port on the loopback interface pointin to dbs | 15:42 |
regXboi | russellb: my turn to ping | 15:42 |
Mic22 | is there any other way (without a VIP) to pass more arguments for example? | 15:43 |
openstackgerrit | litong01 proposed openstack/networking-ovn: Vagrant: make provider network creation configurable https://review.openstack.org/276475 | 15:43 |
russellb | Mic22: i dont' see your patch. are you subscribed to the ovs dev mailing list? | 16:08 |
russellb | you probably have to subscribe before you can post the patch | 16:09 |
*** jckasper has quit IRC | 16:10 | |
*** jckasper has joined #openstack-neutron-ovn | 16:14 | |
regXboi | russellb: I see it | 16:14 |
regXboi | and am commenting on it... | 16:15 |
russellb | gmail marked it as spam | 16:15 |
regXboi | bad gmail | 16:15 |
russellb | ovs build broken, bah | 16:18 |
regXboi | russellb: what broke? | 16:19 |
russellb | might just be local issue | 16:19 |
*** armax has joined #openstack-neutron-ovn | 16:20 | |
russellb | patch posted to fix .. | 16:21 |
* Sam-I-Am gets on the ovs dev list | 16:31 | |
*** numans has joined #openstack-neutron-ovn | 16:32 | |
Sam-I-Am | moarr email | 16:35 |
*** allan_h has joined #openstack-neutron-ovn | 16:35 | |
*** salv-orl_ has joined #openstack-neutron-ovn | 16:40 | |
*** salv-orlando has quit IRC | 16:43 | |
Sam-I-Am | that moment you realize you have to rebase every part of a tiny patch | 16:45 |
*** armax has quit IRC | 16:46 | |
openstackgerrit | Matthew Kassawara proposed openstack/networking-ovn: Add support for net-mtu extension https://review.openstack.org/277490 | 16:47 |
regXboi | Mic22: ping - do you have patches for your HA work? | 16:55 |
*** mamulsow has joined #openstack-neutron-ovn | 17:00 | |
*** salv-orl_ has quit IRC | 17:05 | |
*** salv-orlando has joined #openstack-neutron-ovn | 17:05 | |
*** yamamoto has joined #openstack-neutron-ovn | 17:21 | |
*** yamamoto has quit IRC | 17:22 | |
Mic22 | regXboi: I will put tomorrow a documentation in place. It's not really about patching It's more about a slightly different installation | 17:42 |
Mic22 | But ATM I have a running HA proof of concept using ceph rbd for sharing the configuration and I will document step by step this way. Probably I will put the ceph part in a separate document | 17:43 |
*** yamamoto has joined #openstack-neutron-ovn | 17:43 | |
Mic22 | I was just thinking if make it with virtual IP or without. The advantage of having this without is to be able to use boxes on different L2 segments. | 17:45 |
Mic22 | But maybe it complexify without a really need | 17:46 |
regXboi | Mic22: ack | 17:46 |
*** roeyc has quit IRC | 17:49 | |
Mic22 | russellb: ovs build broken, bah => something to be done to fix? | 17:53 |
*** armax has joined #openstack-neutron-ovn | 17:56 | |
regXboi | Mic22: IIRC russellb has pushed a patch already to fix | 18:04 |
Mic22 | where I don't see it | 18:04 |
Mic22 | by the way I posted the new patch that has tovn-northd updated to read the new dbs | 18:05 |
regXboi | It hasn't made it to me | 18:05 |
regXboi | the patch discussion starts at http://openvswitch.org/pipermail/dev/2016-February/065853.html | 18:06 |
Mic22 | ok | 18:06 |
Mic22 | you are even in cc in the new patch i changed the topic because it was too long as russel noticed | 18:06 |
*** armax has quit IRC | 18:07 | |
*** armax has joined #openstack-neutron-ovn | 18:12 | |
regXboi | Mic22: yep, I can see it | 18:22 |
Mic22 | I will go home now It starts to be late here, have a good day! | 18:23 |
*** markmcclain has quit IRC | 18:24 | |
*** markmcclain has joined #openstack-neutron-ovn | 18:25 | |
*** yamamoto has quit IRC | 18:26 | |
*** armax has quit IRC | 18:31 | |
*** armax has joined #openstack-neutron-ovn | 18:44 | |
openstackgerrit | Richard Theis proposed openstack/networking-ovn: Document neutron API extensions supported https://review.openstack.org/277531 | 18:46 |
*** chandrav has joined #openstack-neutron-ovn | 18:55 | |
*** salv-orl_ has joined #openstack-neutron-ovn | 18:58 | |
*** salv-orlando has quit IRC | 19:01 | |
*** s3wong has joined #openstack-neutron-ovn | 19:13 | |
*** chandrav has quit IRC | 19:14 | |
openstackgerrit | Merged openstack/networking-ovn: Add support for net-mtu extension https://review.openstack.org/277490 | 19:25 |
*** armax has quit IRC | 19:26 | |
*** cz_ has joined #openstack-neutron-ovn | 19:32 | |
*** cz_ is now known as c_z | 19:33 | |
*** chandrav has joined #openstack-neutron-ovn | 19:34 | |
*** armax has joined #openstack-neutron-ovn | 19:34 | |
*** armax has quit IRC | 19:36 | |
openstackgerrit | Richard Theis proposed openstack/networking-ovn: Document neutron API extensions supported https://review.openstack.org/277531 | 19:40 |
*** salv-orl_ has quit IRC | 19:53 | |
*** tongli has quit IRC | 19:57 | |
*** numans has quit IRC | 20:00 | |
openstackgerrit | Merged openstack/networking-ovn: Document neutron API extensions supported https://review.openstack.org/277531 | 20:01 |
regXboi | russellb: do you understand the difference between <FOO>_FOR_EACH and <FOO>_FOR_EACH_TRACKED ? | 20:06 |
russellb | yes | 20:07 |
russellb | we don't use the _TRACKED functionality yet | 20:07 |
russellb | it lets you iterate over rows that have actually changed | 20:07 |
regXboi | I'm trying to rewrite the lflow processing in ovn-controller to use _TRACKED and I'm getting an odd result | 20:07 |
russellb | did you enable change tracking on the tables/columns of interest? | 20:08 |
regXboi | I believe I did, yes | 20:08 |
russellb | ok | 20:08 |
regXboi | I just did an _add_all for the purposes of this test | 20:08 |
regXboi | and that maybe where I'm getting nailed | 20:08 |
regXboi | because it looks like I'm processing one more row with _TRACKED than without and that row appears to have garbage in it | 20:09 |
*** chandrav has quit IRC | 20:12 | |
*** armax has joined #openstack-neutron-ovn | 20:13 | |
regXboi | yep, at initialization with _FOR_EACH, I'm looking at 12 flows and with _FOR_EACH_TRACKED (and add_all) I'm looking at 13 flows | 20:17 |
regXboi | so I've goofed something :) | 20:17 |
*** armax has quit IRC | 20:17 | |
*** armax has joined #openstack-neutron-ovn | 20:18 | |
*** salv-orlando has joined #openstack-neutron-ovn | 20:27 | |
*** armax has quit IRC | 20:30 | |
regXboi | russellb: ok, next question that I'm banging my head against... how do I dig a specific column out of an ovsdb_idl structure? or do I have to reparse the ovsdb schema and pull out the table and column via strings? | 20:32 |
regXboi | and if you'd rather I move to #openvswitch for these questions, I certainly can :) | 20:33 |
*** salv-orlando has quit IRC | 20:36 | |
russellb | sure let's use #openvswitch | 20:39 |
* Sam-I-Am wonders where his ovs channel went | 20:49 | |
*** c_z has quit IRC | 20:59 | |
*** armax has joined #openstack-neutron-ovn | 20:59 | |
*** salv-orlando has joined #openstack-neutron-ovn | 21:01 | |
*** chandrav has joined #openstack-neutron-ovn | 21:06 | |
*** gangil has joined #openstack-neutron-ovn | 21:41 | |
*** gangil has joined #openstack-neutron-ovn | 21:41 | |
*** regXboi has quit IRC | 22:01 | |
*** rtheis has quit IRC | 22:07 | |
*** jckasper has quit IRC | 22:53 | |
*** jckasper has joined #openstack-neutron-ovn | 22:54 | |
*** jckasper has quit IRC | 22:58 | |
*** allan_h has quit IRC | 22:59 | |
*** salv-orlando has quit IRC | 23:14 | |
*** salv-orlando has joined #openstack-neutron-ovn | 23:15 | |
*** jckasper has joined #openstack-neutron-ovn | 23:16 | |
openstackgerrit | Merged openstack/networking-ovn: Use compile_ovs() from Neutron tree https://review.openstack.org/267159 | 23:32 |
*** chandrav has quit IRC | 23:33 | |
Sam-I-Am | moo. | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!