Wednesday, 2018-03-28

*** s-shiono has joined #openstack-cinder00:04
*** salv-orlando has joined #openstack-cinder00:05
*** salv-orlando has quit IRC00:10
*** liverpooler has joined #openstack-cinder00:11
*** openstack has joined #openstack-cinder00:18
*** ChanServ sets mode: +o openstack00:18
*** stakeda has joined #openstack-cinder00:20
*** dave-mccowan has joined #openstack-cinder00:21
*** wanghao has quit IRC00:26
*** wanghao has joined #openstack-cinder00:26
*** abishop has quit IRC00:30
*** markstur has joined #openstack-cinder00:39
*** markstur has quit IRC00:43
*** odyssey4me has quit IRC00:53
*** odyssey4me has joined #openstack-cinder00:53
*** yangyapeng has joined #openstack-cinder00:58
*** yangyapeng has quit IRC01:03
*** itlinux has joined #openstack-cinder01:07
*** gyankum has joined #openstack-cinder01:12
*** yangyapeng has joined #openstack-cinder01:14
*** itlinux has quit IRC01:15
openstackgerritwanghao proposed openstack/cinder master: Transfer snapshots with volumes  https://review.openstack.org/53356401:16
*** armaan has quit IRC01:30
*** armaan has joined #openstack-cinder01:31
*** vivsoni has quit IRC01:39
*** gouthamr has joined #openstack-cinder01:40
*** vivsoni has joined #openstack-cinder01:41
openstackgerrityenai proposed openstack/cinder master: Use check_virtual_size to do the size check  https://review.openstack.org/55316401:57
*** salv-orlando has joined #openstack-cinder02:07
*** dikonoo has joined #openstack-cinder02:09
*** psachin has joined #openstack-cinder02:11
*** salv-orlando has quit IRC02:12
*** AlexeyAbashkin has joined #openstack-cinder02:12
*** AlexeyAbashkin has quit IRC02:16
*** markstur has joined #openstack-cinder02:28
*** knylande_ has quit IRC02:29
*** zhurong has joined #openstack-cinder02:32
*** markstur has quit IRC02:32
*** dalgaaf has quit IRC02:35
*** dalgaaf has joined #openstack-cinder02:36
openstackgerritTommyLike proposed openstack/cinder-specs master: Support mark backend pool sold out  https://review.openstack.org/55711202:37
openstackgerrityixuan zhang proposed openstack/cinder master: Storwize: self assign the SCSI lun id for volume attaching  https://review.openstack.org/55564102:52
*** armaan has quit IRC02:59
*** knylande_ has joined #openstack-cinder03:12
*** knylande_ has quit IRC03:13
openstackgerritTommyLike proposed openstack/cinder-specs master: Support AZ in volume type  https://review.openstack.org/54269103:18
*** itlinux has joined #openstack-cinder03:18
openstackgerritTommyLike proposed openstack/cinder master: Support availability-zone type  https://review.openstack.org/55224303:31
*** markstur has joined #openstack-cinder03:45
*** andreas_s has joined #openstack-cinder03:46
*** markstur has quit IRC03:50
*** andreas_s has quit IRC03:51
*** dave-mccowan has quit IRC03:55
*** yangyapeng has quit IRC03:56
*** stakeda has quit IRC04:07
*** salv-orlando has joined #openstack-cinder04:08
*** salv-orlando has quit IRC04:13
*** yuan_ has joined #openstack-cinder04:13
*** yuanmingpeng has joined #openstack-cinder04:15
*** lkuchlan_ has joined #openstack-cinder04:15
*** gouthamr has quit IRC04:22
*** psachin has quit IRC04:25
*** namnh has joined #openstack-cinder04:27
*** abhishekk has joined #openstack-cinder04:29
*** yuanmingpeng has left #openstack-cinder04:31
*** psachin has joined #openstack-cinder04:42
*** gkadam has joined #openstack-cinder04:44
*** links has joined #openstack-cinder04:46
*** lpetrut has joined #openstack-cinder04:55
*** chhagarw has joined #openstack-cinder05:03
*** armaan has joined #openstack-cinder05:04
*** gkadam has quit IRC05:07
*** salv-orlando has joined #openstack-cinder05:09
*** salv-orlando has quit IRC05:09
*** salv-orlando has joined #openstack-cinder05:09
*** chhagarw has quit IRC05:12
*** yangyapeng has joined #openstack-cinder05:14
*** yangyapeng has quit IRC05:19
*** pgadiya has joined #openstack-cinder05:19
*** moshele has joined #openstack-cinder05:22
*** lpetrut has quit IRC05:26
*** e0ne has joined #openstack-cinder05:26
*** lpetrut has joined #openstack-cinder05:27
*** e0ne has quit IRC05:33
openstackgerritVivek Soni proposed openstack/cinder master: 3PAR: Report backend state in service list  https://review.openstack.org/55676505:35
*** lpetrut has quit IRC05:39
*** armaan has quit IRC05:49
*** e0ne has joined #openstack-cinder05:50
*** lpetrut has joined #openstack-cinder05:52
*** belmoreira has joined #openstack-cinder05:54
*** gaoyan has joined #openstack-cinder06:00
*** kaisers has joined #openstack-cinder06:02
openstackgerritMerged openstack/python-brick-cinderclient-ext stable/queens: Updated from global requirements  https://review.openstack.org/55708906:04
*** kaisers has quit IRC06:05
*** markstur has joined #openstack-cinder06:05
*** kaisers has joined #openstack-cinder06:06
*** e0ne has quit IRC06:10
*** markstur has quit IRC06:10
*** swamireddy has quit IRC06:11
openstackgerritPeter Wang proposed openstack/cinder master: VNX: fix performance in create/delete_volume  https://review.openstack.org/55715606:13
*** Gorian has quit IRC06:14
*** moshele has quit IRC06:14
*** armaan has joined #openstack-cinder06:15
*** voelzmo has joined #openstack-cinder06:18
*** Gorian has joined #openstack-cinder06:21
*** chenying_ has joined #openstack-cinder06:27
*** chenying has quit IRC06:27
openstackgerritTommyLike proposed openstack/cinder-specs master: Support image signature verification  https://review.openstack.org/38414306:32
*** pcaruana has joined #openstack-cinder06:38
*** alexchadin has joined #openstack-cinder06:39
*** kaisers has quit IRC06:40
*** yangyapeng has joined #openstack-cinder06:42
*** voelzmo has quit IRC06:55
*** andreas_s has joined #openstack-cinder06:55
*** voelzmo has joined #openstack-cinder06:55
*** lpetrut has quit IRC06:58
*** yangyapeng has quit IRC07:00
*** yangyapeng has joined #openstack-cinder07:01
*** zhurong has quit IRC07:08
*** voelzmo has quit IRC07:08
*** voelzmo has joined #openstack-cinder07:10
*** voelzmo has quit IRC07:11
*** rcernin has quit IRC07:15
*** voelzmo has joined #openstack-cinder07:15
*** kaisers has joined #openstack-cinder07:17
*** pooja_jadhav has quit IRC07:21
*** kaisers has quit IRC07:22
*** salv-orlando has quit IRC07:22
*** pooja_jadhav has joined #openstack-cinder07:22
*** salv-orlando has joined #openstack-cinder07:22
*** tesseract has joined #openstack-cinder07:25
*** alexchadin has quit IRC07:26
*** salv-orlando has quit IRC07:27
*** alexchadin has joined #openstack-cinder07:31
*** voelzmo has quit IRC07:36
openstackgerritRyan Liang proposed openstack/cinder master: Unity: fail to detach lun when auto zone enabled  https://review.openstack.org/55680307:39
openstackgerritPeter Wang proposed openstack/cinder master: VNX: fix performance in create/delete_volume  https://review.openstack.org/55715607:43
*** kaisers has joined #openstack-cinder07:44
*** salv-orlando has joined #openstack-cinder07:45
*** belmoreira has quit IRC07:49
*** salv-orlando has quit IRC07:54
*** salv-orlando has joined #openstack-cinder07:54
*** salv-orlando has quit IRC07:59
*** AlexeyAbashkin has joined #openstack-cinder08:00
*** sar has quit IRC08:04
*** voelzmo has joined #openstack-cinder08:04
*** belmoreira has joined #openstack-cinder08:05
*** kaisers has quit IRC08:07
*** kaisers has joined #openstack-cinder08:17
*** gkadam has joined #openstack-cinder08:19
*** kaisers has quit IRC08:21
*** voelzmo has quit IRC08:22
*** voelzmo has joined #openstack-cinder08:22
*** voelzmo has quit IRC08:23
*** voelzmo has joined #openstack-cinder08:23
*** voelzmo has quit IRC08:24
*** voelzmo has joined #openstack-cinder08:24
*** rcernin has joined #openstack-cinder08:24
*** voelzmo has quit IRC08:24
*** voelzmo has joined #openstack-cinder08:25
*** voelzmo has quit IRC08:25
*** voelzmo has joined #openstack-cinder08:25
*** s-shiono has quit IRC08:26
*** voelzmo has quit IRC08:26
*** e0ne has joined #openstack-cinder08:27
*** lihi has quit IRC08:32
openstackgerritwanghui proposed openstack/python-brick-cinderclient-ext master: Update links in README  https://review.openstack.org/55719108:35
*** lihi has joined #openstack-cinder08:39
*** lpetrut has joined #openstack-cinder08:40
*** rcernin has quit IRC08:42
*** hoangcx has quit IRC08:42
*** lpetrut has quit IRC08:47
*** hoangcx has joined #openstack-cinder08:47
*** lpetrut has joined #openstack-cinder08:47
*** alexchadin has quit IRC08:57
*** alexchadin has joined #openstack-cinder08:58
*** chenying_ has quit IRC08:59
*** wanghao has quit IRC09:01
*** wanghao has joined #openstack-cinder09:02
*** wanghao has quit IRC09:02
*** wanghao has joined #openstack-cinder09:02
*** wanghao has quit IRC09:03
*** wanghao has joined #openstack-cinder09:03
*** wanghao has quit IRC09:04
*** wanghao has joined #openstack-cinder09:04
*** wanghao has quit IRC09:04
*** wanghao has joined #openstack-cinder09:05
*** wanghao has quit IRC09:05
*** wanghao has joined #openstack-cinder09:06
*** wanghao has quit IRC09:06
*** wanghao has joined #openstack-cinder09:06
*** wanghao has quit IRC09:07
*** wanghao has joined #openstack-cinder09:08
*** wanghao has quit IRC09:08
*** wanghao has joined #openstack-cinder09:09
*** alexchadin has quit IRC09:34
*** swamireddy has joined #openstack-cinder09:34
*** itlinux has quit IRC09:36
*** lpetrut_ has joined #openstack-cinder09:41
*** hoangcx has quit IRC09:51
*** hoangcx has joined #openstack-cinder09:56
*** salv-orlando has joined #openstack-cinder09:58
*** liverpooler has quit IRC09:59
*** hoangcx has quit IRC10:03
*** namnh has quit IRC10:03
*** hoangcx has joined #openstack-cinder10:10
*** markstur has joined #openstack-cinder10:19
*** markstur has quit IRC10:24
openstackgerritSilvan Kaiser proposed openstack/cinder master: Adds Overlay Volumes Created from Snapshots to Quobyte  https://review.openstack.org/50705010:24
*** armaan has quit IRC10:28
*** jiaopengju has quit IRC10:34
*** alexchadin has joined #openstack-cinder10:35
*** gaoyan has quit IRC10:36
*** moshele has joined #openstack-cinder10:36
*** jiaopengju has joined #openstack-cinder10:37
*** abhishekk has quit IRC10:45
*** swamireddy has quit IRC10:51
*** salv-orl_ has joined #openstack-cinder10:53
*** zhurong has joined #openstack-cinder10:55
*** salv-orlando has quit IRC10:56
*** gyankum has quit IRC10:56
*** markstur has joined #openstack-cinder10:57
*** salv-orlando has joined #openstack-cinder10:58
*** salv-orl_ has quit IRC10:58
openstackgerritMichael McAleer proposed openstack/cinder master: VMAX driver - VMAX list manageable volumes and snapshots  https://review.openstack.org/55729710:58
openstackgerritSilvan Kaiser proposed openstack/cinder master: Allow extension of volumes with snapshots for VZstorage driver  https://review.openstack.org/46290510:58
*** markstur has quit IRC11:02
*** gkadam_ has joined #openstack-cinder11:03
*** pgadiya_ has joined #openstack-cinder11:03
*** chkumar246 has joined #openstack-cinder11:04
*** links has quit IRC11:04
*** chandankumar has quit IRC11:04
*** pgadiya has quit IRC11:04
*** gkadam__ has joined #openstack-cinder11:04
*** salv-orlando has quit IRC11:05
*** links has joined #openstack-cinder11:05
*** armaan has joined #openstack-cinder11:06
*** salv-orlando has joined #openstack-cinder11:06
*** kaisers has joined #openstack-cinder11:06
*** gkadam has quit IRC11:07
*** gkadam_ has quit IRC11:08
*** chkumar246 has quit IRC11:08
*** kaisers has quit IRC11:08
*** pgadiya_ has quit IRC11:08
*** dave-mccowan has joined #openstack-cinder11:09
openstackgerritMichael McAleer proposed openstack/cinder master: VMAX driver - VMAX list manageable volumes and snapshots  https://review.openstack.org/55729711:15
*** chkumar246 has joined #openstack-cinder11:20
openstackgerritMichael McAleer proposed openstack/cinder master: VMAX driver - VMAX list manageable volumes and snapshots.  https://review.openstack.org/55729711:26
*** mvk has quit IRC11:35
*** salv-orl_ has joined #openstack-cinder11:36
*** salv-orlando has quit IRC11:41
*** gkadam__ has quit IRC11:44
*** moshele has quit IRC11:47
*** abishop has joined #openstack-cinder11:48
*** andreas_s has quit IRC12:01
*** andreas_s_ has joined #openstack-cinder12:02
*** andreas__ has joined #openstack-cinder12:03
*** tpsilva has joined #openstack-cinder12:05
*** edmondsw has joined #openstack-cinder12:06
*** sapd__ has joined #openstack-cinder12:06
*** andreas_s_ has quit IRC12:07
*** chhagarw has joined #openstack-cinder12:08
*** sapd_ has quit IRC12:09
*** armaan has quit IRC12:09
*** lpetrut_ has quit IRC12:15
*** sapd__ has quit IRC12:17
*** sapd_ has joined #openstack-cinder12:17
*** moshele has joined #openstack-cinder12:18
*** gkadam has joined #openstack-cinder12:19
*** odyssey4me has quit IRC12:22
*** odyssey4me has joined #openstack-cinder12:22
*** moshele has quit IRC12:23
*** sapd_ has quit IRC12:27
*** sapd_ has joined #openstack-cinder12:27
openstackgerritAlan Bishop proposed openstack/cinder stable/queens: Reduce scope of the lock for image volume cache  https://review.openstack.org/55731812:28
*** sapd_ has quit IRC12:32
*** sapd_ has joined #openstack-cinder12:32
*** zhurong has quit IRC12:34
*** andreas__ has quit IRC12:35
*** andreas_s has joined #openstack-cinder12:35
*** pchavva has joined #openstack-cinder12:36
*** andreas_s has quit IRC12:37
*** andreas_s has joined #openstack-cinder12:39
*** gman-tx has joined #openstack-cinder12:42
*** chkumar246 is now known as chandankumar12:43
*** MarkMielke has joined #openstack-cinder12:48
*** sapd__ has joined #openstack-cinder12:52
*** sapd_ has quit IRC12:52
*** salv-orlando has joined #openstack-cinder12:56
*** salv-or__ has joined #openstack-cinder12:57
*** salv-orl_ has quit IRC12:58
*** mvk has joined #openstack-cinder12:58
*** kaisers has joined #openstack-cinder12:58
*** dustins has joined #openstack-cinder12:58
*** salv-orlando has quit IRC13:02
*** salv-or__ has quit IRC13:02
*** salv-orlando has joined #openstack-cinder13:03
*** lbragstad has joined #openstack-cinder13:04
*** mriedem has joined #openstack-cinder13:06
*** salv-orlando has quit IRC13:08
*** gman-tx has quit IRC13:09
*** armaan has joined #openstack-cinder13:11
*** eharney has joined #openstack-cinder13:14
*** gouthamr has joined #openstack-cinder13:19
*** kaisers has quit IRC13:29
*** psachin has quit IRC13:30
*** mmethot has quit IRC13:31
*** lhx_ has joined #openstack-cinder13:31
*** yangyapeng has quit IRC13:35
*** alexchadin has quit IRC13:37
*** zhaochao has joined #openstack-cinder13:37
*** mmethot has joined #openstack-cinder13:37
*** alexchadin has joined #openstack-cinder13:39
*** liverpooler has joined #openstack-cinder13:41
*** kaisers has joined #openstack-cinder13:42
*** kongwei has joined #openstack-cinder13:42
*** kaisers has quit IRC13:47
*** kongwei has quit IRC13:47
*** gman-tx has joined #openstack-cinder13:50
*** knylande_ has joined #openstack-cinder13:52
*** salv-orlando has joined #openstack-cinder13:53
*** armaan has quit IRC13:59
*** kongwei has joined #openstack-cinder14:00
*** armaan has joined #openstack-cinder14:05
*** dklyle has joined #openstack-cinder14:05
*** david-lyle has quit IRC14:05
*** dklyle has quit IRC14:07
*** dklyle has joined #openstack-cinder14:07
*** hemna_ has joined #openstack-cinder14:08
*** jbernard has quit IRC14:09
*** jbernard has joined #openstack-cinder14:10
*** yangyapeng has joined #openstack-cinder14:10
*** mchlumsky has joined #openstack-cinder14:17
*** kongwei has quit IRC14:19
*** _alastor_ has quit IRC14:20
eharneyjungleboyj: xyang: so what's the plan here?  https://review.openstack.org/#/c/556640/14:22
xyangeharney, jungleboyj: what's the new merge policy?  I'm fine getting this merged.  Looks good14:24
*** alexchadin has quit IRC14:24
smcginniseharney, xyang: Spot check of third party CI failures didn't look related. +W14:25
*** kongwei has joined #openstack-cinder14:27
*** r-daneel has joined #openstack-cinder14:28
*** r-daneel has quit IRC14:32
*** r-daneel has joined #openstack-cinder14:33
*** felipemonteiro has joined #openstack-cinder14:33
*** moshele has joined #openstack-cinder14:33
tommylikehuhey eharney, what's your suggestion on the strict mode of signature verification?14:34
*** crose has joined #openstack-cinder14:35
tommylikehusmcginnis: do you mind to take a look at this spec? https://review.openstack.org/#/c/557112/14:35
eharneytommylikehu: i'm not sure, it's not obvious yet how to combine the idea of strict mode meaning "always require verification" with the fact that some drivers won't do verification14:35
eharneytommylikehu: because the current proposal makes it sound like "always require verification except it won't work in conditions X, Y, and Z" and so nobody is going to be able to figure out what it's actually doing14:36
tommylikehueharney: that's is true and what's why you asked to add metadata14:36
eharneyadding metadata is also helpful for the "optional" case... i'm not sure it helps with this issue14:37
*** sapd__ has quit IRC14:37
eharneywe decided we needed a "strict" mode after Duncan suggested that, but nobody has figured out how it would work exactly, so far14:38
*** sapd__ has joined #openstack-cinder14:38
tommylikehuDuncanT: :)14:38
eharneyi'm wondering if we should just rely on setting the user-visible metadata which pushes the issue of requiring it or not back to the user14:39
tommylikehulooks it meaningless when we can't cover every cases14:39
eharneythen users that demand verification can check for it14:39
*** kongwei has quit IRC14:39
eharneybecause the signatures are handled by users anyway, but the "strict" option is a deployer concern... sounds like having it as a Cinder option may not be the right place14:40
*** sapd__ has quit IRC14:40
*** moshele has quit IRC14:40
eharneyas always, it depends on mapping out what issues specifically this is protecting against14:40
tommylikehueharney:  sounds reasonable14:40
DuncanTHey, there are smart people to figure out solutions, I just find the holes :-)14:40
*** sapd__ has joined #openstack-cinder14:41
tommylikehuDuncanT:  lol14:41
*** sapd__ has quit IRC14:41
DuncanTIn general I couldn't see any real point to it without a working strict mode14:41
*** sapd__ has joined #openstack-cinder14:41
eharneywell, the strict mode as proposed doesn't seem to really add much14:41
*** sapd__ has quit IRC14:41
*** sapd__ has joined #openstack-cinder14:42
eharneybecause it can only be implemented as "strict some of the time"14:42
tommylikehueharney, DuncanT  maybe we can add this option when we really can do it in the backend14:42
DuncanTAnd without strict mode, what is the point of the feature?14:42
*** moshele has joined #openstack-cinder14:43
DuncanTThat was what I didn't understant14:43
eharneythe point of the feature is that a user provides a signature and the user can verify that the signature was validated at the time of the volume was created, protecting against mitm and corruption problems around the glance service14:43
DuncanT...except if they're using ceph as an image store14:43
eharneywell, that's the part i'm still trying to figure out14:44
eharneyhow does Nova's version of this same feature work when using Ceph for both Glance and the instance store?14:44
tommylikehuDuncanT:  can we cover the case when volume is created via clone image or clone image volume?14:45
*** moshele has quit IRC14:45
eharneydoes it do anything?14:45
DuncanTI think the ability of a user to manually query the validity of the signature is of very limited value from a security PoV14:45
eharneyi kind of agree, but i'm not sure that making this a cinder.conf option makes sense14:46
tommylikehueharney:  need to check it, but last time when I see it, they only validate signature when image is downloaded via glanceclient14:47
tommylikehueharney: https://github.com/openstack/nova/blob/24379f1822e3ae1d4f7c8398e60af6e52b386c32/nova/image/glance.py#L33314:48
*** kongwei has joined #openstack-cinder14:51
*** gman-tx_ has joined #openstack-cinder14:55
eharneytommylikehu: yeah, it looks like it's skipped if it's a direct backend transfer from glance (allowed_direct_url_schemas above)14:55
*** gman-tx has quit IRC14:57
*** gman-tx_ is now known as gman-tx14:57
*** r-daneel has quit IRC15:00
*** jiaopengju has quit IRC15:02
*** links has quit IRC15:03
eharneytommylikehu: it makes me think we should go with our original plan, and figure out how to make a "strict" mode as a later feature15:03
*** jiaopengju has joined #openstack-cinder15:03
*** felipemonteiro_ has joined #openstack-cinder15:04
*** ganso has joined #openstack-cinder15:06
*** r-daneel has joined #openstack-cinder15:07
*** felipemonteiro has quit IRC15:07
tommylikehueharney: agree:)15:07
*** pcaruana has quit IRC15:09
*** luizbag has joined #openstack-cinder15:10
*** kongwei has quit IRC15:11
DuncanTeharney: tommylikehu: Under what circumstances is this information actually checked / useful?15:11
*** gyankum has joined #openstack-cinder15:11
*** zhaochao has quit IRC15:13
*** kaisers has joined #openstack-cinder15:19
*** lhx_ has quit IRC15:20
*** kaisers_ has joined #openstack-cinder15:21
*** _alastor_ has joined #openstack-cinder15:23
*** kaisers has quit IRC15:23
*** kaisers_ has quit IRC15:26
*** imacdonn has quit IRC15:26
*** imacdonn has joined #openstack-cinder15:27
*** lkuchlan_ has quit IRC15:28
tommylikehuDuncanT:  in cinder, this is useful when image content is downloaded from glance (and not cinder backend image)15:28
eharneyDuncanT: this is described in the spec... mitm between Cinder and Glance, or corruption on the Glance backend, which can be caught by Cinder15:29
tommylikehuhey geguileo, smcginnis, that's is true we discussed to use json filter to support this. but I removed it from spec as it can not really work without modify json filter codes.15:31
tommylikehuhttps://review.openstack.org/#/c/557112/15:31
DuncanTeharney: My point being, if it only works for some images some of the time, is it actually useful?15:32
eharneyDuncanT: it is if the user can validate that it happens15:32
DuncanTeharney: I'm dubious, but if you're sure...15:33
eharneyand it still would succeed in catching issues with signed images15:33
DuncanTeharney: I really can't see any user actually checking it15:33
tommylikehusmcginnis, geguileo: plus it's not convenient for administrator to use, I would suggest to have the cloud vendor to write their own filter plugins to support it15:33
eharneyeven without checking it, it catches a whole class of issues15:34
*** gman-tx has quit IRC15:43
jgriffithHas Tempest become highly unstable for other CI's as of late or is it just *mine*?15:44
jgriffithswap and snapshot related tests inparticular15:44
*** dustins has quit IRC15:45
*** abhishekk has joined #openstack-cinder15:48
*** andreas_s has quit IRC15:51
*** andreas_s has joined #openstack-cinder15:51
*** armaan has quit IRC15:52
*** gman-tx has joined #openstack-cinder15:53
*** itlinux has joined #openstack-cinder15:54
*** andreas_s_ has joined #openstack-cinder15:55
*** andreas_s has quit IRC15:56
openstackgerritEric Harney proposed openstack/cinder stable/pike: Cinder Tempest plugin tests: Switch to stable interface  https://review.openstack.org/55738515:56
*** belmoreira has quit IRC15:57
*** felipemonteiro_ has quit IRC16:00
*** andreas_s_ has quit IRC16:00
smcginnistommylikehu: geguileo has more experience there, and seemed to have some ideas around it, so I'd like to get his input.16:03
tommylikehusmcginnis:  waiting for his response too :)16:04
smcginnistommylikehu: But if you wanted to expand on that spec in the meantime, that might be good. I don't think there is enough detail there to fully understand the proposal.16:04
tommylikehusmcginnis:  ok16:04
smcginnistommylikehu: Thanks!16:05
*** andreas_s has joined #openstack-cinder16:06
*** andreas_s has quit IRC16:10
openstackgerritMerged openstack/cinder master: Don't use custom password generating code  https://review.openstack.org/55664016:11
*** alkhodos has joined #openstack-cinder16:30
*** yangyapeng has quit IRC16:32
*** gman-tx_ has joined #openstack-cinder16:38
*** gman-tx_ has quit IRC16:38
*** gman-tx has quit IRC16:40
*** armaan has joined #openstack-cinder16:40
openstackgerritJohn Griffith proposed openstack/cinder master: Add IPV6 support for SolidFire MVIP  https://review.openstack.org/55700116:43
*** lpetrut has quit IRC16:45
*** gouthamr has quit IRC16:55
*** mvk has quit IRC16:56
*** r-daneel_ has joined #openstack-cinder16:59
*** e0ne has quit IRC17:01
*** r-daneel has quit IRC17:01
*** r-daneel_ is now known as r-daneel17:01
alkhodosjungleboyj: Hi, you have commented about documentation for the driver in this review https://review.openstack.org/#/c/513143/, could you help me out on how to post docs? Google did not help :(17:08
*** AlexeyAbashkin has quit IRC17:08
jungleboyjalkhodos:  Hey.17:15
jungleboyjalkhodos:  What do you need help with?17:15
*** vivsoni_ has joined #openstack-cinder17:15
smcginnisalkhodos: Here's an existing example - https://raw.githubusercontent.com/openstack/cinder/master/doc/source/configuration/block-storage/drivers/nexentastor5-driver.rst17:16
*** bswartz1 is now known as bswartz17:16
jungleboyjalkhodos:  You can find information on how to contribute documentation here:  https://docs.openstack.org/cinder/latest/contributor/documentation.html17:17
alkhodossmcginnis: jungleboyj: So the *.rst file jsut goes in the patch?17:17
*** stvnoyes has joined #openstack-cinder17:17
vivsoni_jungleboyj: please review https://review.openstack.org/#/c/556765/17:17
jungleboyjCorrect.  You add an rst file similar to the one smcginnis references.17:18
jungleboyjAdd it as part of the patch in doc/source/configuration/block-storage/drivers17:18
alkhodosoh, thanks. For some reason I was sure that there was a separate openstack-docs project17:18
*** stvnoyes1 has quit IRC17:18
jungleboyjalkhodos:  There is but they no longer handle the project documentation.17:19
jungleboyjWe now have to do that.17:19
alkhodosjungleboyj: oh I see. I remember posting old docs to mitaka and I had to make a separate patch/review for that.17:21
jungleboyjalkhodos:  Ah, that is why you are confused.  Yeah, new process now.17:21
alkhodosjungleboyj: I like the new one more :) Thanks for your help17:24
jungleboyjGood.  Yeah, I think it is good that we have more control over our documentation.17:25
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: FC: refactor fczm utils decorators to functions  https://review.openstack.org/47285517:26
hemna_speaking of docs...17:26
hemna_docs still fails on that ^^17:26
hemna_:(17:26
jungleboyjhemna_:  Hey, I tried to recreate that and couldn't.17:27
hemna_I don't get it17:28
hemna_it shouldn't fail17:28
*** mvk has joined #openstack-cinder17:28
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Add VMAX specific metadata to volume  https://review.openstack.org/55744817:28
*** gouthamr has joined #openstack-cinder17:29
*** moshele has joined #openstack-cinder17:30
*** lpetrut has joined #openstack-cinder17:30
*** gouthamr has quit IRC17:32
*** dustins has joined #openstack-cinder17:33
jungleboyjYeah, I tried to make it fail locally and it wouldn't.17:35
*** tesseract has quit IRC17:35
smcginnishemna_: Docs just built fine locally for me for that patch.17:37
*** armaan has quit IRC17:39
*** abhishekk has quit IRC17:41
*** felipemonteiro has joined #openstack-cinder17:43
*** felipemonteiro_ has joined #openstack-cinder17:43
*** felipemonteiro has quit IRC17:47
jungleboyjsmcginnis:  Ok, so you too.17:47
*** moshele has quit IRC17:49
*** r-daneel_ has joined #openstack-cinder17:51
*** psachin has joined #openstack-cinder17:52
*** r-daneel has quit IRC17:53
*** r-daneel_ is now known as r-daneel17:53
*** gkadam has quit IRC17:54
*** jmccrory has quit IRC17:59
*** itlinux has quit IRC18:00
*** lpetrut has quit IRC18:02
*** lvdombrkr has joined #openstack-cinder18:05
*** jmccrory has joined #openstack-cinder18:06
eharneyhemna_: jungleboyj: smcginnis: it will fail locally if you run the same commands that the gate job runs18:06
eharneyour tox docs job doesn't run with warning-as-error enabled18:06
eharneysource .tox/docs/bin/activate18:07
eharneysphinx-build -b html -W doc/source doc/build/html18:07
eharneyhttp://logs.openstack.org/55/472855/9/check/build-openstack-sphinx-docs/33681d4/ara/    <- click on the red FAILED and you'll see what the job runs18:07
*** swamireddy has joined #openstack-cinder18:08
eharneywe are missing a -W compared in our tox command compared to what runs there18:09
*** liverpooler has quit IRC18:11
*** itlinux has joined #openstack-cinder18:15
*** chhagarw has quit IRC18:16
*** lpetrut has joined #openstack-cinder18:16
*** psachin has quit IRC18:16
*** e0ne has joined #openstack-cinder18:19
*** abishop_ has joined #openstack-cinder18:20
jungleboyjeharney:  Oh, interesting.  I thought we had set warning-as-error .18:22
*** lvdombrkr has quit IRC18:22
jungleboyjThat should be fixed.18:22
eharneyyes18:22
jungleboyjOk, I will recreate and take a look again.18:22
*** abishop has quit IRC18:23
hemna_https://www.youtube.com/watch?v=IzpLiO1QixU18:25
hemna_lunch break :)18:25
*** vivsoni_ has quit IRC18:31
*** voelzmo has joined #openstack-cinder18:33
*** gyankum has quit IRC18:36
jungleboyjhemna_:  That looks more like playing than lunching.18:38
hemna_it is :)18:39
*** gyankum has joined #openstack-cinder18:40
smcginnisOh! It's probably our autodoc stuff.18:46
smcginnisHmm, nope. We don't have that in setup.cfg.18:47
jungleboyjsmcginnis:  Yeah.  that is the autodoc.18:47
gansojungleboyj: ping18:48
jungleboyjganso: Pong.18:48
*** gyankum has quit IRC18:48
smcginnisjungleboyj: Normally I've seen that in setup.cfg. Do you know where we have that?18:48
*** voelzmo has quit IRC18:48
jungleboyjsmcginnis:  Seen what?18:48
gansojungleboyj: hey Jay, quick question, I saw several driver patches are recently merging in stable/pike. Isn't this branch supposed to be open for security and critical fixes only?18:48
gansojungleboyj: based on your comment in https://review.openstack.org/#/c/528783/ I thought it was closed18:49
jungleboyjganso:  That was the plan and then the new proposal to keep the branches open longer came along so I eased up on that a bit.18:49
smcginnisjungleboyj: This stuff - https://github.com/openstack/glance/blob/master/setup.cfg#L10918:50
gansojungleboyj: the merged patches I'm referring to are: https://review.openstack.org/#/c/554656/, https://review.openstack.org/#/c/554701/ and https://review.openstack.org/#/c/551831/18:50
jungleboyjganso:  That is why I have added the discussion in next week's meeting on the topic.18:50
gansojungleboyj: ok, so that means the branch is still open and we could backport this https://review.openstack.org/#/c/528782/ ?18:51
jungleboyjsmcginnis:  Yeah, that has been in there for quite a while.18:51
jungleboyjThat is why I did all that work to fix up docstrings.18:51
smcginnisjungleboyj: Do you know where we do that from? It's not in Cinder's setup.cfg where I expected to find it.18:52
jungleboyjganso:  Yeah, that is small.  I think I am ok with that backport without further discussion.18:52
gansojungleboyj: ok, I will test it downstream and work on it18:52
gansojungleboyj: thank you!18:52
jungleboyjganso:  Welcome.18:53
*** moshele has joined #openstack-cinder18:54
*** AlexeyAbashkin has joined #openstack-cinder18:54
*** lpetrut has quit IRC18:54
jungleboyjsmcginnis:  Sorry, I am not following what you are asking about.18:55
smcginnisjungleboyj: Normally the bit that triggers the automatic documentation of modules using their docstrings is done by something like that [pbr] section I pointed to in glance.18:56
smcginnisjungleboyj: Cinder does not have that.18:56
smcginnisBut we are getting that automatic documentation generation, so I'm wondering if you know where we are triggering that to happen.18:56
*** moshele has quit IRC18:56
jungleboyjsmcginnis:  Oh .... *Sigh* I didn't notice that was Glance.  Have too many things going on.18:56
jungleboyjHold on.18:57
*** AlexeyAbashkin has quit IRC18:59
jungleboyjsmcginnis:  I think it happens due to doc/source/conf.py19:01
jungleboyjAs part of sphinx-build -b html doc/source doc/build/html19:02
smcginnisLooks like we have some hook that must call this: https://github.com/openstack/cinder/blob/54d6c2da9c1043ba7a9d24fe7395150daa362aae/doc/generate_autodoc_index.sh19:02
smcginnisGood, more code I might be able to get rid of soon. :)19:02
jungleboyjsmcginnis:  Right.19:03
jungleboyjIf there is a better way to do that.  ++19:03
jungleboyjIn the mean time let me figure out hemna_ 's issue19:03
jungleboyjI know I have fixed that before but can't remember what it was.19:04
smcginnisI would think it's just a tweak to the unity driver docstring.19:04
jungleboyjRight.  I need to play with it as all the recommendations I have hemna_  didn't work.19:05
*** gkadam has joined #openstack-cinder19:06
smcginnisYeah, adding ".. code-block::" like I had suggested seems to fix it locally for me.19:08
jungleboyjsmcginnis:  Verifying.19:17
*** dikonoo has quit IRC19:19
hemna_in the comment block?19:20
hemna_why doesn't it fail on other drivers then?19:20
eharneyprobably they don't have docstrings that look like json structures19:21
jungleboyjHe he.19:22
smcginnisIt's an embedded chunk of rst, so one option is to paste it into a temporary file and run doc8 against it.19:24
hemna_https://review.openstack.org/#/c/472855/10/cinder/volume/drivers/hpe/hpe_3par_fc.py19:27
hemna_pretty much the same19:27
hemna_doesn't fail19:27
smcginnishemna_: Have you tried? Once I fixed the first error, then it went on to the next, etc.19:30
hemna_nah, not yet, I'm stuck ceph'ing at the moment19:31
smcginnishemna_: I think we have a bunch of latent doc format errors in there.19:31
hemna_oh, so it fails on the 3par fc driver the same way?19:31
smcginnisI didn't go that far. After I got by the unity driver error, then it gives on in the vmax driver.19:32
smcginnisI'm assuming after that there would be another one, so likely it would eventually fail on the 3par one.19:32
hemna_damn, so same issue then in multiple drivers19:32
smcginnisYeah, I think so.19:32
hemna_arg19:32
hemna_thanks for the info19:33
hemna_I'll hack on it when I get a chance19:33
smcginnisWe never _really_ treated them as rst before, so now that we have warnings enabled for doc builds that pull in autodoc'ing module docstrings, we're going to have to pay attnetion to that I guess.19:33
hemna_yah it sounds like it.   I just got lucky and hit it first.19:34
hemna_wee!19:34
smcginnisHah, lucky you! :D19:35
*** MarkMielke has quit IRC19:37
*** felipemonteiro_ has quit IRC19:40
*** felipemonteiro_ has joined #openstack-cinder19:40
*** asbishop has joined #openstack-cinder19:44
*** abishop_ has quit IRC19:46
Swansonhttps://pbs.twimg.com/media/DZY7_V0X4AU0bt2.jpg19:50
openstackgerritSean McGinnis proposed openstack/cinder master: Remove unused doc extensions  https://review.openstack.org/55749519:53
*** felipemonteiro__ has joined #openstack-cinder19:53
*** felipemonteiro_ has quit IRC19:58
eharneySwanson: the longer i think about that, the more i'm disappointed that "Learn More" isn't a working link... but it's probably for the best20:05
*** moshele has joined #openstack-cinder20:11
Swansoneharney, That was kind of my thought, too. I saw it, wanted more information, but couldn't get myself to actually Bing "long egg".20:11
*** gkadam has quit IRC20:12
*** eharney has quit IRC20:13
itlinuxhello all, I wonder if anyone suggest to have a sep bond for the controllers as well (running on Tripleo) or just the compute nodes.. That's my plan there but want to see from cinders' gurus20:15
itlinuxthanks20:15
openstackgerritJay Bryant proposed openstack/cinder master: FC: refactor fczm utils decorators to functions  https://review.openstack.org/47285520:15
*** knylande_ has quit IRC20:18
*** sapd__ has quit IRC20:30
*** kaisers has joined #openstack-cinder20:31
*** sapd__ has joined #openstack-cinder20:31
*** r-daneel_ has joined #openstack-cinder20:32
*** r-daneel has quit IRC20:33
*** r-daneel_ is now known as r-daneel20:33
*** kaisers has quit IRC20:35
*** nicolasbock has joined #openstack-cinder20:39
*** pchavva has quit IRC20:41
*** erlon has joined #openstack-cinder20:43
*** moshele has quit IRC20:44
*** andreas_s has joined #openstack-cinder20:45
*** erlon has quit IRC20:48
*** e0ne has quit IRC20:48
*** andreas_s has quit IRC20:49
mnaseris there a way to recalculate quota manually?20:53
mnaserwe ran into a race condition again20:53
mnaserquota limit is hit but the real usage is not the same20:55
*** luizbag has quit IRC20:57
*** r-daneel_ has joined #openstack-cinder21:01
*** r-daneel has quit IRC21:03
*** r-daneel_ is now known as r-daneel21:03
*** moshele has joined #openstack-cinder21:08
*** asbishop has quit IRC21:13
*** itlinux has quit IRC21:14
*** crose has quit IRC21:20
*** moshele has quit IRC21:26
*** e0ne has joined #openstack-cinder21:29
smcginnismnaser: Unfortunately no. That seems like a good idea. We would need to be very careful to make sure it's not racy, but we could probably add something that tries to reconcile quota against actual usage.21:32
*** sticker has joined #openstack-cinder21:40
mnasersmcginnis: that's why it would be useful to figure out maybe as an audit21:40
mnaserlike a noop mode of trying to catch issues, and then something to fix it, hopefully it doesn't stay as a 'permanent' workaround21:41
smcginnismnaser: So just output a message where it sees a possible out of sync condition?21:41
mnasersmcginnis: yeah maybe like cinder-manage check-quotas and cinder-manage check-quotas --fix21:42
*** edmondsw has quit IRC21:43
smcginnisI can put it on my list as something to look into, but it likely won't be any time soon.21:45
smcginnismnaser: Might be something good to bring up in the cinder meeting.21:45
mnasersmcginnis: few hours late, but i'll try to look at it21:45
smcginnismnaser: It seems like a reasonable idea to me, but there may be some reason I don't know about why something similar hasn't been done before.21:45
mnasersmcginnis: yeah, that's possible, we'll see, i'll try to bring it up21:46
smcginnismnaser: Cool, thanks!21:47
-openstackstatus- NOTICE: the zuul web dashboard will experience a short downtime as we roll out some changes - no job execution should be affected21:51
*** harlowja has joined #openstack-cinder21:53
*** e0ne has quit IRC21:55
openstackgerritSean McGinnis proposed openstack/cinder master: FC: refactor fczm utils decorators to functions  https://review.openstack.org/47285522:05
openstackgerritSean McGinnis proposed openstack/cinder master: Fix doc format errors in module docstrings  https://review.openstack.org/55753122:05
*** dustins has quit IRC22:19
*** ganso has quit IRC22:27
*** rcernin has joined #openstack-cinder22:28
*** mriedem has quit IRC22:34
*** mriedem has joined #openstack-cinder22:36
*** felipemonteiro__ has quit IRC22:43
*** tpsilva has quit IRC22:44
*** mriedem has quit IRC22:47
*** salv-orlando has quit IRC23:21
*** salv-orlando has joined #openstack-cinder23:22
*** salv-orlando has quit IRC23:27
*** wanghao_ has joined #openstack-cinder23:46
*** wanghao has quit IRC23:48
-openstackstatus- NOTICE: Zuul has been restarted to update to the latest code; existing changes have been re-enqueued, you may need to recheck changes uploaded in the past 10 minutes23:57

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!