| tkajinam | abhishekk, hi. are your around ? | 07:54 |
|---|---|---|
| tkajinam | abhishekk, I'm wondering if I could hear your thoughts on https://review.opendev.org/c/openstack/glance_store/+/1002875 | 07:54 |
| tkajinam | the main question is whether we should revert the original change to disable secure hash (proposed in https://review.opendev.org/c/openstack/glance_store/+/1002880 ) instead | 07:55 |
| tkajinam | abhishekk, I don't know clearly why we needed the feature disabled, and I saw you added that flag | 07:55 |
| opendevreview | Cyril Roelandt proposed openstack/glance master: Requirements: requests is always required https://review.opendev.org/c/openstack/glance/+/1002563 | 10:06 |
| tkajinam | croeland1, hi. could you check https://review.opendev.org/c/openstack/glance_store/+/1002875 and https://review.opendev.org/c/openstack/glance_store/+/1002875 and share your preference ? We need one of these to fix the failing test in glance_store | 10:17 |
| tkajinam | oops. I posted the same link twice... | 10:17 |
| tkajinam | croeland1, this is the correct link https://review.opendev.org/c/openstack/glance_store/+/1002880 | 10:18 |
| tkajinam | I don't fully understand why the secure hash had to be disabled. I also asked abhishekk to check these, because he added that override in the CI job. | 10:18 |
| abhishekk | tkajinam: the reason was mentioned in the commit message itself | 10:25 |
| abhishekk | @pdeore please look at above patches and decide which is the better way to move forward | 10:28 |
| tkajinam | abhishekk, yes though the same feature is enabled in ceph job in glance repo and has been passing afaik | 10:48 |
| tkajinam | I don't know why we should disable it specifically in glance_store | 10:49 |
| tkajinam | maybe we could try enabling the feature later and merge https://review.opendev.org/c/openstack/glance_store/+/1002875 as a solution ? this is required to fix broken ceph job in glance_store | 10:50 |
| tkajinam | as a solution for now, I mean | 10:50 |
| abhishekk | sounds good | 10:54 |
| croeland1 | ok, looks like I'll still be PTL https://review.opendev.org/c/openstack/governance/+/1002415 | 13:27 |
| croeland1 | y'all won't get rid of me this cycle! | 13:27 |
| *** croeland1 is now known as croelandt | 13:27 | |
| opendevreview | Merged openstack/glance-specs master: Add 2027.1 directory for specs https://review.opendev.org/c/openstack/glance-specs/+/999612 | 13:33 |
| *** tobias-u1 is now known as tobias-urdin | 14:23 | |
| opendevreview | Cyril Roelandt proposed openstack/glance master: Test requirements: remove oslotest https://review.opendev.org/c/openstack/glance/+/1003066 | 15:00 |
| opendevreview | Cyril Roelandt proposed openstack/glance master: Test requirements: remove boto3 https://review.opendev.org/c/openstack/glance/+/1003069 | 15:08 |
| opendevreview | Cyril Roelandt proposed openstack/glance master: Test requirements: remove privsep https://review.opendev.org/c/openstack/glance/+/1003075 | 15:32 |
| opendevreview | Cyril Roelandt proposed openstack/glance_store master: Test requirements: remove testscenarios https://review.opendev.org/c/openstack/glance_store/+/1003077 | 15:54 |
| opendevreview | Cyril Roelandt proposed openstack/glance_store master: Test requirements: remove httplib2 https://review.opendev.org/c/openstack/glance_store/+/1003078 | 15:56 |
| opendevreview | Anton Iacobaeus proposed openstack/glance master: Update metadata definition for Intel TDX support in nova https://review.opendev.org/c/openstack/glance/+/1003083 | 16:41 |
| tkajinam | croelandt, hi could you check https://review.opendev.org/c/openstack/glance_store/+/1002875 when you have time . this unblocks glance_store CI (which you may notice in your changes later) | 16:42 |
| opendevreview | Merged openstack/glance master: Requirements: requests is always required https://review.opendev.org/c/openstack/glance/+/1002563 | 18:33 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!