| *** yoctozepto8 is now known as yoctozepto | 00:33 | |
| opendevreview | Merged openstack/os-brick master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/os-brick/+/805705 | 00:42 |
|---|---|---|
| opendevreview | Merged openstack/cinder master: [SVF] Fixed update_rep_properties for empty values https://review.opendev.org/c/openstack/cinder/+/801617 | 01:05 |
| *** yoctozepto6 is now known as yoctozepto | 01:21 | |
| Girish_IBM_ | hemna can you please review this patch https://review.opendev.org/c/openstack/cinder/+/782390 | 01:27 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Prototype base policy file https://review.opendev.org/c/openstack/cinder/+/803748 | 02:01 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: Fake policies - wallaby https://review.opendev.org/c/openstack/cinder/+/805749 | 02:02 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: Fake policies - xena https://review.opendev.org/c/openstack/cinder/+/805770 | 02:03 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: Fake policies - yoga https://review.opendev.org/c/openstack/cinder/+/805771 | 02:04 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: Fake policies - AA https://review.opendev.org/c/openstack/cinder/+/805773 | 02:04 |
| *** yoctozepto3 is now known as yoctozepto | 02:12 | |
| *** yoctozepto8 is now known as yoctozepto | 03:43 | |
| *** yoctozepto8 is now known as yoctozepto | 03:57 | |
| *** yoctozepto0 is now known as yoctozepto | 04:12 | |
| *** yoctozepto1 is now known as yoctozepto | 04:40 | |
| *** yoctozepto7 is now known as yoctozepto | 05:43 | |
| *** yoctozepto0 is now known as yoctozepto | 06:24 | |
| *** redrobot0 is now known as redrobot | 06:29 | |
| *** yoctozepto9 is now known as yoctozepto | 07:00 | |
| *** yoctozepto1 is now known as yoctozepto | 07:52 | |
| opendevreview | GirishIBM proposed openstack/cinder-specs master: Replication disable reason for replication backend https://review.opendev.org/c/openstack/cinder-specs/+/804204 | 07:53 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Log connection info returned from driver https://review.opendev.org/c/openstack/cinder/+/803696 | 08:47 |
| *** akekane_ is now known as abhishekk | 08:53 | |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: HPE 3PAR: Add HPE Alletra 9k related information https://review.opendev.org/c/openstack/cinder/+/805796 | 09:08 |
| venkatakrishnat | Hi eharney, jungleboyj, rosmaita, whoami-rajat: Waiting for workflow +2 for https://review.opendev.org/c/openstack/cinder/+/786524 which is to manage global mirror change volumes on separate storage pools. Please review in your free time | 10:24 |
| opendevreview | Merged openstack/cinder stable/victoria: [SVF]:Storwize hyperswap volume is not deleting. https://review.opendev.org/c/openstack/cinder/+/798104 | 10:33 |
| opendevreview | Eric Xie proposed openstack/cinder stable/victoria: Fix instance locality scheduler filter https://review.opendev.org/c/openstack/cinder/+/798098 | 10:53 |
| *** dviroel|out is now known as dviroel|ruck | 11:23 | |
| opendevreview | Merged openstack/cinder stable/ussuri: NFS: Update connection info on online snap create https://review.opendev.org/c/openstack/cinder/+/802045 | 11:48 |
| raghavendrat | hi reviewers, request someone to have a look at: https://review.opendev.org/c/openstack/cinder/+/805796 | 12:19 |
| raghavendrat | HPE 3PAR: Add HPE Alletra 9k related information | 12:19 |
| raghavendrat | Appreciate your feedback. | 12:19 |
| whoami-rajat | simondodsley, hey, I've requested access for the nvme doc, can you help with that? | 12:43 |
| simondodsley | whoami-rajat: Done | 12:51 |
| whoami-rajat | thanks simondodsley | 12:55 |
| opendevreview | Eric Harney proposed openstack/cinder master: Snapshot in-use volumes without force flag https://review.opendev.org/c/openstack/cinder/+/789564 | 13:34 |
| simondodsley | geguileo: for your LVM NVMe-TCP patch I assume that LVM cannot be used for NVMe-FC so any work for FC transport will have to depend on this. Correct? | 14:18 |
| geguileo | simondodsley: We had NVMe-RDMA already working, that patch adds the NVMe-TCP, and I was thinking of adding support for NVMe-FC to the LVM driver as well | 14:19 |
| geguileo | I think it should be doable with a machine with HBA that I have here | 14:20 |
| venkatakrishnat | Hi eharney, jungleboyj, rosmaita, whoami-rajat: Please review the patch https://review.opendev.org/c/openstack/cinder/+/786524. Waiting for workflow +2 and this patch is to manage GM change volumes on separate storage pools. | 14:20 |
| geguileo | The FC array I have here doesn't support NVMe | 14:21 |
| geguileo | it's a very old one | 14:21 |
| simondodsley | geguilo: so amending this current patch for FC transport support? Sounds good., but there are issues in os-brick as the `_wait_for_blk` routine uses hardcoded RDMA parameters. | 14:22 |
| simondodsley | check out my document where I have jsut added this detail | 14:22 |
| simondodsley | geguileo ^ | 14:22 |
| geguileo | simondodsley: I see you added a paragraph related to that. For FC there may be os-brick changes necessary | 14:23 |
| simondodsley | https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connectors/nvmeof.py#L287 | 14:23 |
| geguileo | and my "may" I mean, it looks like they are | 14:23 |
| rosmaita | venkatakrishnat: would like to see some reviews from you on non-ibm patches, please | 14:23 |
| venkatakrishnat | rosmaita: I am Spending atleast 30 minutes everyday to review some non-IBM patches. | 14:41 |
| rosmaita | nice! happy to hear that | 14:41 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Add infrastructure for testing new RBAC policies https://review.opendev.org/c/openstack/cinder/+/805316 | 14:44 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Implement project personas for attachments https://review.opendev.org/c/openstack/cinder/+/759943 | 14:44 |
| abishop | rosmaita, lbragstad: ^^ pushed new framework code | 14:45 |
| lbragstad | sounds good - i have some cinder reviews queued up for this morning, i'll add those and take a look | 14:45 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Implement project personas for attachments https://review.opendev.org/c/openstack/cinder/+/759943 | 15:08 |
| whoami-rajat | walshh_, hey, I've left some questions on the backports you've proposed for powermax driver, can you take a look? https://review.opendev.org/c/openstack/cinder/+/798088 | 15:09 |
| whoami-rajat | and this https://review.opendev.org/c/openstack/cinder/+/798094 | 15:09 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Support Images API v2.11 https://review.opendev.org/c/openstack/cinder/+/755654 | 15:18 |
| simondodsley | geguileo: just added the JSON output for `list-subsys` for FC transport to the doc | 15:38 |
| amar___ | rosmaita : Hi, I have addressed you comments on https://review.opendev.org/c/openstack/cinder/+/790910. Please have a look and share your thoughts. | 16:08 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NetApp ONTAP: Add storage assisted migration support https://review.opendev.org/c/openstack/cinder/+/799668 | 16:17 |
| opendevreview | Venkata krishna Thumu proposed openstack/cinder master: [SVF] Update rccg details for mirror volumes https://review.opendev.org/c/openstack/cinder/+/788782 | 16:37 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Fix: Nova thinks deleted volume is attached https://review.opendev.org/c/openstack/cinder/+/801913 | 16:53 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: 3PAR: Fix extend error for online copy https://review.opendev.org/c/openstack/cinder/+/756709 | 17:00 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: 3PAR: Allow disabling online copy for cloning https://review.opendev.org/c/openstack/cinder/+/756710 | 17:00 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: 3PAR: Fix race in concurrent attachments https://review.opendev.org/c/openstack/cinder/+/756711 | 17:00 |
| opendevreview | Merged openstack/cinder stable/wallaby: LVM: Retry lvextend commands on code 139 https://review.opendev.org/c/openstack/cinder/+/805144 | 17:01 |
| opendevreview | Eric Harney proposed openstack/cinder stable/victoria: LVM: Retry lvextend commands on code 139 https://review.opendev.org/c/openstack/cinder/+/805825 | 17:13 |
| opendevreview | Eric Harney proposed openstack/cinder master: Delete attachment on remove_export failure https://review.opendev.org/c/openstack/cinder/+/801912 | 17:26 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NetApp ONTAP: Add option to report storage provisioned capacity https://review.opendev.org/c/openstack/cinder/+/798198 | 17:27 |
| opendevreview | Merged openstack/cinder stable/train: NetApp ONTAP: Fix QoS lost after moving volume https://review.opendev.org/c/openstack/cinder/+/800231 | 17:27 |
| sfernand | hey folks, some of the changes in this patch appears to be unrelated with the implementation of multi attach support | 18:09 |
| sfernand | https://review.opendev.org/c/openstack/cinder/+/794962 | 18:09 |
| sfernand | I would suggest to remove the fixes from the bp https://blueprints.launchpad.net/cinder/+spec/open-e-joviandss-minor-documentation-and-code-fixes and create a separate patch for it | 18:11 |
| sfernand | Anyone from Open-E online? | 18:14 |
| opendevreview | Fábio Oliveira proposed openstack/cinder master: Netapp ONTAP: Add support to revert to snapshot https://review.opendev.org/c/openstack/cinder/+/804093 | 18:18 |
| opendevreview | Sofia Enriquez proposed openstack/cinder stable/wallaby: Support Images API v2.11 https://review.opendev.org/c/openstack/cinder/+/805877 | 18:23 |
| opendevreview | Fábio Oliveira proposed openstack/cinder master: Netapp ONTAP: Add support to revert to snapshot https://review.opendev.org/c/openstack/cinder/+/804093 | 18:23 |
| sfernand | hey anyone knows Tushar Trambak Gite's irc? | 18:30 |
| eharney | sfernand: just his full name i think, i don't see him on now | 18:32 |
| sfernand | ok! | 18:40 |
| sfernand | just to make sure: Does this kind of patch requires a reno? https://review.opendev.org/c/openstack/cinder/+/773985 | 18:48 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: RBD driver https://review.opendev.org/c/openstack/cinder/+/774288 | 19:04 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: exception.py https://review.opendev.org/c/openstack/os-brick/+/786730 | 19:06 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: utils.py https://review.opendev.org/c/openstack/os-brick/+/786731 | 19:24 |
| *** dviroel|ruck is now known as dviroel|ruck|afk | 19:33 | |
| hemna | mep | 19:39 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: quota.py https://review.opendev.org/c/openstack/cinder/+/788366 | 19:45 |
| opendevreview | Brian Rosmaita proposed openstack/python-cinderclient master: Add functional jobs to the gate https://review.opendev.org/c/openstack/python-cinderclient/+/804115 | 19:52 |
| sfernand | enriquetaso: do you need reviews on this one? https://review.opendev.org/c/openstack/cinder/+/779114. Don't know if it is planned to be merged this release | 19:56 |
| enriquetaso | sfernand, i'm stuck on that bp.. that patch isn't the right approach so abandoning it for now | 20:33 |
| sfernand | ahh ok | 20:33 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: Add snapshot deletion test https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/805884 | 20:36 |
| opendevreview | Merged openstack/cinder master: Support Images API v2.11 https://review.opendev.org/c/openstack/cinder/+/755654 | 20:38 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: WIP Try regular delete for snapshot or deferred deletion https://review.opendev.org/c/openstack/cinder/+/805885 | 20:41 |
| opendevreview | Merged openstack/cinder master: [SVF]:Retype in-use hyperswap volume https://review.opendev.org/c/openstack/cinder/+/782390 | 21:03 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: DNM WIP Try regular delete for snapshot or deferred deletion https://review.opendev.org/c/openstack/cinder/+/805887 | 21:05 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: WIP Try regular delete for snapshot or deferred deletion https://review.opendev.org/c/openstack/cinder/+/805887 | 21:39 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: [DNM] testing NetApp CI https://review.opendev.org/c/openstack/cinder/+/794021 | 21:55 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!