opendevreview | yuhao.zhu proposed openstack/cinder master: Delete quotas when deleting volume type https://review.opendev.org/c/openstack/cinder/+/768592 | 03:08 |
---|---|---|
*** akekane_ is now known as abhishekk | 05:51 | |
opendevreview | yuhao.zhu proposed openstack/cinder master: Delete quotas when deleting volume type https://review.opendev.org/c/openstack/cinder/+/768592 | 08:00 |
*** bhagyashris_ is now known as bhagyashris | 08:27 | |
opendevreview | Zohar Mamedov proposed openstack/cinder-specs master: NVMe-oF connection agent https://review.opendev.org/c/openstack/cinder-specs/+/796365 | 08:49 |
opendevreview | Ivan Pchelintsev proposed openstack/cinder master: Add Cinder NFS driver for Dell EMC PowerStore https://review.opendev.org/c/openstack/cinder/+/797608 | 08:52 |
*** dviroel|out is now known as dviroel | 11:37 | |
opendevreview | Sofia Enriquez proposed openstack/cinder master: LVM: Retry lvdisplay and lvcreate calls on segfault https://review.opendev.org/c/openstack/cinder/+/796950 | 13:11 |
opendevreview | Merged openstack/cinder-specs master: Temporary Resource Tracking https://review.opendev.org/c/openstack/cinder-specs/+/795842 | 13:25 |
opendevreview | Merged openstack/cinder-specs master: Snapshot attached volumes w/o force flag https://review.opendev.org/c/openstack/cinder-specs/+/781914 | 13:28 |
opendevreview | Walt proposed openstack/cinder master: Ensure driver is initialized before attach https://review.opendev.org/c/openstack/cinder/+/794001 | 13:28 |
opendevreview | Alan Bishop proposed openstack/cinder-specs master: Expose ``user visible`` extra specs https://review.opendev.org/c/openstack/cinder-specs/+/796166 | 13:49 |
opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Improve error handling around deletes https://review.opendev.org/c/openstack/cinder/+/796286 | 13:53 |
hemna | are we not mirroring to github.com ? | 13:54 |
hemna | https://github.com/openstack/rbd-iscsi-client | 13:55 |
hemna | that's empty | 13:55 |
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 lseki sfernand rajinir enriquetaso hemna ^^ | 13:58 |
rosmaita | hemna: maybe i was supposed to set something up for that, i will check | 13:58 |
opendevreview | Merged openstack/python-brick-cinderclient-ext master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/789695 | 13:58 |
opendevreview | Brian Rosmaita proposed openstack/cinder-specs master: Spec for "Consistent and Secure RBAC" in Cinder https://review.opendev.org/c/openstack/cinder-specs/+/797655 | 14:01 |
smcginnis | hemna: rosmaita: Yeah, there's a job that needs to be added per-repo to do the mirroring. | 14:07 |
smcginnis | hemna: Any reason not to approve https://review.opendev.org/c/openstack/cinder/+/786926 | 14:07 |
hemna | ok | 14:07 |
hemna | missed the +A | 14:08 |
smcginnis | OK, cool. Didn't want to approve if you had held it back for some reason. ;) | 14:08 |
opendevreview | Alan Bishop proposed openstack/cinder-specs master: Expose ``user visible`` extra specs https://review.opendev.org/c/openstack/cinder-specs/+/796166 | 14:24 |
hemna | This is a good example of an out of tree cinder driver that has never been in cinder and is a major fail: https://github.com/iXsystems/cinder/pull/25 | 14:26 |
opendevreview | Merged openstack/os-brick master: Ussuri+ is python3 only https://review.opendev.org/c/openstack/os-brick/+/789964 | 14:59 |
tosky | before the bug meeting: I understand it's not critical and it's quite unusual ansible usage but please take a look at this review which removes one of the two devstack-based legacy jobs left in the whole openstack/master: https://review.opendev.org/c/openstack/cinder/+/751579 | 15:02 |
enriquetaso | hello | 15:02 |
enriquetaso | #startmeeting cinder_bs | 15:02 |
opendevmeet | Meeting started Wed Jun 23 15:02:30 2021 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:02 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:02 |
opendevmeet | The meeting name has been set to 'cinder_bs' | 15:02 |
rosmaita | time for cinder BS! | 15:02 |
enriquetaso | tosky++ | 15:03 |
enriquetaso | Only 2 topics for today's meeting: | 15:03 |
enriquetaso | (1) Regarding the LVM CI failatures & retry work. I'd like to ask for reviews & opinions on https://review.opendev.org/c/openstack/cinder/+/796950. In this way I can submit an os-brick patch to solve this same issue on os-brick. | 15:03 |
enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/796950 | 15:03 |
enriquetaso | As a final work, the idea is to follow up walter's patch and remove duplication code https://review.opendev.org/c/openstack/cinder/+/260756 (in order to do so we may need a os-brick release) | 15:03 |
enriquetaso | #topic LVM CI failatures & retry work | 15:05 |
enriquetaso | #topic LVM CI failatures & retry work | 15:05 |
rosmaita | sorry, still catching up to the previous topic | 15:05 |
enriquetaso | don't worry rosmaita | 15:06 |
rosmaita | i'll review 796950 today | 15:06 |
rosmaita | will get to it later after hopefully zuul has said +1 | 15:06 |
enriquetaso | I just wonder why opendevmeet bot doesn't show the topic title | 15:06 |
rosmaita | there's a problem with the oftc interface to the meetbot | 15:07 |
enriquetaso | thanks rosmaita | 15:07 |
enriquetaso | ahh | 15:07 |
rosmaita | it's still worth using the #topic because the meeting log formatter understands it | 15:07 |
enriquetaso | sure, OK, last one it's a quick question: | 15:07 |
enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/796950 | 15:07 |
enriquetaso | #topic bug_1 "scaleio connector disables HTTPS certificate validation". Unassigned | 15:08 |
enriquetaso | #link https://bugs.launchpad.net/os-brick/+bug/1929223 | 15:08 |
enriquetaso | Do we have any work done for this? I couldn't find any | 15:08 |
rosmaita | no, no work on that | 15:08 |
rosmaita | we need to ping someone from dell to get a comment on it | 15:09 |
enriquetaso | by email? | 15:09 |
rosmaita | i added some tags to the bug, but maybe not the right ones | 15:09 |
eharney | i think this may require some brick changes to fix, i can help if they get someone started on it | 15:09 |
rosmaita | yeah, i think look at who has committed to the scaleio driver recently and send an email asking them to look at the bug | 15:10 |
eharney | it's powerflex now | 15:10 |
rosmaita | and you can mention eric's offer to help if they get things started | 15:10 |
eharney | previously known as vxflexos | 15:10 |
eharney | previously known as scaleio | 15:10 |
rosmaita | yeah, i can't keep them straight | 15:10 |
enriquetaso | cool | 15:11 |
enriquetaso | #action(enriquetaso) contact powerflex last commiter with bug 1929223 and mention eric's offer | 15:12 |
enriquetaso | #topic open discussion | 15:13 |
rosmaita | looks like Ivan Pchelintsev was the last committer, but i think walshh_ has worked on that one too | 15:13 |
enriquetaso | OK, i'll ask him then | 15:14 |
enriquetaso | That's all for me | 15:14 |
eharney | since it's bug related, i'll mention that https://etherpad.opendev.org/p/cinder-xena-ci-tracking is still going | 15:14 |
enriquetaso | do we have any topic for open discussion? | 15:14 |
eharney | rosmaita: any particular goals for that now, since the lvm bugs seem to be the most interesting issues? | 15:15 |
enriquetaso | #link https://etherpad.opendev.org/p/cinder-xena-ci-tracking | 15:16 |
rosmaita | no, not really | 15:16 |
hemna | enriquetaso it would be nice to get that code out of cinder and in os-brick done | 15:16 |
rosmaita | i mean it would be good to keep it alive a bit so that we have some data that the lvm bugs aren't causing failures any more | 15:16 |
hemna | I think I ran into some small differences between nova's lvm code and cinder's and then ran out of time to work on it | 15:16 |
enriquetaso | agree | 15:17 |
whoami-rajat | enriquetaso: left a comment on your patch, minor change, else it looks pretty straightforward https://review.opendev.org/c/openstack/cinder/+/796950 | 15:18 |
enriquetaso | thank rajat | 15:19 |
whoami-rajat | np | 15:19 |
enriquetaso | OK, if we don't have any topics, i'll close the meeting | 15:20 |
enriquetaso | thanks everyone | 15:21 |
enriquetaso | #endmeeting | 15:22 |
opendevmeet | Meeting ended Wed Jun 23 15:22:02 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:22 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-06-23-15.02.html | 15:22 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-06-23-15.02.txt | 15:22 |
opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-06-23-15.02.log.html | 15:22 |
opendevreview | Eric Harney proposed openstack/cinder master: mypy: create_volume flows https://review.opendev.org/c/openstack/cinder/+/780933 | 15:23 |
opendevreview | Eric Harney proposed openstack/cinder master: mypy: image cache https://review.opendev.org/c/openstack/cinder/+/788363 | 15:25 |
opendevreview | Brian Rosmaita proposed openstack/cinder-specs master: Spec for "Consistent and Secure RBAC" in Cinder https://review.opendev.org/c/openstack/cinder-specs/+/797655 | 15:36 |
rosmaita | whoami-rajat: thanks for your comments ^^ | 15:36 |
opendevreview | Sofia Enriquez proposed openstack/cinder master: LVM: Retry lvdisplay and lvcreate calls on segfault https://review.opendev.org/c/openstack/cinder/+/796950 | 15:41 |
opendevreview | Alan Bishop proposed openstack/cinder-specs master: Expose ``user visible`` extra specs https://review.opendev.org/c/openstack/cinder-specs/+/796166 | 15:44 |
_pewp_ | hemna \(-_- ) | 15:45 |
opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Retype fix for replication https://review.opendev.org/c/openstack/cinder/+/797510 | 16:41 |
walshh_ | Hi, could someone take a look at https://review.opendev.org/c/openstack/cinder/+/772388. It is patch for tempest CI so that manage/unmanage tests for volume and snapshots can be run successfully. Thank you | 16:49 |
opendevreview | Gorka Eguileor proposed openstack/cinder master: Improve quota usage for temporary resources https://review.opendev.org/c/openstack/cinder/+/786385 | 17:57 |
opendevreview | Gorka Eguileor proposed openstack/cinder master: Expose volume & snapshot use_quota field https://review.opendev.org/c/openstack/cinder/+/786386 | 17:57 |
opendevreview | Gorka Eguileor proposed openstack/cinder master: RBD: Don't flatten temporary resources https://review.opendev.org/c/openstack/cinder/+/790492 | 17:57 |
opendevreview | Gorka Eguileor proposed openstack/python-cinderclient master: Add consumes quota field support https://review.opendev.org/c/openstack/python-cinderclient/+/787407 | 18:02 |
opendevreview | Merged openstack/cinder master: Remove six from cinder.tests.unit.objects https://review.opendev.org/c/openstack/cinder/+/780757 | 19:01 |
opendevreview | Merged openstack/cinder master: HPMSA: Report SAN driver options https://review.opendev.org/c/openstack/cinder/+/770807 | 19:01 |
opendevreview | Merged openstack/cinder master: Fix instance locality scheduler filter https://review.opendev.org/c/openstack/cinder/+/786025 | 19:02 |
opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Fix minimum SDK version required https://review.opendev.org/c/openstack/cinder/+/792462 | 20:14 |
opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Fix minimum SDK version required https://review.opendev.org/c/openstack/cinder/+/792462 | 20:18 |
opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Fix minimum SDK version required https://review.opendev.org/c/openstack/cinder/+/792462 | 20:34 |
opendevreview | Alan Bishop proposed openstack/cinder-specs master: Expose ``user visible`` extra specs https://review.opendev.org/c/openstack/cinder-specs/+/796166 | 20:56 |
*** dviroel is now known as dviroel|out | 21:13 | |
opendevreview | Merged openstack/cinder master: db: Compact migrations to Train https://review.opendev.org/c/openstack/cinder/+/786926 | 23:05 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!