| opendevreview | Keigo Noha proposed openstack/cinder master: [WIP] Consolidate schema validation based on volumes. https://review.opendev.org/c/openstack/cinder/+/884958 | 02:56 |
|---|---|---|
| *** amoralej|off is now known as amoralej | 07:26 | |
| *** chuanm9 is now known as chuanm | 09:52 | |
| opendevreview | Rajat Dhasmana proposed openstack/cinder-specs master: Spec to introduce new backup_status field for volume https://review.opendev.org/c/openstack/cinder-specs/+/868761 | 11:10 |
| *** amoralej is now known as amoralej|lunch | 11:11 | |
| raghavendrat | hi whoami-rajat: are you around ? | 11:34 |
| raghavendrat | whoami-rajat: if you get time, could you please have a look at below patch. It has one +2. Thanks. | 12:27 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/878684 | 12:28 |
| opendevreview | Merged openstack/cinder-specs master: Spec to introduce new backup_status field for volume https://review.opendev.org/c/openstack/cinder-specs/+/868761 | 13:03 |
| *** amoralej|lunch is now known as amoralej | 13:16 | |
| whoami-rajat | jungleboyj, geguileo eharney hemna jbernard enriquetaso can you take a look at this spec? today is the spec merge deadline https://review.opendev.org/c/openstack/cinder-specs/+/868761 | 13:31 |
| enriquetaso | sure | 13:31 |
| enriquetaso | it's merged whoami-rajat | 13:32 |
| whoami-rajat | oh simondodsley did it, didn't notice | 13:32 |
| whoami-rajat | thanks enriquetaso | 13:32 |
| whoami-rajat | thanks simondodsley | 13:32 |
| enriquetaso | whoami-rajat, jungleboyj or any interesting in rbd backups :) would you mind double check https://review.opendev.org/c/openstack/cinder/+/880965 I've rebase the patch | 13:45 |
| sfv880_ | Hello reviewers! Could you please review backports: https://review.opendev.org/c/openstack/cinder/+/886067 and https://review.opendev.org/c/openstack/cinder/+/886141 ? Thank you very much! | 14:21 |
| *** chuanm1 is now known as chuanm | 14:46 | |
| ganso | whoami-rajat, jungleboyj: hi! When you have a minute could you please take a look at the last backport in the series? https://review.opendev.org/c/openstack/cinder/+/885822 Thanks in advance! | 14:46 |
| *** chuanm0 is now known as chuanm | 14:57 | |
| *** chuanm1 is now known as chuanm | 15:06 | |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Ceph: Fix restore backups to diff backend https://review.opendev.org/c/openstack/cinder/+/880965 | 15:15 |
| enriquetaso | As discussed I've updated the commit message ^ | 15:16 |
| enriquetaso | Let's wait for Zuul to vote +1 and I'll ask for reviews again :) thanks! | 15:16 |
| opendevreview | Cyril Roelandt proposed openstack/cinder master: Remove all occurrences of six.text_type https://review.opendev.org/c/openstack/cinder/+/886219 | 15:17 |
| opendevreview | Cyril Roelandt proposed openstack/cinder master: Remove all occurrences of six.add_metaclass https://review.opendev.org/c/openstack/cinder/+/886220 | 15:17 |
| opendevreview | Cyril Roelandt proposed openstack/cinder master: Remove all occurrences of six.assertCountEqual https://review.opendev.org/c/openstack/cinder/+/886221 | 15:17 |
| opendevreview | Cyril Roelandt proposed openstack/cinder master: Remove all occurrences of six.iteritems https://review.opendev.org/c/openstack/cinder/+/886222 | 15:17 |
| opendevreview | Cyril Roelandt proposed openstack/cinder master: Remove all occurrences of six.PY3 https://review.opendev.org/c/openstack/cinder/+/886223 | 15:17 |
| opendevreview | Cyril Roelandt proposed openstack/cinder master: Remove all occurrences of six.wraps https://review.opendev.org/c/openstack/cinder/+/886224 | 15:17 |
| opendevreview | Cyril Roelandt proposed openstack/cinder master: Remove all occurrences of six.{string,integer}_types https://review.opendev.org/c/openstack/cinder/+/886225 | 15:17 |
| opendevreview | Cyril Roelandt proposed openstack/cinder master: Remove all occurrences of six.binary_type https://review.opendev.org/c/openstack/cinder/+/886226 | 15:17 |
| opendevreview | Cyril Roelandt proposed openstack/cinder master: Remove all occurrences of six.b https://review.opendev.org/c/openstack/cinder/+/886227 | 15:18 |
| opendevreview | Cyril Roelandt proposed openstack/cinder master: Remove all occurrences of six.reraise https://review.opendev.org/c/openstack/cinder/+/886228 | 15:18 |
| *** amoralej is now known as amoralej|off | 17:28 | |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [WIP][DNM] Remote Ceph with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/876747 | 20:39 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [WIP][DNM] Remote Ceph with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/876747 | 20:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!