14:00:01 <abhishekk> #startmeeting glance
14:00:01 <opendevmeet> Meeting started Thu Feb 24 14:00:01 2022 UTC and is due to finish in 60 minutes.  The chair is abhishekk. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:01 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:01 <opendevmeet> The meeting name has been set to 'glance'
14:00:03 <abhishekk> #topic roll call
14:00:11 <abhishekk> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:00:16 <abhishekk> o/
14:00:31 <jokke_> o/
14:00:37 <pdeore> o/
14:00:40 <rosmaita> o/
14:00:45 <mrjoshi> o/
14:00:49 <abhishekk> cool, lets start
14:00:55 <abhishekk> #topic Updates
14:01:22 <abhishekk> Zed PTG planning etherpad is up, you can add your interested topics in belpw etherpad
14:01:29 <abhishekk> #link https://etherpad.opendev.org/p/zed-ptg-glance-planning
14:02:03 <abhishekk> I will finalize the topics 1 week before PTG i.e last week of March
14:02:13 <abhishekk> #topic release/periodic jobs update
14:02:50 <abhishekk> this is M3 release week, but we are going to skip it and tag rc1 directly next week
14:03:05 <abhishekk> Client release patch is up and will be merge today
14:03:18 <abhishekk> #link https://review.opendev.org/c/openstack/python-glanceclient/+/829653
14:04:06 <abhishekk> Periodic job, 1 failure today and 2 failures yesterday, test_delayed_delete is failing with lock on SQLALchemy db
14:04:39 <abhishekk> I will run it locally to see whether it is recurring or not
14:05:05 <abhishekk> I need to prepare final release note patch for glance release
14:05:25 <abhishekk> will do it shortly after the meeting and add you guys as a reviewer
14:05:45 <abhishekk> moving ahead
14:05:48 <abhishekk> #topic glanceclient-dsvm-functional job
14:06:09 <abhishekk> 2 days before croelandt asked me why don't we have functional tests on client side
14:06:14 <abhishekk> and I found this job
14:06:28 <abhishekk> but to my surprise it is running unit tests
14:06:41 <abhishekk> #link https://c084ebf856caeb602128-bb81120481bfc1a9d88d226642da11d8.ssl.cf1.rackcdn.com/800172/7/gate/glanceclient-dsvm-functional/ba0d0b5/testr_results.html
14:07:05 <abhishekk> jokke_, rosmaita do you have any idea about this job and why it is running unit tests
14:07:12 <rosmaita> looking
14:07:24 <abhishekk> because we are setting entire openstack and it runs unit tests which makes no sense
14:08:52 <jokke_> I have no idea about that test job, I think the reason we do not have functional run for gc is because we practically have no functional tests for it
14:09:13 <abhishekk> we do have couple of tests in functional directory
14:09:48 <jokke_> Yes, Louis started to write them years ago, but that work got never carried on
14:10:04 <jokke_> there is couple, that's why "practically no functional tests"
14:10:08 <rosmaita> the job def in .zuul.yaml doesn't set a tox_envlist
14:10:18 <rosmaita> it inherits from devstack-tox-functional
14:10:29 <rosmaita> which should call tox -e functional
14:10:41 <rosmaita> (looking at what's in tox.ini)
14:11:05 <abhishekk> right, but I think it's not calling it
14:11:40 <jokke_> Yeah, not sure where that is coming from but if it's dsvm job just running unit tests, we should get rid of it
14:11:52 <abhishekk> I think we should retire this job until we have some useful functional tests
14:11:53 <rosmaita> no, we should fix it to run the functional tests
14:13:04 <abhishekk> rosmaita, we can add it back once we have valid/reasonable functional tests
14:13:30 <abhishekk> at this moment it sounds like wasting resources
14:15:29 <rosmaita> let's see if this does anything: https://review.opendev.org/c/openstack/python-glanceclient/+/830824
14:15:42 <abhishekk> I will have a look and find out whether it is easy to add more tests or it will need whole restructuring to get this job do what it is intended to do
14:15:44 <abhishekk> rosmaita, ak
14:15:46 <abhishekk> ack
14:16:09 <rosmaita> i'm pretty sure it used to work
14:16:34 <abhishekk> ok, we will revisit this topic next week
14:16:35 <jokke_> abhishekk: I agree, it spends half an hour of infra resources to set up environment if we run the functional part, that consists literally 1 test excercising that devstack it sets up
14:17:17 <jokke_> the 2 more tests there are testing just cli (help and version)
14:17:29 <abhishekk> yeah, lets see the output of rosmaita's patch and we can decide next week
14:17:46 <abhishekk> anyways now there will be no patch we are going to submit for client
14:18:16 <abhishekk> We can take the decision in next meeting about it
14:18:27 <abhishekk> that's all from me
14:18:43 <abhishekk> moving to Open discussion
14:18:50 <abhishekk> #topic Open discussion
14:19:25 <abhishekk> can any one of you submit the final release patch for glance by next week
14:19:41 <jokke_> I can do that
14:19:47 <jokke_> You're gonna be on PTO?
14:19:50 <abhishekk> I might not be around on Monday and Tuesday for some urgency
14:19:56 <jokke_> kk
14:20:00 <jokke_> sure thing
14:20:15 <abhishekk> yeah, most probably
14:20:19 <abhishekk> thank you
14:20:34 <abhishekk> I will submit the reno patch tonight and then we are good to go for release
14:20:46 <jokke_> cool
14:21:02 <abhishekk> anything else guys?
14:21:09 <jokke_> Nothing in our bug list that should have been addressed?
14:21:22 <jokke_> I don't think we have been doing scrubbing for a while
14:21:22 <abhishekk> nothing important
14:21:26 <jokke_> nice
14:21:59 * croelandt sends patches :D
14:22:16 <abhishekk> 2 weeks before croelandt and me gone through recent bugs
14:22:17 * jokke_ starts stretching -2 finger
14:22:32 <abhishekk> so we are good for this cycle
14:22:35 <croelandt> abhishekk: yeah I looked at a bunch of glanceclient patches
14:22:46 <croelandt> and there are leftover patches from the reveiw party that we could +2 :p
14:22:56 <croelandt> jokke_: do you have a dedicated finger for -2?
14:23:02 <jokke_> croelandt: ofc
14:23:04 <croelandt> I wonder which one it could be
14:23:12 <jokke_> the middle one
14:23:12 <abhishekk> haha
14:23:21 <jokke_> ;P
14:23:47 * croelandt was not really wondering
14:24:01 <croelandt> next gerrit update: the -2 red button is replaced with a middle finger emoji
14:24:03 <jokke_> nothing more from me at least if we want to cut out early
14:24:13 <abhishekk> we can wrap up early if there is nothing to discuss
14:24:39 <abhishekk> croelandt, we can suggest this to rosmaita if he becomes TC
14:24:53 <croelandt> we should
14:25:09 <abhishekk> looking forward to it :D
14:25:25 <abhishekk> ok, lets wrap up for today
14:25:28 <abhishekk> thank you all!!
14:25:50 <abhishekk> have a nice weekend ahead
14:26:05 <jokke_> thanks all
14:26:16 <abhishekk> #endmeeting