| opendevreview | yangzhe proposed openstack/cinder master: reimage api image invalid,volume status should be collback https://review.opendev.org/c/openstack/cinder/+/850989 | 03:11 |
|---|---|---|
| opendevreview | yangzhe proposed openstack/cinder master: Add volume status rollback in reimage API https://review.opendev.org/c/openstack/cinder/+/850989 | 03:13 |
| opendevreview | yangzhe proposed openstack/cinder master: Add volume status rollback in reimage API https://review.opendev.org/c/openstack/cinder/+/850989 | 06:01 |
| opendevreview | yangzhe proposed openstack/cinder master: Add volume status rollback in reimage API https://review.opendev.org/c/openstack/cinder/+/850989 | 06:02 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Initial commit for Yadro Tatlin.UNIFIED driver https://review.opendev.org/c/openstack/cinder/+/825492 | 07:06 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Add manage related support https://review.opendev.org/c/openstack/cinder/+/851170 | 07:41 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Add manage related support https://review.opendev.org/c/openstack/cinder/+/851170 | 07:44 |
| opendevreview | Alexander Malashenko proposed openstack/cinder master: Add Cinder NFS driver for Dell PowerStore https://review.opendev.org/c/openstack/cinder/+/797608 | 08:17 |
| amalashenko | Hi sfernand: jungleboyj: I updated comments, could you please take a look. https://review.opendev.org/c/openstack/cinder/+/797608 | 08:19 |
| opendevreview | Merged openstack/cinder master: Report tri-state shared_targets for NVMe volumes https://review.opendev.org/c/openstack/cinder/+/836070 | 08:47 |
| opendevreview | Ke Niu proposed openstack/cinder-specs master: remove unicode from code https://review.opendev.org/c/openstack/cinder-specs/+/851181 | 09:49 |
| geguileo | tosky: I have a question regarding zuul change queues for https://review.opendev.org/c/openstack/os-brick/+/836063 | 11:06 |
| geguileo | don't know how familiar you are with those (me 0) | 11:07 |
| tosky | geguileo: with zuul somehow, with the changes in that patch not much | 11:09 |
| geguileo | lol | 11:10 |
| geguileo | thanks | 11:10 |
| tosky | I mean, there are no zuul changes in that patch O.o | 11:10 |
| *** dviroel|out is now known as dviroel | 11:35 | |
| whoami-rajat | enriquetaso_, hey, can you take another look at this patch, the dependent patch you mentioned is merged now https://review.opendev.org/c/openstack/cinder/+/812429 | 11:39 |
| Zenkuro | hi, have anybody faced issues with CI/devstack cinder testing recently? like neutron does not start with previously working local.conf for devstack? | 11:48 |
| enriquetaso_ | sure whoami-rajat | 12:04 |
| whoami-rajat | thanks enriquetaso_ | 12:19 |
| whoami-rajat | geguileo, tosky can you take a look at this? it's regarding making the ceph job voting https://review.opendev.org/c/openstack/os-brick/+/849759 | 12:19 |
| whoami-rajat | (on selected files) ^ | 12:20 |
| geguileo | tosky: if I understood correctly it's complaining about not being a job that can test that os-brick patch with the cinder "depend-on" patch | 12:20 |
| geguileo | whoami-rajat: done | 12:24 |
| whoami-rajat | thanks | 12:32 |
| opendevreview | Sofia Enriquez proposed openstack/devstack-plugin-nfs master: Runs all encrypted scenario tests https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/742905 | 13:30 |
| opendevreview | Sofia Enriquez proposed openstack/devstack-plugin-nfs master: Runs all encrypted scenario tests https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/742905 | 13:32 |
| whoami-rajat | Cinder video + IRC meeting in #openstack-meeting-alt at 1400 UTC | 13:58 |
| whoami-rajat | meeting link: https://bluejeans.com/556681290 | 13:58 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin | 13:58 |
| geguileo | whoami-rajat: will join shortly, trying to start firefox (it takes a couple of minutes) | 14:00 |
| whoami-rajat | geguileo, sure np | 14:00 |
| geguileo | whoami-rajat: and now even chrome has died... | 14:04 |
| whoami-rajat | :( | 14:05 |
| opendevreview | Merged openstack/os-brick master: Make ceph job voting (sometimes) https://review.opendev.org/c/openstack/os-brick/+/849759 | 14:11 |
| *** akekane_ is now known as abhishekk | 14:12 | |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: mypy: NFS https://review.opendev.org/c/openstack/cinder/+/841315 | 14:36 |
| enriquetaso_ | #startmeeting cinder_bs | 14:56 |
| opendevmeet | Meeting started Wed Jul 27 14:56:20 2022 UTC and is due to finish in 60 minutes. The chair is enriquetaso_. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:56 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:56 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 14:56 |
| enriquetaso_ | i'll try to fix bj for next time | 14:56 |
| whoami-rajat | enriquetaso_, with all the issues you were facing i thought it would be difficult so suggested here | 14:56 |
| whoami-rajat | difficult for you* | 14:57 |
| whoami-rajat | Hi | 14:57 |
| enriquetaso_ | whoami-rajat++ | 14:57 |
| enriquetaso_ | thanks | 14:57 |
| enriquetaso_ | Welcome back to the bug meeting, cinder has 6 new bugs | 14:57 |
| enriquetaso_ | #topic Reimage volume API image is invalid, the status should not be downloading | 14:57 |
| *** dviroel is now known as dviroel|lunch | 14:57 | |
| enriquetaso_ | #link https://bugs.launchpad.net/cinder/+bug/1982436 | 14:58 |
| enriquetaso_ | If reimage of a volume fails, the status of the volume is still downloaded. | 14:58 |
| enriquetaso_ | Fix proposed to master:#link https://review.opendev.org/c/openstack/cinder/+/850318The fix has some pep8 issues and whoami-rajat left some comments that haven't been replied to it yet. | 14:58 |
| enriquetaso_ | #link https://review.opendev.org/c/openstack/cinder/+/850318 | 14:58 |
| eharney | this bug seems to have two different patches open for it | 14:58 |
| enriquetaso_ | i see, looks like the second one is the updated | 15:00 |
| enriquetaso_ | #action(enriquetaso): asks to abandon the first patch | 15:00 |
| whoami-rajat | yeah, i will probably update that patch as a co author since i think the author isn't very familiar with unit tests and releasenotes | 15:01 |
| whoami-rajat | (and i want the bug to be fixed) | 15:01 |
| enriquetaso_ | cool, thanks! | 15:01 |
| whoami-rajat | oh so we've 2 patches ... | 15:01 |
| whoami-rajat | i will take a look anyway | 15:01 |
| enriquetaso_ | yes, in the second patch the owner addressed your comments from the first patch | 15:02 |
| whoami-rajat | but i still don't see tests and releasenote | 15:02 |
| eharney | the commit message needs some more info on what it is actually doing | 15:02 |
| enriquetaso_ | probably as you mentioned the author isn't familiar with the way we submit updates on patches | 15:02 |
| whoami-rajat | yeah suggested that as well ^ | 15:02 |
| enriquetaso_ | makes sense | 15:02 |
| whoami-rajat | yep, that too, anyway i will help out with that | 15:03 |
| enriquetaso_ | and the topic is wrong as well | 15:03 |
| whoami-rajat | :D | 15:03 |
| whoami-rajat | too many wrong things in a single patch | 15:03 |
| enriquetaso_ | Sounds good to me! thanks Rajat, moving on with the next one | 15:03 |
| enriquetaso_ | #topic Creating from source volume tries to create it multiple times if rekeying fails | 15:04 |
| whoami-rajat | np | 15:04 |
| enriquetaso_ | #link https://bugs.launchpad.net/cinder/+bug/1982848 | 15:04 |
| enriquetaso_ | geguileo reported this one. When creating an encrypted volume from an encrypted volume the cinder-volume manager code calls the driver to clone a volume multiple times if rekeying fails. | 15:04 |
| geguileo | yeah, I think eharney was interested on that one | 15:05 |
| enriquetaso_ | geguileo, one question, you only see this issue using RBD or it's a general problem? | 15:05 |
| geguileo | enriquetaso_: the problem is general | 15:06 |
| eharney | it's not an RBD problem, it's an exception handling problem in the create volume flow | 15:06 |
| geguileo | what he said ;-) | 15:06 |
| enriquetaso_ | nice | 15:06 |
| enriquetaso_ | OK, we don't have fix proposed or anyone assigned | 15:06 |
| enriquetaso_ | so for now, it's just confirmed | 15:07 |
| enriquetaso_ | Triaged * | 15:07 |
| enriquetaso_ | Thanks! | 15:07 |
| eharney | i'll assign it to me since nobody else has it | 15:07 |
| enriquetaso_ | thanks Eric | 15:08 |
| eharney | (and i probably am the one that introduced it) | 15:08 |
| enriquetaso_ | :P | 15:08 |
| enriquetaso_ | moving on | 15:08 |
| enriquetaso_ | #topic Infinidat driver multi-attach and migration is broken | 15:09 |
| enriquetaso_ | #link https://bugs.launchpad.net/cinder/+bug/1982350 | 15:09 |
| enriquetaso_ | #link https://bugs.launchpad.net/cinder/+bug/1982405 | 15:09 |
| enriquetaso_ | Both bug reports have a fix proposed to master, so feel free to review them | 15:10 |
| enriquetaso_ | moving on | 15:11 |
| enriquetaso_ | #topic PowerMax does not work with workload in extra specs | 15:11 |
| enriquetaso_ | #link https://bugs.launchpad.net/cinder/+bug/1982568 | 15:11 |
| enriquetaso_ | Gorka found this bug last week "According to the PowerMax driver code the workload in a volume type's extra specs should be treated as None. But that doesn't seem to be the case, at least not in all parts of the driver code, and instance live migration doesn't work for attached volumes that have the "pool_name" extra spec with workloads different than "None"." | 15:12 |
| enriquetaso_ | OK, last one | 15:15 |
| enriquetaso_ | #topic NFS Attach Encrypted fails only on the first attempt | 15:15 |
| enriquetaso_ | #link https://bugs.launchpad.net/cinder/+bug/1982891 | 15:16 |
| enriquetaso_ | Plain generic NFS and encrypted NFS do not perform the attach on the first attempt. | 15:16 |
| enriquetaso_ | The operation is shown as successful "attachment_update completed successfully", but immediately afterward c-vol shows "Terminate volume connection completed successfully" and the volume is not attached to the instance. | 15:16 |
| enriquetaso_ | This happens only on the first attempt and works as expected when you try a second or third time. | 15:16 |
| enriquetaso_ | I'm seeing this on my local devstack, but i'd like to see it on upstream ci https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/742905 | 15:17 |
| eharney | i'm not too sure what is going on with the info available on this (need n-cpu info for sure), but i know you're working on it | 15:18 |
| opendevreview | Merged openstack/os-brick master: Support shared_targets tristate value https://review.opendev.org/c/openstack/os-brick/+/836063 | 15:18 |
| opendevreview | Merged openstack/os-brick stable/train: Use file locks in connectors https://review.opendev.org/c/openstack/os-brick/+/846957 | 15:20 |
| enriquetaso_ | I will add some n-cpu logs. Anyway, the upstream nfs job does not show the same results I see when I run tempest locally, that's bad. | 15:21 |
| enriquetaso_ | on the upstream job all the encryption test are failed. On my devstack those passed. | 15:22 |
| enriquetaso_ | well, i guess it's not for this meeting tho | 15:22 |
| enriquetaso_ | #topic Open Discussion | 15:22 |
| enriquetaso_ | Feel free to mention bugs now | 15:22 |
| enriquetaso_ | looks like no bugs for the open discussion | 15:27 |
| enriquetaso_ | Thanks everyone!! | 15:27 |
| enriquetaso_ | #endmeeting | 15:27 |
| opendevmeet | Meeting ended Wed Jul 27 15:27:46 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:27 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-07-27-14.56.html | 15:27 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-07-27-14.56.txt | 15:27 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-07-27-14.56.log.html | 15:27 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: initiator https://review.opendev.org/c/openstack/os-brick/+/786732 | 15:28 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: privileged/nvmeof.py https://review.opendev.org/c/openstack/os-brick/+/849017 | 15:31 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: improve utils.py coverage, update format https://review.opendev.org/c/openstack/os-brick/+/849018 | 15:31 |
| opendevreview | Eric Harney proposed openstack/os-brick master: Bump mypy version to 0.960 https://review.opendev.org/c/openstack/os-brick/+/849019 | 15:31 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: privileged/rbd.py https://review.opendev.org/c/openstack/os-brick/+/849020 | 15:31 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: initiator/connectors/rbd.py https://review.opendev.org/c/openstack/os-brick/+/849021 | 15:31 |
| opendevreview | Eric Harney proposed openstack/os-brick master: Move mypy job to check queue (non-voting) https://review.opendev.org/c/openstack/os-brick/+/849840 | 15:31 |
| eharney | (had to pull all of those ^ out of merge conflict) | 15:32 |
| Alexander | there is incorrect link for nfs driver https://etherpad.opendev.org/p/cinder-zed-new-drivers, the link must be with cinder endpoint https://review.opendev.org/c/openstack/cinder/+/797608 | 15:38 |
| opendevreview | Merged openstack/cinder master: mypy: work around mypy bug #13214 https://review.opendev.org/c/openstack/cinder/+/851032 | 15:45 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add additional transport type constants https://review.opendev.org/c/openstack/cinder/+/849690 | 15:47 |
| geguileo | simondodsley: I have just reviewed your nvme-of driver patch. | 15:51 |
| *** dviroel|lunch is now known as dviroel | 15:52 | |
| eharney | Alexander: i added a question on the powerstore nfs driver patch | 15:52 |
| Alexander | eharney: thank you! i answered on your comment | 16:27 |
| eharney | Alexander: i don't see a response posted from you there | 16:29 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 16:33 |
| Alexander | done, forgot to push comments | 16:34 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 16:42 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 17:00 |
| Alexander | eharney: i updated comment, could you please check? | 17:04 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 17:18 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 17:20 |
| eharney | when leaving a comment on the PowerStore NFS review... | 19:03 |
| eharney | "Error 409 (Conflict): Change 797608 may not exceed 1000 updates. It may still be abandoned, submitted and you can add/remove reviewers to/from the attention-set." | 19:03 |
| eharney | not great, never seen this limitation before | 19:03 |
| eharney | Alexander: i can't leave an update on gerrit, but i am not sure if cloning a volume from a snap works right | 19:04 |
| Alexander | eharney: got it, i push a new patch where i am using remotefs do_create_snapshot | 19:06 |
| eharney | Alexander: please make a new review w/ a new Change-Id so we can continue to review it | 19:07 |
| Alexander | when i tried to push patch i get this exception ERROR: Change 797608 may not exceed 1000 updates. It may still be abandoned, submitted and you can add/remove reviewers to/from the attention-set. To continue working on this change, recreate it with a new Change-Id, then abandon this one. | 19:07 |
| eharney | yeah :/ | 19:07 |
| eharney | delete the Change-Id from your commit message and it will generate a new one | 19:07 |
| Alexander | ok | 19:08 |
| opendevreview | Alexander Malashenko proposed openstack/cinder master: Add Cinder NFS driver for Dell PowerStore https://review.opendev.org/c/openstack/cinder/+/851249 | 19:08 |
| Alexander | done | 19:09 |
| opendevreview | Merged openstack/cinder master: Remove return from rpc cast method https://review.opendev.org/c/openstack/cinder/+/849181 | 19:09 |
| Alexander | we are starting from a blank page :) | 19:10 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 19:20 |
| opendevreview | Sofia Enriquez proposed openstack/devstack-plugin-nfs master: Runs all encrypted scenario tests https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/742905 | 20:46 |
| opendevreview | Alexander Malashenko proposed openstack/cinder master: Add Cinder NFS driver for Dell PowerStore https://review.opendev.org/c/openstack/cinder/+/851249 | 21:52 |
| opendevreview | Alexander Malashenko proposed openstack/cinder master: Add Cinder NFS driver for Dell PowerStore https://review.opendev.org/c/openstack/cinder/+/851249 | 21:58 |
| opendevreview | Alexander Malashenko proposed openstack/cinder master: Add Cinder NFS driver for Dell PowerStore https://review.opendev.org/c/openstack/cinder/+/851249 | 21:59 |
| Alexander | sorry for 3 patches in a row ^ | 22:00 |
| Alexander | eharney: zuul passing for changes with return _do_create_snapshot, about CI our CI team turned off powerstore nfs ci automatically check because they trying to resolve problem with powerstore nvme-otcp driver. If check nfs driver check can see CI can run automatically. | 22:02 |
| Alexander | sfernand: thank you for comment, i pushed a new patch with changes! | 22:04 |
| *** dviroel is now known as dviroel|afk | 22:36 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!