| akawai | Hi, I would like to ask you how to add 'Zuul' as reviewers. | 02:35 |
|---|---|---|
| akawai | My patch(https://review.opendev.org/c/openstack/cinder/+/885986) does not have Zuul as reviewers. | 02:36 |
| akawai | My patch(https://review.opendev.org/c/openstack/cinder/+/885986) does not have Zuul as reviewers. So, I've tried to add reviewers by pushing the blue pencil-icon and typing `Zuul`, chosing `Zuul <test.openstack.test@gmail.com>, but the Zuul does not run any tests. | 02:36 |
| akawai | Although I've also tried reply 'recheck' and made new patchset, the status is not changed. Pls tell me what to do. | 02:36 |
| opendevreview | caiqilong proposed openstack/cinder master: Add Cinder driver for TOYOU NetStor TYDS https://review.opendev.org/c/openstack/cinder/+/886942 | 03:28 |
| opendevreview | Merged openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/886333 | 03:35 |
| *** enick_178 is now known as diablo_rojo | 04:03 | |
| *** amoralej|off is now known as amoralej | 06:00 | |
| *** geguileo is now known as Guest4517 | 07:03 | |
| MichielPiscaer[m] | Is it possible when you have multiple Availability zone to migrate volumes between them, with a simple command like "openstack volume migrate"? The storages are based on Ceph RBD. | 08:29 |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder master: Implement disk format validation in volume metadata https://review.opendev.org/c/openstack/cinder/+/887081 | 09:55 |
| Supriya_Kotwal_ | Gerrit review for - https://review.opendev.org/c/openstack/cinder/+/886612 : DSM unable to fetch WWNs which are in lowercase | 11:13 |
| happystacker | senrique: regarding https://review.opendev.org/c/openstack/cinder/+/887081, how can I achieve that: "Please rebase the patch : https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/887003" ? | 12:00 |
| *** amoralej is now known as amoralej|lunch | 12:10 | |
| senrique | thank you for rebase, I'll remove the -1 and wait for Zuul output | 12:28 |
| hemna- | Sad to see RedHat shit the bed on opensource. That's really troubling. How does that jive with patches for cinder now, since cinder source is publicly available? | 12:47 |
| happystacker | senrique: still failing, did I miss something? | 13:10 |
| *** amoralej|lunch is now known as amoralej | 13:13 | |
| senrique | let me check happystacker | 13:16 |
| senrique | happystacker, not sure now... Unfortunately, the CI has been failing without reason since last month. We need to assign someone to debug and fix it because both the granade and openstack-tox-functional-py310, among others, are constantly failing regardless of the patch. | 13:20 |
| senrique | the only thing you can do it's to recheck again :/ | 13:21 |
| happystacker | ok I'll do it, thanks Sofia | 13:22 |
| happystacker | senrique: let me know if I can help | 13:24 |
| opendevreview | Eric Harney proposed openstack/cinder master: SC: Add unit test for mismatched case WWNs https://review.opendev.org/c/openstack/cinder/+/887290 | 14:43 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: Add test for create volume from backup https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/887294 | 15:14 |
| *** amoralej is now known as amoralej|off | 16:09 | |
| opendevreview | Supriya Kotwal proposed openstack/cinder master: DSM unable to fetch WWNs which are in lowercase https://review.opendev.org/c/openstack/cinder/+/886612 | 17:55 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: Add test for create volume from backup https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/887294 | 18:06 |
| opendevreview | Merged openstack/cinder master: Remove six from qnap driver https://review.opendev.org/c/openstack/cinder/+/885289 | 23:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!