| *** mhen_ is now known as mhen | 01:51 | |
| opendevreview | Merged openstack/glance master: Remove glance-centralized-cache job https://review.opendev.org/c/openstack/glance/+/953120 | 04:53 |
|---|---|---|
| opendevreview | Rajiv Mucheli proposed openstack/glance-specs master: Add spec to support domain_id in Glance Policy https://review.opendev.org/c/openstack/glance-specs/+/953435 | 12:25 |
| croelandt | #startmeeting glance | 14:01 |
| 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'glance' | 14:01 |
| croelandt | #topic roll call | 14:01 |
| croelandt | o/ | 14:01 |
| priteau | o/ | 14:01 |
| abhishek_ | o/ | 14:02 |
| croelandt | not many people around, but let's go :) | 14:03 |
| croelandt | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:03 |
| croelandt | #topic Periodic jobs | 14:03 |
| croelandt | glance-multistore-cinder-import-fips is failing because it required Python3.10, and our node only has 3.9 | 14:03 |
| croelandt | so 2 ways of fixing this: | 14:03 |
| croelandt | 1) switch to ubuntu-noble | 14:03 |
| croelandt | 2) wait for centos-10 to be available in the infra (and send patches so it gets there faster) | 14:03 |
| croelandt | I'm not sure why we use centos for this job, does anyone know? | 14:04 |
| abhishek_ | Ubuntu itself does not natively include FIPS | 14:05 |
| croelandt | oh | 14:05 |
| croelandt | yep that explains it | 14:05 |
| croelandt | ok a centos 10 image is available in diskimage-builder | 14:05 |
| croelandt | and we need patches in other projects | 14:05 |
| croelandt | I was looking at it earlier, so I'll try and send patches | 14:06 |
| croelandt | Moving on | 14:06 |
| croelandt | #topic M2 release 2025-07-03 (in 1 week) | 14:06 |
| croelandt | #link https://review.opendev.org/q/topic:%22bug/2110185-all-backends%22 | 14:06 |
| croelandt | So ^ is our top priority | 14:06 |
| croelandt | dansmith: rosmaita: do you think you'll have time to review these in the coming days? | 14:06 |
| dansmith | sorry, lost track of time | 14:07 |
| 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 |
| croelandt | dansmith: happens to all of us :-( | 14:07 |
| dansmith | I'm out tomorrow, here next week and then gone the following | 14:07 |
| dansmith | but yeah, I know I need to get back to those | 14:07 |
| dansmith | I'll try to do that today, but keep hounding me if not so we can get them done by next week | 14:07 |
| dansmith | (I give you permission to nag) :P | 14:08 |
| croelandt | the release is on the 3rd | 14:08 |
| croelandt | so I guess I need to send the patch to openstack/releases on the 2nd? | 14:08 |
| abhishek_ | yes | 14:08 |
| dansmith | ack | 14:08 |
| croelandt | if we don't have common config options from Rajat, that is not optimal, but fine | 14:09 |
| croelandt | #topic Open Discussion | 14:09 |
| 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 |
| * dansmith nods | 14:09 | |
| abhishek_ | croelandt: could you please review my pending functional test migration patch? | 14:09 |
| rajiv | hi, i wanted feedback on https://review.opendev.org/c/openstack/glance-specs/+/953435 | 14:10 |
| abhishek_ | https://review.opendev.org/c/openstack/glance/+/947845 | 14:10 |
| croelandt | abhishek_: yes, will do right after this meeting | 14:10 |
| priteau | Hello. I was asked by abhishek_ if I could join the meeting today to discuss a patch | 14:11 |
| abhishek_ | croelandt: thanks | 14:11 |
| croelandt | rajiv: are you willing to do the dev work on this? | 14:11 |
| rajiv | sure, need little guidance, is this a valid request ? | 14:12 |
| rajiv | the background is to control image visibility per domain | 14:12 |
| croelandt | rajiv: well I guess users requested this? | 14:13 |
| abhishek_ | priteau: this is related to fix directly proposed to stable branch, right? | 14:14 |
| 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 |
| rajiv | croelandt: yes, i was looking for options to hide images, --tag and --hidden were not accurate as policy changes | 14:15 |
| priteau | abhishek_: I submitted it to master but I wanted to discuss if it would even be considered for backports. | 14:15 |
| priteau | Happy to give more context but I don't want to interrupt the other discussion | 14:15 |
| croelandt | rajiv: if I were you I'd add links to explain how and why we evolve towards... | 14:16 |
| croelandt | because I honestly have no idea about that topic | 14:16 |
| abhishek_ | yes, you can after the current one | 14:16 |
| 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 |
| 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:17 |
| croelandt | so point out to the documentation of these other services | 14:18 |
| croelandt | prove that we are lagging behind | 14:18 |
| rajiv | okay | 14:19 |
| croelandt | Also run tox locally, that will help catch issues that the CI pointed out :) | 14:20 |
| croelandt | priteau: ok, you have the mic :) | 14:22 |
| 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 |
| 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:22 |
| 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:24 |
| 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 |
| croelandt | You tested this and it solves your issue? | 14:25 |
| 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 |
| priteau | I was just wondering if there is any chance my patch would be approved + backportable? | 14:26 |
| croelandt | so if abhishek_ is working on a proper fix, we might not want to work around it | 14:27 |
| croelandt | priteau: what distro are you using? Would they be opened to patch glance_store? | 14:27 |
| priteau | Patching is not a problem, we use our own Kolla containers | 14:27 |
| croelandt | oh ok | 14:28 |
| croelandt | abhishek_: what do you think about this patch? Is it not the "proper" way of fixing the issue? | 14:28 |
| croelandt | Is this going to be solved with your size patch series? | 14:28 |
| 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 |
| abhishek_ | I think it will force to use multipart upload for all images | 14:28 |
| croelandt | priteau: so ideally, two things | 14:29 |
| croelandt | 1) I think you should patch your containers for now | 14:29 |
| croelandt | 2) If you could try https://review.opendev.org/c/openstack/glance_store/+/949129 that would help us a lot | 14:29 |
| croelandt | we are planning to release glance_store with this patch next week | 14:30 |
| abhishek_ | croelandt: I don't know whether we allow fixing bugs directly towards stable branches | 14:30 |
| priteau | croelandt: thanks, I could try patch 949129. Is it independant or do I need to pull others? | 14:31 |
| croelandt | abhishek_: yes, we'll have to figure somehting later for stable branches | 14:31 |
| croelandt | abhishek_: does priteau need to cherry-pick something else? | 14:32 |
| abhishek_ | priteau: it's independent, you can apply s3 patch only in your env | 14:32 |
| priteau | OK, I can give it a try. | 14:32 |
| priteau | Thanks | 14:32 |
| croelandt | if you can easily do that and come back to tell us whether that works, it would be really helpful | 14:33 |
| priteau | Will do | 14:33 |
| croelandt | \o/ | 14:34 |
| croelandt | Do we have anything else? | 14:34 |
| croelandt | I think not, thanks everyone for joining! | 14:37 |
| croelandt | #endmeeting | 14:37 |
| opendevmeet | Meeting ended Thu Jun 26 14:37:08 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:37 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2025/glance.2025-06-26-14.01.html | 14:37 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2025/glance.2025-06-26-14.01.txt | 14:37 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2025/glance.2025-06-26-14.01.log.html | 14:37 |
| mhen | oh snap I missed the call for open discussion, I sneaked in the image encryption topic into the agenda last minute but then was distracted by other work while the meeting was held | 14:51 |
| abhishek_ | mhen: any specific thing you want to point out? | 14:52 |
| mhen | essentially what I wrote into the etherpad | 14:53 |
| abhishek_ | ack | 14:53 |
| mhen | implemented scenario tests but had to add a skip check that effectively makes them being skipped by Zuul CI | 14:53 |
| mhen | see my comment here: https://review.opendev.org/c/openstack/barbican-tempest-plugin/+/952699/comments/752a67e9_cdabbcd9?tab=comments | 14:54 |
| mhen | I copied this check from other existing tests that seem to have the same issue | 14:54 |
| mhen | dunno if there is anything that can be done about that or if I'm doing it wrong as a whole | 14:55 |
| abhishek_ | I will have a look, but will take some time | 14:55 |
| mhen | thanks! | 14:55 |
| * abhishek_ not around machine for next couple of hours | 14:56 | |
| abhishek_ | np! | 14:56 |
| *** mhen_ is now known as mhen | 17:57 | |
| opendevreview | Rajat Dhasmana proposed openstack/glance_store master: Add support for backend_defaults group https://review.opendev.org/c/openstack/glance_store/+/946512 | 20:02 |
| opendevreview | Rajat Dhasmana proposed openstack/glance_store master: Add support for backend_defaults group https://review.opendev.org/c/openstack/glance_store/+/946512 | 20:06 |
| opendevreview | Cyril Roelandt proposed openstack/glance-tempest-plugin master: Zuul: do not use USE_PYTHON3 https://review.opendev.org/c/openstack/glance-tempest-plugin/+/953494 | 22:06 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!