| opendevreview | Sophie Huang proposed openstack/os-brick master: [WIP] iSCSI connections are not reinitiated after reboot https://review.opendev.org/c/openstack/os-brick/+/810331 | 01:51 |
|---|---|---|
| opendevreview | Merged openstack/cinder stable/xena: Raise min of oslo.policy to 3.8.1 https://review.opendev.org/c/openstack/cinder/+/810198 | 03:46 |
| opendevreview | Merged openstack/cinder master: Add deprecation notices to deprecated policy rules https://review.opendev.org/c/openstack/cinder/+/809894 | 05:34 |
| opendevreview | Merged openstack/cinder master: Specify xena os-brick series in requirements https://review.opendev.org/c/openstack/cinder/+/810118 | 05:34 |
| opendevreview | Merged openstack/cinder stable/xena: Update SQLAlchemy related requirements https://review.opendev.org/c/openstack/cinder/+/810201 | 05:34 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder-specs master: Add spec to optimize upload volume for RBD backend https://review.opendev.org/c/openstack/cinder-specs/+/810363 | 08:05 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder-specs master: Add spec to optimize upload volume for RBD backend https://review.opendev.org/c/openstack/cinder-specs/+/810363 | 08:20 |
| opendevreview | Helen Walsh proposed openstack/cinder stable/victoria: PowerMax Driver - Allow for case mismatch in SGs https://review.opendev.org/c/openstack/cinder/+/810367 | 08:30 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder stable/xena: Add deprecation notices to deprecated policy rules https://review.opendev.org/c/openstack/cinder/+/810205 | 09:34 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder stable/xena: Specify xena os-brick series in requirements https://review.opendev.org/c/openstack/cinder/+/810206 | 09:35 |
| *** dviroel|out is now known as dviroel | 11:18 | |
| whoami-rajat | rosmaita, jungleboyj hey, when you get time can you take a look at this backport, it's the last one remaining in ussuri https://review.opendev.org/c/openstack/cinder/+/809669 | 11:32 |
| rosmaita | whoami-rajat: done | 12:06 |
| whoami-rajat | thanks rosmaita :) | 12:07 |
| opendevreview | Oleg Nesterenkov proposed openstack/cinder master: Closes-Bug: #1929223 https://review.opendev.org/c/openstack/cinder/+/810415 | 12:27 |
| opendevreview | Oleg Nesterenkov proposed openstack/os-brick master: Closes-Bug: #1929223 https://review.opendev.org/c/openstack/os-brick/+/810419 | 12:28 |
| opendevreview | Vladislav Belogrudov proposed openstack/os-brick master: Fix PowerFlex connector HTTPS certificate validation https://review.opendev.org/c/openstack/os-brick/+/810419 | 12:53 |
| whoami-rajat | tosky, rosmaita correct me if I'm wrong but this patch doesn't affect release right? https://review.opendev.org/c/openstack/cinder/+/809448 | 13:48 |
| rosmaita | whoami-rajat: you are correct | 13:49 |
| rosmaita | (though i am sure tosky would like to see it merged!) | 13:49 |
| whoami-rajat | ack | 13:50 |
| rosmaita | Courtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:58 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna ^^ | 13:58 |
| tosky | whoami-rajat: no, it doesn't - I think we may ship the zuul files in the tarballs but I doubt any downstream consumer of the tarballs uses them! | 13:59 |
| enriquetaso | rosmaita, thanks | 13:59 |
| tosky | whoami-rajat: but anyway, wallaby needs to be merge first https://review.opendev.org/c/openstack/cinder/+/809439 | 14:00 |
| tosky | 805887 is cursed, it's going to fail again due to doc8 issues, which are probably infrastructural | 14:05 |
| whoami-rajat | ack, thanks tosky | 14:06 |
| opendevreview | Oleg Nesterenkov proposed openstack/os-brick master: Fix PowerFlex connector HTTPS certificate validation Closes-Bug: #1929223 https://review.opendev.org/c/openstack/os-brick/+/810419 | 14:29 |
| opendevreview | Sofia Enriquez proposed openstack/cinder stable/xena: RBD: Call trash operation when plain deletion fails https://review.opendev.org/c/openstack/cinder/+/810402 | 14:37 |
| opendevreview | Moritz Wanzenböck proposed openstack/cinder master: Add install docs for LINSTOR driver dependencies https://review.opendev.org/c/openstack/cinder/+/773870 | 14:40 |
| enriquetaso | #startmeeting cinder_bs | 15:00 |
| opendevmeet | Meeting started Wed Sep 22 15:00:39 2021 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 | #topic os-brick bug | 15:01 |
| enriquetaso | simondodsley :) | 15:01 |
| simondodsley | Thanks | 15:01 |
| simondodsley | Bug https://bugs.launchpad.net/os-brick/+bug/1944474 was raised yesterday and the customer (a large global telco) are very keen for it to be looked at. | 15:01 |
| simondodsley | It appears that when an error 6 is retuned from iscsiadm there are no reties made even after the latest patch https://review.opendev.org/c/openstack/os-brick/+/796671 which is mmetrged into os-brick 3.0.7 | 15:01 |
| _pewp_ | hemna ( ・_・)ノ | 15:01 |
| simondodsley | The customer would like these retries otherwise they have to keep checking the hypervisor and manually rescanning to get all the paths back. | 15:01 |
| simondodsley | Could someone who is more knowlegeable of the iscsi connector code in os-brick take a look at this? TBH I don't want to mess with os-brick code... | 15:02 |
| enriquetaso | #link https://bugs.launchpad.net/os-brick/+bug/1944474 | 15:02 |
| enriquetaso | Fix proposed: | 15:02 |
| enriquetaso | #link https://review.opendev.org/c/openstack/os-brick/+/810331 | 15:02 |
| simondodsley | Ah - so this is already in the works - great to know | 15:03 |
| simondodsley | can we get this bug added to the commit message for the fix? | 15:04 |
| simondodsley | I love how on the ball you all are... | 15:05 |
| rosmaita | looks like that was sofia's -1 | 15:05 |
| simondodsley | yep - just saw that | 15:05 |
| simondodsley | I'll let the customer know about this patch and they can try it out | 15:05 |
| rosmaita | simondodsley: if you could keep an eye on that patch, and propose backports (you can do it from gerrit) that would be helpful | 15:05 |
| simondodsley | will do | 15:06 |
| rosmaita | we will probably be releasing from ussuri soon after the xena release | 15:06 |
| rosmaita | because i think ussuri goes into EM mode like 2 weeks afterwards | 15:06 |
| rosmaita | so we'll be doing a final ussuri release of everything | 15:07 |
| rosmaita | so simondodsley your other task is to make sure we don't propose a ussuri os-brick release before that fix is included | 15:07 |
| simondodsley | on it | 15:07 |
| whoami-rajat | I will also keep a note | 15:08 |
| rosmaita | cool, ty whoami-rajat | 15:08 |
| enriquetaso | thanks | 15:08 |
| simondodsley | thanks everyone for this rapid response | 15:08 |
| rosmaita | cinder has 2 modes: rapid and glacial | 15:08 |
| rosmaita | :) | 15:09 |
| enriquetaso | Cool, can I move on with the rest of the bugs? | 15:09 |
| enriquetaso | #topic [RBD] Volume can't be removed if other volumes were created from a snapshot of this volume and were not removed | 15:09 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/810402 | 15:10 |
| enriquetaso | -I'm confused about this rosmaita. Do I need to write up a release note as a known issue ? Because the target is RC2 instead of RC1. | 15:10 |
| rosmaita | looking | 15:10 |
| rosmaita | no, your current release note is fine | 15:11 |
| rosmaita | i think we can target this one to rc-2 | 15:11 |
| rosmaita | i just did ... are there any objections to considering this a release-critical bug? | 15:12 |
| rosmaita | ok, hearing no objections, i will add it to the etherpad | 15:13 |
| enriquetaso | thanks! | 15:13 |
| enriquetaso | #topic Announcements | 15:13 |
| enriquetaso | Cinder has six new bugs reported this period. | 15:13 |
| enriquetaso | Three of them are driver related so I won't mention them today. | 15:13 |
| enriquetaso | One it's the cinder-manage db version fails with: AttributeError: 'Engine' object has no attribute 'get_main_option' that we've already mentioned in the main meeting today.#link http://lists.openstack.org/pipermail/openstack-discuss/2021-September/024994.html | 15:13 |
| enriquetaso | oops | 15:13 |
| enriquetaso | #link http://lists.openstack.org/pipermail/openstack-discuss/2021-September/024994.html | 15:13 |
| enriquetaso | The next one is from last week, but I think I should mentioned it here | 15:14 |
| enriquetaso | #topic bug_1 Create a separate [quota] section for quota parameters | 15:14 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1934025 | 15:14 |
| enriquetaso | Summary: Takashi Kajinami proposed to migrate all quota parameters to a new [quota] section so that users and operators can easily understand these options are for quota features. | 15:15 |
| enriquetaso | Manila and Nova have all quota parameters in the [quota] section. | 15:15 |
| enriquetaso | Patch proposed to master: | 15:15 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/772181 | 15:15 |
| eharney | should mark as Wishlist importance | 15:15 |
| enriquetaso | moved to Wishlist | 15:16 |
| rosmaita | we should put this on the PTG etherpad | 15:16 |
| rosmaita | would be good to distinguish quotas from limits | 15:16 |
| rosmaita | and get ready for the unified-quotas stuff | 15:16 |
| enriquetaso | sounds good | 15:16 |
| enriquetaso | OK, that's all I have for today's meeting | 15:17 |
| eharney | i wrote https://bugs.launchpad.net/cinder/+bug/1944577 for the manage+encryption stuff | 15:17 |
| enriquetaso | #topic Open Discussion | 15:17 |
| eharney | definitely needs some work, i'll go chase fixing things up there | 15:17 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1944577 | 15:17 |
| eharney | we block unmanaging encrypted volumes, we probably need to block managing them too | 15:18 |
| eharney | er, managing to encrypted types, that is | 15:18 |
| rosmaita | ok, that makes sense | 15:18 |
| eharney | because doing that would require a user being able to associate a key w/ the volume | 15:18 |
| eharney | and we don't have a way to do that | 15:18 |
| eharney | so you just end up with a volume that isn't put together right | 15:18 |
| rosmaita | good point! | 15:18 |
| eharney | could make that work eventually, but for now we need to at least make it not dangerous | 15:19 |
| rosmaita | ok, i will write up a release note saying don't do this and ping you for reveiw | 15:19 |
| opendevreview | Merged openstack/cinder stable/ussuri: Allow removing NFS snapshots in error status https://review.opendev.org/c/openstack/cinder/+/809669 | 15:19 |
| eharney | ok | 15:19 |
| opendevreview | Sofia Enriquez proposed openstack/cinder stable/train: Allow removing NFS snapshots in error status https://review.opendev.org/c/openstack/cinder/+/810406 | 15:20 |
| rosmaita | eharney: is it easy block managing an encrypted volume? | 15:20 |
| enriquetaso | thanks eharney for the new bug, it's definitely more detailed than the original | 15:20 |
| rosmaita | might be better to just do that now and include it in RC-2 | 15:21 |
| eharney | rosmaita: should be, i just need to, as usual, determine which place in the code to do it | 15:21 |
| eharney | can just do it up front in the API on the request, i guess | 15:21 |
| rosmaita | that's what i was thinking | 15:21 |
| rosmaita | ok, let's touch base on monday and we can see where we are on this | 15:23 |
| enriquetaso | sound good | 15:23 |
| enriquetaso | sounds* | 15:24 |
| enriquetaso | any other bug for the open discussion? | 15:24 |
| enriquetaso | OK | 15:26 |
| enriquetaso | ending meeting, thanks !! | 15:26 |
| enriquetaso | #endmeeting | 15:26 |
| opendevmeet | Meeting ended Wed Sep 22 15:26:16 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:26 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-09-22-15.00.html | 15:26 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-09-22-15.00.txt | 15:26 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-09-22-15.00.log.html | 15:26 |
| rosmaita | thanks enriquetaso | 15:26 |
| pots | Hi folks. I'd really like to close https://bugs.launchpad.net/cinder/+bug/1922255 for Xena. I just need reviews for https://review.opendev.org/c/openstack/cinder/+/808837, https://review.opendev.org/c/openstack/cinder/+/809965, https://review.opendev.org/c/openstack/cinder/+/809965. | 15:26 |
| pots | The first 2 are trivial and the last one is pretty straightforward. CI is green and unit tests and releases notes are included. The only thing missing is your review. :) | 15:27 |
| jungleboyj | pots: Looks good. Can you address the one comment here? https://review.opendev.org/c/openstack/cinder/+/809968 | 15:40 |
| jungleboyj | Easy fix. | 15:40 |
| pots | sure, thanks! | 15:41 |
| opendevreview | Jan Hartkopf proposed openstack/cinder master: Ceph: add option to keep only last n snapshots per backup https://review.opendev.org/c/openstack/cinder/+/810457 | 15:42 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Raise min of oslo.db to 11.0.0 https://review.opendev.org/c/openstack/cinder/+/810465 | 16:03 |
| opendevreview | Chris M proposed openstack/cinder master: Seagate driver: fix get_volume_size() https://review.opendev.org/c/openstack/cinder/+/809968 | 16:04 |
| opendevreview | Oleg Nesterenkov proposed openstack/cinder master: Fix PowerFlex connector HTTPS certificate validation Closes-Bug: #1929223 https://review.opendev.org/c/openstack/cinder/+/810415 | 16:36 |
| bbezak | Hi, I've upgraded today environment to latest stable/victoria build, and I am not able to boot from rbd volume - actually cinder cannot fetch flance images from RBD - it worked on train and ussuri. Ceph is Pacific (16.2.6) - https://paste.opendev.org/raw/809512/ | 16:37 |
| bbezak | Looks like related to this one https://review.opendev.org/c/openstack/cinder/+/808475 (it is not merged to ussuri yet) | 16:37 |
| bbezak | I've tested rbd_store_chunk_size = 8 (the same as in glance). But same issue. | 16:37 |
| opendevreview | Oleg Nesterenkov proposed openstack/os-brick master: Fix PowerFlex connector HTTPS certificate validation Closes-Bug: #1929223 https://review.opendev.org/c/openstack/os-brick/+/810419 | 17:27 |
| -opendevstatus- NOTICE: Zuul has been restarted in order to address a performance regression related to event processing; any changes pushed or approved between roughly 17:00 and 18:30 UTC should be rechecked if they're not already enqueued according to the Zuul status page | 18:36 | |
| enriquetaso | I've forgotten to add this to today's agenda but If anyone is interested in proposing Cinder for the next round of Outreachy, the deadline has been extended to Sep 29thh. http://lists.openstack.org/pipermail/openstack-discuss/2021-September/024992.html | 18:39 |
| opendevreview | Oleg Nesterenkov proposed openstack/cinder master: Fix PowerFlex connector HTTPS certificate validation https://review.opendev.org/c/openstack/cinder/+/810415 | 20:12 |
| opendevreview | Merged openstack/cinder master: Dell PowerVault driver: Fix documentation typos https://review.opendev.org/c/openstack/cinder/+/809965 | 20:50 |
| opendevreview | Merged openstack/cinder master: Fix detach notification https://review.opendev.org/c/openstack/cinder/+/778534 | 23:06 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!