| opendevreview | Fernando Ferraz proposed openstack/devstack-plugin-nfs master: [DNM] Run Glance with Cinder as backend https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/940482 | 00:52 |
|---|---|---|
| *** mhen_ is now known as mhen | 01:59 | |
| opendevreview | Fernando Ferraz proposed openstack/devstack-plugin-nfs master: [DNM] Run Glance with Cinder as backend https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/940482 | 02:04 |
| opendevreview | Merged openstack/cinder master: Show volume attachment host information for services https://review.opendev.org/c/openstack/cinder/+/962409 | 02:28 |
| erlon | rosmaita: so, I was following the meeting discussions yesterday, and you mentioned about OS brick being released this week. We are very early in the Gazpacho release right? The Proposal is to have osbrick, this early so people can use it along this release? | 11:37 |
| erlon | I have a pending patch that was in the flamingo bug list, that wasn't released. I just need to sort something with rajat, but I'm finding it hard to get a hold of him to respond my comment there. Unfortunately he was in the meeting yesterday and I couldn't join. | 11:39 |
| erlon | The patch is this one by the way: https://review.opendev.org/c/openstack/os-brick/+/955379 | 11:40 |
| VolodymyrBoiko[m] | simondodsley: abishop simondodsley could you review pls: https://review.opendev.org/c/openstack/cinder/+/939005 | 12:08 |
| rosmaita | erlon: i don't remember saying anything about os-brick yesterday, but the release team likes early library releases, usually before milestone-1, so they would not object if we release earlier | 12:16 |
| rosmaita | so if we get your changes merged, we can have an early G release of os-brick because presumably you will need it for cinder driver changes | 12:17 |
| rosmaita | or let me know if i completely misunderstood what you are asking about | 12:24 |
| sagigrimberg[m] | Hey all, I also wanted to ask about the vastdata driver submission: https://review.opendev.org/c/openstack/cinder/+/939005. It has been sitting waiting for another +1 for a long time now, 8 months now if I'm not mistaken. Can we understand what is needed to get it over the hump (I work with Volodymyr (figured I'd join the group and ask)? | 12:24 |
| rosmaita | sagigrimberg[m]: jbernard was saying yesterday that we will only be using 2 days of the PTG, maybe you could propose a driver review day? | 12:29 |
| rosmaita | i think the issue is just that it's a lot of lines of code, so people tend to put off reviewing until they have more time that never comes | 12:29 |
| rosmaita | another thing would be to put something on the agenda for the cinder weekly meeting (wednesdays at 1400 utc in IRC) to say | 12:29 |
| rosmaita | (1) your driver patch is up, (2) it has third-party CI running and passing, and (3) are there any quirks of your backend that people should be aware of? | 12:29 |
| rosmaita | probably both would be good, because when people show up at the PTG they'll already have heard your discussion at the weekly meeting | 12:30 |
| rosmaita | final thing is to make sure you and your team review some other driver patches, shows that you have good community spirit | 12:31 |
| sagigrimberg[m] | @rosamita we want to help, I know that @Volodymyr Boiko was helping with reviews over the last few weeks. I think we are kind of surprised that a driver inclusion process is a multi-month or potentially a multi-year effort. | 12:34 |
| sagigrimberg[m] | If your suggestion is that we collectively dedicate a day for driver review over the course of say next week (or once every xxx weeks), I am supportive of that. We can also agree on a regular cadance. We just need to understand how we can help our interest move forward, and willing to contribute for that. | 12:38 |
| sagigrimberg[m] | Is there a dial details for the weekly meeting? we would like to join | 12:39 |
| erlon | rosmaita: about the change, it is driver agnostic, and only iSCSI. It should not affect how drivers interact | 12:41 |
| abishop | VolodymyrBoiko[m]: ack, but I already reviewed the patch | 12:45 |
| rosmaita | erlon: ok, get it merged and then poke me or Jon to propose a release | 12:45 |
| rosmaita | sagigrimberg[m]: https://meetings.opendev.org/#Cinder_Team_Meeting | 12:46 |
| erlon | rosmaita: about the getting it merged part, I needed some input from rajat, but I don't know if he's been active lately. I never find him around | 12:47 |
| erlon | *or someone that can judge the discussions we are having there. I think he basically misunderstood what the patch meant to do | 12:47 |
| rosmaita | sagigrimberg[m]: we have a "festival of extra small reviews": https://meetings.opendev.org/#Cinder_Festival_of_XS_Reviews | 12:48 |
| rosmaita | supposed to be monthly, but attendance has been lagging ... maybe you can propose changing it to being focused on driver reviews | 12:48 |
| rosmaita | i think a lot of driver maintainers would support that (and hopefully show up!) | 12:48 |
| rosmaita | erlon: rajat is definitely active, but really busy ... you might want to put an item on the meeting agenda to discuss the change to remove misunderstandings (and make people aware of the importance of the patch) | 12:49 |
| erlon | shure, will do it | 12:50 |
| rosmaita | great! | 12:52 |
| opendevreview | Fernando Ferraz proposed openstack/devstack-plugin-nfs master: [DNM] Run Glance with Cinder as backend https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/940482 | 13:16 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: WIP: Test multiple create volume from image with two passes https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/963558 | 13:30 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: WIP: Test multiple create volume from image with two passes https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/963558 | 13:36 |
| opendevreview | Gireesh Awasthi proposed openstack/cinder master: Added more debug logs for cinder volume creation https://review.opendev.org/c/openstack/cinder/+/963197 | 13:47 |
| jbernard | gireesh: ^ it can be helpful to have 'netapp: ' prefix in your commit message subject line | 13:50 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Remove duplicate key in qos dict https://review.opendev.org/c/openstack/cinder/+/963568 | 14:19 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP Prototype running with threads instead of eventlet https://review.opendev.org/c/openstack/cinder/+/954650 | 14:20 |
| opendevreview | Eric Harney proposed openstack/cinder master: Replace _add_to_threadpool with native threads https://review.opendev.org/c/openstack/cinder/+/959383 | 14:20 |
| opendevreview | Eric Harney proposed openstack/cinder master: Add Slow Fake volume driver https://review.opendev.org/c/openstack/cinder/+/959384 | 14:20 |
| opendevreview | Eric Harney proposed openstack/cinder master: RBD changes for eventlet removal https://review.opendev.org/c/openstack/cinder/+/959385 | 14:20 |
| opendevreview | Eric Harney proposed openstack/cinder master: Fix issue with workers = None https://review.opendev.org/c/openstack/cinder/+/959386 | 14:20 |
| opendevreview | Eric Harney proposed openstack/cinder master: Add thread info logging to volume manager for debug purposes https://review.opendev.org/c/openstack/cinder/+/959387 | 14:20 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP cinder-scheduler threading prototyping https://review.opendev.org/c/openstack/cinder/+/959388 | 14:20 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: WIP: Test multiple create volume from image with two passes https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/963558 | 14:32 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: WIP: Add fake volume driver job https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/963309 | 14:43 |
| *** vhari_ is now known as vhari | 14:57 | |
| Sai | Hi team | 15:10 |
| Sai | https://review.opendev.org/c/openstack/cinder/+/961431 | 15:10 |
| Sai | https://review.opendev.org/c/openstack/cinder/+/961432 | 15:10 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: NFS driver https://review.opendev.org/c/openstack/cinder/+/841315 | 15:10 |
| Sai | Have +2 and request more approvals for the above backporting patches! We have been waiting and any help would be appreciated. Thanks!! | 15:11 |
| Sai | Also would request team to review the below patch please! | 15:19 |
| Sai | https://review.opendev.org/c/openstack/cinder/+/961819 | 15:19 |
| opendevreview | Gireesh Awasthi proposed openstack/cinder master: Added additional debug logs for cinder volume creation and volume update stats https://review.opendev.org/c/openstack/cinder/+/963197 | 15:26 |
| opendevreview | Gireesh Awasthi proposed openstack/cinder master: NetApp - Bug Fix 2124264: Pools not getting populated for ASAr2 platform https://review.opendev.org/c/openstack/cinder/+/961525 | 16:51 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: WIP: Add fake volume driver job https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/963309 | 17:04 |
| opendevreview | Gireesh Awasthi proposed openstack/cinder master: Added additional debug logs for cinder volume creation and volume update stats https://review.opendev.org/c/openstack/cinder/+/963197 | 17:25 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: WIP: Add fake volume driver job https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/963309 | 17:58 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: New snapshot dependency test https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/946703 | 18:26 |
| opendevreview | Eric Harney proposed openstack/cinder master: Add thread info logging to volume manager for debug purposes https://review.opendev.org/c/openstack/cinder/+/959387 | 18:31 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP cinder-scheduler threading prototyping https://review.opendev.org/c/openstack/cinder/+/959388 | 18:31 |
| opendevreview | Eric Harney proposed openstack/cinder master: Add Slow Fake volume driver https://review.opendev.org/c/openstack/cinder/+/959384 | 18:46 |
| opendevreview | Eric Harney proposed openstack/cinder master: RBD changes for eventlet removal https://review.opendev.org/c/openstack/cinder/+/959385 | 18:46 |
| opendevreview | Eric Harney proposed openstack/cinder master: Fix issue with workers = None https://review.opendev.org/c/openstack/cinder/+/959386 | 18:46 |
| opendevreview | Eric Harney proposed openstack/cinder master: Add thread info logging to volume manager for debug purposes https://review.opendev.org/c/openstack/cinder/+/959387 | 18:46 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP cinder-scheduler threading prototyping https://review.opendev.org/c/openstack/cinder/+/959388 | 18:46 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Add extend_volume to fake driver https://review.opendev.org/c/openstack/cinder/+/963618 | 18:46 |
| opendevreview | Akshay Ranade proposed openstack/cinder master: [IBM SVf Driver] Added support for volumegroup-snapshot for temporary volumegroup https://review.opendev.org/c/openstack/cinder/+/926844 | 19:06 |
| opendevreview | Akshay Ranade proposed openstack/cinder master: [IBM SVf Driver] Added support for restorefromsnapshot CLI for volumegroups https://review.opendev.org/c/openstack/cinder/+/916722 | 19:06 |
| opendevreview | Akshay Ranade proposed openstack/cinder master: [IBM SVf Driver] Added support of clone operation for volumegroup https://review.opendev.org/c/openstack/cinder/+/938771 | 19:06 |
| opendevreview | Akshay Ranade proposed openstack/cinder master: Doc: Documentation updates for ibm-svf-volumegroup feature. https://review.opendev.org/c/openstack/cinder/+/929906 | 19:06 |
| opendevreview | Akshay Ranade proposed openstack/cinder master: [IBM SVf driver] Adding support for PBHA partition backend https://review.opendev.org/c/openstack/cinder/+/943364 | 19:06 |
| opendevreview | Akshay Ranade proposed openstack/cinder master: [IBM SVf driver] PBHA Volume Extend Support https://review.opendev.org/c/openstack/cinder/+/949483 | 19:06 |
| opendevreview | Akshay Ranade proposed openstack/cinder master: [IBM SVf driver] Adding support for PBHA volumegroup update https://review.opendev.org/c/openstack/cinder/+/950860 | 19:06 |
| opendevreview | Akshay Ranade proposed openstack/cinder master: [IBM SVf driver] Support PBHA properties reporting https://review.opendev.org/c/openstack/cinder/+/952422 | 19:06 |
| opendevreview | Akshay Ranade proposed openstack/cinder master: [IBM SVf driver] Support PBHA group-snappshot and restore https://review.opendev.org/c/openstack/cinder/+/952432 | 19:06 |
| opendevreview | Akshay Ranade proposed openstack/cinder master: [IBM SVf driver] Adding policing for PBHA partition https://review.opendev.org/c/openstack/cinder/+/958138 | 19:06 |
| opendevreview | Akshay Ranade proposed openstack/cinder master: [IBM SVf driver] Support location affinity to hosts in PBHA partition. https://review.opendev.org/c/openstack/cinder/+/963620 | 19:06 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!