Friday, 2016-06-17

*** suyash has quit IRC00:05
*** akapil has joined #openstack-cinder00:06
*** karthikp_ has quit IRC00:14
*** smoriya has joined #openstack-cinder00:17
*** akapil has quit IRC00:20
*** laughterwym has joined #openstack-cinder00:20
*** laughterwym has quit IRC00:26
*** laughterwym has joined #openstack-cinder00:26
*** laughterwym has joined #openstack-cinder00:27
*** arch-nemesis has quit IRC00:28
*** zul has joined #openstack-cinder00:28
*** itzdilip has joined #openstack-cinder00:37
*** itzdilip has quit IRC00:42
*** cknight has joined #openstack-cinder00:45
*** cknight1 has quit IRC00:45
*** diogogmt has joined #openstack-cinder00:52
*** cknight1 has joined #openstack-cinder00:53
*** cknight has quit IRC00:54
*** sdake has joined #openstack-cinder00:58
*** bardia has quit IRC01:00
*** Apoorva has quit IRC01:01
*** bardia has joined #openstack-cinder01:05
*** diogogmt has quit IRC01:06
*** cheneydc has joined #openstack-cinder01:13
*** sdake has quit IRC01:14
*** merooney has quit IRC01:14
*** iceyao has joined #openstack-cinder01:15
*** sdake has joined #openstack-cinder01:16
*** EinstCrazy has joined #openstack-cinder01:21
*** itzdilip has joined #openstack-cinder01:22
*** sdake_ has joined #openstack-cinder01:22
*** sdake has quit IRC01:24
*** yangyapeng has joined #openstack-cinder01:25
*** wN has quit IRC01:27
jgriffithsmcginnis: so do you want to just merge your fix rather than that revert?01:30
jgriffithsmcginnis: I'd suggest we abandon that revert if that's the case.  I haven't had time to load both up and do a mysql-dump... but assuming you or Jay did?01:31
*** Apoorva has joined #openstack-cinder01:33
*** Lee1092 has joined #openstack-cinder01:35
*** geguileo_phone has quit IRC01:37
*** itzdilip has quit IRC01:37
*** itzdilip has joined #openstack-cinder01:38
openstackgerritwuyuting proposed openstack/cinder: Fix infortrend driver bug: cloned with src vol size  https://review.openstack.org/32679201:41
*** mtanino has quit IRC01:45
openstackgerritwuyuting proposed openstack/cinder: ProphetStor driver create vol from snap with wrong size  https://review.openstack.org/32974101:48
openstackgerritliuke proposed openstack/cinder: Huawei: Support reporting disk type of pool  https://review.openstack.org/32416301:50
mtreinishjgriffith: any idea why cinder unit tests dump tracebacks even when they pass?: http://logs.openstack.org/70/330570/2/check/gate-cinder-python34-db/ffc9b11/console.html#_2016-06-16_15_41_42_62946401:52
openstackgerritMerged openstack/cinder: SolidFire driver should return ceiling of volume size  https://review.openstack.org/32677601:55
*** itzdilip has quit IRC02:01
*** Apoorva has quit IRC02:05
*** garthb has joined #openstack-cinder02:10
*** sdake has joined #openstack-cinder02:16
*** huanan has quit IRC02:16
*** akapil has joined #openstack-cinder02:18
*** sdake_ has quit IRC02:19
*** mtanino has joined #openstack-cinder02:22
*** houming has joined #openstack-cinder02:24
*** Suyash has joined #openstack-cinder02:25
*** akapil has quit IRC02:38
openstackgerritxiexs proposed openstack/cinder: Move _check_metadata_properties to cinder/utils  https://review.openstack.org/33050202:41
*** garthb has quit IRC02:52
*** adisky has joined #openstack-cinder02:53
openstackgerrithuanan proposed openstack/cinder: Huawei: Fixed problems caused by password expiration  https://review.openstack.org/30563302:53
*** huanan has joined #openstack-cinder02:55
*** jamielennox is now known as jamielennox|away03:03
*** gouthamr_ has quit IRC03:12
*** jamielennox|away is now known as jamielennox03:19
openstackgerritwuyuting proposed openstack/cinder: Fix infortrend driver bug: cloned with src vol size  https://review.openstack.org/32679203:29
openstackgerritMerged openstack/cinder: Define context.roles with base class  https://review.openstack.org/28535403:34
*** bardia has quit IRC03:37
*** markvoelker has quit IRC03:47
*** mriedem has quit IRC03:51
*** yhayashi has joined #openstack-cinder03:53
*** cknight1 has quit IRC03:55
*** GB21 has joined #openstack-cinder04:02
*** lpetrut has joined #openstack-cinder04:03
openstackgerrithaobing proposed openstack/cinder: Allow setting CG name or description to empty value  https://review.openstack.org/32446504:12
nikeshmhi04:19
nikeshmdo cinder supports cloning a volume from a attached volume?04:19
*** links has joined #openstack-cinder04:20
nikeshme0ne: there04:22
nikeshm?04:22
*** sgotliv_ has joined #openstack-cinder04:24
*** laughterwym has quit IRC04:24
*** lpetrut has quit IRC04:28
*** ociuhandu has quit IRC04:29
*** sgotliv_ has quit IRC04:29
*** karthikp has joined #openstack-cinder04:33
*** cheneydc has quit IRC04:35
openstackgerritVijay Ladani proposed openstack/cinder: Add unit tests in cinder for HPE Lefthand driver  https://review.openstack.org/31794804:40
*** Suyash has quit IRC04:41
*** ociuhandu has joined #openstack-cinder04:42
*** markvoelker has joined #openstack-cinder04:47
*** sdake_ has joined #openstack-cinder04:50
*** markvoelker has quit IRC04:52
*** sdake has quit IRC04:53
*** GB21 has quit IRC04:53
*** garthb has joined #openstack-cinder04:57
*** garthb has quit IRC04:58
*** lprice has joined #openstack-cinder05:01
*** Poornima has joined #openstack-cinder05:03
*** salv-orl_ has joined #openstack-cinder05:09
*** ChubYann has quit IRC05:11
*** salv-orlando has quit IRC05:12
*** pgadiya has joined #openstack-cinder05:12
*** lpetrut has joined #openstack-cinder05:14
openstackgerritParashuram Hallur proposed openstack/cinder: Implementation for CoprHD Cinder Drivers  https://review.openstack.org/31764305:14
*** akapil has joined #openstack-cinder05:21
*** links has quit IRC05:27
*** akapil has quit IRC05:27
*** sdake has joined #openstack-cinder05:27
*** links has joined #openstack-cinder05:28
*** yangyape_ has joined #openstack-cinder05:28
*** cknight has joined #openstack-cinder05:29
*** sdake_ has quit IRC05:31
*** yangyapeng has quit IRC05:31
*** cknight has quit IRC05:33
lixiaoy1nikeshm: Cinder supports cloning a volume from in-use volume05:34
*** GB21 has joined #openstack-cinder05:34
nikeshmthanks05:35
lixiaoy1nikeshm: as clone is done in backend storages05:35
*** laughterwym has joined #openstack-cinder05:36
*** sdake_ has joined #openstack-cinder05:38
*** sdake has quit IRC05:41
*** bhavik has joined #openstack-cinder05:42
*** bhavik has quit IRC05:47
*** karthikp has quit IRC05:48
*** sdake_ has quit IRC05:50
openstackgerritCao ShuFeng proposed openstack/python-cinderclient: Fix useless api_version of Manager class  https://review.openstack.org/30928305:50
*** lprice1 has joined #openstack-cinder05:58
*** lprice has quit IRC05:59
*** karthikp has joined #openstack-cinder06:01
*** bhavik has joined #openstack-cinder06:03
*** karthikp has quit IRC06:05
*** belmoreira has joined #openstack-cinder06:06
*** nkrinner has joined #openstack-cinder06:10
*** rcernin has joined #openstack-cinder06:24
*** mtanino has quit IRC06:24
*** david-lyle has quit IRC06:26
*** wanghao has quit IRC06:28
*** lpetrut has quit IRC06:33
*** arecknag has joined #openstack-cinder06:35
openstackgerritwanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly  https://review.openstack.org/23243606:49
*** sheel has joined #openstack-cinder06:50
openstackgerritwanghao proposed openstack/cinder: Fix snapshot stuck in 'deleting' after reboot c-v  https://review.openstack.org/32241806:52
*** sticker has quit IRC06:54
openstackgerritVipin Balachandran proposed openstack/cinder: VMware:Persist storage policy ID in vmdk meta-data  https://review.openstack.org/33091106:58
openstackgerritxiexs proposed openstack/cinder: Add validation for the metadata properties  https://review.openstack.org/33091306:59
*** itzdilip has joined #openstack-cinder07:01
*** ihrachys has joined #openstack-cinder07:03
*** cknight has joined #openstack-cinder07:04
*** ihrachys has quit IRC07:08
*** salv-orl_ has quit IRC07:09
*** salv-orlando has joined #openstack-cinder07:09
*** cknight has quit IRC07:09
*** itzdilip has quit IRC07:10
*** tesseract has joined #openstack-cinder07:10
*** hogepodge has quit IRC07:12
*** bauzas is now known as bauwser07:13
*** bauwser is now known as bauzas07:14
openstackgerritningwei proposed openstack/cinder: Added backend driver for Huawei FusionStorage  https://review.openstack.org/32888707:14
*** bauzas has left #openstack-cinder07:18
*** GB21 has quit IRC07:23
*** salv-orlando has quit IRC07:35
*** gcb has joined #openstack-cinder07:35
*** EinstCra_ has joined #openstack-cinder07:38
*** EinstCrazy has quit IRC07:40
*** yangyape_ has quit IRC07:40
huyangkindly review: https://review.openstack.org/#/c/325682/07:40
*** claudiub has joined #openstack-cinder07:41
*** iceyao has quit IRC07:42
*** iceyao has joined #openstack-cinder07:42
*** EinstCra_ has quit IRC07:43
*** EinstCrazy has joined #openstack-cinder07:43
*** zhangjn has quit IRC07:45
*** rcernin has quit IRC07:45
openstackgerrithuanan proposed openstack/cinder: Huawei: Fixed problems caused by password expiration  https://review.openstack.org/30563307:46
*** GB21 has joined #openstack-cinder07:49
*** zhangjn has joined #openstack-cinder07:50
*** lpetrut has joined #openstack-cinder07:57
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:00
*** cknight has joined #openstack-cinder08:06
*** yangyapeng has joined #openstack-cinder08:07
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
*** rcernin has joined #openstack-cinder08:10
*** cknight has quit IRC08:12
*** itzdilip has joined #openstack-cinder08:12
*** huyang_ has joined #openstack-cinder08:20
*** ameade has quit IRC08:21
*** thurloat has quit IRC08:21
*** v12aml has quit IRC08:22
*** ameade has joined #openstack-cinder08:22
*** tyrola has joined #openstack-cinder08:22
*** huyang has quit IRC08:23
*** huyang_ is now known as huyang08:23
*** thurloat has joined #openstack-cinder08:23
*** v12aml has joined #openstack-cinder08:23
*** tyrola has quit IRC08:23
*** itzdilip has quit IRC08:23
openstackgerritnikesh proposed openstack/cinder: Provide Kaminario K2 all-flash array iSCSI driver  https://review.openstack.org/32728508:33
*** bhagyashris has joined #openstack-cinder08:36
*** mkrcmari__ has joined #openstack-cinder08:36
*** rmstar has joined #openstack-cinder08:36
*** kaisers has quit IRC08:37
*** pgadiya_ has joined #openstack-cinder08:37
*** dramakri has quit IRC08:37
*** liverpoo1er has joined #openstack-cinder08:38
*** TheBall has joined #openstack-cinder08:38
*** kaisers has joined #openstack-cinder08:38
*** ntpttr_ has quit IRC08:38
*** bswartz has quit IRC08:39
*** dramakri has joined #openstack-cinder08:39
*** itzdilip has joined #openstack-cinder08:39
*** bbhavsar_ has joined #openstack-cinder08:39
*** Poornima_K has joined #openstack-cinder08:40
*** ntpttr_ has joined #openstack-cinder08:40
*** harlowja_ has quit IRC08:41
*** winston-1 has quit IRC08:42
*** anteaya has quit IRC08:42
*** jay-mehta has quit IRC08:42
*** mvk_ has quit IRC08:42
*** bhagyashri has quit IRC08:42
*** rmstar_ has quit IRC08:42
*** ekarlso has quit IRC08:42
*** bhavik has quit IRC08:42
*** winston-d has joined #openstack-cinder08:42
*** The_Ball has quit IRC08:42
*** TheBall is now known as The_Ball08:43
*** arecknag has quit IRC08:43
*** mattymo has quit IRC08:43
*** pgadiya has quit IRC08:43
*** mdavidson has quit IRC08:43
*** liverpooler has quit IRC08:43
*** kaisers1 has quit IRC08:43
*** Poornima has quit IRC08:43
*** SergeyLukjanov2 has joined #openstack-cinder08:43
*** asselin_ has joined #openstack-cinder08:43
*** SergeyLukjanov has quit IRC08:43
*** xek has quit IRC08:43
*** arecknag has joined #openstack-cinder08:43
*** arecknag has joined #openstack-cinder08:43
openstackgerritHelen Walsh proposed openstack/cinder: VMAX Driver - SnapVX licensing checks for VMAX3  https://review.openstack.org/32297208:43
*** SergeyLukjanov2 is now known as SergeyLukjanov08:43
*** xek has joined #openstack-cinder08:43
*** asselin__ has quit IRC08:44
*** mattymo has joined #openstack-cinder08:44
*** mtreinish has quit IRC08:44
*** e0ne has joined #openstack-cinder08:44
*** ihrachys has joined #openstack-cinder08:44
*** mdavidson has joined #openstack-cinder08:44
*** kaisers1 has joined #openstack-cinder08:45
*** anteaya has joined #openstack-cinder08:45
*** mtreinish has joined #openstack-cinder08:46
*** jay-mehta has joined #openstack-cinder08:46
*** tpeoples has quit IRC08:47
*** tpeoples has joined #openstack-cinder08:49
*** ekarlso has joined #openstack-cinder08:51
*** laughterwym has quit IRC08:52
*** EinstCrazy has quit IRC08:55
*** itzdilip has quit IRC08:58
*** EinstCrazy has joined #openstack-cinder08:59
openstackgerritAbhishek Kekane proposed openstack/python-cinderclient: Log request-id for each api call  https://review.openstack.org/31592509:03
*** EinstCra_ has joined #openstack-cinder09:06
*** cknight has joined #openstack-cinder09:07
*** EinstCrazy has quit IRC09:09
*** cknight has quit IRC09:12
zigoHi there! Running tempest on Newton b1 of Cinder, I have a small issue. The test:09:13
zigotempest.api.volume.admin.test_volume_services.VolumesServicesV1TestJSON.test_get_service_by_host_name09:13
zigofails with this return:09:13
zigo    reference = [u'cinder-backup', u'cinder-scheduler', u'cinder-volume']09:13
zigo    actual    = [u'cinder-backup', u'cinder-scheduler']09:13
zigoI believe the issue is that cinder-volume replies debian@lvm as a host, instead of just "debian" as the others do. How can I fix this?09:13
zigoe0ne: ^09:13
*** e0ne has quit IRC09:13
*** openstackgerrit has quit IRC09:18
*** openstackgerrit has joined #openstack-cinder09:19
*** geguileo_phone has joined #openstack-cinder09:20
*** nobody_ has joined #openstack-cinder09:27
*** karthikp has joined #openstack-cinder09:31
*** karthikp has quit IRC09:35
*** bbhavsar_ has quit IRC09:42
*** itzdilip has joined #openstack-cinder09:44
*** cknight has joined #openstack-cinder09:54
*** savihou has quit IRC09:55
openstackgerritningwei proposed openstack/cinder: Added backend driver for Huawei FusionStorage  https://review.openstack.org/32888709:58
*** cknight has quit IRC09:58
*** gouthamr has joined #openstack-cinder09:59
*** gouthamr_ has joined #openstack-cinder10:00
*** yhayashi has quit IRC10:01
*** openstackgerrit has quit IRC10:03
*** openstackgerrit has joined #openstack-cinder10:04
*** gouthamr has quit IRC10:04
*** rcernin has quit IRC10:05
*** GB21 has quit IRC10:09
*** e0ne has joined #openstack-cinder10:12
openstackgerritzhangsong proposed openstack/os-brick: Fix the bug of 'AttributeError' when connect volume  https://review.openstack.org/33100710:14
openstackgerritParashuram Hallur proposed openstack/cinder: Implementation for CoprHD Cinder Drivers  https://review.openstack.org/31764310:16
openstackgerritYuriy Nesenenko proposed openstack/cinder: Fix quota-usage list information with invalid tenant-id  https://review.openstack.org/31567310:23
*** ociuhandu has quit IRC10:23
*** e0ne has quit IRC10:24
openstackgerritSzymon Borkowski proposed openstack/cinder: Convert backup_device to OVO  https://review.openstack.org/32933210:28
*** e0ne has joined #openstack-cinder10:29
*** iceyao has quit IRC10:40
*** gouthamr_ is now known as gouthamr10:40
*** GB21 has joined #openstack-cinder10:40
*** ociuhandu has joined #openstack-cinder10:43
*** huyang has quit IRC10:44
*** huyang has joined #openstack-cinder10:45
*** permalac has joined #openstack-cinder10:47
openstackgerritStefan Amann proposed openstack/os-brick: WIP: Support FICON-attached ECKD volumes on System z  https://review.openstack.org/32053610:47
*** e0ne has quit IRC10:47
*** hogepodge has joined #openstack-cinder10:54
*** cknight has joined #openstack-cinder10:55
openstackgerritStefan Amann proposed openstack/os-brick: WIP: Support FICON-attached ECKD volumes on System z  https://review.openstack.org/32053610:56
*** GB21 has quit IRC10:59
*** cknight has quit IRC11:00
*** smoriya has quit IRC11:05
*** GB21 has joined #openstack-cinder11:06
*** itzdilip has quit IRC11:14
*** itzdilip has joined #openstack-cinder11:15
*** itzdilip has quit IRC11:28
*** nobody_ has quit IRC11:29
*** itzdilip has joined #openstack-cinder11:30
openstackgerritnikesh proposed openstack/cinder: Provide Kaminario K2 all-flash array iSCSI driver  https://review.openstack.org/32728511:30
*** bjornar_ has joined #openstack-cinder11:31
*** itzdilip has quit IRC11:31
*** itzdilip has joined #openstack-cinder11:31
*** gcb has quit IRC11:33
*** gouthamr has quit IRC11:43
*** rcernin has joined #openstack-cinder11:48
*** gcb has joined #openstack-cinder11:49
*** houming has quit IRC11:49
*** gouthamr has joined #openstack-cinder11:50
*** EinstCra_ has quit IRC11:53
*** eharney has joined #openstack-cinder11:55
openstackgerritVijay Ladani proposed openstack/cinder: Add unit tests in cinder for HPE Lefthand driver  https://review.openstack.org/31794811:55
*** cknight has joined #openstack-cinder11:56
*** EinstCrazy has joined #openstack-cinder11:57
*** EinstCra_ has joined #openstack-cinder12:00
*** cknight has quit IRC12:00
*** EinstCrazy has quit IRC12:02
*** gouthamr_ has joined #openstack-cinder12:02
*** itzdilip has quit IRC12:02
*** gouthamr_ has quit IRC12:02
*** gouthamr_ has joined #openstack-cinder12:03
*** cdelatte has quit IRC12:03
*** EinstCrazy has joined #openstack-cinder12:03
*** dave-mccowan has joined #openstack-cinder12:04
*** timcl has joined #openstack-cinder12:04
*** gouthamr_ has quit IRC12:04
*** markvoelker has joined #openstack-cinder12:04
*** gouthamr_ has joined #openstack-cinder12:04
*** gouthamr has quit IRC12:05
*** gouthamr_ is now known as gouthamr12:05
*** EinstCra_ has quit IRC12:06
*** kfarr has joined #openstack-cinder12:07
*** e0ne has joined #openstack-cinder12:07
*** mtanino has joined #openstack-cinder12:07
*** itzdilip has joined #openstack-cinder12:09
*** EinstCra_ has joined #openstack-cinder12:11
openstackgerritYuriy Nesenenko proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request  https://review.openstack.org/30044312:11
*** laughterwym has joined #openstack-cinder12:11
*** EinstCra_ has quit IRC12:11
*** EinstCra_ has joined #openstack-cinder12:13
*** iceyao has joined #openstack-cinder12:14
*** EinstCrazy has quit IRC12:14
*** laughterwym has quit IRC12:15
*** itzdilip has quit IRC12:16
*** itzdilip has joined #openstack-cinder12:19
*** zhangjn has quit IRC12:19
*** cdelatte has joined #openstack-cinder12:19
*** EinstCra_ has quit IRC12:21
*** GB21 has quit IRC12:23
*** zhangjn has joined #openstack-cinder12:23
*** yangyapeng has quit IRC12:24
*** iceyao has quit IRC12:24
jgriffithmtreinish: No I don't...  looks like CloudByte is dumping a bunch of logging .  I'll have a look at it today12:24
*** cdelatte has quit IRC12:25
e0nejgriffith: morning, John. one short qurestion according to your comment to my patch12:27
e0nejgriffith: I like the idea to have functional tests with fake driver. shoult I propose spec for it?12:27
jgriffithe0ne: I don't think spec is necessary, but if you want to that's cool12:30
*** cdelatte has joined #openstack-cinder12:30
e0nejgriffith: I'll try to bring this topic to the next testing relating meeting. it community requests spec, I'll do it12:31
*** jsheeren has joined #openstack-cinder12:31
*** pauloewerton has joined #openstack-cinder12:34
*** edmondsw has joined #openstack-cinder12:34
*** timcl has quit IRC12:39
*** cdelatte has quit IRC12:39
*** alaski is now known as lascii12:40
*** cdelatte has joined #openstack-cinder12:41
*** adrianofr has joined #openstack-cinder12:43
*** bswartz has joined #openstack-cinder12:43
*** Srinu has quit IRC12:44
*** timcl has joined #openstack-cinder12:44
claudiubhemna: smcginnis: hello. can you take a look at the os-brick Windows iSCSI / fibre channel / SMB connectors? lpetrut has posted tempest test results for each of them:  https://review.openstack.org/#/c/272522/2912:46
smcginnisclaudiub: Will try to get to it today.12:48
smcginnisjgriffith: Saw your ping from yesterday afternoon. Yeah, I'd rather just fix things than revert that whole thing.12:50
smcginnisjgriffith: Though if folks prefer and this ends up more difficult than it should be, I could go release by release and compact that way to make things easier.12:51
smcginnisdulek: Saw your comments. Any reason why we would need to make sure the index names are the same? I didn't think that mattered.12:51
nikeshmsmcginnis: luckily i saw your recent merge and added in kaminario driver too12:55
nikeshminterface12:55
nikeshmthing12:55
smcginnisnikeshm: Just noticed that. Thanks!12:56
jgriffithsmcginnis: yuck... no thanks12:57
smcginnisjgriffith: Yeah... :)12:58
duleksmcginnis: My perception is that unified names will be useful if we'll happen to remove the constraint.13:01
smcginnisdulek: Oh, good point.13:01
jgriffithdulek: aren't the unified names sort of what's causing issue/confusion now though?13:02
*** faiz89 has joined #openstack-cinder13:02
*** gcb has quit IRC13:02
*** cdelatte has quit IRC13:03
dulekjgriffith: I don't think I follow.13:03
jgriffithdulek: it may be me that's not following :)13:03
jgriffithdulek: sorry...13:03
jgriffithdulek: I was looking at your comment on smcginnis 's compaction update13:03
*** akerr has joined #openstack-cinder13:04
smcginnisI can add explicit naming. That's pretty trivial.13:04
*** faiz89 has quit IRC13:04
jgriffithI'm trying to figure out if it matters :)13:04
smcginnisNot sure if that's the same across all DBs anyway. If we remove an index, we wouldn't do so by name via sqlalchemy, would we?13:05
jgriffithwe've added so much crap to the DB stuff it's sort of ridic13:05
jgriffithsmcginnis: I would hope we'd always enforce ID, but who knows13:05
smcginnisjgriffith: I didn't realize how true that was until I tried following its evolution. :)13:05
dulekjgriffith: I think it may be DB-specific, but if we wanted to change or delete a constraint, having same names across deployments would be useful.13:05
jgriffithsmcginnis: yeah, it's kinda crazy.13:06
smcginnisdulek: Yeah, I'll add it. Not sure it's necessary, but I don't think it will hurt anything.13:06
duleksmcginnis: It's possible via SQLA I think. Let me dig one old commit…13:06
smcginnisdulek: Had a minute of confusion here: http://paste.openstack.org/show/516822/13:06
smcginnisWith the CREATE SEQUENCE bit. That must be an interim step, because a few migrations into it we've ended up switching ID over to a varchar(36) to accomodate UUIDs.13:07
duleksmcginnis: Oh!13:07
smcginnisdulek: So I'm pretty sure we don't want to start at 1 and increment by 1. :)13:07
duleksmcginnis: Nice explanation.13:07
smcginnisdulek: A lot of these little things have ended up being stuff like that. That's made it a little harder to have to follow the whole story.13:08
duleksmcginnis: I totally believe it was incredibly hard.13:09
smcginnisdulek: It just needed some patience. ;)13:09
smcginnisA lot of "why the heck..."13:09
dulekAnd I don't want to be overreacting, but I really cannot predict if these small differences wouldn't hurt us in the future.13:09
smcginnisThough "heck" wasn't usually the word I used.13:09
smcginnisdulek: Yeah, I appreciate that!13:10
*** xyang1 has joined #openstack-cinder13:10
dulekWith all the DBs implementation specifics… It's just impossible to tell.13:10
smcginnisdulek: I did want to get this in early for that reason. The longer we have in the cycle to shake out any gotchas the better.13:10
dulekgeguileo_phone knows something about implementation specifics of DBs. :D13:10
smcginnisdulek: Hah! I'm sure more than I care to know.13:11
smcginnisjgriffith: You cool if I just set the index names? Or do you still have concerns with that?13:12
*** nkrinner has quit IRC13:14
*** laughterwym has joined #openstack-cinder13:16
*** nkrinner has joined #openstack-cinder13:19
*** diablo_rojo has joined #openstack-cinder13:19
*** dustins has joined #openstack-cinder13:20
*** laughterwym has quit IRC13:20
zigoe0ne: Around?13:22
e0nezigo: yes13:22
*** faiz89 has joined #openstack-cinder13:22
zigoHi there! Running tempest on Newton b1 of Cinder, I have a small issue. The test:13:22
zigotempest.api.volume.admin.test_volume_services.VolumesServicesV1TestJSON.test_get_service_by_host_name13:22
zigofails with this return:13:22
zigo    reference = [u'cinder-backup', u'cinder-scheduler', u'cinder-volume']13:22
zigo    actual    = [u'cinder-backup', u'cinder-scheduler']13:22
zigoI believe the issue is that cinder-volume replies debian@lvm as a host, instead of just "debian" as the others do. How can I fix this?13:22
zigoe0ne: ^13:22
*** jsheeren has quit IRC13:23
*** akshai has joined #openstack-cinder13:23
*** nkrinner has quit IRC13:24
e0nezigo: do you use latest tempest?13:25
zigoe0ne: I use version 12.0.0, and I had a look, that piece of code hasn't changed, seemingly.13:25
*** timcl has quit IRC13:26
zigoI may be wrong.13:26
zigo(if one of the functions used in the test case has changed, I didn't check for it)13:26
*** takedakn has joined #openstack-cinder13:26
e0nezigo: give me few minutes to take a look on it13:27
zigoe0ne: Oh, I found something...13:28
zigoFrom tempest.conf: [volume-feature-enabled]/volume_services13:29
zigoComments say: Extract correct host info from host@backend (boolean value)13:29
zigoI think I should try that.13:29
*** merooney has joined #openstack-cinder13:30
openstackgerritMichal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls  https://review.openstack.org/25245813:30
openstackgerritMichal Dulko proposed openstack/cinder: Switch request_spec in create volume calls to ovo  https://review.openstack.org/24660013:30
openstackgerritMichal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO  https://review.openstack.org/25303313:30
openstackgerritMichal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype)  https://review.openstack.org/25730513:30
*** mriedem has joined #openstack-cinder13:31
*** timcl has joined #openstack-cinder13:31
*** kongwei has joined #openstack-cinder13:31
*** mtanino has quit IRC13:35
*** dustins has quit IRC13:35
*** kongwei has quit IRC13:39
e0nezigo: ok. let me know please, if it helps you13:39
eantysheve0ne, jgriffith: Thank you for +2 and workflow on https://review.openstack.org/#/c/262246 . But it depends on a refactoring change: https://review.openstack.org/261558 (improvement in unitttests). could you please look at it?13:39
openstackgerritStefan Amann proposed openstack/os-brick: Support FICON-attached ECKD volumes on System z  https://review.openstack.org/32053613:40
scottdadulek: I'm thinking that we could take the same approach for multi-node gate testing as for multi-backend, i.e create an experimental job and try to get those project-config changes through. What do you think?13:40
*** dansmith is now known as superdan13:41
*** cdelatte has joined #openstack-cinder13:42
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/32967713:44
openstackgerritSean McGinnis proposed openstack/cinder: Correct DB modifications from compaction  https://review.openstack.org/33075613:44
*** porrua has joined #openstack-cinder13:46
*** links has quit IRC13:46
*** cdelatte has quit IRC13:49
*** itzdilip has quit IRC13:49
*** xinli has joined #openstack-cinder13:50
*** ametts has joined #openstack-cinder13:50
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring  https://review.openstack.org/28546913:50
*** e0ne has quit IRC13:51
*** e0ne has joined #openstack-cinder13:52
*** sgotliv_ has joined #openstack-cinder13:57
*** mriedem has quit IRC13:58
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file  https://review.openstack.org/29407613:58
xinligeguileor: could you please help to review this patch 251989 (VersionObject) : https://review.openstack.org/#/c/251989/13:58
*** mriedem has joined #openstack-cinder13:59
dulekxinli: geguileo is on vacations right now. I remember about your patch, just couldn't get to it lately due to DB migrations issues we've had.14:02
xinlidulek: thank you very much for your response. I have resubmitted another patchset (22) due to code merging conflict. if you have time, please help to review the patch.14:05
*** porrua has quit IRC14:06
*** cdelatte has joined #openstack-cinder14:06
yuriy_n17DuncanT: Hi14:11
DuncanTHi14:11
yuriy_n17DuncanT: Very sorry for disturbing. Please take a look at https://review.openstack.org/#/c/279202/ whenever you have free time.14:12
*** sdake has joined #openstack-cinder14:12
DuncanTCertainly14:13
yuriy_n17DuncanT: Thanks.14:13
DuncanTAh yes, the API working group approved this, didn't they?14:13
yuriy_n17DuncanT: Sure, they approved and that patch was merged.14:14
*** cdelatte has quit IRC14:14
DuncanTOk. I'll get to this one shortly, that was all I was waiting for, then forgot to follow up.14:14
*** nikeshm has quit IRC14:15
*** zul has quit IRC14:16
*** zul has joined #openstack-cinder14:16
*** hemanthm|afk is now known as hemanthm14:20
*** zul_ has joined #openstack-cinder14:21
*** mtanino has joined #openstack-cinder14:21
*** nikeshm has joined #openstack-cinder14:22
*** zul has quit IRC14:23
*** arecknag has quit IRC14:24
*** cdelatte has joined #openstack-cinder14:31
*** david-lyle has joined #openstack-cinder14:32
*** e0ne has quit IRC14:34
*** e0ne has joined #openstack-cinder14:35
*** cdelatte has quit IRC14:37
e0neeantyshev: did you investigate why quobyte and netapp ci failed on your patch? it doesn't look related to you changes14:41
*** ihrachys has quit IRC14:45
openstackgerritSean McGinnis proposed openstack/cinder: Correct DB modifications from compaction  https://review.openstack.org/33075614:48
*** cdelatte has joined #openstack-cinder14:48
*** krtaylor has joined #openstack-cinder14:53
eantysheve0ne: honestly, no14:55
*** daneyon has joined #openstack-cinder14:55
eantysheve0ne: quobyte CI, n-cpu.log: "Could not open '/opt/stack/data/nova/instances/8a395dbd-efa4-490d-b81d-52f0f3bc7e9f/console.log': Permission denied"14:56
eantysheve0ne: doesn't look like consequence of our refactoring14:57
*** laughterwym has joined #openstack-cinder15:00
*** gouthamr has quit IRC15:01
*** gouthamr has joined #openstack-cinder15:01
*** merooney_ has joined #openstack-cinder15:01
e0neeantyshev: I'm agree with you, just wanted to be sure15:02
*** diogogmt has joined #openstack-cinder15:02
smcginnisdulek: Thanks for all your help with checking on that patch!15:03
*** merooney has quit IRC15:04
*** diogogmt has quit IRC15:04
eantysheve0ne: Netapp CI also doesn't seem related: http://paste.openstack.org/show/51688415:05
duleksmcginnis: I'm happy we've got it right now (hopefully ;)).15:05
e0neeantyshev: thanks for the verification15:05
e0neeantyshev: I'll review your patch later today15:05
*** faiz89 has quit IRC15:05
*** tesseract has quit IRC15:05
*** rcernin has quit IRC15:07
smcginnisdulek: (fingers crossed)15:07
openstackgerritPavlo Shchelokovskyy proposed openstack/cinder: Fail to list backups of no backup services  https://review.openstack.org/33121715:07
*** faiz89 has joined #openstack-cinder15:08
*** bjornar_ has quit IRC15:10
*** faiz89 has quit IRC15:13
*** jgregor has joined #openstack-cinder15:19
*** kongwei has joined #openstack-cinder15:19
*** sdake has quit IRC15:20
*** jseiler__ has quit IRC15:21
*** yangyapeng has joined #openstack-cinder15:22
*** porrua has joined #openstack-cinder15:22
*** yangyapeng has quit IRC15:22
*** yangyapeng has joined #openstack-cinder15:22
*** kongwei has quit IRC15:23
*** cdelatte has quit IRC15:24
*** afazekas|dentist is now known as afazekas15:25
*** cdelatte has joined #openstack-cinder15:28
*** timcl has quit IRC15:30
*** sgotliv_ has quit IRC15:31
*** pgadiya_ has quit IRC15:32
*** rajinir has joined #openstack-cinder15:32
*** yangyapeng has quit IRC15:34
*** yuriy_n17 has quit IRC15:34
*** mriedem has quit IRC15:35
*** jdillaman has quit IRC15:37
*** mriedem has joined #openstack-cinder15:38
*** xinli has left #openstack-cinder15:39
openstackgerritxiexs proposed openstack/cinder: Move _check_metadata_properties to cinder/utils  https://review.openstack.org/33050215:39
openstackgerritxiexs proposed openstack/cinder: Add validation for the metadata properties  https://review.openstack.org/33091315:39
*** dustins has joined #openstack-cinder15:39
*** xinli has joined #openstack-cinder15:39
*** belmoreira has quit IRC15:40
*** arch-nemesis has joined #openstack-cinder15:42
*** karthikp has joined #openstack-cinder15:50
*** sdake has joined #openstack-cinder15:51
*** yangyapeng has joined #openstack-cinder15:53
*** karthikp has quit IRC15:54
*** daneyon has quit IRC15:56
*** Poornima_K has quit IRC15:57
*** adisky has quit IRC16:00
nikeshmtrying FC driver  http://paste.openstack.org/show/516901/  http://paste.openstack.org/show/516903/  http://paste.openstack.org/show/516904/  attach volume is failing in initialize connection16:02
nikeshmProcessExecutionError: Unexpected error while running command.oslo.privsep.daemon privsep log: Command: /lib/udev/scsi_id --page 0x83 --whitelisted /dev/disk/by-path/pci-0000:03:00.1-fc-0x50024f40d6030101-lun-116:02
*** permalac has quit IRC16:03
openstackgerritMerged openstack/cinder-specs: Deleting volume metadata keys with a single request  https://review.openstack.org/27920216:03
openstackgerritMerged openstack/cinder: Update manage_existing to use volume object  https://review.openstack.org/25813216:05
*** sdake has quit IRC16:07
*** jungleboyj has joined #openstack-cinder16:09
jungleboyjsmcginnis: Any reason to not merge this once CI passes?  https://review.openstack.org/#/c/330756/16:09
*** mdavidson has quit IRC16:11
*** ivase has joined #openstack-cinder16:12
*** timcl has joined #openstack-cinder16:15
*** xinli has quit IRC16:15
*** cknight has joined #openstack-cinder16:21
*** takedakn has quit IRC16:25
*** faiz89 has joined #openstack-cinder16:26
*** e0ne has quit IRC16:27
*** geguileo_phone has quit IRC16:27
*** diablo_rojo has quit IRC16:28
*** geguileo_phone has joined #openstack-cinder16:28
*** chris_morrell has joined #openstack-cinder16:29
*** mriedem is now known as mriedem_lunch16:30
jgriffitheharney: ping16:31
-openstackstatus- NOTICE: zuul was restarted for a software upgrade; events between 16:08 and 16:30 were missed, please recheck any changes uploaded during that time16:32
*** ebalduf has joined #openstack-cinder16:33
*** jungleboyj has quit IRC16:33
adrianofrsmcginnis: ping16:34
*** Apoorva has joined #openstack-cinder16:36
*** Apoorva has quit IRC16:37
smcginnisadrianofr: pong16:37
adrianofrsmcginnis: I didn't understand your comment regarding the decorator on https://review.openstack.org/#/c/285469/16:38
adrianofrsmcginnis: it already has the decorator added by your patch16:38
adrianofrsmcginnis: anything else to be done?16:38
smcginnisadrianofr: Oh, I could have sworn I looked and it didn't/16:39
smcginnisadrianofr: Sorry, I must have been looking at an older patch rev.16:39
smcginnisadrianofr: Just ignore me. :)16:39
adrianofrsmcginnis: Np  :P16:39
adrianofrsmcginnis: ops...  :)16:40
smcginnisadrianofr: ;)16:40
adrianofrsmcginnis: another question: a release note about a code refactoring should be described on "others" section?16:40
smcginnisadrianofr: Oh, wait...16:42
smcginnisadrianofr: Just ignore me completely.16:42
smcginnisadrianofr: That patch was listed here as a new driver: https://etherpad.openstack.org/p/cinder-spec-review-tracking16:42
smcginnisadrianofr: But it's not.16:42
smcginnisadrianofr: So no need for a release note on that.16:42
*** karthikp has joined #openstack-cinder16:43
claudiubhemna: hello. can you take a look at the os-brick Windows iSCSI / fibre channel / SMB connectors? lpetrut has posted tempest test results for each of them:  https://review.openstack.org/#/c/272522/2916:44
*** karthikp_ has joined #openstack-cinder16:45
*** karthikp_ has quit IRC16:45
adrianofrsmcginnis: Oh, I see. We put it there because it's equivalent to a new driver review (trying to enforce cores to look at it)16:45
*** karthikp_ has joined #openstack-cinder16:45
adrianofrsmcginnis: It was done during the meeting16:45
adrianofrsmcginnis: any problem with this?16:45
*** jungleboyj has joined #openstack-cinder16:47
*** karthikp_ has quit IRC16:47
*** karthikp has quit IRC16:47
smcginnisadrianofr: I'd rather just have new drivers listed there as they have a shorter timeframe.16:48
smcginnisadrianofr: Updates to existing drivers have until n-3, but new drivers need to be added by n-2.16:49
*** bardia has joined #openstack-cinder16:49
adrianofrsmcginnis: OK. I'm going to remove it from the list then.16:51
*** manjrem has joined #openstack-cinder16:51
smcginnisadrianofr: Thanks!16:52
smcginnisadrianofr: I'll still try to review it and we can likely get it in earlier than that.16:53
*** harlowja has joined #openstack-cinder16:53
adrianofrsmcginnis: Thanks  :)16:56
openstackgerritClinton Knight proposed openstack/cinder: Capability lists in Cinder scheduler  https://review.openstack.org/33125816:59
*** faiz89 has quit IRC17:01
*** faiz89 has joined #openstack-cinder17:02
*** Apoorva has joined #openstack-cinder17:02
hemnaclaudiub, sure17:02
openstackgerritClinton Knight proposed openstack/cinder: Capability lists in Cinder scheduler  https://review.openstack.org/33125817:04
*** akerr has quit IRC17:05
*** ccesario has quit IRC17:05
*** sheel has quit IRC17:05
*** xyang1 has quit IRC17:08
*** xyang1 has joined #openstack-cinder17:08
*** laughterwym has quit IRC17:08
*** laughterwym has joined #openstack-cinder17:09
*** mvk_ has joined #openstack-cinder17:11
*** e0ne has joined #openstack-cinder17:12
*** faiz891 has joined #openstack-cinder17:13
*** faiz89 has quit IRC17:13
eharneyjgriffith: pong17:13
*** laughterwym has quit IRC17:13
jgriffitheharney: hey... I was remembering a while back I think you cleaned up a bunch of the spewing of log info in the unit tests17:15
*** mkrcmari__ has quit IRC17:15
*** GB21 has joined #openstack-cinder17:15
jgriffitheharney: mtreinish pointed out last night that cloudbyte is spewing info currently:  http://logs.openstack.org/43/323343/4/check/gate-cinder-python27-db/80036ea/console.html#_2016-06-16_05_29_26_77153517:15
jgriffitheharney: thought I'd ping you and see if you remembered off the top of your head how to clean those things up :)17:16
jgriffitheharney: or at least if you had some helpful tips for me :)17:16
mtreinishjgriffith: there were other things leaking through to stdout, but I thought a stacktrace was more suspicious17:16
*** daneyon has joined #openstack-cinder17:16
jgriffithmtreinish: yeah, I noticed a few things.  But in the case of cloudbyte it appears to be pretty much all of their logging17:17
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring  https://review.openstack.org/28546917:17
jgriffithmtreinish: that particular test is expecting exceptions... although not that particular one :)17:17
mtreinishheh17:18
eharneyjgriffith: hmm, i'm not sure what that driver/test is doing differently from any others17:18
jgriffithyeah, me neither17:19
*** daneyon_ has joined #openstack-cinder17:19
*** ccesario has joined #openstack-cinder17:19
eharneyjgriffith: oh, i see now -- it inherits from testtools directly instead of cinder's test.TestCase17:20
eharneyjgriffith: so it's not getting the right fixtures set up probably17:20
eharneyclass CloudByteISCSIDriverTestCase(testtools.TestCase):17:20
*** faiz891 has left #openstack-cinder17:21
*** faiz891 has joined #openstack-cinder17:21
jgriffithahhh.. indeed17:21
*** daneyon has quit IRC17:22
*** dustins has quit IRC17:23
*** e0ne has quit IRC17:24
*** dustins has joined #openstack-cinder17:24
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file  https://review.openstack.org/29407617:27
openstackgerritMerged openstack/cinder: Fix db purge for volume_types FK constraint  https://review.openstack.org/33032817:27
nikeshmattach volume worked, i did mistake in initiator-target mapping17:30
nikeshmone doubt, i saw multipath=False in connector properties, how can we make it True17:31
nikeshmin initialize_connection17:31
nikeshmfor attach_volume17:31
hemnanikeshm, that is set by 2 factors17:32
hemna1) if multipath is enabled in nova.conf  [libvirt] section17:32
hemna2) multipath daemon is also running.17:33
hemnaeither of those 2 aren't set, then multipath=False17:33
*** jungleboyj has quit IRC17:33
nikeshmhemna: is this is the option "iscsi_use_multipath" we have to use under [libvirt], what about fiber channel17:35
hemnanikeshm, yes17:35
hemnait's the same setting for iscsi and fc17:35
jgriffitheharney: mtreinish that was it17:36
jgriffitheharney: thanks for that17:36
jgriffitheharney: mtreinish I'll post a patch for it17:36
nikeshmhemna: thanks, shall we not use generic name for this option,   may be use_multipath17:36
hemnawe'd have to change nova to do that17:37
hemnaand deprecate the existing and change the name17:37
hemnameh17:37
openstackgerritMerged openstack/cinder: Add parameter info for manageable snapshots  https://review.openstack.org/32866817:37
hemnabut sure.  I can put together a patch for that17:37
openstackgerritMerged openstack/cinder: Move test from test_glusterfs and test_quobyte to test_remotefs  https://review.openstack.org/26154917:37
nikeshmhemna:  thanks17:38
*** sdake has joined #openstack-cinder17:40
openstackgerritMerged openstack/cinder: Use versionedobjects in remotefs.py  https://review.openstack.org/26155817:43
openstackgerritMerged openstack/cinder: vzstorage: add ploop volume format support  https://review.openstack.org/26224617:43
*** lpetrut has quit IRC17:43
jgriffitheharney: bahhh... although that exposed some other issues :(  Like several hundred lock files littered about the working dir17:43
*** pauloewerton has quit IRC17:43
openstackgerritMerged openstack/cinder: VMware:Persist storage policy ID in vmdk meta-data  https://review.openstack.org/33091117:44
*** GB21 has quit IRC17:45
jgriffithhmmmm17:48
jgriffithanybody try running unit tests locally from master recently?17:48
smcginnisFairly recently.17:49
jgriffithOh... I bet I know what broke it17:49
*** dustins has quit IRC17:49
jgriffithsmcginnis: so if you run currrent master you get a bunch of lock files17:49
jgriffithI think it's that patch I just approved last night that replaces them with the tooz stuff17:49
smcginnisjgriffith: Didn't we have that once before and fixed it?17:49
smcginnisjgriffith: Ah, sounds like a likely culprit.17:49
*** pauloewerton has joined #openstack-cinder17:50
*** kfarr has quit IRC17:50
*** Suyash has joined #openstack-cinder17:50
openstackgerritJohn Griffith proposed openstack/cinder: Revert "Replace locks in volume manager"  https://review.openstack.org/33128217:53
jgriffithsmcginnis: yeah17:53
*** manjrem has quit IRC17:57
*** dustins has joined #openstack-cinder17:57
*** timcl1 has joined #openstack-cinder17:57
*** mriedem_lunch is now known as mriedem17:59
*** timcl has quit IRC18:00
*** harlowja has quit IRC18:04
*** jungleboyj has joined #openstack-cinder18:07
openstackgerritJohn Griffith proposed openstack/cinder: Use cinders test class in cloudbyte unit tests  https://review.openstack.org/33128718:08
jgriffithpatrickeast: hehe.. yeah... what's up with that!18:10
jgriffith:)18:10
openstackgerritJohn Griffith proposed openstack/cinder: Revert "Replace locks in volume manager"  https://review.openstack.org/33128218:11
smcginnisThat's a little easier to read. :)18:11
patrickeast haha18:11
hemnaoopsie18:12
patrickeasti was tempted to just edit it in gerrit18:12
hemnadie gerrit die18:12
smcginnisIt's really too bad you can't update commit messages in gerrit without it needing to kick off new jobs.18:12
jgriffithsmcginnis: yeah... didn't the last version detect that?18:13
jgriffithI could've sworn it did18:13
jgriffiththat and trivial rebase18:13
*** timcl1 has quit IRC18:13
*** timcl has joined #openstack-cinder18:14
smcginnisCan't remember now.18:15
jgriffithsmcginnis: you're getting as bad as me :)18:15
smcginnisI just miss the "f" shortcut key to switch between files while reviewing.18:15
smcginnisjgriffith: Haha, I suppose. :)18:15
openstackgerritTom Swanson proposed openstack/cinder: WIP: Dell SC: Use Live Volume for replication  https://review.openstack.org/32953118:23
*** mkrcmari__ has joined #openstack-cinder18:25
*** timcl has quit IRC18:26
*** timcl has joined #openstack-cinder18:26
*** ChubYann has joined #openstack-cinder18:27
*** mvk_ has quit IRC18:28
*** karthikp_ has joined #openstack-cinder18:40
*** simondodsley has joined #openstack-cinder18:40
*** dustins has quit IRC18:43
*** timcl has quit IRC18:46
*** porrua has quit IRC18:47
*** timcl has joined #openstack-cinder18:48
*** Lee1092 has quit IRC18:53
*** locust has joined #openstack-cinder18:54
*** locust has quit IRC19:00
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: [WIP] Switch service capabilities to ovo  https://review.openstack.org/31904019:00
*** e0ne has joined #openstack-cinder19:08
*** dustins has joined #openstack-cinder19:08
*** karthikp has joined #openstack-cinder19:10
*** geguileo_phone has quit IRC19:12
*** karthikp has quit IRC19:12
*** shakamunyi has joined #openstack-cinder19:13
*** karthikp has joined #openstack-cinder19:15
*** faiz891 has quit IRC19:22
*** faiz89 has joined #openstack-cinder19:22
*** geguileo_phone has joined #openstack-cinder19:25
*** dustins has quit IRC19:26
*** harlowja has joined #openstack-cinder19:32
*** gouthamr has quit IRC19:32
*** ociuhandu has quit IRC19:34
bluexjgriffith: it seems volume manager locks patch was reverted but it's not yet visible in repo, should I start new patch replacing locks or create simple 2 line fix for current implementation?19:37
jgriffithbluex: hey19:38
jgriffithbluex: hasn't merged yet19:38
jgriffithif you have a fix that's that easy post it19:38
jgriffithbluex: I can revert my revert :)19:38
jgriffithbluex: make sense?19:40
bluexsure, I'll send fix, give me 10 min19:40
*** merooney_ has quit IRC19:40
*** cdelatte has quit IRC19:40
jgriffithbluex: awesome19:41
*** rajinir has quit IRC19:44
*** mriedem has quit IRC19:46
openstackgerritAlyson proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages  https://review.openstack.org/31078519:47
smcginnisGood, I was hoping that could just get fixed rather than starting over.19:48
*** pauloewerton has quit IRC19:49
*** Apoorva has quit IRC19:49
*** lpetrut has joined #openstack-cinder19:52
*** mriedem has joined #openstack-cinder19:53
*** e0ne has quit IRC19:53
*** timcl has quit IRC19:54
openstackgerritSzymon Wróblewski proposed openstack/cinder: Mock tooz coordinator for tests  https://review.openstack.org/33132519:54
*** markvoelker has quit IRC19:55
jgriffithbluex: smcginnis cool, looks like that should do the trick!19:56
jgriffithhmm... test fail19:57
*** raildo is now known as raildo-afk19:57
jgriffithbluex: oh... think we need to remove the mocks you had in test_volumes.py19:57
openstackgerritAlyson proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages  https://review.openstack.org/31078520:01
*** dustins has joined #openstack-cinder20:03
jgriffithbluex: hmm, seems to be some side efffects... let me checkout a fresh env20:04
bluexhmm, tests passed locally, I have to check again, test_volumes mock only cinder.coordination.Coordinator.get_lock and fix I posted mock tooz.coordination.get_coordinator which should override any reference to tooz used by cinder.coordination20:05
jgriffithbluex: let me make sure it's not just something on my side that's messed up20:05
*** daneyon_ has quit IRC20:06
jgriffithbluex: http://logs.openstack.org/25/331325/1/check/gate-cinder-python27-db/35b85ad/console.html#_2016-06-17_20_03_14_61607320:09
*** markvoelker has joined #openstack-cinder20:10
*** daneyon has joined #openstack-cinder20:10
jgriffithbluex: oh... actually probably failing because of the mock20:12
*** bardia has quit IRC20:12
*** karthikp has quit IRC20:13
jgriffithbluex: so those two tests actually check to make sure creates fail because the lock is set20:13
*** bardia has joined #openstack-cinder20:14
jgriffithso the mock kinda screws those up20:14
*** karthikp has joined #openstack-cinder20:14
*** gouthamr has joined #openstack-cinder20:14
bluexyeah, I mocked whole locking behavior too, locks are not created so files are not created and nothing is really locked20:15
*** akshai has quit IRC20:18
*** Apoorva has joined #openstack-cinder20:18
jgriffithbluex: right, so those that were actually testing the locks get a bit grumpy :)20:19
SwansonJenkins should just fix errors like W391 blank line at end of file20:19
*** karthikp has quit IRC20:20
*** karthikp has joined #openstack-cinder20:22
openstackgerritKendall Nelson proposed openstack/cinder: Add VolumeAttachStatus Enum  https://review.openstack.org/29088520:22
nikeshmsmcginnis:  https://review.openstack.org/#/c/327285/ is not showing in gate20:26
nikeshmeven after reverify20:27
nikeshmit went in check pipeline and passed20:27
nikeshmbut not going in gate pipeline20:27
nikeshmis any issue in gate20:27
smcginnisnikeshm: Hmm. And it's not dependent on another patch?20:27
nikeshmit is independent20:29
*** karthikp has quit IRC20:29
openstackgerritMerged openstack/cinder: Correct DB modifications from compaction  https://review.openstack.org/33075620:31
openstackgerritMerged openstack/cinder: Use cinders test class in cloudbyte unit tests  https://review.openstack.org/33128720:31
*** karthikp has joined #openstack-cinder20:32
*** dustins has quit IRC20:33
nikeshmsmcginnis: it is not dependent on any patch20:33
*** mvk_ has joined #openstack-cinder20:34
*** karthikp has quit IRC20:37
*** mkrcmari__ has quit IRC20:38
nikeshmsmcginis xyang: i am going to sleep, please check when you get time that why it is not entering in gate pipeline20:41
smcginnisnikeshm: It's going now.20:42
smcginnisnikeshm: Should be merged by the time you wake up. ;)20:42
nikeshmwhat was issue20:42
nikeshmgood night20:44
smcginnisGerrit irregularity.20:45
*** csky has quit IRC20:46
*** jungleboyj has quit IRC20:50
*** crose has joined #openstack-cinder20:52
*** mkrcmari__ has joined #openstack-cinder20:54
*** crose has quit IRC20:57
*** faiz89 has quit IRC20:57
*** mvk_ has quit IRC20:57
hemnadie gerrit die20:58
*** karthikp has joined #openstack-cinder21:00
*** ivase has quit IRC21:01
*** mvk has joined #openstack-cinder21:01
*** mkrcmari__ has quit IRC21:02
openstackgerritTom Swanson proposed openstack/cinder: WIP: Dell SC: Use Live Volume for replication  https://review.openstack.org/32953121:02
*** karthikp has quit IRC21:06
*** karthikp has joined #openstack-cinder21:09
*** cknight has quit IRC21:10
*** ametts has quit IRC21:13
bluexjgriffith: I have two solutions - question is whether we want to use threading.Lock on top of tooz locks - if yes I'll change implementation of locks in cinder.coordination if not I'll just mock tooz get_lock for tests to return named threading.Lock21:21
*** markvoelker has quit IRC21:21
*** gouthamr has quit IRC21:22
jgriffithbluex: well... you're the tooz impl expert at this point :)21:24
jgriffithbluex: I can't say as I understand the pros/cons of the coordination impl21:25
jgriffithbluex: it does seem that not using threading.lock causes some weirdness for us21:25
*** karthikp_ has quit IRC21:25
*** dave-mccowan has quit IRC21:26
bluexPrevious implementation used threading.lock on top of file locks, current implementation uses only tooz locks, tests assumed only local (threading) locks are used so no files was created21:28
*** lascii is now known as alaski21:28
bluexI remember some previous implementation of coordination module also used threading.lock, but I have to check again what was the reason to not use that in final implementation21:29
*** erlon has quit IRC21:36
bluexOk, as I see it - additional threading.lock would be useful only when connection with coordination service is dropped, can't be restored and we would want node to continue locked operation, but as we don't want this situation to ever happen - it wouldn't add anything21:43
bluexAnd with this I think I answered my question :)21:43
*** xyang1 has quit IRC21:48
jgriffithbluex: :)21:50
openstackgerritVasanthi Thirumalai proposed openstack/cinder: Violin Memory ISCSI storage for 7000 series AFA  https://review.openstack.org/26979621:52
*** alyson_ has quit IRC21:58
openstackgerritMerged openstack/cinder: Provide Kaminario K2 all-flash array iSCSI driver  https://review.openstack.org/32728521:58
*** mriedem has quit IRC21:59
*** lpetrut has quit IRC22:04
*** karthikp has quit IRC22:06
openstackgerritSzymon Wróblewski proposed openstack/cinder: Mock tooz coordinator for tests  https://review.openstack.org/33132522:11
openstackgerritSzymon Wróblewski proposed openstack/cinder: Mock tooz coordinator for tests  https://review.openstack.org/33132522:22
*** tpsilva has quit IRC22:27
openstackgerritSzymon Wróblewski proposed openstack/cinder: Fix lock files littering working dir during tests  https://review.openstack.org/33132522:33
*** sdake has quit IRC22:38
*** mvk_ has joined #openstack-cinder22:39
*** sdake has joined #openstack-cinder22:41
*** adrianofr has quit IRC22:42
*** mvk has quit IRC22:42
*** dustins has joined #openstack-cinder22:58
*** edmondsw has quit IRC22:59
jgriffithbluex: thanks!  Seems to be a problem with py34 maybe though23:09
jgriffithbut at least the other problems appear to be fixed up23:09
*** IlyaG has joined #openstack-cinder23:13
*** markvoelker has joined #openstack-cinder23:16
openstackgerritRodrigo Freire proposed openstack/os-brick: Multipath: fix race when detaching devices  https://review.openstack.org/33137523:18
*** dave-mccowan has joined #openstack-cinder23:22
*** karthikp has joined #openstack-cinder23:25
*** dave-mccowan has quit IRC23:26
*** dustins has quit IRC23:26
*** karthikp has quit IRC23:30
*** karthikp has joined #openstack-cinder23:31
*** claudiub has quit IRC23:38
*** eharney has quit IRC23:46
*** karthikp has quit IRC23:47
*** karthikp has joined #openstack-cinder23:48
*** asselin_ has quit IRC23:49
*** Suyash has quit IRC23:53

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