| opendevreview | Joan Gilabert proposed openstack/hacking master: Add H302 check: import modules, not objects https://review.opendev.org/c/openstack/hacking/+/1011889 | 06:25 |
|---|---|---|
| opendevreview | Abhishek Kekane proposed openstack/tempest master: Accept 410 from legacy tasks API https://review.opendev.org/c/openstack/tempest/+/1011263 | 06:25 |
| opendevreview | Merged openstack/devstack-plugin-jaeger master: Install the OTLP exporter for tracing https://review.opendev.org/c/openstack/devstack-plugin-jaeger/+/1011468 | 08:32 |
| opendevreview | Michael Still proposed openstack/devstack master: Tolerate zombie processes in get-stats.py https://review.opendev.org/c/openstack/devstack/+/1011786 | 08:39 |
| opendevreview | Michael Still proposed openstack/devstack master: Record peak, anonymous and swapped memory in get-stats.py https://review.opendev.org/c/openstack/devstack/+/1011787 | 08:39 |
| opendevreview | Michael Still proposed openstack/devstack master: Generate performance.json without a global venv https://review.opendev.org/c/openstack/devstack/+/1011794 | 08:39 |
| opendevreview | Michael Still proposed openstack/devstack master: Tolerate zombie processes in get-stats.py https://review.opendev.org/c/openstack/devstack/+/1011786 | 09:41 |
| opendevreview | Michael Still proposed openstack/devstack master: Record peak, anonymous and swapped memory in get-stats.py https://review.opendev.org/c/openstack/devstack/+/1011787 | 09:42 |
| opendevreview | Michael Still proposed openstack/devstack master: Generate performance.json without a global venv https://review.opendev.org/c/openstack/devstack/+/1011794 | 09:42 |
| opendevreview | Michael Still proposed openstack/devstack master: Record service process memory in get-stats.py https://review.opendev.org/c/openstack/devstack/+/1011992 | 09:42 |
| opendevreview | Abhishek Kekane proposed openstack/devstack master: Install glance_store S3 extra https://review.opendev.org/c/openstack/devstack/+/1011993 | 10:00 |
| opendevreview | Grzegorz Grasza proposed openstack/tempest master: Support write-only credentials and S3 signing keys https://review.opendev.org/c/openstack/tempest/+/1012025 | 13:30 |
| abhishek_ | Hi, if any core is around (cc @gmaan) kindly review https://review.opendev.org/c/openstack/devstack/+/1011993 as this will help unblcok glance CI failure related to s3 job | 15:08 |
| gmaan | abhishek_: ack, how it is broken now?something changed recently in devstack? | 16:26 |
| tkajinam | gmaan, I guess it's a side effect of recent cinder change which made boto3 as its optional dependency | 16:36 |
| tkajinam | Oct 23 12:57:08.907562 np96d7cafe86b84 devstack@g-api.service[85791]: WARNING glance_store.driver [None req-171e1872-58a4-49af-8e7c-9f0073b284bb admin admin] Failed to configure store correctly: Store s3 could not be configured correctly. Reason: boto3 or botocore is not available. Disabling add method.: glance_store.exceptions.BadStoreConfiguration: Store s3 could not be configured correctly. Reason: boto3 or botocore is not available. | 16:39 |
| tkajinam | I assume one of the affected jobs is https://zuul.opendev.org/t/openstack/builds?job_name=glance-s3-multistore | 16:39 |
| tkajinam | https://review.opendev.org/c/openstack/cinder/+/902104 is the cinder change I'm referring to. | 16:40 |
| tkajinam | and we no longer install boto3 for cinder unless c-bak is enabled and s3 driver is used | 16:40 |
| abhishek_ | thanks tkajinam san! | 16:45 |
| tkajinam | ;-) | 16:46 |
| tkajinam | that cinder change was merged during 2026.2 cycle so we likely have to backport that devstack update later | 16:47 |
| abhishek_ | ack | 16:49 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily to restart onto updated container images, but should return within a few minutes | 17:07 | |
| gmaan | tkajinam: abhishek_ ok but how it is installed now, i mean if boto3 is deps of s3 store then it should be in the requirements either on glance-store side or so | 17:23 |
| gmaan | and it seems it was broken since 2026.2 released? abhishek_ maybe you should add coverage of s3 store in cinder side to avoid such breakage | 17:24 |
| abhishek_ | ack | 17:25 |
| gmaan | abhishek_: coming back to my question, 'Store s3 could not be configured correctly. Reason: boto3 or botocore is not available.' so where we are installing boto3 or botocore now? | 17:27 |
| gmaan | abhishek_: also, can you push a testing change where that job run to know if that fixes the issue or we need to install boto3 or botocore somwhere? | 17:28 |
| abhishek_ | OK, I will push a DNM patch depending on above devstack fix | 17:29 |
| gmaan | thanks | 17:29 |
| abhishek_ | boto3 is present in test-requirements for glance_store and glance | 17:30 |
| gmaan | cool | 17:33 |
| gmaan | if it hard deps then it should be in reqiuirements.txt though but anyways that you can check later, once glance change pass this job, I am good on devstack change | 17:35 |
| gmaan | abhishek_: oh maybe you need to do that ^^ as test-requirements not installed by devstack, I think it should be detected in glance testing change, let's wait fro that | 17:36 |
| gmaan | abhishek_: if it is late for you, I can push glance testing change | 17:36 |
| abhishek_ | I am working on that, will push it in 5-10 mins | 17:37 |
| gmaan | cool | 17:37 |
| abhishek_ | if it takes more than that then I will ping you, thank you :) | 17:37 |
| gmaan | sure | 17:37 |
| abhishek_ | gmaan: https://review.opendev.org/c/openstack/glance/+/1012082 | 17:40 |
| gmaan | abhishek_: thanks, I will monitor it. it might be late for you, have a good weekend | 17:41 |
| abhishek_ | thank you, same to you!! | 17:42 |
| abhishek_ | gmaan: https://github.com/openstack/glance_store/blob/master/setup.cfg#L75 | 17:48 |
| gmaan | oh nice, that works fine. | 17:48 |
| tkajinam | gmaan, boto3 isn't a hard dep. it's an optional dep of glance_store and is installed only when s3 driver is used | 21:58 |
| tkajinam | it's intentionally made optional | 21:59 |
| tkajinam | I agree there should be a voting s3 store driver job. s3 driver wasn't maintained well in the past but its maintenance status has been restored for recent cycles | 22:00 |
| gmaan | yea its there in setup.cfg and job passing in glance testing change, I am +2 on the fix | 23:26 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!