| *** zseguin is now known as Guest16914 | 06:31 | |
| *** rosmaita1 is now known as rosmaita | 12:58 | |
| croelandt | tkajinam: I think we need to discuss https://review.opendev.org/c/openstack/glance/+/973102/10 tomorrow | 14:46 |
|---|---|---|
| croelandt | abhishek_: can we still use keystone v1/v2 to authenticate? | 14:46 |
| tkajinam | croelandt, no identity v2 (and even v1) was already removed | 14:50 |
| tkajinam | croelandt, ok > we need to discuss | 14:50 |
| croelandt | yeah so... v3 must work | 14:50 |
| tkajinam | I tried to make changes step by step but I'm wondering if we want to squash the next one (to use keystoneauth) into that one instead. | 14:50 |
| tkajinam | yup | 14:50 |
| croelandt | which means I wonder why your patch needed to add the required two fields | 14:51 |
| croelandt | since you say it fails when they are not there | 14:51 |
| tkajinam | croelandt, yeah but it seems the client implementation is called by the cache managenent utilities and utilities always pass down domain ids, it seems | 14:52 |
| tkajinam | https://github.com/openstack/glance/blob/4b369fc380fa17aa6a21b4a46530439bedee211e/glance/image_cache/client.py#L116-L119 | 14:53 |
| tkajinam | (I've posted the same in review | 14:53 |
| tkajinam | I mean, KeystoneStrategy fails if domain_ids are not passed, but the only code using KeystoneStrategy always pass down domain_ids | 14:56 |
| tkajinam | I find it quite confusing that these creds keys are validated in KeystoneStrategy while inputs are defined far distant place... | 14:56 |
| abhishek_ | croelandt: No | 14:58 |
| croelandt | abhishek_: if you review https://review.opendev.org/c/openstack/glance/+/973112/7 we can start pushing a few of tkajinam's patches | 15:05 |
| croelandt | tkajinam: maintaining software is also maintaining a headache, it seems | 15:06 |
| tkajinam | X-( | 15:06 |
| tkajinam | croelandt, by the way could you check https://review.opendev.org/c/openstack/glance_store/+/1002875 to fix CI . | 15:06 |
| abhishek_ | croelandt: done | 15:11 |
| croelandt | tkajinam: is there a default that changed in Tempest? | 15:12 |
| abhishek_ | @croelandt I have added cross project sessions in ptg planning etherpad for nova and cinder, I guess we need to communicate with them in advance for the same | 15:12 |
| abhishek_ | https://etherpad.opendev.org/p/2027.1-ptg-glance-planning | 15:12 |
| opendevreview | Takashi Kajinami proposed openstack/glance_store stable/2026.1: DNM: testing https://review.opendev.org/c/openstack/glance_store/+/1003551 | 15:14 |
| tkajinam | croelandt, no. it has been enabled by default. https://review.opendev.org/c/openstack/tempest/+/951592 | 15:14 |
| tkajinam | I'll check whether we should backport it to stable branches. (tempest is branchless so that test may affect stable branches too) | 15:15 |
| opendevreview | Takashi Kajinami proposed openstack/glance_store stable/2026.1: DNM: testing https://review.opendev.org/c/openstack/glance_store/+/1003551 | 15:16 |
| tkajinam | I guess that ceph job hasn't been run for some time due to limited trigger and stable rbd driver code | 15:17 |
| tkajinam | I'll try to be online around the meeting time tomorrow | 15:18 |
| croelandt | tkajinam: ok so a default changed :) | 15:48 |
| croelandt | abhishek_: thanks! | 15:48 |
| tkajinam | yeah | 15:48 |
| tkajinam | though it's strange that the change was merged several months ago | 15:49 |
| croelandt | abhishek_: didn't we also need a Cinder cross-session for erasure coded pools? | 15:49 |
| abhishek_ | No, not needed, today cinder don't support rbd + add_location optimization so that is not needed | 15:50 |
| croelandt | oh I thought they did | 15:53 |
| croelandt | ok ok | 15:53 |
| abhishek_ | I also thought, but today I confirmed its not there yet | 15:54 |
| croelandt | thanks for checking | 16:07 |
| croelandt | https://review.opendev.org/c/openstack/glance/+/1001296 is this what you had in mind when writing that comment? | 16:07 |
| opendevreview | Merged openstack/glance_store master: tempest: Configure do_secure_hash option https://review.opendev.org/c/openstack/glance_store/+/1002875 | 18:39 |
| opendevreview | Eimad Zamrik proposed openstack/glance master: Add configurable qemu-img process limits https://review.opendev.org/c/openstack/glance/+/952950 | 19:46 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!