opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/894451 | 03:13 |
---|---|---|
*** ozzzo1 is now known as ozzzo | 07:02 | |
opendevreview | Stephen Finucane proposed openstack/cinder master: db: Don't rely on branched connections https://review.opendev.org/c/openstack/cinder/+/880657 | 13:47 |
opendevreview | Stephen Finucane proposed openstack/cinder master: db: Replace use of Connection.execute https://review.opendev.org/c/openstack/cinder/+/887032 | 13:47 |
opendevreview | Stephen Finucane proposed openstack/cinder master: Add job to test with SQLAlchemy master (2.x) https://review.opendev.org/c/openstack/cinder/+/886152 | 13:47 |
opendevreview | Stephen Finucane proposed openstack/cinder master: WIP: db: Use same session in tests https://review.opendev.org/c/openstack/cinder/+/887033 | 13:47 |
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 happystacker dosaboy hillpd | 13:59 |
opendevreview | Jean Pierre Roquesalane proposed openstack/cinder stable/yoga: Dell PowerMax : Disabling the renaming of a volume before deletion https://review.opendev.org/c/openstack/cinder/+/894922 | 14:03 |
sfv880_ | Hello reviewers, Could you please review https://review.opendev.org/c/openstack/cinder/+/881188 ? This change already have one +2 vote. Thank you very much! | 14:18 |
rosmaita | sfv880_: i'll take a look, looks like i had comments on an earlier version | 14:20 |
*** lpiwowar|PTO is now known as lpiwowar | 14:33 | |
rosmaita | sfv880_: approved the patch, but left a comment for you to think about | 14:44 |
sfv880_ | Hello rosmaita, thank you very much for your review and comment! I will improve all the next patches! | 14:51 |
rosmaita | :D | 14:51 |
sfv880_ | Hello whoami-rajat, thank you very much for your review! | 14:51 |
whoami-rajat | np, i also initially thought we have a new config option but later saw it was the existing one being modified, I think rosmaita has a good point to replace it in next cycle with a new one | 14:54 |
jbernard | quick bug meeting, here we go | 15:12 |
jbernard | #startmeeting cinder_bs | 15:13 |
opendevmeet | Meeting started Wed Sep 13 15:13:04 2023 UTC and is due to finish in 60 minutes. The chair is jbernard. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:13 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:13 |
opendevmeet | The meeting name has been set to 'cinder_bs' | 15:13 |
jbernard | full report | 15:13 |
jbernard | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-September/035073.html | 15:13 |
jbernard | #topic Fail to migrate, resize, evacuate VM with volume state backing-up | 15:13 |
jbernard | #link https://bugs.launchpad.net/cinder/+bug/2034677 | 15:13 |
jbernard | this was seen in a yoga env | 15:15 |
rosmaita | that log extract in the bug looks weird, looks like nova is asking cinder to reserve the volume | 15:16 |
rosmaita | which is strange for an already existing volume | 15:16 |
happystacker | yeah that's weird | 15:17 |
jbernard | do we test vm migration in tempest/devstack jobs? | 15:18 |
happystacker | do we have to wait for the back up to complete before making any further action? | 15:18 |
jbernard | i wonder if a tempest test could reinforce this bug | 15:18 |
jbernard | happystacker: i would think, if this were no allowed, that nova would refuse the operation if the backing volume wasn't in the correct state.. | 15:20 |
rosmaita | i'm pretty sure there are migration tests, but probably not when a volume is being backed up | 15:20 |
happystacker | yes that's my thinking too | 15:20 |
happystacker | I can try to recreate the issue | 15:23 |
jbernard | this one needs a closer look by us | 15:24 |
jbernard | happystacker: if you're able to, especially in a current env, steps and findings would be great | 15:24 |
happystacker | ok let me see if I can find anything relevant | 15:24 |
jbernard | thanks | 15:24 |
jbernard | ok, next one: | 15:25 |
jbernard | #topic Dell PowerMax Live Migration Fails Without a Pool Name | 15:25 |
jbernard | #link https://bugs.launchpad.net/cinder/+bug/2034937 | 15:25 |
jbernard | although the title suggests it might be driver-specific | 15:27 |
happystacker | I have powermax env available, I can test | 15:27 |
jbernard | it appears this could be a general bug in the volume manager | 15:27 |
jbernard | a second comment adds additional info | 15:27 |
jbernard | this one should be fairly easy to reproduce | 15:27 |
happystacker | yes I'll ask for command and steps to recreate the issue | 15:28 |
jbernard | happystacker: that would help, this one looks like something we can/should fix quickly | 15:28 |
happystacker | absolutely | 15:29 |
jbernard | ok, | 15:29 |
jbernard | #topic Cinder Backups Fail with cinder.exception.ServiceNotFound with multiple availibility zones | 15:29 |
jbernard | #link https://bugs.launchpad.net/cinder/+bug/2035007 | 15:29 |
jbernard | ive not used juju before, im not sure if that info is relevant or not | 15:30 |
jbernard | rosmaita: ^ could this be an issue with how juju deploys/configures cinder? | 15:31 |
rosmaita | looking | 15:31 |
rosmaita | i don't think it's a juju issue | 15:35 |
rosmaita | left a comment for some more info | 15:38 |
jbernard | thanks rosmaita | 15:38 |
jbernard | ok, last one: | 15:38 |
jbernard | #topic Dell PowerFlex (Scaleio) connector doesn't handle volume disconnection/unmapping properly | 15:39 |
jbernard | #link https://bugs.launchpad.net/os-brick/+bug/2034685 | 15:39 |
jbernard | this sounds like what gorka deals with a lot | 15:40 |
jbernard | who is our scaleio person? | 15:41 |
happystacker | that's me | 15:41 |
happystacker | ScaleIO is PowerFlex | 15:41 |
jbernard | this might not be a scaleio bug, but os-brick in genera | 15:42 |
jbernard | general | 15:42 |
jbernard | some version/env info would be helpful | 15:42 |
happystacker | yes that was also my discussion with them | 15:43 |
happystacker | I'll ask them more info | 15:43 |
jbernard | i left a comment | 15:44 |
jbernard | happystacker: if it's a common env, you might have the necessary backend access to verify | 15:44 |
jbernard | ok, that's all | 15:45 |
jbernard | #topic open discussion | 15:45 |
jbernard | alright, thanks | 15:48 |
jbernard | #endmeeting | 15:48 |
opendevmeet | Meeting ended Wed Sep 13 15:48:30 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:48 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-09-13-15.13.html | 15:48 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-09-13-15.13.txt | 15:48 |
opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-09-13-15.13.log.html | 15:48 |
stephenfin | This is pretty late in the day, but there are two outstanding patches needed to close out sqla 2.0 support https://review.opendev.org/c/openstack/cinder/+/880657/ https://review.opendev.org/c/openstack/cinder/+/887032/ | 17:11 |
stephenfin | There's another issue that I've yet to get to the bottom of, but it only affects tests | 17:11 |
stephenfin | (see comments on https://review.opendev.org/c/openstack/cinder/+/886152/) | 17:12 |
opendevreview | Merged openstack/cinder master: Fix Infinidat driver to inherit compression https://review.opendev.org/c/openstack/cinder/+/881188 | 17:28 |
opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Fix missing pod causing driver failure https://review.opendev.org/c/openstack/cinder/+/895008 | 23:07 |
*** chuanm41 is now known as chuanm4 | 23:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!