| opendevreview | Eric Xie proposed openstack/cinder master: Using `clone` instead `download` when pool in the 'host' field https://review.opendev.org/c/openstack/cinder/+/822278 | 01:40 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/818037 | 02:29 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVF] Add cleanrate in volume-type extra specs https://review.opendev.org/c/openstack/cinder/+/811877 | 04:45 |
| opendevreview | Abhishek Kekane proposed openstack/devstack-plugin-ceph master: Allow glance ceph osd cinder,nova pool access https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/819476 | 05:12 |
| TusharTgite | hi team , i'm stuck in here to zolve this zuul -1 on this patch can anyone please give me diection on how i can resolve this https://review.opendev.org/c/openstack/cinder/+/804035 | 07:26 |
| *** _pewp__ is now known as _pewp_ | 07:37 | |
| opendevreview | katari manoj kumar proposed openstack/cinder master: Doc: Typos in Spectrum Virtulize family user guide https://review.opendev.org/c/openstack/cinder/+/819082 | 08:33 |
| *** dviroel__ is now known as dviroel | 11:14 | |
| *** brinzhang_ is now known as brinzhang | 12:38 | |
| rosmaita | geguileo: tiny cinderlib patch when you have a minute: https://review.opendev.org/c/openstack/cinderlib/+/819258 | 13:13 |
| venkatakrishnat | Hi reviewers, Could someone please review these patches https://review.opendev.org/c/openstack/cinder/+/810917 and https://review.opendev.org/c/openstack/cinder/+/809830 | 13:34 |
| tosky | rosmaita: not sure you have seen this simple "import translations" change, but I think it make sense to officially rule that such changes can be merged by a single reviewer: https://review.opendev.org/c/openstack/cinder/+/818037 | 13:48 |
| rosmaita | tosky: yes, that is the policy, and thanks for pointing out the patch | 13:50 |
| tosky | rosmaita: oh, I didn't know whether was something we did in sahara only, whether it was global, or still project-specific but widespread among the projects | 13:53 |
| rosmaita | tosky: not sure where it's stated, maybe the translations guide? or maybe it's folklore ... in any case, it does make sense | 13:55 |
| geguileo | rosmaita: +2 to cinderlib patch, merging now | 15:33 |
| rosmaita | geguileo: ty | 15:34 |
| rosmaita | geguileo: hate to bug you more, but could you take a quick look at the nvmeof brick spec? wanted to make sure you saw zohar's response to your point about using device mapper as a fallback: https://review.opendev.org/c/openstack/cinder-specs/+/815613/comment/cf1efcaa_775abdfd/ | 15:37 |
| rosmaita | other than that, he's updated with all your suggestions | 15:38 |
| opendevreview | Merged openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/818037 | 16:05 |
| opendevreview | Ghanshyam proposed openstack/cinder stable/train: DNM: test tempest train-last tag https://review.opendev.org/c/openstack/cinder/+/816596 | 16:13 |
| geguileo | rosmaita: added comments to the spec | 17:07 |
| rosmaita | geguileo: ty | 17:08 |
| geguileo | rosmaita: I feel bad for Zohar, because the spec is really good but I keep complaining ... | 17:08 |
| *** bauzas is now known as bauzas_pto_till_Jan3rd | 17:13 | |
| rosmaita | geguileo: hopefully he understands ... | 17:16 |
| geguileo | rosmaita: or he'll learn voodoo to make me suffer ;-) | 17:16 |
| rosmaita | :) | 17:17 |
| opendevreview | Merged openstack/cinderlib master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/cinderlib/+/819258 | 19:32 |
| *** dviroel is now known as dviroel|afk | 20:06 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!