14:01:14 <croelandt> #startmeeting glance
14:01:14 <opendevmeet> Meeting started Thu Jun 26 14:01:14 2025 UTC and is due to finish in 60 minutes.  The chair is croelandt. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:01:14 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:01:14 <opendevmeet> The meeting name has been set to 'glance'
14:01:18 <croelandt> #topic roll call
14:01:18 <croelandt> o/
14:01:37 <priteau> o/
14:02:15 <abhishek_> o/
14:03:00 <croelandt> not many people around, but let's go :)
14:03:05 <croelandt> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:03:19 <croelandt> #topic Periodic jobs
14:03:32 <croelandt> glance-multistore-cinder-import-fips is failing because it required Python3.10, and our node only has 3.9
14:03:36 <croelandt> so 2 ways of fixing this:
14:03:40 <croelandt> 1) switch to ubuntu-noble
14:03:56 <croelandt> 2) wait for centos-10 to be available in the infra (and send patches so it gets there faster)
14:04:07 <croelandt> I'm not sure why we use centos for this job, does anyone know?
14:05:18 <abhishek_> Ubuntu itself does not natively include FIPS
14:05:36 <croelandt> oh
14:05:41 <croelandt> yep that explains it
14:05:50 <croelandt> ok a centos 10 image is available in diskimage-builder
14:05:58 <croelandt> and we need patches in other projects
14:06:04 <croelandt> I was looking at it earlier, so I'll try and send patches
14:06:20 <croelandt> Moving on
14:06:26 <croelandt> #topic M2 release 2025-07-03 (in 1 week)
14:06:32 <croelandt> #link https://review.opendev.org/q/topic:%22bug/2110185-all-backends%22
14:06:37 <croelandt> So ^ is our top priority
14:06:51 <croelandt> dansmith: rosmaita: do you think you'll have time to review these in the coming days?
14:07:01 <dansmith> sorry, lost track of time
14:07:08 <croelandt> I could merge them with only my +2 but they are very impactful patches and I'd love to have a second opinion
14:07:13 <croelandt> dansmith: happens to all of us :-(
14:07:22 <dansmith> I'm out tomorrow, here next week and then gone the following
14:07:32 <dansmith> but yeah, I know I need to get back to those
14:07:52 <dansmith> I'll try to do that today, but keep hounding me if not so we can get them done by next week
14:08:04 <dansmith> (I give you permission to nag) :P
14:08:04 <croelandt> the release is on the 3rd
14:08:17 <croelandt> so I guess I need to send the patch to openstack/releases on the 2nd?
14:08:25 <abhishek_> yes
14:08:31 <dansmith> ack
14:09:00 <croelandt> if we don't have common config options from Rajat, that is not optimal, but fine
14:09:28 <croelandt> #topic Open Discussion
14:09:32 <abhishek_> dansmith: IF you review one rest are the same for other backends, I think rbd and s3 should be good to review first
14:09:50 * dansmith nods
14:09:52 <abhishek_> croelandt: could you please review my pending functional test migration patch?
14:10:09 <rajiv> hi, i wanted feedback on https://review.opendev.org/c/openstack/glance-specs/+/953435
14:10:25 <abhishek_> https://review.opendev.org/c/openstack/glance/+/947845
14:10:31 <croelandt> abhishek_: yes, will do right after this meeting
14:11:17 <priteau> Hello. I was asked by abhishek_ if I could join the meeting today to discuss a patch
14:11:32 <abhishek_> croelandt: thanks
14:11:34 <croelandt> rajiv: are you willing to do the dev work on this?
14:12:24 <rajiv> sure, need little guidance, is this a valid request ?
14:12:45 <rajiv> the background is to control image visibility per domain
14:13:05 <croelandt> rajiv: well I guess users requested this?
14:14:28 <abhishek_> priteau: this is related to fix directly proposed to stable branch, right?
14:15:02 <croelandt> I am not sure whether OpenStack is "evolving towards better domain-scoped operations" but I am also not an admin for any real OpenStack deployment :)
14:15:15 <rajiv> croelandt: yes, i was looking for options to hide images, --tag and --hidden were not accurate as policy changes
14:15:39 <priteau> abhishek_: I submitted it to master but I wanted to discuss if it would even be considered for backports.
14:15:59 <priteau> Happy to give more context but I don't want to interrupt the other discussion
14:16:16 <croelandt> rajiv: if I were you I'd add links to explain how and why we evolve towards...
14:16:23 <croelandt> because I honestly have no idea about that topic
14:16:27 <abhishek_> yes, you can after the current one
14:17:09 <croelandt> rajiv: this also seems like something we'll end up discussing at the next PTG, but having a good spec and a PoC by then will help
14:17:45 <rajiv> okay, i will update the spec. the current policy only supports project_id bases scoping, i wanted to enhance it to domain_id based like other openstack services.
14:18:29 <croelandt> so point out to the documentation of these other services
14:18:33 <croelandt> prove that we are lagging behind
14:19:21 <rajiv> okay
14:20:37 <croelandt> Also run tox locally, that will help catch issues that the CI pointed out :)
14:22:41 <croelandt> priteau: ok, you have the mic :)
14:22:51 <priteau> I submitted https://review.opendev.org/c/openstack/glance_store/+/946382 a few months ago to work around an issue we have with a customer. They are using Glance with S3 backend, the S3 being NetApp StorageGRID. When we create an image from a Cinder volume, the upload to S3 is done without multipart upload. It works but StorageGRID is logging these errors: A client is
14:22:53 <priteau> attempting a PUT Object operation that exceeds S3 size limits: https://docs.netapp.com/us-en/storagegrid/troubleshoot/troubleshoot-s3-put-object-size-too-large-alert.html
14:24:08 <priteau> I understand this is because Glance doesn't know the size of the Cinder volume and thus doesn't enable multipart upload, even though the volume is bigger than the threshold
14:25:01 <priteau> The size is known as 0 during the upload, so I made this patch which just enables to force multipart upload for 0-sized images
14:25:29 <croelandt> You tested this and it solves your issue?
14:26:13 <priteau> It appears to be (I don't have direct access to this NetApp, but got told that the error went away). Although this is clearly just a workaround, and I understand that abhishek_ is working on a 2025.2 spec for fixing it properly.
14:26:37 <priteau> I was just wondering if there is any chance my patch would be approved + backportable?
14:27:15 <croelandt> so if abhishek_ is working on a proper fix, we might not want to work around it
14:27:30 <croelandt> priteau: what distro are you using? Would they be opened to patch glance_store?
14:27:58 <priteau> Patching is not a problem, we use our own Kolla containers
14:28:03 <croelandt> oh ok
14:28:16 <croelandt> abhishek_: what do you think about this patch? Is it not the "proper" way of fixing the issue?
14:28:23 <croelandt> Is this going to be solved with your size patch series?
14:28:26 <priteau> It's more about fixing it for other people, especially for those who will be on 2025.1 or 2024.1 for a while
14:28:44 <abhishek_> I think it will force to use multipart upload for all images
14:29:24 <croelandt> priteau: so ideally, two things
14:29:30 <croelandt> 1) I think you should patch your containers for now
14:29:49 <croelandt> 2) If you could try https://review.opendev.org/c/openstack/glance_store/+/949129 that would help us a lot
14:30:01 <croelandt> we are planning to release glance_store with this patch next week
14:30:53 <abhishek_> croelandt: I don't know whether we allow fixing bugs directly towards stable branches
14:31:44 <priteau> croelandt: thanks, I could try patch 949129. Is it independant or do I need to pull others?
14:31:58 <croelandt> abhishek_: yes, we'll have to figure somehting later for stable branches
14:32:19 <croelandt> abhishek_: does priteau need to cherry-pick something else?
14:32:29 <abhishek_> priteau: it's independent, you can apply s3 patch only in your env
14:32:47 <priteau> OK, I can give it a try.
14:32:52 <priteau> Thanks
14:33:18 <croelandt> if you can easily do that and come back to tell us whether that works, it would be really helpful
14:33:45 <priteau> Will do
14:34:25 <croelandt> \o/
14:34:28 <croelandt> Do we have anything else?
14:37:06 <croelandt> I think not, thanks everyone for joining!
14:37:08 <croelandt> #endmeeting