| opendevreview | Merged openstack/cinder master: Add 'enforce_multipath' in connection_properties https://review.opendev.org/c/openstack/cinder/+/934011 | 01:19 |
|---|---|---|
| *** mhen_ is now known as mhen | 02:59 | |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Improve process about copysession https://review.opendev.org/c/openstack/cinder/+/910482 | 05:57 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Added checking for cascade copy. https://review.opendev.org/c/openstack/cinder/+/910483 | 05:59 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Improve Snap Pool selection https://review.opendev.org/c/openstack/cinder/+/910494 | 06:16 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Change the function of attach/detach https://review.opendev.org/c/openstack/cinder/+/910495 | 06:42 |
| opendevreview | Vasyl Saienko proposed openstack/cinder master: [HPE Nimble] walk over own volumes during startup https://review.opendev.org/c/openstack/cinder/+/936760 | 06:44 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Add mapping file support https://review.opendev.org/c/openstack/cinder/+/912051 | 07:18 |
| *** geguileo is now known as Guest2958 | 10:55 | |
| raghavendrat | hi whoami-rajat: are you around ? | 12:34 |
| whoami-rajat | raghavendrat, hey | 12:35 |
| raghavendrat | If you get some time, could you please re-visit below patch. | 12:35 |
| raghavendrat | i think, i have addressed review comments. | 12:36 |
| raghavendrat | HPE 3par - Fix retype volume | 12:36 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/928599 | 12:36 |
| whoami-rajat | raghavendrat, thanks, I've updated my vote | 12:38 |
| raghavendrat | thank you whoami-rajat: for quick approval | 12:38 |
| tobias-urdin | anybody that could review this one being stuck for a while? https://review.opendev.org/c/openstack/cinder/+/804470 thanks! | 13:48 |
| raghavendrat | hi reviewers, it would great to receive feedback on below XS patch. Thanks. | 14:16 |
| raghavendrat | HPE 3par: getWsApiVersion login/logout | 14:16 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/926583 | 14:16 |
| raghavendrat | Zuul and HPE Storage CI have passed. | 14:16 |
| simondodsley | rosmaita jbernard eharney whoami-rajat looks like we might have an issue with the build `cinder-tempest-plugin-protection-functional` in Zuul. Seems to be contuniually failing since 12/12: https://zuul.opendev.org/t/openstack/builds?job_name=cinder-tempest-plugin-protection-functional&project=openstack/cinder | 14:48 |
| eharney | TypeError: EncryptionTypesClient.delete_encryption_type() missing 1 required positional argument: 'encryption_id' | 14:49 |
| eharney | well it looks like a test bug, at least | 14:50 |
| whoami-rajat | strange, looks like something changed in tempest | 14:50 |
| opendevreview | Takashi Kajinami proposed openstack/os-brick master: Fix outdated notes in requirement files https://review.opendev.org/c/openstack/os-brick/+/937694 | 14:51 |
| eharney | what changed in tempest is https://review.opendev.org/c/openstack/tempest/+/935526 | 14:52 |
| opendevreview | Takashi Kajinami proposed openstack/os-brick master: Use oslo.utils to strip scope from an IPv6 address https://review.opendev.org/c/openstack/os-brick/+/937697 | 14:57 |
| rosmaita | i think we need to propose a patch to the tempest zuul.yaml that runs a cinder-tempest-plugin job | 15:04 |
| opendevreview | Takashi Kajinami proposed openstack/python-cinderclient master: Remove Python 3.8 support https://review.opendev.org/c/openstack/python-cinderclient/+/933224 | 15:04 |
| simondodsley | rosmaita i agree | 15:06 |
| opendevreview | Takashi Kajinami proposed openstack/os-brick master: Switch back to built-in md5 method https://review.opendev.org/c/openstack/os-brick/+/930899 | 15:08 |
| opendevreview | Takashi Kajinami proposed openstack/os-brick master: Switch back to built-in md5 method https://review.opendev.org/c/openstack/os-brick/+/930899 | 15:10 |
| tosky | rosmaita: zuul, or the test code? | 15:16 |
| rosmaita | tosky: independent of the test code, the tempest team needs to be aware when they make breaking changes | 15:17 |
| tosky | rosmaita: right (gmann ^^) | 15:20 |
| tosky | but I would advise against pinning tempest, if that was the idea | 15:20 |
| tosky | especially (need to check whether this is the case) if we use bits which are not part of the stable tempest API | 15:22 |
| opendevreview | Merged openstack/os-brick master: Fix multipathd dependency in NVMe-oF connections https://review.opendev.org/c/openstack/os-brick/+/933250 | 15:41 |
| opendevreview | Merged openstack/python-cinderclient master: Fix outdated notes in requirement files https://review.opendev.org/c/openstack/python-cinderclient/+/937621 | 15:42 |
| rosmaita | is anyone working on fixing the cinder-tempest-plugin to address what simondodsley reported? if not, i can do it | 15:55 |
| jbernard | rosmaita: i say go for it | 15:57 |
| rosmaita | ok | 15:57 |
| rosmaita | (just don't want to duplicate efforts) | 15:57 |
| wncslln | hi cinder o/. An user tried to delete a volume when it comes to error_deleting state. why this occur? Its the 4th time that this happen i guess... any clue? | 17:17 |
| opendevreview | Brian Rosmaita proposed openstack/cinder-tempest-plugin master: Add 'encryption_id' parameter to enc type calls https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/937725 | 17:21 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [DNM] Get cephadm log for ingress https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/931212 | 17:59 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [DNM] Get cephadm log for ingress https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/931212 | 18:00 |
| opendevreview | Merged openstack/python-brick-cinderclient-ext master: Updating testing as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/935588 | 19:40 |
| gmann | rosmaita: eharney tonyb sorry for getting breaking change in tempest. I agree that without testing it is hard to know all plugin usage. | 20:12 |
| gmann | I agree with rosmaita idea to run some cinder-tempest-plugin tests in tempest job to catch such breaking change. let me do that | 20:13 |
| rosmaita | gmann: np, it was kind of our fault | 20:13 |
| gmann | tosky_ I mean to ping. tonyb please ignore this ping | 20:14 |
| *** elodilles is now known as elodilles_pto | 20:51 | |
| simondodsley | eharney jbernard whoami-rajat the patch to fix the cinder tempest plugin job has passed zuul. Could we get this merged as it is stopping all cinder jobs currently? | 23:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!