Tuesday, 2015-11-10

*** haomaiwang has quit IRC00:01
*** haomaiwang has joined #openstack-cinder00:01
*** markvoelker_ has quit IRC00:01
*** cdelatte has quit IRC00:04
*** zhangjn has joined #openstack-cinder00:05
*** ndipanov has joined #openstack-cinder00:06
*** setmason has left #openstack-cinder00:07
*** xyang1 has quit IRC00:15
*** EinstCrazy has quit IRC00:15
*** zhangjn has quit IRC00:16
*** mc_nair has left #openstack-cinder00:17
*** mc_nair has joined #openstack-cinder00:17
*** harlowja has joined #openstack-cinder00:20
*** mudassirlatif has quit IRC00:24
*** mtanino has quit IRC00:28
*** gouthamr_ has quit IRC00:28
*** breitz has quit IRC00:32
*** dims_ has quit IRC00:33
*** jamielennox is now known as jamielennox|away00:33
*** dims has joined #openstack-cinder00:36
patrickeasthemna: ping00:40
*** IlyaG has joined #openstack-cinder00:46
*** timcl has joined #openstack-cinder00:47
*** timcl has quit IRC00:49
hemnahey00:50
hemnapatrickeast,00:50
patrickeasthemna: hey so i was hoping to get your opinion on os-brick + cinder testing00:50
hemnaok00:50
patrickeastso my ci system right now uses the pypi os-brick for cinder jobs00:51
hemnayah as does ours I believe00:51
patrickeastbuut things like that fc permission fix are still causing problems for them00:51
hemnafor cinder patches00:51
*** dims_ has joined #openstack-cinder00:51
*** dims has quit IRC00:51
hemnaand the patch in brick that landed fixes it ?00:51
patrickeastand it makes me wonder if maybe it should just be using tip of tree os-brick for cinder master branch00:51
*** setmason has joined #openstack-cinder00:51
patrickeastyea i don't think i've seen it happen on the os-brick jobs since00:51
hemnaI think that's what the os-brick CI stuff is supposed to vet out00:52
patrickeasti have seen some new fun cisco fczm stuff00:52
patrickeastso whats the timing then for releases?00:52
hemnawhenever really00:52
hemnaI think as needed00:52
patrickeastwhat i'm wondering is if it is really better to test the older os-brick one against cinder master00:52
patrickeastif that isn't ever what would be released when cinder makes a cut00:53
jgriffithFWIW I pull from master00:53
hemnaat a minimum, I think we should evaluate doing drops to pypi every milestone00:53
hemnabut I think that's a bit long for issues like bug fixes directly related to CI00:53
jgriffithpatrickeast: hemna I've actually been torn on whether I should do that or not though00:53
patrickeastif i had more hardware i would just test both >.<00:53
hemnaheh00:53
hemnayah00:53
patrickeastjgriffith: have you run into any issues using the latest os-brick?00:54
patrickeastim always a little gun-shy about diverging my ci config from the jenkins one00:54
hemnasame here00:54
jgriffithpatrickeast: Not really... seems solid00:54
hemnawe fight FC passthrough crap constantly00:54
jgriffithpatrickeast: I did start seeing some failures but did a reset on my cluster and haven't seen them since00:55
patrickeastok cool00:55
jgriffithpatrickeast: hemna I'll keep an eye on it the next couple days and shout if I see anything weird00:55
hemnapatrickeast, so maybe a run for a day on master and see what happens ?00:55
patrickeastwell, maybe for a while i'll try out pulling os-brick from git00:55
patrickeastcouldn't be any worse ;)00:55
patrickeastwell00:55
hemnamaster is supposed to work00:55
patrickeastthats not true, it could be so much worse00:55
patrickeasthemna: haha yea00:56
hemnasince we have os-brick CI as well00:56
patrickeastit works better for me actually00:56
patrickeastwell00:56
patrickeastin my os-brick ci job00:56
jgriffithpatrickeast: hehe.. that's why I switched to it TBH :)00:56
hemnathe only downside is that if a brick patch lands that causes unforseen issues....00:57
hemnabut I suppose that's also a good thing as well00:57
patrickeastyea00:57
patrickeastprobably better for alarms to go off00:57
hemnabut the os-brick CI should be catching those.00:57
patrickeastyea00:57
patrickeastmaking the switch means it would basically be the same job now for my system on os-brick and cinder00:58
patrickeastso it shouldn't have too many suprises00:58
* patrickeast hopes00:58
hemnaand by the time we push to pypi00:58
hemnathe pypi release should be blessed.00:58
hemnaI'm working on new brick stuffs now00:58
*** zhangjn has joined #openstack-cinder00:59
hemnaand just fixed a unit test problem today in it00:59
hemnahttps://review.openstack.org/#/c/243338/00:59
hemnathat00:59
* hemna isn't thrilled with the unit tests00:59
patrickeasthaha00:59
hemnaI'm close to getting the extend volume stuff working01:00
hemnafor attached volumes01:00
hemnaat least the brick code :P01:00
patrickeastoo nice01:00
*** haomaiwang has quit IRC01:01
*** 20WACGE4T has joined #openstack-cinder01:01
hemnapatrickeast, http://paste.openstack.org/show/478399/01:02
hemnaship it!01:02
*** kmartin has quit IRC01:03
*** pv_ has quit IRC01:03
patrickeasthaha01:03
patrickeast+101:03
*** EinstCra_ has joined #openstack-cinder01:05
*** crose has quit IRC01:05
*** lei-zh has joined #openstack-cinder01:06
*** asselin__ has joined #openstack-cinder01:06
patrickeastok pure ci is now using only upstream os-brick01:07
patrickeasthemna: jgriffith: thanks for the input :)01:07
*** asselin_ has quit IRC01:08
hemnanp01:08
hemnahave a good evenin...01:08
*** hemna is now known as hemnafk01:08
*** kmartin has joined #openstack-cinder01:13
*** yangxi has joined #openstack-cinder01:14
*** EinstCra_ has quit IRC01:16
*** EinstCrazy has joined #openstack-cinder01:16
*** Lee1092 has joined #openstack-cinder01:17
*** garthb_ has quit IRC01:19
Lickitysplittedwould anyone happen to have links to info regarding iser in kilo?01:21
Lickitysplittedand yes i've googled the hell out of it01:21
*** markvoelker has joined #openstack-cinder01:23
*** timcl has joined #openstack-cinder01:25
*** timcl has quit IRC01:27
*** zhenguo has joined #openstack-cinder01:28
*** markvoelker has quit IRC01:28
openstackgerritMENJO, Takashi proposed openstack/cinder: Sheepdog: support Cinder multi backend  https://review.openstack.org/24048101:28
*** EinstCra_ has joined #openstack-cinder01:30
*** zhangjn_ has joined #openstack-cinder01:30
*** p0rtal has quit IRC01:31
*** terryyao has joined #openstack-cinder01:32
*** EinstCrazy has quit IRC01:33
*** zhangjn has quit IRC01:33
*** diogogmt has quit IRC01:37
*** dave-mccowan has quit IRC01:42
openstackgerritwanghao proposed openstack/cinder: CG creation should be scheduled on backend level  https://review.openstack.org/22440501:43
*** setmason has quit IRC01:46
*** mc_nair has quit IRC01:54
*** leeantho has quit IRC01:55
*** yangxi has quit IRC01:58
*** 20WACGE4T has quit IRC02:01
*** haomaiwang has joined #openstack-cinder02:01
*** dims_ has quit IRC02:01
*** yangxi has joined #openstack-cinder02:03
*** dims has joined #openstack-cinder02:05
*** haomaiwang has quit IRC02:09
*** salv-orlando has joined #openstack-cinder02:12
*** timcl has joined #openstack-cinder02:12
*** timcl has quit IRC02:14
openstackgerritxing-yang proposed openstack/cinder: ScaleIO driver: update_migrated_volume  https://review.openstack.org/24093702:22
openstackgerritWilson Liu proposed openstack/cinder: Encode username and password in Huawei driver  https://review.openstack.org/21608502:25
*** yangxi has quit IRC02:31
*** diogogmt has joined #openstack-cinder02:31
*** apoorvad has quit IRC02:33
*** yangxi has joined #openstack-cinder02:34
openstackgerritzhangsong proposed openstack/cinder: fix the bug of OSError when convert image  https://review.openstack.org/24315202:34
*** jamielennox|away is now known as jamielennox02:34
*** hemna has joined #openstack-cinder02:40
*** setmason has joined #openstack-cinder02:42
*** hodos has quit IRC02:46
*** haomaiwang has joined #openstack-cinder02:51
*** hemna has quit IRC02:51
*** timcl has joined #openstack-cinder02:52
*** bapalm has quit IRC02:54
*** timcl has quit IRC02:54
*** davechen has joined #openstack-cinder02:56
*** dave-mccowan has joined #openstack-cinder02:57
*** haomaiwang has quit IRC03:01
*** haomaiwang has joined #openstack-cinder03:01
*** hemna has joined #openstack-cinder03:02
*** bapalm has joined #openstack-cinder03:02
*** hemna has quit IRC03:03
*** hemna has joined #openstack-cinder03:10
*** sghanekar has quit IRC03:11
openstackgerritWilson Liu proposed openstack/cinder: Rename Huawei drivers  https://review.openstack.org/23804103:12
*** dims has quit IRC03:13
*** hemna has quit IRC03:14
*** timcl has joined #openstack-cinder03:15
*** timcl has quit IRC03:17
*** hemna has joined #openstack-cinder03:18
*** lixiaoy1 has quit IRC03:18
*** yangxi has quit IRC03:18
*** lixiaoy1 has joined #openstack-cinder03:19
*** hemna has quit IRC03:19
*** dave-mccowan has quit IRC03:22
*** markvoelker has joined #openstack-cinder03:24
*** markvoelker has quit IRC03:28
openstackgerritWilson Liu proposed openstack/cinder: Rename Huawei drivers  https://review.openstack.org/23804103:32
*** yangxi has joined #openstack-cinder03:36
*** haomaiwang has quit IRC03:38
*** houming-wang has joined #openstack-cinder03:38
*** haomaiwang has joined #openstack-cinder03:38
*** IlyaG has quit IRC03:49
smcginnishemnafk, patrickeast: Let me know if I should get a new os-brick release going.03:50
*** xiaohui_ has joined #openstack-cinder03:50
*** xiaohui_ has quit IRC03:54
*** davechen has left #openstack-cinder03:58
*** links has joined #openstack-cinder03:59
*** salv-orlando has quit IRC03:59
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-cinder04:01
*** mriedem_away has quit IRC04:05
*** david-lyle has joined #openstack-cinder04:07
*** martyturner has joined #openstack-cinder04:08
*** martyturner has quit IRC04:13
*** diogogmt has quit IRC04:17
*** setmason has quit IRC04:20
*** yangxi has quit IRC04:27
*** jerrygb has quit IRC04:28
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/24006504:28
openstackgerritxing-yang proposed openstack/cinder: Backup snapshots  https://review.openstack.org/24340604:33
*** jerrygb has joined #openstack-cinder04:36
*** lei-zh has quit IRC04:40
*** lprice has joined #openstack-cinder04:40
*** wilson-1 has quit IRC04:51
*** wilson-1 has joined #openstack-cinder04:53
*** haomaiwang has quit IRC04:55
*** haomaiwang has joined #openstack-cinder04:55
*** salv-orlando has joined #openstack-cinder05:01
*** haomaiwang has quit IRC05:01
*** jamielennox is now known as jamielennox|away05:01
*** 18WAA337V has joined #openstack-cinder05:01
*** salv-orlando has quit IRC05:05
openstackgerritMENJO, Takashi proposed openstack/cinder: Sheepdog: support Cinder multi backend  https://review.openstack.org/24048105:10
*** yangxi has joined #openstack-cinder05:14
*** [2]Thelo has joined #openstack-cinder05:17
*** Thelo has quit IRC05:19
*** [2]Thelo is now known as Thelo05:19
*** jerrygb has quit IRC05:20
*** BharatK_ has joined #openstack-cinder05:21
*** gcb has joined #openstack-cinder05:22
*** BharatK_ has quit IRC05:22
*** jerrygb has joined #openstack-cinder05:23
*** BharatK_ has joined #openstack-cinder05:23
*** jerrygb has quit IRC05:23
*** BharatK has quit IRC05:24
*** garthb has joined #openstack-cinder05:24
*** BharatK_ has quit IRC05:24
*** BharatK_ has joined #openstack-cinder05:25
*** markvoelker has joined #openstack-cinder05:25
*** BharatK_ has quit IRC05:26
*** BharatK_ has joined #openstack-cinder05:27
*** BharatK_ has quit IRC05:28
*** BharatK_ has joined #openstack-cinder05:29
*** markvoelker has quit IRC05:29
*** BharatK_ has quit IRC05:34
*** laughterwym has joined #openstack-cinder05:45
*** garthb has quit IRC05:45
*** Yogi1 has joined #openstack-cinder05:52
*** anshul has joined #openstack-cinder05:52
*** Yogi1 has quit IRC05:56
openstackgerritXi Yang proposed openstack/cinder: Support libgfapi's backing file path format  https://review.openstack.org/24341605:57
*** 18WAA337V has quit IRC06:01
*** 20WACGGAC has joined #openstack-cinder06:01
*** ig0r__ has quit IRC06:05
*** ig0r__ has joined #openstack-cinder06:10
*** setmason has joined #openstack-cinder06:10
*** jgregor has joined #openstack-cinder06:11
*** jgregor has quit IRC06:18
openstackgerritXi Yang proposed openstack/cinder: Remove db access in VNX driver  https://review.openstack.org/23958006:19
*** lei-zh1 has joined #openstack-cinder06:19
*** lpetrut has joined #openstack-cinder06:20
*** jerrygb has joined #openstack-cinder06:24
*** belmoreira has joined #openstack-cinder06:24
*** jerrygb has quit IRC06:28
*** cfriesen__ has quit IRC06:40
*** bswartz has quit IRC06:47
*** lpetrut has quit IRC06:48
*** bswartz has joined #openstack-cinder06:53
*** chenying has quit IRC06:55
*** chenying has joined #openstack-cinder06:55
*** mudassirlatif has joined #openstack-cinder06:56
*** vgridnev has joined #openstack-cinder06:57
*** 20WACGGAC has quit IRC07:01
*** 7GHABMC07 has joined #openstack-cinder07:01
*** yrabl has joined #openstack-cinder07:11
*** yrabl has quit IRC07:15
*** alexpilotti has joined #openstack-cinder07:15
*** nkrinner has joined #openstack-cinder07:20
openstackgerritchenying proposed openstack/cinder: Use oslo_config new type PortOpt for port options  https://review.openstack.org/24343307:20
*** lpetrut has joined #openstack-cinder07:20
openstackgerritDeepti Ramakrishna proposed openstack/python-cinderclient: snapshot-list now supports filtering by tenant  https://review.openstack.org/24239107:25
*** jerrygb has joined #openstack-cinder07:25
*** markvoelker has joined #openstack-cinder07:26
*** jerrygb has quit IRC07:29
*** markvoelker has quit IRC07:31
*** alexpilotti has quit IRC07:35
*** mudassirlatif has quit IRC07:37
*** liverpooler has joined #openstack-cinder07:45
*** openstackgerrit has quit IRC07:46
*** openstackgerrit has joined #openstack-cinder07:47
*** EinstCrazy has joined #openstack-cinder07:52
*** EinstCra_ has quit IRC07:52
*** jwcroppe_ has joined #openstack-cinder07:53
*** jwcroppe_ has quit IRC07:53
*** jwcroppe_ has joined #openstack-cinder07:54
*** yangxi has quit IRC07:55
*** yangxi has joined #openstack-cinder07:55
*** jwcroppe has quit IRC07:55
*** jwcroppe_ has quit IRC07:56
*** jwcroppe has joined #openstack-cinder07:56
*** 7GHABMC07 has quit IRC08:01
*** jwcroppe has quit IRC08:01
*** haomaiwang has joined #openstack-cinder08:01
*** zigo has quit IRC08:01
*** zigo has joined #openstack-cinder08:03
openstackgerritwanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly  https://review.openstack.org/23243608:08
*** haomaiwang has quit IRC08:11
*** haomaiwang has joined #openstack-cinder08:11
*** haomaiwang has quit IRC08:16
*** 18WAA3459 has joined #openstack-cinder08:18
*** EinstCrazy has quit IRC08:19
*** setmason has quit IRC08:19
*** EinstCrazy has joined #openstack-cinder08:19
*** jerrygb has joined #openstack-cinder08:26
*** geguileo_Holiday is now known as geguileo08:28
*** vgridnev has quit IRC08:30
*** jerrygb has quit IRC08:31
*** openstackgerrit has quit IRC08:31
*** openstackgerrit has joined #openstack-cinder08:32
*** vgridnev has joined #openstack-cinder08:32
*** deepakcs has joined #openstack-cinder08:34
*** vgridnev has quit IRC08:38
*** ZZelle has quit IRC08:39
*** ZZelle has joined #openstack-cinder08:39
*** jordanP has joined #openstack-cinder08:41
*** andrewfly has quit IRC08:44
*** laughterwym has quit IRC08:46
*** yusuke has quit IRC08:57
*** 18WAA3459 has quit IRC09:01
*** haomaiwa_ has joined #openstack-cinder09:02
*** EinstCra_ has joined #openstack-cinder09:02
*** EinstCrazy has quit IRC09:05
*** gouthamr has joined #openstack-cinder09:21
openstackgerritwanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly  https://review.openstack.org/23243609:24
*** EinstCra_ has quit IRC09:24
*** EinstCrazy has joined #openstack-cinder09:25
*** lixiaoy1_ has joined #openstack-cinder09:26
*** lixiaoy1 has left #openstack-cinder09:26
*** markvoelker has joined #openstack-cinder09:27
*** lprice has quit IRC09:27
*** jerrygb has joined #openstack-cinder09:27
*** subscope has joined #openstack-cinder09:29
*** markvoelker has quit IRC09:31
*** lprice has joined #openstack-cinder09:32
*** vgridnev has joined #openstack-cinder09:32
*** jerrygb has quit IRC09:32
*** lpetrut has quit IRC09:34
*** terryyao has quit IRC09:35
*** lixiaoy1_ has quit IRC09:40
*** lixiaoy1_ has joined #openstack-cinder09:40
openstackgerritnandal proposed openstack/cinder: Bug Fix : 1514739  https://review.openstack.org/24348909:41
openstackgerritnandal proposed openstack/cinder: Bug Fix : 1514739 Change assertEqual(True/False, value) to assertTrue/False(value)  https://review.openstack.org/24349009:41
*** sgotliv has joined #openstack-cinder09:44
*** jistr has joined #openstack-cinder09:45
*** laughterwym has joined #openstack-cinder09:51
*** zhenguo has quit IRC10:00
*** haomaiwa_ has quit IRC10:01
*** haomaiwa_ has joined #openstack-cinder10:01
*** lei-zh1 has quit IRC10:04
*** can8dnSix has joined #openstack-cinder10:04
*** lei-zh has joined #openstack-cinder10:04
*** aix has joined #openstack-cinder10:04
*** lprice has quit IRC10:07
*** houming-wang has quit IRC10:09
*** yangxi has quit IRC10:09
*** gcb has quit IRC10:09
*** jamielennox|away is now known as jamielennox10:24
*** subscope has quit IRC10:26
*** jerrygb has joined #openstack-cinder10:28
*** EinstCra_ has joined #openstack-cinder10:30
*** EinstCr__ has joined #openstack-cinder10:32
*** ociuhandu has joined #openstack-cinder10:33
*** EinstCrazy has quit IRC10:33
*** lei-zh has quit IRC10:34
*** jerrygb has quit IRC10:34
*** EinstCra_ has quit IRC10:35
*** EinstCr__ has quit IRC10:38
*** sgotliv_ has joined #openstack-cinder10:39
*** yuriy_n17 has joined #openstack-cinder10:39
*** ig0r__ has quit IRC10:40
openstackgerritnandal proposed openstack/cinder: Change assertEqual(True/False, value) to assertTrue/False(value)  https://review.openstack.org/24348910:41
*** sgotliv has quit IRC10:42
openstackgerritnandal proposed openstack/cinder: Change assertEqual(True/False, value) to assertTrue/False(value)  https://review.openstack.org/24348910:50
openstackgerritMichal Dulko proposed openstack/cinder-specs: Add Cinder Volume A/A HA spec  https://review.openstack.org/20261510:51
openstackgerritMichal Dulko proposed openstack/cinder-specs: Add Tooz locs to support A/A HA spec  https://review.openstack.org/20261510:51
openstackgerritHelen Walsh proposed openstack/cinder: Fix for randomly selecting a portgroup. Before now the utils function was always returning the first portgroup in the list.  It should randomly select.  https://review.openstack.org/24330410:57
openstackgerritMichal Dulko proposed openstack/cinder-specs: Add Tooz locs to support A/A HA spec  https://review.openstack.org/20261510:58
*** haomaiwa_ has quit IRC11:01
*** haomaiwang has joined #openstack-cinder11:01
*** EinstCrazy has joined #openstack-cinder11:01
dulekgeguileo: Hi! I've revived Tooz locks spec if you're looking for something to review. You would be able to take revenge on me for my -1's. ;)11:02
geguileoXD XD XD11:02
geguileoMy backlog is huge from my vacation time11:03
geguileoBut I'll certainly review that spec11:03
geguileodulek: Thanks for the ping11:03
dulekgeguileo: I know that feeling. Hope your vacation time was totally worth it. :)11:04
geguileodulek: The first day is always the day I'm not sure it was worth it  ;-)11:04
*** marcusvrn_ has joined #openstack-cinder11:04
*** dims has joined #openstack-cinder11:05
dulekgeguileo: Oh, so it's like normal Monday?11:06
openstackgerritMichal Dulko proposed openstack/cinder-specs: Add Tooz locks to support A/A HA spec  https://review.openstack.org/20261511:06
geguileodulek: But on Tuesday!!  ;-)11:06
dulekgeguileo: Ah, right. :D This Tuesday is actually a logical Friday for me - tomorrow's state holiday in Poland.11:08
geguileodulek: Lucky you! In Madrid yesterday was a local holiday, so I've been on holidays since the 1st of this month11:10
*** deepakcs has quit IRC11:10
dulekgeguileo: So who's actually lucky? …or was?11:12
geguileodulek: I was lucky, but today I don't feel so much... XD11:12
openstackgerritzhangsong proposed openstack/cinder: fix the bug of OSError when convert image  https://review.openstack.org/24315211:15
*** lpetrut has joined #openstack-cinder11:16
openstackgerritArne Wiebalck proposed openstack/cinder: Fix limited retype support for rbd  https://review.openstack.org/24353711:18
*** dims has quit IRC11:21
*** dims has joined #openstack-cinder11:22
*** markvoelker has joined #openstack-cinder11:27
*** markvoelker has quit IRC11:32
*** aix has quit IRC11:33
*** aix has joined #openstack-cinder11:33
openstackgerritHelen Walsh proposed openstack/cinder: Fix for randomly selecting a portgroup. Before now the utils function was always returning the first portgroup in the list.  It should randomly select. This addresses the following bug https://bugs.launchpad.net/cinder/+bug/1501919  https://review.openstack.org/24330411:34
openstackLaunchpad bug 1501919 in Cinder "VMAX driver always picks the first port group" [Medium,Triaged] - Assigned to Helen Walsh (walshh2)11:34
*** smatzek has joined #openstack-cinder11:40
*** subscope has joined #openstack-cinder11:41
*** jaypipes has joined #openstack-cinder11:42
*** houming-wang has joined #openstack-cinder11:48
*** anshul has quit IRC11:51
*** can8dnSix has quit IRC11:53
*** julim has joined #openstack-cinder11:53
openstackgerritwanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly  https://review.openstack.org/23243611:54
openstackgerritArne Wiebalck proposed openstack/cinder: Fix limited retype support for rbd  https://review.openstack.org/24353711:56
*** haomaiwang has quit IRC12:01
*** haomaiwang has joined #openstack-cinder12:01
*** cdelatte has joined #openstack-cinder12:05
*** Yogi1 has joined #openstack-cinder12:19
*** Yogi1 has quit IRC12:24
*** wilson-1 has quit IRC12:24
*** smoriya has quit IRC12:24
*** wilson has joined #openstack-cinder12:26
*** wilson is now known as Guest9783112:27
*** jwcroppe has joined #openstack-cinder12:27
openstackgerritPetrut Lucian proposed openstack/cinder: Add os-win to requirements.txt  https://review.openstack.org/24356412:29
openstackgerritPetrut Lucian proposed openstack/cinder: Refactor Windows drivers using os-win  https://review.openstack.org/24356512:29
*** julim has quit IRC12:31
*** jerrygb has joined #openstack-cinder12:32
*** angela-s has quit IRC12:35
*** jerrygb has quit IRC12:37
*** diegows has joined #openstack-cinder12:37
*** zhangjn_ has quit IRC12:39
*** NightKhaos has quit IRC12:48
*** NightKhaos has joined #openstack-cinder12:49
*** haomaiwang has quit IRC12:51
dulekgeguileo: I'm trying to rebase and understand https://review.openstack.org/#/c/18353712:52
*** haomaiwang has joined #openstack-cinder12:52
dulekgeguileo: Do you remember why we have MultiLock there? Where do we need to do multiple locks in one place? This seems a little odd to me.12:53
*** dims has quit IRC12:53
geguileodulek: Trying to remember...12:54
geguileodulek: Ok, volume creation from a source12:54
geguileodulek: You need to lock volume being created as well as the source12:55
geguileoI think that was the reason12:55
geguileoBut I haven't looked at that since July...12:56
*** dims has joined #openstack-cinder12:56
*** haomaiwang has quit IRC12:57
*** baumann has joined #openstack-cinder12:58
*** markvoelker has joined #openstack-cinder12:58
dulekgeguileo: Me neither. I don't think we need to lock on both, volume that's being created should be locked by "creating" state.12:59
geguileodulek: But we may want to prevent force delete12:59
geguileoI think it was somewhere in a TODO (I think in the create flow)13:00
*** gouthamr has quit IRC13:00
*** timcl has joined #openstack-cinder13:01
*** diegows has quit IRC13:01
*** jgregor has joined #openstack-cinder13:01
dulekgeguileo: Ah, right… I think I can get that multi-stuff into another patch and add fixing that into my backlog.13:02
dulekgeguileo: Patch is complicated by itself, no point in increasing its complexity.13:03
*** markvoelker has quit IRC13:03
dulekgeguileo: Also as first version it's probably safer to do it without adding new locks, just replacing the old ones.13:04
dulekgeguileo: Thanks for explanation!13:04
geguileodulek: np13:05
geguileodulek: And you are right, better add the functionality in multiple patches13:05
*** jwcroppe has quit IRC13:07
*** zhangjn has joined #openstack-cinder13:07
*** gouthamr has joined #openstack-cinder13:10
*** stevemar_ has joined #openstack-cinder13:11
*** gcb has joined #openstack-cinder13:17
*** subscope has quit IRC13:17
*** stevemar_ has quit IRC13:20
*** markvoelker has joined #openstack-cinder13:20
*** markvoelker has quit IRC13:20
*** stevemar_ has joined #openstack-cinder13:21
*** salv-orlando has joined #openstack-cinder13:23
*** jerrygb has joined #openstack-cinder13:24
*** markvoelker has joined #openstack-cinder13:25
*** changbl has quit IRC13:28
*** akerr has joined #openstack-cinder13:29
*** akerr is now known as akerr_away13:29
*** akerr_away is now known as akerr13:29
*** jerrygb has quit IRC13:29
*** kaisers has quit IRC13:33
*** subscope has joined #openstack-cinder13:34
*** houming-wang has quit IRC13:35
*** houming-wang has joined #openstack-cinder13:35
*** porrua has joined #openstack-cinder13:36
*** bluex has joined #openstack-cinder13:40
*** bluex has quit IRC13:40
*** kaisers has joined #openstack-cinder13:40
*** bluex has joined #openstack-cinder13:40
openstackgerritArne Wiebalck proposed openstack/cinder: Fix limited retype support for rbd  https://review.openstack.org/24353713:43
*** martyturner has joined #openstack-cinder13:43
*** vincent_hou has joined #openstack-cinder13:47
*** haomaiwang has joined #openstack-cinder13:50
*** akshai has joined #openstack-cinder13:51
openstackgerritzhangsong proposed openstack/cinder: fix the bug of OSError when convert image  https://review.openstack.org/24315213:56
*** haomaiwang has quit IRC14:01
*** xyang1 has joined #openstack-cinder14:01
*** haomaiwang has joined #openstack-cinder14:01
*** dustins has joined #openstack-cinder14:03
*** stevemar_ has quit IRC14:04
openstackgerritxing-yang proposed openstack/cinder: Backup snapshots  https://review.openstack.org/24340614:08
*** links has quit IRC14:08
*** setmason has joined #openstack-cinder14:12
*** vincent_hou has quit IRC14:12
*** diogogmt has joined #openstack-cinder14:13
*** stevemar_ has joined #openstack-cinder14:13
*** timcl has quit IRC14:20
openstackgerritEduard Biceri-Matei proposed openstack/cinder: Re-add the OpenvStorage Cinder driver  https://review.openstack.org/24117414:20
openstackgerritPetrut Lucian proposed openstack/cinder: Refactor Windows drivers using os-win  https://review.openstack.org/24356514:20
openstackgerritzhu.rong proposed openstack/python-cinderclient: Update CONTRIBUTING.md to CONTRIBUTING.rst  https://review.openstack.org/24362114:23
*** ndipanov has quit IRC14:24
*** tsufiev_ has quit IRC14:26
openstackgerritzhangsong proposed openstack/cinder: Fix the bug of OSError when convert image  https://review.openstack.org/24315214:27
*** tsufiev has joined #openstack-cinder14:30
*** zhangjn has quit IRC14:31
openstackgerritMichal Dulko proposed openstack/cinder-specs: Add Tooz locks to support A/A HA spec  https://review.openstack.org/20261514:32
*** timcl has joined #openstack-cinder14:38
*** takedakn has joined #openstack-cinder14:40
*** dustins has quit IRC14:41
*** salv-orlando has quit IRC14:46
*** thangp has joined #openstack-cinder14:46
*** zhangjn has joined #openstack-cinder14:47
*** cfriesen__ has joined #openstack-cinder14:49
*** ntpttr has quit IRC14:49
*** eharney has joined #openstack-cinder14:50
*** ntpttr has joined #openstack-cinder14:50
*** mriedem has joined #openstack-cinder14:51
*** lprice has joined #openstack-cinder14:51
*** stevemar_ has quit IRC14:53
*** stevemar_ has joined #openstack-cinder14:54
*** zhangjn has quit IRC14:55
*** diegows has joined #openstack-cinder14:55
*** ZZelle has quit IRC14:56
*** zhangjn has joined #openstack-cinder14:56
*** anshul has joined #openstack-cinder14:57
*** mc_nair has joined #openstack-cinder14:59
*** diegows has quit IRC14:59
*** zhangjn has quit IRC15:00
*** akshai has quit IRC15:00
*** haomaiwang has quit IRC15:01
*** haomaiwang has joined #openstack-cinder15:01
*** links has joined #openstack-cinder15:01
*** akshai has joined #openstack-cinder15:02
*** lprice has quit IRC15:03
*** breitz has joined #openstack-cinder15:04
*** links has quit IRC15:05
*** diogogmt has quit IRC15:06
*** lprice has joined #openstack-cinder15:06
*** dave-mccowan has joined #openstack-cinder15:08
*** flip214 has quit IRC15:08
openstackgerritzhu.rong proposed openstack/python-cinderclient: Update CONTRIBUTING.md to CONTRIBUTING.rst  https://review.openstack.org/24362115:09
openstackgerritzhu.rong proposed openstack/python-cinderclient: Update CONTRIBUTING.md to CONTRIBUTING.rst  https://review.openstack.org/24362115:09
openstackgerritzhu.rong proposed openstack/cinder: Update CONTRIBUTING.md to CONTRIBUTING.rst  https://review.openstack.org/24364715:10
*** takedakn has quit IRC15:12
*** jwcroppe has joined #openstack-cinder15:14
*** andrewfly has joined #openstack-cinder15:14
*** edtubill has joined #openstack-cinder15:14
*** [2]Thelo has joined #openstack-cinder15:18
*** Thelo has quit IRC15:20
*** [2]Thelo is now known as Thelo15:20
*** jungleboyj has joined #openstack-cinder15:22
*** asselin__ has quit IRC15:22
*** setmason has quit IRC15:23
openstackgerritAccela Zhao proposed openstack/cinder: OpenStack typo  https://review.openstack.org/24365615:23
*** flip214 has joined #openstack-cinder15:26
*** Thelo has quit IRC15:27
*** Thelo has joined #openstack-cinder15:28
*** asselin_ has joined #openstack-cinder15:30
*** akshai has quit IRC15:31
*** jerrygb has joined #openstack-cinder15:33
*** dustins has joined #openstack-cinder15:33
*** akshai has joined #openstack-cinder15:33
*** rhagarty_ has quit IRC15:33
*** gcb has quit IRC15:35
*** jerrygb has quit IRC15:37
openstackgerritMikhail Khodos proposed openstack/cinder: Re-add Nexenta driver.  https://review.openstack.org/18887715:37
*** rhagarty has joined #openstack-cinder15:38
*** rhagarty has quit IRC15:40
*** rhagarty has joined #openstack-cinder15:40
*** changbl has joined #openstack-cinder15:40
*** andrewfly has quit IRC15:41
*** garthb has joined #openstack-cinder15:41
*** rhagarty_ has joined #openstack-cinder15:42
*** changbl has quit IRC15:43
*** changbl has joined #openstack-cinder15:44
*** rhagarty has quit IRC15:45
*** asselin_ has quit IRC15:45
*** salv-orlando has joined #openstack-cinder15:46
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API  https://review.openstack.org/24367315:50
*** subscope has quit IRC15:51
*** mudassirlatif has joined #openstack-cinder15:52
*** belmoreira has quit IRC15:53
*** asselin_ has joined #openstack-cinder15:54
*** diogogmt has joined #openstack-cinder15:55
*** martyturner has quit IRC15:55
*** DaveJ has joined #openstack-cinder15:56
*** martyturner has joined #openstack-cinder15:57
*** andrewfly has joined #openstack-cinder15:57
*** salv-orlando has quit IRC15:57
*** houming-wang has quit IRC15:57
*** mtanino has joined #openstack-cinder15:57
*** diablo_rojo has quit IRC15:59
*** diogogmt has quit IRC15:59
*** houming-wang has joined #openstack-cinder16:00
*** vgridnev has quit IRC16:00
*** diablo_rojo has joined #openstack-cinder16:00
*** haomaiwang has quit IRC16:01
*** haomaiwang has joined #openstack-cinder16:01
*** dustins has quit IRC16:01
*** diogogmt has joined #openstack-cinder16:02
*** baumann has quit IRC16:02
*** aix has quit IRC16:04
*** dustins has joined #openstack-cinder16:06
*** baumann has joined #openstack-cinder16:07
*** jerrygb has joined #openstack-cinder16:07
*** bluex has quit IRC16:08
*** hemnafk is now known as hemna16:09
hemnajungleboyj, hey man16:11
*** jerrygb has quit IRC16:12
openstackgerritHelen Walsh proposed openstack/cinder: Fix for randomly selecting a portgroup  https://review.openstack.org/24330416:12
*** jerrygb has joined #openstack-cinder16:13
jungleboyjWhat's up?16:13
jungleboyjhemna: ^^16:13
hemnaany idea what's up w/ the storewize CI ?16:13
hemnahasn't been happy on me for a while16:13
hemnahttps://review.openstack.org/#/c/199764/16:13
hemnahttp://dal05.objectstorage.softlayer.net/v1/AUTH_58396f85-2c60-47b9-aaf8-e03bc24a1a6f/IBMStorwizeCI/64/199764/20/check/storwize-cinder-ci-iscsi-os-brick/3bdfbbd/console.html16:14
hemna:(16:14
hemnaI need to get that patch in so I can move on16:14
hemnanot getting reviews, but I'd like to see if the storewize CI to pass16:14
*** crose has joined #openstack-cinder16:14
*** jerrygb_ has joined #openstack-cinder16:15
*** merooney has joined #openstack-cinder16:15
jungleboyjThe CI was down due to a hardware problem at one point.  Let me see if Ivy is still around.16:15
smcginnisjungleboyj: It ran on the npiv patch, but only the FC one. iSCSI not so happy.16:16
*** jerrygb has quit IRC16:17
*** esker has joined #openstack-cinder16:18
hemnaany idea how to issue a recheck on it ?16:18
openstackgerritArne Wiebalck proposed openstack/cinder: Fix limited retype support for rbd  https://review.openstack.org/24353716:18
jungleboyjI can make that happen.16:19
hemnacool thank you16:19
jungleboyjjgregor: Can you recheck Walt's patch and look at what is up with the storwize CI?16:20
jgregorjungleboyj: On it.16:20
hemnajungleboyj, jgregor thanks guys16:20
*** timcl has quit IRC16:24
*** nkrinner has quit IRC16:25
*** dzamboni has joined #openstack-cinder16:27
*** laughterwym has quit IRC16:27
*** laughterwym has joined #openstack-cinder16:28
*** jamielennox has quit IRC16:28
*** sgundur has joined #openstack-cinder16:31
*** setmason has joined #openstack-cinder16:32
*** yuriy_n17 has quit IRC16:34
*** subscope has joined #openstack-cinder16:38
*** subscope has quit IRC16:39
*** jamielennox has joined #openstack-cinder16:41
*** salv-orlando has joined #openstack-cinder16:42
*** subscope has joined #openstack-cinder16:44
*** timcl has joined #openstack-cinder16:44
*** vgridnev has joined #openstack-cinder16:47
*** haomaiwang has quit IRC16:48
*** laughterwym has quit IRC16:51
*** subscope has quit IRC16:54
openstackgerritMerged openstack/cinder: Removal of deprecated NPIV option in Storwize  https://review.openstack.org/24331816:58
hemnaI like the new merged notification16:58
hemnathat's cool16:58
*** sgundur has quit IRC17:00
*** merooney has quit IRC17:00
*** houming-wang has quit IRC17:01
*** martyturner has quit IRC17:01
*** sgundur has joined #openstack-cinder17:01
jungleboyjOh cool, I didn't know we were getting those now.17:02
openstackgerritPetrut Lucian proposed openstack/cinder: Refactor Windows drivers using os-win  https://review.openstack.org/24356517:02
*** martyturner has joined #openstack-cinder17:04
openstackgerritPetrut Lucian proposed openstack/cinder: [WIP] Refactor Windows drivers using os-win  https://review.openstack.org/24356517:05
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder-specs: Add extend_volume API to connectors  https://review.openstack.org/24373017:06
*** leeantho has joined #openstack-cinder17:06
*** martyturner has quit IRC17:08
*** salv-orlando has quit IRC17:08
smcginnishemna: Yeah, added that when I checked in on the os-brick notifications. I like it too. Helps to see what's going on.17:12
mriedemgeguileo: is https://review.openstack.org/#/c/200927/ something that can/should be backported to stable/kilo?17:14
*** garthb_ has joined #openstack-cinder17:14
*** garthb has quit IRC17:15
geguileomriedem: That was part of the fix for metadata import bug in Kilo17:15
geguileomriedem: So if we want to backport the patch that fixes that bug without changing it completely we would probably have to17:16
smcginnismriedem: Kilo is security only now. Not sure if we can do that.17:16
mriedemwell, it's also an api version change since that change adds a remotable method to the object17:17
mriedemhttps://review.openstack.org/#/c/200927/6/cinder/objects/backup.py17:17
geguileosmcginnis: You are totally right!!17:17
mriedembut the version wasn't bumped17:17
*** Guest97831 has quit IRC17:17
*** Guest97831 has joined #openstack-cinder17:17
mriedemdoes cinder have the same unit tests that nova has which fails if it detects you've added remotable methods or changed fields/methods but not bumped the version?17:17
geguileomriedem: I don't think we do17:17
mriedemheh, cinder/tests and cinder/testing17:18
mriedemnot confusing at all17:18
mriedemyeah, https://github.com/openstack/nova/blob/master/nova/tests/unit/objects/test_objects.py#L125317:19
mriedemi can look into adding that17:19
mriedemit keeps nova from hurting nova17:19
geguileomriedem: That would be a good addition17:20
*** jwcroppe has quit IRC17:25
*** jwcroppe has joined #openstack-cinder17:26
*** dims has quit IRC17:27
*** dims has joined #openstack-cinder17:28
*** sgundur has quit IRC17:30
*** jwcroppe has quit IRC17:30
*** sgundur has joined #openstack-cinder17:33
*** esker has quit IRC17:35
*** wilson1 has joined #openstack-cinder17:37
*** Guest97831 has quit IRC17:39
mriedemjungleboyj: fyi https://bugs.launchpad.net/cinder/+bug/1514926 - i'm going to start looking at that today probably but if it becomes a mess for me, your team might want to take that over17:40
openstackLaunchpad bug 1514926 in Cinder "Cinder has no testing infrastructure for when objects require a version bump" [High,Confirmed] - Assigned to Matt Riedemann (mriedem)17:40
jungleboyjmriedem: Ok.  Let me know.17:42
*** lpetrut has quit IRC17:44
*** sghanekar has joined #openstack-cinder17:48
*** jistr has quit IRC17:48
*** baumann has quit IRC17:57
*** rady has joined #openstack-cinder17:57
*** apoorvad has joined #openstack-cinder17:59
*** garthb_ has quit IRC18:02
*** merooney has joined #openstack-cinder18:02
*** martyturner has joined #openstack-cinder18:02
*** garthb_ has joined #openstack-cinder18:02
*** krtaylor has quit IRC18:03
*** jordanP has quit IRC18:05
hemnahttp://paste.openstack.org/show/478473/18:05
hemnajust about done18:05
hemnagot the raw volume and the mpath volume paths do resize while attached18:06
*** willsama has joined #openstack-cinder18:06
*** jerrygb has joined #openstack-cinder18:06
*** jerrygb_ has quit IRC18:09
*** jgregor has quit IRC18:09
*** rebase has joined #openstack-cinder18:09
openstackgerritDeepti Ramakrishna proposed openstack/python-cinderclient: snapshot-list now supports filtering by tenant  https://review.openstack.org/24239118:15
*** krtaylor has joined #openstack-cinder18:16
*** lcurtis has joined #openstack-cinder18:23
*** timcl1 has joined #openstack-cinder18:24
*** timcl has quit IRC18:27
*** liverpooler has quit IRC18:27
*** garthb__ has joined #openstack-cinder18:31
*** smatzek has quit IRC18:31
*** garthb_ has quit IRC18:31
*** p0rtal has joined #openstack-cinder18:38
openstackgerritMikhail Khodos proposed openstack/cinder: Re-add Nexenta driver.  https://review.openstack.org/18887718:41
*** angela-s has joined #openstack-cinder18:45
*** lpetrut has joined #openstack-cinder18:46
*** lpetrut has quit IRC18:52
*** lpetrut has joined #openstack-cinder18:52
*** mriedem has quit IRC18:53
*** dave-mcc_ has joined #openstack-cinder18:58
*** mriedem has joined #openstack-cinder18:58
*** dave-mccowan has quit IRC18:58
*** diablo_rojo1 has joined #openstack-cinder18:59
*** dave-mccowan has joined #openstack-cinder18:59
openstackgerritnandal proposed openstack/cinder: Use assertTrue/False(A) Methods  https://review.openstack.org/24348919:01
*** diablo_rojo has quit IRC19:01
*** jerrygb_ has joined #openstack-cinder19:01
*** baumann has joined #openstack-cinder19:02
*** dave-mcc_ has quit IRC19:03
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: WIP: brick add extend_volume API  https://review.openstack.org/24377619:03
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: WIP: brick add extend_volume API  https://review.openstack.org/24377619:03
*** julim has joined #openstack-cinder19:03
*** sgundur has quit IRC19:03
*** jerrygb has quit IRC19:04
*** merooney has quit IRC19:05
*** jerrygb has joined #openstack-cinder19:05
*** jerrygb_ has quit IRC19:08
*** kfarr has joined #openstack-cinder19:09
openstackgerritnandal proposed openstack/cinder: Use assertTrue/False(A) Methods  https://review.openstack.org/24348919:11
*** ociuhandu has quit IRC19:12
*** jgregor has joined #openstack-cinder19:12
*** skraynev has quit IRC19:16
*** anshul has quit IRC19:25
*** skraynev has joined #openstack-cinder19:27
*** lpabon has joined #openstack-cinder19:32
*** martyturner has quit IRC19:33
*** martyturner has joined #openstack-cinder19:34
*** jwcroppe has joined #openstack-cinder19:37
*** subscope has joined #openstack-cinder19:39
mriedemdansmith: HI!19:43
dansmithohai!19:44
mriedemyeah, so there are some of these https://github.com/openstack/cinder/blob/master/cinder/objects/backup.py#L8719:44
mriedemwhich are completely pointless19:44
mriedemthe parent class doesn't do your compat for you19:44
dansmithwell, it does for object fields19:44
mriedemhttps://github.com/openstack/cinder/blob/master/cinder/objects/service.py#L51 really doesn't do anything19:44
dansmithbut there's no reason to calculate target_version afterwards19:44
mriedemyeah, so i'll remove all of those19:45
mriedemas for child_versions19:45
dansmiththe base class of VersionedObject does do magic backporting of sub-objects for you19:45
dansmithwe should ask the cinder people here if they plan to need that.. if so, those should be converted to relationships, and if not, they should be removed entirely19:45
dansmiththangp: ?19:45
*** DaveJ has quit IRC19:45
mriedemclass ObjectListBase(base.ObjectListBase):19:46
mriedem    pass19:46
mriedemhmmm19:46
dansmithor if they're planning to keep a single version for everything then it doesn't much matter19:46
dansmithbut that's going to make it hard for them to do hash checking :/19:46
mriedemyeah19:46
mriedemwell, what's the point in using versioned objects if you never bump versions?19:46
mriedemi guess it hides some of the rpc/db stuff19:47
mriedemand enforces type checking on fields19:47
dansmithyeah, I think they still plan to bump versions,19:48
dansmithbut because they don't plan to have anything like conductor, they have to figure out how they're going to handle being compatible with older nodes19:48
dansmithsmcginnis: do you know any of cinder's plans in this regard, or know who does?19:48
dansmithdulek: maybe?19:48
smcginnisdansmith: Yeah, that's an area I definitely need a better understanding of all the implications.19:52
*** rebase has quit IRC19:52
dansmithsmcginnis: okay19:52
smcginnisdansmith: For now I'm looking to thangp and maybe dulek.19:52
dansmithmriedem: at this point, I think it's probably best if you convert the child_versions to obj_relationships19:53
dansmithmriedem: I think the o.vo checker has the same thing for the relationship maps, but if not, you can port that from nova19:53
dansmithmriedem: honestly, I didn't think about the non-conductor-using cinder implications of manifests when I did it19:53
dansmithmeaning I was hoping we could deprecate/remove that stuff entirely19:54
*** daneyon has joined #openstack-cinder19:54
dansmitheven if we keep it for them, the indirection methods that relied on them can still go away as they don't use indirection in cinder at all at the moment, we just have to keep that bit of the base class to handle doing things that way19:54
*** daneyon__ has joined #openstack-cinder19:55
mriedemok, so despite the child_versions/obj_relationships stuff, is there value in cinder running ObjectVersionChecker.test_hashes?19:56
mriedemi would think yes19:56
dansmithyes definitely19:56
mriedemok19:56
mriedemthat i can do19:56
*** daneyon_ has quit IRC19:56
*** daneyon has quit IRC19:58
*** martyturner has quit IRC19:59
*** Lee1092 has quit IRC19:59
openstackgerritJacob Gregor proposed openstack/cinder: Storwize: Split up __init__ into separate files  https://review.openstack.org/24032919:59
*** martyturner has joined #openstack-cinder20:00
*** apoorvad has quit IRC20:01
* thangp reading the backlog20:05
*** merooney has joined #openstack-cinder20:06
*** merooeny_ has joined #openstack-cinder20:08
*** merooney has quit IRC20:11
thangpmriedem, dansmith: so the object versioning was not properly enforced, since not everything was using objects.  we should enforce bumping up the version when changes are made.  as far as the child_versions, we put it there to be consistent with how nova did it.  if there's no value in it, then it should be removed.20:11
thangpas for the obj_make_compatible, right now it does not do anything since we do not support rolling upgrades, but we can fix it up when we do and properly send the correct versioned objects20:13
mriedemthangp: ok, i'm fixing that first part20:14
mriedemshould be up shortly20:14
thangpmriedem: ok cool, thx!20:14
smcginnisthangp: Thanks!20:14
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: WIP: brick add extend_volume API  https://review.openstack.org/24377620:15
*** lpabon has quit IRC20:16
*** r-daneel has joined #openstack-cinder20:18
mriedemgdi, ostestr20:19
mriedemmtreinish: we have problems20:19
*** cbader has left #openstack-cinder20:19
*** crose has quit IRC20:19
mriedemmtreinish: apparently this doesn't work with ostestr (trying to do the same in cinder): https://review.openstack.org/#/c/243800/2/tox.ini20:20
openstackgerritVictor Stinner proposed openstack/cinder: Port IBM storewize_svc driver to Python 3  https://review.openstack.org/24287820:20
mtreinishwhy would ostestr change that?20:20
mriedemwas just going to say, that shoudn't matter20:20
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: WIP: brick add extend_volume API  https://review.openstack.org/24377620:21
*** sparr has quit IRC20:21
*** sparr has joined #openstack-cinder20:21
mtreinishmriedem: do you have an example of when it's not working?20:22
*** cbader has joined #openstack-cinder20:23
mriedemjust add that to cinder tox.ini20:23
mriedemDouble requirement given: oslo.versionedobjects>=0.9.0 (from -r /home/mriedem/git/cinder/requirements.txt (line 25)) (already in oslo.versionedobjects[fixtures], name='oslo.versionedobjects'20:24
mriedemweird, that didn't fail for nova20:24
*** timcl has joined #openstack-cinder20:24
*** timcl1 has quit IRC20:24
mriedemoh i see why20:25
*** lifeless has quit IRC20:26
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API  https://review.openstack.org/24367320:26
*** dave-mccowan has quit IRC20:48
openstackgerritMerged openstack/os-brick: Update minimum tox version to 1.8  https://review.openstack.org/24143320:49
*** dave-mccowan has joined #openstack-cinder20:50
*** jerrygb has quit IRC20:51
*** dave-mcc_ has joined #openstack-cinder20:52
*** dave-mccowan has quit IRC20:55
*** akerr has quit IRC20:55
*** timcl has quit IRC20:56
*** lpetrut has quit IRC20:57
*** jerrygb has joined #openstack-cinder20:59
*** jerrygb has quit IRC20:59
*** jerrygb has joined #openstack-cinder20:59
*** apoorvad has joined #openstack-cinder21:02
*** dzamboni has quit IRC21:03
*** ociuhandu has joined #openstack-cinder21:03
hemnaugh21:04
hemnaboth of my +A'd brick patches failed due to random neutron failures.21:04
hemna*sigh*21:04
smcginnishemna: There have been a lot of those lately. Haven't had a chance to dig in to what's going on.21:05
smcginnisIt's happening often enough that I don't think it's just a glitch.21:05
hemnaboth of them are different, but random21:05
smcginnishemna: Yeah, that's what I've seen in the few I've looked at.21:05
smcginnishemna: Hopefully one more recheck will get by.21:05
smcginnisI'm getting concerned though.21:06
hemnayah it's starting to get more frequent21:06
*** apoorvad has quit IRC21:06
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API  https://review.openstack.org/24367321:06
*** lifeless has joined #openstack-cinder21:10
*** salv-orlando has joined #openstack-cinder21:10
*** jerrygb_ has joined #openstack-cinder21:12
*** jerrygb has quit IRC21:16
*** jerrygb_ is now known as jerrygb21:17
openstackgerritMatt Riedemann proposed openstack/cinder: Test for object version hash changes  https://review.openstack.org/24382621:17
mriedemthangp: dansmith: ^ needs some work though21:17
mriedemthe global cinder object registratoin is pulling in way more things than we care about21:17
dansmithmriedem: they're still using the metaclass I bet21:18
*** p0rtal has quit IRC21:18
thangpmriedem: how do you get the hashes?21:19
mriedemdansmith: no, just21:19
mriedem@CinderObjectRegistry.register21:19
mriedemclass CinderObject(base.VersionedObject):21:19
mriedemso everything that extends that is registered21:19
dansmithoh and they're registering their test objects I guess?21:19
mriedemi think so21:19
thangpdansmith: yes21:20
mriedemthangp: just had the test fail against {} thefirst time21:20
dansmithmriedem: I'll pull that down and see about fixing that on top21:20
mriedemok, i was just going to fix in the same change21:20
mriedembut whatevs21:20
dansmithmight as well do it separately for clarity I think21:20
mriedemsure21:21
dansmithat least, I like my things-that-touch-the-hashes patches to be super small21:21
mriedemtheir registry doesn't have this versoin logic either https://github.com/openstack/nova/blob/master/nova/objects/base.py#L5121:21
mriedemi'll remove the WIP21:21
dansmiththat's up to them21:22
dansmiththat hook is in the library purely so nova can do its nova.objects.FooObject thing21:22
mriedemso, the MyObj and MyOwnedObj are actually pulled in from ovo's test_objects21:23
mriedemcinder doesn't have a class called MyObj21:23
dansmithhmm21:23
dansmithnor does that hash set include cinder's test object(s)21:25
*** jamielennox is now known as jamielennox|away21:25
*** jungleboyj has quit IRC21:26
*** apoorvad has joined #openstack-cinder21:26
*** rebase has joined #openstack-cinder21:26
dansmithmriedem: if they're really the ones from o.vo then somewhere we must be importing those test modules21:27
thangpmriedem: i cant find any reference to MyOwnedObj in o.vo or cinder21:29
thangponly MyObj is in o.vo21:29
mriedemi see it21:29
mriedemdansmith: https://github.com/openstack/cinder/blob/master/cinder/tests/unit/objects/__init__.py21:30
mriedemextends ovo and registers all objects21:30
dansmithoh jeez21:30
dansmithyeah21:30
mriedemso i can remove that21:31
dansmithyeah21:31
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Remove the HP CLIQ proxy driver  https://review.openstack.org/24140421:33
mriedemmaybe i cleanup the object registration setup stuff in a dependent change, and then do my hash check on top of that21:33
*** andrewfly has quit IRC21:37
*** rlrossit has joined #openstack-cinder21:38
*** baumann has quit IRC21:38
*** changbl has quit IRC21:41
*** lpetrut has joined #openstack-cinder21:41
*** esker has joined #openstack-cinder21:42
*** dustins has quit IRC21:46
SwansonDoes anything use replication_driver_status or is it replication_driver_data I should be using for driver specific info about repls?21:49
SwansonV221:50
*** esker has quit IRC21:51
*** esker has joined #openstack-cinder21:52
openstackgerritHelen Walsh proposed openstack/cinder: Adding more error handling for unconfigured SLOs  https://review.openstack.org/24383721:54
openstackgerritHelen Walsh proposed openstack/cinder: Proper error handling for invalid SLO/Workload combination  https://review.openstack.org/24383721:55
*** esker has quit IRC21:56
angela-ssmcginnis: should py34 be added to tox.ini since it's voting in gate?21:56
*** thangp has quit IRC21:57
smcginnisangela-s: py34 is added. Not sure if I understand what you're asking.21:57
patrickeastSwanson: i believe it is replication_driver_data for the driver specific info if this comment is to be trusted https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L320421:57
*** jerrygb has quit IRC21:57
angela-ssmcginnis: https://github.com/openstack/cinder/blob/master/tox.ini#L4, here only py2721:57
*** xiaohui has joined #openstack-cinder21:58
angela-si got tripped up on a review because i ran tox and everything was fine, but then Jenkins failed because of py3421:58
smcginnisangela-s: Oh, I see. As a default test to run when just calling tox.21:58
Swansonpatrickeast,  Thanks!  That's what I thought.  I've an ancient solidfire patch I keep checking against...21:58
angela-syeah21:58
smcginnisIt can be called explicitly: https://github.com/openstack/cinder/blob/master/tox.ini#L3721:58
smcginnisangela-s: I think we were waiting initially for more py34 conversion and support.21:58
patrickeastSwanson: ah, yea, things shifted around a little bit with the model_update (actually... that might have been my fault if i check git blame...)21:59
smcginnisangela-s: But yeah, since we are testing that in gate now, it probably should be added to the default targets to run.21:59
angela-ssmcginnis: ok, thanks.  I'll try to remember to run this test too21:59
smcginnisangela-s: At least so there aren't any more surprises like you had. :)21:59
angela-ssmcginnis: yeah, that would be best i think21:59
Swansonpatrickeast, If I ever pushed up code I'd have more skin in this game.  As it is I just have to chase.22:00
smcginnisangela-s: The problem is, on my main dev machine I still get an error when I try to run py34 tests.22:00
smcginnisangela-s: But that's my problem. :D22:00
patrickeastSwanson: unfortunately it seems like its always a chase... things are always shifting around22:01
angela-ssmcginnis: LOL, i'll probably have the same problem22:01
*** salv-orlando has quit IRC22:01
angela-ssmcginnis: i was blindly thinking my new code was magically py34 compliant until Jenkins told me different22:01
Swansonpatrickeast, I'm tempted to push this up before I have unit tests.  Just to see if I can get some eyes on it.22:02
patrickeastSwanson: not a bad idea, i've done that before with WIP patches22:02
smcginnisangela-s: If we could only be so lucky.22:04
angela-ssmcginnis: yep, as you said, py34 fails miserably on my dev machine too!  Oh joy! :(22:08
*** vgridnev has quit IRC22:09
*** salv-orlando has joined #openstack-cinder22:10
*** vgridnev has joined #openstack-cinder22:11
mriedemdansmith: so how bad of an idea is it to re-use https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/tests/test_objects.py#L462 in cinder?22:12
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: WIP: brick add extend_volume API  https://review.openstack.org/24377622:12
mriedemb/c as i'm fixing some things in the cinder objects unit tests, i'm hitting issues with how objects are compared to db dicts22:12
*** baumann has joined #openstack-cinder22:12
*** baumann has left #openstack-cinder22:12
*** salv-orlando has quit IRC22:13
*** xiaohui has quit IRC22:15
dansmithmriedem: not terrible, but we could move that somewhere into the fixture or something if it's useful22:16
*** vgridnev has quit IRC22:16
*** porrua has quit IRC22:17
mriedemdansmith: i just saw nova was doing the same thing22:18
mriedembut it looks like nova doesn't actually use compare_obj anywhere22:18
mriedemoh nvm, it is22:19
*** gouthamr has quit IRC22:20
*** jamielennox|away is now known as jamielennox22:24
*** jgregor has quit IRC22:26
*** p0rtal has joined #openstack-cinder22:27
*** boris-42 has joined #openstack-cinder22:30
*** tsufiev has quit IRC22:34
*** rebase has quit IRC22:37
*** tsufiev has joined #openstack-cinder22:37
*** rebase has joined #openstack-cinder22:37
smcginnisangela-s: See if deleting the .tox directory helps in your case. I've heard that might make a difference. Not for me.22:38
*** lpetrut has quit IRC22:44
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: 3PAR Fix find_existing_vluns  https://review.openstack.org/24385622:49
*** edtubill has quit IRC22:51
*** akshai has quit IRC22:51
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: 3PAR Fix find_existing_vluns  https://review.openstack.org/24385622:53
*** akshai has joined #openstack-cinder22:56
*** diablo_rojo1 has quit IRC23:07
*** jerrygb has joined #openstack-cinder23:08
*** xyang1 has quit IRC23:09
*** rady has quit IRC23:12
*** jwcroppe has quit IRC23:13
*** rlrossit has left #openstack-cinder23:14
*** dims has quit IRC23:15
*** esker has joined #openstack-cinder23:16
*** jdg has joined #openstack-cinder23:20
*** esker has quit IRC23:20
*** garthb__ has quit IRC23:22
*** merooeny_ has quit IRC23:23
*** garthb__ has joined #openstack-cinder23:23
*** stevemar_ has quit IRC23:25
*** stevemar_ has joined #openstack-cinder23:26
*** jdg has quit IRC23:29
*** stevemar_ has quit IRC23:30
*** lcurtis has quit IRC23:42
*** kfarr has quit IRC23:42
*** jwcroppe has joined #openstack-cinder23:43
*** salv-orlando has joined #openstack-cinder23:45
angela-ssmcginnis: thanks.  looks like i need to install py34 on my machine. :D  there is a directory for it, but it's empty.23:46
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Fetch and return SCSI WWN  https://review.openstack.org/24222123:46
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Add new Connector APIs for path validation  https://review.openstack.org/19976423:47
angela-ssmcginnis: definitely explains why it doesn't work. doh23:47
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Fix test_connect_volume when skip is bypassed  https://review.openstack.org/24333823:47
openstackgerritMatt Riedemann proposed openstack/cinder: Fix cinder objects unit test registration  https://review.openstack.org/24386823:49
*** yusuke has joined #openstack-cinder23:50
*** akshai has quit IRC23:51
*** chris_morrell has quit IRC23:54
angela-ssmcginnis: i'm confused, py3 is installed by default on ubuntu14 and when trying to install it, it says it's already installed.23:54
*** jwcroppe has quit IRC23:54
*** jwcroppe has joined #openstack-cinder23:55
openstackgerritMatt Riedemann proposed openstack/cinder: Test for object version hash changes  https://review.openstack.org/24382623:58
mriedemyou guys might want to prioritize that ^ to keep from breaking your object version compat23:58

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