Tuesday, 2015-09-08

*** takedakn has joined #openstack-cinder00:00
*** haomaiwang has quit IRC00:01
*** zhongjun has joined #openstack-cinder00:01
*** haomaiwang has joined #openstack-cinder00:01
*** dimsum__ has quit IRC00:03
*** haomaiwang has quit IRC00:36
*** esker has quit IRC00:42
*** zhongjun has quit IRC00:45
*** takedakn has quit IRC00:46
*** takedakn has joined #openstack-cinder00:46
*** takedakn has quit IRC00:47
*** dimsum__ has joined #openstack-cinder00:48
*** esker has joined #openstack-cinder00:50
*** lixiaoy1 has joined #openstack-cinder01:00
*** lixiaoy11 has joined #openstack-cinder01:01
*** lixiaoy1 has quit IRC01:04
*** baojg has joined #openstack-cinder01:07
*** rick-chen has joined #openstack-cinder01:10
rick-chenanteaya: Hi, do you know when Miki return?01:11
rick-chenHi, do you know when Mike return?01:12
*** erhudy1 has quit IRC01:12
rick-chenthingee: HI Mike, Do you return?01:13
*** zhangjn has joined #openstack-cinder01:19
*** Lee1092 has joined #openstack-cinder01:28
openstackgerritwanghao proposed openstack/cinder: Add UniqueConstraint for quota usages table  https://review.openstack.org/21746501:32
*** esker has quit IRC01:32
*** esker has joined #openstack-cinder01:33
*** haomaiwang has joined #openstack-cinder01:35
*** zhenguo has joined #openstack-cinder01:35
*** dimsum__ has quit IRC01:40
*** chenying has joined #openstack-cinder01:43
*** david-ly_ has joined #openstack-cinder01:51
*** krtaylor has quit IRC01:52
*** david-lyle has quit IRC01:54
*** esker has quit IRC01:57
*** haomaiwang has quit IRC02:01
*** haomaiwang has joined #openstack-cinder02:01
*** krtaylor has joined #openstack-cinder02:05
*** rick-chen has quit IRC02:05
*** esker has joined #openstack-cinder02:06
openstackgerritThang Pham proposed openstack/cinder: Conversion to volume object  https://review.openstack.org/17705402:10
*** zxyf has joined #openstack-cinder02:15
*** esker has quit IRC02:31
*** davechen has joined #openstack-cinder02:31
openstackgerritPeter Wang proposed openstack/os-brick: Use 'device' instead of 'volume_path'  https://review.openstack.org/22090202:32
*** zxyf has quit IRC02:33
*** zxyf has joined #openstack-cinder02:33
*** esker has joined #openstack-cinder02:38
*** dimsum__ has joined #openstack-cinder02:40
openstackgerritThang Pham proposed openstack/cinder: WIP: Make RPC and objects backwards compatible  https://review.openstack.org/21494302:44
*** dimsum__ has quit IRC02:46
lixiaoy11Thang?02:50
lixiaoy11thangp: here?02:51
*** haomaiwang has quit IRC03:01
*** haomaiwang has joined #openstack-cinder03:01
*** esker has quit IRC03:05
*** esker has joined #openstack-cinder03:10
openstackgerritLisaLi proposed openstack/cinder: Fix KeyError  https://review.openstack.org/21956603:22
*** esker has quit IRC03:26
*** baojg has quit IRC03:31
*** esker has joined #openstack-cinder03:35
*** zxyf has quit IRC03:42
*** zxyf has joined #openstack-cinder03:43
*** julim has quit IRC03:47
*** zxyf has quit IRC03:50
*** zxyf has joined #openstack-cinder03:50
*** zhangjn_ has joined #openstack-cinder03:52
*** esker has quit IRC03:54
*** zhangjn has quit IRC03:56
*** jamielennox is now known as jamielennox|away03:56
*** haomaiwang has quit IRC04:01
*** esker has joined #openstack-cinder04:01
*** haomaiwang has joined #openstack-cinder04:01
*** baojg has joined #openstack-cinder04:04
openstackgerritCedric Zhuang proposed openstack/cinder: Add retype logic in manage_existing for VNX  https://review.openstack.org/20917604:22
*** chenying has quit IRC04:22
*** resker has joined #openstack-cinder04:23
*** esker has quit IRC04:24
*** zhangjn_ has quit IRC04:27
*** jseiler_ has joined #openstack-cinder04:32
*** jseiler has quit IRC04:35
*** zxyf has quit IRC04:37
*** zxyf has joined #openstack-cinder04:38
*** zxyf has quit IRC04:42
*** zxyf has joined #openstack-cinder04:43
*** resker has quit IRC04:44
*** esker has joined #openstack-cinder04:48
*** zxyf has quit IRC04:59
*** haomaiwang has quit IRC05:01
*** haomaiwang has joined #openstack-cinder05:01
*** esker has quit IRC05:16
*** esker has joined #openstack-cinder05:22
openstackgerritVincent Hou proposed openstack/cinder: Adds migration abortion for available volumes  https://review.openstack.org/20775405:25
*** zxyf has joined #openstack-cinder05:31
openstackgerritPeter Wang proposed openstack/cinder: Update update_migrated_volume in VNX driver  https://review.openstack.org/21503605:32
*** BharatK has joined #openstack-cinder05:39
*** kjelly has quit IRC05:39
*** vgridnev has joined #openstack-cinder05:45
*** esker has quit IRC05:45
*** bkopilov has joined #openstack-cinder05:49
*** esker has joined #openstack-cinder05:51
*** chenying has joined #openstack-cinder06:00
*** haomaiwang has quit IRC06:01
*** alexpilotti_ has joined #openstack-cinder06:01
*** haomaiwang has joined #openstack-cinder06:01
*** lpetrut has joined #openstack-cinder06:05
*** vlaza has joined #openstack-cinder06:12
*** sgotliv_ has quit IRC06:14
*** yrabl has joined #openstack-cinder06:14
*** yrabl is now known as yrabl_wfh06:18
*** yrabl_wfh has quit IRC06:18
*** yrabl has joined #openstack-cinder06:19
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex  https://review.openstack.org/21874106:20
*** lpetrut has quit IRC06:22
*** esker has quit IRC06:25
*** esker has joined #openstack-cinder06:27
*** salv-orlando has joined #openstack-cinder06:33
*** baojg has quit IRC06:36
*** esker has quit IRC06:45
nikeshmhi06:48
nikeshmi have a doubt in https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/rbd.py#L74106:49
nikeshmis this correct?06:49
nikeshmvolume.create_snap(snap)06:49
*** esker has joined #openstack-cinder06:50
nikeshmshould not be it volume.volume.create_snap(snap)06:50
nikeshmdid any check rbd recently06:52
*** lixiaoy11 has quit IRC06:54
*** lixiaoy1 has joined #openstack-cinder06:54
lixiaoy1nikeshm: I had do changes in rbd.  https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/rbd.py#L73906:56
lixiaoy1nikeshm: here volume is RBDVolumeProxy06:57
lixiaoy1nikeshm: it call driver.rbd.Image.create_snap which is from librbd06:58
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-cinder07:01
*** anshul has joined #openstack-cinder07:02
openstackgerritVincent Hou proposed openstack/cinder: Adds migration abortion for available volumes  https://review.openstack.org/20775407:05
*** zhongjun has joined #openstack-cinder07:08
*** yrabl has quit IRC07:09
*** yrabl has joined #openstack-cinder07:09
*** lixiaoy11 has joined #openstack-cinder07:09
*** lixiaoy11 has left #openstack-cinder07:09
*** lixiaoy11 has joined #openstack-cinder07:10
nikeshmlixiaoy1 : hi, with RBDVolumeProxy(self, snapshot['volume_name']) as volume, here volume is RBDVolumeProxy object and in definition of this class, self.volume = driver.rbd.Image and07:11
*** lixiaoy1 has quit IRC07:11
nikeshmso how can we call (RBDVolumeProxy object).create_snap07:12
nikeshmshould not be it (RBDVolumeProxy object).volume.create_snap07:13
*** fifieldt has joined #openstack-cinder07:13
*** alexschm has joined #openstack-cinder07:13
lixiaoy11nikeshm: you may look at the function __getattr__()07:13
lixiaoy11https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/rbd.py#L24307:14
nikeshmlixiaoy11 : thanks07:14
*** abhijeetm has joined #openstack-cinder07:14
lixiaoy11nikeshm: welcome07:14
*** esker has quit IRC07:17
*** zhangjn has joined #openstack-cinder07:19
*** esker has joined #openstack-cinder07:23
*** zhangjn_ has joined #openstack-cinder07:30
*** deepakcs has joined #openstack-cinder07:33
*** zhangjn has quit IRC07:34
*** zhangjn_ has quit IRC07:35
*** shyama has joined #openstack-cinder07:48
*** sgotliv_ has joined #openstack-cinder07:49
*** bluex-pl has joined #openstack-cinder07:52
*** esker has quit IRC07:52
*** chlong has quit IRC07:56
*** esker has joined #openstack-cinder07:59
*** zxyf has quit IRC08:00
*** haomaiwang has quit IRC08:01
*** haomaiwang has joined #openstack-cinder08:01
*** baojg has joined #openstack-cinder08:01
*** lpetrut has joined #openstack-cinder08:01
*** sgotliv_ has quit IRC08:03
wanghaohi~, Need cores' help to review this bug patch: https://review.openstack.org/#/c/203530/. It's a performance bug.08:06
*** ronis has joined #openstack-cinder08:06
*** ociuhandu has quit IRC08:12
*** vgridnev has quit IRC08:12
openstackgerritXinXiaohui proposed openstack/cinder: Calculate virtual free capacity and notify  https://review.openstack.org/20692308:15
*** baojg has quit IRC08:18
*** sgotliv_ has joined #openstack-cinder08:23
*** esker has quit IRC08:24
*** esker has joined #openstack-cinder08:29
*** jistr has joined #openstack-cinder08:30
*** baojg has joined #openstack-cinder08:46
openstackgerritLisaLi proposed openstack/cinder: Data transfer between encrypted volumes and images  https://review.openstack.org/21656708:48
*** aix has quit IRC08:53
*** esker has quit IRC08:54
*** vgridnev has joined #openstack-cinder08:58
*** lixiaoy11 has quit IRC09:00
*** haomaiwang has quit IRC09:01
*** haomaiwa_ has joined #openstack-cinder09:01
*** jordanP has joined #openstack-cinder09:01
*** esker has joined #openstack-cinder09:05
*** sgotliv_ has quit IRC09:10
*** e0ne has joined #openstack-cinder09:16
*** aix has joined #openstack-cinder09:18
*** afazekas_ has joined #openstack-cinder09:22
*** afazekas_ has quit IRC09:22
*** afazekas_ has joined #openstack-cinder09:23
openstackgerritVincent Hou proposed openstack/cinder: Adds migration abortion for available volumes  https://review.openstack.org/20775409:23
*** esker has quit IRC09:25
*** esker has joined #openstack-cinder09:33
*** salv-orlando has quit IRC09:36
*** chenying has quit IRC09:38
*** NightKhaos has quit IRC09:39
*** sgotliv_ has joined #openstack-cinder09:40
*** e0ne has quit IRC09:41
*** NightKhaos has joined #openstack-cinder09:42
*** dimsum__ has joined #openstack-cinder09:46
*** bluex-pl has quit IRC09:48
openstackgerritYuriy Nesenenko proposed openstack/cinder: WIP: Small optimization in Block Device driver  https://review.openstack.org/22056109:48
*** bluex-pl has joined #openstack-cinder09:48
*** bluex-pl has quit IRC09:49
*** bluex-pl has joined #openstack-cinder09:49
*** dims_ has joined #openstack-cinder09:50
*** davechen has left #openstack-cinder09:54
*** dimsum__ has quit IRC09:55
*** baojg has quit IRC09:56
*** baojg has joined #openstack-cinder09:56
*** e0ne has joined #openstack-cinder09:57
*** baojg has quit IRC10:00
*** haomaiwa_ has quit IRC10:01
*** haomaiwang has joined #openstack-cinder10:01
*** IanGovett has joined #openstack-cinder10:02
*** esker has quit IRC10:05
*** ociuhandu has joined #openstack-cinder10:07
*** bluex-pl has quit IRC10:10
*** bluex-pl has joined #openstack-cinder10:10
*** esker has joined #openstack-cinder10:11
*** julim has joined #openstack-cinder10:29
*** julim has quit IRC10:33
*** esker has quit IRC10:33
*** aix has quit IRC10:35
*** aix has joined #openstack-cinder10:36
*** esker has joined #openstack-cinder10:39
*** zhongjun has quit IRC10:43
openstackgerritBharat Kumar Kobagana (BharatK) proposed openstack/cinder: GlusterFS: Fixing 'migrate_volume' attribute error  https://review.openstack.org/22125110:48
*** chenying has joined #openstack-cinder10:53
*** kodokuu has joined #openstack-cinder10:55
*** ronis has quit IRC10:59
*** zhongjun has joined #openstack-cinder10:59
*** salv-orlando has joined #openstack-cinder11:00
*** haomaiwang has quit IRC11:01
*** 92AAB1O7Y has joined #openstack-cinder11:01
kodokuuHi, I update cinder-client 1.1 to 1.2 and now I can't make cinder list11:02
kodokuupossible to have patch note ?11:04
*** sgotliv_ has quit IRC11:05
*** esker has quit IRC11:10
*** kjelly_ has joined #openstack-cinder11:12
*** 92AAB1O7Y has quit IRC11:12
kodokuudhellmann Hi Can you tell me why I have this issue ? http://paste.openstack.org/show/450103/11:14
*** esker has joined #openstack-cinder11:18
*** marcusvrn_ has joined #openstack-cinder11:25
*** timcl has joined #openstack-cinder11:26
e0nekodokuu: hi. could you please share cinder api logs with failure?11:30
*** jaypipes has joined #openstack-cinder11:31
e0nekodokuu: looks like updated cinderclient doesn't pass tenant_id:(11:32
*** zhongjun has quit IRC11:32
kodokuue0ne yes, I downgrade cinderclient and works :)11:34
kodokuu0 log in API because cinder client GET on WRONG URL11:36
*** esker has quit IRC11:36
kodokuuI think the pb is How cinder client make URL for contact API11:36
e0nekodokuu: maybe it was buh #1464160 but I don't sure yet11:38
e0nebug #146416011:38
openstackbug 1464160 in python-cinderclient "ERROR: Bad Request (HTTP 400) with cinder 1.2.1" [Critical,Fix released] https://launchpad.net/bugs/1464160 - Assigned to John Griffith (john-griffith)11:38
kodokuue0ne I use public_endpoint11:39
*** dobson has quit IRC11:39
kodokuuI don't see the fix for this bug11:39
*** esker has joined #openstack-cinder11:45
*** dobson has joined #openstack-cinder11:45
*** jecarey has quit IRC11:56
*** ankit_ag has joined #openstack-cinder11:56
*** bkopilov has quit IRC12:00
*** kodokuu has quit IRC12:01
*** e0ne has quit IRC12:01
*** alexpilotti_ has quit IRC12:03
*** tpsilva has joined #openstack-cinder12:04
*** lkong has joined #openstack-cinder12:04
*** e0ne has joined #openstack-cinder12:04
*** esker has quit IRC12:07
*** amoturi has joined #openstack-cinder12:10
openstackgerritVipin Balachandran proposed openstack/cinder: Enable certificate verification during image copy  https://review.openstack.org/22127212:11
*** pots has joined #openstack-cinder12:12
*** raildo-afk is now known as raildo12:12
*** zhangjn has joined #openstack-cinder12:15
*** zhangjn has quit IRC12:17
*** esker has joined #openstack-cinder12:17
*** scottda has joined #openstack-cinder12:17
*** abhijeetm has quit IRC12:17
*** dalgaaf has joined #openstack-cinder12:18
*** markvoelker has joined #openstack-cinder12:19
*** deepakcs has quit IRC12:20
*** markvoelker has quit IRC12:20
*** markvoelker has joined #openstack-cinder12:20
*** ronis has joined #openstack-cinder12:21
*** chlong has joined #openstack-cinder12:22
*** sgotliv_ has joined #openstack-cinder12:23
*** daemontool_ has quit IRC12:23
*** e0ne has quit IRC12:26
*** salv-orlando has quit IRC12:34
*** esker has quit IRC12:38
*** merooney has joined #openstack-cinder12:38
*** edmondsw has joined #openstack-cinder12:41
*** merooeny_ has joined #openstack-cinder12:41
*** smoriya has quit IRC12:42
*** daemontool_ has joined #openstack-cinder12:43
openstackgerritSzymon Wr√≥blewski proposed openstack/cinder: Support initialization state in Backup Manager  https://review.openstack.org/21672812:44
*** esker has joined #openstack-cinder12:44
*** salv-orlando has joined #openstack-cinder12:44
*** merooney has quit IRC12:44
*** crose has joined #openstack-cinder12:50
*** merooeny_ has quit IRC12:50
*** esker has quit IRC12:52
*** esker has joined #openstack-cinder12:52
*** ankit_ag has quit IRC12:54
*** esker has quit IRC12:54
*** esker has joined #openstack-cinder12:56
*** esker has quit IRC12:56
*** jordanP has quit IRC12:58
*** erhudy has joined #openstack-cinder13:01
*** zhangjn has joined #openstack-cinder13:01
*** eduardo has joined #openstack-cinder13:03
*** bswartz has joined #openstack-cinder13:03
*** pots has left #openstack-cinder13:03
*** vgridnev has quit IRC13:07
*** vgridnev has joined #openstack-cinder13:07
*** e0ne has joined #openstack-cinder13:08
*** porrua has joined #openstack-cinder13:08
*** Yogi1 has joined #openstack-cinder13:09
*** jgregor has joined #openstack-cinder13:10
*** haomaiwang has joined #openstack-cinder13:10
*** Yogi11 has joined #openstack-cinder13:10
*** changbl has quit IRC13:11
*** jecarey has joined #openstack-cinder13:14
*** Yogi1 has quit IRC13:14
*** jgregor has quit IRC13:15
*** gouthamr has joined #openstack-cinder13:15
*** chenying has quit IRC13:16
openstackgerritWilson Liu proposed openstack/cinder: Fix way of get LUN id in Huawei driver  https://review.openstack.org/20974913:16
*** dustins has joined #openstack-cinder13:17
*** zhangjn has quit IRC13:20
*** crose has quit IRC13:20
*** kjelly_ has quit IRC13:20
*** BharatK has quit IRC13:22
*** daemontool_ has quit IRC13:22
*** Yogi11 has left #openstack-cinder13:22
*** breitz has joined #openstack-cinder13:23
*** akerr has joined #openstack-cinder13:24
*** jgregor has joined #openstack-cinder13:24
jgriffithe0ne: So the problem is the version discovery stuff that was aded in 1.213:27
*** lprice has joined #openstack-cinder13:27
e0nejgriffith: thanks for the clarification. I mentioned bug #1464160 earlier. kodokuu ^^13:28
openstackbug 1464160 in python-cinderclient "ERROR: Bad Request (HTTP 400) with cinder 1.2.1" [Critical,Fix released] https://launchpad.net/bugs/1464160 - Assigned to John Griffith (john-griffith)13:28
jgriffithAhh... yep, that looks like the right one13:28
jgriffith:(13:28
*** ronis has quit IRC13:29
e0nejgriffith: I found it in the "revert commit message"13:29
smcginnisjgriffith: I thought we fixed those version discovery issues?13:30
smcginnisOr rather: /we/you/13:30
jgriffithsmcginnis: not *really*, my fix was to revert it13:31
smcginnis:[13:31
amoturijgriffith: could you take another look at this patchset. https://review.openstack.org/#/c/211299/13:34
jgriffithamoturi: seems "ok"13:36
jgriffithamoturi: but if you're using pools don't you want to report them?  Otherwise that will break the scheduler pool assignment for you?13:37
jgriffithamoturi: or are you not exposing them?13:37
jgriffithamoturi: ahh... never mind13:38
jgriffithI think I get it13:38
amoturijgriffith: pools are not really relevant because all the volumes are created in a project and that report better usage data.13:38
*** martyturner has joined #openstack-cinder13:39
jgriffithamoturi: yeah, I kinda get that looking at how your rest call works13:39
jgriffithemphasis on "kinda" :)13:39
amoturijgriffith: :)13:39
jgriffithamoturi: I'm +213:39
*** lprice1 has joined #openstack-cinder13:39
amoturijgriffith: i am also not sure if the 'bracket instead of \' is valid13:39
jgriffithamoturi: well, in some cases it's not13:40
jgriffithamoturi: and I'm not sure about your case there, I was going to ask you to respond/have a look at that13:40
jgriffithamoturi: the \ has fallen out of favor :(13:40
jgriffithamoturi: but there are cases like "with" where the parens don't work13:41
amoturijgriffith: it did pass pep8 :(13:41
jgriffithamoturi: which is why I'm fine with it frankly :)13:41
jgriffithamoturi: until it's automated it's a "guideline/preferrence"13:41
openstackgerritYuriy Nesenenko proposed openstack/cinder: Implement extend_volume method to Block Device driver  https://review.openstack.org/22130313:42
*** lprice has quit IRC13:42
amoturijgriffith: thanks :) . ahh i see.13:43
*** gouthamr has quit IRC13:44
*** gouthamr has joined #openstack-cinder13:44
jgriffithsmcginnis: is the cross dependency tag actually working?13:48
jgriffithsmcginnis: ie https://review.openstack.org/#/c/220830/13:49
jgriffithsmcginnis: passes because it's actually loading the version of devstack?13:49
*** takedakn has joined #openstack-cinder13:49
jgriffithsmcginnis: I didn't think that was possible (node pool and all)13:49
*** gouthamr has quit IRC13:55
*** gouthamr has joined #openstack-cinder13:56
*** r-daneel has joined #openstack-cinder13:57
*** haomaiwang has quit IRC14:01
*** haomaiwang has joined #openstack-cinder14:01
*** jistr is now known as jistr|call14:01
openstackgerritYuriy Nesenenko proposed openstack/cinder: Implement extend_volume method to Block Device driver  https://review.openstack.org/22130314:03
e0nejgriffith: afaik, if depended(parent) patch doesn't have +A, the child one won't be merged14:04
*** eharney has joined #openstack-cinder14:04
jgriffithe0ne: yeah14:04
jgriffithe0ne: I didn't know that worked with devstack :)14:04
smcginnisjgriffith: I think the devstack patch went in last night. I need to check.14:05
jgriffithe0ne: because the devstack nodes are actually "pre-built"14:05
e0nejgriffith: https://review.openstack.org/#/c/194726/14:05
yuriy_n17jgriffith: Thanks a lot for review.14:05
e0nejgriffith: it's a patch for devstack. parent is not approved yet.14:05
smcginnisI got around it on our CI for now by pulling in a Nova patch to explicitly add the necessary os-brick rootwrap cmd.14:05
jgriffithyuriy_n17: thank you for the quick update14:05
e0nejgriffith: calls 'git clone devstack-repo' for every job14:06
e0nes/calls/infra calls14:06
jgriffithe0ne: but that's different I think.. that's a devstack patch depending on another project, not the other way around14:06
jgriffithe0ne: ahh... ok, cool14:06
jgriffithe0ne: so I learn something new already today :)14:06
jgriffithsmcginnis: and FWIW I did look and it hadn't merged yet14:07
smcginnisDidn't land yet: https://review.openstack.org/#/c/207677/14:07
smcginnis;)14:07
jgriffiththe devstack change, thus my comment :)14:07
jgriffithbut I'm a gonna take a chance and "trust" you and e0ne :)14:08
smcginnisjgriffith: Yeah, kind of an odd one with it being devstack.14:08
smcginnisA bit chicken and egg.14:08
e0nejgriffith: http://logs.openstack.org/50/218750/11/check/gate-tempest-dsvm-full/31f5860/logs/devstack-gate-setup-workspace-new.txt.gz#_2015-09-03_06_34_46_565 - clonning devstack14:08
jgriffithe0ne: well, that pretty much settles that :)14:09
smcginnisStill have a bad feeling about needing to copy over os-brick's rootwrap filters though.14:09
smcginnisI think we're going to run in to trouble with deployments missing that necessary step.14:10
*** rick-chen has joined #openstack-cinder14:10
smcginnisI would rather it be a little more automatic, but I guess we'll see.14:10
e0nejgriffith: thanks for +2 for https://review.openstack.org/#/c/221303/. but what about feature freeze?14:10
*** mriedem has joined #openstack-cinder14:12
*** diablo_rojo has joined #openstack-cinder14:15
yuriy_n17jgriffith: Sorry for disturbing. What about to review https://review.openstack.org/#/c/220561/14:16
jgriffithe0ne: that's on master14:17
jgriffithe0ne: the branch is cut14:18
jgriffithe0ne: https://www.dropbox.com/s/vj6osneq7q8b6zd/Screenshot%202015-09-08%2008.18.35.png?dl=014:18
jgriffithno?14:18
e0nejgriffith: hm... I don't see it on github14:18
e0nejgriffith: may be it is not synced yet14:19
e0ne"file is uploading...."14:19
jgriffithe0ne: that link is a screenshot of the tag in github14:20
jgriffith:)14:20
*** jistr|call is now known as jistr14:20
e0nejgriffith: I missed something. will we have a branch from the tag?14:21
jgriffithe0ne: that all has changed a bit and now they do tags instead of branches until stable I believe14:24
jgriffithe0ne: in other words, any release gets a "tag"14:24
jgriffithhmmm14:25
e0nejgriffith: is any wiki or openstack-dev ml thread about it?14:25
jgriffithI'll check with ttx as things have changed since I did this14:25
jgriffithI dunno :)14:25
*** dustins has quit IRC14:25
rick-chenjgriffith: HI John, Does Mike return?14:25
smcginnisI seem to remember an ML thread about it.14:25
e0nejgriffith: if we'll merge something for Mitaka now, how will we create stable branch for L?14:25
*** zhangjn has joined #openstack-cinder14:26
*** zhangjn has quit IRC14:26
*** dims_ has quit IRC14:26
*** dimsum__ has joined #openstack-cinder14:27
*** bkopilov has joined #openstack-cinder14:27
smcginnise0ne: Doesn't appear this has been updated: https://wiki.openstack.org/wiki/Branch_Model14:27
*** vlaza has quit IRC14:29
jgriffithe0ne: you are correct14:29
jgriffithsmcginnis: e0ne Mast is currently still feature freeze until an rc branch is cut14:29
jgriffithI'll go fix up my screw ups14:30
tbarronwhen are our rc targets (or is that a meeting topic) ?14:30
e0nejgriffith: thanks for clarification. it's good to be on the same page14:30
openstackgerritDerrick Wippler proposed openstack/cinder: Hacking log format arg check  https://review.openstack.org/22024614:30
openstackgerritIvan Kolodyazhny proposed openstack/python-cinderclient: Remove unused code from cinderclient.utils module  https://review.openstack.org/22132314:32
e0netbarron: here is a schedule https://wiki.openstack.org/wiki/Liberty_Release_Schedule14:32
*** alexschm has quit IRC14:33
*** edtubill has joined #openstack-cinder14:33
tbarrone0ne: ty!  so rc0, rc1, etc. may vary within the window shown, right? (trying to map  https://wiki.openstack.org/wiki/Branch_Model onto the schedule you supplied).14:34
*** raildo is now known as raildo-afk14:34
jgriffithsmcginnis: e0ne DuncanT hemna_ geguileo eharney winston-d so just to make sure we're all on the same page; master branch is still under feature freeze14:35
geguileojgriffith: Thanks14:35
jgriffithYou all probably already knew that and I was the only one that didn't :(14:35
jgriffithbut just to make sure :)14:35
smcginnisjgriffith: Bug fix only, right? Real bugs, not "my driver doesn't support feature x" bugs.14:35
jgriffithsmcginnis: correct14:35
jgriffithsmcginnis: and even more so, think of it as "this is going in to Liberty"14:36
smcginnisYep14:36
jgriffithso don't break stuff :)14:36
smcginnisI do think there are some good things out there right now to get in to L.14:36
jgriffithsmcginnis: agreed14:36
smcginnisjgriffith: But yeah, definitely should consider the risk of some of it going in at this point.14:36
*** raildo-afk is now known as raildo14:37
*** zhangjn has joined #openstack-cinder14:37
*** rhagarty has quit IRC14:38
*** teju has joined #openstack-cinder14:38
jgriffithsmcginnis: I'm lookign trhough https://review.openstack.org/#/q/status:merged+cinder,n,z to make sure I didn't merge anything i shouldn't have ;)14:38
*** rhagarty_ has quit IRC14:38
smcginnisjgriffith: I hadn't noticed anything questionable. :)14:38
*** zhangjn has quit IRC14:39
smcginnisWell, that Oracle one maybe, but that wasn't you.14:39
jgriffithyeah... I"m fine with it14:39
*** takedakn has quit IRC14:39
jgriffithand I think tbarron 's stuff got merged so that's cool too14:39
tbarronjgriffith: yes, thank you and too all that swarmed on that queue.14:39
tbarronto all14:39
jgriffithgeguileo: ping14:42
geguileojgriffith: pong14:45
jgriffithgeguileo: hey... if you have some time later or maybe tomorrow morning...14:45
jgriffithgeguileo: I was wondering if you could help me understand how https://review.openstack.org/#/c/216377/10/cinder/volume/api.py improves things14:46
geguileojgriffith: Sure, when is a good time for you?14:47
jgriffithgeguileo: I was hoping right now, but sadly some thing has come up... and you're in an opposite TZ if I recall no?14:48
jgriffithgeguileo: maybe tomorrow morning before weekly meeting if that works ok for you?14:48
geguileojgriffith: Yep, here it's 16:4814:48
jgriffithgeguileo: :)14:48
geguileojgriffith: Sure, just ping me and most likely I'll be available14:48
jgriffithgeguileo: I'll play around with it some more on my own tonight and see if I can educate myself a bit :)14:48
jgriffithgeguileo: sounds great, thank you!14:49
geguileojgriffith: np14:49
*** baojg has joined #openstack-cinder14:50
*** david-ly_ is now known as david-lyle14:53
*** dustins has joined #openstack-cinder14:54
*** tsekiyama has joined #openstack-cinder14:55
*** tpsilva has quit IRC14:55
*** rick-chen has quit IRC14:57
*** dustins has quit IRC14:57
*** cbits has joined #openstack-cinder14:58
*** anshul has quit IRC14:59
*** martyturner has quit IRC14:59
*** timcl1 has joined #openstack-cinder15:00
*** zhipeng has joined #openstack-cinder15:00
*** fifieldt has quit IRC15:00
*** cbader has joined #openstack-cinder15:00
*** haomaiwang has quit IRC15:01
*** martyturner has joined #openstack-cinder15:01
*** haomaiwang has joined #openstack-cinder15:01
*** haomaiwang has quit IRC15:01
*** timcl has quit IRC15:02
*** hodos has joined #openstack-cinder15:02
*** dustins has joined #openstack-cinder15:02
*** cbits has left #openstack-cinder15:03
openstackgerritVipin Balachandran proposed openstack/cinder: Enable certificate verification during image copy  https://review.openstack.org/22127215:06
*** teju has quit IRC15:08
*** baojg has quit IRC15:09
*** akerr is now known as akerr_away15:12
*** akerr_away is now known as akerr15:12
openstackgerritIvan Kolodyazhny proposed openstack/python-cinderclient: Remove unused code from cinderclient.utils module  https://review.openstack.org/22132315:14
*** changbl has joined #openstack-cinder15:14
openstackgerritYuriy Nesenenko proposed openstack/cinder: Small optimization in Block Device driver  https://review.openstack.org/22056115:16
openstackgerritYuriy Nesenenko proposed openstack/cinder: Implement extend_volume method to Block Device driver  https://review.openstack.org/22130315:18
*** agarciam has joined #openstack-cinder15:18
*** jungleboyj has joined #openstack-cinder15:20
*** salv-orlando has quit IRC15:20
*** tpsilva has joined #openstack-cinder15:21
*** rushil has joined #openstack-cinder15:23
*** dims_ has joined #openstack-cinder15:25
*** afazekas_ has quit IRC15:28
*** monsoon has joined #openstack-cinder15:29
*** dimsum__ has quit IRC15:29
monsoonhi guys!15:30
*** salv-orlando has joined #openstack-cinder15:30
monsoonis the volume backend remotefs a real back-end? or is it just a base class?15:31
*** jdurgin1 has joined #openstack-cinder15:31
eharneymonsoon: just a base class15:31
monsoonthank you)15:32
*** vgridnev has quit IRC15:33
monsoonso that means that this documentation http://docs.openstack.org/kilo/config-reference/content/section_cinder.conf.html15:33
monsoonis autogenerated?15:33
monsoonsince it has configuration options for remotefs15:33
eharneymonsoon: it is, but drivers do use those options15:34
monsoonok, now it's totally clear. thank you again)15:34
*** guest5 has joined #openstack-cinder15:35
*** dobson has quit IRC15:36
*** salv-orlando has quit IRC15:36
*** dobson has joined #openstack-cinder15:39
*** devlaps has joined #openstack-cinder15:43
*** timcl1 has quit IRC15:45
*** edtubill has left #openstack-cinder15:45
*** edtubill has joined #openstack-cinder15:45
*** hodos has quit IRC15:46
*** timcl has joined #openstack-cinder15:49
*** martyturner has quit IRC15:54
*** agarciam1 has joined #openstack-cinder15:55
*** guest5 has quit IRC15:56
*** agarciam has quit IRC15:56
openstackgerritYusuke Hayashi proposed openstack/python-cinderclient: Fix incorrect exception message in cinderclient  https://review.openstack.org/21082715:56
*** tpsilva has quit IRC15:57
*** rushil has quit IRC15:57
*** hemna_ is now known as hemna15:57
*** r-daneel has quit IRC15:58
*** timcl1 has joined #openstack-cinder15:59
*** tsekiyama has quit IRC16:00
*** timcl has quit IRC16:01
lpetrutHi guys. I was wondering: shouldn't there be a check for missing volume type id here? https://github.com/openstack/cinder/blob/master/cinder/objects/volume.py#L241-L24316:03
*** martyturner has joined #openstack-cinder16:03
*** annegentle has joined #openstack-cinder16:05
*** Apoorva has joined #openstack-cinder16:06
*** zhenguo has quit IRC16:08
*** zhipeng has quit IRC16:08
*** jistr has quit IRC16:11
*** leeantho has joined #openstack-cinder16:15
*** esker has joined #openstack-cinder16:16
*** Zhongjun_ has joined #openstack-cinder16:19
*** tpsilva has joined #openstack-cinder16:19
*** rhagarty has joined #openstack-cinder16:26
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/22104016:26
*** tpsilva has quit IRC16:26
*** rhagarty_ has joined #openstack-cinder16:26
*** bluex-pl has quit IRC16:31
*** dims_ has quit IRC16:31
*** dimsum__ has joined #openstack-cinder16:32
*** Yogi1 has joined #openstack-cinder16:32
*** dannywilson has joined #openstack-cinder16:36
*** sghanekar_ has joined #openstack-cinder16:36
*** vgridnev has joined #openstack-cinder16:37
openstackgerritAnthony Lee proposed openstack/cinder: Fix QoS keys not being available to scheduler filter and weighers  https://review.openstack.org/21482616:38
*** zhangjn has joined #openstack-cinder16:38
*** andreykurilin__ has joined #openstack-cinder16:39
*** mdbooth has quit IRC16:39
*** Guest60469 is now known as mgagne16:40
*** mgagne has joined #openstack-cinder16:40
*** jdurgin1 has quit IRC16:42
*** leeantho has quit IRC16:43
*** dflorea has joined #openstack-cinder16:44
*** winston-d has quit IRC16:46
*** winston-d has joined #openstack-cinder16:47
*** leeantho has joined #openstack-cinder16:47
*** ronis has joined #openstack-cinder16:49
*** tsekiyama has joined #openstack-cinder16:50
*** crose has joined #openstack-cinder16:50
*** lkong has quit IRC16:53
*** lpetrut has quit IRC16:56
*** e0ne has quit IRC16:56
*** earlephilhower has quit IRC16:57
*** aix has quit IRC16:58
*** tsekiyama has quit IRC17:00
*** harlowja has joined #openstack-cinder17:03
*** afazekas_ has joined #openstack-cinder17:05
*** dannywilson has quit IRC17:06
*** pv has joined #openstack-cinder17:08
pvi had a question on volume-attach, currently my attach sometimes goes through and sometimes it doesn't17:09
pvwhen it doesnt go through my n-cpu log outputs Looking for Fibre Channel dev /dev/disk/by-path/pci-0000:02:00.0-fc-0x10003c912b666600-lun-0 _wait_for_device_discovery which it keeps trying until it eventually fails17:09
pvi don't get why that path is sometimes available and sometimes it isnt found, i try to ls the path and when volume-attach fails it says the path doesn't exist17:10
pvthe target port is the only available target on the machine I'm using and everything else seems to be in place, i know for a fact it has worked before it just is really sporadic on when it does work and when it doesnt17:11
pvany ideas?17:11
hemnais there a FC zone for it ?17:11
hemnaif the volume doesn't show up, then there is nothing that nova/cinder can do17:11
pvyeah17:11
pvit is zoned17:12
hemnaneed to check the fabric17:12
pvok thought so, its not really an openstack issue i just wanted to see if anyone's ever run into something similar17:12
pvwith their own FC driver17:12
hemnayah17:12
hemnathat typically happens to me when the zone isn't available or is down between the 2 endpoints.17:12
pvhmm alright, ill check the zoning again and see if its all properly zoned17:13
pvbefore what use to work was if it wasnt found, i'd do a quick rescan and then the device would be found17:13
pvbut even that doesn't seem to be working today haha17:13
pvthanks anyways!17:14
hemnanp17:14
*** tsekiyama has joined #openstack-cinder17:14
*** markstur has quit IRC17:18
*** ronis has quit IRC17:20
*** zhangjn has quit IRC17:21
*** afazekas_ has quit IRC17:24
*** tsekiyama has quit IRC17:24
*** Apoorva has quit IRC17:25
*** Apoorva has joined #openstack-cinder17:26
*** ociuhandu has quit IRC17:26
*** earlephilhower has joined #openstack-cinder17:27
*** dflorea has quit IRC17:29
jungleboyjhemna: You still around?17:31
*** markstur has joined #openstack-cinder17:31
*** dflorea_ has joined #openstack-cinder17:31
hemnajungleboyj, yah17:32
jungleboyjOh, I think I found the answer.17:33
jungleboyjNeed to use the --allow-multiattach option to mark a volume as multi-attach capable.17:33
hemnaah yah17:33
hemnaat create time17:33
jungleboyjWhich hasn't yet been released as we haven't done a new cinderclient drop yet.17:34
*** sghanekar_ has quit IRC17:37
*** crose has quit IRC17:38
*** mudassirlatif has joined #openstack-cinder17:42
openstackgerritEric Harney proposed openstack/cinder: Add migrate_volume to BaseVD  https://review.openstack.org/22142517:44
*** anshul has joined #openstack-cinder17:45
*** e0ne has joined #openstack-cinder17:47
openstackgerritEric Harney proposed openstack/cinder: Add migrate_volume to BaseVD  https://review.openstack.org/22142517:51
*** merooney has joined #openstack-cinder17:53
*** vgridnev has quit IRC18:00
*** aix has joined #openstack-cinder18:01
*** r-daneel has joined #openstack-cinder18:02
*** rushil has joined #openstack-cinder18:04
*** dflorea_ has quit IRC18:05
*** vgridnev has joined #openstack-cinder18:06
*** vgridnev has quit IRC18:06
*** dflorea has joined #openstack-cinder18:11
openstackgerritPatrick East proposed openstack/cinder: Correctly report multiattach in Pure drivers  https://review.openstack.org/22143918:14
*** dflorea has quit IRC18:15
*** shyama has quit IRC18:15
openstackgerritGorka Eguileor proposed openstack/cinder: Remove API races from delete methods  https://review.openstack.org/20583518:17
openstackgerritGorka Eguileor proposed openstack/cinder: Add atomic conditional updates to objects  https://review.openstack.org/20583418:17
openstackgerritGorka Eguileor proposed openstack/cinder: Remove API races on extend and volume_upload_image  https://review.openstack.org/21637818:17
openstackgerritGorka Eguileor proposed openstack/cinder: Move get_by_id to CinderObject  https://review.openstack.org/21801218:17
openstackgerritGorka Eguileor proposed openstack/cinder: Improve metadata update operations  https://review.openstack.org/21637618:17
openstackgerritGorka Eguileor proposed openstack/cinder: Remove API races from attach and detach methods  https://review.openstack.org/21637718:17
openstackgerritGorka Eguileor proposed openstack/cinder: Remove API races on replication methods  https://review.openstack.org/22144118:17
openstackgerritGorka Eguileor proposed openstack/cinder: WIP: Remove API races from migrate_volume  https://review.openstack.org/22144218:17
*** merooeny_ has joined #openstack-cinder18:19
*** merooney has quit IRC18:22
*** rushil has quit IRC18:25
openstackgerritEric Harney proposed openstack/cinder: Add migrate_volume to BaseVD  https://review.openstack.org/22142518:26
openstackgerritEric Harney proposed openstack/cinder: Add manage_existing and unmanage to BaseVD  https://review.openstack.org/22144518:26
*** Zhongjun_ has quit IRC18:27
*** merooeny_ has quit IRC18:28
*** e0ne has quit IRC18:29
*** Apoorva_ has joined #openstack-cinder18:30
*** merooney has joined #openstack-cinder18:30
*** openstackgerrit has quit IRC18:31
*** openstackgerrit has joined #openstack-cinder18:32
*** dflorea has joined #openstack-cinder18:32
*** Apoorva has quit IRC18:33
*** mudassirlatif has quit IRC18:37
*** dflorea has quit IRC18:37
*** dflorea has joined #openstack-cinder18:39
*** ociuhandu has joined #openstack-cinder18:39
*** dflorea has quit IRC18:41
*** angela-s has joined #openstack-cinder18:44
*** vgridnev has joined #openstack-cinder18:48
*** garthb has joined #openstack-cinder18:49
*** dflorea has joined #openstack-cinder18:50
*** dflorea has quit IRC18:50
*** dflorea has joined #openstack-cinder18:52
*** tpsilva has joined #openstack-cinder18:53
jungleboyjhemna: eharney jgriffith Any reviews that we are finding associated with BPs should be -2'd right now?  Agreed?18:57
jgriffithjungleboyj: correct18:57
eharneyjungleboyj: yes18:58
jgriffithjungleboyj: we're still under feature freeze18:58
jgriffithjungleboyj: and keep in mind what you merge now, goes in to Liberty18:58
jgriffithjungleboyj: so be picky :)18:58
jungleboyjOk, great.18:58
jungleboyjJust didn't want to be an jerk for now reason.18:59
jungleboyj*no19:00
eharneyjungleboyj: don't raise the bar too high for the rest of us :)19:02
jungleboyj:-)19:02
*** vgridnev has quit IRC19:09
ameadehemna: /usr/local/bin/nova-rootwrap: Unauthorized command: scsi_id19:13
ameadehemna: any idea why it's using nova-rootwrap? should it be doing brick rootwrap?19:14
*** aorourke has joined #openstack-cinder19:16
*** vgridnev has joined #openstack-cinder19:17
eharneyameade: iirc it uses nova-rootwrap when loaded in nova, and cinder-rootwrap when loaded in cinder?19:18
ameadeeharney: i think that's ok but nova's rootwrap filters dont have scsi_d in there19:20
ameadei wonder how gate is working.. gonna go poke around in their configs19:20
eharneyameade: there's a rootwrap filters config in os-brick itself that needs to get installed into /etc/nova/...19:21
ameadeoh maybe i just need ot update devstack or somthing19:21
ameadei'm sure thats it, thanks!19:22
*** p0rtal has joined #openstack-cinder19:24
*** martyturner has quit IRC19:26
scottdaAnyone interested in nova force_detach of a volume there's a working patch up for review:19:27
scottdahttps://review.openstack.org/#/c/184537/19:27
*** dflorea has quit IRC19:28
*** martyturner has joined #openstack-cinder19:30
*** pots1 has joined #openstack-cinder19:30
*** aix has quit IRC19:32
*** palendae has joined #openstack-cinder19:32
*** dflorea has joined #openstack-cinder19:34
*** tpsilva has quit IRC19:34
smcginnisameade: https://review.openstack.org/#/c/207677/719:43
smcginnisameade: It's in the works. Hopefully soon.19:43
smcginnisameade: That command (as far as I know) is only used for FC, so it's just affecting the FC driver test runs.19:44
smcginnisameade: Therefore Jenkins is fine.19:44
smcginnisameade: Temporarily added scsi_id to cinder, but once that devstack patch lands it should be removed.19:44
smcginnisameade: Nova didn't want the temporary workaround.19:44
ameadeah gotcha, thanks19:45
smcginnisameade: You can add NOVA_BRANCH=refs/changes/50/220750/1 to your local.conf to pull in the nova side.19:45
*** andreykurilin__ has quit IRC19:46
*** pots1 has quit IRC19:46
*** dflorea has quit IRC19:47
*** theanalyst has quit IRC19:48
*** theanalyst has joined #openstack-cinder19:52
hemnaameade, the rootwrap is in brick now19:53
hemnaameade, there is a devstack patch still outstanding that fixes it19:53
smcginnishemna: Any thoughts on how to make the rootwrap piece a little more automatic?19:56
smcginnishemna: I'm worried we are going to have issues with the brick filters not getting copied over to the right place.19:56
*** dflorea has joined #openstack-cinder19:56
*** jecarey has quit IRC19:56
hemnaafaik, currently the devstack patch is the only way19:57
smcginnishemna: It would be great if the filter file for os-brick would just get pulled in, but that probably means updates to oslo-rootwrap.19:57
hemnanova folks objected to having the filters added in nova's existing computer.filters file19:57
hemnawhich makes sense, because new filters can be added in brick and then they don't exist in nova/cinder19:58
smcginnishemna: Yeah, for both cinder and nova that could be a pain.19:58
eharneyhemna: i wonder if they would support some kind of #include /etc/os-brick/filters ability?  does rootwrap have that?19:58
smcginnishemna: Hopefully my fears are unfounded and it's not an issue.19:58
*** palendae has left #openstack-cinder19:58
hemnanah, they aren't unfounded :P19:58
smcginnisProbably too late in the cycle to really do much about it anyway other than document needing to copy the file.19:59
smcginnisBah, meeting time! bbl19:59
*** dimsum__ is now known as dims20:03
*** dustins has quit IRC20:04
*** dims is now known as Guest8007120:04
*** timcl1 has quit IRC20:04
*** lcurtis has joined #openstack-cinder20:06
*** Guest80071 is now known as dims__20:08
*** martyturner has quit IRC20:09
*** mudassirlatif has joined #openstack-cinder20:09
*** martyturner has joined #openstack-cinder20:10
openstackgerritPatrick East proposed openstack/cinder: Correctly report multiattach in Pure drivers  https://review.openstack.org/22143920:10
*** martyturner1 has joined #openstack-cinder20:13
*** martyturner has quit IRC20:15
*** guest5 has joined #openstack-cinder20:24
patrickeasthey so for the image cache and internal tenant feature... whats the right (best?) place to add documentation for that? in the config reference somewhere like here(ish) http://docs.openstack.org/kilo/config-reference/content/ch_configuring-openstack-block-storage.html ?20:25
*** amoturi has left #openstack-cinder20:25
patrickeastor maybe somewhere in here http://docs.openstack.org/admin-guide-cloud/blockstorage-manage-volumes.html ?20:25
annegentlepatrickeast: both. the first table is auto-generated by scraping the cinder codebase20:30
patrickeastannegentle: ok cool, thanks!20:31
annegentlepatrickeast: you'll find the source in the openstack/openstack-manuals repo20:32
annegentlefor the admin-guide-cloud, and then use openstack/openstack-doc-tools to find the codescraper20:32
patrickeastannegentle: sweet, found it, thanks for the pointers20:33
*** raildo is now known as raildo-afk20:34
*** tiny-hands has joined #openstack-cinder20:35
*** ekarlso has quit IRC20:38
*** vgridnev has quit IRC20:40
mtreinisheharney: heh, maybe I should start reading the bugs in the commit msg :)20:41
*** Yogi1 has quit IRC20:41
eharneymtreinish: thanks for the quick feedback though20:41
mtreinisheharney: sure np, it actually stood out to me because I remember having a long conversation with someone about testing it20:42
mtreinishthat's how I learned about the lvm thing20:43
openstackgerritEric Harney proposed openstack/cinder: Add unmanage default implementation to VolumeDriver  https://review.openstack.org/22149720:43
*** Lee1092 has quit IRC20:43
*** raildo-afk is now known as raildo20:44
eharneymtreinish: i think i actually don't want it to have a skip check20:44
eharneymtreinish: so that it can ensure something sane happens when the feature isn't supported?20:45
eharneyor maybe that needs to be a separate test20:45
mtreinisheharney: like check the response of the not supported error from cinder?20:46
mtreinishI think that might be a seperate test20:46
eharneymtreinish: i think it would have to check volume state instead20:46
eharneymtreinish: or, check that force delete works after the unmanage fails, which was what i was going for there (but I missed an else:)20:47
*** salv-orlando has joined #openstack-cinder20:48
mtreinisheharney: that feels like it should be a seperate test, because you're verifying a distinct path20:48
eharneymtreinish: makes sense20:49
mtreinishalthough tbh, that's not the kinda thing we normally verify in tempest20:49
eharneyhmm, i probably don't really know where the line is for sensible tests to put in there or not20:49
mtreinisheharney: well if there were in-tree cinder functional tests I'd suggest doing it there. Because it's kinda of an edge case which is very config specific20:51
eharneymtreinish: so maybe this should aim more at "if supported, run the basic path"20:52
mtreinisheharney: yep, that's what I was going for. Which was what I was suggesting in my review :)20:52
eharney:)20:53
*** vgridnev has joined #openstack-cinder20:53
*** jwang_ has joined #openstack-cinder20:54
*** raildo is now known as raildo-afk20:56
*** cdelatte has quit IRC20:56
*** patrickeast has quit IRC20:56
*** patrickeast has joined #openstack-cinder20:57
*** jwang has quit IRC20:57
*** ekarlso has joined #openstack-cinder20:57
*** ekarlso has quit IRC20:57
*** ekarlso has joined #openstack-cinder20:57
*** bswartz has quit IRC20:57
openstackgerritAngela Smith proposed openstack/cinder: Adds VF support with HTTPS southbound connector  https://review.openstack.org/18479921:00
*** gouthamr has quit IRC21:00
*** dflorea has quit IRC21:00
*** aix has joined #openstack-cinder21:01
*** dflorea has joined #openstack-cinder21:03
*** dflorea has quit IRC21:04
*** changbl has quit IRC21:07
*** vgridnev has quit IRC21:13
*** lcurtis has quit IRC21:13
*** anshul has quit IRC21:14
*** dflorea has joined #openstack-cinder21:26
*** IanGovett has quit IRC21:27
*** tiny-hands has quit IRC21:31
*** dflorea has quit IRC21:31
*** vgridnev has joined #openstack-cinder21:31
*** akerr has quit IRC21:32
openstackgerritJohn Griffith proposed openstack/cinder: Return a tuple from SolidFire update_provider_info  https://review.openstack.org/22151021:32
*** dflorea has joined #openstack-cinder21:34
*** IlyaG has joined #openstack-cinder21:35
*** devlaps has quit IRC21:38
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Move the rootwrap filters  https://review.openstack.org/22151321:40
*** mriedem is now known as mriedem_away21:42
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Move the rootwrap filters  https://review.openstack.org/22151321:43
*** vgridnev has quit IRC21:44
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Move the rootwrap filters  https://review.openstack.org/22151321:45
*** guest5 has quit IRC21:46
*** guest5 has joined #openstack-cinder21:47
*** dflorea has quit IRC21:49
*** xyang1 has joined #openstack-cinder21:49
*** jungleboyj has quit IRC21:50
*** jamielennox|away is now known as jamielennox21:51
*** dflorea has joined #openstack-cinder21:52
*** eduardo has quit IRC21:53
*** marcusvrn_ has quit IRC21:55
*** edmondsw has quit IRC21:57
*** jgregor has quit IRC22:00
*** p0rtal_ has joined #openstack-cinder22:00
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/22104022:01
*** p0rtal has quit IRC22:04
*** rushil has joined #openstack-cinder22:06
*** annegentle has quit IRC22:13
*** edtubill has quit IRC22:14
*** IlyaG has quit IRC22:20
openstackgerritAnthony Lee proposed openstack/cinder: Fix QoS keys not being available to scheduler  https://review.openstack.org/21482622:25
*** r-daneel has quit IRC22:26
*** IlyaG has joined #openstack-cinder22:30
*** rushil has quit IRC22:30
*** IlyaG_ has joined #openstack-cinder22:30
openstackgerritAnthony Lee proposed openstack/cinder: LeftHand Add update_migrated_volume to driver  https://review.openstack.org/22153422:30
*** rushil has joined #openstack-cinder22:30
*** IlyaG_ has quit IRC22:31
*** dannywilson has joined #openstack-cinder22:32
*** martyturner has joined #openstack-cinder22:32
*** dannywilson has quit IRC22:32
*** dannywilson has joined #openstack-cinder22:33
*** martyturner1 has quit IRC22:33
*** IlyaG has quit IRC22:34
*** xyang1 has quit IRC22:36
*** martyturner has quit IRC22:38
*** dalgaaf has quit IRC22:42
*** earlephilhower has quit IRC22:42
*** guest5 has quit IRC22:43
openstackgerritEric Harney proposed openstack/cinder: Filter scheduler: Fix KeyError on invalid create request  https://review.openstack.org/22153722:44
*** agarciam1 has quit IRC22:45
openstackgerritAnthony Lee proposed openstack/cinder: LeftHand Add update_migrated_volume to drivers  https://review.openstack.org/22153422:45
openstackgerritAnthony Lee proposed openstack/cinder: LeftHand Add update_migrated_volume to drivers  https://review.openstack.org/22153422:48
*** krtaylor has quit IRC22:49
*** rushil has quit IRC22:54
*** dflorea has quit IRC22:58
*** angela-s has quit IRC23:01
*** dflorea has joined #openstack-cinder23:03
*** eharney has quit IRC23:06
*** dflorea has quit IRC23:07
*** hemna is now known as hemnafk23:10
*** erlon has quit IRC23:18
*** chlong has quit IRC23:27
*** chlong has joined #openstack-cinder23:28
*** cdelatte has joined #openstack-cinder23:34
*** cdelatte has quit IRC23:37
*** salv-orlando has quit IRC23:41
*** lixiaoy11 has joined #openstack-cinder23:43
*** smoriya has joined #openstack-cinder23:54
*** jungleboyj has joined #openstack-cinder23:58

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