| opendevreview | Rajat Dhasmana proposed openstack/cinder master: WIP: Remove leftover nfs backup dirs https://review.opendev.org/c/openstack/cinder/+/804603 | 04:48 |
|---|---|---|
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: HPE 3PAR: Reuse existing session https://review.opendev.org/c/openstack/cinder/+/804698 | 07:10 |
| opendevreview | Vladislav Belogrudov proposed openstack/os-brick master: Add support for multiple volumes within subsystem to NVMe-OF connector https://review.opendev.org/c/openstack/os-brick/+/800014 | 07:18 |
| venkatakrishnat | Hi eharney, hemna, jungleboyj, whoami-rajat: In your available time, please review https://review.opendev.org/c/openstack/cinder/+/792800 to modify the rccg_name format on IBM SVC cinder driver. CI also passed. | 09:40 |
| opendevreview | Zohar Mamedov proposed openstack/os-brick master: NVMeOF Connection Agent https://review.opendev.org/c/openstack/os-brick/+/802691 | 10:47 |
| opendevreview | Girish Chilukuri proposed openstack/cinder master: [SVF]:Retype in-use hyperswap volume https://review.opendev.org/c/openstack/cinder/+/782390 | 10:58 |
| *** brinzhang0 is now known as brinzhang | 11:05 | |
| *** dviroel|ruck|out is now known as dviroel|ruck | 11:12 | |
| raghavendrat | test message | 12:03 |
| raghavendrat | test msg 2 | 12:19 |
| opendevreview | Zohar Mamedov proposed openstack/os-brick master: NVMeOF Connection Agent https://review.opendev.org/c/openstack/os-brick/+/802691 | 12:41 |
| opendevreview | yangheng proposed openstack/cinder master: Support Cinder FC driver for TOYOU NetStor https://review.opendev.org/c/openstack/cinder/+/803000 | 12:51 |
| walshh_ | Hi all, would really appreciate eyes on https://review.opendev.org/c/openstack/cinder/+/782995 which is customer issue that I need to get backported ASAP. It has clean CI and feedback has been applied. Much appreciated. Helen | 13:26 |
| opendevreview | yangheng proposed openstack/cinder master: Support Cinder FC driver for TOYOU NetStor https://review.opendev.org/c/openstack/cinder/+/803000 | 13:26 |
| opendevreview | Eric Harney proposed openstack/cinder master: LVM: Retry lvextend commands on code 139 https://review.opendev.org/c/openstack/cinder/+/805026 | 13:52 |
| opendevreview | Emilien Macchi proposed openstack/cinder master: API/logging: improve message when a volume can't be removed https://review.opendev.org/c/openstack/cinder/+/805027 | 13:54 |
| opendevreview | Emilien Macchi proposed openstack/cinder master: API/logging: improve message when a volume can't be removed https://review.opendev.org/c/openstack/cinder/+/805027 | 13:54 |
| rosmaita | Courtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:58 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir enriquetaso hemna ^^ | 13:58 |
| shoffmann | Like discussed last week I proposed an overview about nas_secure options. I guess, we won't have time at todays meeting but maybe you can have a look later and give some hints, what is missing and how we can proceed. https://etherpad.opendev.org/p/gSotXYAZ3JfJE8FEpMpS | 14:03 |
| rosmaita | shoffmann: ack | 14:08 |
| venkatakrishnat | Hi rosmaita, Addressed your comment on https://review.opendev.org/c/openstack/cinder/+/792800. Please review in your available time | 14:16 |
| enriquetaso | OK, cinder bug meeting :) | 15:01 |
| enriquetaso | maybe two meetings in a row it's too much | 15:01 |
| enriquetaso | #startmeeting cinder_bs | 15:01 |
| opendevmeet | Meeting started Wed Aug 18 15:01:38 2021 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:01 |
| enriquetaso | #link https://etherpad.opendev.org/p/cinder-bug-squad-meeting | 15:01 |
| enriquetaso | Hello, welcome to the bug meeting, we have a bunch of bugs for this week. Full list of bugs: | 15:01 |
| enriquetaso | #link http://lists.openstack.org/pipermail/openstack-discuss/2021-August/024220.html | 15:01 |
| enriquetaso | The way I used to request the new bugs to Launchpad is failing or something so I lost track of most of the new reported bugs. For this week's report I manually checked the launchpad. If anyone knows how to filter dates on Launchpad that actually works please let me know. | 15:02 |
| enriquetaso | OK.. the first two bugs are High importance bugs and that's why I've mentioned them here but both have patches proposed to master. | 15:02 |
| enriquetaso | #topic bug_1 "Original volume size is lost when uploading a volume to a qcow2 image" | 15:03 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1939972 | 15:03 |
| enriquetaso | Summary: The virtual size of the image(2GB) doesn't fit in a volume size(1GB), but this error is not exposed to users. This makes it difficult to let users know the actual cause of the error. | 15:03 |
| whoami-rajat | enriquetaso, i think you can Order by: Date last updated | 15:03 |
| enriquetaso | whoami-rajat, yep, i need to double check again I guess, because it wasnt working for me today | 15:03 |
| whoami-rajat | ok | 15:04 |
| enriquetaso | Also, regarding the bug_1 Rajat proposed a new user message that will notify users regarding any async failure with the given operation. In addition, the reporter wrote that cinder already has a mechanism to validate a given volume size based on the min_disk property of a source image. | 15:04 |
| whoami-rajat | yeah so i will be working on the user message part for this bug (as I've a series of patches for user-messages and can add this in one of them) | 15:05 |
| enriquetaso | whoami-rajat++ | 15:05 |
| enriquetaso | thanks | 15:05 |
| enriquetaso | moving on.. | 15:05 |
| rosmaita | i guess cinder could set virtual_size for glance, and/or min_disk when we upload | 15:05 |
| enriquetaso | The patch proposed upstream uses min_disk https://review.opendev.org/c/openstack/cinder/+/804584 | 15:06 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/804584 | 15:06 |
| rosmaita | min_disk would be better, we check that | 15:06 |
| rosmaita | thanks for the link | 15:06 |
| rosmaita | that sounds like a good solution | 15:07 |
| whoami-rajat | glance has code inplace to set virtual_size after creating an image, maybe they can add that to this flow as well | 15:07 |
| enriquetaso | do you mind doing that whoami-rajat ? I'm not sure what should glance do | 15:08 |
| enriquetaso | OK, i can check that later, moving on.. | 15:09 |
| enriquetaso | #topic bug_2 " Nova thinks deleted volume is still attached" | 15:09 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1937084 | 15:09 |
| enriquetaso | Summary: There are cases where a cinder volume no longer exists yet nova still thinks it is attached to an instance and we cannot detach it anymore. This has been observed when running cinder-csi, where it makes a volume delete request as soon as the volume status says it's available. | 15:09 |
| enriquetaso | Fix proposed to branch: master | 15:10 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/801913 | 15:10 |
| enriquetaso | Please review when you have time, nothing else to add there from my side and I think gorka is on holiday | 15:10 |
| rosmaita | and it's #4 in a series of patches! | 15:11 |
| eharney | the whole series of 4 patches is stuck on a merge conflict on the first patch that needs resolution | 15:11 |
| opendevreview | Eric Harney proposed openstack/cinder master: Fix detach notification https://review.opendev.org/c/openstack/cinder/+/778534 | 15:12 |
| enriquetaso | oh true, sorry. forgot to mention that | 15:12 |
| rosmaita | i think eric just resolved the conflict | 15:12 |
| eharney | rebased ^ | 15:13 |
| enriquetaso | eharney++ | 15:13 |
| opendevreview | Eric Harney proposed openstack/cinder master: Delete attachment on remove_export failure https://review.opendev.org/c/openstack/cinder/+/801912 | 15:13 |
| opendevreview | Eric Harney proposed openstack/cinder master: Expose volume_attachments in Volume OVO https://review.opendev.org/c/openstack/cinder/+/779576 | 15:13 |
| enriquetaso | #topic bug_3 " Create instance from volume fails on reschedule " | 15:13 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1936474 | 15:13 |
| enriquetaso | whoami-rajat sorry for bothering you here but I think it's easier, are you facing this problem with rbd for both glance and cinder or it's something different? | 15:14 |
| whoami-rajat | enriquetaso, it's backend independent, I've also replied on the bug | 15:14 |
| enriquetaso | thanks! | 15:14 |
| enriquetaso | Last 2 bugs: | 15:14 |
| eharney | should be moved back to "in progress" from "incomplete" | 15:15 |
| enriquetaso | done | 15:15 |
| enriquetaso | #topic bug_4 "Removing a volume in wrong state doesn't report its state" | 15:15 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1940437 | 15:15 |
| enriquetaso | Summary: When trying to remove a volume in a wrong state, the error message says about what's the states should be for the volume to be removable; however we don't report its current state. It would be useful for operators to know about it. | 15:15 |
| enriquetaso | I thought that we already covered this somewhere, but it looks like not. Feel free to reproduce this and review Emilien's patch. | 15:16 |
| eharney | i think the reason we don't do this is because the logged status could be wrong due to race conditions | 15:16 |
| eharney | i remember looking at it a while back | 15:16 |
| eharney | to do it completely accurately you need conditional_update() to report the relevant values back, not query them again later | 15:17 |
| enriquetaso | yes.. sounds a bit tricky | 15:18 |
| enriquetaso | Fix proposed on master | 15:18 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/805027 | 15:18 |
| enriquetaso | guess this fix isn't completely accurate then | 15:19 |
| eharney | i'm not sure, need to take a closer look again | 15:19 |
| enriquetaso | sure | 15:20 |
| enriquetaso | Last one, we've already discussed this in the main meeting but a little reminder. | 15:20 |
| enriquetaso | #topic bug_5 "[LVM] lvextend command crashes with code 139" | 15:20 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1940436 | 15:20 |
| enriquetaso | #action(enriquetaso) take a look to os-brick | 15:20 |
| enriquetaso | The rest of the bugs are driver related sort of bugs and already have patches proposed upstream so feel free to check the list and review them :) | 15:20 |
| enriquetaso | #topic open discussion | 15:20 |
| enriquetaso | Do we have any bugs for open discussion? | 15:20 |
| rosmaita | nothing from me ... thanks for organizing this, sofia | 15:24 |
| enriquetaso | I guess we don't have any bugs for the open discussion then :P | 15:24 |
| enriquetaso | thank you brian | 15:24 |
| enriquetaso | #endmeeting | 15:24 |
| opendevmeet | Meeting ended Wed Aug 18 15:24:23 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:24 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-08-18-15.01.html | 15:24 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-08-18-15.01.txt | 15:24 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-08-18-15.01.log.html | 15:24 |
| enriquetaso | thanks everyone! | 15:24 |
| opendevreview | Merged openstack/cinder master: PowerMax Driver - Fix for create snapshot https://review.opendev.org/c/openstack/cinder/+/782995 | 15:38 |
| venkatakrishnat | Hi eharney, hemna, jungleboyj, rosmaita,whoami-rajat: Could you please review https://review.opendev.org/c/openstack/cinder/+/786524 which is to manage Global mirror change volumes on separate storage pools on IBM SVC cinder driver. | 15:47 |
| opendevreview | Eric Harney proposed openstack/cinder master: LVM: Retry lvextend commands on code 139 https://review.opendev.org/c/openstack/cinder/+/805026 | 16:32 |
| opendevreview | Helen Walsh proposed openstack/cinder stable/wallaby: PowerMax Driver - Fix for create snapshot https://review.opendev.org/c/openstack/cinder/+/805060 | 16:54 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [WIP] Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 17:03 |
| opendevreview | Stefan Hoffmann proposed openstack/cinder master: Try nas_secure access with cinder user at retype https://review.opendev.org/c/openstack/cinder/+/802882 | 17:06 |
| opendevreview | Stefan Hoffmann proposed openstack/cinder master: Try nas_secure access with cinder user at retype https://review.opendev.org/c/openstack/cinder/+/802882 | 17:06 |
| shoffmann | eharney: please check, if this fits your idea to check the file permissions. | 17:08 |
| shoffmann | Detailed review can be done later, I know, other topics are more important right now. But a hint, if this is the right direction would be helpful. | 17:09 |
| kukacz | hi, having cinder configured as glance backend, I am dealing with issue that volumes are byte-streamed from images on creation, instead of being just thin cloned on the cinder backend (powerflex). what might be wrong? | 17:13 |
| rosmaita | kukacz: what release are you using? | 17:39 |
| kukacz | rosmaita: victoria | 17:39 |
| rosmaita | kukacz: i think you may have hit this bug: https://bugs.launchpad.net/cinder/+bug/1898075 | 17:40 |
| opendevreview | saikrishnagk proposed openstack/cinder master: [SVF] Fixed update_rep_properties for empty values https://review.opendev.org/c/openstack/cinder/+/801617 | 17:42 |
| kukacz | rosmaita: yes, it seems so. my image's property includes this location string: `locations='[{'url': 'cinder://cinder/8a2e9a1a-4988-4a4f-a081-6793539ef840', 'metadata': {'store': 'cinder'}}]'` | 17:45 |
| gksk | To the Core reviewers , Please review the below patch related to update replication properties fix for empty values | 17:48 |
| gksk | https://review.opendev.org/c/openstack/cinder/+/801617 | 17:48 |
| rosmaita | kukacz: a few people have expressed interest in fixing it, so hopefully soon | 17:48 |
| kukacz | rosmaita: great , thanks a lot for identifying this! | 17:51 |
| opendevreview | katari manoj kumar proposed openstack/cinder master: [SVF]:Fix add volumes to GMCV group https://review.opendev.org/c/openstack/cinder/+/784092 | 18:22 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Support Images API v2.11 https://review.opendev.org/c/openstack/cinder/+/755654 | 18:33 |
| opendevreview | mitya-eremeev-2 proposed openstack/cinder master: Fix simultaneous volume reservation error. https://review.opendev.org/c/openstack/cinder/+/805072 | 18:33 |
| opendevreview | mitya-eremeev-2 proposed openstack/cinder master: Fix simultaneous volume reservation error. https://review.opendev.org/c/openstack/cinder/+/805072 | 18:38 |
| gksk | To the reviewers please review this patch related to fix for update replication properties fix for empty values | 19:04 |
| gksk | https://review.opendev.org/c/openstack/cinder/+/801617 | 19:05 |
| *** dviroel|ruck is now known as dviroel|ruck|out | 19:07 | |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: mypy: update retry decorator in utils.py https://review.opendev.org/c/openstack/cinder/+/805077 | 19:44 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Support Images API v2.11 https://review.opendev.org/c/openstack/cinder/+/755654 | 20:01 |
| opendevreview | Eric Harney proposed openstack/cinder master: Remove unused config imports/objects https://review.opendev.org/c/openstack/cinder/+/805083 | 20:46 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Test glance with cinder backend optimization https://review.opendev.org/c/openstack/cinder/+/757936 | 22:02 |
| opendevreview | Merged openstack/cinder master: mypy: create_volume flows https://review.opendev.org/c/openstack/cinder/+/780933 | 22:29 |
| opendevreview | Merged openstack/cinder master: mypy: image cache https://review.opendev.org/c/openstack/cinder/+/788363 | 22:29 |
| opendevreview | Merged openstack/cinder master: Doc: IBM Storages user documentation update https://review.opendev.org/c/openstack/cinder/+/803901 | 22:29 |
| opendevreview | Merged openstack/cinder master: LVM: Retry lvextend commands on code 139 https://review.opendev.org/c/openstack/cinder/+/805026 | 22:29 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!