Thursday, 2023-06-01

pdeore#startmeeting glance13:59
opendevmeetMeeting started Thu Jun  1 13:59:26 2023 UTC and is due to finish in 60 minutes.  The chair is pdeore. Information about MeetBot at http://wiki.debian.org/MeetBot.13:59
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.13:59
opendevmeetThe meeting name has been set to 'glance'13:59
pdeore#topic roll call13:59
pdeore#link https://etherpad.openstack.org/p/glance-team-meeting-agenda13:59
pdeoreo/13:59
abhishekko/13:59
dansmitho/13:59
pdeorelets wait few minutes for others to join14:00
mrjoshi_o/14:01
pdeoreok let's start, Cyril is on leave14:01
pdeore#topic release/periodic jobs updates14:01
abhishekkI think rosmaita is in another meeting14:01
pdeoreyeah14:01
pdeoreM2 is 1 month from now i.e, in first week of July which will be spec freeze for us as well14:02
pdeoreSo, Kindly please give your some time on spec reviews before the deadline :)14:02
pdeorewe will discuss about the specs in detail in next topic14:03
pdeorePeriodic job all green except some functional-py39 jobs failing intermittently ..14:03
abhishekkneed to check those failures ^^14:03
rosmaitao/14:03
pdeoreyeah will check that14:04
pdeoremoving to next 14:04
pdeore#topic Important reviews14:04
pdeoreAs discussed during PTG regarding the RBD trash feature, eharney has submitted the patch in last week14:05
pdeore#link https://review.opendev.org/c/openstack/glance_store/+/88452414:05
pdeorenow we would need to work on the spec for the same14:05
eharneyyes should i start a draft of that shortly?14:05
pdeoreeharney, yeah sure14:06
eharneyok, will do14:06
abhishekkeharney, thanks, I have added couple of questions on the patch, please confirm14:06
pdeoreeharney, thanks !14:06
abhishekk*when you have time14:06
eharneysure, thanks, will give them a look today14:07
abhishekkack, thank you!14:07
pdeorelet's move to the next topic 14:08
pdeore#topic Specs14:08
pdeoreWe have few specs up for review which need cores attention, so kindly please have a look,14:08
pdeore#link https://review.opendev.org/c/openstack/glance-specs/+/88349114:09
dansmithI think we (and when I say we I mean you) should merge the location api spec update :)14:09
pdeoreRajat has updated the new location api spec please have a look, I hope this would be the final design now :P14:09
pdeoredansmith, yeah ofcourse :D but need 2 +2 again :D14:10
abhishekkEven while implementing if we found loophole we can amend it again14:10
dansmithpdeore: you've got them14:10
abhishekkit has 2 +2s14:10
pdeoreohh i missed that :/14:10
dansmithrosmaita was -1 only on a formatting thing also, so I expect he would be +2 on the current version as well14:10
pdeorerosmaita, if you have time please have a final look on this, so that I will approve it :)14:13
rosmaitaack14:13
pdeoreok, next is 14:13
pdeore#link https://review.opendev.org/c/openstack/glance-specs/+/881951 - Spec Lite: Add new location strategy14:13
pdeoreabhishekk, ohh you updated it ? Thanks !14:14
* abhishekk learning from dansmith to work fast :D14:14
dansmithlol14:14
pdeoreand i more we have is , #link https://review.opendev.org/c/openstack/glance-specs/+/880627 14:14
pdeoreRepropose Spec for the Glance part of Image Encryption (Bobcat) 14:15
abhishekkfor this you need Luzi to confirm14:15
pdeoreabhishekk, :D 14:15
pdeoreabhishekk, yeah14:15
pdeoreohh i missed to add her on this , will do that 14:15
abhishekkack14:15
pdeoreok, i think that's it from me14:16
pdeoremoving to next topic 14:16
pdeore#topic Missing rbac tests in plugin14:16
pdeoreyes we have location api, cache api, import api  missing in plugin14:17
abhishekkWe have some tests missing in tempest plugin for some policies, i think those needs to be added14:17
pdeoredo we need to add tests for existing location apis? since we are anyways going to add for new loc api ?14:18
abhishekkyes, since it will be there for next two cycles (or more)14:18
dansmithyes agree14:18
pdeorehmmm ok :/14:18
abhishekkthat's it, I think we should cross check plugin tests with API documentation14:19
pdeoreabhishekk, yes14:20
pdeoreI will try to add these missing test while adding tests for new location apis 14:21
dansmithpriority should be the new location implementation though I'd say14:22
pdeoredansmith, yeah of course14:23
pdeoreThat's it from me, let's move to open discussions14:24
pdeore#topic Open Discussions14:24
abhishekkdansmith, I have one question for you related to unified limits14:24
abhishekkwe have default value say 1000 for image_size_total14:25
abhishekkI want to increase the default size, is it possible?14:25
dansmithin devstack you mean?14:25
abhishekkanywhere, lets say devstack14:26
abhishekkLooking at current documentation it is not possible14:26
dansmithwhy do you say that?14:26
abhishekkhttps://docs.openstack.org/api-ref/identity/v3/?expanded=update-registered-limit-detail#update-registered-limit14:26
abhishekkit requires either service_id, region or resource_name different14:26
abhishekkI don't know the reason behind this condition though14:27
dansmiththat's only if you want to change them right? they're optional14:27
abhishekkAs of now, once we registered limit through default value, we will not able to increase or decrease it14:28
dansmithif you specify the limit by id, you can change just the default_limit14:28
abhishekkno it gives 409 error14:28
dansmiththen that's a keystone (or osc) bug I think based on that description14:28
dansmithwhat command are you running?14:28
abhishekkThe UUID of the service to update to which the registered limit belongs. Either service_id, resource_name, or region_id must be different than existing value otherwise it will raise 409.14:29
dansmithbut those are optional14:29
abhishekkopenstack registered limit set14:29
dansmithif you provide them, you're saying "change X to X" which is why it 409s I think14:29
dansmithabhishekk: the full command :)14:29
abhishekkohhh14:30
abhishekklet me paste full command, I have not tried just passing limit id and default limit14:30
dansmith... :)14:30
abhishekkI will try it and check with you offline14:31
dansmithtbh I don't like that 409 behavior because it means deployment tools can't be idempotent, but alas :/14:31
dansmithabhishekk: cool14:31
abhishekkopenstack --os-cloud devstack-system-admin registered limit set 3b75aca754c44cd9b7ff836579cfb9aa --default-limit 20014:32
abhishekkthis is working 14:32
dansmithcool I thought so :)14:32
abhishekkthanks!!14:32
abhishekkThat's it from me14:32
pdeoreack, anyone has anything else to discuss? 14:33
dansmithnot me14:33
pdeorecool14:33
pdeorelet's conclude for the day 14:34
abhishekkthank you all!14:34
pdeoreThanks everyone for joining!!14:34
pdeore#endmeeting14:35
opendevmeetMeeting ended Thu Jun  1 14:35:15 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:35
opendevmeetMinutes:        https://meetings.opendev.org/meetings/glance/2023/glance.2023-06-01-13.59.html14:35
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/glance/2023/glance.2023-06-01-13.59.txt14:35
opendevmeetLog:            https://meetings.opendev.org/meetings/glance/2023/glance.2023-06-01-13.59.log.html14:35

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!