Wednesday, 2017-07-05

*** gcb has joined #openstack-cinder00:15
*** edmondsw has joined #openstack-cinder00:16
*** stakeda has joined #openstack-cinder00:17
*** iceyao has joined #openstack-cinder00:20
*** edmondsw has quit IRC00:20
*** iceyao has quit IRC00:25
*** iceyao has joined #openstack-cinder00:41
*** iceyao has quit IRC00:46
*** openstackgerrit has joined #openstack-cinder00:46
openstackgerritWang Xuguang proposed openstack/cinder master: api-ref: Add parameter "offset" to list(volumes/snapshots/types) apis  https://review.openstack.org/48032000:46
*** wanghao has joined #openstack-cinder00:48
*** winston-d_ has joined #openstack-cinder00:52
*** iceyao has joined #openstack-cinder01:02
openstackgerritWang Xuguang proposed openstack/cinder master: api-ref: Add parameter "offset" to list(volumes/snapshots/types) apis  https://review.openstack.org/48032001:06
*** iceyao has quit IRC01:06
*** zhangguoqing has joined #openstack-cinder01:12
*** wanghao has quit IRC01:17
*** wanghao has joined #openstack-cinder01:18
*** iceyao has joined #openstack-cinder01:22
openstackgerritjunbo.li proposed openstack/cinder master: Use GroupSnapshotStatus enum field  https://review.openstack.org/48020601:24
*** Shunli has joined #openstack-cinder01:24
openstackgerritliuyamin proposed openstack/cinder master: Use QoSConsumerValues enum field  https://review.openstack.org/47967401:34
*** markvoelker has joined #openstack-cinder01:36
*** bkopilov has quit IRC01:59
*** iceyao has quit IRC02:02
*** iceyao has joined #openstack-cinder02:03
*** zhaochao has joined #openstack-cinder02:04
*** edmondsw has joined #openstack-cinder02:04
*** ChubYann has quit IRC02:06
*** markvoelker has quit IRC02:08
*** edmondsw has quit IRC02:09
*** zhurong has joined #openstack-cinder02:12
*** lbragstad has joined #openstack-cinder02:17
*** ChubYann has joined #openstack-cinder02:19
openstackgerritlihaijing proposed openstack/cinder master: Api-ref: fix v2/v3 hosts extension api doc  https://review.openstack.org/47779802:40
openstackgerritlihaijing proposed openstack/cinder master: Api-ref: add 'os-services' v3 api doc  https://review.openstack.org/47878902:50
openstackgerritWang Xuguang proposed openstack/cinder master: api-ref: Add parameter "offset" to list apis  https://review.openstack.org/48032002:54
*** abhitechie has joined #openstack-cinder02:57
openstackgerritqtlu proposed openstack/cinder master: Api-ref: Update doc for show backups  https://review.openstack.org/47252602:59
*** abhitechie has quit IRC03:05
*** abhitechie has joined #openstack-cinder03:05
*** markvoelker has joined #openstack-cinder03:06
*** bkopilov has joined #openstack-cinder03:13
*** gkadam has joined #openstack-cinder03:24
*** abhitechie has quit IRC03:25
*** gkadam has quit IRC03:26
*** gkadam has joined #openstack-cinder03:27
*** markvoelker has quit IRC03:39
*** links has joined #openstack-cinder03:44
*** zhangguoqing has quit IRC03:44
*** adisky__ has joined #openstack-cinder03:46
*** zhangguoqing has joined #openstack-cinder03:49
*** iceyao has quit IRC03:53
*** lbragstad has quit IRC03:55
*** chhavi has joined #openstack-cinder04:04
*** zhangguoqing has quit IRC04:08
*** lpetrut has joined #openstack-cinder04:09
*** lpetrut has quit IRC04:12
*** iceyao has joined #openstack-cinder04:15
*** Nel1x has quit IRC04:18
*** _pewp_ has quit IRC04:18
*** lpetrut has joined #openstack-cinder04:19
*** _pewp_ has joined #openstack-cinder04:19
*** iceyao has quit IRC04:20
openstackgerritMerged openstack/cinder master: Move driver options to "backend_defaults" section  https://review.openstack.org/45365404:23
*** Nel1x has joined #openstack-cinder04:23
*** psachin has joined #openstack-cinder04:31
*** iceyao has joined #openstack-cinder04:33
*** markvoelker has joined #openstack-cinder04:36
*** arnewiebalck_ has joined #openstack-cinder04:38
*** iceyao has quit IRC04:38
*** chhavi has quit IRC04:38
*** chhavi has joined #openstack-cinder04:41
*** iceyao has joined #openstack-cinder04:42
*** chhavi has quit IRC04:44
*** mtreinish has quit IRC04:44
openstackgerritliuyamin proposed openstack/cinder master: Use QoSConsumerValues enum field  https://review.openstack.org/47967404:46
*** yangyapeng has joined #openstack-cinder04:50
*** mtreinish has joined #openstack-cinder04:51
*** yangyapeng has quit IRC04:57
*** yangyapeng has joined #openstack-cinder04:58
*** ChubYann has quit IRC05:00
openstackgerritThomas Bechtold proposed openstack/cinder master: Revert "Don't use ignoreskipactivation for thin LVM"  https://review.openstack.org/48022605:01
*** lpetrut has quit IRC05:02
openstackgerritThomas Bechtold proposed openstack/os-brick master: Revert "Don't use ignoreskipactivation for thin LVM"  https://review.openstack.org/48022405:03
*** yangyapeng has quit IRC05:05
*** yangyapeng has joined #openstack-cinder05:06
*** yangyape_ has joined #openstack-cinder05:07
*** markvoelker has quit IRC05:08
*** yangyapeng has quit IRC05:11
*** nikeshm has joined #openstack-cinder05:15
*** yangyape_ has quit IRC05:20
openstackgerritKushal Wathore proposed openstack/cinder master: HPE 3PAR: ISCSI/FC drivers – code refactoring  https://review.openstack.org/44647505:20
*** yangyapeng has joined #openstack-cinder05:21
*** lpetrut has joined #openstack-cinder05:25
openstackgerritzhangdaolong proposed openstack/os-brick master: the ioctx should be close when open rbd pool failed  https://review.openstack.org/44411405:37
*** iceyao has quit IRC05:39
*** arnewiebalck_ has quit IRC05:40
*** edmondsw has joined #openstack-cinder05:40
*** yangyapeng has quit IRC05:40
*** yangyapeng has joined #openstack-cinder05:41
*** Nel1x has quit IRC05:41
*** edmondsw has quit IRC05:45
*** zhangguoqing has joined #openstack-cinder05:50
*** iceyao has joined #openstack-cinder05:58
*** iceyao has quit IRC06:02
*** markvoelker has joined #openstack-cinder06:05
*** iceyao has joined #openstack-cinder06:09
*** iceyao has quit IRC06:13
nikeshmgeguileo:  hi06:18
vivsoni_hi06:20
openstackgerritKushal Wathore proposed openstack/cinder master: 3PAR: Fixed image cache enabled capability.  https://review.openstack.org/46174406:21
nikeshmvivsoni_ : hi06:22
*** lpetrut has quit IRC06:27
*** wxy- has joined #openstack-cinder06:35
*** belmoreira has joined #openstack-cinder06:36
*** iceyao has joined #openstack-cinder06:36
*** markvoelker has quit IRC06:39
*** iceyao has quit IRC06:41
*** pgadiya has joined #openstack-cinder06:43
*** Poornima has joined #openstack-cinder06:43
*** wxy- has quit IRC06:44
*** lpetrut has joined #openstack-cinder06:50
*** tesseract has joined #openstack-cinder07:02
openstackgerritParashuram Hallur proposed openstack/cinder master: Added CG capability to volume group in CoprHD  https://review.openstack.org/45961007:05
*** gouthamr has joined #openstack-cinder07:15
*** lpetrut has quit IRC07:16
*** rcernin has joined #openstack-cinder07:19
*** lpetrut has joined #openstack-cinder07:23
*** edmondsw has joined #openstack-cinder07:29
*** lpetrut has quit IRC07:32
*** edmondsw has quit IRC07:33
*** markvoelker has joined #openstack-cinder07:36
nikeshmthe issue in the bug https://bugs.launchpad.net/tempest/+bug/1702102 is not coming by using n-api-meta in OVERRIDE_ENABLED_SERVICES in our CI07:40
openstackLaunchpad bug 1702102 in tempest "Failed to establish authenticated ssh connection to cirros" [Undecided,New]07:40
nikeshmthanks Dr. Jens Rosenboom (j-rosenboom-j) for suggestion07:40
openstackgerritzheng yin proposed openstack/cinder master: make incremental effective when cinder-backup create  https://review.openstack.org/48006207:45
*** arnewiebalck_ has joined #openstack-cinder07:47
*** e0ne has joined #openstack-cinder07:52
openstackgerritzhangdaolong proposed openstack/cinder master: Fix Log parameter modification  https://review.openstack.org/48039807:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:01
openstackgerritzhangdaolong proposed openstack/cinder master: Fix Log parameter modification  https://review.openstack.org/48039808:01
openstackgerritzhangdaolong proposed openstack/cinder master: Fix Log parameter  https://review.openstack.org/48039808:03
*** mdovgal has joined #openstack-cinder08:04
*** markvoelker has quit IRC08:08
*** lkuchlan has joined #openstack-cinder08:14
*** Poornima has quit IRC08:15
*** Poornima has joined #openstack-cinder08:16
*** Poornima has quit IRC08:16
*** Poornima has joined #openstack-cinder08:16
*** Poornima has quit IRC08:17
*** Poornima has joined #openstack-cinder08:17
*** chenying has quit IRC08:19
*** chenying has joined #openstack-cinder08:19
*** arnewiebalck_ has quit IRC08:20
*** wxy- has joined #openstack-cinder08:20
*** Poornima has quit IRC08:23
*** Poornima has joined #openstack-cinder08:26
*** e0ne has quit IRC08:29
*** wxy- has quit IRC08:34
*** crushil has quit IRC08:36
*** links has quit IRC08:37
*** crushil has joined #openstack-cinder08:37
openstackgerritNgo Quoc Cuong proposed openstack/cinder master: Enable H904 check  https://review.openstack.org/48043908:39
*** crushil has quit IRC08:39
*** iceyao has joined #openstack-cinder08:40
*** crushil has joined #openstack-cinder08:40
*** lhx__ has joined #openstack-cinder08:41
*** yangyape_ has joined #openstack-cinder08:42
*** e0ne has joined #openstack-cinder08:42
*** yangyape_ has quit IRC08:46
*** yangyapeng has quit IRC08:46
*** yangyapeng has joined #openstack-cinder08:48
openstackgerritwangxiyuan proposed openstack/cinder master: Update version show response example  https://review.openstack.org/44846908:50
*** wxy- has joined #openstack-cinder08:51
*** links has joined #openstack-cinder08:53
*** wxy- has quit IRC08:56
*** wxy- has joined #openstack-cinder08:59
*** yangyapeng has quit IRC08:59
*** yangyapeng has joined #openstack-cinder09:00
*** wxy- has quit IRC09:00
openstackgerritChason Chan proposed openstack/cinder master: Switch to openstackdocstheme  https://review.openstack.org/48045209:01
*** yangyapeng has quit IRC09:04
*** yangyapeng has joined #openstack-cinder09:05
*** markvoelker has joined #openstack-cinder09:06
openstackgerritChason Chan proposed openstack/cinder master: Switch to openstackdocstheme  https://review.openstack.org/48045209:09
*** Shunli has quit IRC09:12
openstackgerritliuyamin proposed openstack/cinder master: Use QoSConsumerValues enum field  https://review.openstack.org/47967409:13
openstackgerritChason Chan proposed openstack/cinder master: Switch to openstackdocstheme  https://review.openstack.org/48045209:15
*** edmondsw has joined #openstack-cinder09:17
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Fix wrong usage of rbd.list_snaps return value  https://review.openstack.org/45578609:19
*** edmondsw has quit IRC09:21
*** wanghao_ has joined #openstack-cinder09:23
*** wanghao_ has quit IRC09:24
*** wanghao has quit IRC09:26
openstackgerritjunbo.li proposed openstack/python-cinderclient master: Add force delete volume in v3.23  https://review.openstack.org/47844709:33
*** Poornima has quit IRC09:36
*** Poornima has joined #openstack-cinder09:36
openstackgerritChason Chan proposed openstack/python-cinderclient master: Switch to openstackdocstheme  https://review.openstack.org/48047809:37
openstackgerritParashuram Hallur proposed openstack/cinder master: Added CG capability to volume group in CoprHD  https://review.openstack.org/45961009:38
*** stakeda has quit IRC09:38
*** markvoelker has quit IRC09:39
openstackgerritjunbo.li proposed openstack/cinder master: Add params for API v3 snapshot-create  https://review.openstack.org/47683109:42
*** links has quit IRC09:49
*** dalgaaf has quit IRC09:50
*** dalgaaf has joined #openstack-cinder09:54
*** yangyapeng has quit IRC09:55
*** yangyapeng has joined #openstack-cinder09:55
*** wxy- has joined #openstack-cinder10:00
*** links has joined #openstack-cinder10:03
*** iceyao has quit IRC10:08
*** yangyapeng has quit IRC10:12
openstackgerritjunbo.li proposed openstack/cinder master: [Api-ref]: Add the new added key/values  https://review.openstack.org/47576110:17
*** iceyao has joined #openstack-cinder10:19
*** iceyao has quit IRC10:23
*** wxy- has quit IRC10:27
*** wxy- has joined #openstack-cinder10:33
*** avishay has joined #openstack-cinder10:35
*** markvoelker has joined #openstack-cinder10:36
openstackgerritMerged openstack/cinder master: Unity: temp snapshot for backup deleted twice  https://review.openstack.org/47914810:42
avishayHi everyone.  Anyone from EMC available for a driver question?10:44
*** zhurong has quit IRC10:45
*** iceyao has joined #openstack-cinder10:46
openstackgerritMerged openstack/cinder master: Optimize: Replace 'fields.Enum' with 'object'  https://review.openstack.org/48006110:46
*** iceyao has quit IRC10:50
*** Poornima has quit IRC10:58
*** bkopilov has quit IRC10:59
*** edmondsw has joined #openstack-cinder11:05
*** Poornima has joined #openstack-cinder11:07
*** edmondsw has quit IRC11:09
*** markvoelker has quit IRC11:10
*** Poornima has quit IRC11:11
*** Poornima has joined #openstack-cinder11:11
ericyoungavishay: which driver?11:16
*** wxy- has quit IRC11:20
*** iceyao has joined #openstack-cinder11:22
*** lhx__ has quit IRC11:25
*** lhx__ has joined #openstack-cinder11:25
avishayericyoung: VMAX.  I'd like to know if its possible to provide credentials that are limited to the specified VMAX pool, rather than providing admin credentials.11:30
*** wxy- has joined #openstack-cinder11:31
*** wxy- has quit IRC11:37
*** pgadiya has quit IRC11:38
*** Poornima has quit IRC11:38
*** Poornima has joined #openstack-cinder11:41
ericyoungah... you are looking for the VMAX person then. I know ScaleIO but not VMAX. I'll see if I can find you an IRC userid o ping11:43
ericyoungavishay: ^^^11:45
avishayericyoung: excellent, thank you!11:45
*** abishop has joined #openstack-cinder11:46
ericyoungavishay: it may take a little while. It's still a bit early in the US and we had a holiday this week. I know who you need to speak to, but not their IRC handle11:49
avishayericyoung: Right, no worries11:50
*** pgadiya has joined #openstack-cinder11:55
*** nikeshm has quit IRC11:59
*** gcb has quit IRC12:01
*** yangyapeng has joined #openstack-cinder12:02
*** avishay has quit IRC12:03
*** avishay has joined #openstack-cinder12:03
*** markvoelker has joined #openstack-cinder12:07
*** yangyapeng has quit IRC12:07
*** Poornima has quit IRC12:08
*** liverpooler has joined #openstack-cinder12:09
*** pgadiya has quit IRC12:11
*** wxy- has joined #openstack-cinder12:13
openstackgerritShay Halsband proposed openstack/cinder master: XtremIO: fix fetching FC targets from X2 array  https://review.openstack.org/48054612:15
*** arnewiebalck has quit IRC12:15
openstackgerritIsaac Beckman proposed openstack/cinder master: XIV\A9000: Added replication group support  https://review.openstack.org/46873812:15
*** arnewiebalck has joined #openstack-cinder12:16
*** markvoelker has quit IRC12:16
*** markvoelker has joined #openstack-cinder12:17
*** edmondsw has joined #openstack-cinder12:22
openstackgerritiswarya vakati proposed openstack/cinder master: Replace six.iteritems() with .items()  https://review.openstack.org/47885512:23
*** pgadiya has joined #openstack-cinder12:23
*** jmlowe has quit IRC12:28
*** dims has quit IRC12:29
*** lkuchlan has quit IRC12:33
*** pgadiya has quit IRC12:33
openstackgerritPooja Jadhav proposed openstack/python-cinderclient master: Fix output of update command  https://review.openstack.org/47347212:34
*** yangyapeng has joined #openstack-cinder12:37
*** bkopilov has joined #openstack-cinder12:40
*** eharney has joined #openstack-cinder12:41
*** wxy- has quit IRC12:42
*** dims has joined #openstack-cinder12:44
*** e0ne has quit IRC12:46
*** e0ne has joined #openstack-cinder12:46
*** e0ne has quit IRC12:47
*** e0ne has joined #openstack-cinder12:47
*** Julien-zte has quit IRC12:56
*** Julien-zte has joined #openstack-cinder12:57
*** gcb has joined #openstack-cinder12:57
*** lucasxu has joined #openstack-cinder13:00
*** wxy- has joined #openstack-cinder13:02
*** wxy-_ has joined #openstack-cinder13:04
*** jmlowe has joined #openstack-cinder13:06
*** wxy- has quit IRC13:07
*** lhx__ has quit IRC13:07
*** avishay has quit IRC13:09
*** wxy-_ has quit IRC13:09
smcginniseharney: Morning. Did you see someone put up a couple revert patches for those LVM changes?13:11
eharneysmcginnis: yes13:11
smcginniseharney: Wasn't a lot of detail there, so I haven't looked in to why13:11
eharneyalready +2d https://review.openstack.org/#/c/480226/13:11
eharneyour thin LVM/LIO CI job caught the error13:12
smcginniseharney: OK, so there are legit reasons after all for how it was?13:12
*** avishay has joined #openstack-cinder13:13
eharneysmcginnis: there's still definitely a problem in there, i'm going to have to go figure out what and how to fix it13:13
eharneyi'm not sure the failure i was fixing shows up on any of our CI13:13
avishayericyoung: hey sorry, got disconnected, i'm back13:13
smcginniseharney: Anything we need to hold off on an os-brick release for?13:14
jungleboyjeharney:  smcginnis  Ok, so at least he added some detail in there.13:14
e0neeharney: hi. I would like to discuss LIO job failures if you have a time13:14
eharneysmcginnis: we should probably land the os-brick revert too and get back to the previous state before releasing os-brick13:14
eharneye0ne: we may be discussing that here already...13:15
smcginniseharney: Yeah, just approved that.13:15
jungleboyjeharney:  Where is the os-brick refert?13:15
jungleboyjNM.13:15
eharneyhttps://review.openstack.org/#/c/480224/13:15
smcginnisErr, actually that's the one I rechecked. Hopefully it's ahppier now.13:15
e0neeharney: not sure that it's related. I saw errors during backup-related tests with permission denied from swift13:15
smcginnis*happier13:15
eharneysmcginnis: it probably needs to Depends-On the Cinder patch13:15
smcginnisAh13:16
eharneysmcginnis: i'll go ahead and do that13:16
ericyoungavishay: still working on it13:16
smcginniseharney: Thanks13:16
openstackgerritEric Harney proposed openstack/os-brick master: Revert "Don't use ignoreskipactivation for thin LVM"  https://review.openstack.org/48022413:16
*** catintheroof has joined #openstack-cinder13:18
eharneye0ne: i'm not familiar with this issue13:19
jungleboyjFeel bad that patch got through but I am glad the gate caught it.13:19
e0neeharney: something like http://logs.openstack.org/18/446518/9/check/gate-tempest-dsvm-full-lio-ubuntu-xenial/fccf596/logs/screen-c-bak.txt.gz?level=TRACE#_Jul_04_19_34_14_71782713:19
e0neeharney: the issue was introduced with barbican in devstack:(13:19
eharneye0ne: smcginnis: seeing that the LIO job is the only Cinder job testing Barbican (and thin LVM), we probably need to fix it up and get it to voting status13:21
smcginnis+10013:21
e0neI agree13:21
e0nethe issue is that barbican creates user with 'creator' role but swift accept only 'admin' and 'Member' roles13:22
e0neI didn't find how to configure tempest to use it without any dirty hacks13:22
*** mriedem has joined #openstack-cinder13:22
eharneysounds like maybe something the barbican devstack plugin needs to do13:23
*** felipemonteiro_ has joined #openstack-cinder13:23
jungleboyjeharney:  ++13:23
*** links has quit IRC13:24
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Fix wrong usage of rbd.list_snaps return value  https://review.openstack.org/45578613:24
*** felipemonteiro__ has joined #openstack-cinder13:24
scottdaAnyone know why I get this error with a cinder command when using 'demo' or 'admin' creds in devstack?13:25
scottdaERROR: Unable to determine the Keystone version to authenticate with using the given auth_url.13:25
scottdahttps://www.irccloud.com/pastebin/290qQmpJ/13:26
smcginnisscottda: I haven't hit that. Yet.13:27
*** felipemonteiro_ has quit IRC13:28
scottdalikely something on my end. But it's a fresh clone from July 3rd. I just sourced the creds and tried to create a volume...13:28
*** xyang1 has joined #openstack-cinder13:28
ildikovsmcginnis: eharney: hi :)13:28
ildikovsmcginnis: eharney: with jgriffith we bumped into a probably policy issue with implementing the new attach/detach API flow in Nova and I was wondering whether you might be able to help13:28
smcginnisildikov: Hey13:28
smcginnisscottda: I'll try to spin up a fresh instance.13:29
ildikovsmcginnis: eharney: the issue is this one http://logs.openstack.org/85/330285/85/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/eaae918/logs/screen-n-api.txt.gz?level=ERROR13:29
ildikovsmcginnis: eharney: the use case is detaching a volume from a shelved offloaded server13:29
scottdasmcginnis: Thanks. But I think I'm just borked, so don't rush to do it. I'll just re-stack.13:30
smcginnisildikov: But it works fine for non-shelved instances?13:30
eharneyis that call to Cinder done with user creds or service creds in that case?13:30
ildikovsmcginnis weirdly everything else seems to work according to tempest...13:30
smcginniseharney: Oh, good though.13:31
ildikovsmcginnis: test_detach_volume_shelved_or_offload_server is the only test now that fails which is related to our changes as far as I saw the logs from the last run13:32
*** gkadam has quit IRC13:33
*** lhx_ has joined #openstack-cinder13:34
ildikovsmcginnis: we tried to use the elevated context in Nova when issuing the attachment_delete call, but nothing has changed13:35
ildikovsmcginnis: personally I don't have much knowledge about how policy enforcement works and therefore I don't have too many ideas on where to start to figure this one out13:36
smcginnisildikov: Is that in nova/compute/manager?13:36
smcginnisildikov: Did you see eharney's question? ^13:36
smcginnisildikov: Policy enforcement shouldn't be any different, so I would think it would have to be the way it's getting called from nova.13:37
ildikovit's the same as all the other calls13:37
ildikoveharney: how do I figure out the answer to your question? :)13:37
* eharney drinks more coffee13:37
eharneyumm13:38
smcginnisbrb - network is being wonky. Think I need to turn it off and back on again.13:38
eharneyi think the cinder API log for that call should show the user id, can see if it's different from the other user ids in similar tests?13:38
ildikoveharney: ok, I'll look into that13:38
tommylikehuscottda: many reasons could lead to that problem. Add the debug option would be a help?13:43
*** zhurong has joined #openstack-cinder13:45
ildikoveharney: the Auth-Token and Servic-Token look the same for both create and delete13:48
ildikoveharney: on the Nova side logs at least, but I would guess it didn't get changed on the wire...13:48
eharneyildikov: ok13:49
smcginnisildikov: Anything in the cinder logs?13:49
smcginnisNot that I expect it to tell us much, but maybe at least give a clue how/if this is going down a different code path.13:50
ildikovsmcginnis: not much in the API logs13:51
ildikovsmcginnis: it just tells that HTTP 403: Fault: User does not have admin privileges13:52
ildikovsmcginnis: where else should I look?13:52
*** e0ne has quit IRC13:54
ildikovsmcginnis: http://logs.openstack.org/85/330285/85/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/eaae918/logs/screen-c-api.txt.gz#_Jul_04_22_05_01_50322413:54
*** e0ne has joined #openstack-cinder13:55
scottdatommylikehu: Thanks. Keystone is not starting. I'll just re-spin Devstack and see if it is a real issue, but probably just do to my workstation reboot.13:59
*** jwcroppe has quit IRC14:01
*** jwcroppe has joined #openstack-cinder14:02
smcginnisildikov, jgriffith: Is this an issue? https://github.com/openstack/cinder/blob/c22231474659203ceb42594f9a846aa9e6aaee73/cinder/volume/api.py#L9514:03
*** winston-d_ has quit IRC14:03
smcginnishttps://github.com/openstack/cinder/blob/c22231474659203ceb42594f9a846aa9e6aaee73/cinder/volume/api.py#L198114:04
openstackgerritIsaac Beckman proposed openstack/cinder master: XIV\A9000: Added replication group support  https://review.openstack.org/46873814:04
ildikovsmcginnis: it's attachment_delete that fails14:05
ildikovsmcginnis: and even that fails in only one case14:05
ildikovsmcginnis: and the new flow should run in every test :S14:05
smcginnisildikov: Hmm... and that doesn't look any different.14:05
smcginnisildikov: Something must be different in how that is called. We might need to have more debug logging in there to find the difference.14:06
*** jwcroppe has quit IRC14:06
smcginnisildikov: It's rule:admin_or_owner, so I really don't see how this is different unless Nova is passing down a different context.14:07
*** wxy- has joined #openstack-cinder14:08
ildikovsmcginnis: can I get more debug logs in tempest?14:08
*** takedakn has joined #openstack-cinder14:09
smcginnisildikov: Not sure. You might have to instrument it yourself. :/14:09
ildikovsmcginnis: at least locally I mean14:09
*** zhurong has quit IRC14:09
smcginnisildikov: It does debug by default, but looking through that code path we don't really capture much.14:09
ildikovsmcginnis: which side you mean?14:10
*** wanghao has joined #openstack-cinder14:10
smcginnisildikov: Maybe both, but you could at least add some logging in Cinder to print out what we're receiving.14:10
ildikovsmcginnis: ok, I'll try that14:12
smcginnisildikov: Hopefully that gives us a clue.14:13
ildikovsmcginnis: I know jgriffith ran all the commands as admin and everything worked, not sure whether he tried as non-admin since14:13
smcginnisJust always run as admin. Problem solved. :)14:14
ildikovsmcginnis: when I get there I'll see whether it fails manually too, might be easier to debug if it does14:14
ildikovsmcginnis: lol :)14:14
smcginnisildikov: Yeah, if you can manually reproduce the failure, that should make it a lot easier to debug.14:15
ildikovsmcginnis: I'll report back :)14:16
ildikovsmcginnis: if you have any idea in the meantime on what it can be, don't hesitate to share :)14:16
*** jwcroppe has joined #openstack-cinder14:16
smcginnisildikov: Will do!14:17
ildikovTnx!14:17
*** wxy- has quit IRC14:18
*** wxy- has joined #openstack-cinder14:18
*** zhangguoqing has quit IRC14:24
*** zhangguoqing has joined #openstack-cinder14:25
*** yangyape_ has joined #openstack-cinder14:29
*** zhangguoqing has quit IRC14:29
*** felipemonteiro__ has quit IRC14:30
*** chlong_ has joined #openstack-cinder14:31
*** yangyapeng has quit IRC14:32
*** crushil_ has joined #openstack-cinder14:43
openstackgerritMerged openstack/cinder master: Revert "Don't use ignoreskipactivation for thin LVM"  https://review.openstack.org/48022614:43
*** takedakn has quit IRC14:52
jgriffithnon-admin isn't working on delete14:56
jgriffithand I haven't quite figured out why, kinda need some help on that from somebody that understands policy better than I do14:56
jgriffithone thing I *think* might be a bit of a problem.  The attachment table does not have a project_id column :(14:56
jgriffithso I think that might be part of the issue (or the entire issue)14:57
smcginnisjgriffith: Hmm, so since it can't find its project, it fails verifying the owner maybe?14:57
jgriffithsmcginnis that's my current theory14:57
* smcginnis doesn't know enough about policy enforcement either14:57
*** mmehan has joined #openstack-cinder14:58
*** crushil_ has quit IRC14:58
*** crushil_ has joined #openstack-cinder14:59
*** lucasxu has quit IRC15:00
*** markstur has joined #openstack-cinder15:01
*** cebruns__ is now known as cebruns15:03
*** lbragstad has joined #openstack-cinder15:05
*** belmoreira has quit IRC15:06
*** bkopilov has quit IRC15:14
jgriffithsmcginnis yeah, and I'm not quite sure about that because there is an old api call there for get-all-by-project which I assume may have worked :)15:21
jgriffithso maybe it just gets it via the relationship to the volume.. not sure15:21
*** rcernin has quit IRC15:21
jgriffithpolicy file and everything *looks* right, so I'm kinda stumped15:21
*** chlong has joined #openstack-cinder15:23
smcginnisjgriffith: Really weird that it works everywhere else except under this condition too.15:26
*** chlong has quit IRC15:26
jgriffithyeah, I'm really not sure what to make of this15:26
jgriffithhaha!15:27
jgriffithI add the column, do the migration...15:27
jgriffithnow I can't create attachments "project_id is not nullable"15:27
jgriffithLOL15:27
jgriffithWTF15:27
smcginnisjgriffith: Only thing I can think is to just shotgun LOG.debug calls all throughout the process.15:27
jgriffithsmcginnis well, it's the policy check in volume-api15:27
jgriffithOh.. yeah, maybe put log statements in the policy check itself15:28
smcginnisjgriffith: Hack oslo.policy?15:28
jgriffithbut that goes through the oslo spaghetti15:28
jgriffith:)15:28
smcginnisMmmm, spaghetti.15:28
jgriffithblack-holes full of magic and indirection15:28
smcginnis;)15:28
jgriffithtoo early (and hot) for spaghetti right now15:28
jgriffithmaybe tomorrow15:28
*** Nel1x has joined #openstack-cinder15:29
smcginnisjgriffith: Are you trying to one up MN again?15:29
smcginnisHigh of 90 here today.15:29
*** walshh has joined #openstack-cinder15:31
jgriffith97 here :(15:32
jgriffithanything over 80 I'm pretty miserable15:33
smcginnisSheesh.15:33
jgriffithalthough you have humidity!!!!15:33
smcginnisMe too.15:33
jgriffithso that would be like 120 here :)15:33
smcginnisFor sure.15:33
*** lucasxu has joined #openstack-cinder15:35
*** crushil_ has quit IRC15:36
*** psachin has quit IRC15:37
*** bkopilov has joined #openstack-cinder15:40
jgriffithSo it is infact only the reserved case that blocks the regular user15:43
smcginnis?15:43
jgriffithso there's *something* in all the crazy calls that update etc that toggle something to allow access15:43
* jgriffith should've just created a whole new table and db apis for this :(15:43
ericyoungwe have a patch for nova, to use ScaleIO for ephemeral storage. It's passed CI and mdbooth had offered to review it for us.15:44
ericyoungWe haven't been able to get ahold of him, can someone else take a look for us at https://review.openstack.org/#/c/407440/15:45
jgriffithoh.. I think I found my problem15:47
jgriffithderp15:47
ericyoungdang... wrong channel. Ignore me today15:48
smcginnisjgriffith: It always ends up being something minor.15:48
jgriffithYeah :(15:49
jgriffithSo I tried to be fancy and take a short cut15:49
jgriffithjust call the objects destroy method15:49
smcginnisjgriffith: Usually the path to trouble. :)15:49
jgriffiththat's no bueano15:49
jgriffithhehe..15:49
* jgriffith will never learn15:49
*** Apoorva has joined #openstack-cinder15:50
*** felipemonteiro_ has joined #openstack-cinder15:55
jgriffithsmcginnis ildikov ok.. fixed, let me open a bug and I'll post a patch15:57
*** chlong_ has quit IRC15:58
smcginnisjgriffith: Awesone16:05
smcginnisGah! *awesome16:05
openstackgerritJohn Griffith proposed openstack/cinder master: Fix attachment_delete for reserved case  https://review.openstack.org/48065716:08
*** wanghao has quit IRC16:10
*** wanghao has joined #openstack-cinder16:11
*** wanghao has quit IRC16:11
*** wanghao has joined #openstack-cinder16:11
*** chlong_ has joined #openstack-cinder16:15
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Remove periodic_tasks method from base manager  https://review.openstack.org/48065916:19
*** adisky__ has quit IRC16:22
*** avishay has quit IRC16:22
*** markvoelker has quit IRC16:24
openstackgerritJohn Griffith proposed openstack/cinder master: Fix attachment_delete for reserved case  https://review.openstack.org/48065716:26
*** lucasxu has quit IRC16:27
*** markvoelker has joined #openstack-cinder16:30
*** abhitechie has joined #openstack-cinder16:31
ildikovsmcginnis: BTW, hopefully an easy one: https://review.openstack.org/#/c/480165/ plz :)16:38
smcginnisildikov: Easy peasy. Thanks16:39
ildikovsmcginnis: awesome, thanks :)16:39
*** e0ne has quit IRC16:46
*** lhx_ has quit IRC16:49
*** cebruns has quit IRC16:50
*** cebruns has joined #openstack-cinder16:54
*** itlinux has joined #openstack-cinder16:54
*** eharney has quit IRC17:02
*** iceyao has quit IRC17:03
*** eharney has joined #openstack-cinder17:04
*** Poornima has joined #openstack-cinder17:09
*** Poornima has quit IRC17:09
*** Poornima has joined #openstack-cinder17:09
openstackgerritMerged openstack/os-brick master: Revert "Don't use ignoreskipactivation for thin LVM"  https://review.openstack.org/48022417:10
*** chlong_ has quit IRC17:21
*** abhitechie has quit IRC17:22
openstackgerritEric Harney proposed openstack/cinder master: LVM: Don't initialize VG with lvm_type='auto'  https://review.openstack.org/47929517:23
*** iceyao has joined #openstack-cinder17:30
*** chlong_ has joined #openstack-cinder17:34
*** iceyao has quit IRC17:34
*** ChubYann has joined #openstack-cinder17:40
*** dhinesh has joined #openstack-cinder17:44
*** gkadam has joined #openstack-cinder17:48
*** Guest39045 is now known as med_18:05
*** med_ has joined #openstack-cinder18:05
*** e0ne has joined #openstack-cinder18:15
*** gkadam has quit IRC18:19
*** cebruns has quit IRC18:21
*** dave-mccowan has joined #openstack-cinder18:21
*** cebruns has joined #openstack-cinder18:26
harlowjaquestion for folks here!18:28
harlowjain our standup, we were talking about how admins/operators have awareness into how much capacity say is left (used, taken...) in a given cinder backend18:29
harlowjaand there seems to be a few ways to do this right now, some via this volume usage audit, som specs that mention some new apis?, some via others...18:29
harlowjasoooo, was wondering what people normally do :-P18:30
harlowjahttps://review.openstack.org/#/c/206923 seems to shove info into ceilometer?18:30
harlowja(and ya, i know its not always obvious what the capacity is, espeically with thin provisioning)18:31
openstackgerritMerged openstack/os-brick master: Fix ceph incremental backup fail  https://review.openstack.org/47650318:34
*** iceyao has joined #openstack-cinder18:35
*** iceyao has quit IRC18:39
e0neharlowja: hi. you can use pools api https://developer.openstack.org/api-ref/block-storage/v3/?expanded=show-all-back-end-capabilities-detail,list-all-back-end-storage-pools-detail18:42
harlowjakk18:43
*** e0ne has quit IRC18:44
*** rcernin has joined #openstack-cinder18:47
jgriffithsmcginnis and all... thanks to ildikov fixing a bunch of things up over the week-end, we have our first clean run on Nova for the new attach api's:  https://review.openstack.org/#/c/330285/18:59
smcginnisjgriffith: Sweet!19:00
ildikovand special thanks to jgriffith to write it up and let me fix just a few things :)19:01
*** iceyao has joined #openstack-cinder19:02
openstackgerritMerged openstack/cinder master: Add 'reserved' status to api-ref  https://review.openstack.org/48016519:04
*** iceyao has quit IRC19:06
*** chlong_ has quit IRC19:14
*** dalgaaf has quit IRC19:17
*** xyang1 has quit IRC19:20
*** chlong_ has joined #openstack-cinder19:30
*** markstur has quit IRC19:30
*** markstur has joined #openstack-cinder19:38
*** chlong_ has quit IRC19:55
*** tesseract has quit IRC19:56
openstackgerritxing-yang proposed openstack/python-cinderclient master: Support revert to snapshot in client  https://review.openstack.org/46490320:02
openstackgerritxing-yang proposed openstack/python-cinderclient master: Tiramisu: replication group support  https://review.openstack.org/35222920:02
*** rcernin has quit IRC20:05
*** Nel1x has quit IRC20:07
*** Nel1x has joined #openstack-cinder20:08
*** xyang1 has joined #openstack-cinder20:11
*** Apoorva has quit IRC20:11
*** Apoorva has joined #openstack-cinder20:12
*** dalgaaf has joined #openstack-cinder20:12
*** Apoorva has quit IRC20:13
*** Apoorva has joined #openstack-cinder20:14
*** rcernin has joined #openstack-cinder20:14
mriedemsmcginnis: just remembered the required nova version for this https://review.openstack.org/#/c/454287/20:24
mriedemgood thing i added the depends-on :)20:24
smcginnismriedem: Doh20:30
mriedemi'm working on the tempest test, which would have caught this, but made me think about it20:30
smcginnismriedem: Saw that. Looks like the tempest test won't be so easy.20:31
mriedemwill see what the qa people let me get away with20:31
openstackgerritMerged openstack/cinder master: Fix attachment_delete for reserved case  https://review.openstack.org/48065720:32
*** chlong_ has joined #openstack-cinder20:44
mriedemrandom question, but don't some cinder backends require the block size to be in some multiple?20:46
mriedemlike scaleio requires block size in increments of 8GB?20:46
mriedemer multiples of 8GB?20:46
mriedembecause if so, tempest won't work for those https://github.com/openstack/tempest/blob/master/tempest/api/volume/test_volumes_extend.py#L3220:47
eharneyscaleio does, i don't know of any others20:47
mriedemxyang1: ^20:47
smcginnismriedem: It's either ScaleIO or VMAX, just one.20:47
xyang1mriedem: hi20:47
xyang1mriedem: ScaleIO requires multiples of 8 GB20:48
mriedemok so those tempest volume extend tests won't work for scaleio20:48
mriedemmaybe scaleio doesn't support volume extend yet20:49
mriedem?20:49
xyang1mriedem: it does support20:49
mriedemso the scaleio ci doesn't run them?20:49
*** liverpooler has quit IRC20:49
xyang1mriedem: I think in the code, it increase the size by 820:49
xyang1Let me check on that test20:50
mriedemso if i have an 8gb volume, and the user asks to extend it to 9gb, the backend automatically bumps that up to 16gb?20:50
jgriffithmriedem :)20:50
jgriffitheven better I believe it still reports that it's only 9gb... unless they changed that20:50
mriedemfun20:51
mriedemwell, the ci blew up in http://publiclogs.emc.com/vipr_ostack/EMC_CoprHD_ScaleIO/1243/console.html before tests ran it looks like20:51
mriedemso can't really verify20:51
mriedemno i guess it did run tests20:51
mriedemtempest run --regex ^(?=.*volume)(?!.*extend)(?!.*test_force_detach_volume).* --concurrency=120:52
mriedemheh,20:52
mriedemso it explicitly does not run the volume extend tests20:52
mriedemxyang1: ^20:52
xyang1mriedem: is this from coprhd CI?20:53
mriedemEMC_CoprHD_ScaleIO20:53
xyang1mriedem:do you have the gerrit review URL?20:55
mriedemhttps://review.openstack.org/#/c/454287/20:55
xyang1mriedem: thanks20:55
mriedemi'll just continue writing this tempest test by doing volume['size'] * 220:56
xyang1mriedem:I'll check with them about coprhd scallions CI.  The ScaleIO CI does run extend volume tests.  We'll run it on that patch20:58
mriedemok thanks20:59
*** swamireddy has joined #openstack-cinder21:00
*** jmlowe has quit IRC21:01
*** dalgaaf has quit IRC21:16
*** chlong_ has quit IRC21:18
ericyoungmriedem xyang1: ScaleIO CI does NOT run the extend tests. I just checked the last to patches I submitted. They were excluded both times.21:20
ericyoungI am going to enable them in my environmment and see what happens21:20
mriedemok, i expect them to blow up given the size + 1 thing in tempest noted above21:20
ericyoungunderstood. I want to see them run to see what we/I can do about making them pass :)21:21
xyang1ericyoung: ScaleIO driver does support it though.  It rounds it up to 8GB multiples21:22
ericyoungyep, but without tempest running the tests, I'd like a little more confidence that they work. After all, people call me when it doesn't21:23
mriedemwhat's your number btw?21:23
smcginnisericyoung: Good motivator. ;)21:23
ericyoungmriedem: 1800-worksforme21:23
mriedem867-5309?21:24
ericyoungnah, that's Jenny21:24
mriedemha21:24
mriedemglad you got the reference21:24
ericyoungmanaged music store to get through college21:24
*** dalgaaf has joined #openstack-cinder21:26
*** abishop has quit IRC21:30
*** edmondsw has quit IRC21:31
*** edmondsw has joined #openstack-cinder21:33
*** edmondsw has quit IRC21:38
*** eharney has quit IRC21:39
openstackgerritxing-yang proposed openstack/cinder master: Tiramisu: Add groups param to failover_host  https://review.openstack.org/47768821:47
*** iceyao has joined #openstack-cinder21:48
*** iceyao has quit IRC21:53
*** felipemonteiro_ has quit IRC22:00
*** edmondsw has joined #openstack-cinder22:11
*** edmondsw has quit IRC22:16
*** markstur has quit IRC22:21
*** wanghao has quit IRC22:24
*** dhinesh has quit IRC22:36
*** markstur has joined #openstack-cinder22:41
*** jwcroppe has quit IRC22:43
*** jwcroppe has joined #openstack-cinder22:43
*** rcernin has quit IRC22:45
*** markstur has quit IRC22:46
*** jwcroppe has quit IRC22:47
*** iceyao has joined #openstack-cinder22:50
*** itlinux has quit IRC22:50
*** iceyao has quit IRC22:54
*** jwcroppe has joined #openstack-cinder23:00
*** jwcroppe has quit IRC23:01
*** jwcroppe has joined #openstack-cinder23:01
*** xyang1 has quit IRC23:05
*** jwcroppe has quit IRC23:06
*** iceyao has joined #openstack-cinder23:11
*** iceyao has quit IRC23:15
*** _pewp_ has quit IRC23:22
*** markstur has joined #openstack-cinder23:23
*** hemna is now known as assbutt23:25
*** _pewp_ has joined #openstack-cinder23:25
*** assbutt is now known as buttass23:26
*** buttass is now known as butt23:26
*** markstur has quit IRC23:28
*** iceyao has joined #openstack-cinder23:31
*** jwcroppe has joined #openstack-cinder23:31
ericyoungsmcginnis xyang1 mriedem: I enabled the tempest extend tests and they all passed with ScaleIO. I'll ask the team that maintains CI to enable them as well.23:33
*** iceyao has quit IRC23:36
*** jwcroppe has quit IRC23:36
*** wxy- has quit IRC23:37
*** hoonetorg has quit IRC23:47
*** catintheroof has quit IRC23:47
*** dhinesh has joined #openstack-cinder23:48
*** iceyao has joined #openstack-cinder23:52
*** wxy- has joined #openstack-cinder23:52
*** Julien-z_ has joined #openstack-cinder23:53
*** dhinesh has quit IRC23:53
*** iceyao has quit IRC23:56
*** Julien-z_ has quit IRC23:56
*** Julien-zte has quit IRC23:57

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