Wednesday, 2016-08-24

SridarKyushiro: hi no worries00:32
yushiroSridarK: Thanks.00:32
SridarKyushiro: thx for the review of the plugin00:32
SridarKi am doing some integration testing - with the agent - i think there are some things that need fixing there00:33
yushiroSridarK: no warries!00:34
yushiroSridarK: great!  I'm reviewing mfranc213 patch and updating CLI patch.  If you ask me to something, please let me know :)00:36
yushiros/to/to do/00:37
SridarKyushiro: ok great - let me leave some comments in gerrit00:37
SridarKso u dont hit the same issues if u pull things in devstack00:37
SridarKyamamoto: thx for ur patience - i hope things are resolved for u. And thx for pushing the patches out00:39
yushiroSridarK: I built devstack with master(yesterday) repositories and succeeded.  I don't check with agents but I'll try it now.00:43
yamamotoSridarK: the issues settled for us now.  thank you!00:44
SridarKyushiro: just hang on for a few mins - i am debugging this and will let u know so u dont hit the same issue, we have one issue where we pick up the the v1 agent00:44
yushiroSridarK: Sure :)00:45
SridarKyamamoto: no many thx for helping to take care of things00:45
SridarKyushiro: i specified some of the issues i saw - mfranc213 is on PTO - i will ask if chandanc can push a new rev. Since we have been doing the end to end testing - hopefully with this no other issues will crop up01:05
*** vishwanathj has quit IRC01:11
yushiroSridarK: Thanks for your information. I check it.01:22
SridarKyushiro: ok thx01:29
SridarKyushiro: i am heading back home from the office - will get online b4 our mtg01:30
yushiroSridarK: Sure.01:31
*** SridarK has quit IRC01:47
*** mickeys has quit IRC02:17
*** chandanc_ has joined #openstack-fwaas02:55
njohnstonchandanc_: Are you working on SridarK's comments to https://review.openstack.org/#/c/337699/23/neutron_fwaas/services/firewall/agents/l3reference/firewall_l3_agent_v2.py@12603:11
*** vishwanathj has joined #openstack-fwaas03:11
chandanc_yes, just verifying the comments03:11
chandanc_the rpc part is done, checking the dict problem now03:12
njohnstonchandanc_: Excellent, thank you!03:13
chandanc_nate, sure happy to be of help :)03:13
*** vishwanathj is now known as vishwanathj_zzz03:19
njohnstonbiab03:23
*** yushiro has quit IRC03:38
njohnstonback03:48
*** SarathMekala has joined #openstack-fwaas03:56
*** yushiro has joined #openstack-fwaas03:57
*** padkrish has joined #openstack-fwaas04:00
*** _SarathMekala_ has joined #openstack-fwaas04:07
*** _SarathMekala_ has quit IRC04:07
*** _SarathMekala_ has joined #openstack-fwaas04:07
*** _SarathMekala_ has quit IRC04:08
*** _SarathMekala_ has joined #openstack-fwaas04:08
*** SarathMekala has quit IRC04:10
*** yamamoto has quit IRC04:39
*** yamamoto has joined #openstack-fwaas04:39
*** yamamoto has quit IRC04:41
*** yamamoto has joined #openstack-fwaas04:42
*** yamamoto has quit IRC04:42
*** mickeys has joined #openstack-fwaas04:51
*** mickeys has quit IRC04:57
yushiroI'm home.05:02
njohnstonchandanc_: We never got a chance to talk about https://review.openstack.org/333338  "Make ip_conntrack.IpConntrackManager a singleton"05:02
njohnstonchandanc_: How is that going?05:02
chandanc_Hello njohnston05:02
chandanc_yes, just fixing up the l3 comments from sridark05:02
yushiroping padkrish05:03
chandanc_njohnston, do you have anh question on conntrack patch05:03
padkrishyushiro# yes, i skimmed through https://review.openstack.org/#/c/323971/605:03
njohnstonchandanc_: No, I just want to see how far away it is from being ready.  Do you want me to take a look at the test failures in the morning?05:04
chandanc_sure, that will help a lot05:04
njohnstonmust save maximum time for slow neutron reviews05:04
chandanc_yes agree05:04
njohnstonOK, I will devoute time to that in the morning.05:04
njohnstonchandanc_: Please feel free to tap me to help, if help can be used.05:04
padkrishyushiro# handle_port needs more details like subnets etc from the plugin, right?05:05
chandanc_njohnston, sure will do05:05
chandanc_njohnston, i will be pushing a update on the l3 PS today05:06
*** SridarK has joined #openstack-fwaas05:06
njohnstonchandanc_: I look forward to testing it out05:06
chandanc_:) thanks05:06
SridarKnjohnston: quick q b4 u head to bed05:06
njohnstonyes SridarK>05:07
njohnston?05:07
SridarKon the gate check for tempest05:07
SridarKshall we wait until we get the L3 Agent ext patch in05:07
SridarKsorry moving from non-voting05:07
njohnstonSridarK: you mean https://review.openstack.org/35932005:07
njohnstonNo, it will still be non-voting05:08
SridarKyes05:08
SridarKoh ok05:08
njohnstonCurrently it's in the check-experimental queue05:08
njohnstonthis would move it to the check queue05:08
njohnstonso we would always see it's results05:08
njohnstonbut it would not affect gating if it misfired05:08
SridarKi was concerned as we will likely hit some speed bumps as we move to the new framework05:08
njohnstonindeed, thats partially why I want to give this visibility... but not handicap us in the process05:09
SridarKnjohnston: we can discuss more on the coexistence of v1, v2 etc tomorrow am05:09
njohnstonsounds good, ping me when you are ready05:09
SridarKok i just put a comment and no -105:10
SridarKwe can sync tomorrow and i will also do a +105:10
njohnstonok05:10
njohnstonthats good because I worry Ihar might think the comment "I would like to wait to open this up" is like a -105:10
SridarKnjohnston: ok i am hoping my preface will be clear :-)05:11
njohnston:-)05:11
njohnstonand on that note, I can not keep my eyes open any longer... talk to you all tomorrow05:11
SridarKno issues - i did not want to bug given the time - so i just put in a cautionary note in case it merged05:12
SridarK:-)05:12
SridarKGN05:12
SridarKchandanc_: hi05:12
chandanc_Hello SridarK05:12
SridarKchandanc_: pls let me know if u had any clarifications on the comments05:12
chandanc_SridarK, I found some more small issues05:13
SridarKchandanc_: ok05:13
chandanc_I could verify the create workflow till the iptable rule generation05:13
chandanc_now fixing the delete issues05:14
SridarKchandanc_: ok05:14
chandanc_should we have a short call on the l2 driver05:14
chandanc_or is it too late for you ?05:14
SridarKchandanc_: no we can do that05:14
chandanc_should i login to the same webex link ?05:15
SridarKchandanc_: 2 mins05:16
*** yamamoto has joined #openstack-fwaas05:39
*** mickeys has joined #openstack-fwaas05:53
*** mickeys has quit IRC05:58
*** chandanc_ has quit IRC06:23
*** padkrish has quit IRC06:26
yushiroping SridarK07:25
*** _SarathMekala_ has quit IRC08:09
*** _SarathMekala_ has joined #openstack-fwaas08:29
*** mickeys has joined #openstack-fwaas08:40
*** mickeys has quit IRC08:43
*** _SarathMekala_ is now known as SarathMekala09:25
*** yushiro is now known as yushiro_dinner09:58
*** chandanc_ has joined #openstack-fwaas10:43
*** SarathMekala is now known as SarathMekala_Off11:02
*** SarathMekala_Off has quit IRC11:02
*** chandanc_ has quit IRC11:04
*** xdcc has joined #openstack-fwaas12:32
*** yamamoto has quit IRC12:50
*** vishwanathj_zzz is now known as vishwanthj13:03
*** yamamoto has joined #openstack-fwaas13:32
*** yamamoto has quit IRC13:40
*** yamamoto has joined #openstack-fwaas13:52
*** yamamoto has quit IRC14:20
*** chandanc_ has joined #openstack-fwaas14:34
*** yamamoto has joined #openstack-fwaas14:43
*** chandanc_ has quit IRC14:52
*** chandanc_ has joined #openstack-fwaas14:52
SridarKchandanc_: hi15:08
chandanc_Hello SridarK15:09
SridarKchandanc_: i think we ran into some UT failures15:09
chandanc_sorry I am little out of context15:09
chandanc_you mean the l3 patch ?15:09
SridarKchandanc_: sorry on the L3 Agent15:09
SridarKyes15:10
chandanc_what change is causing the issue ?15:10
SridarKi am also juggling another mtg - shd be done soon - going thru the patch15:10
chandanc_ok15:11
*** yushiro_dinner has quit IRC15:12
SridarKchandanc_: it seems like the cfg opt has some issues15:43
SridarKwith the UT failures15:43
chandanc_sorry was away15:52
chandanc_I think as i have moved the opt registration to the cmd file UT is failing15:53
SridarKchandanc_: no prob15:55
*** vishwanthj has quit IRC15:56
SridarKyes it seems so15:56
chandanc_SridarK, please leave a comment if you feel this should be reverted15:56
SridarKchandanc_: no i think what u have is good15:57
chandanc_ok15:57
SridarKchandanc_: just trying to see if register of the opts needs a tweak15:57
SridarKchandanc_: if u are busy on iptables stuff - pls go ahead - let me pull the PS and rerun UT15:58
chandanc_ok, i could not find any better place to register the opts15:58
chandanc_sure please leave a comment, i will get back15:58
chandanc_SridarK, just wanted to know how do i resolve the merge conflict showing up in gerrit for this patch15:59
SridarKchandanc_: ok let me debug15:59
SridarKif u update ur repo and then cherry-pick the changeset16:01
SridarKit will show u the conflict, which u will need to resolve16:01
SridarKbasically the cherry-pick will tell u the steps16:02
chandanc_got it, will do that . Thanks for the help16:02
SridarKno prob at all16:02
*** yamamoto has quit IRC16:37
*** mickeys has joined #openstack-fwaas16:45
*** vishwanathj has joined #openstack-fwaas16:52
*** chandanc__ has joined #openstack-fwaas17:02
*** chandanc_ has quit IRC17:04
*** xdcc has quit IRC17:04
*** chandanc_ has joined #openstack-fwaas17:13
*** chandanc__ has quit IRC17:14
*** chandanc_ has quit IRC17:30
*** chandanc_ has joined #openstack-fwaas17:30
*** xdcc has joined #openstack-fwaas17:34
*** yamamoto has joined #openstack-fwaas17:38
*** yamamoto has quit IRC17:44
*** chandanc_ has quit IRC18:01
*** mickeys has quit IRC18:20
njohnstonSridarK: I was wondering if you could help me with something.  It looks like the current code gets a "router_id" when looking for router information.  But I thought that it got a port id.  Which is correct?18:21
njohnstonSridarK: If it is port id, can you show me where that lookup happens, similarly to the lookup using router id that occurs here: https://github.com/openstack/neutron-fwaas/blob/master/neutron_fwaas/services/firewall/agents/l3reference/firewall_l3_agent.py#L10718:21
SridarKnjohnston: ok18:21
SridarKmfranc213: had changes to translate the port-id18:22
SridarKto reflect the owner router18:22
SridarKto get to the ns18:22
njohnstonok, I'll check her patches then18:22
SridarKin this case with v1 it is the router-id18:23
SridarKhttps://review.openstack.org/#/c/337699/24/neutron_fwaas/services/firewall/agents/l3reference/firewall_l3_agent_v2.py L#13918:24
*** vishwanathj has quit IRC18:25
*** vishwanathj has joined #openstack-fwaas18:26
*** xdcc has quit IRC18:57
*** mickeys has joined #openstack-fwaas19:21
*** mickeys has quit IRC19:25
*** mickeys has joined #openstack-fwaas19:59
*** vishwanathj has quit IRC22:02
*** mickeys has quit IRC22:11
*** SridarK has quit IRC22:31
*** yamamoto has joined #openstack-fwaas22:48
*** zigo has quit IRC22:56
*** zigo has joined #openstack-fwaas22:57
*** mickeys has joined #openstack-fwaas23:55
*** yushiro has joined #openstack-fwaas23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!