*** armax has quit IRC | 00:19 | |
*** armax_ has joined #openstack-neutron-ovn | 00:19 | |
*** allan_h has joined #openstack-neutron-ovn | 00:55 | |
*** allan_h has quit IRC | 01:12 | |
russellb | quack | 01:56 |
---|---|---|
Sam-I-Am | moo | 01:58 |
Sam-I-Am | why am i still here | 01:58 |
russellb | because you love it | 01:58 |
Sam-I-Am | perhaps | 01:59 |
russellb | and it's cold outside | 01:59 |
*** yamamoto has joined #openstack-neutron-ovn | 01:59 | |
Sam-I-Am | it was nice here today. i got outside for a bit. | 01:59 |
Sam-I-Am | now i'm back to troubleshooting what appears to be a lab network problem that has set back most of this afternoons progress on anything | 01:59 |
Sam-I-Am | git is broken | 01:59 |
Sam-I-Am | and nothing uses that, luckily | 01:59 |
Sam-I-Am | changed git_base to use http and now tox blows up | 02:00 |
Sam-I-Am | was also looking at more mtu business in create_network | 02:01 |
russellb | yaks everywhere | 02:05 |
Sam-I-Am | yep nothing is working with vagrant anymore | 02:06 |
Sam-I-Am | so my lab is dead | 02:06 |
Sam-I-Am | on that note, it becomes tomorrow's problem | 02:07 |
russellb | ++ | 02:08 |
Sam-I-Am | have a good night. dont work too hard. | 02:09 |
russellb | we'll see about that | 02:11 |
russellb | i can't seem to feel caught up these days | 02:11 |
*** yamamoto has quit IRC | 02:24 | |
*** gangil has quit IRC | 02:29 | |
*** s3wong has quit IRC | 02:44 | |
*** armax_ is now known as armax | 02:54 | |
*** gangil has joined #openstack-neutron-ovn | 03:09 | |
*** gangil has joined #openstack-neutron-ovn | 03:09 | |
*** yamamoto has joined #openstack-neutron-ovn | 03:12 | |
openstackgerrit | litong01 proposed openstack/networking-ovn: Vagrant: make provider network creation configurable https://review.openstack.org/276475 | 04:16 |
*** armax_ has joined #openstack-neutron-ovn | 04:18 | |
*** armax has quit IRC | 04:21 | |
*** armax_ is now known as armax | 04:21 | |
*** brad_behle has quit IRC | 04:31 | |
*** salv-orl_ has joined #openstack-neutron-ovn | 04:41 | |
*** salv-orlando has quit IRC | 04:43 | |
*** allan_h has joined #openstack-neutron-ovn | 04:44 | |
*** allan_h has quit IRC | 05:28 | |
*** jckasper has quit IRC | 05:40 | |
*** chandrav has joined #openstack-neutron-ovn | 06:11 | |
*** chandrav has quit IRC | 06:12 | |
*** numans has joined #openstack-neutron-ovn | 06:19 | |
*** armax has quit IRC | 06:49 | |
*** jckasper has joined #openstack-neutron-ovn | 07:41 | |
*** jckasper has quit IRC | 07:45 | |
*** gangil has quit IRC | 08:14 | |
*** salv-orlando has joined #openstack-neutron-ovn | 10:40 | |
*** salv-orl_ has quit IRC | 10:43 | |
*** yamamoto has quit IRC | 11:05 | |
openstackgerrit | Numan Siddique proposed openstack/networking-ovn: Refactor ovn_nb_sync.OvnNbSynchronizer https://review.openstack.org/277792 | 11:29 |
openstackgerrit | Numan Siddique proposed openstack/networking-ovn: Refactor ovn_nb_sync.OvnNbSynchronizer https://review.openstack.org/277792 | 11:37 |
*** jckasper has joined #openstack-neutron-ovn | 11:43 | |
*** jckasper has quit IRC | 11:47 | |
*** yamamoto has joined #openstack-neutron-ovn | 11:51 | |
openstackgerrit | Numan Siddique proposed openstack/networking-ovn: Neutron ovn northbound db sync tool https://review.openstack.org/277805 | 12:03 |
openstackgerrit | Numan Siddique proposed openstack/networking-ovn: Neutron ovn northbound db sync tool https://review.openstack.org/277805 | 12:06 |
*** chandrav has joined #openstack-neutron-ovn | 12:09 | |
*** rtheis has joined #openstack-neutron-ovn | 12:23 | |
*** openstackgerrit_ has joined #openstack-neutron-ovn | 12:39 | |
*** yamamoto has quit IRC | 12:46 | |
*** numans has quit IRC | 13:02 | |
*** yamamoto has joined #openstack-neutron-ovn | 13:03 | |
*** yamamoto has quit IRC | 13:16 | |
*** brad_behle has joined #openstack-neutron-ovn | 13:22 | |
*** yamamoto has joined #openstack-neutron-ovn | 13:23 | |
russellb | rtheis: you around? there seems to be a failing unit test related to AZs | 13:28 |
rtheis | russellb: yes I am | 13:29 |
russellb | i first saw the failure here https://review.openstack.org/#/c/277792/2 | 13:29 |
*** numans has joined #openstack-neutron-ovn | 13:29 | |
russellb | but i can reproduce locally without any patches applied | 13:29 |
rtheis | russellb: does it fail consistently? | 13:30 |
* russellb checks | 13:30 | |
russellb | yes, fails every time for me | 13:31 |
*** regXboi has joined #openstack-neutron-ovn | 13:31 | |
russellb | i had to get a fresh tox env to reproduce it though (tox -r -e py27, or just rm -rf .tox) | 13:31 |
russellb | which makes it sound related to a fairly recent change in neutron | 13:32 |
rtheis | russellb: yeah, I'll take a look | 13:32 |
russellb | ok thank you! | 13:32 |
*** numans has quit IRC | 13:34 | |
regXboi | russellb: morning - I *think* I've figured out what's going on with column tracking, but if I'm correct, it looks like there is an off by one error in the code that loops through tracked rows | 13:38 |
russellb | nice | 13:39 |
russellb | it could be, i'm not sure that code is used anywhere yet | 13:39 |
regXboi | I've dropped in instrumentation to check how much it improves | 13:39 |
russellb | though there is test coverage | 13:39 |
russellb | see tests/test-ovsdb.c and search for TRACKED | 13:40 |
regXboi | yes, I did | 13:40 |
russellb | ok | 13:40 |
regXboi | but here's why I say there is an off-by-one | 13:40 |
regXboi | I dropped in instrumentation in lflow.c to check the number of flows looped through | 13:40 |
regXboi | and after an initial stack, that instrumentation says 12 | 13:40 |
regXboi | but, when I put in the code to loop through tracked rows, that instrumentation says 13 *AND* I see occasional WARN messages about not being able to process filters | 13:41 |
regXboi | which leads me to think there is an off-by-one | 13:41 |
regXboi | I'm trying to finish getting the seqno checking right and then I'll chase this off-by-one some more | 13:42 |
*** jckasper has joined #openstack-neutron-ovn | 13:42 | |
*** jckasper_ has joined #openstack-neutron-ovn | 13:43 | |
rtheis | russellb: I think the failure is caused by https://github.com/openstack/neutron/commit/62c0c0771fb1e46b275ab9694ae94cbada6b922c | 13:44 |
*** jckasper has quit IRC | 13:47 | |
Sam-I-Am | moo | 13:47 |
rtheis | anyone seeing "/opt/stack/networking-ovn/devstack/plugin.sh: line 86: /opt/stack/neutron/devstack/lib/ovs: No such file or directory" when they try to stack.sh with OVN? | 13:48 |
regXboi | rtheis: no, but I don't think I'm quite tip of the tree anymore | 13:48 |
russellb | rtheis: you need to update neutron for that error | 13:48 |
russellb | rtheis: we're now using some devstack code in the neutron repo | 13:49 |
russellb | rtheis: and "the test just got added" would explain it :) | 13:49 |
rtheis | russellb: this is a fresh stack...no neutron yet | 13:49 |
russellb | oh. | 13:50 |
russellb | well... we may have broken things then | 13:50 |
rtheis | Looks like I don't even have /opt/stack/neutron | 13:50 |
russellb | EVERYTHING IS BROKEN TODAY | 13:50 |
rtheis | ouch | 13:51 |
regXboi | moar yelling? | 13:51 |
Sam-I-Am | russellb: yeah i saw that last night | 13:51 |
russellb | :) | 13:51 |
russellb | rtheis: we must assume neutron is already there, which is true for the gate env | 13:51 |
* regXboi is guessing he won't update code basis today | 13:51 | |
Sam-I-Am | what changed that would break this? | 13:51 |
regXboi | Sam-I-Am: did you just ask what changed to break devstack? | 13:52 |
russellb | http://git.openstack.org/cgit/openstack/networking-ovn/commit/?id=d36d8e5078f7eafe375a87ec2074ccfdc187169b | 13:52 |
russellb | that one. | 13:52 |
russellb | easy fix, one sec ... | 13:53 |
Sam-I-Am | ohhhh yeah | 13:53 |
Sam-I-Am | that would do it | 13:53 |
* Sam-I-Am looks at blame | 13:54 | |
*** openstackgerrit_ has quit IRC | 13:56 | |
Sam-I-Am | are we just debugging why this patch doesnt work, or reverting it? | 14:01 |
russellb | i know the problem | 14:02 |
russellb | i'll post a patch in a sec | 14:02 |
russellb | trying to work out how to get it to merge | 14:02 |
russellb | since there's also a new unit test added in neutron that's failing | 14:02 |
Sam-I-Am | there inlies the fun problem | 14:02 |
Sam-I-Am | yeah i saw that too | 14:02 |
Sam-I-Am | this sounds like a good reason to go back to bed | 14:02 |
openstackgerrit | Richard Theis proposed openstack/networking-ovn: WIP: Add extension fields on network create https://review.openstack.org/277844 | 14:04 |
openstackgerrit | Russell Bryant proposed openstack/networking-ovn: Disable failing new unit test. https://review.openstack.org/277845 | 14:05 |
openstackgerrit | Russell Bryant proposed openstack/networking-ovn: devstack: Source ovs devstack lib later. https://review.openstack.org/277846 | 14:05 |
*** numans has joined #openstack-neutron-ovn | 14:12 | |
regXboi | russellb: it's not an off-by-one - it's a problem with processing a delete twice | 14:18 |
*** numans has quit IRC | 14:20 | |
*** numans has joined #openstack-neutron-ovn | 14:22 | |
regXboi | or maybe I should say, not releasing a row once it is deleted | 14:22 |
regXboi | well... now *that's* annoying | 14:29 |
regXboi | the logical datapath check is getting in my way | 14:30 |
*** yamamoto has quit IRC | 14:32 | |
*** yamamoto has joined #openstack-neutron-ovn | 14:33 | |
*** yamamoto has quit IRC | 14:37 | |
*** tongli has joined #openstack-neutron-ovn | 14:40 | |
regXboi | russellb: what would you say if I said I wanted to take out the logical datapath check in lflow_run in favor of seqno tracking? | 14:56 |
openstackgerrit | Richard Theis proposed openstack/networking-ovn: WIP: Add extension fields on network and port create https://review.openstack.org/277844 | 15:03 |
*** yamamoto has joined #openstack-neutron-ovn | 15:07 | |
*** shettyg has joined #openstack-neutron-ovn | 15:11 | |
*** palexster has quit IRC | 15:12 | |
*** palexster has joined #openstack-neutron-ovn | 15:14 | |
Sam-I-Am | russellb: huh @ fail on 277845 - secgroup ext not enabled? | 15:17 |
Sam-I-Am | er, wait... a bit further down | 15:17 |
russellb | some ipv6 test fail, no idea | 15:18 |
*** chandrav has quit IRC | 15:18 | |
Sam-I-Am | might be worth a recheck | 15:19 |
russellb | we've got a db issue in the q-svc log that we still need to fix | 15:19 |
Sam-I-Am | linky? | 15:19 |
russellb | http://logs.openstack.org/45/277845/1/check/gate-tempest-dsvm-networking-ovn/4e5ba49/logs/screen-q-svc.txt.gz?level=TRACE | 15:20 |
*** numans has quit IRC | 15:20 | |
Sam-I-Am | ew | 15:21 |
Sam-I-Am | wonder whats up with that | 15:23 |
*** numans has joined #openstack-neutron-ovn | 15:23 | |
*** palexster has quit IRC | 15:24 | |
regXboi | argh! | 15:31 |
regXboi | the tracking code gets rows LIFO, which means the sequence numbers aren't monotonically increasing | 15:32 |
*** palexster has joined #openstack-neutron-ovn | 15:39 | |
regXboi | so that snippet of code gets more complex now | 15:41 |
*** numans has quit IRC | 15:43 | |
*** yamamoto has quit IRC | 15:48 | |
Sam-I-Am | russellb: 277846 passed but has a variety of interesting tbacks | 15:57 |
* russellb nods | 15:57 | |
russellb | the db tracebacks are fairly consistently there i think | 15:57 |
Sam-I-Am | the equivalent neutron/ml2 jobs dont seem to have them | 15:58 |
* russellb nods | 15:59 | |
*** armax has joined #openstack-neutron-ovn | 16:08 | |
*** palexster has quit IRC | 16:34 | |
*** salv-orl_ has joined #openstack-neutron-ovn | 16:41 | |
regXboi | ha ha... ok, this works | 16:44 |
*** salv-orlando has quit IRC | 16:44 | |
regXboi | now to run a test to see how well it does and then update the patch set | 16:44 |
* regXboi dances a slow jig | 16:44 | |
*** c_z has joined #openstack-neutron-ovn | 17:12 | |
*** allan_h has joined #openstack-neutron-ovn | 17:20 | |
*** c_z has quit IRC | 17:23 | |
-openstackstatus- NOTICE: Gerrit is restarting now, to alleviate current performance impact and WebUI errors. | 17:24 | |
*** c_z has joined #openstack-neutron-ovn | 17:26 | |
*** palexster has joined #openstack-neutron-ovn | 17:31 | |
rtheis | Testing out provider networks by following http://docs.openstack.org/developer/networking-ovn/testing.html ... however I didn't "observe that ovn-controller automatically created patch ports between br-int and br-provider". Is the documentation still accurate? | 17:32 |
russellb | rtheis: it is not accurate, thanks | 17:35 |
Sam-I-Am | isnt that the ovs-vsctl step? | 17:36 |
openstackgerrit | Russell Bryant proposed openstack/networking-ovn: testing: Remove comment about looking at patch ports. https://review.openstack.org/277985 | 17:37 |
russellb | rtheis: ^ | 17:37 |
russellb | Sam-I-Am: the patch ports now aren't created until later, and it's more of an implementation detail anyway.. | 17:37 |
*** c_z has quit IRC | 17:40 | |
*** palexster has quit IRC | 17:46 | |
*** palexster has joined #openstack-neutron-ovn | 18:04 | |
russellb | rtheis: patch ports will appear once you start creating neutron ports, if you want to look | 18:07 |
russellb | but it doesn't really matter | 18:07 |
rtheis | russellb: thanks, I'll take a look | 18:07 |
russellb | well, it matters a lot, but i'm not sure it's important to look at | 18:07 |
russellb | depends how much you care about diving into how OVN is setting up what you asked for | 18:08 |
*** palexster has quit IRC | 18:14 | |
rtheis | indeed they appear after creating a subnet and port for the provider network. | 18:16 |
*** gangil has joined #openstack-neutron-ovn | 18:20 | |
*** gangil has joined #openstack-neutron-ovn | 18:20 | |
openstackgerrit | Richard Theis proposed openstack/networking-ovn: Add extension fields on network and port create https://review.openstack.org/277844 | 18:20 |
openstackgerrit | Russell Bryant proposed openstack/networking-ovn: devstack: Source ovs devstack lib later. https://review.openstack.org/277846 | 18:24 |
*** palexster has joined #openstack-neutron-ovn | 18:27 | |
*** aginwala has joined #openstack-neutron-ovn | 18:32 | |
*** aginwala has quit IRC | 18:45 | |
*** c_z has joined #openstack-neutron-ovn | 18:46 | |
*** arosen12 has joined #openstack-neutron-ovn | 18:48 | |
*** aginwala has joined #openstack-neutron-ovn | 18:49 | |
*** gangil has quit IRC | 19:02 | |
*** palexster has quit IRC | 19:03 | |
*** gangil has joined #openstack-neutron-ovn | 19:11 | |
*** gangil has joined #openstack-neutron-ovn | 19:11 | |
*** jckasper_ has quit IRC | 19:13 | |
*** jckasper has joined #openstack-neutron-ovn | 19:14 | |
*** jckasper has quit IRC | 19:14 | |
*** jckasper has joined #openstack-neutron-ovn | 19:14 | |
*** Mic22 has quit IRC | 19:23 | |
*** c_z has quit IRC | 19:32 | |
*** aginwala has quit IRC | 19:49 | |
*** aginwala has joined #openstack-neutron-ovn | 19:55 | |
*** jckasper has quit IRC | 20:19 | |
*** jckasper has joined #openstack-neutron-ovn | 20:19 | |
*** jckasper has quit IRC | 20:23 | |
*** salv-orl_ has quit IRC | 20:36 | |
*** aginwala has quit IRC | 20:40 | |
*** aginwala has joined #openstack-neutron-ovn | 20:43 | |
*** jckasper has joined #openstack-neutron-ovn | 20:44 | |
*** salv-orlando has joined #openstack-neutron-ovn | 20:53 | |
*** aginwala has quit IRC | 20:57 | |
tongli | @Sam-I-Am, matt, is the gate back up now? | 21:02 |
*** jckasper has quit IRC | 21:06 | |
*** jckasper has joined #openstack-neutron-ovn | 21:06 | |
Sam-I-Am | tongli: no | 21:07 |
*** jckasper has quit IRC | 21:08 | |
*** jckasper has joined #openstack-neutron-ovn | 21:09 | |
mestery | salv-orlando tongli: I believe based on what rtheis said we need https://review.openstack.org/#/c/277844/ and https://review.openstack.org/#/c/277846/ to merge to fix the gate | 21:13 |
mestery | Not salv-orlando but salv-orlando | 21:13 |
mestery | Not salv-orlando or salv-orlando but Sam-I-Am | 21:14 |
russellb | armax: lol | 21:14 |
mestery | Stupid autocorrect | 21:14 |
mestery | rofl | 21:14 |
mestery | russellb: armax? :) | 21:14 |
russellb | patch to fix is in the gate | 21:14 |
mestery | sweetness! | 21:14 |
Sam-I-Am | yeah | 21:14 |
Sam-I-Am | it'll be fixed soonish i hope | 21:14 |
Sam-I-Am | in the meantime plenty of other stuff that i've neglected | 21:14 |
armax | what did I do? | 21:14 |
russellb | 277844 indeed | 21:14 |
russellb | armax: mestery may have summoned you by accident. | 21:14 |
tongli | @Sam-I-Am, thanks. | 21:15 |
*** Mic22 has joined #openstack-neutron-ovn | 21:15 | |
russellb | 277844 is running gate jobs, should be finished in < 30 mins | 21:15 |
armax | ok, I need to change nick | 21:15 |
armax | this world is too small for me | 21:15 |
mestery | armax: There's so much love for you in all the channels :) | 21:16 |
armax | I kno | 21:16 |
armax | despite the many attempts to get people to hate me | 21:16 |
mestery | armax: There's just too much of you to love ;) | 21:16 |
armax | mestery: ditto | 21:17 |
mestery | :) | 21:17 |
Sam-I-Am | tongli: your recheck wont work | 21:17 |
Sam-I-Am | tongli: not until those patches merge and you rebase | 21:17 |
tongli | @Sam-I-Am, yeah, I thought the gate is up so I did that before I asked. I probably should have waited a bit longer. | 21:18 |
tongli | @Sam-I-Am, sorry about that. | 21:18 |
Sam-I-Am | no prob | 21:18 |
openstackgerrit | Merged openstack/networking-ovn: Add extension fields on network and port create https://review.openstack.org/277844 | 21:19 |
mestery | russellb: I'd like to create a job for kuryr+ovn, even if it's just an install job, to ensure things work there. Thoughts? | 21:21 |
russellb | mestery: sounds great. | 21:21 |
russellb | and i'm fine with an install job to start with | 21:21 |
mestery | russellb: Thinking of putting it into the networking-ovn job list | 21:21 |
russellb | nothing < install job < actually using the thing | 21:21 |
russellb | fine with me | 21:21 |
Sam-I-Am | would it use conventional l3? | 21:21 |
Sam-I-Am | i also have ideas for a native l3 job | 21:22 |
mestery | coolio | 21:22 |
mestery | Sam-I-Am: For now, it would just install :) | 21:22 |
Sam-I-Am | i was trying to use the kuryr vagrant yesterday and its broken | 21:23 |
Sam-I-Am | at least with vbox | 21:24 |
Sam-I-Am | then i lost git access from the lab | 21:24 |
mestery | WFM? :) | 21:25 |
Sam-I-Am | yeaaaaaah | 21:25 |
openstackgerrit | Merged openstack/networking-ovn: devstack: Source ovs devstack lib later. https://review.openstack.org/277846 | 21:37 |
mestery | russellb Sam-I-Am: https://review.openstack.org/278101 | 21:39 |
Sam-I-Am | i thought i was working on that | 21:41 |
mestery | Sam-I-Am: You were, but I got antsy and had to do something while sitting in all these meetings :) | 21:42 |
Sam-I-Am | hmm ok | 21:42 |
Sam-I-Am | also was thinking it'd go in ovn | 21:42 |
mestery | Wait, isn't it in ovn? | 21:42 |
* regXboi goes to look at the sun | 21:42 | |
Sam-I-Am | errr... non-voting/experimental | 21:43 |
Sam-I-Am | one step down :) | 21:43 |
*** regXboi has quit IRC | 21:43 | |
mestery | Yes | 21:44 |
mestery | Sam-I-Am: This is what happens when I roll patches while in meetings :) | 21:44 |
* Sam-I-Am backs away slowly | 21:45 | |
mestery | rofl | 21:45 |
*** azbiswas has joined #openstack-neutron-ovn | 21:48 | |
*** aginwala has joined #openstack-neutron-ovn | 22:33 | |
*** shettyg has quit IRC | 22:38 | |
*** salv-orl_ has joined #openstack-neutron-ovn | 22:40 | |
*** salv-orlando has quit IRC | 22:43 | |
*** rtheis has quit IRC | 22:46 | |
*** tongli has quit IRC | 22:56 | |
*** jckasper has quit IRC | 23:04 | |
*** jckasper has joined #openstack-neutron-ovn | 23:05 | |
*** aginwala has quit IRC | 23:14 | |
*** aginwala has joined #openstack-neutron-ovn | 23:16 | |
*** jckasper has quit IRC | 23:17 | |
*** jckasper has joined #openstack-neutron-ovn | 23:17 | |
*** jckasper has quit IRC | 23:18 | |
*** jckasper has joined #openstack-neutron-ovn | 23:18 | |
*** yamamoto_ has joined #openstack-neutron-ovn | 23:21 | |
*** yamamoto_ has quit IRC | 23:23 | |
*** gangil1 has joined #openstack-neutron-ovn | 23:33 | |
*** gangil has quit IRC | 23:33 | |
*** allan_h has quit IRC | 23:39 | |
*** azbiswas has quit IRC | 23:44 | |
*** yamamoto_ has joined #openstack-neutron-ovn | 23:50 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!