| *** bauzas_ is now known as bauzas | 00:12 | |
| *** bauzas_ is now known as bauzas | 01:44 | |
| *** bauzas_ is now known as bauzas | 08:07 | |
| *** bauzas_ is now known as bauzas | 12:08 | |
| *** haleyb|out is now known as haleyb | 13:34 | |
| pdeore | #startmeeting glance | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Thu Jun 20 14:00:19 2024 UTC and is due to finish in 60 minutes. The chair is pdeore. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'glance' | 14:00 |
| pdeore | #topic roll call | 14:00 |
| pdeore | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:00 |
| pdeore | o/ | 14:00 |
| dansmith | o/ | 14:00 |
| croelandt | o/ | 14:00 |
| abhishekk | o/ | 14:01 |
| pdeore | Mridula will not be able to join | 14:01 |
| pdeore | so let's get started | 14:01 |
| pdeore | #topic release/periodic jobs updates | 14:01 |
| pdeore | M2 is just 2 weeks from now which will be spec freeze date for us as well | 14:01 |
| pdeore | since we have 2 new specs recently so shall we extend our spec freeze date by a week? | 14:02 |
| abhishekk | We can do that | 14:02 |
| abhishekk | Unless those gets merged within next two weeks | 14:03 |
| pdeore | yeah I think we should do that .. | 14:03 |
| pdeore | I will send an email about the same then | 14:03 |
| dansmith | have they been undergoing active discussion? | 14:03 |
| dansmith | meaning, will the extra two weeks really matter? | 14:03 |
| abhishekk | yes, I have added some comments on new specs | 14:04 |
| croelandt | Are they Stephen's specs? | 14:04 |
| pdeore | New Calculate Hash API : #link https://review.opendev.org/c/openstack/glance-specs/+/922011 | 14:04 |
| pdeore | Add image-import-plugins-api spec: #link https://review.opendev.org/c/openstack/glance-specs/+/922116 | 14:04 |
| abhishekk | yeeah | 14:04 |
| abhishekk | I am ok with plugins spec but have concerns about hash calculation | 14:05 |
| pdeore | I have updated calculate hash as per your comments | 14:05 |
| abhishekk | ack | 14:05 |
| dansmith | ah those okay I haven't even started looking at those, but that makes sense now | 14:05 |
| pdeore | gr8 let's extend it then | 14:06 |
| abhishekk | I still don't understand the positive side of renaming existing parameter for which we need to follow deprecation process and introducing new one with ListOpt and also passing desired algorithm as input | 14:06 |
| abhishekk | may be I am not able to convey it in right manner, will try to rephrase it again on the spec | 14:07 |
| pdeore | ack | 14:08 |
| pdeore | moving ahead | 14:09 |
| pdeore | should we have review party next week ? | 14:10 |
| croelandt | hm it's also the bug hunt so I think we might miss at least Mridula and Brian | 14:10 |
| pdeore | ohh yeah | 14:11 |
| abhishekk | IMO we should drop the idea of review parties | 14:11 |
| abhishekk | it is not serving the purpose why we started it | 14:11 |
| abhishekk | the motive was to encourage people to attend it and start providing reviews | 14:12 |
| abhishekk | but I don't see that is happening | 14:12 |
| croelandt | It gets a bunch of small patches merged though | 14:12 |
| abhishekk | It's alway dan, cyril me and sometimes brian | 14:12 |
| croelandt | yes | 14:12 |
| abhishekk | that is not really good imo | 14:12 |
| croelandt | maybe it can be a bit more informal | 14:12 |
| abhishekk | For example, I was expecting reviews on new specs but I didn't saw any | 14:13 |
| abhishekk | same goes with image encryption spec, it was sitting ideal for last 3 weeks | 14:13 |
| pdeore | I will try to spend more time on reviews | 14:16 |
| croelandt | ok so no review party next week | 14:16 |
| abhishekk | so, will we be able to do weekly meeting because of bug hunt? | 14:17 |
| dansmith | I've got something else going on next week that will take top priority, so I might be around for stuff, but might not | 14:17 |
| abhishekk | ack, pdeore, croelandt ^^ | 14:18 |
| pdeore | not sure, will that be for every week ? | 14:18 |
| abhishekk | nope its only for next week | 14:19 |
| croelandt | abhishekk: I'll be there I trhink | 14:19 |
| abhishekk | but if you are not participating in bug hunt then you can chair | 14:19 |
| abhishekk | croelandt: ack, I will be around as well | 14:19 |
| pdeore | ohh ok ok, I will also there to chair this meeting | 14:20 |
| abhishekk | ok | 14:20 |
| pdeore | moving ahead | 14:21 |
| pdeore | periodic jobs are all green | 14:21 |
| pdeore | #topic Important Reviews | 14:21 |
| pdeore | so as discussed above we have these 2 imp specs to review | 14:22 |
| pdeore | New Calculate Hash API : #link https://review.opendev.org/c/openstack/glance-specs/+/922011 | 14:22 |
| pdeore | Add image-import-plugins-api spec: #link https://review.opendev.org/c/openstack/glance-specs/+/922116 | 14:22 |
| pdeore | and may be Dalmatian project priorities: #link https://review.opendev.org/c/openstack/glance-specs/+/916279 :) | 14:22 |
| pdeore | there are few bug fixes which we need to backport as well | 14:23 |
| pdeore | Fix 500 if multi-tenant swift is enabled along with conf file - #link https://review.opendev.org/c/openstack/glance/+/920170 | 14:23 |
| pdeore | Make location URL compatible with cinder backend : #link https://review.opendev.org/c/openstack/glance/+/906359/16 | 14:23 |
| pdeore | Fix: optimized upload volume in Cinder store: #link https://review.opendev.org/c/openstack/glance/+/922316 | 14:23 |
| pdeore | so kindly please have a look | 14:23 |
| pdeore | that's it from me for today | 14:24 |
| abhishekk | I am good with last patch, cyril can approve it | 14:24 |
| abhishekk | means, last two patches | 14:25 |
| croelandt | let me look | 14:25 |
| pdeore | gr8 | 14:25 |
| croelandt | https://review.opendev.org/c/openstack/glance/+/922316 <- Zuul unhappy | 14:25 |
| croelandt | same for https://review.opendev.org/c/openstack/glance/+/906359 | 14:26 |
| pdeore | how about the swift one? it already has 1 +2 from abhishek | 14:26 |
| croelandt | damn the CI is also not happy | 14:27 |
| croelandt | what's happening here :) | 14:27 |
| croelandt | tempest apparently | 14:27 |
| pdeore | seems mostly timeouts | 14:28 |
| pdeore | for #906359 everything was passing earlier | 14:29 |
| croelandt | ok I'll go through the list | 14:29 |
| pdeore | Thanks! | 14:29 |
| pdeore | anything else to be highlighted? | 14:30 |
| abhishekk | yeah, my oslo.middleware patch also need reviews form glance side | 14:30 |
| abhishekk | https://review.opendev.org/c/openstack/oslo.middleware/+/920055?usp=search | 14:31 |
| abhishekk | this one | 14:31 |
| croelandt | oh yeah, easy to forget about this one | 14:31 |
| abhishekk | :d | 14:32 |
| pdeore | ohh i forgot to change the topic :/ | 14:34 |
| pdeore | #topic Open Discussions | 14:34 |
| abhishekk | nothing from me | 14:35 |
| croelandt | nothing from me | 14:36 |
| pdeore | cool, let's conclude for the day then .. | 14:36 |
| pdeore | Thanks everyone for joining !! | 14:37 |
| pdeore | #endmeeting | 14:37 |
| opendevmeet | Meeting ended Thu Jun 20 14:37:26 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:37 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2024/glance.2024-06-20-14.00.html | 14:37 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2024/glance.2024-06-20-14.00.txt | 14:37 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2024/glance.2024-06-20-14.00.log.html | 14:37 |
| *** Guest10177 is now known as dasm | 15:15 | |
| *** jph5 is now known as jph | 17:46 | |
| *** bauzas_ is now known as bauzas | 18:02 | |
| *** bauzas_ is now known as bauzas | 18:14 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!