| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/873350 | 03:30 |
|---|---|---|
| Sathya | Hi Reviewers, Can I get some review on the patch https://review.opendev.org/c/openstack/cinder/+/869302 | 04:38 |
| gksk | Hi Team Can someone review the below patch | 04:58 |
| gksk | https://review.opendev.org/c/openstack/cinder/+/869704 | 04:58 |
| opendevreview | Konrad Gube proposed openstack/cinder master: Add the os-extend_volume_completion volume action https://review.opendev.org/c/openstack/cinder/+/873557 | 08:45 |
| opendevreview | Konrad Gube proposed openstack/cinder master: NFS: Use extend volume completion to add online extend support https://review.opendev.org/c/openstack/cinder/+/873686 | 08:45 |
| Sathya | Hi Reviewers, Can I get some review on the patch https://review.opendev.org/c/openstack/cinder/+/869302 | 08:48 |
| *** dviroel|out is now known as dviroel | 10:58 | |
| harsh | Hello core reviewers, requesting you to please review the patch : https://review.opendev.org/c/openstack/cinder/+/844140 | 13:28 |
| harsh | it has been pending for a long time. Zuul and IBM CI have passed on this already. got multiple +1's also. | 13:28 |
| rosmaita | harsh: which IBM third-party CI handles SVf ? | 14:01 |
| rosmaita | nobody is going to review a patch like that unless the third-party CI is green ... there is no way for us to know what calls to the backend are strictly necessary and which are redundant | 14:02 |
| harsh | IBM Storage CI | 14:04 |
| harsh | this is the IBM third party CI that we run on SVf patches | 14:04 |
| rosmaita | ok, so SVf is actually storwize? | 14:04 |
| harsh | it is green on this patch. | 14:05 |
| harsh | yes | 14:05 |
| rosmaita | ok, i can see that now by looking at the file paths ... it would be helpful if you put [storwize] in the commit message header instead of [SVf] ... you can keep the [Spectrum Virtualize family] in the body of the commit message like you have now | 14:06 |
| rosmaita | (your goal is to make it as easy as possible to review your patches!) | 14:07 |
| harsh | okie. i will take care of that from now on | 14:07 |
| rosmaita | thanks! | 14:07 |
| harsh | :) | 14:07 |
| hemna- | mornin | 14:20 |
| opendevreview | Rodrigo Barbieri proposed openstack/cinder stable/yoga: Filter reserved image properties https://review.opendev.org/c/openstack/cinder/+/873145 | 14:31 |
| opendevreview | Konrad Gube proposed openstack/cinder master: Add the os-extend_volume_completion volume action https://review.opendev.org/c/openstack/cinder/+/873557 | 14:55 |
| opendevreview | Konrad Gube proposed openstack/cinder master: NFS: Use extend volume completion to add online extend support https://review.opendev.org/c/openstack/cinder/+/873686 | 14:55 |
| opendevreview | Merged openstack/python-brick-cinderclient-ext master: Fix: tox4 issues https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/873635 | 15:27 |
| opendevreview | Merged openstack/cinder master: Objects: Make OPTIONAL_FIELDS a tuple https://review.opendev.org/c/openstack/cinder/+/865034 | 15:50 |
| opendevreview | Merged openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/873350 | 16:25 |
| opendevreview | Sathyanarayana R proposed openstack/cinder master: [SVf] As part of Flashcopy 2.0 adding config parameter to support volumegroup https://review.opendev.org/c/openstack/cinder/+/869302 | 16:55 |
| rosmaita | when a stable core (other than eric) has a minute, this will unclog the ussuri gate: https://review.opendev.org/c/openstack/cinder/+/871792 | 19:03 |
| rosmaita | e0ne enriquetaso geguileo hemna jungleboyj smcginnis whoami-rajat ^^ | 19:03 |
| jungleboyj | +W from me. | 19:05 |
| jungleboyj | rosmaita: ^^ | 19:05 |
| rosmaita | jungleboyj: ty! | 19:05 |
| jungleboyj | Welcome. Try to be useful on occasion. :-) | 19:06 |
| opendevreview | Merged openstack/python-brick-cinderclient-ext master: Update minimum requirements https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/873595 | 20:36 |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder stable/xena: PowerMax Driver - Get manageable volumes fix https://review.opendev.org/c/openstack/cinder/+/873755 | 22:06 |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder stable/xena: PowerMax Driver - Get manageable volumes fix https://review.opendev.org/c/openstack/cinder/+/821739 | 22:21 |
| opendevreview | Merged openstack/cinder master: Dell PowerMax Driver - Add support for Unisphere for PowerMax 10.0 https://review.opendev.org/c/openstack/cinder/+/844563 | 22:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!