| opendevreview | Merged openstack/devstack-plugin-ceph master: Bump Ceph default version to Quincy in cephadm deploy https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/851769 | 00:18 |
|---|---|---|
| opendevreview | Pete Zaitcev proposed openstack/cinder master: Restore into sparse volumes https://review.opendev.org/c/openstack/cinder/+/852654 | 05:27 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVf]: Attach operation fails when volume and back-end host are on different iogroup. The assignment of the node IP addresses is done outside the try block, and the "for ip_data in lsip_resp" loop. Because of this the loop was not consistent before, so only latest node was detected instead of all nodes https://review.opendev.org/c/openstack/cinder/+/853126 | 06:41 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVf]: mkhost failure when volume and node are on different iogrp https://review.opendev.org/c/openstack/cinder/+/853126 | 06:43 |
| whoami-rajat | geguileo, hey, can you take a look at this https://review.opendev.org/c/openstack/rbd-iscsi-client/+/852783 | 07:39 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin driver: add more tests and improve code https://review.opendev.org/c/openstack/cinder/+/853315 | 07:52 |
| geguileo | whoami-rajat: ok, I downvoted the patch };-) | 08:10 |
| whoami-rajat | ack | 08:50 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: WIP: [NetApp] Fix QoS policy cleanup https://review.opendev.org/c/openstack/cinder/+/853450 | 10:40 |
| *** dviroel|afk is now known as dviroel | 11:32 | |
| gksk | Hi team | 11:44 |
| opendevreview | Merged openstack/cinderlib stable/ussuri: [ussuri-only] Remove cinderlib-lvm-functional job https://review.opendev.org/c/openstack/cinderlib/+/850653 | 12:11 |
| opendevreview | Merged openstack/cinderlib stable/ussuri: [ussuri-only] Don't use yum to install ceph https://review.opendev.org/c/openstack/cinderlib/+/850683 | 12:11 |
| harshailani | Hello reviewers, Please review the feature for SVf driver: https://review.opendev.org/c/openstack/cinder/+/852364 | 12:16 |
| harshailani | It is targeted for Zed release | 12:16 |
| harshailani | Please review the following as well. I was waiting for the IBM CI to be fixed. Its fixed and passed now.: | 12:18 |
| harshailani | https://review.opendev.org/c/openstack/cinder/+/850310 | 12:18 |
| harshailani | https://review.opendev.org/c/openstack/cinder/+/844140 | 12:19 |
| harshailani | https://review.opendev.org/c/openstack/cinder/+/844572 | 12:19 |
| harshailani | https://review.opendev.org/c/openstack/cinder/+/853126 | 12:19 |
| opendevreview | Merged openstack/cinderlib stable/ussuri: Update deprecated zuul syntax https://review.opendev.org/c/openstack/cinderlib/+/850404 | 12:30 |
| opendevreview | Merged openstack/cinderlib stable/ussuri: Migrate jobs from CentOS 8 to CentOS 8 Stream https://review.opendev.org/c/openstack/cinderlib/+/824421 | 12:34 |
| opendevreview | Merged openstack/cinderlib stable/train: [train-only] Update train CI https://review.opendev.org/c/openstack/cinderlib/+/850756 | 12:34 |
| opendevreview | Merged openstack/cinderlib stable/train: Fix pep8 job https://review.opendev.org/c/openstack/cinderlib/+/730147 | 12:34 |
| opendevreview | Merged openstack/cinderlib stable/train: Update deprecated zuul syntax https://review.opendev.org/c/openstack/cinderlib/+/850405 | 12:34 |
| yuval | Hello nice folks, if I want to add some bugfix for my driver to zed release - is it to late? | 13:27 |
| opendevreview | Sergey Drozdov proposed openstack/cinder master: rbd backend qos implementation https://review.opendev.org/c/openstack/cinder/+/820027 | 13:32 |
| yuval | hey this report: http://cinderstats.ivehearditbothways.com/cireport.txt | 13:43 |
| yuval | where I can see the source code? | 13:43 |
| yuval | whoami-rajat ? | 13:44 |
| jungleboyj | yuval: smcginnis runs that service. He isn't on right now, but that is who I would check with. | 13:45 |
| whoami-rajat | yuval, nope, you can propose bug fixes till RC1 (and even after that in exceptions), deadline for RC1 is 16th Sept | 13:45 |
| yuval | thanks | 13:46 |
| yuval | whoami-rajat thanks | 13:46 |
| whoami-rajat | np | 13:46 |
| yuval | in this report: http://cinderstats.ivehearditbothways.com/cireport.txt - our ci (lightbits) is not reporting success rate | 13:47 |
| yuval | any clue what I missing? is it checking the comment text? | 13:47 |
| rosmaita | yuval: compare these 2 gerrit queries: | 13:58 |
| rosmaita | https://review.opendev.org/q/commentby:%2522Yadro+Tatlin+CI%2522 | 13:58 |
| rosmaita | https://review.opendev.org/q/commentby:%2522Yadro+Tatlin+Unified+CI%2522 | 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 tobias-urdin adiare | 13:59 |
| rosmaita | yuval: https://opendev.org/openstack/cinder/src/branch/master/cinder/volume/drivers/yadro/tatlin_iscsi.py#L49 has CI_WIKI_NAME = "Yadro_Tatlin_Unified_CI", but you report as "Yadro Tatlin CI" to gerrit | 13:59 |
| yuval | rosmita the second one is getting 404 not found error | 14:01 |
| rosmaita | yuval: correct | 14:02 |
| rosmaita | i think that's the source of the "Account 'Yadro_Tatlin_Unified_CI' not found" in the cireport | 14:02 |
| yuval | my issue is that I see: "last success: None" in the lightbits section | 14:03 |
| opendevreview | Eric Harney proposed openstack/os-brick master: (WIP) Encryptors: remove execute parameter https://review.opendev.org/c/openstack/os-brick/+/853500 | 14:03 |
| opendevreview | Eric Harney proposed openstack/cinder master: RBD: Flattening of child volumes during deletion https://review.opendev.org/c/openstack/cinder/+/835384 | 14:11 |
| opendevreview | Brian Rosmaita proposed openstack/rbd-iscsi-client master: Replace testing template with individual jobs https://review.opendev.org/c/openstack/rbd-iscsi-client/+/852783 | 14:31 |
| opendevreview | Brian Rosmaita proposed openstack/rbd-iscsi-client master: Address G002 in flake8-logging-format 0.7.4 https://review.opendev.org/c/openstack/rbd-iscsi-client/+/852738 | 14:31 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: NVMe-oF: read mdstat in Python https://review.opendev.org/c/openstack/os-brick/+/836056 | 14:39 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: NVMe-oF: Get system uuid in privsep https://review.opendev.org/c/openstack/os-brick/+/836057 | 14:39 |
| enriquetaso | #startmeeting cinder_bs | 14:42 |
| opendevmeet | Meeting started Wed Aug 17 14:42:14 2022 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:42 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:42 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 14:42 |
| enriquetaso | Hello, welcome back to the cinder bug meeting | 14:42 |
| enriquetaso | Full list of bugs: | 14:42 |
| enriquetaso | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-August/030011.html | 14:42 |
| enriquetaso | Only 3 bugs were reported! Let's see the first one: | 14:42 |
| enriquetaso | #topic wrong volume_quota exception msg when extending volume_size | 14:42 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1985962 | 14:42 |
| enriquetaso | Summary: When extending volume size, the quota exceeds a certain quota class and raises the wrong exception msg to the frontend. It does not consider the quota class now so the exception message below gives the wrong actual quota_size. | 14:43 |
| enriquetaso | I need some help to prioritize this bug, since it's a quota bug maybe geguileo could take a quick look at it :) pls | 14:43 |
| eharney | if it's just that the message is wrong, we should fix it, but not a huge issue IMO | 14:44 |
| enriquetaso | i'll move it to low importance then | 14:45 |
| enriquetaso | thanks eharney | 14:45 |
| * geguileo looks | 14:45 | |
| geguileo | mmmm, both messages from the LP bug look wrong to me... | 14:47 |
| enriquetaso | Fix proposed to master | 14:49 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/852945 | 14:50 |
| enriquetaso | OK so we need to fix the error msg geguileo.. besides that the fix ^ removes the error logging so i think it's not the right way to go | 14:52 |
| eharney | it doesn't remove it, it calls process_reserve_over_quota which logs the error | 14:53 |
| eharney | (whether this is all correct, i don't know) | 14:53 |
| geguileo | enriquetaso: it doesn't remove it, it just uses the process_reserve_over_quota method to do it | 14:53 |
| enriquetaso | oki :) | 14:53 |
| geguileo | (as always eharney was faster, lucky this is not the wild west...) | 14:54 |
| enriquetaso | hahaha | 14:54 |
| enriquetaso | I moved the bug to low importance, please review the fix when you have time. Moving on.. | 14:54 |
| enriquetaso | The last two bugs are driver related and both have fix proposed to master: | 14:55 |
| enriquetaso | #topic [Storwize] mkhost command failure | 14:55 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1985065 | 14:55 |
| enriquetaso | Summary: attach fails Bad or unexpected response from the storage volume backend API: _get_single_iscsi_data: No node found in I/O group 0 for volume volume-4cd0ae48-30b0-41ed-af9f-646fba247e30. | 14:56 |
| enriquetaso | Fix proposed to master | 14:57 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/853126 | 14:57 |
| enriquetaso | Last one: | 14:57 |
| enriquetaso | #topic NetApp driver is hitting the QoS policy limit due soft deletion | 14:57 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1986658 | 14:57 |
| enriquetaso | Summary: Customer cannot create a new volume with Quality of Service (QoS) due the limit of QoS policy on the storage. | 14:59 |
| enriquetaso | felipe_rodrigues, is working on the fix | 15:00 |
| enriquetaso | Fix proposed to master (Zuul vote -1) | 15:00 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/853450 | 15:00 |
| *** adiare is now known as abdi | 15:01 | |
| enriquetaso | That's all i have for today's meeting! | 15:01 |
| enriquetaso | #topic open discussion | 15:02 |
| enriquetaso | Feel free to proposed bugs now | 15:02 |
| rosmaita | i have a question about https://bugs.launchpad.net/cinder/+bug/1735285 | 15:02 |
| rosmaita | abishop is fixing it by a new feature | 15:02 |
| rosmaita | so how do we want to handle the bug? | 15:02 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1735285 | 15:02 |
| rosmaita | i don't see a good Status to change it to | 15:03 |
| abishop | oh, I didn't realize there's a bug against the feature (whoops!) | 15:03 |
| eharney | i think that bug has another "fix" already done | 15:03 |
| rosmaita | or do we want alan to add "Closes-bug: #1735285" to his patch? | 15:03 |
| abishop | maybe the "fix" was to add code that rejected the xfer? | 15:03 |
| rosmaita | eharney: that was declared to be a "related-bug" i think | 15:03 |
| eharney | abishop: correct | 15:03 |
| eharney | the bug was "you can do this and end up horribly broken", the fix was "don't allow users to do this", IIRC | 15:04 |
| rosmaita | ok, i will just close it as fix released | 15:04 |
| rosmaita | thanks! | 15:05 |
| abishop | rosmaita: I can dredge up the patch that did that, if it helps | 15:05 |
| rosmaita | i have it, don't worry | 15:06 |
| rosmaita | from Queens! | 15:06 |
| abishop | "what, me worry?" | 15:06 |
| abishop | not sure how many people get that reference these days | 15:06 |
| rosmaita | i will need to refer to you as albert from now on | 15:06 |
| rosmaita | actually that would not work | 15:06 |
| abishop | lol | 15:06 |
| rosmaita | Alfred! | 15:06 |
| enriquetaso | lol | 15:07 |
| *** dviroel is now known as dviroel|lunch | 15:07 | |
| rosmaita | enriquetaso: are you familiar with Alfred E. Newman? | 15:07 |
| enriquetaso | so #1735285 should be marked as released? | 15:07 |
| enriquetaso | rosmaita, nop :P | 15:07 |
| rosmaita | i didn't even get that right | 15:08 |
| rosmaita | Alfred E. Neuman | 15:08 |
| abishop | it's safe to google | 15:08 |
| enriquetaso | i think i saw that boy somewhere before | 15:08 |
| abishop | rosmaita and I are carbon dating ourselves | 15:09 |
| rosmaita | yes, and i am proving that my memory is shot | 15:09 |
| abishop | enriquetaso: "mad magazine" | 15:10 |
| rosmaita | though this is still kind of relevant: https://en.wikipedia.org/wiki/Mad_(magazine)#/media/File:Mad_Magazine,_August_2017.jpg | 15:10 |
| rosmaita | speaking of my memory being shot, i have a question about https://bugs.launchpad.net/cinder/+bug/1918932 also | 15:11 |
| rosmaita | did we merge a change that prevents a volume transfer if the volume has snapshots, or automatically transfers the snapshots too, or did we just talk about it? | 15:11 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1918932 | 15:11 |
| eharney | it's a little more complicated than that | 15:12 |
| rosmaita | that bug is about the error message not being complete, but one of the parts of the message is "or be disassociated from snapshots after volume transfer" | 15:12 |
| rosmaita | so it made me wonder about the current volume transfer situation | 15:13 |
| eharney | we fixed transfer to transfer snapshots, but left an argument that allows people to get the old behavior (i.e. don't transfer the snapshots, and therefore basically break things) | 15:13 |
| rosmaita | ok | 15:13 |
| eharney | https://review.opendev.org/c/openstack/cinder/+/533564 | 15:13 |
| eharney | if that helps... | 15:13 |
| rosmaita | yes, it does | 15:14 |
| rosmaita | thanks | 15:14 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/533564 | 15:14 |
| opendevreview | Merged openstack/cinder master: Veritas: Eliminate timed delays in unit tests https://review.opendev.org/c/openstack/cinder/+/853324 | 15:14 |
| opendevreview | Merged openstack/cinder master: Inspur: : Eliminate timed delays in unit tests https://review.opendev.org/c/openstack/cinder/+/853325 | 15:15 |
| enriquetaso | Do we need https://review.opendev.org/c/openstack/cinder/+/826608 ? | 15:15 |
| opendevreview | Nahim Alves de Souza proposed openstack/cinder master: NetApp ONTAP: Implemented core functions on REST API https://review.opendev.org/c/openstack/cinder/+/852208 | 15:17 |
| enriquetaso | OK, i think it would be nice to have | 15:17 |
| enriquetaso | That's all I have for today's meeting | 15:17 |
| enriquetaso | anything else to discuss? :) | 15:18 |
| enriquetaso | guess not :) | 15:19 |
| enriquetaso | thank you for joining today! | 15:19 |
| enriquetaso | #endmeeting | 15:20 |
| opendevmeet | Meeting ended Wed Aug 17 15:20:03 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:20 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-08-17-14.42.html | 15:20 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-08-17-14.42.txt | 15:20 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-08-17-14.42.log.html | 15:20 |
| rosmaita | enriquetaso: thanks! | 15:21 |
| opendevreview | Alan Bishop proposed openstack/cinder-tempest-plugin master: Test the ability to transfer encrypted volumes https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/852675 | 15:39 |
| opendevreview | Alan Bishop proposed openstack/cinder-tempest-plugin master: Unblock cinder-tempest-plugin-lvm-multiattach job https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/853513 | 15:39 |
| *** dviroel|lunch is now known as dviroel | 16:12 | |
| opendevreview | Alan Bishop proposed openstack/cinder-tempest-plugin master: Unblock cinder-tempest-plugin-lvm-multiattach job https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/853513 | 18:03 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: os_brick/encryptors/__init__.py https://review.opendev.org/c/openstack/os-brick/+/853457 | 18:05 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: initiator/linuxrbd https://review.opendev.org/c/openstack/os-brick/+/853458 | 18:05 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: [docs] Add info about releases https://review.opendev.org/c/openstack/cinder/+/850698 | 18:24 |
| opendevreview | Sergey Drozdov proposed openstack/cinder master: rbd backend qos implementation https://review.opendev.org/c/openstack/cinder/+/820027 | 18:49 |
| rosmaita | whoami-rajat: geguileo: https://lists.openstack.org/pipermail/openstack-discuss/2022-August/030013.html | 19:43 |
| *** dviroel is now known as dviroel|afk | 21:02 | |
| opendevreview | Alan Bishop proposed openstack/cinder-tempest-plugin master: Test the ability to transfer encrypted volumes https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/852675 | 21:33 |
| opendevreview | Merged openstack/cinder stable/yoga: Log when waiting to acquire coordinator lock https://review.opendev.org/c/openstack/cinder/+/848821 | 21:50 |
| opendevreview | Merged openstack/cinder master: Tests: RBD: Refactor mocks https://review.opendev.org/c/openstack/cinder/+/850548 | 21:50 |
| opendevreview | Sergey Drozdov proposed openstack/cinder master: rbd backend qos implementation https://review.opendev.org/c/openstack/cinder/+/820027 | 22:24 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: [NetApp] Fix QoS policy cleanup https://review.opendev.org/c/openstack/cinder/+/853450 | 23:01 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!