| opendevreview | Jon Bernard proposed openstack/cinder master: RBD: use correct stripe unit in clone operation https://review.opendev.org/c/openstack/cinder/+/786260 | 02:43 |
|---|---|---|
| opendevreview | 王昊 proposed openstack/cinder master: Support to query snapshot filter by updated_at/created_at https://review.opendev.org/c/openstack/cinder/+/733062 | 02:56 |
| opendevreview | yangyang proposed openstack/cinder master: FIX BUG: https://bugs.launchpad.net/cinder/+bug/1567263 https://review.opendev.org/c/openstack/cinder/+/802648 | 02:58 |
| opendevreview | 王昊 proposed openstack/cinder master: Support multiple filters together for volume time comparison filter https://review.opendev.org/c/openstack/cinder/+/740146 | 03:26 |
| opendevreview | Amar proposed openstack/cinder master: [SVF]: Fix to Enable Replication to Storwize_V3700 https://review.opendev.org/c/openstack/cinder/+/802653 | 04:24 |
| opendevreview | saikrishnagk proposed openstack/cinder master: [SVF] Fixed update_rep_properties for empty values https://review.opendev.org/c/openstack/cinder/+/801617 | 04:31 |
| GirishChilukuri | Hi Team | 08:26 |
| GirishChilukuri | can we use "volume_name_template" in volume_type extra specs | 08:27 |
| opendevreview | 江治林 proposed openstack/cinder master: when backend is not support consistency group, creating generic group is successful https://review.opendev.org/c/openstack/cinder/+/802669 | 08:46 |
| *** diablo_rojo__ is now known as diablo_rojo | 09:33 | |
| opendevreview | saikrishnagk proposed openstack/cinder master: [SVF]: Get RCCG State of a Group https://review.opendev.org/c/openstack/cinder/+/800431 | 09:59 |
| *** mabrams is now known as Guest2562 | 11:51 | |
| *** mabrams1 is now known as mabrams | 11:51 | |
| *** mabrams is now known as Guest2563 | 11:53 | |
| *** mabrams1 is now known as mabrams | 11:53 | |
| opendevreview | Zohar Mamedov proposed openstack/os-brick master: NVMeOF Connection Agent https://review.opendev.org/c/openstack/os-brick/+/802691 | 12:10 |
| *** whoami-rajat is now known as Guest2569 | 12:41 | |
| *** whoami-rajat__ is now known as whoami-rajat | 12:41 | |
| rosmaita | Early reminder: today's cinder meeting is on video + #openstack-meeting-alt at 1400 UTC | 13:55 |
| rosmaita | connection info: https://bluejeans.com/3228528973 | 13:55 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir enriquetaso hemna ^^ | 13:55 |
| jungleboyj | Doh! | 13:56 |
| whoami-rajat | enriquetaso_, I've one question about your patch, it says in the commit message that there is possibility of data truncating but your code logic works only for destination volumes greater than source volumes, can you explain this scenario | 14:38 |
| enriquetaso_ | #startmeeting cinder_bs | 14:39 |
| opendevmeet | Meeting started Wed Jul 28 14:39:00 2021 UTC and is due to finish in 60 minutes. The chair is enriquetaso_. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:39 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:39 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 14:39 |
| enriquetaso_ | #topic bug_1: extra_capabilities not added to pool stats | 14:40 |
| enriquetaso_ | #link https://bugs.launchpad.net/cinder/+bug/1937086 | 14:40 |
| enriquetaso_ | Summary: When a driver section is configured with the extra_capabilities entry and the driver is reporting pools in get_volume_stats, the extra_capabilities are not added to each pool entry. | 14:41 |
| simondodsley | Patch for this: https://review.opendev.org/c/openstack/cinder/+/801633 | 14:42 |
| enriquetaso_ | thanks | 14:42 |
| enriquetaso_ | #action (enriquetaso) in addition would be nice to ask gorka. | 14:43 |
| enriquetaso_ | #action (enriquetaso) the fix should have the closes-bug in the commit msg | 14:44 |
| enriquetaso_ | #topic bug_2: retype available volume fails with non root file access | 14:44 |
| enriquetaso_ | #link https://bugs.launchpad.net/cinder/+bug/1938196 | 14:44 |
| eharney | this sounds related to the usual problems around use of the nas_secure options -- we should probably fix it | 14:45 |
| enriquetaso_ | summary The reporter uses Netapp nfs and is configured that only one cinder user (by uid) can write to this share. | 14:45 |
| enriquetaso_ | When running retype on a volume not in-use, it fails, because the manager.py asks to be executed as root. But root is not allowed to write to that nfs share. | 14:45 |
| enriquetaso_ | eharney, nas_secure option is nfs specific option? | 14:47 |
| eharney | yes | 14:47 |
| enriquetaso_ | #action (enriquetaso) add the nas_secure option to the launchpad bug. | 14:48 |
| eharney | it's nfs/remotefs specific, most users just won't hit it because they are running with a configuration where everything is done as root | 14:48 |
| eharney | the nas_secure options to run as a non-root user have had endless problems because they aren't well tested and have never been very robust, presumably the reporter is using that configuration | 14:49 |
| enriquetaso_ | #topic bug_3: misleading message about api version negotiation | 14:50 |
| enriquetaso_ | #link https://bugs.launchpad.net/python-cinderclient/+bug/1938203 | 14:50 |
| enriquetaso_ | Summary: In the scenario where cinderclient supports a maximum API version of 3.64, but the server supports 3.65, the client will say "downgrading to 3.64 based on server support". This is wrong because it is downgrading based on the client, not the server support. | 14:51 |
| enriquetaso_ | #topic bug_4: when the backend does not support consistency groups, creating generic groups is successful and because of this Can't delete the consistency group, which this group backend does not support | 14:52 |
| enriquetaso_ | #link https://bugs.launchpad.net/cinder/+bug/1938240 | 14:52 |
| enriquetaso_ | #link https://bugs.launchpad.net/cinder/+bug/1938256 | 14:52 |
| enriquetaso_ | #topic open discussion | 14:54 |
| enriquetaso_ | #endmeeting | 14:57 |
| opendevmeet | Meeting ended Wed Jul 28 14:57:12 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:57 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-07-28-14.39.html | 14:57 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-07-28-14.39.txt | 14:57 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-07-28-14.39.log.html | 14:57 |
| enriquetaso_ | whoami-rajat, sure: | 15:05 |
| enriquetaso_ | The first scenario is when destination and source volume are the same size. | 15:05 |
| enriquetaso_ | This fix proposed that create_volume_from_snapshot() won't perform any resize. | 15:05 |
| enriquetaso_ | In this way the data won't truncate. | 15:05 |
| enriquetaso_ | The second case scenario is when we would like to increase the size of the destination volume. As far as I can tell this won't face the encryption header problem but we still need to calculate the difference size to provide the size that the user is expecting. | 15:05 |
| enriquetaso_ | i.e. RBD workflow would be something like this: source luks volume would be 1026M, we write some data and create a snap from it. We like to create a new luks volume from a snapshot so the create_volume_from_snapshot() method performs a RBD clone first and then a resize if needed. | 15:07 |
| enriquetaso_ | If the desired size of the destination luks volume is 1G the create_volume_from_snapshot() won't perform any resize and will be 1026M as the parent. | 15:07 |
| enriquetaso_ | the second scenario isn't so obvious because won't face the encryption header problem but we still need to calculate the difference size to provide the size that the user is expecting. | 15:11 |
| whoami-rajat | enriquetaso_, ack, can you summarize the second scenario in a commit msg or add a note in the code, i think it's not obvious from current commit msg which only states the problem with same size volumes. | 15:14 |
| enriquetaso_ | whoami-rajat, sounds good, let me update | 15:27 |
| whoami-rajat | enriquetaso_, thanks, ping me once you update it | 15:27 |
| amar___ | To Core Reviewers, Please have a look at this fix : https://review.opendev.org/c/openstack/cinder/+/782405 and share your reviews/comments | 16:32 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: [rbd] Fix create encrypted volume from snapshot https://review.opendev.org/c/openstack/cinder/+/784623 | 17:45 |
| enriquetaso_ | done whoami-rajat ^ | 17:45 |
| hemna | is sqlalchemy still the cool kid? | 20:22 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: [rbd] Fix create encrypted volume from snapshot https://review.opendev.org/c/openstack/cinder/+/784623 | 21:00 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!