| *** dviroel is now known as dviroel|pto | 00:40 | |
| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/856002 | 03:19 |
|---|---|---|
| whoami-rajat | thanks rosmaita | 05:07 |
| geguileo | whoami-rajat: I have made some clarifications to your comment on https://review.opendev.org/c/openstack/cinder/+/851640 | 08:47 |
| geguileo | whoami-rajat: it's not as crazy as it sounds, there are other drivers doing it, but I would like to hear some confirmation on the Infinidat storage capabilities | 08:47 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin driver: add more tests and improve code https://review.opendev.org/c/openstack/cinder/+/853315 | 09:09 |
| whoami-rajat | geguileo, ack, thanks for looking, it didn't seem right since they're performing same thing as creating a volume from snapshot in clone_volume code but if it's a limitation that they've to workaround like this, we can't do much there | 09:15 |
| whoami-rajat | my idea was just if they could explore other ways to fix the issue raised in commit message | 09:15 |
| geguileo | whoami-rajat: yeah, I'm trying to do the same thing, that's why I'm asking if they can attach snapshots and asking for confirmation of current available features in the storage system | 09:16 |
| whoami-rajat | yep, that would be a good alternative | 09:17 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 09:38 |
| opendevreview | yuval proposed openstack/cinder master: Lightos - add lightos new volume states https://review.opendev.org/c/openstack/cinder/+/856259 | 09:46 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 09:50 |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder master: PowerMax Driver - Fix for renaming GVG https://review.opendev.org/c/openstack/cinder/+/801354 | 13:53 |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 14:00 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin adiare | 14:00 |
| enriquetaso | thanks whoami-rajat | 14:00 |
| whoami-rajat | np | 14:00 |
| enriquetaso | #startmeeting cinder_bs | 15:02 |
| opendevmeet | Meeting started Wed Sep 7 15:02:26 2022 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:02 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:02 |
| whoami-rajat | Hi | 15:02 |
| enriquetaso | Hello, welcome to the bug meeting | 15:02 |
| enriquetaso | we have 2 bugs for today's meeting and two old bugs | 15:03 |
| rosmaita | o/ | 15:03 |
| enriquetaso | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-September/030380.html | 15:03 |
| enriquetaso | #topic With image-volume cache enabled backend, volume creation for a smaller volume than image-volume cache has the same size as the image-volume cache. | 15:03 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1988803 | 15:03 |
| enriquetaso | Long title. | 15:03 |
| enriquetaso | This issue has a kind of long history on the Bugzilla link included in the launchpad report for those interested in the case. | 15:04 |
| enriquetaso | Summary: Cinder creates a volume whose requested size is smaller than the image cache volume and has the same size as the image cache volume. That results in an inconsistency between Cinder and the storage backend. | 15:04 |
| enriquetaso | The reporter forgot to mention the backend but I think it's reproducible throw the backends. | 15:04 |
| enriquetaso | The reporter suggests a possible solutions: https://bugs.launchpad.net/cinder/+bug/1988803/comments/1 | 15:05 |
| enriquetaso | Quoting abishop: Any design changes like that would require writing a spec and more. | 15:05 |
| enriquetaso | Should we treat this like a bug or as a feature improvement? | 15:05 |
| whoami-rajat | I was thinking at looking at it at some point | 15:05 |
| whoami-rajat | s/at/of | 15:06 |
| whoami-rajat | this looks doable | 15:06 |
| rosmaita | seems like a bug if cinder says the volume is 1GB but it is really 5GB | 15:06 |
| abishop | yes, that aspect is definitely a bug | 15:07 |
| whoami-rajat | i think the second one is the way to go, just create the volume as big as required by the image | 15:07 |
| eharney | i agree, it's a bug | 15:07 |
| abishop | some of the discussions that proposed changes got complicated | 15:07 |
| enriquetaso | yes, i got confused from the discussion | 15:08 |
| anskiy | I've filed this change, that addresses similiar issue, but I've found, that it doesn't resize create volume _file_ to the volume size at all: https://review.opendev.org/c/openstack/cinder/+/855964 | 15:08 |
| yuval__ | I did saw the patch and was not sure why its needed | 15:10 |
| yuval__ | I see it was trying to workaround this bug... | 15:10 |
| whoami-rajat | left a comment on the patch, looks like it won't work if the new volume has size < original volume since we don't support shrinking a volume, resize=extend | 15:11 |
| abishop | some proposed (in email discussions) changes felt rather "opionated" to me (i.e. behavior that not all operators would want), which is why I felt it was time for a spec | 15:12 |
| anskiy | yuval__: I saw your comment, just didn't have time to respond properly, sorry. No, I wasn't trying to workaround it, I've found this problem the other way around: small image, big volume and on NFS (haven't seen such problems with LVM driver, and not sure about the others, so fix is for NFS only) | 15:12 |
| rosmaita | i was just going to ask, is there any reason to think this is NFS-specific? | 15:13 |
| whoami-rajat | I think it's generic to all drivers, as our image cache feature is | 15:13 |
| rosmaita | that's my thought too | 15:13 |
| eharney | seems like it would be for any driver | 15:14 |
| anskiy | it's broken in the point, where new volume create from snapshot of other volume -- the snapshot size is set to the original volume size | 15:14 |
| anskiy | in qemu-img command, that is. | 15:14 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin driver: add more tests and improve code https://review.opendev.org/c/openstack/cinder/+/853315 | 15:14 |
| rosmaita | anskiy: i see, the snapshot could be really big, but is going to be set to whatever cinder thinks the original volume size is, which could be pretty small | 15:15 |
| anskiy | yes, and with LVM driver it is not triggered, because it creates snapshot via lv* things. I'm totally unsure about drivers besides these ones, sorry | 15:16 |
| anskiy | so that's why I've sent this change specifically for NFS driver | 15:17 |
| rosmaita | ok | 15:17 |
| enriquetaso | Makes sense. However, looks like we should probably go for a generic / driver-independent approach if possible. | 15:18 |
| rosmaita | agreed | 15:19 |
| enriquetaso | Not sure to understand the snapshot case correctly... Should we create a new bug report for the snapshot case + nfs driver? | 15:20 |
| enriquetaso | anskiy, would you mind creating a launchpad report describing the snapshot size issue you mentioned. It would be nice to link it to https://review.opendev.org/c/openstack/cinder/+/855964 too | 15:22 |
| rosmaita | my impression is: create a 10GB volume A from an image with image-cache enabled, then create a 1 GB volume B from the same image, then use volume B and add 2GB of data to it, then snapshot volume B, create volume C from the snapshot, volume C gets resized to 1 GB | 15:23 |
| rosmaita | and the "extra" data goes missing | 15:23 |
| rosmaita | anskiy: is that roughly correct? | 15:23 |
| anskiy | rosmaita: nah, it's easier :) create 10G volume A from image with image-cache, then it's _real_ size on filesystem (qemu-img info, ls...) would be the size of cache volume. | 15:25 |
| anskiy | it just gets created via qemu-img snapshot mechanically | 15:27 |
| eharney | sounds worth investigating | 15:28 |
| rosmaita | ok, hopefully i misunderstood and there's not a data loss issue here | 15:28 |
| anskiy | enriquetaso: sure, I hope to find some time to do it this week | 15:29 |
| rosmaita | anskiy: sounds like what you are describing is the same as https://bugs.launchpad.net/cinder/+bug/1988803 , just for NFS | 15:30 |
| anskiy | rosmaita: yeah, looks similar, and I can't see with which driver this bug happens, as I've said earlier, it's fine with LVM... | 15:34 |
| enriquetaso | OK, looks like worth investigating so please anskiy fill a bug on cinder launchpad please :D | 15:39 |
| enriquetaso | moving on | 15:39 |
| enriquetaso | #topic Remove IET iSCSI target | 15:40 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1988317 | 15:40 |
| enriquetaso | As we discussed in our last meeting I've created a bug report to remember us that we need to deprecate IET ISCSI target. Looks like a low-hanging-fruit to me. | 15:40 |
| eharney | it's already deprecated -- we need to remove it :) | 15:40 |
| enriquetaso | oh sorry about that | 15:41 |
| rosmaita | yes, the deprecation period is over, and we forgot to remove it | 15:42 |
| enriquetaso | any volunteer to do it? :) Maybe I can work with an inter with this | 15:42 |
| rosmaita | so we can remove it whenever we want to | 15:42 |
| rosmaita | i think it might be a good intern bug | 15:42 |
| rosmaita | not critical, but needs to get done | 15:42 |
| enriquetaso | sounds good then :) | 15:43 |
| enriquetaso | OK so the old and last bug for today's meeting is: | 15:44 |
| enriquetaso | #topic Out of space when migrating encrypted volume between two LVM backends | 15:44 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1720885 | 15:44 |
| enriquetaso | It would be nice to check if this is still a problem present in the LVM backend. Looks like a problem, we would like to fix this since it's kind of a basic functionality between encrypted volumes. | 15:44 |
| enriquetaso | Do you have any thoughts? It would be nice if a great person offer to confirm this and/or work on this | 15:44 |
| enriquetaso | Since we don't have updates on this bug in a long time. | 15:44 |
| enriquetaso | If nobody would like to confirm I'll try to do it by the end of this week. | 15:45 |
| eharney | isn't this the encryption header size issues? | 15:45 |
| enriquetaso | I think the header size issues involves mostly volumes that are not encrypted when migrating them to encrypted types. The original volume lacks of space and the operation fails. | 15:48 |
| enriquetaso | I think this may be related of not :/ I'm not sure | 15:49 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: NetApp NFS: Clone image using copy file operation https://review.opendev.org/c/openstack/cinder/+/847732 | 15:52 |
| enriquetaso | OK, we are running out of time | 15:59 |
| enriquetaso | thank you for attending | 16:01 |
| enriquetaso | #endmeeting | 16:01 |
| opendevmeet | Meeting ended Wed Sep 7 16:01:34 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-09-07-15.02.html | 16:01 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-09-07-15.02.txt | 16:01 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-09-07-15.02.log.html | 16:01 |
| whoami-rajat | sfernand, left a comment on the deprecation patch, i think we require a releasenote mentioning it as well | 16:17 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: encryptors: Unbind LuksEncryptor and CryptsetupEncryptor https://review.opendev.org/c/openstack/os-brick/+/791271 | 17:22 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: privsep: Move luks utils to privsep https://review.opendev.org/c/openstack/os-brick/+/791274 | 17:22 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: privsep: Convert 'os_brick.caches.opencas' https://review.opendev.org/c/openstack/os-brick/+/791275 | 17:22 |
| *** dviroel is now known as dviroel|out | 17:58 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!