17:32:59 <mfedosin> #startmeeting glare
17:33:00 <openstack> Meeting started Mon May 16 17:32:59 2016 UTC and is due to finish in 60 minutes.  The chair is mfedosin. Information about MeetBot at http://wiki.debian.org/MeetBot.
17:33:02 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
17:33:05 <openstack> The meeting name has been set to 'glare'
17:33:17 <mfedosin> #topic agenda
17:33:28 <mfedosin> #link https://etherpad.openstack.org/p/glance-artifacts-sub-team-meeting-agenda
17:33:52 <mfedosin> #topic Updates
17:34:35 <mfedosin> So, I haven't been involved in Glare last two weeks
17:35:00 <mfedosin> but kairat and dshakray made a great work
17:35:26 <mfedosin> before the summit we defined a list of items that must be done before merging
17:35:36 <mfedosin> there were 11 points
17:35:45 <mfedosin> 10 of them are implemented
17:35:52 <docaedo> nice!
17:35:58 <mfedosin> only db is left
17:36:23 <mfedosin> it lead to the fact that after some testing we are ready to merge it
17:36:40 <mfedosin> I updated API spec in late April
17:36:40 <nikhil> mfedosin: can I say something here?
17:36:45 <mfedosin> sure
17:37:07 <nikhil> so, the only way this will merge it soon is if people know what is being implemented and how
17:37:18 <nikhil> and the way to do that is working upstream
17:37:31 <nikhil> otherwise, we will again have last minute -1, -2s
17:37:44 <nikhil> as people will not see things fit for some of the use cases
17:37:52 <nikhil> So, like I mentioned at the summit:
17:37:53 <mfedosin> yeah, it was my next topic :)
17:38:12 <nikhil> ok, let's carry on for now. I can add comments later.
17:38:21 <mfedosin> for sure we need to have good documentation
17:38:34 <nikhil> mfedosin: that's not enough
17:38:41 <nikhil> to be really honest
17:38:43 <mfedosin> what else?
17:39:03 <nikhil> I can share the plan to "how to do it" upstream later with you
17:39:23 <nikhil> you should carry on with your agenda for now. I may take 15-20 mins for that.
17:39:37 <mfedosin> okay, great
17:40:03 <mfedosin> we can talk later about "how to do it in upstream"
17:40:08 <nikhil> k
17:40:49 <mfedosin> on Wednesday we're going to have a meeting dedicated to final Glare API
17:41:13 <nikhil> send that notice to ML
17:41:23 <mfedosin> walk through the spec and update it if needed
17:41:51 <mfedosin> good point, but I meant inner meeting
17:42:03 <nikhil> mfedosin: from now on, no inner meetings please
17:42:06 <mfedosin> but upstream is good idea
17:42:26 <mfedosin> for example, on Friday
17:42:33 <nikhil> I think for a POC that's fine. but once the idea is in good shape, let's do the discussion 100% upstream
17:42:46 <mfedosin> I'll send this email in ML
17:42:49 <nikhil> k
17:43:22 <mfedosin> #topic Merge plans
17:43:37 <mfedosin> let's define it
17:43:56 <mfedosin> what is definition of done for the spec?
17:44:18 <mfedosin> I mean how detailed it should be
17:44:32 <nikhil> I think for now let's focus on the bare minimum
17:44:42 <nikhil> and we need to clearly indicate that it is so
17:44:56 <nikhil> and also document what will/can be implemented and what are future plans
17:45:07 <nikhil> let us focus on "just" Newton now
17:45:24 <nikhil> let us tell the story of how this API will improve in features over time
17:45:42 <mfedosin> so it should be another spec?
17:45:51 <nikhil> so, the first and foremost thing to do is to come up with the "minimal" set of calls needed for say HEAT
17:46:04 <nikhil> and let's say that in the spec
17:46:07 <mfedosin> or it's a part of existing one?
17:46:10 <mfedosin> ok, I see
17:46:19 <nikhil> for now, just one spec
17:46:26 <nikhil> we can discuss future plans later.
17:46:43 <mfedosin> what about API examples there?
17:46:56 <nikhil> mfedosin: the more the better
17:47:07 <nikhil> mfedosin: but let's not go in too much detail for now
17:47:12 <mfedosin> the spec will be huge :)
17:47:20 <nikhil> mfedosin: for example let's not worry about tags if that is not needed
17:47:31 <nikhil> mfedosin: let us not worry about import or sharing
17:48:04 <mfedosin> yeah, only base part
17:48:07 <nikhil> mfedosin: the first spec will be reasonable big but that's okay.
17:48:23 <nikhil> reasonably*
17:48:48 <mfedosin> and about the code
17:49:09 <nikhil> mfedosin: also, I think it will be really useful if we can get a liaison from heat and someone else from glance actually working on the code
17:49:14 <mfedosin> we agreed to separate in small patches before merge - not this one big commit
17:49:21 <nikhil> correct
17:49:32 <mfedosin> nikhil: I told you - we have one
17:49:49 <nikhil> k, let's make sure all that is done upstream
17:49:53 <mfedosin> and he's already writing a spec for Heat
17:50:21 <nikhil> people have a _strong_ dislike for code that is _dumped_ upstream
17:51:48 <mfedosin> also, we decided don't include images artifact type initially
17:52:06 <nikhil> correct
17:52:48 <mfedosin> and no special features :)
17:53:24 <mfedosin> I predict that amount of code without tests will be 5000-7000 LOC
17:53:30 <nikhil> 👍
17:53:30 <mfedosin> may be less
17:53:49 <nikhil> umm
17:54:01 <nikhil> ok, let's see what we can remove as possible
17:54:21 <mfedosin> db part is very big
17:54:23 <nikhil> just make sure each patch is 250LOC with unit tests
17:54:36 <mfedosin> all these migrations...
17:54:58 <nikhil> eh
17:54:58 <mfedosin> I think there will be about 15 patches
17:55:18 <mfedosin> 200-300 LOC each
17:55:23 <nikhil> ok, let's see
17:55:40 <nikhil> once they are out we need to meet to discuss what can be done first
17:56:02 <nikhil> note: people are getting worried about code not being upstream
17:56:03 <mfedosin> yup
17:56:20 <mfedosin> it's all on review :)
17:56:30 <nikhil> with WIP ?
17:56:30 <mfedosin> except db part
17:56:33 <mfedosin> yeah
17:56:43 <nikhil> :)
17:56:50 <docaedo> yeah as far as I knew everything was out there (that's what we played with in app-catalog land)
17:57:05 <nikhil> I see
17:57:13 <nikhil> so for consumers it is out there
17:57:34 <nikhil> for reviewers (hard part) we need to make sure it's in the right shape for it to be reviewed
17:57:40 <mfedosin> kind of :)
17:57:53 <nikhil> mfedosin: 250LOC /each :)
17:58:09 <mfedosin> as I mentioned on the summit will start to split it in early June
17:58:44 <mfedosin> and if it's possible will have the spec been merged by that time
17:58:47 <nikhil> mfedosin: hmm, ok. let me know in detail what are the blockers so that I can plan accordingly.
17:59:13 <nikhil> mfedosin:  this week we are supposed to focus on spec reviews as per my email
17:59:30 <nikhil> the more you review someone else's work, the more review you can get on your work ;-)
17:59:44 <mfedosin> gotcha ;)
17:59:54 <nikhil> check my emails to the ML
18:00:02 <mfedosin> for sure!
18:00:09 <nikhil> I am sending all that for people to use the approach to get things done fast!
18:00:22 <nikhil> gtg
18:00:25 <nikhil> next meeting
18:00:26 <nikhil> thanks!
18:00:27 <mfedosin> thank you for that
18:00:39 <mfedosin> #endmeeting