18:08:55 <SumitNaiksatam> #startmeeting networking_policy
18:08:55 <openstack> Meeting started Thu Feb 26 18:08:55 2015 UTC and is due to finish in 60 minutes.  The chair is SumitNaiksatam. Information about MeetBot at http://wiki.debian.org/MeetBot.
18:08:56 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
18:08:58 <openstack> The meeting name has been set to 'networking_policy'
18:09:04 <SumitNaiksatam> yapeng: s3wong: :-)
18:09:28 <SumitNaiksatam> #info agenda https://wiki.openstack.org/wiki/Meetings/Neutron_Group_Policy#Feb_26th.2C_19th.2C_12th.2C_2015
18:09:47 <SumitNaiksatam> so i wanted to make a proposal up front
18:10:00 <SumitNaiksatam> how about having a bug squashing day sometime early next week?
18:10:18 <SumitNaiksatam> that will help us plan for the next cut of the stable release as well
18:10:58 <SumitNaiksatam> the idea would be squash as many as high and medium priority bugs on that
18:11:07 <SumitNaiksatam> and have reviewers handy to review and merge them
18:11:23 <rkukura> +1, but not on Monday (I’m flying)
18:11:29 <SumitNaiksatam> rkukura: sure
18:11:45 <SumitNaiksatam> we can coordinate on the day, perhaps, tuesday/wednesday
18:11:52 <mageshgv> +1, sure, we can have it next week
18:11:54 <SumitNaiksatam> if everyone is on board with the idea
18:11:58 <Yi> +1
18:12:25 <ivar-lazzaro> SumitNaiksatam: +1
18:12:34 <SumitNaiksatam> note that this will also help with regards to the planned refacto (to have these bugs fixed before the refactor)
18:12:50 <SumitNaiksatam> Yi: ivar-lazzaro: mageshgv: ok thanks for the confirmation
18:13:03 <yapeng> +1, wed. works for me.
18:13:19 <SumitNaiksatam> lets tenatively plan that for tuesday or wednesday (or perhaps even thursday)
18:13:20 <SumitNaiksatam> yapeng: thanks
18:13:36 <SumitNaiksatam> i will reach out to everyone, and follow up with a note to the ML
18:14:05 <SumitNaiksatam> LouisF: hi
18:14:11 <LouisF> SumitNaiksatam: hi
18:14:30 <SumitNaiksatam> LouisF: FYI - we are planning a bug squashing day sometime next week
18:14:41 <LouisF> ok
18:14:43 <SumitNaiksatam> okay anything else to share with the rest of the team upfront?
18:15:07 <SumitNaiksatam> #topic Bugs
18:15:12 <SumitNaiksatam> so still on the topic of bugs
18:15:34 <SumitNaiksatam> one request, please check if the bugs you are fixing have a “juno-backport-potential” tag
18:15:50 <SumitNaiksatam> and if so, please propose a cherry-pick backport patch to the stable/juno branch
18:16:17 <SumitNaiksatam> we dont have any criticals show up during the week
18:16:31 <SumitNaiksatam> any other bugs/fixes we need to discuss here?
18:17:02 <SumitNaiksatam> mageshgv: and krishnaK have a couple of bug fixe patches on review
18:17:26 <SumitNaiksatam> mageshgv: i think ivar-lazzaro had some comments for you, anything we need to discuss with the rest of the team here?
18:17:30 <SumitNaiksatam> songole: hi!
18:17:38 <songole> Hi SumitNaiksatam
18:18:30 <mageshgv> SumitNaiksatam: I think the comments are addressed unless ivar-lazzaro has some more comments to bring up
18:18:43 <SumitNaiksatam> mageshgv: ah, i did not check today
18:18:45 <ivar-lazzaro> mageshgv: will look at the review today
18:19:08 <SumitNaiksatam> yeah, lets circle back on that
18:19:18 <SumitNaiksatam> oh, while on that
18:19:23 <SumitNaiksatam> one informational point to the team -
18:19:46 <SumitNaiksatam> when creating a foreign key or unique constraints, please provide a name to those
18:20:01 <SumitNaiksatam> there are at least a couple of places in the code where we are not doing that
18:20:15 <ivar-lazzaro> SumitNaiksatam: +1!
18:20:17 <SumitNaiksatam> it becomes a little bit messy to drop those constructs later
18:20:29 <SumitNaiksatam> ivar-lazzaro: thanks for doing the investigation on this as well
18:20:36 <ivar-lazzaro> SumitNaiksatam: magesh found a way of doing that in a backward compatible way
18:20:42 <SumitNaiksatam> ivar-lazzaro: true
18:20:52 <ivar-lazzaro> mageshgv: that's awesome!
18:20:53 <SumitNaiksatam> and thats great, but i think it will still help to name
18:21:18 <mageshgv> As a general rule it will help later if we name the constraints though
18:21:24 <SumitNaiksatam> mageshgv: yeah
18:21:34 <rkukura> SumitNaiksatam: no need to name the constraints
18:21:48 <SumitNaiksatam> mageshgv: for the fix you have, is there any chance that column names can have overlaps?
18:22:28 <rkukura> see https://review.openstack.org/#/c/116122/23/neutron/db/migration/alembic_migrations/versions/2d2a8a565438_hierarchical_binding.py for a migration that removes unnamed constraints
18:22:29 <SumitNaiksatam> mageshgv: i know its not true in our case currrently
18:22:35 <mageshgv> SumitNaiksatam: I do not think there will be any overlap issue there
18:23:07 <SumitNaiksatam> rkukura: i am sure it can be done, as mageshgv also figured out
18:23:21 <ivar-lazzaro> rkukura: thanks for the link
18:23:48 <SumitNaiksatam> rkukura: i think it makes the code more readable and less prone to mistakes if the constraint is identified by a name
18:23:58 <rkukura> SumitNaiksatam: makes sense
18:24:30 <SumitNaiksatam> any other bugs that we need to discuss?
18:24:46 <SumitNaiksatam> i dont see a follow up on: 1416527 and 1417206
18:24:57 <SumitNaiksatam> so i will check with jishnub again
18:25:28 <SumitNaiksatam> #topic Re-factor Group Based Policy with Neutron RESTful APIs
18:25:43 <SumitNaiksatam> so we spent another week deliberating on this
18:25:50 <SumitNaiksatam> and i think it was time well spent
18:26:00 <yapeng> yes, thanks for comments.
18:26:12 <SumitNaiksatam> meanwhile yapeng and Yi also have some WIP patches
18:26:12 <yapeng> I updated the spec yesterday. please review it.
18:26:28 <SumitNaiksatam> yapeng: yes, i saw that, and I have +2’ed it
18:26:49 <SumitNaiksatam> i think we should close on this now, and have further discussion in the implementation patches
18:27:07 <SumitNaiksatam> if we need to revisit design, we can revise the spec by posting a follow up patch
18:27:23 <Yi> SumitNaiksatam: +1
18:27:29 <ivar-lazzaro> SumitNaiksatam: on you comment on L102 of #link https://review.openstack.org/#/c/153126/7/specs/kilo/gbp-refactor-neutron-restful-api.rst
18:27:58 <ivar-lazzaro> SumitNaiksatam: when "self.create_policy_target" is used in the UTs, that is a wrapper for neutron-wsgi
18:27:58 <SumitNaiksatam> ivar-lazzaro: yes
18:28:06 <ivar-lazzaro> SumitNaiksatam: that's not a direct call
18:28:18 <ivar-lazzaro> SumitNaiksatam: erm "neutron-plugin" call
18:28:57 <ivar-lazzaro> SumitNaiksatam: under the hood that method does a new_create_request
18:30:07 <SumitNaiksatam> ivar-lazzaro: is that what is being mentioned there?
18:30:13 <ivar-lazzaro> SumitNaiksatam: so as long as the UTs client wrapper uses neutron-wsgi calls, we should be ok
18:30:30 <ivar-lazzaro> "When a call such as "create_policy_target" [1] is made in the RMD UT, the RMD implementation is making a neutron-plugin call."
18:31:00 <ivar-lazzaro> SumitNaiksatam: did I understand it wrong?
18:31:31 <SumitNaiksatam> ivar-lazzaro: checking
18:31:52 <yapeng> ivar-lazzaro: I think this is the current implementation..
18:33:34 <SumitNaiksatam> ivar-lazzaro: by neutron-plugin calls i am referring to #link https://github.com/stackforge/group-based-policy/blob/master/gbpservice/neutron/services/grouppolicy/drivers/resource_mapping.py#L1358-L1416
18:34:21 <ivar-lazzaro> SumitNaiksatam: ok, that seemed to be referring to the UT implementation
18:34:27 <ivar-lazzaro> SumitNaiksatam: thanks for the clarification
18:34:32 <SumitNaiksatam> ivar-lazzaro: okay
18:35:14 <SumitNaiksatam> ivar-lazzaro: ah i see, i put [1] as a reference to the UT calls
18:36:03 <SumitNaiksatam> okay so anyone not on board to make progress with the way the current spec is defined?
18:36:41 <s3wong> +1
18:37:06 <SumitNaiksatam> s3wong: thanks :-)
18:38:10 <SumitNaiksatam> okay, so unless there are any other major concerns lets close on this spec today
18:38:27 <SumitNaiksatam> cores - request you to take a look at the earliest and push this forward
18:38:28 <s3wong> SumitNaiksatam: I can review the spec one more time today and give the second +2
18:38:41 <SumitNaiksatam> s3wong: okay great, thanks!
18:39:50 <SumitNaiksatam> #topic Floating IP support
18:39:58 <SumitNaiksatam> #undo
18:39:59 <openstack> Removing item from minutes: <ircmeeting.items.Topic object at 0x8b0aa50>
18:40:16 <SumitNaiksatam> just realized that i did not post the link of the refactoring spec...
18:40:32 <SumitNaiksatam> so belatedly - #link https://review.openstack.org/#/c/153126 ;-)
18:40:47 <SumitNaiksatam> #topic Floating IP support
18:40:58 <SumitNaiksatam> #link https://review.openstack.org/157298
18:41:06 <SumitNaiksatam> i think mageshgv dropped off
18:41:37 <SumitNaiksatam> myself and ivar-lazzaro reviewed this, and we are waiting for mageshgv to post the next patch set
18:41:53 <SumitNaiksatam> anyone else had a chance to look at the above spec?
18:41:59 <SumitNaiksatam> questions/comments?
18:42:19 <SumitNaiksatam> i believe songole is here and might be able to address those on mageshgv’s behalf
18:42:28 <rkukura> SumitNaiksatam: I have not yet, but will
18:42:34 <SumitNaiksatam> rkukura: great, thanks
18:42:48 <songole> SumitNaiksatam: I have not reviewed yet. I will review.
18:42:58 <SumitNaiksatam> songole: ah okay :-)
18:43:18 <SumitNaiksatam> rkukura: songole any chance that you guys can review this by EoD?
18:43:28 <songole> ok
18:43:50 <SumitNaiksatam> that way when mageshgv is back by evening our time, he can post a consolidated/updated patch set
18:44:16 <rkukura> SumitNaiksatam: OK
18:44:34 <SumitNaiksatam> rkukura: thanks
18:45:14 <SumitNaiksatam> rkukura: over to you for the next two topics
18:45:16 <SumitNaiksatam> #topic Taskflow investigation
18:45:31 <SumitNaiksatam> you mentioned you are reviewing the neutron patches
18:45:37 <SumitNaiksatam> and gaining better understanding
18:46:02 <rkukura> I haven’t made any progress on this since last week, but will get to it shortly
18:46:21 <rkukura> I’d also like to look at how other projects are using TaskFlow
18:46:29 <SumitNaiksatam> rkukura: sure
18:46:42 <SumitNaiksatam> rkukura: would it help if we have a sesson with manish next week?
18:46:48 <SumitNaiksatam> since you are going to be here?
18:46:58 <SumitNaiksatam> i can try and reach out to him
18:47:28 <SumitNaiksatam> rest of the team can also participate
18:47:34 <rkukura> SumitNaiksatam: could be a good idea, but lets see when we schedule the bug squashing first
18:47:58 <SumitNaiksatam> rkukura: yes sure, we can do those on separate days
18:48:14 <SumitNaiksatam> okay so two action items for me in terms of planning for next week
18:48:15 <rkukura> I think we may have some different requirements with the need to validate and implicitly create resources before transactions, ...
18:48:23 <SumitNaiksatam> rkukura: sure
18:48:46 <SumitNaiksatam> rkukura: thanks for the update
18:48:51 <rkukura> np
18:49:05 <SumitNaiksatam> #topic Packaging update
18:49:20 <SumitNaiksatam> i believe no developments on the RH/Fedora packing, right?
18:50:08 <SumitNaiksatam> rkukura: ^^^ ?
18:50:12 <rkukura> SumitNaiksatam: one
18:50:33 <rkukura> The heat package’s dependencies have become out of date, so this needs an update
18:51:03 <rkukura> would be nice to update to stable releases for everything at the same time
18:51:20 <SumitNaiksatam> rkukura: ah, was just asking if we needed to cut a new stable
18:51:45 <rkukura> Its actially the horizon package, not heat
18:51:45 <SumitNaiksatam> so on that, susaant has made a couple of fixes in gbpautomation which should be backported
18:52:01 <SumitNaiksatam> rkukura: ah okay
18:52:18 <SumitNaiksatam> i will be posting a fix in horizon as well
18:52:37 <rkukura> and it might just be that fedora rawhide has already moved to a kilo milestone - need to investigate
18:52:56 <SumitNaiksatam> rkukura: okay
18:52:59 <rkukura> how about if we target end of next week for stable update?
18:53:15 <rkukura> after we’ve squashed and backported bugs
18:53:21 <SumitNaiksatam> rkukura: sure, end of next week
18:54:10 <SumitNaiksatam> so horizon we can cut new stable after i post the patch
18:54:14 <SumitNaiksatam> please watch out for that
18:54:35 <rkukura> was just looking at the broken dependency email - I think we will need kilo-compatible versions to resolve this
18:54:51 <SumitNaiksatam> rkukura: oh, thats tricky
18:54:58 <SumitNaiksatam> rkukura: lets discuss offline
18:54:59 <rkukura> but this issue is just with Fedora rawhide (F22) right now, not RDO, which is still Juno
18:55:08 <SumitNaiksatam> rkukura: whew!
18:55:36 <SumitNaiksatam> slightly off current-topic, but since we brought up heat - susaant has two important bug fixes waiting to be reviewed:
18:55:44 <SumitNaiksatam> #link https://review.openstack.org/155096
18:55:51 <SumitNaiksatam> #link https://review.openstack.org/155153
18:55:58 <SumitNaiksatam> if anyone has time, please review
18:56:13 <SumitNaiksatam> the above will need to be backported
18:56:29 <SumitNaiksatam> there is another one which is for a namespace change: #link https://review.openstack.org/155172
18:56:35 <SumitNaiksatam> which will not be backported
18:56:44 <SumitNaiksatam> also regarding the Ubuntu packages
18:57:05 <SumitNaiksatam> we have not documented the install process (similar to what has been done for RH/Fedora)
18:57:13 <SumitNaiksatam> i think magesh tried this and has some notes
18:57:18 <SumitNaiksatam> he will be posting them on the wiki
18:57:28 <SumitNaiksatam> but if anyone else has tried this, please chime in
18:57:33 <SumitNaiksatam> on the wiki page
18:57:51 <SumitNaiksatam> this would be the place to put it: #link https://wiki.openstack.org/wiki/GroupBasedPolicy#Try_Group-based_Policy
18:57:57 <SumitNaiksatam> #topic Open Discussion
18:58:33 <SumitNaiksatam> LouisF: nicolas mentioned that there was someone from his team who was going to follow up on your blueprints
18:58:40 <SumitNaiksatam> LouisF: havent seen acitivity there
18:59:20 <SumitNaiksatam> anything else anyone want to discuss?
18:59:32 <SumitNaiksatam> grand total of one min remaining ;-)
19:00:19 <SumitNaiksatam> okay, i guess nothing more for today
19:00:24 <rkukura> anyone using the extension driver APIs?
19:00:33 <SumitNaiksatam> rkukura: not yet
19:00:44 <SumitNaiksatam> need to check with banix
19:00:50 <rkukura> some changes in the neutron version of those we might want to pull in
19:00:53 <SumitNaiksatam> and i think ivar-lazzaro also plans to use
19:01:09 <SumitNaiksatam> lets take it to #openstack-gbp
19:01:13 <SumitNaiksatam> thanks everyone
19:01:16 <SumitNaiksatam> bye!
19:01:21 <ivar-lazzaro> ciao!
19:01:31 <yapeng> bye!
19:01:34 <SumitNaiksatam> #endmeeting