| *** zbr is now known as Guest2488 | 05:03 | |
| *** rpittau|afk is now known as rpittau | 07:22 | |
| *** akekane__ is now known as abhishekk | 08:04 | |
| abhishekk | #startmeeting glance | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Thu Jun 17 14:00:08 2021 UTC and is due to finish in 60 minutes. The chair is abhishekk. 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 |
| abhishekk | #topic roll call | 14:00 |
| abhishekk | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:00 |
| abhishekk | o/ | 14:00 |
| jokke_ | o/ | 14:00 |
| dansmith | o/ | 14:00 |
| abhishekk | lets wait couple of minutes for others | 14:00 |
| Steap_ | o/ | 14:01 |
| *** pdeore_ is now known as pdeore | 14:01 | |
| abhishekk | may be rosmaita is busy in other meeting | 14:02 |
| pdeore | o/ | 14:02 |
| abhishekk | he is here | 14:02 |
| abhishekk | lets start | 14:02 |
| rosmaita | o/ | 14:02 |
| abhishekk | #topic release/periodic jobs update | 14:02 |
| rosmaita | sorry for being late, and sorry in advance for not paying attention | 14:02 |
| abhishekk | 4 weeks for M2 | 14:02 |
| abhishekk | no worries | 14:03 |
| abhishekk | Means we have 3 weeks for quotas and cache work | 14:03 |
| abhishekk | Periodic jobs, 1 failure related to image failed to reach active state in 15 seconds | 14:03 |
| abhishekk | might be due to some non related issues, I will go through logs | 14:04 |
| abhishekk | moving to next topic | 14:04 |
| abhishekk | #topic M2 Target progress check | 14:04 |
| abhishekk | Unified quota spec | 14:04 |
| abhishekk | Patches are up and we need reviews from volunteer reviewers | 14:04 |
| abhishekk | Its in good shape now | 14:04 |
| abhishekk | So kindly review, rosmaita, Steap_ ^^ | 14:05 |
| abhishekk | Cache API | 14:05 |
| abhishekk | Spec is approved, client spec is under review | 14:05 |
| abhishekk | We have 3 weeks to complete the work | 14:05 |
| dansmith | I haven't seen any more movement on the server side cache code, did I miss it? | 14:05 |
| jokke_ | dansmith: nope, haven't got to it yet | 14:06 |
| abhishekk | jokke_, please let me know if you need additional resource | 14:06 |
| dansmith | okay, just didn't want to be ignoring it if so | 14:06 |
| abhishekk | I know you are busy with glance/swift stuff | 14:06 |
| abhishekk | I can volunteer myself for client side work | 14:07 |
| jokke_ | cheers, next week should be fine to look into that too. Have some other things on my plate I need to get done by tomorrow afternoon | 14:07 |
| abhishekk | cool, lets sync next week then | 14:07 |
| abhishekk | moving ahead | 14:07 |
| abhishekk | pdeore, is also with us now, so we can share the work | 14:08 |
| abhishekk | moving ahead, | 14:08 |
| abhishekk | #topic Policy refactoring | 14:08 |
| abhishekk | With help of dansmith and his PoC, I have written a spec for policy refactoring work | 14:09 |
| abhishekk | #link https://review.opendev.org/c/openstack/glance-specs/+/796753 | 14:09 |
| abhishekk | #link https://review.opendev.org/q/topic:%22policy-poc%22+(status:open%20OR%20status:merged) | 14:09 |
| abhishekk | This is going to be big task and we need all possible help to achieve this goal | 14:09 |
| dansmith | s/big/massive/ | 14:10 |
| abhishekk | So kindly have a look at the spec and PoC | 14:10 |
| abhishekk | We need to start putting active efforts into this ASAP | 14:10 |
| abhishekk | any questions around this? | 14:11 |
| abhishekk | I take that as no, I guess next week we will have more discussion around this | 14:11 |
| abhishekk | moving ahead | 14:11 |
| abhishekk | #topic Launchpad and Gerrit team restructuring | 14:12 |
| abhishekk | Last week when rosmaita was not around, I noticed that we were unable to set/add new release or milestones in launchpad | 14:12 |
| abhishekk | He was the only active launchpad admin for glance till then | 14:13 |
| abhishekk | With help of fungi I have added myself and jokke_ to it | 14:13 |
| abhishekk | There are lots of people who are not active and yet are LP administrator for glance | 14:13 |
| rosmaita | sorry, i didn't realize how important i was | 14:13 |
| abhishekk | :D | 14:14 |
| abhishekk | Same thing goes for our gerrit core reviewers team | 14:14 |
| abhishekk | I would like to remove/cleanup both the teams | 14:14 |
| rosmaita | i thought any core member can administer the core team? | 14:14 |
| Steap_ | rosmaita: we were quite surprised as well | 14:14 |
| abhishekk | core team can administer in gerrit but LP administrator is different | 14:15 |
| jokke_ | rosmaita: yeah, gerrit teams are bit different. We just have people like Nikhil still in glance core | 14:15 |
| rosmaita | ah, ok | 14:15 |
| abhishekk | If all are onboard then I will cleanup our both the teams after the meeting | 14:16 |
| jokke_ | ++ | 14:16 |
| fungi | team members can be set as administrators by other administrators | 14:17 |
| fungi | i recommend having several | 14:17 |
| abhishekk | yes, I want to remove only those which are not active | 14:17 |
| fungi | and at least making sure to update it to include the ptl as an administrator when there's a change in leadership | 14:17 |
| abhishekk | Yep, I made a note of this, and will also include it in our documentation somewhere | 14:18 |
| abhishekk | thanks fungi for your help | 14:18 |
| abhishekk | moving ahead | 14:18 |
| abhishekk | #topic Bi-weekly Bug discussion (3rd Meet) | 14:19 |
| abhishekk | Last week we skipped it as there were less members | 14:19 |
| abhishekk | Lets go through them one by one | 14:19 |
| abhishekk | #link https://bugs.launchpad.net/glance/+bug/1889664 | 14:19 |
| abhishekk | this was filed by jokke_ | 14:20 |
| Steap_ | DO we know when thre is going to be another eventlet release? | 14:21 |
| abhishekk | jokke_, could you please update the recent status of eventlet release on the bug? | 14:21 |
| jokke_ | I think we got rid of caps already, probably need to check the minimums before closing that off | 14:21 |
| jokke_ | abhishekk: Yeah, I'll add that to my next weeks todo list | 14:21 |
| abhishekk | ack, thank you | 14:22 |
| abhishekk | #link https://bugs.launchpad.net/glance/+bug/1843772 | 14:22 |
| abhishekk | glance-manage utility tool doesn't recognize config dirs | 14:23 |
| abhishekk | Do we need to fix this issue? | 14:24 |
| abhishekk | glance-manage has now nothing to do with any other config file than glance-api.conf | 14:24 |
| Steap_ | if glance-manage is not supposed to read any other file, then this bug feels invalid | 14:25 |
| Steap_ | doesn't it? | 14:25 |
| abhishekk | but, glance-manage help text shows --config-dir option | 14:26 |
| Steap_ | do we have --config-dir for any other utility? | 14:27 |
| abhishekk | Steap_, we removed registry so glance-manage now only need to read api.conf | 14:27 |
| abhishekk | AFAIK, no | 14:27 |
| jokke_ | I think it still initially defaults to glance-manage.conf, and then gapi.conf | 14:28 |
| Steap_ | maybe we just want to remove --config-dir | 14:28 |
| abhishekk | #action abhishekk to submit a patch to remove --config-dir option and defaults utility to read api.conf | 14:29 |
| abhishekk | Next one is trivial, so skipping it, I will submit a patch for that | 14:29 |
| abhishekk | Non existing property protection file raises 500 Internal server error Edit | 14:29 |
| jokke_ | but I'm not sure what is not working as the bug mentions conf.d dirs which IIRC we have never had anything to do with | 14:30 |
| Steap_ | apparently we read both glance-api.conf and glance-manage.conf | 14:30 |
| Steap_ | if I'm reading the source correctly | 14:30 |
| jokke_ | Steap_: yup, that's what I just said we do go through them both if they exists in the --config-dir specified | 14:31 |
| abhishekk | I think generally the utility looks for config files in 3 locations | 14:32 |
| abhishekk | 1. default one, 2. --config-dir path to config files and 3. If neither then from conf.d directory | 14:32 |
| abhishekk | Next one is doc issue | 14:34 |
| abhishekk | #link https://bugs.launchpad.net/glance/+bug/1930932 | 14:34 |
| abhishekk | this is also related to config file | 14:34 |
| abhishekk | I think this is deployment specific issue | 14:35 |
| jokke_ | yeah, that bug should be closed as invalid | 14:36 |
| abhishekk | cool | 14:37 |
| abhishekk | and last one is related to our policy refactoring stuff, I have added it as reference to my spec | 14:37 |
| abhishekk | https://bugs.launchpad.net/glance/+bug/1915582 | 14:37 |
| dansmith | ++ | 14:38 |
| abhishekk | So, what should be the status of this bug, as we are going to consider it as a feature | 14:38 |
| dansmith | confirmed? it's definitely correct :) | 14:38 |
| abhishekk | Ok, we need to keep it in mind to refer this bug in our patches in that case | 14:39 |
| dansmith | I dunno, | 14:39 |
| dansmith | we don't have to be that strict, IMHO, and we could also refer to it in the final doc patch or something | 14:40 |
| jokke_ | opinion maybe? | 14:40 |
| dansmith | opinion means we're not going to fix it (according to the description) | 14:40 |
| jokke_ | IMHO one should not have change perms without read but | 14:41 |
| dansmith | I dunno, you can have -w- perms on a dir in unix, but not r-- | 14:41 |
| dansmith | but either way, this is just an example of get/modify | 14:42 |
| abhishekk | OK, we can continue discussion on the spec :D | 14:42 |
| dansmith | the real confusion comes when you end up with things like get-image-location and get-image conflicting or something like that | 14:42 |
| dansmith | abhishekk: maybe mark the spec as related-bug to this and then be done with it? | 14:42 |
| abhishekk | hmm, sounds good | 14:42 |
| abhishekk | will do it in next revision | 14:42 |
| dansmith | cool | 14:43 |
| abhishekk | moving to Open discussion | 14:43 |
| abhishekk | #topic Open discussion | 14:43 |
| * abhishekk miss old bots :D | 14:43 | |
| dansmith | Steap_: would love to see you take another trip through the quota patches if you can, just to get your +1 on things | 14:43 |
| abhishekk | rosmaita, Steap_ reminder to review quota patches | 14:44 |
| Steap_ | dansmith: I will :) | 14:44 |
| dansmith | it'll help rosmaita when he gets there too, knowing more eyes have looked | 14:44 |
| rosmaita | ack | 14:44 |
| dansmith | Steap_: thanks :) | 14:44 |
| * rosmaita can use all the help he can get | 14:44 | |
| abhishekk | Nothing from me | 14:44 |
| jokke_ | I'm good too | 14:45 |
| Steap_ | I'm good | 14:45 |
| rosmaita | nothing from me | 14:45 |
| abhishekk | So just to inform all of you | 14:45 |
| abhishekk | pdeore, is going to help us in maintaining glance-tempest-plugin | 14:46 |
| rosmaita | \o/ | 14:46 |
| abhishekk | she is also going to increase her presence in glance in terms of reviews and contribution | 14:47 |
| Steap_ | \o/ | 14:47 |
| abhishekk | That's it | 14:47 |
| abhishekk | I guess we can wrap it then | 14:48 |
| abhishekk | thank you all | 14:48 |
| abhishekk | have a good weekend, and don't forget to review spec and patches :D | 14:48 |
| abhishekk | #endmeeting | 14:48 |
| opendevmeet | Meeting ended Thu Jun 17 14:48:56 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:48 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2021/glance.2021-06-17-14.00.html | 14:48 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2021/glance.2021-06-17-14.00.txt | 14:48 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2021/glance.2021-06-17-14.00.log.html | 14:48 |
| *** rpittau is now known as rpittau|afk | 16:09 | |
| *** akekane_ is now known as abhishekk | 16:44 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!