14:01:06 <abhishekk> #startmeeting glance
14:01:08 <openstack> Meeting started Thu Sep  3 14:01:06 2020 UTC and is due to finish in 60 minutes.  The chair is abhishekk. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:01:09 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:01:11 <openstack> The meeting name has been set to 'glance'
14:01:12 <abhishekk> #topic roll call
14:01:17 <abhishekk> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:01:21 <abhishekk> o/
14:01:40 <dansmith> o/
14:02:07 <abhishekk> lets wait couple of minutes for others
14:03:02 <Steap> o/
14:03:31 <rosmaita> i am only partially here
14:03:38 <abhishekk> ack
14:03:42 <abhishekk> lets start
14:03:43 <abhishekk> #topic Updates
14:04:19 <abhishekk> I have created planning etherpad for W PTG which will be between October 26 to 30
14:04:34 <abhishekk> #link  https://etherpad.opendev.org/p/Glance-Wallaby-PTG-planning
14:04:49 <abhishekk> I have added some of the topics which we will discuss during the PTG
14:05:11 <abhishekk> Kindly go through it and add any topics you want to discuss or contribute
14:06:07 <abhishekk> Also, I will create specs folder for Wallaby so that we can move/add specs for reviews
14:06:24 <abhishekk> moving ahead
14:06:36 <abhishekk> #topic release/periodic jobs updates
14:06:51 <abhishekk> glance-store 2.3.0 released for Victoria
14:07:35 <abhishekk> I may need to submit another patch to create stable/victoria branch for glance-store (which I discuss with smcginnis latter)
14:08:02 <abhishekk> Victoria milestone 3 is next week
14:08:19 <abhishekk> We have couple of important patches left but those are in good shape
14:08:38 <abhishekk> We are maintaining the progress/status in below etherpad
14:08:41 <abhishekk> #link https://etherpad.opendev.org/p/glance-victoria-milestone-3-todo
14:09:20 <abhishekk> Periodic jobs are all green
14:09:30 <abhishekk> Moving ahead
14:09:38 <abhishekk> #topic Important reviews
14:09:44 <abhishekk> Cinder multiple stores support - https://review.opendev.org/748039
14:10:14 <abhishekk> dansmith, has given couple of comments related to tests and whoami-rajat__ is working on those
14:10:31 <abhishekk> documentation patch for the same is also up and mentioned in above etherpad
14:10:53 <abhishekk> Kindly put some time to review those patches
14:10:57 <whoami-rajat__> just writing my thoughts on the functional tests in glance channel
14:12:04 <abhishekk> dansmith, there is one major concern in writing functional test for the same
14:12:48 <dansmith> what, mocking cinder?
14:13:08 <abhishekk> so you are ok with mocking cinder call, right?
14:13:12 <dansmith> of course
14:13:27 <abhishekk> ok
14:13:48 <abhishekk> whoami-rajat__, do you have any other concerns?
14:14:11 <whoami-rajat__> no
14:14:23 <abhishekk> cool, may be we can discuss this offline
14:14:44 <abhishekk> Moving to open discussion
14:14:45 <whoami-rajat__> sure, thanks for the feedback
14:14:53 <abhishekk> #topic Open discussion
14:15:21 <abhishekk> I am not sure who has added that topic under open discussion
14:15:24 <smcginnis> abhishekk: We can do the glance_store stable branch separately. It's easier when it's done with the "final" release for the cycle, but not a requirement.
14:15:25 <abhishekk> may be Steap ?
14:15:49 <abhishekk> smcginnis, cool, thank you
14:16:35 <Steap> oh yeah that was me :)
14:16:42 <Steap> SOrry, forgot to add my name
14:17:06 <Steap> So I feel like it's not a very urgent topic, but some users complain that we can give gibberish to "--file" instead of an actual image
14:17:07 <abhishekk> ack, do you want to spread some lights on it?
14:17:16 <Steap> and would like some form of validation
14:17:26 <Steap> which might be done through a plugin
14:17:33 <Steap> I wonder is that topic has ever come up before?
14:17:46 <abhishekk> not that I am aware of
14:18:20 <Steap> so I understand the need, but I'm not sure exactly how important it is
14:18:26 <dansmith> presumably the only way we would validate, is if they say it's something like a qcow but isn't right?
14:18:33 <Steap> yeah
14:18:41 <dansmith> raw can be garbage by design
14:18:49 <Steap> yep, that is the issue
14:18:50 <dansmith> so we merged the image stream inspection code,
14:19:00 <Steap> the user gave me an example where they did "echo blabla > file.qcow"
14:19:02 <dansmith> which can detect that an image is at least on its face a given format,
14:19:11 <dansmith> and we could make that enforceable
14:19:37 <rosmaita> of course, you could have a qcow containing a bunch of garbage
14:19:40 <Steap> yes
14:19:42 <dansmith> i.e. stop accepting the stream once it gets far enough for us to determine that it's not the format they say it is
14:19:52 <Steap> I'm afraid that if we say "hey, we validate images", users might get their hopes up
14:19:54 <abhishekk> I am also not sure how this will be beneficial or overhead, but we can have a lite spec for the same
14:19:59 <rosmaita> so we just need to be clear that we are only validating the format
14:20:18 <Steap> this could be optional, with a --validate-format option
14:20:20 <dansmith> rosmaita: yeah, this would be more like a "I think you gave us the wrong file" sort of thing to save time and bandwidth and maybe user frustration trying to boot an instance they don't realize won't work
14:20:22 <Steap> and/or config option
14:20:49 <Steap> of course they could also write a shell script that does the validation bfore creatingthe image, in a true UNIX fashion
14:21:12 <abhishekk> Introducing config option for it doesn't sound good, or we can also do it at client side?
14:21:39 <Steap> why doesn't it sound good?
14:21:59 <dansmith> don't we already have a "reject images not in format X" option?
14:22:15 <dansmith> which just requires them to claim the image is the right type
14:22:30 <dansmith> having another one that says "...and check that they're honest" seems good to me
14:23:02 <Steap> the details could be discussed during the next PTG (or the one after that :p)
14:23:25 <abhishekk> I would suggest add this topic in PTG etherpad I just shared above
14:23:26 <Steap> I just wanted to make sure this was not a "hard NO" so that I can get back to a potentially impatient user :p
14:23:31 <Steap> abhishekk: yes
14:23:33 <Steap> will do
14:23:42 <abhishekk> Cool
14:24:34 <abhishekk> Do we have any other topic to discuss?
14:25:39 <abhishekk> Just for FYI, Monday 07 September will be US holiday and most of the people will not be around
14:25:50 <dansmith> yup
14:26:00 <abhishekk> So I would recommend to review cinder-glance patch between today and tomorrow
14:26:11 <abhishekk> we have devstack dependency on that patch as well
14:26:43 <abhishekk> I have reno as well config-referesh patch up as well
14:26:54 <dansmith> we should poke qa people about the devstack patch now I think
14:26:54 <abhishekk> So we only need release patch now
14:27:12 <abhishekk> yes, just poked them before the meeting
14:27:32 <dansmith> cool
14:28:11 <abhishekk> great, we are good for today then
14:28:40 <abhishekk> lets wrap this up and utilize this time for reviews and other work
14:28:53 <abhishekk> thank you all
14:28:56 <smcginnis> Thanks!
14:29:23 <abhishekk> #endmeeting