| opendevreview | Yian Zong proposed openstack/cinder master: Dell PowerFlex: add NVMe-oF support https://review.opendev.org/c/openstack/cinder/+/912058 | 04:54 |
|---|---|---|
| opendevreview | Liron Kuchlani proposed openstack/cinder-tempest-plugin master: Test create a backup when latest incremental backup is deleted https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/912379 | 05:33 |
| frickler | jbernard: rosmaita: what's the status of cinder for rc1? https://review.opendev.org/c/openstack/releases/+/912386 | 07:04 |
| opendevreview | Yian Zong proposed openstack/cinder master: Dell PowerFlex: add NVMe-oF support https://review.opendev.org/c/openstack/cinder/+/912058 | 07:04 |
| opendevreview | Liron Kuchlani proposed openstack/cinder-tempest-plugin master: Add volume snapshot tests https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/885829 | 07:13 |
| opendevreview | Josephine Seifert proposed openstack/cinder-specs master: Spec for user visible information in volume types https://review.opendev.org/c/openstack/cinder-specs/+/909195 | 10:56 |
| andrei | rosmaita: hi! May be you can give advise on what can I do to encourage others to check my patch? | 11:25 |
| jbernard | frickler: i think the deadline is today, so that patch should merge very soon | 11:25 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVf]: Enable support to retype ‘iogrp’ of an ‘in-use’ volume https://review.opendev.org/c/openstack/cinder/+/912060 | 11:28 |
| frickler | jbernard: well it usually would be better to have PTL approval. whoami-rajat_ wasn't around earlier but maybe can check now? | 11:32 |
| whoami-rajat_ | frickler, jbernard which patch are we referring to? | 11:33 |
| jbernard | https://review.opendev.org/c/openstack/releases/+/912386 | 11:34 |
| whoami-rajat_ | jbernard, i was waiting for one feature to merge since it was approved and stuck because of the releasenote job | 11:37 |
| jbernard | whoami-rajat_: ahh, ok, hash is current at the moment | 11:37 |
| jbernard | whoami-rajat_: ok, i updated my review | 11:38 |
| whoami-rajat_ | thanks | 11:39 |
| andrei | guys can you please push my patch? | 11:42 |
| andrei | if deadlkine today | 11:42 |
| andrei | if deadline today | 11:42 |
| andrei | https://review.opendev.org/c/openstack/cinder/+/889284 | 11:42 |
| andrei | it is hanging for a very long time | 11:43 |
| andrei | it have +2 | 11:43 |
| andrei | if there are any typos or anything you believe is questionable I will fix it | 11:45 |
| opendevreview | OpenStack Release Bot proposed openstack/devstack-plugin-ceph stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/913002 | 11:59 |
| opendevreview | OpenStack Release Bot proposed openstack/devstack-plugin-ceph stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/913003 | 11:59 |
| andrei | whoami-rajat_: hi, can you please check https://review.opendev.org/c/openstack/cinder/+/889284, Ive worked through all recommendations | 12:18 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: Add snapshot deletion test https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/805884 | 13:22 |
| raghavendrat | hi rosmaita: are you around ? | 13:29 |
| rosmaita | raghavendrat: what's up? | 13:29 |
| raghavendrat | If you get some time, it would great to have your feedback on below patch. | 13:30 |
| raghavendrat | HPE 3par: Unable to create clone of replicated vol | 13:30 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/884594 | 13:30 |
| raghavendrat | Thanks | 13:30 |
| rosmaita | it's small and has tests, so i'll put it on my list | 13:30 |
| raghavendrat | ok, great | 13:31 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: Bump hacking to 6.1.0 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/913185 | 13:35 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: annotate lvm https://review.opendev.org/c/openstack/cinder/+/733624 | 13:59 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph stable/2023.1: Fix stable CI https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/909055 | 14:10 |
| wncslln | hello cinder o/, any core could give some review on this? https://review.opendev.org/c/openstack/cinder/+/905608 | 15:04 |
| andrei | to this patch a as well https://review.opendev.org/c/openstack/cinder/+/889284 | 15:24 |
| andrei | it needs just a single +2 | 15:25 |
| opendevreview | Merged openstack/devstack-plugin-ceph stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/913002 | 15:52 |
| opendevreview | Merged openstack/devstack-plugin-ceph stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/913003 | 15:52 |
| andrei | hi, core, Im not sure about future of https://review.opendev.org/c/openstack/cinder/+/889284 , should I write email and apply for some kind of exception so my patch could be merged tomorrow? | 16:05 |
| whoami-rajat_ | andrei, hey, I've approved the patch but just as a note, we had feature freeze 2 weeks ago and to merge any features, we have to apply for feature freeze exception on the mailing list that would give a 1 week extension. since we lacked reviews on this patch so it was given an exception but just something to keep a note about in the future | 16:09 |
| whoami-rajat_ | and sorry it took long time to merge it | 16:10 |
| andrei | whoami-rajat_: thank you! What are my actions now? Should I do something specific? | 16:21 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph stable/2023.1: Fix stable CI https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/909055 | 16:30 |
| opendevreview | Merged openstack/cinder master: reno: Update master for Unmaintained branches https://review.opendev.org/c/openstack/cinder/+/911329 | 16:45 |
| opendevreview | Merged openstack/cinder master: JovianDSS: Rework Open-E JovianDSS driver https://review.opendev.org/c/openstack/cinder/+/889284 | 18:16 |
| opendevreview | Merged openstack/cinder master: reno: Update master for xena Unmaintained status https://review.opendev.org/c/openstack/cinder/+/911348 | 19:02 |
| opendevreview | Merged openstack/cinder master: Dell EMC: Deprecate volume drivers for EOL products https://review.opendev.org/c/openstack/cinder/+/907346 | 21:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!