| *** mhen_ is now known as mhen | 01:10 | |
| croelandt | mhen: regarding the encryption patch, dansmith has left a few comments on Thursday. abhishekk will be away for a few weeks starting next Monday, so if dansmith and yourself can come to an agreement, we could potentially merge this by the end of the week. Otherwise it will have to wait a little bit longer | 15:31 |
|---|---|---|
| mhen | croelandt: understood, I had seen that dansmith commented but didn't manage to look at it today; I will most likely find some time tomorrow | 15:32 |
| opendevreview | Cyril Roelandt proposed openstack/glance_store master: Cinder: improve unit tests factorization https://review.opendev.org/c/openstack/glance_store/+/955082 | 15:38 |
| masahito | hit glance team, please review this patch. https://review.opendev.org/c/openstack/glance/+/958196 It's an important information for all S3 compatible API backend user. | 16:08 |
| masahito | s/hit/hi/ | 16:10 |
| abhishekk | masahito: I see s3 job on glance patches is passing without any issue | 16:51 |
| abhishekk | https://review.opendev.org/c/openstack/glance/+/958715 see this one for reference which is installing boto3-1.40.1 | 16:52 |
| abhishekk | is that because of swift? | 16:53 |
| masahito | yes, i think so. The swift supports the new S3 API change, so the CI passes. | 16:55 |
| abhishekk | ack, there is formatting issue in reno which needs to fix and I think we can not merge this in master because for master upper constraint for boto3 is 1.40.1 and not 1.36 | 16:57 |
| abhishekk | I think this needs to be directly proposed to stable/2025.1 | 16:59 |
| abhishekk | @croelandt fyi ^^ | 16:59 |
| masahito | thank. i want to talk this point as well. the problem started from stable/2025.1 so we're wondering the patch should be in both master and stable/2025.1 or only in stable/2025.1. | 17:00 |
| abhishekk | I don't think this makes any sense to master branch | 17:01 |
| masahito | the 1.36 version has the issue and stable/2025.1 upper-constraints is 1.36.26. https://github.com/openstack/requirements/blob/stable/2025.1/upper-constraints.txt#L306 | 17:01 |
| abhishekk | but you can seek the opinion of our PTL | 17:01 |
| abhishekk | yes, that is why this patch directly needs to be proposed to stable and not master | 17:02 |
| masahito | ok. anyway, let us push another commit to stable/2025.1 branch, too. it's required for either case. | 17:05 |
| opendevreview | Merged openstack/glance_store master: Fix missing dependency on retrying https://review.opendev.org/c/openstack/glance_store/+/954192 | 17:33 |
| opendevreview | Merged openstack/glance_store master: Run ceph-tempest job specifically for RBD https://review.opendev.org/c/openstack/glance_store/+/949702 | 17:33 |
| opendevreview | Merged openstack/glance_store master: Migrate tips jobs to Noble https://review.opendev.org/c/openstack/glance_store/+/958987 | 18:22 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!