| *** harlowja has quit IRC | 00:16 | |
| idella4 | ok markmcclain I re-ran with; testr init \ testr run --parallel --concurrency=3 | 00:19 |
|---|---|---|
| idella4 | got; FAIL: process-returncode \ tags: worker-1; Binary content: traceback (test/plain; charset="utf8") | 00:20 |
| markmcclain | hmmm | 00:20 |
| *** dkliban has joined #openstack-neutron | 00:21 | |
| idella4 | + : FAIL: process-returncode \ tags: worker-1; \ ... \ FRan 18183 tests in 1827.552s \ AILED (id=0, failures=2, skips=328) | 00:21 |
| markmcclain | no traceback? | 00:21 |
| idella4 | exactly | 00:21 |
| idella4 | 30 min. is a good run time | 00:22 |
| idella4 | no traceback means I am totally blinded | 00:22 |
| idella4 | a run without --parallel takes around 100 min. | 00:23 |
| idella4 | re-running now | 00:23 |
| idella4 | this is why I can't get a 'move-on' with this and I am at day 6 | 00:23 |
| idella4 | though some of my re-runs I aborted of my own invocation | 00:24 |
| idella4 | the traceback comes from the tests' code, being on a gentoo system afaik makes sodall difference | 00:25 |
| lifeless | idella4: if there is no traceback, it probably means a test is exiting the process | 00:26 |
| idella4 | It's kind of the first time I've encountered such a phenomenon, and gee, it's kind ofthe first time I've called testr as a test runner | 00:26 |
| lifeless | idella4: by calling sys.exit() | 00:26 |
| idella4 | right | 00:26 |
| *** gdubreui has joined #openstack-neutron | 00:27 | |
| idella4 | still, no traceback | 00:27 |
| idella4 | nosetests routinely provides a coverage output which captures any such shenanigans | 00:27 |
| *** dkliban has quit IRC | 00:27 | |
| idella4 | re-running now, eta ~ 100 minutes | 00:28 |
| *** matsuhashi has joined #openstack-neutron | 00:29 | |
| idella4 | I aborted the use of --parallel 3 days ago, these 2 such errors I decided came from its invocation | 00:29 |
| idella4 | that though amounts to a best guess on the info, may not be quite right | 00:30 |
| *** coolsvap has quit IRC | 00:30 | |
| idella4 | I mean have you turned logging off or something? | 00:32 |
| *** devlaps has quit IRC | 00:36 | |
| *** carl_baldwin has quit IRC | 00:38 | |
| *** networkstatic has joined #openstack-neutron | 00:45 | |
| *** matsuhashi has quit IRC | 00:48 | |
| *** matsuhashi has joined #openstack-neutron | 00:49 | |
| *** obondarev has quit IRC | 00:52 | |
| *** matsuhashi has quit IRC | 00:53 | |
| *** enikanorov__ has quit IRC | 00:54 | |
| *** alexpilotti has quit IRC | 00:59 | |
| *** reed has quit IRC | 01:02 | |
| *** matsuhashi has joined #openstack-neutron | 01:08 | |
| *** z0rkito has left #openstack-neutron | 01:12 | |
| *** wenjianhn has joined #openstack-neutron | 01:18 | |
| *** matsuhashi has quit IRC | 01:34 | |
| *** matsuhashi has joined #openstack-neutron | 01:35 | |
| *** matsuhashi has quit IRC | 01:40 | |
| *** matsuhashi has joined #openstack-neutron | 01:43 | |
| *** harlowja has joined #openstack-neutron | 01:51 | |
| *** nati_ueno has joined #openstack-neutron | 01:59 | |
| *** alagalah has quit IRC | 02:05 | |
| *** alexpilotti has joined #openstack-neutron | 02:05 | |
| *** changbl has joined #openstack-neutron | 02:09 | |
| *** wenjianhn has quit IRC | 02:10 | |
| *** alagalah has joined #openstack-neutron | 02:10 | |
| *** suresh12 has quit IRC | 02:17 | |
| *** suresh12 has joined #openstack-neutron | 02:17 | |
| *** jsergent has quit IRC | 02:18 | |
| *** changbl has quit IRC | 02:19 | |
| *** suresh12 has quit IRC | 02:19 | |
| *** suresh12 has joined #openstack-neutron | 02:19 | |
| *** unicell has quit IRC | 02:24 | |
| *** unicell has joined #openstack-neutron | 02:24 | |
| *** wenjianhn has joined #openstack-neutron | 02:25 | |
| *** akamyshnikova has quit IRC | 02:26 | |
| *** ogelbukh has quit IRC | 02:26 | |
| *** yamahata_ has joined #openstack-neutron | 02:28 | |
| *** yamahata_ has quit IRC | 02:33 | |
| *** wenjianhn has quit IRC | 02:39 | |
| *** wenjianhn has joined #openstack-neutron | 02:42 | |
| *** steven-weston has quit IRC | 02:45 | |
| *** yamahata_ has joined #openstack-neutron | 02:50 | |
| dkehn | idella4: is this a test for a single situation (i.e. your testing for something you are patching) or does it work under focused testing and your are trying to run it all? | 02:55 |
| idella4 | let's see; | 02:56 |
| idella4 | I am doing no patching | 02:56 |
| idella4 | I'm running the testsuite as is, the only patch I have patching is one committed fromcollective 'you' | 02:57 |
| dkehn | idella4: so your just pulling from trunk and trying to run something like tox -epy27 | 02:57 |
| idella4 | forget tox | 02:57 |
| idella4 | I'm running tesr directly | 02:57 |
| dkehn | idella4: ok, may I ask why | 02:57 |
| idella4 | take tox OUT of the equation, well it's all in the backlog, but sure | 02:58 |
| dkehn | idella4: I can check the log | 02:58 |
| idella4 | in summary, it confounds gentoo's approach to the core | 02:58 |
| idella4 | yep | 02:58 |
| idella4 | I was outlining it this time yesterday with markmcclain | 02:58 |
| idella4 | this is gentoo | 02:58 |
| idella4 | the user of tesr directly works fine | 02:59 |
| *** steven-weston has joined #openstack-neutron | 02:59 | |
| *** changbl has joined #openstack-neutron | 03:02 | |
| idella4 | I am currently suspecting that gentoo's eclass' multiprocessing component is getting some tests to be run out of order | 03:04 |
| idella4 | dealing with a suite the size of neutron's I don't think I've done before | 03:04 |
| dkehn | I'm probably not going to be of much help here, I typically use tox, becuase that is what is used in the gate and I want to simular that as much as possible to reduce other situations, also | 03:07 |
| idella4 | well that places gentoo at odds with neutron and or vica versa | 03:08 |
| dkehn | whne I've got a pathc that is trublesome I'll goto debug mode on a test or a suite to see whats going on | 03:08 |
| dkehn | using the python -m unittest <class> | 03:08 |
| dkehn | with pdb.set-trace() in it to allow me to stop and look around | 03:08 |
| idella4 | eeer, I follow in principle but not being versed not sure how to do | 03:10 |
| idella4 | is <class> a test_.py file? | 03:10 |
| idella4 | but ok thx | 03:11 |
| *** carl_baldwin has joined #openstack-neutron | 03:12 | |
| dkehn | hmm, so lets say put a import pdb; pdb.set_trace() back in https://github.com/CiscoSystems/neutron/blob/odl_ml2/neutron/tests/unit/test_extension_extradhcpopts.py#L140 | 03:12 |
| dkehn | and then within the neutron directory excute the ~/.tox/py27/bin/activate to get int the virt env | 03:12 |
| dkehn | then run something like: | 03:12 |
| dkehn | python -m unittest neutron.tests.unit.test_extension_extradhcpopt.TestExtraDhcpOpt | 03:13 |
| idella4 | ok I follow | 03:14 |
| dkehn | will run just those test (will do all the setup and tear downs) and break on the set_trace() line | 03:14 |
| *** nati_ueno has quit IRC | 03:14 | |
| idella4 | thanks dkehn | 03:14 |
| dkehn | you can pull that class back a bit if you've got other things that aren't cleanning up | 03:14 |
| dkehn | this way you can look around or whatever. | 03:15 |
| *** gongysh has joined #openstack-neutron | 03:15 | |
| dkehn | hope that helps, I'm off to zzzz land | 03:15 |
| idella4 | right thanks | 03:15 |
| *** nati_ueno has joined #openstack-neutron | 03:16 | |
| *** networkstatic has quit IRC | 03:17 | |
| *** nati_ueno has quit IRC | 03:20 | |
| *** carl_baldwin has left #openstack-neutron | 03:20 | |
| *** suresh12 has quit IRC | 03:30 | |
| *** matsuhashi has quit IRC | 03:31 | |
| *** suresh12 has joined #openstack-neutron | 03:31 | |
| *** matsuhashi has joined #openstack-neutron | 03:31 | |
| *** alagalah_ has joined #openstack-neutron | 03:34 | |
| *** suresh12 has quit IRC | 03:36 | |
| *** matsuhashi has quit IRC | 03:36 | |
| *** matsuhashi has joined #openstack-neutron | 03:41 | |
| prometheanfire | idella4: oh, here you are | 03:57 |
| *** yamahata_ has quit IRC | 03:58 | |
| *** gongysh has quit IRC | 04:03 | |
| *** ljjjusti1 has quit IRC | 04:05 | |
| *** akamyshnikova has joined #openstack-neutron | 04:05 | |
| *** AndreyGrebenniko has quit IRC | 04:07 | |
| *** networkstatic has joined #openstack-neutron | 04:14 | |
| *** coolsvap has joined #openstack-neutron | 04:16 | |
| *** ogelbukh has joined #openstack-neutron | 04:24 | |
| *** AndreyGrebenniko has joined #openstack-neutron | 04:25 | |
| *** alagalah_ has quit IRC | 04:42 | |
| *** suresh12 has joined #openstack-neutron | 04:42 | |
| *** alagalah_ has joined #openstack-neutron | 04:42 | |
| *** suresh12 has quit IRC | 04:47 | |
| *** yamahata_ has joined #openstack-neutron | 04:54 | |
| *** nati_ueno has joined #openstack-neutron | 05:02 | |
| *** alagalah_ has quit IRC | 05:04 | |
| *** alagalah_ has joined #openstack-neutron | 05:04 | |
| *** yamahata_ has quit IRC | 05:05 | |
| *** jecarey has quit IRC | 05:06 | |
| *** alagalah has quit IRC | 05:12 | |
| *** rudrarugge has joined #openstack-neutron | 05:13 | |
| *** yamahata_ has joined #openstack-neutron | 05:15 | |
| *** rudrarugge has quit IRC | 05:15 | |
| *** chandankumar has joined #openstack-neutron | 05:22 | |
| *** steven-weston has quit IRC | 05:23 | |
| *** yamahata_ has quit IRC | 05:33 | |
| *** coolsvap has quit IRC | 05:39 | |
| *** ljjjustin has joined #openstack-neutron | 05:46 | |
| *** harlowja has quit IRC | 05:52 | |
| *** marun has joined #openstack-neutron | 05:55 | |
| *** marun has quit IRC | 05:55 | |
| *** mihgen has joined #openstack-neutron | 05:56 | |
| *** coolsvap has joined #openstack-neutron | 05:58 | |
| *** suresh12 has joined #openstack-neutron | 06:01 | |
| *** nati_ueno has quit IRC | 06:03 | |
| *** nati_ueno has joined #openstack-neutron | 06:04 | |
| *** matsuhashi has quit IRC | 06:08 | |
| *** matsuhashi has joined #openstack-neutron | 06:08 | |
| *** matsuhashi has quit IRC | 06:13 | |
| *** mestery has quit IRC | 06:14 | |
| *** yfried has joined #openstack-neutron | 06:17 | |
| *** mestery has joined #openstack-neutron | 06:18 | |
| *** ljjjustin has quit IRC | 06:21 | |
| *** abhraut has joined #openstack-neutron | 06:24 | |
| *** matsuhashi has joined #openstack-neutron | 06:29 | |
| *** marun has joined #openstack-neutron | 06:38 | |
| openstackgerrit | Jenkins proposed a change to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/56864 | 06:41 |
| *** obondarev has joined #openstack-neutron | 06:42 | |
| *** amritanshu_RnD has joined #openstack-neutron | 06:46 | |
| openstackgerrit | berlin proposed a change to openstack/neutron: Support advanced NVP IPsec VPN Service https://review.openstack.org/55207 | 07:02 |
| *** gdubreui has quit IRC | 07:02 | |
| openstackgerrit | Zhang Hua proposed a change to openstack/neutron: Fwaas can't run in operating system without namespace feature https://review.openstack.org/56893 | 07:09 |
| *** ljjjustin has joined #openstack-neutron | 07:11 | |
| *** amuller has joined #openstack-neutron | 07:11 | |
| *** marun has quit IRC | 07:13 | |
| *** marun has joined #openstack-neutron | 07:14 | |
| openstackgerrit | Jian Wen proposed a change to openstack/neutron: Use correct parameter to call neutron client https://review.openstack.org/57383 | 07:20 |
| *** matsuhashi has quit IRC | 07:22 | |
| *** matsuhashi has joined #openstack-neutron | 07:23 | |
| *** afazekas_ has joined #openstack-neutron | 07:29 | |
| *** matsuhas_ has joined #openstack-neutron | 07:30 | |
| openstackgerrit | Assaf Muller proposed a change to openstack/python-neutronclient: [fwaas] Can now create disabled firewall rules https://review.openstack.org/55826 | 07:31 |
| yfried | marun: ping | 07:32 |
| *** matsuhashi has quit IRC | 07:34 | |
| *** suresh12 has quit IRC | 07:54 | |
| *** suresh12 has joined #openstack-neutron | 07:55 | |
| marun | yfried: pong | 07:56 |
| yfried | marun: in your review https://review.openstack.org/#/c/53305/17/tempest/scenario/manager.py you're commenting about sphinx convention. "There should be examples in the tree to crib from" | 07:58 |
| yfried | marun: what do you mean? | 07:58 |
| marun | yfried: I mean that there should be examples of sphinx documentation in the tempest tree | 07:59 |
| marun | yfried: btw, I realize we should probably be talking about tempest in #openstack-qa | 07:59 |
| marun | yfried: my bad | 07:59 |
| *** suresh12 has quit IRC | 08:00 | |
| *** jlibosva has joined #openstack-neutron | 08:02 | |
| *** mihgen has quit IRC | 08:05 | |
| *** marun has quit IRC | 08:05 | |
| *** marun has joined #openstack-neutron | 08:05 | |
| *** matsuhas_ has quit IRC | 08:18 | |
| *** matsuhashi has joined #openstack-neutron | 08:19 | |
| *** rossella_s has joined #openstack-neutron | 08:19 | |
| *** matsuhashi has quit IRC | 08:24 | |
| *** idella4 has quit IRC | 08:27 | |
| *** bvandenh has joined #openstack-neutron | 08:29 | |
| openstackgerrit | A change was merged to openstack/neutron: Reduce the severity of dhcp related log traces https://review.openstack.org/56785 | 08:29 |
| *** matsuhashi has joined #openstack-neutron | 08:31 | |
| *** mihgen has joined #openstack-neutron | 08:41 | |
| *** ygbo has joined #openstack-neutron | 08:41 | |
| *** ljjjustin has quit IRC | 08:52 | |
| *** nati_ueno has quit IRC | 08:59 | |
| *** steven-weston_ has joined #openstack-neutron | 08:59 | |
| openstackgerrit | Evgeny Fedoruk proposed a change to openstack/neutron: Adds tests, fixes Radware LBaaS driver as a result https://review.openstack.org/54155 | 09:04 |
| *** suresh12 has joined #openstack-neutron | 09:06 | |
| *** jpich has joined #openstack-neutron | 09:06 | |
| *** suresh12 has quit IRC | 09:11 | |
| *** marun has quit IRC | 09:13 | |
| *** marun has joined #openstack-neutron | 09:14 | |
| *** steven-weston__ has joined #openstack-neutron | 09:21 | |
| *** steven-weston_ has quit IRC | 09:22 | |
| *** steven-weston__ is now known as steven-weston | 09:22 | |
| *** alexpilotti has quit IRC | 09:23 | |
| *** afazekas_ is now known as afazekas | 09:34 | |
| *** safchain has joined #openstack-neutron | 09:43 | |
| *** matrohon has joined #openstack-neutron | 09:47 | |
| *** safchain has quit IRC | 10:06 | |
| *** safchain has joined #openstack-neutron | 10:07 | |
| *** steven-weston has quit IRC | 10:11 | |
| *** matsuhashi has quit IRC | 10:14 | |
| openstackgerrit | Isaku Yamahata proposed a change to openstack/neutron: Implement service vm framework(WIP): https://review.openstack.org/56892 | 10:18 |
| *** networkstatic is now known as networkstatic_zZ | 10:20 | |
| *** Jianyong has joined #openstack-neutron | 10:27 | |
| *** jp_at_hp has joined #openstack-neutron | 10:40 | |
| *** marun has quit IRC | 10:42 | |
| openstackgerrit | Zhang Hua proposed a change to openstack/neutron: Fwaas can't run in operating system without namespace feature https://review.openstack.org/56893 | 10:49 |
| *** gdubreui has joined #openstack-neutron | 10:51 | |
| openstackgerrit | Isaku Yamahata proposed a change to openstack/neutron: Implement service vm framework(WIP): https://review.openstack.org/56892 | 10:55 |
| *** coolsvap has quit IRC | 10:57 | |
| openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: NVP plugin:fix delete sec group when backend is out of sync https://review.openstack.org/56591 | 10:57 |
| obondarev | salv-orlando: ping | 11:07 |
| salv-orlando | ho obondarev - did I see wrong your patch? | 11:08 |
| obondarev | I've replied on your comment, can you please take a look? | 11:08 |
| obondarev | it's https://review.openstack.org/#/c/53364/ | 11:08 |
| salv-orlando | I agree its not up to db class dealing with the backend consistency issue | 11:15 |
| salv-orlando | obondarev: I have another question however, see gerrit | 11:15 |
| obondarev | yeah, thank you | 11:16 |
| *** mihgen has quit IRC | 11:16 | |
| obondarev | salv-orlando: good catch! | 11:18 |
| obondarev | salv-orlando: need to fetch ports after deleting floating ips I suppose | 11:19 |
| *** pcm_ has joined #openstack-neutron | 11:19 | |
| salv-orlando | obondarev: That would be one way to go about it - but you can also just add a condition to skip AUTO_DELETE_PORT_OWNER or the disassociated floating ips. They both look good so far | 11:22 |
| *** pcm_ has quit IRC | 11:23 | |
| *** pcm_ has joined #openstack-neutron | 11:24 | |
| *** mihgen has joined #openstack-neutron | 11:25 | |
| obondarev | salv-orlando: ok, thanks! | 11:27 |
| openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: Test commit for testing parallel job in experimental queue https://review.openstack.org/57420 | 11:32 |
| openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: Delete disassociated floating ips on external network deletion https://review.openstack.org/53364 | 11:34 |
| *** idella4 has joined #openstack-neutron | 11:40 | |
| idella4 | oh anteaya | 11:40 |
| *** mihgen_ has joined #openstack-neutron | 11:41 | |
| *** bvandenh has quit IRC | 11:43 | |
| *** mihgen has quit IRC | 11:45 | |
| *** mihgen_ is now known as mihgen | 11:45 | |
| *** gdubreui has quit IRC | 11:51 | |
| *** rkukura has quit IRC | 11:59 | |
| *** yamahata_ has joined #openstack-neutron | 12:00 | |
| *** wenjianhn has quit IRC | 12:08 | |
| *** terrylhowe has quit IRC | 12:24 | |
| *** marun has joined #openstack-neutron | 12:24 | |
| *** wenjianhn has joined #openstack-neutron | 12:24 | |
| *** marun has quit IRC | 12:34 | |
| *** marun has joined #openstack-neutron | 12:34 | |
| *** pcm__ has joined #openstack-neutron | 12:49 | |
| *** pcm_ has quit IRC | 12:52 | |
| *** pcm__ has quit IRC | 12:56 | |
| *** changbl has quit IRC | 12:57 | |
| *** pcm_ has joined #openstack-neutron | 12:57 | |
| idella4 | I have lots of dots, they all just need joining together, plus 1 rather nouvelle suggestion which will likely go down like a lead balloon | 13:04 |
| anteaya | hi idella4 | 13:05 |
| *** mvenesio has joined #openstack-neutron | 13:07 | |
| *** afazekas is now known as afazekas_mtg | 13:18 | |
| *** jprovazn has joined #openstack-neutron | 13:26 | |
| *** amotoki has joined #openstack-neutron | 13:32 | |
| anteaya | in case you haven't seen this ml post: http://lists.openstack.org/pipermail/openstack-dev/2013-November/019826.html | 13:38 |
| anteaya | the gate is in really bad shape and 3 of the four bugs causing the most problems are associated with -neutron | 13:39 |
| anteaya | one of them affects both -neutron and -nova | 13:39 |
| anteaya | we talked about this one: https://bugs.launchpad.net/neutron/+bug/1251784 | 13:39 |
| anteaya | at Monday's meeting | 13:39 |
| *** enikanorov_ has joined #openstack-neutron | 13:39 | |
| anteaya | and at the time discussed that it is not a duplicate of https://bugs.launchpad.net/neutron/+bug/1211915 | 13:40 |
| anteaya | <salv-orlando> jog0 said it's a different manifestation; perhaps different root cause. | 13:41 |
| anteaya | we have had 328 hits on that bug | 13:41 |
| anteaya | who is willing to work on it? | 13:42 |
| anteaya | ttx changed its status to critical 2 hours ago | 13:42 |
| *** arahal has joined #openstack-neutron | 13:42 | |
| *** enikanorov_ has quit IRC | 13:43 | |
| anteaya | and I would prefer someone other than salv-orlando marun or arosen since they are already working on gate blocking bugs themselves | 13:43 |
| idella4 | ah hi | 13:44 |
| idella4 | my tv program just finished | 13:44 |
| idella4 | how are you? | 13:44 |
| marun | salv-orlando: is that the one that the polling minimization might fix? | 13:44 |
| marun | ah, guess not | 13:45 |
| anteaya | hi marun | 13:45 |
| marun | hi | 13:45 |
| anteaya | is this the bug you are working on: https://bugs.launchpad.net/neutron/+bug/1251448 | 13:45 |
| salv-orlando | polling minimitazion is for bug 1224001 | 13:45 |
| *** afazekas_mtg is now known as afazekas | 13:45 | |
| marun | si | 13:46 |
| *** enikanorov_ has joined #openstack-neutron | 13:46 | |
| idella4 | hmmm | 13:46 |
| anteaya | marun: how it is going? | 13:47 |
| *** mihgen_ has joined #openstack-neutron | 13:47 | |
| *** ogelbukh has quit IRC | 13:47 | |
| *** akamyshnikova has quit IRC | 13:47 | |
| marun | looking at it now | 13:47 |
| anteaya | salv-orlando: who is Terry? https://review.openstack.org/#/c/57290/ | 13:47 |
| anteaya | marun: thanks | 13:47 |
| salv-orlando | bug 1251784 is looking for an owner. I have my pipeline full until the end of the day, and I've already distributed several other gate blocking bugs to other neutron devs; we need to find an owner for this too or it'll have to wait until tomororw. | 13:47 |
| marun | otherwiseguy | 13:47 |
| salv-orlando | Terry Wilson aka otherwiseguy | 13:47 |
| *** mihgen has quit IRC | 13:47 | |
| *** mihgen_ is now known as mihgen | 13:47 | |
| anteaya | okay thanks | 13:47 |
| anteaya | otherwiseguy: ping | 13:47 |
| anteaya | salv-orlando: we need to find someone to work on 1251784 ASAP and I agree it can't be you | 13:48 |
| marun | salv-orlando: Terry had some concerns about the existing functional test that I've hopefully addressed. He should have a patch to submit today. | 13:48 |
| *** akamyshnikova has joined #openstack-neutron | 13:48 | |
| salv-orlando | marun: ok | 13:49 |
| anteaya | marun sorry are you referencing a patch for 1251448? when you say Terry had concerns about existing functiontional test? | 13:49 |
| marun | sorry, for 12240001 | 13:50 |
| marun | one fewer zero | 13:50 |
| anteaya | k | 13:50 |
| *** ogelbukh has joined #openstack-neutron | 13:50 | |
| marun | the functional test can't run at the moment anyway. speaking of, I should work with infra to get that fixed... | 13:50 |
| anteaya | marun: yes please | 13:50 |
| marun | (need a separate unit test job that allows sudo) | 13:51 |
| anteaya | fungi should be online in about 40 minutes | 13:51 |
| marun | ok | 13:51 |
| anteaya | mordred is on the west coast so isn't up yet | 13:51 |
| anteaya | and so far it is just me there | 13:51 |
| marun | damn west coasters | 13:51 |
| marun | (i'm normally one) | 13:51 |
| anteaya | but we can get something in the backscroll | 13:51 |
| anteaya | :D | 13:51 |
| anteaya | he is on the west coast right now | 13:51 |
| marun | -14h from me right now | 13:52 |
| anteaya | normally he is flying somewhere | 13:52 |
| anteaya | and his mail goes to nyc | 13:52 |
| anteaya | wow | 13:52 |
| anteaya | were are you now? | 13:52 |
| marun | taipei | 13:52 |
| anteaya | how is the weather in taipei? | 13:52 |
| marun | temperate, quite nice really | 13:52 |
| anteaya | cool | 13:53 |
| idella4 | oh my | 13:53 |
| anteaya | ice is forming on the lake out my window | 13:53 |
| marun | i'm a fair-weather canadian. i only go back when it's in the 20s | 13:53 |
| anteaya | ha | 13:54 |
| anteaya | where is home? | 13:54 |
| anteaya | you won't be back for a while | 13:54 |
| *** bvandenh has joined #openstack-neutron | 13:55 | |
| *** abhraut has quit IRC | 13:56 | |
| salv-orlando | marun: Bug 1251448 is also an obvious blocker for enabling parallel testing | 13:56 |
| marun | salv-orlando: roger that | 13:58 |
| marun | anteaya: bay area | 13:58 |
| anteaya | marun: nice | 13:59 |
| anteaya | arosen isn't online right now, he has https://bugs.launchpad.net/nova/+bug/1249065 | 13:59 |
| anteaya | if anyone works with him please ask him to join us in here and share his status | 13:59 |
| anteaya | if we can at least update folks with our progress, they can look for ways to support and help our work | 14:00 |
| *** dkliban has joined #openstack-neutron | 14:01 | |
| idella4 | anteaya: I think your suspected race condition notion might actually be correct in the testsuite fails from what I've observed | 14:03 |
| *** julim has joined #openstack-neutron | 14:04 | |
| idella4 | but to pin it and fix it, ugh | 14:04 |
| anteaya | idella4: yes, knowing there is a race and fixing the race, two different things | 14:04 |
| idella4 | o hyep | 14:04 |
| idella4 | testr run --failing runs all the tests that failed right? | 14:06 |
| *** yamahata_ has quit IRC | 14:06 | |
| *** yamahata_ has joined #openstack-neutron | 14:06 | |
| salv-orlando | idella4: I've barely followed your unit test issues, but have you tried to apply this patch: https://review.openstack.org/#/c/54208/ | 14:08 |
| *** carl_baldwin has joined #openstack-neutron | 14:08 | |
| openstackgerrit | Akihiro Motoki proposed a change to openstack/neutron: Replace stubout with fixtures https://review.openstack.org/55805 | 14:08 |
| idella4 | let's look | 14:08 |
| *** wenjianhn has quit IRC | 14:09 | |
| anteaya | marun: I am composing an email message to the list updating them on our current status on the -neutron bugs | 14:10 |
| anteaya | so far I have Maru Newby and I are talking about it in the -neutron channel and he is looking at it now | 14:10 |
| *** rkukura has joined #openstack-neutron | 14:11 | |
| anteaya | I can hold off on it if you think I can improve on that statement | 14:11 |
| anteaya | hi rkukura | 14:11 |
| idella4 | salv-orlando: that looksquite pertinent | 14:11 |
| idella4 | salv-orlando: the only patch I've addedto the ebuild currently is the one that fixed the erialization issue | 14:12 |
| idella4 | salv-orlando: the only patch I've addedto the ebuild currently is the one that fixed the serialization issue | 14:12 |
| rkukura | anteaya: Hi. I've got a meeting in a moment - will catch up with you in a bit. | 14:12 |
| idella4 | this one looks most apt | 14:12 |
| anteaya | rkukura: k | 14:14 |
| idella4 | hmmm; Updated Nov 19, 2013 3:36 AM == yesterday | 14:14 |
| anteaya | was just saying good morning | 14:14 |
| idella4 | wow | 14:14 |
| salv-orlando | anteaya: perhaps markmclain has a more precise assessment of the current status of each gate-blocking bug (assuming you're talking about those bugs). I know only about those who were initially assigned to me and then I delegated those tasks | 14:16 |
| anteaya | great | 14:16 |
| anteaya | markmcclain: ping | 14:16 |
| anteaya | I need to communicate what I know as imperfect as it is, to the list | 14:17 |
| idella4 | salv-orlando: I won't go into my issues here it's plain as day you're all under seige, but tehy do have the themes of being triggerred by some test and the backtrace pointing at a packahe module rather than a test | 14:17 |
| anteaya | waiting for perfection, the silence that it takes to get there is interpreted as no activity | 14:17 |
| idella4 | a very tricky combo | 14:17 |
| anteaya | because noone else knows what is happening | 14:17 |
| anteaya | since they have no update | 14:18 |
| idella4 | testr run --failing runs all the tests that failed right? | 14:18 |
| anteaya | I'll post what I have and welcome Mark's input and corrections where I err when he can | 14:18 |
| salv-orlando | idella4: yes, but since you're seeing timing issues in your tests tests run --failing will probably just pass | 14:19 |
| *** aymenfrikha has joined #openstack-neutron | 14:19 | |
| idella4 | that's exactly what's going on | 14:19 |
| *** alagalah has joined #openstack-neutron | 14:19 | |
| idella4 | they always just pass which I am interpreting as leaning towards the timing / racing | 14:20 |
| mestery | anteaya: Any chance I can get you to jump over to #openstack-meeting? We have a few Tempest questions for you in our weekly ML2 meeting. | 14:20 |
| anteaya | mestery: yes | 14:21 |
| idella4 | anyway I'll put this patch in and re-run | 14:21 |
| mestery | anteaya: Thanks! | 14:21 |
| idella4 | is it this link? https://review.openstack.org/#q,I697dd8fc509308108ff1f40400f36ac6271bf4bb,n,z | 14:22 |
| idella4 | that didn't work | 14:23 |
| *** peristeri has joined #openstack-neutron | 14:23 | |
| salv-orlando | idella4: what did not work exactly? | 14:23 |
| idella4 | that link | 14:23 |
| idella4 | the page is full of links, which is the final latest patch | 14:24 |
| idella4 | @ https://review.openstack.org/gitweb?p=openstack%2Fneutron.git;a=commitdiff;h=a52ef6ecf19b8b015465ddda2a3ca087f0e12122 ? | 14:24 |
| salv-orlando | I don't understand if you're saying the patch did not solve your unit test problem or if the link did not work | 14:24 |
| idella4 | I'm just trying to get the patch | 14:24 |
| salv-orlando | right. So that particular patch has not landed yet. | 14:24 |
| salv-orlando | you need to cherry pick it into your local branch using git review | 14:25 |
| salv-orlando | git review -x 54208 should do the trick | 14:25 |
| idella4 | local branch hmm | 14:26 |
| salv-orlando | I was assuming you had a local branch - that would be tricker if you're running some automated system which pull from github | 14:27 |
| *** aveiga has joined #openstack-neutron | 14:27 | |
| idella4 | well I can make one I suppose | 14:28 |
| salv-orlando | but if you're using something like jenkins you should be able to tell it to pull code from alternate location | 14:28 |
| idella4 | this looks good https://review.openstack.org/gitweb?p=openstack%2Fneutron.git;a=commitdiff;h=a52ef6ecf19b8b015465ddda2a3ca087f0e12122 | 14:28 |
| *** safchain has quit IRC | 14:29 | |
| idella4 | I'm a gentoo dev, I don't belong to neutron at all, but cloning the repo is pretty basic | 14:29 |
| idella4 | just haven't had to do it yet | 14:30 |
| salv-orlando | oh I see | 14:30 |
| idella4 | that link looks like the patch to me | 14:30 |
| idella4 | thx salv-orlando | 14:30 |
| salv-orlando | it is the patch - I did not know you could just apply it | 14:31 |
| salv-orlando | I was under the assumption you had an automated system pulling code from trunk and running gentoo tests | 14:31 |
| idella4 | via the ebuild sure | 14:31 |
| idella4 | nope, but that sounds a fair assumption | 14:31 |
| idella4 | the ebuild is the bash script that runs the emerge process(es) | 14:32 |
| idella4 | I'll show you my current ebuild | 14:33 |
| otherwiseguy | salv-orlando: pong | 14:34 |
| idella4 | just to help clear the picture for you a little | 14:34 |
| idella4 | @ http://codepad.org/v4Okm7wI | 14:34 |
| salv-orlando | hi otherwiseguy: I did not ping you. I was just asked about who's looking after the minimizepolling patch | 14:35 |
| idella4 | see I'll just add the pasted patch to PATCHES=( "${FILESDIR}"/${P}-sphinx_mapping.patch \ "${FILESDIR}"/${P}-json-tests.patch ) | 14:35 |
| anteaya | I pinged you otherwiseguy | 14:36 |
| anteaya | otherwiseguy: we are talking about https://review.openstack.org/#/c/57290/ | 14:36 |
| otherwiseguy | This is what happens when I roll out of bed and immediately start trying to read. :) | 14:36 |
| anteaya | yeah | 14:36 |
| anteaya | do you need coffee first? | 14:37 |
| idella4 | you should JUMP out of bed | 14:37 |
| anteaya | no, only you JUMP out of bed idella4 | 14:37 |
| idella4 | back in the day | 14:37 |
| anteaya | I have no doubt | 14:37 |
| idella4 | er yep | 14:38 |
| otherwiseguy | My alarm clock is a 50lb dog that tends to leap onto my stomach when she decides it is time for me to wake up. | 14:38 |
| anteaya | ha ha ha | 14:38 |
| anteaya | what breed? | 14:38 |
| otherwiseguy | anteaya: she is the result of many many generations of indiscriminate breeding. German-shepherd mix. maybe some doberman. lots of other things. :) | 14:39 |
| anteaya | nice | 14:40 |
| anteaya | I had a shepherd-border collie mix for almost 14 years | 14:40 |
| anteaya | she was so smart | 14:40 |
| otherwiseguy | That sounds like a great mix. | 14:40 |
| anteaya | she was fantastic | 14:41 |
| anteaya | I would just say to her "I would prefer you didn't do that." and she would go do something else | 14:41 |
| anteaya | so when you are awake, my question for you otherwiseguy is, are you able to take over work on https://review.openstack.org/#/c/57290/ | 14:42 |
| anteaya | what are your thoughts on that? | 14:43 |
| otherwiseguy | yeah, Bailey is more of the opinion that you have somehow slighted her if you tell her not to do something. she'll kind bark at you in disapproval. But, if I tell her to do something like come or sit when she is doing something she shouldn't, she's fine. she really just doesn't like being told "no". | 14:44 |
| otherwiseguy | And yeah. I have a patch that is pretty much ready. Just need to work in some changes to the test. | 14:44 |
| *** enikanorov_ has quit IRC | 14:44 | |
| anteaya | otherwiseguy: great have you a url for that patch? | 14:45 |
| anteaya | should salv-orlando abandon https://review.openstack.org/#/c/57290/ ? | 14:45 |
| anteaya | yeah, every dog is different - we had to take about 2 years to learn each other | 14:45 |
| anteaya | but once we got it, we got it | 14:45 |
| salv-orlando | otherwise guy will take over from me as he's already working on related change. | 14:45 |
| otherwiseguy | anteaya: doesn't really matter. I can leave those changes out and he can commit that one. | 14:45 |
| otherwiseguy | as a dependent thing, or I can add them in. | 14:46 |
| anteaya | salv-orlando: what would you like to do? | 14:46 |
| salv-orlando | otherwiseguy: either way will work for me. | 14:46 |
| anteaya | I don't care, just let me know what you decide | 14:46 |
| otherwiseguy | They're different issues, really. So we could just treat them separately. | 14:46 |
| *** Chicago has joined #openstack-neutron | 14:47 | |
| salv-orlando | otherwiseguy: the patch you're working on is already in review (the rootwrap one merged yesterday I think) | 14:47 |
| otherwiseguy | 1) Make it default 2) Make it fully functional | 14:47 |
| salv-orlando | ok, so i'll keep working on 1 then | 14:47 |
| salv-orlando | and you work on item #2 | 14:47 |
| otherwiseguy | Give me about 15 minutes and I'll post the respawn interval patch for review | 14:48 |
| salv-orlando | cool | 14:48 |
| anteaya | so you are still going to work on https://review.openstack.org/#/c/57290/ salv-orlando? | 14:48 |
| otherwiseguy | Then I'll make some coffee. :) | 14:48 |
| anteaya | otherwiseguy: thanks | 14:48 |
| anteaya | :D | 14:48 |
| salv-orlando | anteaya: yes, that patch is a very tiny change at the end of the day. We already have the feature we shall just turn it on in gate jobs | 14:49 |
| anteaya | great | 14:49 |
| idella4 | that looks like the patch markmcclain alluded to yesterday or so | 14:50 |
| aveiga | anteaya: to continue the discussion from the ML2 meeting, is it possible to have informational tests added for Neutron as a whole to start checking if code works when using IPv6? | 14:51 |
| aveiga | i.e. never gating, but inform of failures when used against an IPv6 or dual-stacked network | 14:51 |
| anteaya | I think we might be talking about third party testing: http://ci.openstack.org/third_party.html | 14:52 |
| anteaya | salv-orlando: added a comment to that patch to reflect its current status | 14:52 |
| anteaya | mestery: ping | 14:53 |
| anteaya | aveiga: sorry the link for third party testing was for you | 14:54 |
| mestery | anteaya: Hey, on a phone call, give me a few minutes ... | 14:54 |
| anteaya | aveiga: let me know if you think that is what you are looking for | 14:54 |
| anteaya | mestery: k | 14:54 |
| aveiga | anteaya: I figured. I'm looking into how I can integrate there | 14:54 |
| anteaya | mestery: I need your help to get someone working on bug 1251784: http://lists.openstack.org/pipermail/openstack-dev/2013-November/019878.html | 14:54 |
| anteaya | aveiga: okay | 14:54 |
| aveiga | I'm just concerned about a lot of work being done that is basically keyed into IPv4 only and is going to block me from deploying things. | 14:55 |
| anteaya | I think that would be the best approach | 14:55 |
| aveiga | i.e. L2 population is basically intercepting ARP, but there's no ARP in IPv6... | 14:55 |
| anteaya | aveiga: I don't have enough techinical knowledge to be able to offer an opinion | 14:55 |
| *** banix has joined #openstack-neutron | 14:55 | |
| aveiga | that's why I'm offering to help :) | 14:55 |
| *** changbl has joined #openstack-neutron | 14:55 | |
| anteaya | aveiga: awesome | 14:56 |
| anteaya | aveiga: can you expand on your thought that work on IPv4 only will block you | 14:56 |
| mestery | aveiga: L2 Pop is not required to be deployed when runing ML2 as an example. | 14:56 |
| mestery | aveiga: So you could just choose not to enable that MechanismDriver for example. | 14:57 |
| *** jecarey has joined #openstack-neutron | 14:57 | |
| aveiga | understood, butwe're running into basic bugs elsewhere too. I still need to file a report for a bug I hit with L2 provider where adding an IPv6 subnet shuts off v4 too | 14:57 |
| *** s3wong has joined #openstack-neutron | 14:57 | |
| aveiga | I'm behind on my test documentation at the moment | 14:57 |
| anteaya | rkukura: can you help me find someone to work on bug 1251784? | 14:58 |
| *** rcurran_ has joined #openstack-neutron | 14:59 | |
| *** networkstatic_zZ has quit IRC | 15:01 | |
| *** networkstatic has joined #openstack-neutron | 15:02 | |
| rcurran_ | rkukura here? | 15:02 |
| *** rkukura has quit IRC | 15:03 | |
| *** nati_ueno has joined #openstack-neutron | 15:03 | |
| *** rkukura has joined #openstack-neutron | 15:03 | |
| *** dkliban has quit IRC | 15:05 | |
| rcurran_ | rkukura here? | 15:05 |
| rkukura | yes, I'm back online | 15:05 |
| *** carl_baldwin has quit IRC | 15:05 | |
| rcurran_ | ok, did you see my response on openstack-meeting? | 15:06 |
| rcurran_ | you asked if i could use network context to get vlan_id | 15:06 |
| rcurran_ | i use the port.bound_segment today for create, update, deletes ... are you saying to use this for port create,updates but not deletes? | 15:07 |
| rkukura | rcurran_: I'm just asking if it works in delete_port_postcommit() | 15:08 |
| rcurran_ | wouldn't i have to do something like this to get the segmentation_id based on the network context: segmentation_id = segments[0]['segmentation_id'] | 15:09 |
| *** enikanorov_ has joined #openstack-neutron | 15:09 | |
| rcurran_ | to answer your q: I don't know. Could be, but i didn't think that's how we should access vlan ID now that portbinding is in place | 15:10 |
| rkukura | rcurran_: You should be able to use PortContext.bound_segment['binding:segmentation_id'], right? | 15:12 |
| *** Chicago has quit IRC | 15:13 | |
| rcurran_ | yes ... but that's what doesn't work on the delete_port_postcommit() ... ml2 common calls .delete_port_binding() [or whatever it's called] before the delete_port_postcommit() call | 15:14 |
| rkukura | rcurran_: Was just looking, and I see that. | 15:14 |
| rcurran_ | at one point it was called before the delete_port_precommit() method. someone switched the calling order to get the delete_port_precommit() to work | 15:14 |
| rkukura | rcurran_: Maybe that is why this sounds familiar | 15:15 |
| rcurran_ | :-) | 15:15 |
| rkukura | Maybe we don't need to call delete_port_binding() at all - let me look. | 15:15 |
| rcurran_ | ok | 15:16 |
| *** coolsvap has joined #openstack-neutron | 15:17 | |
| rkukura | rcurran_: So delete_port_binding() calls MechanismManager.unbind_port() which calls unbind_port() on the bound MechanismDriver. Can your driver do what it needs there? | 15:18 |
| rcurran_ | maybe. the unbind_port() is a new method, right? doesn't look like many md's are using it | 15:20 |
| rkukura | rcurran_: unbind_port() is called inside the transaction. Do you need to be called after the transaction commits? | 15:20 |
| rkukura | rcurran_: It was added as part of the port binding BP. | 15:20 |
| rcurran_ | right about transaction, i don't know why that was. | 15:21 |
| openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: LBaaS: unify haproxy-on-host plugin driver and agent https://review.openstack.org/40381 | 15:21 |
| rcurran_ | i just have been using the rule that for precommit() calls md's do their db transactions. on postcommit() you take specific md actions (for us calling down to the nexus switch to add/delete config) | 15:22 |
| anteaya | obondarev: check your email | 15:22 |
| *** thedodd has joined #openstack-neutron | 15:23 | |
| idella4 | oh anteaya do you have a link for that etherpad thingy for pasting output -> making a link for bug comment? | 15:23 |
| idella4 | I think the etherpad thingy has a number of uses | 15:24 |
| rcurran_ | is there a reason why the delete_port_binding() call is made inside the db transaction? | 15:24 |
| anteaya | idella4: it does | 15:25 |
| anteaya | etherpad: etherpad.openstack.org | 15:25 |
| anteaya | paste: paste.openstack.org | 15:25 |
| anteaya | use them as much as you like | 15:26 |
| anteaya | we still need someone willing to work on https://bugs.launchpad.net/neutron/+bug/1251784 | 15:27 |
| *** ben_duyujie has joined #openstack-neutron | 15:27 | |
| *** ben_duyujie has quit IRC | 15:30 | |
| anteaya | rkukura: any chances you can help to find someone with bug 1251784? | 15:30 |
| mestery | anteaya: Regarding 1251784, I see gongysh asked a question on that bug which has not been answered yet (https://bugs.launchpad.net/neutron/+bug/1251784/comments/2) | 15:31 |
| *** ben_duyujie has joined #openstack-neutron | 15:31 | |
| mestery | I think that would help in neutron devs being able to reproduce this locally | 15:31 |
| anteaya | sure | 15:32 |
| anteaya | who did gongysh ask the question of? | 15:32 |
| anteaya | who is he waiting to get an answer from? | 15:32 |
| anteaya | looks like he is asking it of a devstack dev | 15:33 |
| anteaya | what is stopping him from going into -qa and asking one of the devstack core? https://review.openstack.org/#/admin/groups/50,members | 15:34 |
| mestery | anteaya: He just asked it in the bug I'm afraid. | 15:34 |
| anteaya | okay | 15:34 |
| anteaya | and he isn't in channel | 15:34 |
| mestery | anteaya: Not sure, just noticed that in the bug. :) | 15:34 |
| anteaya | okay | 15:34 |
| mestery | Let me ask in -qa and see what I can find out. | 15:34 |
| *** ben_duyujie1 has joined #openstack-neutron | 15:34 | |
| anteaya | awesome, thank you | 15:35 |
| *** armax has joined #openstack-neutron | 15:36 | |
| *** mihgen has quit IRC | 15:37 | |
| salv-orlando | README.rst in devstack-gate.git has that info perhaps | 15:37 |
| anteaya | http://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/README.rst | 15:38 |
| *** ben_duyujie has quit IRC | 15:38 | |
| anteaya | so does this information help find someone willing to take responsibility for this bug? | 15:43 |
| *** yamahata_ has quit IRC | 15:43 | |
| mestery | salv-orlando: Thanks! | 15:43 |
| *** nati_ueno has quit IRC | 15:50 | |
| anteaya | mestery: any thoughts on how we find someone to work on this bug? | 15:51 |
| mestery | anteaya: I'm looking at this now from the perspective of recreation. | 15:52 |
| mestery | anteaya: I'm learning as I go here, so bear with me. :) | 15:52 |
| anteaya | mestery: awesome, thank you | 15:52 |
| anteaya | no no take your time, just happy you are looking at it | 15:52 |
| mestery | anteaya: thanks to you for your help as well | 15:52 |
| anteaya | let me know if there is anything you need to progress | 15:52 |
| anteaya | happy to help | 15:52 |
| mestery | will do | 15:53 |
| *** carl_baldwin has joined #openstack-neutron | 15:54 | |
| *** ekarlso has joined #openstack-neutron | 15:55 | |
| *** amritanshu_RnD has quit IRC | 15:57 | |
| openstackgerrit | Terry Wilson proposed a change to openstack/neutron: Add configurable ovsdb monitor respawn interval https://review.openstack.org/57475 | 16:00 |
| openstackgerrit | Terry Wilson proposed a change to openstack/neutron: Ensure get_pid_to_kill works with rootwrap script https://review.openstack.org/53188 | 16:00 |
| *** mlavalle has joined #openstack-neutron | 16:03 | |
| otherwiseguy | anteaya: salv-orlando: ^^ it took me a little longer to track down why a test was failing, but hopefully that'll do it | 16:04 |
| salv-orlando | thanks | 16:05 |
| *** marun has quit IRC | 16:06 | |
| anteaya | otherwiseguy: awesome | 16:06 |
| *** yfried has quit IRC | 16:13 | |
| ekarlso | Anyone know why my devstack with neutron is failing ? | 16:14 |
| ekarlso | http://paste.ubuntu.com/6448884/ | 16:14 |
| anteaya | hi ekarlso I'll look | 16:15 |
| anteaya | what image is it on? ubuntu 12.04? | 16:15 |
| ekarlso | 12.04 | 16:16 |
| anteaya | k | 16:16 |
| anteaya | did devstack install fine for you or were there issues on installation? | 16:16 |
| ekarlso | no issues during install | 16:17 |
| anteaya | there is a devstack installation bug that is hitting 12.04 which is why I ask | 16:17 |
| anteaya | okay | 16:17 |
| idella4 | anteaya: you might care to peruse https://bugs.launchpad.net/neutron/+bug/1251657 if you get time, likely post the working day | 16:18 |
| anteaya | have you gotten neutron working with devstack successfully in the past, or is this your first time working with neutron in devstack? | 16:18 |
| anteaya | idella4: thanks, is it related to ekarlso's question? | 16:18 |
| idella4 | nope, just dropped it in having just clicked it | 16:19 |
| anteaya | idella4: very good | 16:19 |
| idella4 | I'm shitting down | 16:19 |
| anteaya | ekarlso: I see this: RuntimeError: Unable to load neutron from configuration file /etc/neutron/api-paste.ini | 16:19 |
| idella4 | shutting ooopsie | 16:19 |
| anteaya | idella4: okay | 16:19 |
| ekarlso | anteaya: yeah | 16:19 |
| anteaya | ttyl | 16:19 |
| ekarlso | -rw-rw-r-- 1 karlsone karlsone 725 Nov 20 07:58 /etc/neutron/api-paste.ini | 16:20 |
| ekarlso | though... | 16:20 |
| anteaya | hmmmm | 16:20 |
| anteaya | I think that file needs to belong to stack | 16:20 |
| mestery | ekarlso: What does your ml2_conf.ini file look like? | 16:21 |
| mestery | ekarlos: Instead of ovs, use openvswitch as the mechanism_driver. | 16:21 |
| mestery | That's your problem. | 16:21 |
| mestery | This line here in your paste output: | 16:21 |
| mestery | 2013-11-20 08:14:32.211 [00;36mINFO neutron.plugins.ml2.managers [[00;36m-[00;36m] [01;35m[00;36mConfigured mechanism driver names: ['ovs', 'logger'][00m | 16:21 |
| mestery | Should be 'openvswitch' instead of 'ovs' | 16:22 |
| *** idella4 has quit IRC | 16:24 | |
| *** jlibosva has quit IRC | 16:25 | |
| *** coolsvap has quit IRC | 16:29 | |
| ekarlso | thnx mestery ! | 16:29 |
| anteaya | ekarlso: let us know how you do with the change | 16:30 |
| ekarlso | that worked! | 16:30 |
| anteaya | awesome | 16:30 |
| *** jlibosva has joined #openstack-neutron | 16:30 | |
| anteaya | I'm afk for a bit | 16:30 |
| *** jlibosva has joined #openstack-neutron | 16:31 | |
| *** jlibosva has joined #openstack-neutron | 16:33 | |
| mestery | ekarlso: Glad it worked. | 16:33 |
| *** jlibosva has quit IRC | 16:33 | |
| *** jsergent has joined #openstack-neutron | 16:41 | |
| *** sgandikota has joined #openstack-neutron | 16:46 | |
| *** s3wong has quit IRC | 16:47 | |
| *** coolsvap has joined #openstack-neutron | 16:48 | |
| *** afazekas has quit IRC | 16:48 | |
| *** sgandikota has quit IRC | 16:49 | |
| *** coolsvap has quit IRC | 16:49 | |
| openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: Test commit for testing parallel job in experimental queue https://review.openstack.org/57420 | 16:52 |
| *** coolsvap has joined #openstack-neutron | 16:52 | |
| ekarlso | mestery: shouldn't br-int be created by devstack ? | 16:55 |
| EmilienM | ekarlso: it is afik | 17:01 |
| *** mihgen has joined #openstack-neutron | 17:01 | |
| *** sgandikota has joined #openstack-neutron | 17:02 | |
| ekarlso | EmilienM: it didn't for me : p | 17:02 |
| mestery | ekarlso: I think so, it's not? | 17:03 |
| ekarlso | nope | 17:04 |
| *** jpich has quit IRC | 17:05 | |
| *** mihgen has quit IRC | 17:05 | |
| openstackgerrit | Terry Wilson proposed a change to openstack/neutron: Add configurable ovsdb monitor respawn interval https://review.openstack.org/57475 | 17:10 |
| *** mihgen has joined #openstack-neutron | 17:12 | |
| *** suresh12 has joined #openstack-neutron | 17:15 | |
| markmcclain | config error | 17:23 |
| markmcclain | "[01;31m2013-11-20 08:14:32.222 TRACE neutron.common.config [01;35m[00mKeyError: 'ovs'" | 17:23 |
| markmcclain | oops.. old send.. ignore | 17:23 |
| anteaya | hello markmcclain | 17:26 |
| markmcclain | hi | 17:27 |
| anteaya | markmcclain: have you been following the top gate bugs post on the ml? http://lists.openstack.org/pipermail/openstack-dev/2013-November/019826.html | 17:31 |
| markmcclain | I haven't read the one from today | 17:32 |
| anteaya | k | 17:32 |
| anteaya | just for the sake of status right now mestery is trying to recreate https://bugs.launchpad.net/neutron/+bug/1251784 locally | 17:33 |
| anteaya | that is the only change not posted to ml or bug report atm | 17:33 |
| *** amuller has quit IRC | 17:36 | |
| *** chandankumar has quit IRC | 17:40 | |
| anteaya | markmcclain: do you have any way to get a hold of arosen to see how he is progressing with this bug? https://bugs.launchpad.net/nova/+bug/1249065 | 17:40 |
| markmcclain | he's not online yet | 17:41 |
| anteaya | okay | 17:41 |
| *** ygbo has quit IRC | 17:41 | |
| anteaya | is he on pacific time? | 17:41 |
| anteaya | mestery: speak up if there is anything anyone can do to unstick you | 17:43 |
| *** reed_ has joined #openstack-neutron | 17:46 | |
| *** salv-orlando has quit IRC | 17:49 | |
| *** salv-orlando has joined #openstack-neutron | 17:54 | |
| *** sgandikota has quit IRC | 17:55 | |
| *** suresh12 has quit IRC | 17:59 | |
| *** suresh12 has joined #openstack-neutron | 18:00 | |
| *** jp_at_hp has quit IRC | 18:02 | |
| *** harlowja has joined #openstack-neutron | 18:02 | |
| *** obondarev has quit IRC | 18:03 | |
| *** enikanorov__ has joined #openstack-neutron | 18:04 | |
| *** suresh12 has quit IRC | 18:04 | |
| *** enikanorov_ has quit IRC | 18:05 | |
| openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: Test commit for testing parallel job in experimental queue https://review.openstack.org/57420 | 18:07 |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/neutron: Midonet plugin, set __native_bulk_support correctly https://review.openstack.org/50665 | 18:14 |
| *** rossella_s has quit IRC | 18:15 | |
| *** ben_duyujie1 has quit IRC | 18:18 | |
| *** marun has joined #openstack-neutron | 18:19 | |
| salv-orlando | anteaya: arosen is unfortunately busy this morning with other work commitments; he shall be available after lunch - pacific time | 18:23 |
| otherwiseguy | salv-orlando: any chance I could get a re-re-re-re-approval on https://review.openstack.org/#/c/53188/ ? :p | 18:25 |
| salv-orlando | otherwiseguy: done | 18:26 |
| otherwiseguy | thx | 18:26 |
| *** aymenfrikha has left #openstack-neutron | 18:26 | |
| *** aymenfrikha has joined #openstack-neutron | 18:26 | |
| * otherwiseguy crosses his fingers and looks for some kind of animal to sacrifice to the God of Chaos^H^H^H^H^HJenkins | 18:27 | |
| *** enikanorov_ has joined #openstack-neutron | 18:27 | |
| *** enikanorov__ has quit IRC | 18:28 | |
| anteaya | salv-orlando: great thanks for letting me know | 18:29 |
| salv-orlando | np | 18:31 |
| anteaya | I think I am going to get my walk in now | 18:33 |
| anteaya | see you in a bit | 18:33 |
| *** mlavalle has quit IRC | 18:40 | |
| *** enikanorov_ has quit IRC | 18:50 | |
| *** enikanorov_ has joined #openstack-neutron | 18:51 | |
| *** suresh12 has joined #openstack-neutron | 18:56 | |
| *** networkstatic has quit IRC | 18:59 | |
| *** devlaps has joined #openstack-neutron | 19:03 | |
| *** yfried has joined #openstack-neutron | 19:05 | |
| roaet | markmcclain: ping | 19:19 |
| roaet | markmcclain: you approved https://blueprints.launchpad.net/python-neutronclient/+spec/pluggable-neutronclient-auth and, while we want to work on it, we would like to submit a 'mini' blueprint that just first provides noauth. What do you recommend for such a procedure? | 19:19 |
| *** enikanorov_ has quit IRC | 19:20 | |
| *** enikanorov_ has joined #openstack-neutron | 19:20 | |
| jog0 | you guys have any patches to fix any ofthe top bugs ready? | 19:23 |
| jog0 | we are planning on bumping bug fixes to the top of the quueue | 19:24 |
| *** jprovazn has quit IRC | 19:27 | |
| *** ivar-lazzaro has joined #openstack-neutron | 19:28 | |
| *** enikanorov_ has quit IRC | 19:30 | |
| *** enikanorov_ has joined #openstack-neutron | 19:31 | |
| *** mjbright_ has joined #openstack-neutron | 19:32 | |
| *** mihgen has quit IRC | 19:33 | |
| salv-orlando | jog0: we might have a patch for 1224001 soon | 19:34 |
| openstackgerrit | Justin Hammond proposed a change to openstack/python-neutronclient: Adds support for os-auth-strategy=noauth https://review.openstack.org/57517 | 19:34 |
| jog0 | salv-orlando: kk | 19:34 |
| salv-orlando | it's dependent on another patch which is backlogged in the queue | 19:35 |
| *** ivar-lazzaro has quit IRC | 19:41 | |
| *** banix has quit IRC | 19:47 | |
| carl_baldwin | Hi, all. Wondering about the "Agent Framework Refactoring" bullet in the NeutronIcehouseProjectPlan ether pad. I'd like to get up to speed. Looking for related blueprints. Can someone point me to where this work is being discussed? | 19:48 |
| *** banix has joined #openstack-neutron | 19:48 | |
| *** carl_baldwin has left #openstack-neutron | 19:50 | |
| *** carl_baldwin has joined #openstack-neutron | 19:50 | |
| anteaya | salv-orlando: which patch is 1224001 depedent on, that patch should get bumped up too | 19:53 |
| salv-orlando | it's in gate queue | 19:53 |
| salv-orlando | otherwiseguy: do you have a quick ref? | 19:53 |
| salv-orlando | https://review.openstack.org/#/c/53188/ | 19:53 |
| *** reed_ is now known as reed | 19:54 | |
| *** reed has quit IRC | 19:54 | |
| *** reed has joined #openstack-neutron | 19:54 | |
| anteaya | jog0: we need 53188 given priority too, please | 19:54 |
| anteaya | jog0: it is currently in the queue but if you are going to reque we need it up top | 19:55 |
| jog0 | tell infra | 19:55 |
| otherwiseguy | salv-orlando: 53188 + 57475. | 19:55 |
| anteaya | can do | 19:55 |
| jog0 | the gate is being restarted | 19:55 |
| anteaya | k | 19:55 |
| jog0 | so ASAP | 19:56 |
| *** carl_baldwin has quit IRC | 19:56 | |
| otherwiseguy | salv-orlando: (in addition to yours actually enabling minimize_polling as default) But w/o 57475, there will be a test that flaps because of a small bug in the original minimize_polling patch. | 19:56 |
| *** carl_baldwin has joined #openstack-neutron | 19:56 | |
| anteaya | we need 57475 approved please | 19:57 |
| anteaya | we are restarting the queue | 19:57 |
| anteaya | it needs to be approved now please | 19:57 |
| anteaya | or we will miss the restart | 19:57 |
| anteaya | salv-orlando markmcclain https://review.openstack.org/#/c/57475/ | 19:57 |
| *** catalyst_xav has joined #openstack-neutron | 19:58 | |
| jog0 | anteaya: and you can just do a reverify | 19:58 |
| anteaya | will that not miss putting 57475 at the top of the queue after the restart? | 19:59 |
| jog0 | anteaya: queue is clean, so it will go to top | 20:00 |
| jog0 | if you run recehck | 20:00 |
| jog0 | recheck | 20:00 |
| jog0 | or reverify | 20:00 |
| anteaya | thanks markmcclain, salv-orlando can you approve? | 20:01 |
| markmcclain | marun: can also review it too | 20:01 |
| salv-orlando | I think I should review it too | 20:01 |
| salv-orlando | not just approve it - that's my understanding | 20:01 |
| marun | i'll check | 20:02 |
| jog0 | salv-orlando: the gate is back up | 20:02 |
| otherwiseguy | marun is currently asleep in Tapei? | 20:02 |
| jog0 | but empty | 20:02 |
| otherwiseguy | oh, no he isn't! | 20:02 |
| *** jecarey has quit IRC | 20:02 | |
| anteaya | thanks | 20:02 |
| otherwiseguy | you keep weird hours, marun. :) | 20:02 |
| anteaya | go go marun | 20:02 |
| marun | not marun, really. his evil twin | 20:02 |
| jog0 | we haven't readded the old stuff | 20:02 |
| anteaya | thanks jog0 | 20:02 |
| jog0 | so any critical bug fixes, make sure they get into the queue ASAP | 20:02 |
| anteaya | let's see if we can get this in | 20:02 |
| anteaya | salv-orlando: how soon is that patch ready? | 20:02 |
| marun | otherwiseguy: hmm, this patch would appear to require respawn | 20:03 |
| salv-orlando | that depends on the review cycle. I can put up the patch in 15 min. | 20:04 |
| *** jecarey has joined #openstack-neutron | 20:04 | |
| anteaya | salv-orlando: post when you have something, let's get eyeballs on it | 20:04 |
| anteaya | and thanks | 20:04 |
| marun | otherwiseguy: ? | 20:04 |
| otherwiseguy | marun: does it? | 20:05 |
| * otherwiseguy looks | 20:05 | |
| marun | otherwiseguy: there's no way to pass None, which is what would disable respawn | 20:05 |
| marun | otherwiseguy: maybe that's not a bad thing, but i figured it was worth discussing at least | 20:05 |
| marun | (an interval of 0 respawns immediately btw | 20:06 |
| otherwiseguy | ah, as in in the config file. I was thinking "why can't you just pass in ovsdb_monitor_respawn_interval=None as an arg" :p | 20:07 |
| marun | patch looks good apart from that | 20:07 |
| marun | yeah | 20:07 |
| marun | maybe that's a non-issue though | 20:07 |
| *** carl_baldwin has quit IRC | 20:07 | |
| marun | salv-orlando, since you're around, can we get your opinion? | 20:08 |
| *** carl_baldwin has joined #openstack-neutron | 20:08 | |
| otherwiseguy | Seems like you'd always want to respawn, though, right? | 20:08 |
| salv-orlando | sorry I was on the patch | 20:08 |
| salv-orlando | scrolling back | 20:08 |
| otherwiseguy | Or just minimize_polling=False | 20:08 |
| marun | otherwiseguy: i'm thinking that's probably the case, yeah | 20:08 |
| salv-orlando | hmmm so I read the patch and the only thing I saw is that now respawn_interval can't be none into async_process | 20:08 |
| *** steven-weston has joined #openstack-neutron | 20:08 | |
| salv-orlando | is that the same thing you noticed marun? | 20:08 |
| marun | salv-orlando: yes | 20:08 |
| marun | salv-orlando: that means that the monitor will have to respawn after some interval | 20:09 |
| salv-orlando | Is that really a case where we might want to have no respawn? | 20:09 |
| marun | salv-orlando: that's probably ok | 20:09 |
| otherwiseguy | "Oh, you died? Well, better give up!" | 20:09 |
| marun | salv-orlando: because we could? I don't think there's a usecase outside of testing. | 20:09 |
| marun | for the agent, I don't think it matters | 20:09 |
| marun | as I said, I just wanted us to talk about it in case there was a reason to allow not respawning | 20:10 |
| marun | we can always fix later if a use case presents itself, but for now, I'm going to +2 and hope the pid patch finally lands | 20:10 |
| salv-orlando | ok let's approve this patch | 20:10 |
| marun | salv-orlando: please do the honours | 20:11 |
| marun | oh wait, mark already +2'd. i'll approve | 20:11 |
| salv-orlando | otherwise guy, marun: this also means we only need to switch minimize_polling to True for enabling the feature, as the respawn interval is set by default? | 20:11 |
| marun | we can't merge until the pid patch lands though :\ | 20:11 |
| marun | salv-orlando: si | 20:11 |
| salv-orlando | cool | 20:11 |
| marun | markmcclain: btw, i only saw external_process.py this week. I probably should have based async_process.py's process management on that. :\ | 20:12 |
| markmcclain | marun: no worries.. we can still refactor and merge them later | 20:14 |
| marun | salv-orlando: what's the rule for backporting when an external dependency is added? will it help if the dependency is already required be ceilometer? | 20:14 |
| marun | markmcclain: ^ | 20:14 |
| marun | be -> by | 20:14 |
| markmcclain | adding deps to backports is a lot trickier because the packages would need changing | 20:15 |
| marun | markmcclain: that's what I figured | 20:15 |
| salv-orlando | idk about decencies that confuses me a lot | 20:15 |
| markmcclain | might need to look at what psutil is doing under the hood | 20:15 |
| marun | markmcclain: I think we could do it at redhat easily enough | 20:15 |
| marun | markmcclain: so reimplement what psutil does for the backport? | 20:15 |
| marun | markmcclain: it should be easy enough, just wanted to make sure | 20:16 |
| otherwiseguy | markmcclain: psutil is using built-in syscalls. | 20:16 |
| marun | otherwiseguy: feel like a backport? ^^ | 20:16 |
| markmcclain | ah ok.. I've never dived into that lib | 20:16 |
| otherwiseguy | yeah, it is primarily C code. | 20:16 |
| otherwiseguy | but supporting lots of different OSes. | 20:17 |
| marun | if this fixes bugs in icehouse that are present in havana, we need to backport. i guess we can wait on the gate results | 20:17 |
| marun | markmcclain: given that a bunch of bugs are represented (the ones implementing polling minimization, fixing process management, and polling interval), how best to manage a cohesive backport effort? | 20:18 |
| markmcclain | yeah.. lets make sure this addresses the gate issue | 20:19 |
| markmcclain | also the support release in Havana was experimental and off by default | 20:19 |
| *** aveiga has quit IRC | 20:19 | |
| marun | markmcclain: I think I'd like to backport regardless. | 20:20 |
| otherwiseguy | and the grenade test is failing with a a timeout... | 20:21 |
| markmcclain | yeah I do think a backport should be considered | 20:21 |
| marun | markmcclain: the polling behavior was negatively impacting maximum ovs performance and redhat would like to ship rhos 4.0 (based on havana release) with polling minimization enabled by default | 20:21 |
| markmcclain | yeah… the perf of the old polling was bad | 20:22 |
| markmcclain | let's make sure this addresses the gate issues first | 20:22 |
| markmcclain | and then we can figure out how we get this backported | 20:22 |
| marun | ok | 20:22 |
| marun | markmcclain: to your knowledge, are request ids being returned to clients of the neutron service on every request? | 20:26 |
| openstackgerrit | Claudiu Belu proposed a change to openstack/neutron: Fixes missing method in Hyper-V Utils (Metering) https://review.openstack.org/57521 | 20:30 |
| markmcclain | marun: API or RPC client? | 20:32 |
| marun | api | 20:32 |
| marun | markmcclain: api | 20:34 |
| anteaya | awesome work on those patches, 53188 and 57475 are in the gate one right behind the other | 20:34 |
| openstackgerrit | Sergio Cazzolato proposed a change to openstack/python-neutronclient: Fix i18n messages in neutronclient https://review.openstack.org/57522 | 20:35 |
| anteaya | let's stay close to review salvatore's patch once it is offered | 20:35 |
| marun | markmcclain: ah, it appears not. nmind | 20:36 |
| openstackgerrit | Youcef Laribi proposed a change to openstack/neutron: Implements a driver for the Neutron LBaaS service plugin that allows for using the Citrix NetScaler loadbalancing devices to provide Neutron LBaaS functionality. https://review.openstack.org/57524 | 20:39 |
| *** enikanorov_ has quit IRC | 20:41 | |
| *** enikanorov_ has joined #openstack-neutron | 20:41 | |
| marun | anteaya: salv-orlando's patch needs to be updated to fix a test failure it introduces | 20:41 |
| otherwiseguy | anteaya: thanks. I hope they help. | 20:43 |
| * otherwiseguy crosses fingers | 20:43 | |
| * otherwiseguy sits and stares at the zuul page | 20:44 | |
| *** mvenesio has quit IRC | 20:45 | |
| *** clev has joined #openstack-neutron | 20:45 | |
| anteaya | otherwiseguy: me too, awesome job on the turn around | 20:46 |
| anteaya | marun: okay, yes I think that is the patch submission we are waiting for, yes? | 20:47 |
| marun | yes, i can't read | 20:47 |
| anteaya | np | 20:48 |
| anteaya | :D | 20:48 |
| salv-orlando | patch updated | 20:48 |
| anteaya | woot | 20:48 |
| openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: Enable polling minimization https://review.openstack.org/57290 | 20:48 |
| salv-orlando | well it's still travelling on the wire | 20:48 |
| salv-orlando | landed now | 20:48 |
| anteaya | marun otherwiseguy markmcclain | 20:48 |
| anteaya | whoever else | 20:48 |
| anteaya | sgran mestery | 20:49 |
| mestery | anteaya: pong. Whats up? | 20:49 |
| anteaya | mestery can you look at https://review.openstack.org/#/c/57290/ | 20:49 |
| mestery | looking | 20:49 |
| anteaya | since this has never passed check it has to go through there first | 20:49 |
| anteaya | mestery: awesome thanks | 20:50 |
| marun | it's good, someone just needs to ride it until it gets through | 20:50 |
| *** mlavalle has joined #openstack-neutron | 20:51 | |
| mestery | marun: Your concerns with this one from prior review are taken care of now? | 20:51 |
| marun | mestery: yes, terry's patches have landed | 20:51 |
| mestery | marun: Great! | 20:51 |
| mestery | Will +2 and merge it now. | 20:51 |
| mestery | done | 20:51 |
| *** same5336_ has quit IRC | 20:51 | |
| anteaya | marun: can ride | 20:51 |
| anteaya | it should pass check first | 20:52 |
| anteaya | I can ride it through check and can ping you mestery when it is ready for the gate | 20:52 |
| anteaya | can we do that? | 20:52 |
| *** nplanel has quit IRC | 20:52 | |
| * mestery nods, works for me. | 20:53 | |
| anteaya | thanks | 20:53 |
| anteaya | will ping once it passes check | 20:53 |
| marun | nite all | 20:53 |
| *** marun has quit IRC | 20:53 | |
| anteaya | nite marun | 20:54 |
| openstackgerrit | Youcef Laribi proposed a change to openstack/neutron: Implements a driver for the Neutron LBaaS service plugin that allows for using the Citrix NetScaler loadbalancing devices to provide Neutron LBaaS functionality. https://review.openstack.org/57524 | 20:54 |
| *** jecarey has quit IRC | 20:55 | |
| otherwiseguy | anteaya: gah, 57475 is broken. not sure how it passed the tests I ran locally? https://jenkins02.openstack.org/job/gate-neutron-python27/3183/console | 20:57 |
| anteaya | haruph | 20:57 |
| anteaya | did 57475 ever pass check? | 20:57 |
| anteaya | yeah, that is a problem with the patch: AttributeError: 'module' object has no attribute 'DEFAULT_OVSDBMON_RESPAWN_INTERVAL' | 20:58 |
| openstackgerrit | Youcef Laribi proposed a change to openstack/neutron: Implements a driver for the Neutron LBaaS service plugin that allows for using the Citrix NetScaler loadbalancing devices to provide Neutron LBaaS functionality. https://review.openstack.org/57524 | 20:58 |
| otherwiseguy | doesn't look like it. might have just ran the wrong subset of tests on it. | 20:58 |
| otherwiseguy | (locally) | 20:58 |
| *** armax has left #openstack-neutron | 20:58 | |
| anteaya | okay | 20:58 |
| otherwiseguy | should be easy fix. | 20:58 |
| anteaya | well good work team coming together so quickly | 20:59 |
| anteaya | some take aways | 20:59 |
| anteaya | 1 - please make sure all patches have passed the check tests before approval | 20:59 |
| anteaya | any one else have anything else to offer? | 20:59 |
| anteaya | 2 - it really helps having everyone focused on the same issue at once | 21:00 |
| *** mjbright_ has quit IRC | 21:01 | |
| anteaya | otherwiseguy: do you have another patchset in you? | 21:02 |
| otherwiseguy | yeah, uploading now | 21:04 |
| openstackgerrit | Terry Wilson proposed a change to openstack/neutron: Add configurable ovsdb monitor respawn interval https://review.openstack.org/57475 | 21:04 |
| otherwiseguy | anteaya: but it will fail without the test fix in salv-orlando's patch | 21:05 |
| anteaya | what happens if 57290 merges? | 21:07 |
| anteaya | without 57475? | 21:07 |
| anteaya | salv-orlando otherwiseguy? | 21:08 |
| anteaya | since 57290 doesn't list any dependencies it might merge | 21:08 |
| anteaya | what would happen if that happens? | 21:08 |
| anteaya | should we -2 it? that will prevent it from merging | 21:09 |
| otherwiseguy | 57290 will enable minimize_polling which has a test that flaps. | 21:09 |
| otherwiseguy | without the fix in 57475. Should have just done them all in one patch, apparently. | 21:09 |
| salv-orlando | I'm in a meeting; I don't think there's a need to impose a strict ordering - but I will let you guys make the call. | 21:09 |
| otherwiseguy | becuase we have a circular dependency now. | 21:09 |
| anteaya | if 57290 lands before 57475, can you still merge 57475, once it passes | 21:10 |
| otherwiseguy | even more fun, with the patch I just uploaded, I end up both needing the change in 57290 and needing it to not be there. | 21:11 |
| otherwiseguy | AssertionError: Expected call: get_polling_manager(False, 'sudo') | 21:11 |
| otherwiseguy | Actual call: get_polling_manager(False, 'sudo', 30) | 21:11 |
| anteaya | and no to one big patch, salvatore's should have listed dependencies | 21:11 |
| otherwiseguy | if we call get_polling_manager(False, 'sudo') it requires True and vice-versa... | 21:11 |
| anteaya | so which do you want? | 21:11 |
| salv-orlando | merge them then and we'll abandon the other | 21:11 |
| *** aymenfrikha has quit IRC | 21:11 | |
| anteaya | I am thinking we need to -2 57290 | 21:11 |
| *** enikanorov__ has joined #openstack-neutron | 21:12 | |
| *** enikanorov_ has quit IRC | 21:12 | |
| anteaya | salv-orlando: so -2 57290? | 21:12 |
| anteaya | otherwiseguy: are you core? | 21:12 |
| salv-orlando | done | 21:12 |
| anteaya | thanks | 21:12 |
| otherwiseguy | anteaya: nope, I'm new here. :) | 21:12 |
| anteaya | okay that is fine | 21:12 |
| anteaya | good to have you | 21:13 |
| anteaya | the floor is yours, let's see other patch and have it pass the check queue this time | 21:13 |
| anteaya | and then go from there | 21:13 |
| otherwiseguy | k. I'll keep looking into what is going on with the test_daemon_loop_uses_polling_manager test. | 21:14 |
| anteaya | great | 21:14 |
| anteaya | let me know how I can help | 21:14 |
| anteaya | if you need a hand navigating the -infra tools let me know | 21:14 |
| anteaya | happy to help | 21:14 |
| otherwiseguy | oh, duh. I see the problem. fixing. | 21:15 |
| openstackgerrit | Terry Wilson proposed a change to openstack/neutron: Add configurable ovsdb monitor respawn interval https://review.openstack.org/57475 | 21:16 |
| otherwiseguy | Ok, I've fixed the test in mine. salv-orlando's patch will need to be slightly updated to pass in the polling respawn interval in the test that he fixed. | 21:17 |
| salv-orlando | I'll be back shortly - busy now | 21:18 |
| otherwiseguy | anteaya: so in my version without his making minimize_polling default, we call get_polling_manager(False, 'sudo', constants.DEFAULT_OVSDBMON_RESPAWN) now instead of (False, 'sudo'). In salv-orlando's patch he changed False to True, but will also need to add the constants.DEFAULT_OVSDBMON_RESPAWN arg. | 21:18 |
| anteaya | okay | 21:19 |
| salv-orlando | otherwise guy: tip - you can push a new patch set on my patch | 21:19 |
| otherwiseguy | salv-orlando: will do. | 21:19 |
| *** alagalah_ has joined #openstack-neutron | 21:20 | |
| otherwiseguy | anteaya: can we add a dependency on 57475 to 57290 as well? | 21:20 |
| anteaya | yes we should be able to | 21:21 |
| anteaya | what is the chain you want to do | 21:21 |
| anteaya | and https://review.openstack.org/#/c/56187/ isn't really a gate blocking bug | 21:21 |
| openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: Test commit for testing parallel job in experimental queue https://review.openstack.org/57420 | 21:21 |
| anteaya | why is it in the gate queue when the rest of openstack is waiting to be requeued? | 21:22 |
| otherwiseguy | 53188 -> 54747 -> 57290 | 21:22 |
| anteaya | otherwiseguy: join me in infra | 21:22 |
| *** rkukura has quit IRC | 21:27 | |
| anteaya | markmcclain: can you ensure all neutron-core read this, and if they have any questions they can ask me about it | 21:28 |
| anteaya | Nachi Ueno isn't on irc and shouldn't have approved https://review.openstack.org/#/c/56187/ given our current status | 21:29 |
| openstackgerrit | Terry Wilson proposed a change to openstack/neutron: Enable polling minimization https://review.openstack.org/57290 | 21:32 |
| markmcclain | anteaya: I can make sure hey read it, but I'm not 100% sure I'm following | 21:32 |
| markmcclain | so we have an ordering we need to maintain? | 21:33 |
| *** mlavalle has quit IRC | 21:33 | |
| otherwiseguy | anteaya: ok, so hopefully all tests will work and dependency tree looks right. | 21:33 |
| anteaya | if the gate is in trouble, like it is right now | 21:33 |
| anteaya | have some communication with the channel before approving patches | 21:33 |
| anteaya | right now it is gate blocking bug fix only | 21:34 |
| markmcclain | ah ok | 21:34 |
| anteaya | markmcclain: this is an ununusual situation and has never happened before, but given the growth of openstack will probably happen again | 21:34 |
| anteaya | there are 100+ patches waiting for the gate to be fixed so they can be requeued | 21:35 |
| anteaya | otherwiseguy: great | 21:35 |
| anteaya | otherwiseguy: let's see check tests pass on each | 21:35 |
| markmcclain | yeah.. I thought some stuff has been submitted out of order | 21:35 |
| anteaya | communication with the channel is the most important element | 21:36 |
| anteaya | if you are core and awake you should be at least pingable from irc | 21:36 |
| anteaya | during working hours | 21:36 |
| *** coolsvap has quit IRC | 21:37 | |
| anteaya | otherwiseguy: okay let's watch https://review.openstack.org/#/c/57475/4 | 21:38 |
| anteaya | and see how it does in the gate | 21:38 |
| *** jecarey has joined #openstack-neutron | 21:38 | |
| anteaya | if it passes we can see about 57290 | 21:38 |
| anteaya | fair enough? | 21:38 |
| otherwiseguy | sounds good to me. | 21:39 |
| anteaya | okay I'm going to have some food | 21:40 |
| otherwiseguy | I've spent a good portion of this day staring at zuul already :) | 21:40 |
| anteaya | cool | 21:40 |
| anteaya | nothing like the entertainment of zuul TV | 21:40 |
| anteaya | bbiab | 21:40 |
| anteaya | also can we get anyone to talk to this developer and have him hold off on submitting more patchsets for a bit: https://review.openstack.org/#/c/57524/ | 21:44 |
| anteaya | just until we get the gate addressed | 21:44 |
| markmcclain | I just -2'd | 21:45 |
| *** nati_ueno has joined #openstack-neutron | 21:46 | |
| anteaya | thx | 21:46 |
| *** armax has joined #openstack-neutron | 21:47 | |
| anteaya | nati_ueno: welcome | 21:54 |
| anteaya | we were just chatting about our status | 21:55 |
| anteaya | we are in a bit of a situtation regarding the gate at the moment | 21:55 |
| anteaya | we being all of openstack | 21:55 |
| anteaya | and currently the only thing that should be in check or queue are gate blocking bug fixes, and their dependencies | 21:56 |
| anteaya | should in the idealistic sense of the word | 21:57 |
| dkehn | markmcclain: you aware of anything concerning the setup_physical_bridges where teh udevadm settle is (i believe) timing out? | 21:58 |
| dkehn | for the last couple of days I'm bumping into it running tox -epy27 | 21:59 |
| anteaya | hey dkehn, I just figured out you are a different person from dekehn, yes? | 21:59 |
| anteaya | also we got a shout out on the ml: http://lists.openstack.org/pipermail/openstack-dev/2013-November/019937.html | 22:00 |
| dkehn | anteaya: I think I'm the same | 22:00 |
| anteaya | so thanks to everyone who continues to help making the gate blocking bug fixes a priority | 22:00 |
| dkehn | dkehn is my handle and dekehn is email user | 22:00 |
| dkehn | yes its all confusing | 22:00 |
| anteaya | dkehn: there is someone name derek with a similar handle on launchpad | 22:01 |
| anteaya | your first name is Don isn't it? | 22:01 |
| dkehn | ok, I'm confused derek not cloese to dekehn or dkehn | 22:01 |
| dkehn | but if you type two chars its a grab bag | 22:02 |
| anteaya | no, his first name was dereck his nick was close to dekehn | 22:02 |
| dkehn | salv-orlando: ^^^^ on the udevadm | 22:02 |
| dkehn | ok | 22:02 |
| dkehn | oh well | 22:02 |
| dkehn | its a common name, not | 22:02 |
| anteaya | yeah, if I see it again I will remember better | 22:02 |
| dkehn | anteaya: check HP email | 22:04 |
| *** Jianyong has quit IRC | 22:04 | |
| anteaya | k | 22:05 |
| *** Jianyong has joined #openstack-neutron | 22:06 | |
| *** mlavalle has joined #openstack-neutron | 22:07 | |
| anteaya | dkehn: awesome thanks | 22:08 |
| *** arahal has quit IRC | 22:08 | |
| dkehn | anteaya: dhopefully that clears it up | 22:08 |
| dkehn | or confuses it more | 22:08 |
| anteaya | yes to both | 22:09 |
| anteaya | :D | 22:09 |
| anteaya | late in my day | 22:09 |
| anteaya | punchy | 22:09 |
| *** rudrarugge has joined #openstack-neutron | 22:12 | |
| *** rudrarugge has quit IRC | 22:14 | |
| *** ekarlso has quit IRC | 22:15 | |
| otherwiseguy | anteaya: looks like https://review.openstack.org/#/c/57475/ passed the check. | 22:17 |
| *** ekarlso has joined #openstack-neutron | 22:17 | |
| anteaya | w00t | 22:19 |
| anteaya | collect reviews and get ready for the gate | 22:19 |
| anteaya | ping folks | 22:19 |
| otherwiseguy | markmcclain: salv-orlando: https://review.openstack.org/#/c/57475/ needs re-approval when you get a chance. this time it even passes the initial check. :p | 22:20 |
| markmcclain | otherwiseguy: looking | 22:20 |
| dkehn | markmcclain: ^^^ any thoughts, I feel like I'm forgetting something | 22:21 |
| jog0 | FYI http://lists.openstack.org/pipermail/openstack-dev/2013-November/019941.html | 22:23 |
| mestery | otherwiseguy: Nice work on 57475. | 22:23 |
| mestery | markmcclain: I can approve 57475 let me know | 22:23 |
| markmcclain | mestery: go ahead | 22:23 |
| mestery | markmcclain: done | 22:23 |
| anteaya | awesome | 22:23 |
| anteaya | great work team | 22:23 |
| anteaya | 57290 is almost done in the check queue | 22:24 |
| otherwiseguy | mestery: thanks! | 22:24 |
| otherwiseguy | or, double thanks, actually. | 22:24 |
| *** clev has quit IRC | 22:25 | |
| *** steven-weston has quit IRC | 22:28 | |
| otherwiseguy | https://review.openstack.org/57290 just passed check too. go, go, go. :) | 22:28 |
| anteaya | go you! | 22:28 |
| anteaya | gather them up again otherwiseguy | 22:29 |
| *** steven-weston has joined #openstack-neutron | 22:29 | |
| otherwiseguy | mestery: markmcclain: https://review.openstack.org/#/c/57290/ is now ready for re-approval. | 22:29 |
| mestery | otherwiseguy: Looking | 22:29 |
| otherwiseguy | thanks everybody. this has been quite a day. | 22:30 |
| otherwiseguy | it it always like this?! :) | 22:30 |
| otherwiseguy | s/^it/is/ | 22:30 |
| mestery | 57290 needs salv-orlando to remove his -2 | 22:30 |
| anteaya | otherwiseguy: not always like this no | 22:30 |
| anteaya | some days are busier | 22:31 |
| anteaya | :D | 22:31 |
| steven-weston | salv-orlando: do you have a few minutes to look over my etherpad for auto ip association? | 22:31 |
| otherwiseguy | anteaya: oh good. | 22:31 |
| anteaya | ha ha ha | 22:32 |
| otherwiseguy | I'd hate to get bored. Or have a regular heartbeat. | 22:32 |
| anteaya | not a chance | 22:32 |
| mestery | otherwiseguy: :D | 22:32 |
| anteaya | steven-weston: including a url is always helpful | 22:32 |
| steven-weston | anteaya: https://etherpad.openstack.org/p/NeutronFloatingIPAutoAssociation | 22:32 |
| steven-weston | anteaya: your input is welcome as well! | 22:33 |
| anteaya | steven-weston: thanks | 22:33 |
| anteaya | would you like to add a link to your etherpad from https://etherpad.openstack.org/p/NeutronIcehouseProjectPlan | 22:34 |
| steven-weston | or anyone else, for that matter :) | 22:34 |
| anteaya | then it helps to see where it fits in with the whole picture | 22:34 |
| steven-weston | anteaya: Yes, that would be wonderful | 22:34 |
| anteaya | awesome, thanks | 22:34 |
| anteaya | salv-orlando: we need you to reassess your -2 on https://review.openstack.org/#/c/57290/ please | 22:35 |
| anteaya | otherwiseguy: you can vote on 57290 | 22:35 |
| anteaya | sgran: if you are around | 22:35 |
| anteaya | nati_ueno: you are core, care to review https://review.openstack.org/#/c/57290/ | 22:36 |
| otherwiseguy | anteaya: ok. since I uploaded the last patch on that one didn't know if my vote was useful there. :) | 22:36 |
| mestery | anteaya: FYI, 57290 has 2 core +2s, just needs salve-orlando to remove his -2 and we can merge it. | 22:36 |
| markmcclain | guessing it's dinner time for him | 22:37 |
| otherwiseguy | I would have thought that uploading patch set 3 would have removed the -2. | 22:38 |
| markmcclain | no those are sticky | 22:38 |
| otherwiseguy | interesting. | 22:38 |
| steven-weston | markmcclain: should I respond to you in the etherpad, or here in irc? | 22:38 |
| markmcclain | and require the person who placed to remove it | 22:38 |
| *** changbl has quit IRC | 22:38 | |
| markmcclain | I'd respond the etherpad | 22:38 |
| markmcclain | it will make the bp more clear | 22:38 |
| steven-weston | markmcclain: okay, thanks :-) | 22:39 |
| anteaya | mestery: okay | 22:39 |
| anteaya | otherwiseguy: and yes you are not the author of that patch so offering a review for it is fine | 22:40 |
| anteaya | reviewing your own patch is frowned upon | 22:40 |
| anteaya | approving your own patch, unless you are the only core, could get you publicly shamed | 22:40 |
| otherwiseguy | note to self: do lots of reviews, be useful, and if you make core don't approve your own patch and use -2 sparingly. :p | 22:41 |
| anteaya | just sent this to the ml: http://lists.openstack.org/pipermail/openstack-dev/2013-November/019949.html | 22:44 |
| anteaya | good work, Neutron | 22:44 |
| anteaya | otherwiseguy: you are catching on nicely | 22:45 |
| *** nplanel has joined #openstack-neutron | 22:45 | |
| otherwiseguy | anteaya: thanks. i've been stuck on packaging duty for a while now. it's good to get out and get back in to actual development. | 22:46 |
| anteaya | s00t | 22:47 |
| *** enikanorov_ has joined #openstack-neutron | 22:47 | |
| anteaya | w00t even | 22:47 |
| anteaya | s00t too | 22:47 |
| *** enikanorov__ has quit IRC | 22:47 | |
| anteaya | what were you packaging? | 22:48 |
| otherwiseguy | so if you are going through a review and you see some simple issues like pep8 test failures, etc., is it kosher to just go in and fix them and re-submit the patch, or is it better to make comments and wait for the original dev? | 22:48 |
| *** clev has joined #openstack-neutron | 22:48 | |
| anteaya | make comments | 22:48 |
| otherwiseguy | openstack-neutron package for RH, also working a bit on the packstack tool. | 22:48 |
| mestery | otherwiseguy: Comment and let the submitter fix them, unless you work with the submitter and get approval. | 22:48 |
| anteaya | your first role as developer is mentor the other developers | 22:49 |
| markmcclain | steven-weston: I've got to head out for a bit. I'll look at responses later on | 22:49 |
| steven-weston | markmcclain: Okay, thanks for your help!!! | 22:49 |
| anteaya | grow as a contributor by asking questions and sharing what you know, as accurately as you can | 22:49 |
| anteaya | encourage new developers to do the same | 22:50 |
| otherwiseguy | ok. just checking as salv-orlando had modify his patch. didn't know if that was the standard procedure to do automatically if you saw an issue, or not. thanks for the input. :) | 22:50 |
| anteaya | always give someone a path for improvement and the option to take it if they wish | 22:50 |
| otherwiseguy | makes sense. | 22:50 |
| anteaya | *nod | 22:50 |
| anteaya | some circumstances are unusual | 22:51 |
| *** pcm_ has quit IRC | 22:51 | |
| anteaya | having a broken gate for all of openstack is unusual and hopefully will be a very rare event | 22:51 |
| steven-weston | anteaya: that's good advice. I am always looking for paths for improvement. | 22:51 |
| anteaya | so in the case of rare events, gather some folks, 5 or more and have a discussion | 22:51 |
| anteaya | steven-weston: awesome, glad to hear it | 22:52 |
| anteaya | me too, I make a lot of mistakes | 22:52 |
| anteaya | but I meet so many good people along the way as I learn to fix them | 22:52 |
| steven-weston | hehe. I think everybody does, there is just a bit of variability as to how many people admit it :-) | 22:52 |
| *** clev has quit IRC | 22:52 | |
| anteaya | steven-weston: very true | 22:52 |
| openstackgerrit | A change was merged to openstack/neutron: Ensure get_pid_to_kill works with rootwrap script https://review.openstack.org/53188 | 22:54 |
| anteaya | w00t | 22:55 |
| anteaya | a merge | 22:55 |
| anteaya | we got a merge | 22:55 |
| anteaya | \o/ | 22:55 |
| mestery | yay! | 22:55 |
| otherwiseguy | anteaya: yay! I've been waiting for that to merge successfully for weeks. :p | 22:55 |
| anteaya | ha ha ha | 22:55 |
| anteaya | today was your day apparently | 22:55 |
| anteaya | working on getting some things in place to reduce the time it takes for patches to get merged | 22:56 |
| anteaya | I think discussing the patches in channel really helps that | 22:56 |
| otherwiseguy | apparently. first commit finally through the gate (that wasn't just a one-line addition to setup.py) | 22:56 |
| otherwiseguy | I agree. This has been a good learning experience. | 22:57 |
| steven-weston | speaking of improvement, do you have or know of any good references regarding solid oo design principles? Also, more specifically, how do I find out the nitty grittys on the data model and implementation in Neutron? | 22:57 |
| *** roampune has joined #openstack-neutron | 22:58 | |
| anteaya | steven-weston: good questions | 22:58 |
| *** peristeri has quit IRC | 22:58 | |
| *** steven-weston_ has joined #openstack-neutron | 22:59 | |
| steven-weston | anteaya: Yay! I do enjoy self edumecation. | 22:59 |
| *** idella4 has joined #openstack-neutron | 23:00 | |
| otherwiseguy | dictionaries. dictionaries everywhere. :p | 23:00 |
| * otherwiseguy ducks | 23:00 | |
| anteaya | solid oo design principles - I don't hav any first hand recommendations myself, perhaps some other folks will when they read the channel log | 23:03 |
| anteaya | if you don't get a good answer to that one, please ask again | 23:03 |
| idella4 | g'day | 23:03 |
| anteaya | the Neutron data model, have you spent much time with the neutron repo? http://git.openstack.org/cgit/openstack/neutron/tree/neutron/db | 23:04 |
| *** same5336 has joined #openstack-neutron | 23:04 | |
| anteaya | or is understanding that code what you are asking about? | 23:04 |
| *** networkstatic has joined #openstack-neutron | 23:04 | |
| anteaya | morning idella4 | 23:05 |
| steven-weston | okay, excellent. I do have some books, I'm just wondering how good they are ;-) | 23:05 |
| idella4 | yep it is! how are you? | 23:05 |
| jog0 | FYI http://lists.openstack.org/pipermail/openstack-dev/2013-November/019941.html | 23:05 |
| jog0 | please don't approve non critical bug fixes until we get gate in order | 23:06 |
| jog0 | anteaya: ^ | 23:06 |
| anteaya | yes doing that | 23:07 |
| anteaya | jog0: this one was approved before we communicated to core: https://review.openstack.org/#/c/56187/ | 23:07 |
| anteaya | nothing else since then | 23:08 |
| anteaya | and thanks | 23:08 |
| jog0 | anteaya: thanks! | 23:08 |
| anteaya | thank you | 23:08 |
| anteaya | steven-weston: ah okay, no idea | 23:08 |
| anteaya | idella4: good thanks, we had some good discussions today | 23:08 |
| anteaya | all sitting around the campfire now waiting for some patches to merge | 23:09 |
| anteaya | the gate is shut down until the bugs are fixed | 23:09 |
| idella4 | oh dear, neutron still under seige | 23:09 |
| idella4 | whoa | 23:09 |
| anteaya | no, not just us | 23:09 |
| anteaya | all of openstack | 23:09 |
| *** carl_baldwin has quit IRC | 23:09 | |
| idella4 | ah it is | 23:09 |
| anteaya | a few bad habits given our size leads to an unstable gate pretty fast | 23:09 |
| anteaya | the size of openstack | 23:10 |
| steven-weston | anteaya: yes, understanding that code is what I was talking about. really would like to learn about the design decisions that went into the model and how to make them myself, as well as contribute. | 23:10 |
| idella4 | makin for very busy littl beavers, or is that platypusses | 23:10 |
| anteaya | steven-weston: understood, I don't know the history of it myself | 23:10 |
| otherwiseguy | huh, looks like the gate jobs just all restarted the tests? | 23:11 |
| anteaya | markmcclain might be able to shed more light when he is around, not sure who else would know | 23:11 |
| anteaya | it happens | 23:11 |
| anteaya | the gate resets | 23:11 |
| anteaya | since every patch after the first one is tested including the ones before | 23:11 |
| * mestery steps out for a while tonight, family calls. | 23:11 | |
| anteaya | if the one in front fails, the gate resets, the one if front is removed and round we go again | 23:12 |
| anteaya | mestery: have a good night, thankss for a great day | 23:12 |
| otherwiseguy | anteaya: ahhh. | 23:12 |
| salv-orlando | I've removed the -2 from my patch. for bug 1224001. It should go through the gate now - if I have not missed anything it should be clear to go, right otherwiseguy? | 23:12 |
| otherwiseguy | salv-orlando: yep! thanks! | 23:12 |
| salv-orlando | ok cool | 23:12 |
| steven-weston | excellent! Thanks for the help! I have to step out as well, but I should be back in a few hours. | 23:13 |
| mestery | same here anteaya! | 23:13 |
| anteaya | okay thanks see you soon steven-weston | 23:13 |
| anteaya | :D | 23:13 |
| steven-weston | :D | 23:13 |
| anteaya | salv-orlando: and another time you would ask someone else to approve your patch, right? | 23:13 |
| anteaya | just teaching the new folk good habits | 23:14 |
| anteaya | we talked about not approving ones own patches | 23:14 |
| otherwiseguy | his was appreoved by others, he just un -2'd it. | 23:14 |
| otherwiseguy | and approved | 23:14 |
| otherwiseguy | :p | 23:14 |
| anteaya | yeah | 23:14 |
| salv-orlando | generally speaking if you want to know good habits - look at me | 23:14 |
| anteaya | :D | 23:14 |
| salv-orlando | and then do the opposite | 23:14 |
| anteaya | ha ha ha | 23:14 |
| otherwiseguy | I'm positive I have other things to do, and yet I can't seem to stop watching zuul. | 23:15 |
| anteaya | it is addictive | 23:16 |
| idella4 | zyyl! | 23:16 |
| idella4 | zuul? | 23:16 |
| otherwiseguy | I bought a 24u dell server enclosure, 3 1u servers, a managed switch, and wired my house for cat6. I really should set those servers up. :p | 23:16 |
| anteaya | idella4: http://status.openstack.org/zuul/ | 23:16 |
| anteaya | ha ha ha | 23:17 |
| *** changbl has joined #openstack-neutron | 23:17 | |
| otherwiseguy | I figured if I was going to work on neutron, I should have a lab. :p | 23:17 |
| anteaya | good call | 23:17 |
| otherwiseguy | And the fact that google fiber was installed might have had something to do with the purchases as well. :p | 23:17 |
| anteaya | I bet | 23:18 |
| idella4 | huh, I was expecting something like "the blacklist" latest episode, and I got a "testing or operation of the OpenStack project" | 23:19 |
| idella4 | now that's devotion! | 23:19 |
| anteaya | ha ha ha | 23:21 |
| *** armax has quit IRC | 23:21 | |
| anteaya | okay we still need to hear from arosen on this: https://bugs.launchpad.net/nova/+bug/1249065 | 23:21 |
| anteaya | if we haven't heard from him by tomorrow, can someone else start working on this | 23:21 |
| anteaya | this bug is a gate blocker | 23:22 |
| anteaya | if you want to start and add to the bug report, so much the better | 23:22 |
| anteaya | this is marun's, so hopefully tomorrow he has an update, if not a patch to look at: https://bugs.launchpad.net/neutron/+bug/1251448 | 23:23 |
| anteaya | since he is in taipai if someone else can nudge him about it, that would be great | 23:23 |
| *** jecarey has quit IRC | 23:23 | |
| otherwiseguy | anteaya: I'll take a look at it and see if I can reproduce it just for fun. Being relatively new, can't promise anything and other people should definitely look at it in parallel. | 23:24 |
| anteaya | fair enough | 23:24 |
| anteaya | feel free to update the bug report with any findings | 23:24 |
| anteaya | and thank you | 23:24 |
| *** clev has joined #openstack-neutron | 23:24 | |
| salv-orlando | otherwiseguy: I hope your employer pays for your electricity bill | 23:25 |
| otherwiseguy | salv-orlando: yeah...not so much. I hope keeping the door closed to the spare bedroom keeps out at least some of the noise of 3 1u dell servers. :p | 23:26 |
| otherwiseguy | If mom comes up for an overnight visit, she might want to bring some earplugs. :p | 23:26 |
| *** mlavalle has quit IRC | 23:28 | |
| *** thedodd has quit IRC | 23:34 | |
| *** mlavalle has joined #openstack-neutron | 23:35 | |
| *** Jianyong has quit IRC | 23:39 | |
| dkehn | salv-orlando: u around? | 23:39 |
| *** banix has quit IRC | 23:45 | |
| *** mlavalle has quit IRC | 23:49 | |
| *** carl_baldwin has joined #openstack-neutron | 23:50 | |
| salv-orlando | dkehn: not for long - going to bed | 23:50 |
| salv-orlando | I shall be around for about 15 minutes | 23:51 |
| dkehn | salv-orlando: I'm having an issue with the neutron/plugins/openvswitch/agent/ovs_neutron_agent.py", line 801, in setup_physical_bridges, and the udevadm settle, I'm assuming failure | 23:51 |
| *** rcurran_ has quit IRC | 23:51 | |
| * salv-orlando going to the code | 23:51 | |
| dkehn | salv-orlando: when running the tox -epy27 on afreash clone | 23:52 |
| dkehn | salv-orlando: its failing in the same place thoughout neutron.tests.unit.openvswitch.test_ovs_tunnel.TunnelTestWithMTU | 23:52 |
| dkehn | salv-orlando: tried lengthing the timeout to no change | 23:53 |
| salv-orlando | ah I see - you're running only openvswitch tests or the whole suite? | 23:53 |
| dkehn | I'm assuming the whole thing | 23:53 |
| dkehn | tox -epy27 pretty much runs it all | 23:53 |
| dkehn | even if I narrow it down it erros on this \ | 23:54 |
| salv-orlando | ok because I saw that today running only ovs tests and I assumed it was some stub out not done properly; was going to look into tomorrow. | 23:54 |
| salv-orlando | anyway - if it fails on your machine but not the gate, is rather weird | 23:54 |
| dkehn | I'm assming it doesn't fail in the gate or everyone would be all over it | 23:55 |
| *** julim has quit IRC | 23:55 | |
| salv-orlando | in that case the patch should not have merged in the first place, as we're talking about unit tests. | 23:55 |
| dkehn | that's why I'm assuming its something in my system | 23:55 |
| salv-orlando | let me quickly check what's mocked in the unit tests | 23:56 |
| salv-orlando | the thing is that in unit test udevadm should be mocked; it should not execute at all, imho | 23:56 |
| dkehn | its mocking the tunnnels | 23:56 |
| lifeless | we run udevadm from neutron?! | 23:56 |
| dkehn | lifeless: yes, neutron/plugins/openvswitch/agent/ovs_neutron_agent.py", line 801, in setup_physical_bridges | 23:57 |
| salv-orlando | lifeless: I discovered it like 5 minutes ago | 23:57 |
| dkehn | hmm, brought it up at the conf | 23:57 |
| salv-orlando | this is the related bug fiy: https://bugs.launchpad.net/neutron/+bug/1218556 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!