18:31:08 #startmeeting Networking FWaaS 18:31:09 Meeting started Wed May 4 18:31:08 2016 UTC and is due to finish in 60 minutes. The chair is SridarK. Information about MeetBot at http://wiki.debian.org/MeetBot. 18:31:10 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 18:31:12 The meeting name has been set to 'networking_fwaas' 18:31:28 #chair xgerman 18:31:30 Current chairs: SridarK xgerman 18:31:44 Hi 18:32:15 Hi 18:32:16 #info According to the ical file from eavesdrop, we are actually in an even week, so today's meeting should be at 8pm, FYI. 18:32:33 I suggest we make normalizing the meeting schedule a topic for the meeting. :-) 18:32:39 Hi 18:32:47 njohnston: sigh ok - since we are in an odd week 18:32:49 my bad 18:32:50 njohnston +1 18:33:00 hence i sent the email out 18:33:12 I didn't look at my calendar until the last second. 18:33:19 yes, that was very helpful for me Sridar. 18:33:21 njohnston: ok lets do that as a topic 18:33:34 njohnston: apologies 18:33:49 No, I think it's good, I would be at a family function at 8pm :-) 18:34:02 serendipity :-) 18:34:26 #topic Summit, v2 priorities 18:34:55 Firstly for the record - thanks all for the great discussion on Fri 18:35:00 +1 18:35:04 +1 18:35:15 +1 18:35:22 +1 18:35:39 we can continue to use the etherpad 18:35:44 #link https://etherpad.openstack.org/p/neutron-fwaas-austin 18:36:42 Are there any blocking issues that we need to discuss and work to resolution 18:37:03 There is the issue I emailed out, the issue with oslo.service 18:37:09 i spent some time with chandanc and Sarath o the gerrit workflow 18:37:28 yes, i looked at the oslo.service thing but not long enough to figure it out unfortunately. 18:37:40 Thanks for the help with gerrit 18:38:00 yes clearly there is an issue - i was able to see that as well - we can try to do some digging in parallel to see what is going on 18:38:39 And if I hack in a fix for that, by adding an argument, then I get to a further issue where it looks like a multiple inheritance issue supplying parameters to an __init__() method 18:38:52 fwiw, the symptomology is simliar to this: http://freshfoo.com/blog/object__init__takes_no_parameters 18:39:45 and actually it appears that the initial bug, the one that nate put a hack into fix, is the same problem as what appears after the hack :) 18:39:53 that's how it appears to me, anyway. 18:40:16 this issue is deep in the weeds, as far as i can tell. an interesting problem. 18:40:16 Here's what I am seeing on the __init__ issue: http://paste.openstack.org/show/496130/ 18:40:50 (what nate has just posted is what is seen after the hack is put in place) 18:41:17 Yes, the first line tries to explain that 18:41:36 ahh, yes, sorry nate 18:41:41 I'm not sure who to reach out to in order to see if we're using oslo.service incorrectly. Perhaps dims? 18:42:04 yes i am puzzled - i will dig more too 18:42:13 josh harlow - he is the oslo PTL 18:42:39 harlowja 18:42:44 Excellent. I will put together a better presentation of the issue and contact him. Thanks xgerman! 18:42:58 yeah, tell him I send you :-) 18:43:03 :-) 18:43:06 :-) 18:43:18 one thing i wanted to do was to see how other callers handle that __init__ in oslo_service. will try to get to that today. think that reaching out to others is good, too. 18:43:26 ok lets continue on this 18:43:42 mfranc213: my thought too - i am sure this is a common pattern 18:43:50 yeah, should be 18:44:13 Other than that I have gotten a start on accounting for all the db objects in the db patch, and trying to make sure that (other than the aforementioned problem) it passes tox. 18:45:06 sorry, guys gotta run today early… should be better next week when I am back home... 18:45:07 other than some methods related to rules, u will need to add for the other resources 18:45:14 xgerman: no worries 18:45:29 Sure, feel free to leave feedback on the PS 18:46:09 that's probably the best way to keep me honest on it :-) 18:46:10 there are orm questions that nate and i will be discussing also. 18:46:40 we can target some very basic sketchy plumbing with the extensions - db - plugin patches in abt 2 weeks 18:46:48 perfect 18:46:53 does that seem reasonable ? 18:46:58 it does to me, yes 18:47:08 sorry, just speaking regarding the db piece 18:47:23 mfranc213: yes no worries 18:47:51 just drawing a line in the sand 18:48:04 sounds good 18:48:10 Sridark: one thing that would be very helpful for me at this stage is TODO item 1 on https://etherpad.openstack.org/p/neutron-fwaas-austin ?? 18:48:26 not sure why i have ?? there. pretend they're not there. 18:48:44 ok i will do that 18:48:57 thank you SridarK! 18:49:37 Perhaps shwetaap can use the same model and resubmit a patchset for the extension 18:50:02 same approach as njohnston and mfranc213 for db patch 18:50:19 SridarK: i'm sorry, can you say more about what you mean here? 18:50:28 i think the ext patch does have most have the pieces in place 18:50:32 ah 18:51:14 sounds good .. i will do that 18:51:36 shwetaap: can u pls run thru the patch to see what could be missing and then either submit another patch or restart the old one 18:52:24 mfranc213: sorry - all i meant is if we want to resurrect the old patch or submit a new one as u guys did for db 18:52:27 sure will do 18:52:39 either is fine - leave it to the new owners 18:52:44 SridarK: ahh, i see. 18:53:31 shwetaap: When you do testing, just ignore py27 and py34, they will always fail until we get the oslo.service issue fixed. 18:54:09 On the agent piece - yushiro is back next week - perhaps then padkrish - u guys can sync as well 18:54:18 once the oslo issue is fixed, we will want the team to know about it. what is the best way to do that? apologies if i've asked this already. is it by email? 18:54:29 njohnston: thanks will keep that in mind 18:54:36 #sridark# sure, i will also sync up with german regarding the existing patch 18:54:44 mfranc213: sure that will be easy 18:54:53 SridarK: ack 18:55:20 mickeys: things u want to bring up on the iptables front ? 18:55:47 I have not made any progress. I need to start on the patch to add a common unwrapped chain. 18:56:26 I have been setting up the dev env , downloaded the current patch 18:56:30 mickeys: ok 18:56:44 chandanc: ok 18:56:49 will ping mickey with questions 18:57:11 chandanc: if u need to sync with mickeys while in the same time zone 18:58:00 other things regarding v2 that anyone wants to bring up ? 18:58:00 sure sridark, may be by tomorrow i will have a better view 18:58:13 chandanc: ok perfect 18:58:38 nothing else to bring up, but I feel like we have gotten a decent start. 18:58:43 +1 18:58:46 +1 18:59:06 once we have the oslo.service issue figured out we should probably put that fix out in a separate patch so we can fast-track merging it 18:59:15 +1 18:59:41 njohnston: yes - pls add xgerman and me to the review - in case we miss it 18:59:43 I've also started looking at all the neutron-lib deprecations, that looks like it will probably not be difficult to fix. 18:59:50 SridarK: Will do. 19:00:16 Regarding 264488, I still feel that position does not belong in rule. We wanted to be able to reference a rule from multiple different policies. IMO this should be abandoned. 19:01:05 njohnston: yes we can prioritize as we see fit once we get some traction on the v2 patches 19:02:15 mickeys: yes agreed 19:02:27 mickeys: From a DB schema perspective, 'position' is a field in the firewall_policy_rule_associations table, so it is an attribute per association, not globally for the rule. 19:02:58 At least, that is what the spec laid out, so that's what I am working towards for the DB 19:03:25 njohnston: having it in the association table enables the reuse 19:03:31 so yes 19:03:42 njohnston: The proposed change is in the REST API section. It should remain in the DB association table. 19:03:43 yes 19:05:29 ok lets move on 19:05:57 But according to that change 'position' only indicates the position within the associated policy, making it a policy-specific bit of data for that rule. 19:05:57 #topic other reviews 19:06:00 OK, leaving that for later 19:06:23 should the rule position not be part of the policy API ? 19:06:39 sure, lets move on 19:06:48 ok lets pick it up in open discussion 19:07:06 we have the observer hierarchy patch out 19:07:20 #link https://review.openstack.org/#/c/278863/ 19:07:38 this is also plagued by the UT issue 19:08:10 i think the patch does what is intended - i am trying to test this as well 19:08:34 i have been talking to the submitter - he going thru some testing and possibly add some more on the UT 19:08:51 request folks to get some eyes on this as well 19:09:21 i will take a look 19:09:23 with this in - will enable us to remove some fwaas stuff from L3Agent 19:09:31 thx 19:09:46 +1 for removing fwaas stuff from L3Agent 19:09:58 any other reviews that folks want to bring up ? 19:10:46 badveli: could u pls also take a look 19:11:34 ok lets move on 19:12:00 #topic FWaaS weekly meeting time 19:12:26 yes 19:12:40 i will take a look sridhark 19:12:41 I think there was consensus for a single meeting time 19:12:58 +1 19:13:11 i realize with some new contributors from the India and Japan we will struggle to find a time that works fantastically for all 19:13:48 i am fine with the single time too - will certainly avoid a snafu like today :-) 19:14:35 possible lets throw some times out over email - so we ensure that active contributors are able to find some common ground 19:14:45 +1 19:15:16 The best guess I can come up with is 9:00am EST == 10:00pm in Japan, 6:30pm in New Delhi 19:15:43 But I shall be as flexible as I can. :-) 19:15:57 We can consider the discussion we had during summit 19:16:15 = 6 am PST 19:16:22 i start my day early - so i can make that work - but 6am Pacific could abe a challenge for some 19:16:57 yushiro is out this week - so lets start the discussion first thing next week 19:17:02 Sounds good. 19:17:04 so the delta between earliest and latest is, ignoring daylight savings, 8 hours. 19:17:05 That is two hours earlier than I normally wake up 19:17:14 sure, better to propose in mail thread, 6:30 pm India is mostly way back from Office. Can we push by 30 mins 19:17:27 if we push pacific time up, japan suffers... 19:17:45 I thought night time US, morning India and Japan might work better 19:18:08 lets see what is upper boundary for yushiro 19:18:34 and if we do evening what is the upper boundary for Eastern time zone 19:18:42 so we can do this over email 19:18:53 When DST ends, it becomes one hour earlier 19:18:56 in the US 19:18:57 sure 19:18:58 i think 11-ish ? 19:19:20 midnight EST == 1:00pm tokyo == 9:30am new delhi == 9:00pm san jose 19:19:23 speaking just for myself. 19:19:32 I would be fine with midnight 19:20:20 ok may be we have a formula there which we can tweak so it is not too late for Eastern and make it work for all 19:20:40 I am also happy to have the opportunity to provide for our esteemed colleagues in Asia the flexibility they have so often given us. 19:21:32 yes, perhaps something like 10:30 pm EST == 11:30am tokyo == 8am new delhi == 7:30pm san jose 19:21:49 i will be flexible too. +1 for what njohnston just said 19:22:01 ok good - lets take it to email then first thing next week 19:22:05 +1 19:22:12 +1 19:22:22 #topic Open Discussion 19:22:43 Regarding 264488, the API rules table is not in the context of a policy. The DB association table is in the context of a policy, so position there makes sense and is a good thing. In the API, the position is specified through the ordered list of rules in the policy table. 19:23:22 Looking forward: I am somewhat unfamiliar with how all this code works in practice; is there anything that is a pre-requisite for our testing fwaas in the lab to make sure the code hangs together, once we get to that point? 19:24:20 njohnston: something as basic as a single node devstack env is good 19:24:44 mickeys: would you be able to add your suggestions and concerns as comments to https://review.openstack.org/#/c/264488/ itself? 19:25:03 SridarK: Very good. I know we had to be much more involved when working on QoS DSCP because we weren't satisfied until we had tcpdumps showing the traffic was behaving as configured. 19:25:06 i have been using that to test out what ever we have albeit with some hacks 19:25:57 SridarK: do you generally pull from master for all other OS components? 19:25:59 i usually have a rule with an obscure port number - so i can quickly grep for that on the iptables to see if it made it all the way 19:26:07 mfranc213: yes 19:26:12 ack 19:26:15 mickey: agree, with the ordered list of rule in policy we should already have the position. right ? 19:26:42 SridarK: Is the devstack directives necessary to set up fwaas documented? Could you share your local.conf with us? 19:26:59 njohnston: ok surely will do that 19:27:13 q-fwaas - will set up the service plugin 19:27:24 mfranc213: I have put comments in 264488 in the past, for example Jan 15 8:19am comment 19:27:35 mickeys: thank you. 19:27:46 just q-fwaas and ofcourse l3-agent in local.conf is all that's needed, i guess 19:27:54 mickeys: do you feel your comments in the PS are exhaustive? 19:27:59 padkrish: yes 19:28:29 mfranc213: Do you think I should add another comment expanding on this, along the lines of the comments I made above? 19:28:32 padkrish and SridarK: does it matter at all what ML2 driver is in use? 19:28:37 2 min warning 19:28:41 mickeys: i think that would help me, yes. 19:29:03 njohnston: it should not 19:29:16 njohnston# good question, my default settings are always ovs as ML2 driver 19:29:22 SridarK: Excellent. That is what I thought, but I am checking my assumptions. 19:29:24 njohnston: do u use linux bridge ? 19:29:51 SridarK: I typically use OVS but I have had trouble in func tests where both OVS and LB were tested scenarios. 19:29:54 +1 to padkrish - i use ovs usually too 19:30:09 njohnston: ok 19:30:16 ok folks times up 19:30:20 two things: 1) i've been adding TODOs to https://etherpad.openstack.org/p/neutron-fwaas-austin 19:30:21 thanks all 19:30:22 , and 2) is there anyone else here who is having difficulty editing https://wiki.openstack.org/wiki/Neutron/FWaaS/NewtonPlan (as i am)? 19:30:27 thanks all! good meeting! 19:30:34 There is a bit of additional code in iptables when using OVS, but it should not make a significant difference. I never did figure out how conntrack zones are assigned in case of LB, should probably get back to that. 19:30:35 thanks 19:30:38 mfranc213: lets pick it up later 19:30:44 thank you 19:30:56 bye all 19:30:59 #endmeeting