| *** dviroel|out is now known as dviroel | 00:20 | |
| *** dviroel is now known as dviroel|out | 00:32 | |
| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/837443 | 02:14 |
|---|---|---|
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Update policy matrix for Zed https://review.opendev.org/c/openstack/cinder/+/835525 | 02:29 |
| opendevreview | Manish proposed openstack/cinder stable/yoga: Dummy commit for Y release on SVC 8.5 for test https://review.opendev.org/c/openstack/cinder/+/838656 | 08:30 |
| *** dviroel|out is now known as dviroel | 11:23 | |
| yuval | Hey, I am trying to understand qos frontend and qos backend and how to configure each | 11:47 |
| yuval | someone can direct me to some doc's about the subject | 11:48 |
| yuval | I see this: https://docs.openstack.org/cinder/latest/admin/basic-volume-qos.html | 11:48 |
| yuval | but dont understand if it refer to frontend qos or back | 11:48 |
| opendevreview | Merged openstack/cinder master: [SVf] Resize of GMCV volumes in group https://review.opendev.org/c/openstack/cinder/+/830787 | 13:13 |
| simondodsley | yuval: that link is for KVM based front-end QoS. Backend QoS has to be defined on a per backend basis, assuming the backend supports QoS in it's driver. Different backends will expose different backend QoS settings | 13:58 |
| 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 | 13:59 |
| geguileo | whoami-rajat: thanks | 13:59 |
| whoami-rajat | np! | 14:00 |
| eharney | jungleboyj: please revisit your -1 on https://review.opendev.org/c/openstack/cinder/+/821893 | 14:55 |
| jungleboyj | eharney: Done. | 14:57 |
| enriquetaso | #startmeeting cinder_bs | 15:04 |
| opendevmeet | Meeting started Wed Apr 20 15:04:55 2022 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:04 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:04 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:04 |
| enriquetaso | Welcome to Cinder Bug Meeting | 15:05 |
| enriquetaso | 8 new bugs! | 15:05 |
| enriquetaso | Full email report: | 15:05 |
| enriquetaso | #link http://lists.openstack.org/pipermail/openstack-discuss/2022-April/028257.html | 15:05 |
| enriquetaso | #link https://etherpad.opendev.org/p/cinder-bug-squad-meeting | 15:05 |
| enriquetaso | #topic [Storwize_SVC] Retype operation failure for GMCV due to update_clean_rate. | 15:05 |
| whoami-rajat | hi | 15:05 |
| fabiooliveira | hi | 15:05 |
| lucasmoliveira059 | hi | 15:05 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1967481 | 15:05 |
| enriquetaso | Cinder Version - Yoga. Retype GMCV operations are failing for several scenarios.No fixes yet. | 15:06 |
| enriquetaso | moving on | 15:06 |
| enriquetaso | hello everyone | 15:06 |
| enriquetaso | #topic [IBM Storwize] Retype failure for replication volume-type | 15:06 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1968159 | 15:06 |
| enriquetaso | Please feel free to review the fix https://review.opendev.org/c/openstack/cinder/+/836944 | 15:06 |
| enriquetaso | next.. | 15:06 |
| enriquetaso | #topic Incorrectly shown volume size while using PowerFlex cinder driver | 15:06 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1968164 | 15:07 |
| enriquetaso | Fix proposed to master https://review.opendev.org/c/openstack/cinder/+/836973 | 15:07 |
| eharney | this fix seems to be incorrect and has a pending review saying so | 15:07 |
| enriquetaso | oh, sorry about that | 15:08 |
| enriquetaso | I need some help with this case: | 15:08 |
| enriquetaso | #topic Cinder fails to retype / migrate large volumes | 15:08 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1969408 | 15:09 |
| enriquetaso | The bug report mentioned that Cinder fails to complete large volume (1TB) retype w/ migration from one ceph custer to another. Small volumes (30GB) complete migration. However, the logs show a 401 Error. "The request you have made requires authentication. (HTTP 401) Code: 401: cinderclient.exceptions.Unauthorized: The request you have made requires authentication. (HTTP 401)"I'm confused, could that be a problem related to working with | 15:09 |
| enriquetaso | big volumes? | 15:09 |
| whoami-rajat | this i think we've discussed before, it's a token expiry issue for long running ops? | 15:09 |
| eharney | IIRC, there is a way to set up keystone tokens so you don't run into this, but i don't know that we document how to do it | 15:09 |
| whoami-rajat | and using service tokens is a way | 15:10 |
| whoami-rajat | but i don't remember exactly | 15:10 |
| rosmaita | there's a doc about this | 15:10 |
| jbernard | is it possible that "bumped up the keystone timeout" wasn't done correctly? | 15:10 |
| rosmaita | https://docs.openstack.org/cinder/latest/configuration/block-storage/service-token.html | 15:10 |
| jbernard | and the default 30 minutes was still in effect when the 1+ hour (yikes) migration occurred? | 15:10 |
| enriquetaso | jbernard, not sure, but I can ask that to the reporter | 15:11 |
| jbernard | i don't know how to verify the change is in effect, but it would be nice to know if the keystone changes were present at runtime | 15:12 |
| enriquetaso | maybe I can reply with the service-token doc and asking those questions | 15:12 |
| enriquetaso | OK, i'll do that, moving on.. | 15:14 |
| enriquetaso | #topic Backend incorrectly reporting cacheable capability | 15:14 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1969366 | 15:14 |
| enriquetaso | When using the LVM cinder driver the cacheable capability is not being reported by the backend to the scheduler when the transport protocol is NVMe-oF (nvmet target driver), but it is properly reported if it's the LIO target driver. | 15:14 |
| enriquetaso | geguileo is going to work on ^ | 15:14 |
| geguileo | enriquetaso: yeah, I have a patch ready | 15:15 |
| geguileo | it's the same from the PTG action of making storage_protocol use constants | 15:15 |
| geguileo | by patch ready, I mean a local patch I'll submit soon | 15:15 |
| enriquetaso | sounds good, thanks! | 15:15 |
| geguileo | funny story, the same thing happens for shared_targets | 15:16 |
| geguileo | when drivers resport 'iscsi' instead of 'iSCSI' as the storage_protocol | 15:16 |
| enriquetaso | oops :P | 15:17 |
| enriquetaso | well.. luckily we have you working on this | 15:18 |
| enriquetaso | #topic pylint tox environment fails on Fedora 35 | 15:18 |
| rosmaita | enriquetaso: ++ | 15:18 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1968048 | 15:18 |
| enriquetaso | "tox -e pylint" fails on Fedora 35. This seems to be due to it running with Python 3.10 and some dependencies not being compatible there. | 15:18 |
| enriquetaso | Fix proposed to master by eharney | 15:18 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/836842 | 15:18 |
| eharney | just a small version bump | 15:19 |
| rosmaita | nice | 15:20 |
| enriquetaso | cool, cinder team please review ^^ | 15:20 |
| enriquetaso | #topic [DEFAULT] use_forwarded_for is a duplicate of the HTTPProxyToWSGI middleware | 15:20 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1967686 | 15:20 |
| enriquetaso | Assigned to Takashi Kajinami but no patch proposed to master yet. | 15:20 |
| enriquetaso | Last bug: | 15:21 |
| enriquetaso | #topic when image properties contain "signature_verified" field，create volume from image will fail | 15:21 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1969213 | 15:22 |
| enriquetaso | If cinder enables image signature verification, image properties contain "signature_verified" field，create volume from image wil fail. | 15:22 |
| enriquetaso | I've closed it as NOTABUG because the error it's the expected behavior I think. | 15:22 |
| eharney | is the message here basically, don't set that field on your glance image? | 15:22 |
| rosmaita | that's a duplicate | 15:22 |
| enriquetaso | #link https://specs.openstack.org/openstack/cinder-specs/specs/rocky/support-image-signature-verification.html | 15:24 |
| rosmaita | https://bugs.launchpad.net/cinder/+bug/1823445 | 15:24 |
| enriquetaso | 'To be clear, we will verify the image’s signature only when image is downloaded from glance and content is copied to volume on host. So when image volume is created via clone_image or clone_image_volume we will skip this verification process regardless of configuration option and provided signature metadata, in order not to confuse end users, we will add verification flag signature_verified in volume’s image metadata when creat | 15:24 |
| enriquetaso | ing from image.' | 15:24 |
| enriquetaso | cool rosmaita, i'll marked it as duplicate | 15:25 |
| eharney | hum i thought we had fixed this | 15:25 |
| rosmaita | not yet | 15:26 |
| enriquetaso | the fix proposed to master isn't merged yet | 15:26 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/764045 | 15:26 |
| eharney | yeah we need to review that :/ | 15:26 |
| enriquetaso | Maybe I can add it to the next XS festival | 15:27 |
| rosmaita | and also https://review.opendev.org/c/openstack/cinder/+/812685 | 15:27 |
| enriquetaso | it's kind of small | 15:27 |
| eharney | it's small, but not simple | 15:27 |
| enriquetaso | true u.u | 15:28 |
| eharney | seems like something we would want tempest coverage for, too | 15:28 |
| enriquetaso | yes.. how are we going to track the tempest ideas ? i think we had a etherpad with ideas a while ago | 15:31 |
| rosmaita | enriquetaso: there are several issues around the bug, i will take the action item to try to organize how these bugs and patches hang together and check back next week | 15:31 |
| *** dviroel is now known as dviroel|lunch | 15:31 | |
| enriquetaso | thanks rosmaita !! | 15:31 |
| eharney | maybe we should create some need-integration-test launchpad bug tag | 15:31 |
| rosmaita | don't thank me yet! | 15:31 |
| enriquetaso | sounds good Eric, we can use a tag.. OK, run out of time | 15:32 |
| enriquetaso | Thank you for joining today | 15:33 |
| enriquetaso | #endmeeting | 15:33 |
| opendevmeet | Meeting ended Wed Apr 20 15:33:23 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:33 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-04-20-15.04.html | 15:33 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-04-20-15.04.txt | 15:33 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-04-20-15.04.log.html | 15:33 |
| rosmaita | thanks enriquetaso | 15:34 |
| enriquetaso | \o/ | 15:37 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: [tests] Deleting Volumes Which are consumed as a basis for others https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/823719 | 16:14 |
| *** dviroel|lunch is now known as dviroel | 16:31 | |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: RBD: flatten child volumes on delete https://review.opendev.org/c/openstack/cinder/+/838756 | 16:35 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: RBD: flatten child volumes on delete https://review.opendev.org/c/openstack/cinder/+/838757 | 16:36 |
| eharney | hum, git-review is doing something weird with my Change-Ids | 16:37 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: RBD: flatten child volumes on delete https://review.opendev.org/c/openstack/cinder/+/835384 | 16:37 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: Fix encryption symlink issues https://review.opendev.org/c/openstack/os-brick/+/836391 | 16:44 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Add statement about CI for backports https://review.opendev.org/c/openstack/cinder/+/838760 | 16:53 |
| opendevreview | Merged openstack/cinder stable/victoria: Fix: Schema validation for attachment create API https://review.opendev.org/c/openstack/cinder/+/836753 | 17:18 |
| opendevreview | Merged openstack/cinder master: [SVf]:Fix retype failure for replication volume-type https://review.opendev.org/c/openstack/cinder/+/836944 | 17:18 |
| opendevreview | Merged openstack/cinder master: Bump pylint to 2.13.4 https://review.opendev.org/c/openstack/cinder/+/836842 | 17:18 |
| opendevreview | Merged openstack/cinder stable/victoria: NetApp ONTAP: Fix sub-clone zapi call https://review.opendev.org/c/openstack/cinder/+/821893 | 17:19 |
| opendevreview | Merged openstack/cinder stable/victoria: Add warning message about slow volume backend https://review.opendev.org/c/openstack/cinder/+/832328 | 17:38 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: [doc] update driver review checklist https://review.opendev.org/c/openstack/cinder/+/838779 | 19:27 |
| hemna | heh added pools support to lvm driver. not sure why it never had that | 20:11 |
| *** dviroel is now known as dviroel|out | 22:27 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!