14:00:54 <SridarK> #startmeeting fwaas
14:00:54 <openstack> Meeting started Tue Jul 18 14:00:54 2017 UTC and is due to finish in 60 minutes.  The chair is SridarK. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:55 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:58 <openstack> The meeting name has been set to 'fwaas'
14:01:08 <SridarK> #chair xgerman_ yushiro
14:01:09 <openstack> Warning: Nick not in channel: yushiro
14:01:10 <openstack> Current chairs: SridarK xgerman_ yushiro
14:01:36 <SridarK> xgerman_: is it ur turn today to chair ?
14:01:43 <xgerman_> I think so
14:01:53 <SridarK> ok all yours pls go ahead
14:01:54 <xgerman_> #topic Announcements
14:02:10 <xgerman_> https://releases.openstack.org/pike/schedule.html
14:02:26 <xgerman_> P-3 is near 7/24
14:02:42 <SridarK> yes we are very close
14:03:20 <xgerman_> #topic Pike
14:03:23 <SridarK> and i think neutron-lib changes need to get in this week ?
14:03:29 <xgerman_> yes
14:03:50 <xgerman_> Final release for non-client libraries  - 7/21
14:04:50 <xgerman_> I think we are still trying to move things… redip?
14:05:09 <xgerman_> reedip
14:05:40 <SridarK> #link https://review.openstack.org/#/c/478363/
14:05:50 <SridarK> this probab needs to get in first
14:05:58 <xgerman_> yep
14:06:05 <xgerman_> and then we have https://review.openstack.org/#/c/474524/ —
14:06:05 <SridarK> and then neutron-lib changes
14:07:10 <xgerman_> yes, it feels we are cutting it short — is there any way we cna help to carry it over the finish line
14:07:28 <SridarK> agree
14:07:41 <xgerman_> the Jenkins error is not encouraging
14:07:44 <SridarK> lets try to have a quick sync everyday at this same time on fwaas irc
14:07:49 <SridarK> yes
14:07:51 <xgerman_> +1
14:08:08 <xgerman_> #action quick sync every day at meeting time in #fwaas
14:08:37 <xgerman_> do we need a prioritized list of things to review?
14:09:29 <SridarK> yes
14:09:46 <SridarK> i think neutron-lib changes is priority 1
14:09:53 <SridarK> and then our L2 changes
14:10:08 <SridarK> if we can undo the revert from neutron-lib
14:10:20 <SridarK> that will be great as step 1
14:10:27 <xgerman_> #link https://etherpad.openstack.org/p/fawns_pike_prioritized_review
14:10:27 <SridarK> i think reedip is on it already
14:11:27 <xgerman_> let’s put the links in here and reviewers should check that and work down the list
14:11:41 <SridarK> xgerman_: typo on link
14:11:47 <SridarK> #link https://etherpad.openstack.org/p/fwaas_pike_prioritized_review
14:11:56 <xgerman_> auto-correct
14:12:00 <SridarK> :-)
14:12:29 <SarathMekala> :)
14:13:29 <xgerman_> should we move on?
14:13:54 <SridarK> yes
14:13:55 <xgerman_> #topic Horizon
14:14:28 <xgerman_> SarathMekala amotoki please go ahead
14:15:03 <SarathMekala> I have submitted a patch after fixing the review comments given by amotoki
14:15:16 <xgerman_> awesome!
14:15:40 <SarathMekala> I am working on adding validations to the forms.. will put in a patch for that by thrusday
14:15:53 <xgerman_> that sounds great!
14:15:59 <SarathMekala> thanks..
14:16:12 <SarathMekala> to give you an overview of the work that needs to be done:
14:16:15 <SarathMekala> 1. Add Validations
14:16:41 <SarathMekala> 2. Need to add a overlay for port add and delete
14:17:24 <SarathMekala> 3. Integrate the plugin with devstack -- amotoki said he will help with this
14:17:31 <amotoki> SarathMekala: what do you mean by "overlay"?
14:18:00 <SarathMekala> its the popup that comes up for the 'add port' button action
14:18:11 <SarathMekala> similarly for 'edit port' btn action
14:18:43 <amotoki> so we can check and test it as normal reviews :)
14:19:25 <SarathMekala> 4. Revamp the table using angular js (stretch goal)
14:19:35 <SarathMekala> table --> Firewall group table
14:20:06 <SarathMekala> today all the ports get listed down as comma separated values, we need to think of some better way if lots of ports get added
14:20:32 <SarathMekala> 5. Make the ports listed in the table as clickable links
14:21:42 <SarathMekala> we can track these items
14:22:33 <SarathMekala> 1,2 and 3 are must do.. 5 gives better UX
14:22:41 <xgerman_> sounds good - do we need to capture that in LP?
14:22:46 <SarathMekala> 4 is better jazz
14:23:15 <SridarK> SarathMekala: given the timeline we can get in basic support first
14:23:37 <SridarK> 4, 5 can follow as a subsequent patch ?
14:23:42 <SarathMekala> xgerman_, sure.. but where in LP?
14:24:04 <SarathMekala> SridarK, right, am of the same opinion
14:24:32 <xgerman_> Maybe on the FWaaS side of things? Not sure? I was just thinking we should capture the list somewhere
14:25:00 <SarathMekala> xgerman_, I was thinking of adding it to our etherpad..
14:25:58 <xgerman_> ok, LP is the place where OpenStack tracks work items but I think ether-pad sounds ok for no
14:26:00 <xgerman_> now
14:26:20 <SarathMekala> xgerman_, ok
14:26:23 <reedip_> hi
14:26:26 <xgerman_> hi
14:26:35 <reedip_> hexchat crashed
14:26:37 <reedip_> :(]
14:26:45 <SarathMekala> I have one question for amotoki
14:27:05 <SarathMekala> one of your comment was to remove internationalization for the debug msgs
14:27:05 <hoangcx_> SarathMekala: https://launchpad.net/neutron-fwaas-dashboard
14:27:32 <hoangcx_> SarathMekala: Is this what you are looking for?
14:27:37 <SarathMekala> is this mandatory as the link you put in seems to suggest
14:28:26 <amotoki> SarathMekala: ping me after the meeting. I am in the neutron team meeting.
14:28:50 <amotoki> the two meetings conflict biweeky :(
14:28:52 <SarathMekala> hoangcx_, I have no clue what to do with LP :) but will check and see if I can add the todo list here
14:28:59 <SarathMekala> amotoki, sure
14:29:41 <hoangcx_> I think that is the best place to track the work (especially feature) rather than etherpad
14:30:14 <SarathMekala> sure will check.. thanks
14:30:27 <SarathMekala> that's all from my side
14:30:29 <xgerman_> yes, didn’t know we had a project there… Thanks hoangcx_ I can assist on adding those tasks
14:30:48 <SarathMekala> amotoki do you have any updates?
14:30:59 <hoangcx_> np :-)
14:31:13 <amotoki> wait for a second
14:31:51 <amotoki> re dashboard fwaas v2 devstack, I will push devstack support as part of SarathMekala's v2 dashboard support.
14:32:20 <amotoki> would like to discuss SarathMekala when it is better i push a devstack support later
14:32:34 <amotoki> for neutron-fwaas-dashboard relase, I would like to have the first release with v1 stuff this week.
14:32:42 <amotoki> that's all from me.
14:34:34 <xgerman_> ok
14:35:09 <xgerman_> #topic pluggable backend
14:35:22 <xgerman_> not sure if we have doude?
14:35:35 <SridarK> xgerman_: shall we also have some time to discuss neutron-lib now that reedip_ is online
14:35:44 <xgerman_> yeah, let’s do that
14:35:45 <SridarK> and also L2
14:35:54 <SridarK> xgerman_: thx
14:36:10 <xgerman_> #topic L2 and Neutron Lib
14:36:39 <xgerman_> reedip_ neutron lib deadline is Friday
14:37:12 <SridarK> Also many thx to yamamoto for getting in midonet non-voting job for fwaas
14:37:16 <reedip_> SridarK , xgerman_ can we proceed without neutron-lib release ?
14:37:35 <SridarK> reedip_: u think it will be a challenge to undo the revert ?
14:37:38 <reedip_> xgerman_ : currently neutron-fwaas needs the release of neutron-lib
14:37:43 <reedip_> the patch in neutron-lib got merged
14:37:57 <reedip_> SridarK : not entirely
14:37:58 <SridarK> the exceptions ?
14:38:30 <reedip_> SridarK : our patches in neutron-lib have merged but our dependency on neutron-lib needs to be resolved
14:38:47 <SridarK> reedip_: ok
14:38:49 <yamamoto> SridarK: np!
14:38:50 <xgerman_> ok
14:40:47 <xgerman_> let’s move on then
14:40:53 <xgerman_> #topic OpenDiscussion
14:41:26 <SridarK> reedip_: maybe we can discuss more on the channel
14:41:46 <SridarK> reedip_: we still need to get our api defn back into neutron-lib
14:41:49 <xgerman_> yeah, I think us overlapping with the Neutron meeing is distracting
14:42:00 <reedip_> SridarK : the api-def is already in neutron-lib :)
14:42:08 <reedip_> we just need to use it in neutron-fwaas
14:42:24 <SridarK> reedip_: hmm ok sorry i missed that
14:42:43 <SridarK> so all we need to do now is to use neutron-lib ?
14:42:51 <xgerman_> well, IF everyhting we need is in neutron-lib then we are good for now and have a few more weeks to get it into FWaaS'
14:42:55 <reedip_> SridarK :  yep
14:43:07 <xgerman_> great!
14:43:15 <reedip_> xgerman_ : Do you think we need to cross analyze it once ?
14:43:17 <SridarK> ok so we are good - let me go and check
14:43:35 <TuanVu> Hi guys, could you please check https://review.openstack.org/#/c/443385/
14:43:47 <SridarK> the other thing to discuss is the L2 race condition
14:43:54 <TuanVu> I know it’ll take some time to read all the code, but it’s a pretty annoying bug which needs to be fixed soon.
14:43:56 <TuanVu> Because currently, firewall rules are not applied immediately when we add a new rule “with port range”.
14:43:56 <SridarK> chandanc: do u want to discuss some here
14:44:19 <chandanc> SridarK: for L2 race i discussed one approach with yushiro
14:44:36 <SridarK> chandanc: yes
14:44:48 <chandanc> I need to investigate the delete port handling part
14:45:05 <chandanc> yushiro was ok with the approach
14:45:39 <chandanc> my openion is the changes needs to go into agent rather then driver
14:45:42 <SridarK> ok if that works for CREATE/UPDATE
14:46:08 <chandanc> ya I will send a mail to yushirio and suggest the updated patch
14:46:23 <SridarK> chandanc: yes i agree we want to keep the driver simple
14:46:38 <chandanc> yes
14:46:59 <xgerman_> yes, but we can have the driver relay events if necessary
14:47:32 <chandanc> xgerman_: sorry dint get that part
14:48:17 <xgerman_> in order to keep the driver simple we can relay things to the agent
14:48:34 <chandanc> ok yes, i agree
14:48:41 <SridarK> chandanc: thx sounds good
14:48:59 <xgerman_> +1
14:49:07 <chandanc> I have some patch level dependency to resolve for testing,
14:49:21 <chandanc> we need l2 ext, default fwg and l2 driver
14:49:28 <SridarK> chandanc: i will allocate time this week - and i can help with testing or any debugging
14:49:38 <SridarK> chandanc: we can coordinate offline
14:49:39 <chandanc> thanks SridarK
14:49:47 <xgerman_> +1
14:50:01 <chandanc> sure lets catch up
14:50:03 <SridarK> chandanc: sorry last week - i was buried on some other things
14:50:12 <chandanc> same here :)
14:51:38 <xgerman_> yes, it has been busy…
14:53:18 <TuanVu> Hi all, regarding to https://review.openstack.org/#/c/443385/ , I really appreciate it if someone can have a look :)
14:53:30 <TuanVu> I’ll try my best to update very quickly if there’s any problem with the code
14:53:42 <xgerman_> ok, we will look
14:53:57 <SridarK> TuanVu: i will look as well
14:53:57 <TuanVu> Hi xgerman_, thank you very much :)
14:54:14 <TuanVu> awesome, SridarK, thanks a lot :)
14:54:32 <reedip_> TaunVu : it would be great if the commit message is changed a bit, looks a bit cluttered :)
14:54:33 <TuanVu> thank you guys in advance, I really appreciate it
14:54:56 <reedip_> just sayin
14:54:59 <TuanVu> thank you, reedip_, I'll change it
14:55:14 <reedip_> thanks TuanVu
14:55:20 <TuanVu> let's try to make it to be esier to understand
14:55:46 <doude_> Hi sorry to be late (too late)
14:55:59 <reedip_> xgerman_ , SridarK : seems neutron-lib got a release this weekend
14:56:08 <xgerman_> ok
14:56:16 <SridarK> reedip_: ok
14:56:42 <reedip_> So just rebasing the patches, to see if the changes are there and if so then we can push the migration patches
14:57:02 <reedip_> http://eavesdrop.openstack.org/meetings/networking/2017/networking.2017-07-18-14.00.log.html#l-17
14:57:03 <xgerman_> yeah, we also need to look in the merge conflicts
14:57:24 <xgerman_> 3 min left
14:57:24 <reedip_> hehe
14:57:59 <reedip_> doude_ : saw your patch, ... will review it this week
14:58:19 <doude_> ok, thanls reedip_
14:58:21 <SridarK> reedip_: and i resolved my confusion on neutron-lib, duh !!
14:58:28 <doude_> I'll try to divide it in two patches
14:58:42 <SridarK> reedip_: too many things in flight and the neurons are dying. :-)
14:58:42 <reedip_> SridarK :)
14:58:50 <SridarK> so we are all good
14:59:01 <reedip_> SridarK : dont tell me about it .. I can feel the same :(
14:59:04 <doude_> one with code reorganization and on whihc introduce the new backenddriver API
14:59:07 <SridarK> hopefully the rehoming of exceptions will make it in
14:59:16 <reedip_> yeah, pushing the rebase now
14:59:26 <xgerman_> seems so - I do so many different things any time amangement breaks down :-(
14:59:34 <SridarK> doude_: thx, i have it in my list also
14:59:43 <doude_> thx
14:59:52 <SridarK> xgerman_: +1
14:59:59 <SridarK> at time
15:00:08 <xgerman_> #endmeeting