18:32:30 <SumitNaiksatam> #startmeeting networking_policy
18:32:31 <openstack> Meeting started Thu Sep 21 18:32:30 2017 UTC and is due to finish in 60 minutes.  The chair is SumitNaiksatam. Information about MeetBot at http://wiki.debian.org/MeetBot.
18:32:32 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
18:32:34 <openstack> The meeting name has been set to 'networking_policy'
18:32:52 <SumitNaiksatam> #info agenda https://wiki.openstack.org/wiki/Meetings/GroupBasedPolicy#Sept_21st.2C_14th.2C_7th_2017
18:33:12 <SumitNaiksatam> #topic DB migration backport
18:33:32 <SumitNaiksatam> gerrit seems to be down
18:33:43 <tbachman> :(
18:33:50 <SumitNaiksatam> tbachman: you abandoned the patches i have linked in the agenda
18:34:01 <tbachman> SumitNaiksatam: those were replaced by the new migration script
18:34:08 <SumitNaiksatam> but you have the new patch which we discussed late couple of weeks
18:34:12 <SumitNaiksatam> tbachman: right
18:34:36 <tbachman> #link https://review.openstack.org/504976 <= new gerrit to address DB migrations (short-term solution)
18:34:43 <SumitNaiksatam> i think your latest patch got over the migration script error, right?
18:34:45 <tbachman> (at least the link is right ;-)
18:34:48 <tbachman> SumitNaiksatam: ack
18:34:51 <SumitNaiksatam> tbachman: :-)
18:35:16 <tbachman> the issue was a subsequent migration script (already committed) was missing the previous pointer
18:35:22 <tbachman> once I updated that, it worked
18:35:35 <tbachman> (even with the method that imports, rather than duplicates, the earlier migrations)
18:35:44 <SumitNaiksatam> tbachman: cool
18:36:07 <tbachman> SumitNaiksatam: the plan is to backport this, dropping some of the migrations as needed, per-stable branch
18:36:10 <SumitNaiksatam> so this is with the non __init__ workaround?
18:36:17 <SumitNaiksatam> tbachman: that makes sense
18:36:23 <tbachman> SumitNaiksatam: it still needs the __init__ workaround
18:36:33 <SumitNaiksatam> tbachman: thanks for getting this patch out!
18:36:37 <tbachman> SumitNaiksatam: np!
18:36:38 <SumitNaiksatam> nice work
18:36:40 <tbachman> thx!
18:36:57 <tbachman> I know that rkukura won’t be able to review it until next week, due to the holiday
18:36:57 <SumitNaiksatam> on the longer term strategy i dont think we have made that much head way
18:37:02 * tbachman nods
18:37:08 <SumitNaiksatam> i am good with the patch
18:37:15 <SumitNaiksatam> so if we get one more reviewer, we can merge it
18:37:24 <tbachman> k
18:37:30 <SumitNaiksatam> on the longer term, i did talk one more colleague
18:37:36 <SumitNaiksatam> tbachman: you know him :-)
18:37:40 <tbachman> :)
18:37:47 <SumitNaiksatam> or you can guess
18:37:49 <tbachman> lol
18:38:04 <SumitNaiksatam> the suggestion was to build the migration chain as a graph
18:38:23 <SumitNaiksatam> where we would have multiple HEADs as leaf nodes
18:39:10 <SumitNaiksatam> in theory this approach would work, but it will be pretty complicated to write the migration scripts since the author will have to spell out the dependency of a migration script on all other scripts (it depends on)
18:39:22 <SumitNaiksatam> difficult to completely explain here
18:39:27 <SumitNaiksatam> will revert to email
18:39:31 <tbachman> SumitNaiksatam: sounds good
18:39:50 <SumitNaiksatam> you can think of a generalization of the multiple chain strategy i was thinking
18:40:17 <SumitNaiksatam> tbachman: i assume you are not in hurry as far as the longer term strategy is concerned?
18:40:22 <tbachman> SumitNaiksatam: ack
18:40:29 <SumitNaiksatam> ok thankds
18:40:47 <SumitNaiksatam> #topic UI patches
18:40:52 <tbachman> I think we need to make sure we have a good strategy long-term, so spending the time to think and investigate is time well-spent, imho
18:40:58 <SumitNaiksatam> tbachman: ack
18:41:19 <SumitNaiksatam> #link https://review.openstack.org/#/q/status:open+project:openstack/group-based-policy-ui
18:41:32 <SumitNaiksatam> so i am still seeing some issues with the patches posted
18:41:37 <tbachman> :(
18:41:40 <SumitNaiksatam> these might very well be issues in my env
18:41:46 <tbachman> strange, b/c it sounds like they’re working for annak
18:41:50 <tbachman> annakk
18:42:05 * tbachman forgot the extra “K"
18:42:16 <SumitNaiksatam> i have been testing some of the patches she has presumably not tested
18:42:36 <SumitNaiksatam> presumed (based on the review comments)
18:43:07 <SumitNaiksatam> we will eventually finish reviewing and merging these patches (hopefully soon)
18:43:27 <SumitNaiksatam> the thing i am not sure about is whether we want to backport these to stable/ocata
18:43:38 <SumitNaiksatam> i started of thinking that we were going to backport these
18:43:58 <tbachman> SumitNaiksatam: it might be good to get annakk’s insights here
18:44:03 <SumitNaiksatam> but there is a huge amount of refactor in these patches and i am leaning towards not backporting these
18:44:05 <tbachman> to see if she’s tried them on stable/ocata as well
18:44:06 <SumitNaiksatam> tbachman: rivht
18:44:11 <SumitNaiksatam> *right
18:44:37 <SumitNaiksatam> so unless its absolutely needed, my proposal will be to not backport it
18:44:52 <SumitNaiksatam> will check with annak if she needs these to be backported
18:44:55 <tbachman> k
18:45:28 <SumitNaiksatam> #topic py27 error messages and copious logs
18:45:37 <tbachman> SumitNaiksatam: thanks for all the patches on this!
18:45:41 <SumitNaiksatam> there were a couple of patches posted to address this
18:45:44 <SumitNaiksatam> tbachman: np
18:45:45 * tbachman still has to review and approve
18:45:53 <SumitNaiksatam> one from annak and one which i posted
18:46:04 <SumitNaiksatam> tbachman: yeah thanks, please take a look
18:46:12 <tbachman> SumitNaiksatam: will do
18:46:14 <SumitNaiksatam> i think annak is planning another rev on her patch
18:46:23 <tbachman> k
18:46:50 <SumitNaiksatam> there are other ones which are not addressed
18:46:59 <SumitNaiksatam> #topic Open Discussion
18:47:09 <SumitNaiksatam> tbachman: anything else you wanted to bring up?
18:47:16 <tbachman> SumitNaiksatam: nothing right now
18:47:25 <SumitNaiksatam> tbachman: ok cool, thanks for joining
18:47:28 <tbachman> SumitNaiksatam: np!
18:47:32 <tbachman> thanks SumitNaiksatam!
18:47:32 <SumitNaiksatam> bye!
18:47:35 <tbachman> bye!
18:47:37 <SumitNaiksatam> #endmeeting