| opendevreview | wanghelin proposed openstack/python-cinderclient master: Fix cinderclient group type update error https://review.opendev.org/c/openstack/python-cinderclient/+/864086 | 06:59 |
|---|---|---|
| sfv880 | Hello, I replied and fixed on https://review.opendev.org/c/openstack/cinder/+/850320 - could you please review? Thank you very much! | 08:11 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Change the function of attach/detach https://review.opendev.org/c/openstack/cinder/+/860997 | 08:15 |
| *** dviroel_ is now known as dviroel | 09:40 | |
| opendevreview | Tobias Urdin proposed openstack/cinder master: Set backup status to error on VolumeNotFound https://review.opendev.org/c/openstack/cinder/+/864100 | 10:15 |
| opendevreview | Merged openstack/cinder master: Update metadata in setup.cfg https://review.opendev.org/c/openstack/cinder/+/855013 | 10:21 |
| opendevreview | Konrad Gube proposed openstack/cinder-specs master: Use assisted volume extend API https://review.opendev.org/c/openstack/cinder-specs/+/864020 | 10:33 |
| opendevreview | Merged openstack/os-brick master: Update metadata in setup.cfg https://review.opendev.org/c/openstack/os-brick/+/855012 | 11:49 |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:59 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin adiare | 13:59 |
| enriquetaso | #startmeeting cinder_bs | 15:00 |
| opendevmeet | Meeting started Wed Nov 9 15:00:22 2022 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:00 |
| enriquetaso | Hey! | 15:00 |
| rosmaita | o/ | 15:00 |
| enriquetaso | We have 6 bugs to discuss today | 15:00 |
| enriquetaso | Full list of bugs | 15:01 |
| enriquetaso | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-November/031133.html | 15:01 |
| enriquetaso | #topic Fail to unify allocation_capacity_gb values among multiple Active-Active Cinder-Volume services | 15:01 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1995204 | 15:01 |
| enriquetaso | Summary: In Active Active deployments, the allocated_capacity_gb returns the partial stats of the hosts instead of return the total cluster's allocated capacity. | 15:01 |
| enriquetaso | As far as I can tell: allocated capacity returns the provisioned volumes created by a single pool. [1] If this is correct, that's why the command is returning a different stat every time that's being used. However, this doesn't look cool and should be an alternative for A/A clusters. | 15:01 |
| enriquetaso | [1] https://specs.openstack.org/openstack/cinder-specs/specs/queens/provisioning-improvements.html | 15:01 |
| enriquetaso | Another thing to highlight is that usually drivers report their stats by pool. For drivers that are not reporting their stats by pool cinder uses the data from the special fixed pool created by _count_allocated_capacity(). | 15:02 |
| enriquetaso | This can be seeing both as a bug and as an new feature. | 15:02 |
| enriquetaso | I know geguileo has been working on this topic for a while. However, maybe another cinder member would like to comment on this | 15:02 |
| enriquetaso | OK, I'll leave some comments on the bug report | 15:06 |
| enriquetaso | #topic cinderclient against wallaby fails to create snapshot | 15:07 |
| enriquetaso | #link https://bugs.launchpad.net/python-cinderclient/+bug/1995883 | 15:07 |
| enriquetaso | Using the latest cinderclient 9.1.0 calling cinder wallaby release, fails to create a snapshot. Because of an invalid input for field/attribute force, the value is None. And None is not of type 'boolean', 'string'. | 15:07 |
| enriquetaso | There were two proposed fixes but one was abandon a couple hours ago | 15:08 |
| enriquetaso | Fix proposed to master | 15:08 |
| enriquetaso | #link https://review.opendev.org/c/openstack/python-cinderclient/+/864027 | 15:08 |
| enriquetaso | I left a comment rosmaita | 15:08 |
| rosmaita | ok, thanks! | 15:09 |
| enriquetaso | moving on | 15:09 |
| enriquetaso | #topic Failed to create multiple instances with boot volumes at the same time in version 20.0.2.dev11. | 15:09 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1995863 | 15:09 |
| enriquetaso | The reporter is not able to boot multiple instances from multiple boot volumes at the same time. The reported mentioned that trying a lower version like cinder 19.1.1 solves the problem. | 15:09 |
| enriquetaso | not sure how to proceed | 15:09 |
| enriquetaso | Maybe a volunteer to reproduce this? | 15:09 |
| enriquetaso | I'm not sure if I can catch this with unittest | 15:10 |
| eharney | the NoneType errors hint at a driver bug | 15:10 |
| rosmaita | enriquetaso: probably not, eharney: my thoughts as well | 15:10 |
| rosmaita | so 20.x is yoga, and 19.x is zena | 15:11 |
| enriquetaso | xena | 15:12 |
| enriquetaso | eharney, what does it means? | 15:13 |
| eharney | it means that the 3par driver owners should look at it | 15:13 |
| enriquetaso | great, thanks eharney | 15:13 |
| enriquetaso | #action(enriquetaso): update bug report | 15:14 |
| enriquetaso | moving on | 15:14 |
| rosmaita | looks like no change in the 3par driver between yoga and xena | 15:14 |
| enriquetaso | #topic Backup fails with VolumeNotFound but not set to error | 15:14 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1996049 | 15:14 |
| enriquetaso | The below is for the Xena release, the actual race should be semi-fixed by the patch to the bug https://bugs.launchpad.net/cinder/+bug/1970768 in Zed | 15:14 |
| enriquetaso | When the backup fails we try to set the volume status but if the volume does not exist we fail on that so the backup is never set to "error" and will be in "creating" state forever until cloud admin fixes the status. | 15:14 |
| enriquetaso | Fix proposed to master: | 15:15 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/864100 | 15:15 |
| eharney | nice find, the fix looks sensible from a quick look | 15:15 |
| enriquetaso | yes.. i was looking at it but we should be careful | 15:16 |
| rosmaita | good point, but it looks like we are already saving and reraising the exception that caused the backup to fail | 15:18 |
| rosmaita | so i think it's probably OK to just swallow the volume-not-found | 15:19 |
| enriquetaso | sounds fine.. maybe we can continue the discussion on the patch | 15:20 |
| opendevreview | Eric Harney proposed openstack/python-cinderclient master: Tests: add coverage for shell group_type_update https://review.opendev.org/c/openstack/python-cinderclient/+/863303 | 15:20 |
| rosmaita | sure | 15:21 |
| enriquetaso | thanks rosmaita | 15:21 |
| enriquetaso | OK, I have marked the last two bugs as incomplete but I think worth the mention: | 15:21 |
| enriquetaso | #topic reimage results to stuck in downloading state | 15:21 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1995838 | 15:21 |
| enriquetaso | The user reported that if we are trying to "reimage" a volume (I'm not quite sure if we is trying to reuse the volume with a different image) with a bigger image that the volume allows, the volume gets stuck in downloading state instead of failing. | 15:21 |
| enriquetaso | maybe i should ask the reporter if the images are encrypted because that would involved the rekey, right? eharney | 15:22 |
| enriquetaso | not sure if that helps but.. | 15:22 |
| eharney | i suspect it isn't that | 15:23 |
| eharney | we should try this w/ nova reimage and see what happens | 15:23 |
| eharney | (to get some logs, etc) | 15:23 |
| opendevreview | Eric Harney proposed openstack/os-brick master: encryptors: Unbind LuksEncryptor and CryptsetupEncryptor https://review.opendev.org/c/openstack/os-brick/+/791271 | 15:24 |
| enriquetaso | #action: try this w/ nova reimage and see what happens (to get some logs, etc) | 15:24 |
| enriquetaso | thanks eharney | 15:24 |
| enriquetaso | #topic Volume State Update Failed After Backup Completed | 15:24 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1996039 | 15:24 |
| enriquetaso | I've market this bug report as incomplete until the reporter provide more information of the version and backend that is being used. | 15:24 |
| enriquetaso | Summary: | 15:24 |
| enriquetaso | Generally, after the backup is created, the state of the volume will be reset to the state before the backup. However, if we create a backup of a volume with status 'in-use'(attached to an instance). When backup creating, the instance get deleted, status will be set to in-use by error, but not reset to available. | 15:24 |
| enriquetaso | When an instance is deleted, the volume will be detached, and the volume status will be set to available. After that, the volume backup is completed, and the status of the volume is reset to in-use. The bug is caused by this. | 15:24 |
| eharney | this is presumably around the whole issue of trying to use the volume status field to communicate the success/failure of backup operations | 15:25 |
| eharney | if we're doing that, we should see if we can stop doing that | 15:25 |
| enriquetaso | good point, haven't thought in that.. | 15:26 |
| enriquetaso | OK, ill add a comment on the bug regarding this so we could focus on why is really falling | 15:30 |
| enriquetaso | thanks everyone | 15:30 |
| enriquetaso | #endmeeting | 15:30 |
| opendevmeet | Meeting ended Wed Nov 9 15:30:24 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:30 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-11-09-15.00.html | 15:30 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-11-09-15.00.txt | 15:30 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-11-09-15.00.log.html | 15:30 |
| *** dviroel is now known as dviroel|lunch | 15:50 | |
| opendevreview | Eric Harney proposed openstack/os-brick master: Remove eventlet from requirements https://review.opendev.org/c/openstack/os-brick/+/863304 | 15:51 |
| opendevreview | Brian Rosmaita proposed openstack/cinderlib master: Open cinderlib for zed development https://review.opendev.org/c/openstack/cinderlib/+/848846 | 15:55 |
| *** dviroel|lunch is now known as dviroel | 16:46 | |
| andrewbogott_ | I don't mean to be a pest but I absolutely cannot figure out how to get attention on my bugfix at https://review.opendev.org/c/openstack/cinder/+/861566. Likely I'm the only deployment actually trying to use that backend for anything but I'd still love a review and/or merge. | 16:54 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Add support for 3-site, trisync, replication. https://review.opendev.org/c/openstack/cinder/+/862365 | 17:02 |
| opendevreview | Eric Harney proposed openstack/python-cinderclient master: Move print operations to shell_utils https://review.opendev.org/c/openstack/python-cinderclient/+/863305 | 17:56 |
| opendevreview | Merged openstack/cinder stable/yoga: rbd: Fix snapshot delete when the source volume doesn't exist https://review.opendev.org/c/openstack/cinder/+/858463 | 18:10 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Add support for 3-site, trisync, replication. https://review.opendev.org/c/openstack/cinder/+/862365 | 18:30 |
| opendevreview | Merged openstack/cinder master: Add doc8 check to docs builds https://review.opendev.org/c/openstack/cinder/+/850700 | 19:10 |
| opendevreview | Merged openstack/python-cinderclient master: Handle downgraded client for snapshot-create https://review.opendev.org/c/openstack/python-cinderclient/+/864027 | 19:38 |
| *** dviroel is now known as dviroel|biba | 19:51 | |
| *** dviroel|biba is now known as dviroel|biab | 19:51 | |
| opendevreview | Brian Rosmaita proposed openstack/python-cinderclient stable/zed: Handle downgraded client for snapshot-create https://review.opendev.org/c/openstack/python-cinderclient/+/864047 | 20:51 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!