Friday, 2021-05-21

*** senrique_ has joined #openstack-cinder00:29
*** sapd1 has quit IRC00:58
*** senrique_ has quit IRC01:12
*** carloss has quit IRC02:24
*** ricolin has quit IRC02:33
*** rcernin has quit IRC03:20
*** rcernin has joined #openstack-cinder03:23
*** gksk has joined #openstack-cinder04:07
*** gksk has quit IRC04:37
*** udesale has joined #openstack-cinder05:44
*** benj_ has quit IRC06:19
*** ricolin has joined #openstack-cinder06:54
*** jawad_axd has joined #openstack-cinder06:54
*** ociuhandu has joined #openstack-cinder07:26
*** rpittau|afk is now known as rpittau07:43
*** ociuhandu has quit IRC07:48
lyarwoodhttps://bugs.launchpad.net/cinder/+bug/1901783 - FYI I'm still seeing this in Nova's gate on master, updated with an example.07:49
openstackLaunchpad bug 1901783 in Cinder xena "volume delete fails because cinder-rootwrap lvs fails with exit code 139" [Critical,Fix committed]07:49
lyarwoodit's a different LVM command so let me know if you'd like a fresh bug07:50
*** ociuhandu has joined #openstack-cinder07:54
*** tosky has joined #openstack-cinder07:59
*** ociuhandu has quit IRC07:59
*** rcernin has quit IRC08:04
*** zenkuro has joined #openstack-cinder08:17
*** ociuhandu has joined #openstack-cinder08:20
*** ociuhandu has quit IRC08:23
*** ociuhandu has joined #openstack-cinder08:24
*** benj_ has joined #openstack-cinder08:27
*** zenkuro has quit IRC08:28
*** zenkuro has joined #openstack-cinder08:28
*** ociuhandu has quit IRC08:30
*** sapd1_x has joined #openstack-cinder08:32
*** rcernin has joined #openstack-cinder08:37
*** rcernin has quit IRC08:38
*** rcernin has joined #openstack-cinder08:38
*** ociuhandu has joined #openstack-cinder08:45
*** ociuhandu has quit IRC08:54
*** ociuhandu has joined #openstack-cinder09:03
*** rcernin has quit IRC09:16
*** rcernin has joined #openstack-cinder09:22
*** rcernin has quit IRC09:36
*** jc has joined #openstack-cinder09:46
*** JaySee has quit IRC09:47
openstackgerritZijian Guo proposed openstack/cinder master: Fix the issue of creating volume from image for vmware  https://review.opendev.org/c/openstack/cinder/+/79256009:50
*** rcernin has joined #openstack-cinder09:54
*** rcernin has quit IRC09:57
*** rcernin has joined #openstack-cinder09:57
*** colby_ has quit IRC10:03
*** e0ne has joined #openstack-cinder10:06
*** snecklifter has joined #openstack-cinder10:16
*** zenkuro has quit IRC10:21
*** zenkuro has joined #openstack-cinder10:21
*** jc is now known as JaySee10:46
*** udesale has quit IRC10:47
*** udesale has joined #openstack-cinder10:47
*** sapd1_x has quit IRC10:49
*** JaySee has quit IRC10:55
*** ociuhandu has quit IRC11:15
*** rcernin has quit IRC11:16
*** ociuhandu has joined #openstack-cinder11:17
*** sapd1_x has joined #openstack-cinder11:22
*** nahimsouza has joined #openstack-cinder11:22
*** ociuhandu has quit IRC11:23
*** ociuhandu has joined #openstack-cinder11:37
*** ociuhandu has quit IRC11:42
*** ociuhandu has joined #openstack-cinder11:42
*** snecklifter has quit IRC11:47
*** ociuhandu has quit IRC11:48
*** sapd1_x has quit IRC11:52
openstackgerritRajat Dhasmana proposed openstack/os-brick master: Don't log error trace when nvme is not used  https://review.opendev.org/c/openstack/os-brick/+/78928912:05
whoami-rajatgeguileo:  ^^ (tested this time)12:06
whoami-rajatgeguileo: also i noticed similar trace related to multipathd in logs http://paste.openstack.org/show/805544/12:07
*** ociuhandu has joined #openstack-cinder12:15
*** ociuhandu has quit IRC12:20
*** ociuhandu has joined #openstack-cinder12:24
*** fabiooliveira has joined #openstack-cinder12:49
*** lyarwood has quit IRC12:55
*** senrique_ has joined #openstack-cinder12:59
*** lyarwood has joined #openstack-cinder13:03
*** ociuhandu has quit IRC13:06
*** ociuhandu has joined #openstack-cinder13:07
*** ociuhandu has quit IRC13:11
*** ociuhandu has joined #openstack-cinder13:11
*** ociuhandu has quit IRC13:12
*** ociuhandu has joined #openstack-cinder13:13
*** ociuhandu has quit IRC13:18
*** ociuhandu has joined #openstack-cinder13:26
*** supamatt has joined #openstack-cinder13:27
*** Underknowledge has quit IRC13:39
*** Underknowledge has joined #openstack-cinder13:40
*** zenkuro has quit IRC13:50
*** zenkuro has joined #openstack-cinder13:51
openstackgerritSimon Dodsley proposed openstack/cinder master: [Pure Storage] Fix minimum SDK version required  https://review.opendev.org/c/openstack/cinder/+/79246213:51
rosmaitareminder: festival of XS reviews at 1400 UTC13:58
rosmaitahttps://etherpad.opendev.org/p/cinder-festival-of-reviews13:58
rosmaitajungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir enriquetaso hemna ^^13:58
senrique_\o/14:00
rosmaita#startmeeting cinder_reviewfest14:04
openstackMeeting started Fri May 21 14:04:02 2021 UTC and is due to finish in 60 minutes.  The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot.14:04
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:04
*** openstack changes topic to " (Meeting topic: cinder_reviewfest)"14:04
openstackThe meeting name has been set to 'cinder_reviewfest'14:04
*** Underknowledge has quit IRC14:04
rosmaita#topic roll call14:06
*** openstack changes topic to "roll call (Meeting topic: cinder_reviewfest)"14:06
rosmaitao/14:06
eharneyhi14:06
walshh_hi14:06
toskyhi14:06
rosmaita#topic let the reviewing begin!14:07
*** openstack changes topic to "let the reviewing begin! (Meeting topic: cinder_reviewfest)"14:07
rosmaita#link https://etherpad.opendev.org/p/cinder-festival-of-reviews14:07
*** Reepicheep has quit IRC14:07
*** Underknowledge has joined #openstack-cinder14:08
*** Reepicheep has joined #openstack-cinder14:08
*** senrique_ is now known as enriquetaso14:09
enriquetasohi14:09
openstackgerritSimon Dodsley proposed openstack/cinder master: [Pure Storage] Fix minimum SDK version required  https://review.opendev.org/c/openstack/cinder/+/79246214:12
*** e0ne has quit IRC14:16
*** Underknowledge has quit IRC14:17
*** Underknowledge has joined #openstack-cinder14:20
*** Underknowledge has quit IRC14:26
*** rpittau is now known as rpittau|afk14:27
openstackgerritEric Harney proposed openstack/cinder master: Doc: Remove incorrect note about encryption flag  https://review.opendev.org/c/openstack/cinder/+/79263814:27
*** Underknowledge has joined #openstack-cinder14:27
jungleboyjo/14:32
enriquetasoI like this title : https://review.opendev.org/c/openstack/cinder/+/79232914:32
enriquetasoonly owners can set the 'work in progress' flag?14:32
eharneyi think so, but we can -W it14:33
openstackgerritSimon Dodsley proposed openstack/cinder master: [Pure Storage] Fix minimum SDK version required  https://review.opendev.org/c/openstack/cinder/+/79246214:37
*** xuany has joined #openstack-cinder14:37
xuanyrosmaita: hello14:37
openstackgerritBrian Rosmaita proposed openstack/cinder master: Remove Block Storage API v2  https://review.opendev.org/c/openstack/cinder/+/79229914:37
rosmaitaxuany: hello, we keep missing each other14:37
xuanyyes14:38
xuanycan you help review my bp spec in cinder, if you have time14:38
rosmaitaxuany: i will promise to do that today, after the festival of reviews14:40
xuanyok, thanks, I really like being able to do some work in cinder14:41
xuany.14:41
openstackgerritBrian Rosmaita proposed openstack/cinder master: Run cinder-mypy job in the check  https://review.opendev.org/c/openstack/cinder/+/79264214:46
xuanygeguileo: hello, i have upload the path about the os-brick bugfix, https://review.opendev.org/c/openstack/os-brick/+/78051114:46
xuanyjunjleboyj: hi,  i have upload the path about the os-brick bugfix, please review if you have time, thanks very much;14:47
xuanyhttps://review.opendev.org/c/openstack/os-brick/+/78051114:47
rosmaitajungleboyj: it's not XS, but it makes some of the XS patches mergeable, so maybe you can look over https://review.opendev.org/c/openstack/os-brick/+/786729/3 ?14:52
*** pcaruana has quit IRC14:52
*** udesale has quit IRC14:56
openstackgerritDavid Vallee Delisle proposed openstack/cinder stable/train: [train-only] Fix mock import for infinidat driver  https://review.opendev.org/c/openstack/cinder/+/79096314:57
openstackgerritSimon Dodsley proposed openstack/cinder master: [Pure Storage] Fix minimum SDK version required  https://review.opendev.org/c/openstack/cinder/+/79246215:01
jungleboyjenriquetaso:  Nothing helps code review like cat butt in your face.15:02
enriquetasocat++15:04
jungleboyj:-)15:04
openstackgerritBrian Rosmaita proposed openstack/os-brick master: Introduce mypy  https://review.opendev.org/c/openstack/os-brick/+/78672915:13
openstackgerritBrian Rosmaita proposed openstack/os-brick master: mypy: exception.py  https://review.opendev.org/c/openstack/os-brick/+/78673015:13
openstackgerritBrian Rosmaita proposed openstack/os-brick master: trivial: correct comment  https://review.opendev.org/c/openstack/os-brick/+/79264515:13
*** ociuhandu has quit IRC15:15
*** ociuhandu has joined #openstack-cinder15:15
eharneyhttps://review.opendev.org/c/openstack/os-brick/+/791272/315:18
*** ociuhandu has quit IRC15:19
openstackgerritMerged openstack/cinder master: Excess availability_zone judgment code  https://review.opendev.org/c/openstack/cinder/+/71425215:25
*** jawad_axd has quit IRC15:31
openstackgerritEric Harney proposed openstack/os-brick master: mypy: exception.py  https://review.opendev.org/c/openstack/os-brick/+/78673015:32
openstackgerritEric Harney proposed openstack/os-brick master: mypy: utils.py  https://review.opendev.org/c/openstack/os-brick/+/78673115:32
rosmaitaenriquetaso: project:openstack/os-brick owner:stephenfin@redhat.com status:open15:36
rosmaitasorry15:36
rosmaitahttps://review.opendev.org/q/project:openstack/os-brick+owner:stephenfin%2540redhat.com+status:open15:36
*** ociuhandu has joined #openstack-cinder15:36
enriquetasorosmaita++15:39
jungleboyjrosmaita:  A backport that can be merged:  https://review.opendev.org/c/openstack/cinder/+/76036215:40
rosmaitawill take a look15:40
*** ociuhandu has quit IRC15:41
*** xuany has quit IRC15:43
jungleboyjrosmaita: https://review.opendev.org/c/openstack/cinder/+/790963  That is an easy one too.15:43
toskyeheh, no one wants to touch https://review.opendev.org/c/openstack/cinder/+/489949/ :)15:48
rosmaitajungleboyj: what did you think of Elod's comments on https://review.opendev.org/c/openstack/cinder/+/760362 ?15:49
jungleboyjtosky:  When geguileo isn't +2, no.15:49
jungleboyjrosmaita:  Oh, missed that.15:50
rosmaitayeah, looks like maybe not a clean backport15:51
jungleboyjYeah, taking my +2 off that.15:52
jungleboyjThanks for pointing that out.15:52
openstackgerritMerged openstack/cinder-tempest-plugin master: Fix stein: use the old terminology for exclusion list  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/78675515:55
openstackgerritMerged openstack/cinder master: remove the oslo_utils.fnmatch  https://review.opendev.org/c/openstack/cinder/+/78838715:55
openstackgerritMerged openstack/cinder master: api-ref: Use key_size 256 in API examples  https://review.opendev.org/c/openstack/cinder/+/79101015:55
*** Underknowledge has quit IRC15:57
*** Underknowledge has joined #openstack-cinder15:58
rosmaitaok, we're just about out of time for the review fest, but feel free to stick around in a festive mood and continue reviewing!16:00
rosmaita#endmeeting16:00
*** openstack changes topic to "The Block Storage Project | https://wiki.openstack.org/wiki/Cinder | https://tiny.cc/CinderPriorities"16:00
openstackMeeting ended Fri May 21 16:00:06 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:00
openstackMinutes:        http://eavesdrop.openstack.org/meetings/cinder_reviewfest/2021/cinder_reviewfest.2021-05-21-14.04.html16:00
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/cinder_reviewfest/2021/cinder_reviewfest.2021-05-21-14.04.txt16:00
openstackLog:            http://eavesdrop.openstack.org/meetings/cinder_reviewfest/2021/cinder_reviewfest.2021-05-21-14.04.log.html16:00
jungleboyjThanks!16:00
*** eharney has quit IRC16:07
*** e0ne has joined #openstack-cinder16:10
*** gmann is now known as gmann_afk16:11
openstackgerritMerged openstack/cinder master: Tests: Don't assert notifier not called  https://review.opendev.org/c/openstack/cinder/+/79240816:17
openstackgerritMerged openstack/python-cinderclient stable/wallaby: Use default nodeset for functional jobs  https://review.opendev.org/c/openstack/python-cinderclient/+/79030616:18
*** e0ne has quit IRC16:37
*** e0ne has joined #openstack-cinder16:38
*** e0ne has quit IRC16:38
openstackgerritMerged openstack/python-cinderclient master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/python-cinderclient/+/78839616:48
*** dave-mccowan has quit IRC17:08
openstackgerritMerged openstack/os-brick master: Enable bandit runs for tox  https://review.opendev.org/c/openstack/os-brick/+/75269217:09
openstackgerritMerged openstack/cinder stable/victoria: RBD: Change rbd_exclusive_cinder_pool's default  https://review.opendev.org/c/openstack/cinder/+/79212817:16
openstackgerritMerged openstack/cinder master: vmware tests: Support different moref backend representations  https://review.opendev.org/c/openstack/cinder/+/78774817:17
openstackgerritMerged openstack/os-brick master: vmware: Use cookiejar from oslo.vmware client directly  https://review.opendev.org/c/openstack/os-brick/+/78842317:21
openstackgerritMerged openstack/os-brick master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/os-brick/+/78793317:21
openstackgerritMerged openstack/cinder master: vmware: Use oslo.vmware's get_moref_value()  https://review.opendev.org/c/openstack/cinder/+/78774917:28
*** __ministry has joined #openstack-cinder17:44
*** __ministry has quit IRC17:49
*** whoami-rajat has quit IRC17:59
*** rosmaita has left #openstack-cinder18:02
*** zenkuro has quit IRC18:05
*** zenkuro has joined #openstack-cinder18:06
*** rosmaita has joined #openstack-cinder18:06
*** ociuhandu has joined #openstack-cinder18:21
*** ociuhandu has quit IRC18:27
*** dave-mccowan has joined #openstack-cinder18:31
*** eharney has joined #openstack-cinder18:35
*** lbragstad has quit IRC18:50
*** coreycb has quit IRC18:50
openstackgerritElod Illes proposed openstack/cinder stable/rocky: [stable-only] Cap bandit to v1.6.2 and fix constraints  https://review.opendev.org/c/openstack/cinder/+/76710618:51
*** ociuhandu has joined #openstack-cinder18:57
*** zenkuro has quit IRC19:04
*** ociuhandu has quit IRC19:04
*** zenkuro has joined #openstack-cinder19:04
*** ociuhandu has joined #openstack-cinder19:09
*** ociuhandu has quit IRC19:15
*** zenkuro has quit IRC19:17
*** zenkuro has joined #openstack-cinder19:17
*** enriquetaso has quit IRC19:21
*** enriquetaso has joined #openstack-cinder19:49
*** e0ne has joined #openstack-cinder19:53
*** e0ne has quit IRC20:01
*** nahimsouza has quit IRC20:21
*** sfernand has joined #openstack-cinder20:53
openstackgerritEric Harney proposed openstack/os-brick master: Introduce mypy  https://review.opendev.org/c/openstack/os-brick/+/78672920:59
*** supamatt has quit IRC21:01
openstackgerritMerged openstack/devstack-plugin-ceph master: Clean up our disk image and mountpoint  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/76201021:08
*** Underknowledge has quit IRC21:14
*** enriquetaso has quit IRC21:15
*** tosky has quit IRC21:15
*** abishop has quit IRC21:15
*** enriquetaso has joined #openstack-cinder21:15
*** abishop has joined #openstack-cinder21:16
*** tosky has joined #openstack-cinder21:16
*** Underknowledge has joined #openstack-cinder21:16
*** fabiooliveira has quit IRC21:29
*** jawad_axd has joined #openstack-cinder21:43
*** jawad_axd has quit IRC21:52
*** jawad_axd has joined #openstack-cinder21:52
*** patrickeast has quit IRC21:53
*** felipe_rodrigues has quit IRC21:53
*** ildikov has quit IRC21:56
*** lxkong has quit IRC21:56
*** knikolla has quit IRC21:56
*** jawad_axd has quit IRC21:57
*** ildikov has joined #openstack-cinder21:59
*** knikolla has joined #openstack-cinder21:59
*** lxkong has joined #openstack-cinder21:59
*** felipe_rodrigues has joined #openstack-cinder21:59
*** patrickeast has joined #openstack-cinder21:59
*** ociuhandu has joined #openstack-cinder22:02
*** ociuhandu has quit IRC22:11
*** zenkuro has quit IRC22:16
*** jawad_axd has joined #openstack-cinder22:23
*** jmlowe has quit IRC22:45
*** jmlowe has joined #openstack-cinder22:48
*** jawad_axd has quit IRC22:57
*** tosky has quit IRC23:03

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!