| *** dasm is now known as dasm|off | 01:10 | |
| *** soniya is now known as soniya29|rover | 05:41 | |
| *** soniya29|rover is now known as soniya29|rover|brb | 09:44 | |
| *** soniya29|rover|brb is now known as soniya29|rover | 11:25 | |
| *** dasm|off is now known as dasm | 13:08 | |
| pdeore | #startmeeting glance | 13:59 |
|---|---|---|
| opendevmeet | Meeting started Thu Jan 19 13:59:42 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:59 |
| opendevmeet | The meeting name has been set to 'glance' | 13:59 |
| pdeore | #topic roll call | 13:59 |
| pdeore | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 13:59 |
| pdeore | o/ | 13:59 |
| mrjoshi__ | o/ | 14:00 |
| abhishekk | o/ | 14:01 |
| pdeore | lets wait few minutes for others to join | 14:01 |
| rosmaita | o/ | 14:01 |
| pdeore | let's start, others may join in between | 14:02 |
| pdeore | #topic release/periodic jobs updates | 14:02 |
| pdeore | Milestone 3 is 4 weeks away | 14:02 |
| pdeore | This is Spec freeze week and we have 2 important specs to get in, which we will see in next topic | 14:02 |
| pdeore | Periodic job all green except TIME_OUT for fips jobs | 14:03 |
| pdeore | moving to next | 14:03 |
| jokke_ | o/ | 14:03 |
| pdeore | #topic Specs | 14:03 |
| pdeore | #link https://review.opendev.org/c/openstack/glance-specs/+/863209 - Repropose new location APIs spec | 14:03 |
| pdeore | #link https://review.opendev.org/c/openstack/glance-specs/+/868901 - (spec-lite) (cinder) Add suport to allow extend attached volumes | 14:03 |
| pdeore | Both these specs looks good to me as per the discussion happened so far on this , so kindly please have a look if you have not yet reviewed the updated one | 14:04 |
| pdeore | today is the last day for approving specs, so if I don't see any objection on the specs today, tomorrow I will approve it | 14:05 |
| abhishekk | ack | 14:05 |
| pdeore | after tomorrow no new/old specs will be approved for this cycle | 14:06 |
| pdeore | so kindly please have a look | 14:06 |
| pdeore | moving to next | 14:07 |
| pdeore | #topic Catching up with reviews | 14:07 |
| pdeore | Update Migration Constant | 14:07 |
| pdeore | #link https://review.opendev.org/c/openstack/glance/+/861752/ | 14:07 |
| pdeore | abhishekk, has updated this, kindly please have a look | 14:07 |
| pdeore | next is | 14:08 |
| pdeore | Pin glance-tempest-plugin with 0.2.0 for wallaby job | 14:08 |
| pdeore | #link https://review.opendev.org/c/openstack/glance-tempest-plugin/+/864005 | 14:08 |
| pdeore | this is blocking other stable/wallaby patches as secure-rbac-protection-functional job is failing for stable/wallaby | 14:08 |
| abhishekk | this one is important ^^ | 14:08 |
| pdeore | yes, please have a look on priority | 14:09 |
| pdeore | that's it from me for today | 14:10 |
| rosmaita | looking at the glance-t-p patch now | 14:10 |
| pdeore | rosmaita, thanks ! | 14:10 |
| pdeore | shall we move to open discussion ? | 14:11 |
| abhishekk | yes | 14:12 |
| pdeore | #topic Open Discussion | 14:12 |
| abhishekk | rosmaita, i have got one +2 on service role patch, if you manage to contact keystone guys then we can get it merged and start on glance location api work | 14:12 |
| abhishekk | https://review.opendev.org/c/openstack/keystone/+/863420 | 14:12 |
| rosmaita | arrrrrrrrgh .... i thought that was taken care of! | 14:12 |
| jokke_ | As we have quite a bit of time I'd like to discuss a bit of the Rajat's cinder expand proposal. Anyone up for it? | 14:13 |
| abhishekk | I think rajat will be busy in cinder midcycle stuff | 14:14 |
| jokke_ | maybe rosmaita can aswer quickly couple of my questions. | 14:15 |
| pdeore | jokke_, i think it would be better to comment direct on the patch, I will ping whoami-rajat after meeting for the same | 14:15 |
| whoami-rajat | I'm here but not sure what the context is | 14:16 |
| abhishekk | lite spec | 14:16 |
| jokke_ | whoami-rajat: ok, cool. Your lite-spec about the cinder extends | 14:16 |
| whoami-rajat | sure | 14:16 |
| whoami-rajat | happy to answer any concerns | 14:17 |
| jokke_ | I'm just wondering if my assumptions are correct: 1) we have no programmatic way to know if the volume happens to support the live expansion 2) we cannot shrink the volume afterwards (or what's the reason we're stuck with 1Gb expansions)? | 14:17 |
| whoami-rajat | 1) no, we don't, it's a backend feature which isn't reported anywhere so the info is only available in our support matrix and the operator using backend is expected to know if their backend supports this capability | 14:19 |
| whoami-rajat | having said that, in current master, apart from nfs i think all backends support it | 14:19 |
| * whoami-rajat quickly checks | 14:19 | |
| jokke_ | Just wondering if 1 is correct, should we do similar what we do with rdb and ramp up the amount we expand and trim once we're done writing. | 14:19 |
| whoami-rajat | https://docs.openstack.org/cinder/latest/reference/support-matrix.html#operation_online_extend_support | 14:19 |
| whoami-rajat | there are few of them but i haven't seen those drivers used a lot recently | 14:20 |
| whoami-rajat | 2) no we cannot, cinder only supports extend and not shrink | 14:20 |
| jokke_ | ohh | 14:20 |
| jokke_ | I had no idea | 14:20 |
| whoami-rajat | yeah, cinder needs to cater for all backend use cases and some backends don't support shrink so it didn't make sense for cinder to do it either | 14:21 |
| whoami-rajat | even though some backends support it like rbd, we can't generalize it for all backends | 14:21 |
| jokke_ | I was worried that the manual config would be error prone, but I guess it's pretty much the only way to improve things then :( | 14:21 |
| jokke_ | whoami-rajat: well, you do online extends even all backends don't support it either ;) | 14:22 |
| whoami-rajat | i know the 1GB expansion isn't great but that's the only way we can get accurate volume size and not overcommit storage | 14:22 |
| jokke_ | do we reserve correct size volume to start with _if_ we know the image size beforehand (Imports)? | 14:23 |
| whoami-rajat | the add method in cinder store does get a size parameter but I've no idea how it gets populated, I haven't seen in my testing the size being used and it always starts with 1 | 14:24 |
| whoami-rajat | but if it's passed, cinder store should use that as initial volume size | 14:24 |
| whoami-rajat | https://github.com/openstack/glance_store/blob/master/glance_store/_drivers/cinder.py#L960-L962 | 14:25 |
| jokke_ | ok, could you quickly add that to the spec as secondary improvement objective (so we don't need another spec for that)? As the volume operations are that heavy I think we should look into reserving correct size volume in the first place if we do have the image file on hand so we know it's size | 14:26 |
| jokke_ | I'd be happy to +2 that as whole | 14:26 |
| whoami-rajat | I can but is it a future effort or expected to be done in the same effort? since it seems like a different issue of size not being passed correctly | 14:27 |
| jokke_ | oh sorry,, missed that while I was writing | 14:28 |
| jokke_ | so all we need is to make sure we actually populate that parameter from glance side, it would not need changes in the store | 14:28 |
| jokke_ | then the proposal is fine as it is as sounds like it's all we really can do | 14:28 |
| whoami-rajat | yes, correct | 14:28 |
| whoami-rajat | ok i will add that to the spec | 14:28 |
| whoami-rajat | thanks for the feedback | 14:29 |
| jokke_ | amazing, thanks | 14:29 |
| pdeore | whoami-rajat, Thanks for joining ! | 14:29 |
| jokke_ | whoami-rajat: and sory for distracting you from the midcycle efforts | 14:30 |
| whoami-rajat | pdeore, no problem, thanks! | 14:30 |
| whoami-rajat | jokke_, oh midcycle was yesterday, but I've another meeting to attend | 14:30 |
| whoami-rajat | thanks guys! | 14:30 |
| jokke_ | cheers, I have nothing else for now | 14:31 |
| pdeore | anyone has anything else to discuss ? or we can wrap up early | 14:31 |
| abhishekk | #metoo | 14:31 |
| pdeore | cool | 14:31 |
| pdeore | Thanks everyone for joining ! | 14:31 |
| jokke_ | tyty | 14:31 |
| abhishekk | thank you! | 14:31 |
| pdeore | #endmeeting | 14:31 |
| opendevmeet | Meeting ended Thu Jan 19 14:31:40 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:31 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2023/glance.2023-01-19-13.59.html | 14:31 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2023/glance.2023-01-19-13.59.txt | 14:31 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2023/glance.2023-01-19-13.59.log.html | 14:31 |
| *** dasm is now known as Guest1846 | 15:29 | |
| *** Guest1846 is now known as dasm | 15:30 | |
| *** dasm is now known as dasm|off | 23:13 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!