Friday, 2016-02-05

*** haomaiwang has quit IRC00:01
*** haomaiwang has joined #openstack-cinder00:01
*** daneyon has quit IRC00:03
smcginnissavihou: Hi, welcome.00:05
smcginnissavihou: If you're running in to issues once you actually start running cinder we can definitely help.00:05
smcginnissavihou: But there are several here that have gone through setting up CI, so chances are you can get some pointers.00:05
openstackgerritScott DAngelo proposed openstack/cinder: cinder-api-microversions code
scottdasmcginnis: I believe we really will need a change in grenade for adding a /v3 endpoint:
scottdaHopefully that ^^ will help Jenkins and CI for the microversions patch00:07
*** lprice has joined #openstack-cinder00:08
*** lprice1 has quit IRC00:08
*** dalgaaf has quit IRC00:08
*** daneyon_ has quit IRC00:09
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Method not being called for V3
smcginnisscottda: OK, good. I'll weigh in on there and see how it goes.00:09
smcginnisscottda: Thanks!00:10
hemnanow I remember why I hate nfs so much00:12
anteayahemna: only now?00:13
*** alonma has joined #openstack-cinder00:14
hemnawell, I'm trying to get it to work so I can test something00:14
hemnahaven't used nfs in ages00:15
hemnasuch a pain00:15
*** alonma has quit IRC00:18
apoorvadsmcginnis thanks for your comments yesterday, I have replied to your questions and worked on suggestions. when you get a chance could you please look at it again? and
smcginnisapoorvad: Sure, I'll try to take a look soon. Thanks for updating them.00:22
*** mylu has quit IRC00:22
apoorvadsmcginnis: appreciate it :)00:22
*** gouthamr_ has joined #openstack-cinder00:22
*** alonma has joined #openstack-cinder00:24
hemnawhat a pain00:24
hemnaStale file handle00:24
*** gouthamr has quit IRC00:25
*** alonma has quit IRC00:28
*** edtubill has quit IRC00:29
*** alonma has joined #openstack-cinder00:33
*** smoriya_ has joined #openstack-cinder00:34
anteayahemna: are you experiencing pain?00:37
anteayais that what I'm hearing?00:37
*** alonma has quit IRC00:37
*** haomaiw__ has joined #openstack-cinder00:38
*** dims_ has joined #openstack-cinder00:39
*** haomaiwang has quit IRC00:40
*** dims has quit IRC00:40
*** alonma has joined #openstack-cinder00:42
*** alonma has quit IRC00:47
*** qeelee has joined #openstack-cinder00:47
*** alonma has joined #openstack-cinder00:50
*** salv-orl_ has quit IRC00:54
*** alonma has quit IRC00:55
*** alonma has joined #openstack-cinder00:57
*** mylu has joined #openstack-cinder00:59
*** haomaiw__ has quit IRC01:01
*** rockeychen has quit IRC01:01
*** alonma has quit IRC01:01
*** haomaiwa_ has joined #openstack-cinder01:01
*** rockeychen has joined #openstack-cinder01:01
*** chlong has joined #openstack-cinder01:01
*** dustins has quit IRC01:04
*** xyang has joined #openstack-cinder01:05
*** xyang has quit IRC01:06
*** daneyon has joined #openstack-cinder01:08
*** daneyon_ has joined #openstack-cinder01:09
*** cheneydc has joined #openstack-cinder01:11
*** daneyon has quit IRC01:13
jbernardDuncanT: so, rbd's iSCSI is a bit better than connecting the block devices and re-exporting those over iSCSI01:18
jbernardDuncanT: it uses both the rbd kernel module and the rbd-lio module01:18
*** daneyon_ has quit IRC01:18
jbernardDuncanT: so that targetcli can be used to create exports for rbd images01:19
jbernardDuncanT: and there is no local block device01:19
jbernardDuncanT: but it does require a dedicated node(s) to provide those exports01:19
jbernardDuncanT: so the solution is not present within the ceph cluster01:20
jbernardDuncanT: ive just been digging through the kernel code, i dont have a tutorial yes01:21
jbernardDuncanT: but i do plan to give this all a try soon and see how it goes01:21
jbernardDuncanT: s/yes/yet01:21
*** r-daneel has quit IRC01:24
*** garthb has quit IRC01:24
*** joqo78 has joined #openstack-cinder01:26
*** merooney has joined #openstack-cinder01:28
*** davechen has joined #openstack-cinder01:28
*** alonma has joined #openstack-cinder01:31
*** mtanino has quit IRC01:34
*** chlong has quit IRC01:34
*** chlong has joined #openstack-cinder01:35
*** alonma has quit IRC01:36
*** wanghao has quit IRC01:41
*** savihou has quit IRC01:41
*** wanghao has joined #openstack-cinder01:41
*** leeantho has quit IRC01:42
*** merooney has quit IRC01:43
*** mylu has quit IRC01:51
*** merooney has joined #openstack-cinder01:52
*** mudassirlatif has quit IRC01:54
*** laughterwym has joined #openstack-cinder01:58
*** mylu has joined #openstack-cinder01:59
*** itzdilip has quit IRC02:00
*** haomaiwa_ has quit IRC02:01
*** laughterwym has quit IRC02:01
*** haomaiwang has joined #openstack-cinder02:01
*** laughterwym has joined #openstack-cinder02:02
*** mylu_ has joined #openstack-cinder02:05
*** mylu has quit IRC02:05
*** edmondsw has quit IRC02:10
*** alkhodos has quit IRC02:17
*** baojg has joined #openstack-cinder02:18
*** itzdilip has joined #openstack-cinder02:25
*** apoorvad has quit IRC02:26
*** haomaiwang has quit IRC02:28
*** haomaiwang has joined #openstack-cinder02:29
*** apoorvad has joined #openstack-cinder02:29
*** alonma has joined #openstack-cinder02:31
*** alonma has quit IRC02:36
*** daneyon has joined #openstack-cinder02:37
*** mylu_ has quit IRC02:39
*** cknight1 has joined #openstack-cinder02:39
*** mylu has joined #openstack-cinder02:40
*** daneyon_ has joined #openstack-cinder02:41
*** cknight has quit IRC02:42
*** daneyon has quit IRC02:43
*** apoorvad has quit IRC02:47
*** haomaiwang has quit IRC03:01
*** haomaiwa_ has joined #openstack-cinder03:01
*** bardia has quit IRC03:03
*** haomaiwa_ has quit IRC03:05
*** mylu has quit IRC03:06
*** mylu has joined #openstack-cinder03:06
*** mylu has quit IRC03:11
*** mylu has joined #openstack-cinder03:12
*** rockeychen has quit IRC03:18
*** rockeychen has joined #openstack-cinder03:19
*** leeantho has joined #openstack-cinder03:31
*** alonma has joined #openstack-cinder03:32
*** dims_ has quit IRC03:33
*** alonma has quit IRC03:36
*** alonma has joined #openstack-cinder03:39
*** alonma has quit IRC03:44
*** qeelee has quit IRC03:47
*** daneyon_ has quit IRC03:57
*** baojg has quit IRC03:57
*** baojg has joined #openstack-cinder04:03
*** vgridnev has joined #openstack-cinder04:04
*** joqo78 has quit IRC04:06
*** cknight1 has quit IRC04:07
*** arch-nemesis has joined #openstack-cinder04:07
*** cknight has joined #openstack-cinder04:07
*** baojg has quit IRC04:08
*** baojg has joined #openstack-cinder04:11
*** yhayashi has joined #openstack-cinder04:25
*** shyama has joined #openstack-cinder04:26
*** 16WAAR2BV has joined #openstack-cinder04:27
*** baojg has quit IRC04:28
*** cknight has quit IRC04:34
*** baojg has joined #openstack-cinder04:39
*** alonma has joined #openstack-cinder04:40
*** links has joined #openstack-cinder04:43
*** alonma has quit IRC04:44
*** alonma has joined #openstack-cinder04:46
*** gouthamr_ has quit IRC04:49
*** alonma has quit IRC04:50
*** leeantho has quit IRC05:00
*** 16WAAR2BV has quit IRC05:01
*** haomaiwang has joined #openstack-cinder05:01
*** sheel has joined #openstack-cinder05:03
*** dalgaaf has joined #openstack-cinder05:05
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements
*** salv-orlando has joined #openstack-cinder05:08
*** laughter_ has joined #openstack-cinder05:12
*** laughterwym has quit IRC05:14
*** rockeychen has quit IRC05:16
*** rockeychen has joined #openstack-cinder05:17
*** mylu has quit IRC05:18
*** salv-orlando has quit IRC05:18
openstackgerritSheel Rana proposed openstack/cinder: Use of metadata id instead of metadata name
*** mylu has joined #openstack-cinder05:19
*** chhavi has joined #openstack-cinder05:29
*** sheel has quit IRC05:38
*** mylu has quit IRC05:41
*** mylu has joined #openstack-cinder05:42
*** alonma has joined #openstack-cinder05:46
*** merooney has quit IRC05:47
*** sgotliv has joined #openstack-cinder05:48
*** alonma has quit IRC05:50
*** alonma has joined #openstack-cinder05:52
*** qeelee has joined #openstack-cinder05:53
*** alonma has quit IRC05:56
*** alonma has joined #openstack-cinder05:59
*** shyama has quit IRC06:00
*** haomaiwang has quit IRC06:01
*** shyama has joined #openstack-cinder06:01
*** haomaiwang has joined #openstack-cinder06:01
*** wanghao has quit IRC06:03
*** wanghao has joined #openstack-cinder06:03
*** shyama has quit IRC06:04
*** shyama has joined #openstack-cinder06:05
*** Poornima has joined #openstack-cinder06:09
*** mylu has quit IRC06:18
*** laughter_ has quit IRC06:19
*** nkrinner has joined #openstack-cinder06:19
*** laughterwym has joined #openstack-cinder06:19
openstackgerritzwei proposed openstack/cinder: When volume_backend_name is None Error
openstackgerritMerged openstack/cinder: Remove useless unit tests mock in Huawei driver
*** alonma has quit IRC06:21
*** alonma has joined #openstack-cinder06:21
*** alonma has quit IRC06:25
*** esp has joined #openstack-cinder06:26
*** alonma has joined #openstack-cinder06:27
*** ChubYann has quit IRC06:30
*** bardia has joined #openstack-cinder06:31
*** alonma has quit IRC06:32
PoornimaHi folks please review :
*** esp has quit IRC06:33
*** alonma has joined #openstack-cinder06:34
*** sgotliv has quit IRC06:35
*** wanghao has quit IRC06:35
*** wanghao has joined #openstack-cinder06:36
*** alonma has quit IRC06:38
wanghaoPoornima: some nits found inline, please have a look.06:40
Poornimathanks wanghao will fix them06:42
wanghaoPoornima: you're welcome.06:42
*** alonma has joined #openstack-cinder06:43
swamireddyPoornima: patch looks fine. found a few nits. Please check.06:45
Poornimaswamireddy, sure thanks :)06:46
*** alonma has quit IRC06:47
*** jwcroppe has joined #openstack-cinder06:58
*** sheel has joined #openstack-cinder07:00
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-cinder07:01
*** qeelee has quit IRC07:01
*** rockeychen has quit IRC07:02
*** anshul has joined #openstack-cinder07:03
*** sheel has quit IRC07:04
*** links has quit IRC07:04
*** bardia has quit IRC07:05
*** stevemar has joined #openstack-cinder07:06
*** baojg has quit IRC07:07
*** lpetrut has joined #openstack-cinder07:08
*** links has joined #openstack-cinder07:14
*** qeelee has joined #openstack-cinder07:15
*** baojg has joined #openstack-cinder07:16
*** belmoreira has joined #openstack-cinder07:29
*** e0ne has joined #openstack-cinder07:30
*** qeelee has quit IRC07:32
openstackgerritwanghao proposed openstack/cinder: Clean up image tmp file if c-vol gets restarted
*** ociuhandu has joined #openstack-cinder07:38
*** alonma has joined #openstack-cinder07:43
*** baojg has quit IRC07:45
*** alonma has quit IRC07:47
*** alonma has joined #openstack-cinder07:52
*** vgridnev has quit IRC07:54
*** baojg has joined #openstack-cinder07:56
*** alonma has quit IRC07:56
*** alonma has joined #openstack-cinder07:58
*** haomaiwang has quit IRC08:01
*** haomaiwang has joined #openstack-cinder08:01
*** alonma has quit IRC08:03
*** alonma has joined #openstack-cinder08:05
*** savihou has joined #openstack-cinder08:06
*** sheel has joined #openstack-cinder08:06
*** Poornima has quit IRC08:09
*** arecknag has joined #openstack-cinder08:09
*** alonma has quit IRC08:09
sheelameade: hi08:11
*** arecknag has quit IRC08:15
*** arecknag has joined #openstack-cinder08:15
*** alonma has joined #openstack-cinder08:15
*** laughterwym has quit IRC08:19
*** jwcroppe has quit IRC08:19
*** alonma has quit IRC08:20
*** arecknag has quit IRC08:21
*** alonma has joined #openstack-cinder08:23
*** arecknag has joined #openstack-cinder08:24
*** geguileo has joined #openstack-cinder08:25
openstackgerritLiucheng Jiang proposed openstack/cinder: Huawei: Implement v2 replication (managed)
*** alonma has quit IRC08:27
*** Poornima has joined #openstack-cinder08:29
*** lpetrut has quit IRC08:29
flip214some cores want to help with DRBD, ?08:30
*** ildikov has quit IRC08:30
flip214only +3 lines, a compat change for newer drbdmanage versions... needed so that I can push other fixes in.08:31
*** sgotliv has joined #openstack-cinder08:32
*** salv-orlando has joined #openstack-cinder08:45
*** salv-orlando has quit IRC08:49
*** savihou has quit IRC08:49
*** savihou has joined #openstack-cinder08:50
*** Poornima has quit IRC08:50
*** Poornima has joined #openstack-cinder08:51
*** chlong has quit IRC08:52
*** savihou has quit IRC08:52
*** savihou has joined #openstack-cinder08:53
*** vgridnev has joined #openstack-cinder08:58
*** salv-orlando has joined #openstack-cinder09:00
*** haomaiwang has quit IRC09:01
*** haomaiwa_ has joined #openstack-cinder09:01
*** EinstCrazy has quit IRC09:01
*** EinstCrazy has joined #openstack-cinder09:02
openstackgerritMerged openstack/cinder: Updated from global requirements
*** yhayashi has quit IRC09:07
*** EinstCrazy has quit IRC09:07
*** baojg has quit IRC09:10
openstackgerritchenzongliang proposed openstack/cinder: Huawei: Support huawei consistency group
*** baojg has joined #openstack-cinder09:14
*** openstackgerrit has quit IRC09:17
*** openstackgerrit has joined #openstack-cinder09:18
*** ociuhandu has quit IRC09:18
*** markvoelker has quit IRC09:20
*** jordanP has joined #openstack-cinder09:23
*** alonma has joined #openstack-cinder09:24
*** baojg has quit IRC09:25
openstackgerritJinxing Fang proposed openstack/cinder: Update the home page
*** alonma has quit IRC09:28
*** ildikov has joined #openstack-cinder09:28
*** arch-nemesis has quit IRC09:33
*** stevemar has quit IRC09:35
*** stevemar has joined #openstack-cinder09:36
*** baojg has joined #openstack-cinder09:36
mkodererDuncanT: btw I added the unit test in
*** stevemar has quit IRC09:40
*** ociuhandu has joined #openstack-cinder09:52
*** boris-42 has joined #openstack-cinder09:52
*** sheel has quit IRC09:54
*** davechen has left #openstack-cinder09:55
*** cheneydc has quit IRC09:56
*** haomaiwa_ has quit IRC10:01
*** mvk has quit IRC10:01
*** haomaiwang has joined #openstack-cinder10:01
*** ankit_ag has joined #openstack-cinder10:08
*** ociuhandu has quit IRC10:08
*** baojg has quit IRC10:14
*** baojg has joined #openstack-cinder10:16
*** lpetrut has joined #openstack-cinder10:17
*** alonma has joined #openstack-cinder10:18
*** dalgaaf has quit IRC10:18
*** markvoelker has joined #openstack-cinder10:21
*** EinstCrazy has joined #openstack-cinder10:22
*** alonma has quit IRC10:22
*** markvoelker has quit IRC10:26
*** mvk has joined #openstack-cinder10:28
openstackgerritMatan Sabag proposed openstack/cinder: Support for consistency groups in ScaleIO driver
*** EinstCrazy has quit IRC10:35
*** EinstCrazy has joined #openstack-cinder10:36
*** lpetrut has quit IRC10:36
*** stevemar has joined #openstack-cinder10:37
*** salv-orl_ has joined #openstack-cinder10:39
*** stevemar has quit IRC10:40
*** EinstCrazy has quit IRC10:40
*** salv-orlando has quit IRC10:42
*** tpsilva has joined #openstack-cinder10:51
*** dulek has joined #openstack-cinder10:52
*** huanan_L has quit IRC10:56
*** huanan has joined #openstack-cinder10:57
*** e0ne has quit IRC10:57
*** haomaiwang has quit IRC11:01
*** haomaiwang has joined #openstack-cinder11:01
*** geguileo has quit IRC11:01
*** zhonghua has joined #openstack-cinder11:09
*** aagrawal has joined #openstack-cinder11:09
DuncanTmkoderer: Thanks. Quick question: Shouldn't line 108 be warning about the import of '_'?11:12
*** ankit_ag has quit IRC11:12
*** skraynev has quit IRC11:12
*** zhonghua-lee has quit IRC11:12
*** dobson has quit IRC11:12
*** mvk has quit IRC11:12
*** ccesario has quit IRC11:12
*** alonma has joined #openstack-cinder11:18
openstackgerritMatan Sabag proposed openstack/cinder: Manage/unmanage volume in ScaleIO driver
*** Poornima has quit IRC11:19
*** laughterwym has joined #openstack-cinder11:20
*** alonma has quit IRC11:22
openstackgerritMatan Sabag proposed openstack/cinder: Support for consistency groups in ScaleIO driver
*** laughterwym has quit IRC11:24
*** mvk has joined #openstack-cinder11:24
*** skraynev has joined #openstack-cinder11:24
*** ccesario has joined #openstack-cinder11:25
*** bkumar has joined #openstack-cinder11:25
*** jwcroppe has joined #openstack-cinder11:26
*** dobson has joined #openstack-cinder11:27
*** alonma has joined #openstack-cinder11:28
*** alonma has quit IRC11:32
*** jwcroppe has quit IRC11:32
*** jwcroppe has joined #openstack-cinder11:33
*** jwcroppe has quit IRC11:37
*** wanghao has quit IRC11:43
*** wanghao has joined #openstack-cinder11:45
*** dims has joined #openstack-cinder11:46
*** baojg has quit IRC11:48
*** ociuhandu has joined #openstack-cinder11:51
*** baojg has joined #openstack-cinder11:51
*** lpetrut has joined #openstack-cinder11:53
*** sgotliv has quit IRC11:54
*** bkumar has quit IRC11:55
*** e0ne has joined #openstack-cinder11:55
*** raildo-afk is now known as raildo11:58
*** haomaiwang has quit IRC12:01
*** salv-orl_ has quit IRC12:01
*** haomaiwang has joined #openstack-cinder12:01
*** thurloat is now known as thurloat_isgone12:04
*** baojg has quit IRC12:09
*** markvoelker has joined #openstack-cinder12:22
*** baojg has joined #openstack-cinder12:26
*** markvoelker has quit IRC12:27
*** alonma has joined #openstack-cinder12:28
*** EinstCrazy has joined #openstack-cinder12:29
*** salv-orlando has joined #openstack-cinder12:29
*** vgridnev has quit IRC12:32
*** alyson_ has quit IRC12:32
*** alonma has quit IRC12:33
*** stevemar has joined #openstack-cinder12:35
*** sinese has joined #openstack-cinder12:35
*** vgridnev has joined #openstack-cinder12:36
*** EinstCrazy has quit IRC12:36
*** alonma has joined #openstack-cinder12:37
*** stevemar has quit IRC12:38
*** baojg has quit IRC12:39
*** alonma has quit IRC12:41
*** links has quit IRC12:43
*** baojg has joined #openstack-cinder12:44
*** savihou has quit IRC12:44
*** laughterwym has joined #openstack-cinder12:46
*** baojg has quit IRC12:47
*** mtanino has joined #openstack-cinder12:50
*** smoriya_ has quit IRC12:54
*** erlon has joined #openstack-cinder12:56
*** sinese_ has joined #openstack-cinder13:00
*** haomaiwang has quit IRC13:01
*** krotscheck has joined #openstack-cinder13:01
*** laughterwym has quit IRC13:01
*** haomaiwang has joined #openstack-cinder13:01
*** sinese has quit IRC13:01
*** laughterwym has joined #openstack-cinder13:01
*** baojg has joined #openstack-cinder13:02
*** laughterwym has quit IRC13:06
*** esp has joined #openstack-cinder13:12
*** salv-orlando has quit IRC13:13
*** laughterwym has joined #openstack-cinder13:13
*** aagrawal has quit IRC13:14
*** akerr has joined #openstack-cinder13:15
*** markvoelker has joined #openstack-cinder13:22
*** boris-42 has quit IRC13:23
*** thurloat_isgone is now known as thurloat13:24
*** markvoelker_ has joined #openstack-cinder13:24
*** markvoelker has quit IRC13:24
*** esp has quit IRC13:27
*** xyang1 has joined #openstack-cinder13:30
*** edmondsw has joined #openstack-cinder13:34
*** takedakn has joined #openstack-cinder13:34
*** alonma has joined #openstack-cinder13:37
*** fthiagogv_ has quit IRC13:38
*** fthiagogv has joined #openstack-cinder13:39
mtaninoDuncanT: hi, are you there?13:40
*** diablo_rojo has joined #openstack-cinder13:40
*** alonma has quit IRC13:41
*** laughterwym has quit IRC13:42
*** laughterwym has joined #openstack-cinder13:43
mtaninoDuncanT: If you have a chance, please revisit glance metadata support?
*** laughterwym has quit IRC13:47
*** laughterwym has joined #openstack-cinder13:49
*** gouthamr has joined #openstack-cinder13:51
*** timcl has joined #openstack-cinder13:51
*** jwang_ has joined #openstack-cinder13:53
*** julim has joined #openstack-cinder13:54
*** gouthamr has quit IRC13:55
*** geguileo has joined #openstack-cinder13:56
*** jwang__ has quit IRC13:57
openstackgerritThang Pham proposed openstack/cinder: Update initialize_connection to use versionedobjects
*** eharney has joined #openstack-cinder13:59
*** takedakn has quit IRC14:00
*** laughterwym has quit IRC14:00
*** haomaiwang has quit IRC14:01
*** laughterwym has joined #openstack-cinder14:01
*** haomaiwang has joined #openstack-cinder14:01
scottdae0ne: Can you ping me when you have a couple minutes?14:03
*** sheel has joined #openstack-cinder14:04
*** laughterwym has quit IRC14:06
e0nescottda: hi14:09
scottdae0ne: Hi. I've added /v3 endpoint to microversions
scottdaThe unit tests pass on my workstation, but they are failing in Jenkins...14:10
scottdaI've fixed one problem (I think) by looking at the code. There is one other issue that I am not sure about.14:10
openstackgerritSzymon Borkowski proposed openstack/cinder: Fix for glance_metadata during volume migration
scottdaI was wondering if you had any ideas why tests can pass for me but not for Jenkins?14:10
*** rlrossit has joined #openstack-cinder14:11
*** sinese_ has quit IRC14:11
scottdaIF you get some time, let me know, I can show you some details.14:12
*** sinese_ has joined #openstack-cinder14:12
*** sinese_ has quit IRC14:12
*** takedakn has joined #openstack-cinder14:12
*** sinese_ has joined #openstack-cinder14:12
*** sinese_ has quit IRC14:12
*** sgotliv has joined #openstack-cinder14:13
e0nescottda: why do we want api v3 now?14:13
scottdaWe discussed this at the midcycle...14:13
scottdaFor microversions, we want it to be explicit that the user is connecting to a new endpoint....14:13
*** salv-orlando has joined #openstack-cinder14:13
e0nescottda: I'm sorry, I've missed the decision:(. my fail14:13
scottdaOtherwise, a newer client might request a microversion and expect new behavior....14:14
e0neit makes sense14:14
scottdaand if it is an older server, it will fail silently14:14
scottdaNew /v3 would start identical to /v2 and change as microversions are added.14:14
scottdaSo I've attempted to add /v3 code, and basic testing works, so do my workstation unit tests. But there are problems with Jenkins and CI14:15
scottdaI thought you had a lot of expertise with your work to deprecate /v1 (not abandonede?) and unit tests, etc.14:15
*** mkoderer__ has quit IRC14:16
e0nedid you change a defaule version to v3 in your code?14:17
e0neor did you create any new keystone endpoints on your env?14:17
*** mkoderer__ has joined #openstack-cinder14:17
e0nescottda: looks like openstack client can't work with your changes to the api14:19
scottdaI did not set any default here:
*** merooney has joined #openstack-cinder14:19
scottdaDoes Jenkins use OSC?14:19
e0neso, you disabled all APIs by default :)14:19
e0nescottda: yes, it uses only OSC during devstack setup14:20
scottdaOK, that would explain some of the failures14:20
e0nescottda: you have to enable APIv2 at least14:20
e0nebut I'm pretty sure that some tempest tests still uses APIv1 too14:20
*** porrua has joined #openstack-cinder14:22
*** baojg has quit IRC14:22
scottdaI did not disable /v1 or /v2. At least they both work on my workstation.14:22
*** nkrinner has quit IRC14:24
*** haomaiwang has quit IRC14:24
e0neand there is no overrides in config:
*** raildo is now known as raildo-afk14:25
e0neoh! got it! you've changed
e0nenot sure how it works now:(14:26
e0ne - your c-api didn't started14:27
*** baojg has joined #openstack-cinder14:29
e0nescottda: - IMO, it's a root cause of failures14:30
*** salv-orlando has quit IRC14:30
*** lprice has quit IRC14:30
e0nescottda: both for tempest and "unit"-tests14:30
*** takedakn has quit IRC14:31
scottdaOK, yes. I saw that and fixed it locally, so I figured that would solve many of the unit tests failures....14:31
scottdalike these:14:32
*** arecknag has quit IRC14:33
openstackgerritMerged openstack/cinder-specs: Support query volume filter by glance metadata
scottdae0ne: I've made this change:14:33
scottdaThe troublesome thing is that I cannot repro the issues locally, so I cannot test this (or any other) fix.14:35
*** alaski is now known as lascii14:35
scottdae0ne: If you think that fix will solve most of the problems, I can push it up and see. I just wish I could test before doing that.14:35
scottdae0ne: I also added a dependency for Grenade to update api-paste.ini with the new endpoint info:
*** thangp has joined #openstack-cinder14:37
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Implement v2 replication
*** alonma has joined #openstack-cinder14:37
*** dustins has joined #openstack-cinder14:39
*** xyang1 has quit IRC14:41
*** alonma has quit IRC14:41
sheelDear Core members: Please have a look at14:42
sheel1. (final +2 required)14:42
sheel2. (final +2 required),14:42
*** jgregor has joined #openstack-cinder14:42
*** baojg has quit IRC14:43
e0nescottda: AFIAK, cinder/ is used only for config generator14:43
scottdae0ne: Are you saying that the patch above is not a fix for the query_volume_filters being registered twice?14:44
diablo_rojoe0ne: scottda that is correct. It is only used for the config generator to generate the sample config opt file14:45
e0nescottda: yes, if I didn't miss anything14:45
e0neeharney: hi! I've answered your comments.14:45
*** bkumar has joined #openstack-cinder14:45
e0neeharney: what is the isssue with osprofiler?14:45
eharneye0ne: that's what we determined was breaking this yesterday14:46
smcginnise0ne: staticmethod is fine, osprofiler doesn't decorate it right.14:46
*** baojg has joined #openstack-cinder14:46
openstackLaunchpad bug 1440027 in osprofiler "osprofiler is Incompatibe with staticmethod within a class" [Undecided,New]14:46
e0nethanks! I'll ask about is to osprofiler team14:46
smcginnisI've just proposed a requirements change:
*** alonma has joined #openstack-cinder14:47
scottdahmm...OK, my bad. e0ne I reckon I should remove it from here:14:47
e0nescottda: yes. it should help14:47
smcginnisI'm behind removing the staticmethod as a workaround, mainly to get my CI to stop failing. ;)14:47
smcginnisBut there are many places in the code where we have static methods.14:47
e0nescottda: or may be move it to some more general place14:47
smcginnisSo this will likely fail somewhere else as well.14:47
eharneydid anyone make sure it's not going to just fail on another test?14:47
smcginniseharney: I'm pretty confident it will, so this just works around a specific case.14:48
e0nesmcginnis: sometimes, we use staticmethods where it's not needed14:48
smcginnisThe reality is, we either need osprofiler fixed or we need to rollback to pre-1.0.014:48
smcginnise0ne: That's a language theory argument. :)14:48
eharneye0ne: that's the sentiment i can't really understand here14:49
smcginnise0ne: It is definitely used correctly.14:49
scottdae0ne: any thoughts on a better place to register query_volume_filters_opt ?14:49
eharneyi mean, most of python is not "needed", it's there to help us produce better code14:49
e0nesmcginnis, eharney:  I don't want to debate on it becase looks like only me thinks that we've got not readable code there. probably, I have to abandone that patch14:50
e0neeharney: I prefer to use simple functions without classes where possible. but it's only IMHO14:51
smcginnise0ne: Maybe. It was very useful though. It definitely helped validate that that was the cause of the failure.14:51
*** alonma has quit IRC14:51
*** cknight has joined #openstack-cinder14:52
e0nescottda: maybe cinder/common/config.py14:52
e0nescottda: or cinder/api/common.py14:52
e0nescottda: the second one looks better for me14:52
flip214e0ne: smcginnis: thank you very much for your help!14:53
smcginnisflip214: np14:53
e0neflip214: you're welcome!14:54
*** bkumar4 has joined #openstack-cinder14:54
*** esp has joined #openstack-cinder14:54
scottdae0ne: I'll move it. Thanks a lot for all the help!14:55
e0nescottda: np. I'm glad to be useful:)14:56
*** jordanP has quit IRC14:56
*** bkumar has quit IRC14:56
*** mriedem has joined #openstack-cinder14:57
e0nesmcginnis: I'm sorry, -1 with a question to your patch14:57
smcginnise0ne: Thanks! My mistake.14:58
*** haomaiwang has joined #openstack-cinder14:59
*** crose has joined #openstack-cinder14:59
*** haomaiwang has quit IRC15:01
*** edtubill has joined #openstack-cinder15:03
*** esp has quit IRC15:05
*** anshul has quit IRC15:06
diablo_rojoscottda: Cadillac Cadillac..15:11
*** dguryanov has quit IRC15:12
diablo_rojoHow do you rerun only the unit tests that failed again?15:16
smcginnisdiablo_rojo: Running tests locally?15:16
diablo_rojosmcginnis: Correct15:16
sheeltox -epy27 -- --no-discover test_case_location15:17
smcginnissheel: Quicker than me. ;)15:17
sheelsmcginnis: i was running this on my local just now15:18
smcginnisdiablo_rojo: Some details here:
diablo_rojosmcginnis: sheel Thank you :)15:18
sheeldiablo_rojo: happy to help.. welcome15:19
*** baojg has quit IRC15:19
*** ildikov has quit IRC15:20
*** xyang1 has joined #openstack-cinder15:24
*** bkumar4 has quit IRC15:25
*** baojg has joined #openstack-cinder15:25
*** bkumar4 has joined #openstack-cinder15:25
*** esker has quit IRC15:26
openstackgerritAlex Meade proposed openstack/cinder-specs: User facing error messages
*** esker has joined #openstack-cinder15:26
*** baumann has joined #openstack-cinder15:28
scottdadiablo_rojo: I'm not sure what to have for lunch, ham sandwich?15:28
*** timcl has quit IRC15:30
*** xyang has joined #openstack-cinder15:30
diablo_rojoscottda: Only if you can clap with no hands15:31
*** jwcroppe has joined #openstack-cinder15:31
*** eharney has quit IRC15:33
*** haomaiwang has joined #openstack-cinder15:33
*** xyang has quit IRC15:36
*** timcl has joined #openstack-cinder15:40
*** arch-nemesis has joined #openstack-cinder15:42
*** sgotliv has quit IRC15:46
*** alonma has joined #openstack-cinder15:47
*** david-lyle has joined #openstack-cinder15:47
*** sgotliv has joined #openstack-cinder15:47
*** mragupat has joined #openstack-cinder15:48
*** sinese has joined #openstack-cinder15:49
*** bkumar4 has quit IRC15:50
*** jungleboyj has joined #openstack-cinder15:51
*** alonma has quit IRC15:52
*** merooney has quit IRC15:53
*** merooney has joined #openstack-cinder15:54
*** mgarza has joined #openstack-cinder15:55
*** thurloat is now known as thurloat_isgone15:56
*** nkrinner has joined #openstack-cinder15:57
*** jdurgin1 has joined #openstack-cinder15:59
*** tongli has quit IRC16:00
*** logan_ has joined #openstack-cinder16:00
*** logan_ is now known as Guest7865416:01
*** haomaiw__ has joined #openstack-cinder16:01
*** logan- has quit IRC16:01
*** haomaiwang has quit IRC16:02
*** baojg has quit IRC16:02
*** tongli has joined #openstack-cinder16:02
*** TheJulia_ has joined #openstack-cinder16:02
*** Guest78654 is now known as logan-16:02
*** sinese_ has joined #openstack-cinder16:02
*** aarefiev_ has joined #openstack-cinder16:03
*** frakt_ has joined #openstack-cinder16:03
*** TheJulia has quit IRC16:04
*** fernnest__ has joined #openstack-cinder16:04
*** cburgess_ has joined #openstack-cinder16:05
*** TheJulia_ is now known as TheJulia16:05
*** sinese_ is now known as sinise16:08
*** akerr_ has joined #openstack-cinder16:08
*** pratap_ has joined #openstack-cinder16:08
*** belmoreira has quit IRC16:13
*** sinese has quit IRC16:14
*** akerr has quit IRC16:14
*** fernnest_ has quit IRC16:14
*** frakt has quit IRC16:14
*** pratap has quit IRC16:14
*** cburgess has quit IRC16:14
*** aarefiev has quit IRC16:14
*** Kimmo_ has quit IRC16:14
*** sinise has quit IRC16:16
*** baojg has joined #openstack-cinder16:17
*** xyang has joined #openstack-cinder16:19
*** vgridnev has quit IRC16:19
*** david-lyle has quit IRC16:23
*** garthb has joined #openstack-cinder16:24
jungleboyjameade: You around?16:25
*** merooney has quit IRC16:27
*** raildo-afk is now known as raildo16:27
*** timcl has quit IRC16:27
ameadejungleboyj: yesish16:28
ameadewhat's up?16:28
jungleboyjameade: Just wanted to check on a couple of things w/r/t the cDOT driver.16:28
*** david-lyle has joined #openstack-cinder16:29
*** baojg has quit IRC16:29
ameadejungleboyj: shoot :)16:30
jungleboyjameade: Are you guys going to be able to get replication in at the FlexVol level for Mitaka?16:30
ameadejungleboyj: no not for mitaka16:30
jungleboyjameade: Doh!  I thought you guys were going to have that in.16:32
*** baojg has joined #openstack-cinder16:33
ameadejungleboyj: :/16:33
ameadetight resources atm16:33
*** mragupat has quit IRC16:34
ameadewe will have it in our manila cdot driver :) :P16:34
jungleboyjameade: :-(  Ok, you did get the QoS changes that IBM was looking for addressed though.  Right?16:34
*** mragupat has joined #openstack-cinder16:34
*** tongli has quit IRC16:35
ameadejungleboyj: for the most part, not upstream yet16:35
jungleboyjameade: SoftLayer has code though.  Right?16:35
ameadejungleboyj: yes16:36
jungleboyjameade: Is that code against Liberty then?16:36
*** e0ne has quit IRC16:37
ameadei should double check we have a bp for this "maxIOPSperGB"16:37
jungleboyjameade: That would be good.16:39
jungleboyjameade: Still planning to get replication for cDOT into Newton but it will be at the flexvol level?16:39
*** baojg has quit IRC16:41
ameadejungleboyj: I believe that to be one of our big priorities for Newton16:42
jungleboyjameade: Ok.  Thanks.  Have a PM in my ear needing info.16:42
ameadejungleboyj: haha yep, totally get that16:42
ameadejungleboyj: heres this btw:
*** baojg has joined #openstack-cinder16:44
jungleboyjameade: Thank you.16:47
hemnalooks like gorka's Remove API races * patches are all merge conflicted now16:49
jungleboyjI feel very conflicted about that.16:52
*** e0ne has joined #openstack-cinder16:52
*** baojg has quit IRC16:53
*** dulek has quit IRC16:56
*** crose has quit IRC16:57
*** haomaiw__ has quit IRC17:01
*** haomaiwang has joined #openstack-cinder17:01
*** bluex has quit IRC17:01
jungleboyjameade: If you can keep me in the loop on those patches I can maybe keep other IBMers off your back.  ;-)17:02
*** crose has joined #openstack-cinder17:03
*** stevemar has joined #openstack-cinder17:03
*** ildikov has joined #openstack-cinder17:10
*** timcl has joined #openstack-cinder17:13
*** tongli has joined #openstack-cinder17:14
*** david-lyle has quit IRC17:14
*** stevemar has quit IRC17:17
*** stevemar has joined #openstack-cinder17:18
e0nesmcginnis, hemna: could you please review - Introduce functional tests for python-brick-cinderclient-ext17:21
*** david-lyle has joined #openstack-cinder17:21
*** leeantho has joined #openstack-cinder17:22
*** stevemar has quit IRC17:22
openstackgerritAnthony Lee proposed openstack/cinder: LeftHand: Implement un/manage snapshot support
*** david-lyle has quit IRC17:25
mc_nairhas anyone run into the issue of trying to access a config option before the config file has been loaded, so the config value is set to the default instead of what's specified in cinder.conf?17:28
mc_nairit's got me pretty stumped right now17:29
*** mudassirlatif has joined #openstack-cinder17:30
openstackgerritxing-yang proposed openstack/cinder: Update db in CGSnapshot create
*** mriedem is now known as mriedem_afk17:31
*** vgridnev has joined #openstack-cinder17:31
*** apoorvad has joined #openstack-cinder17:33
sheelmc_nair: If I got your point right, you want to set some default value even before start of cinder services(config value are loaded during start up)17:34
sheelmc_nair: or you have some other use case?17:35
*** alonma has joined #openstack-cinder17:35
mc_nairsheel: so the issue I'm having I think boils down to the fact that QuotaEngine globally ( so when we get to QuotaEngine's __init__ I don't think the config values are loaded yet17:35
mc_nairso this check isn't working
mc_nairsheel: but I think you've got the crux of the problem17:36
sheelmc_nair: why you want to init these values...are you going to use these globally somewhere...17:39
sheelmc_nair: even before any quota call...17:39
sheelmc_nair: I am aksing these many question as have less idea about quota working..17:40
*** alonma has quit IRC17:40
*** alonma has joined #openstack-cinder17:41
mc_nairsheel: fair question.  So basically the way the quota code currently works is there's a QuotaEngine which has a QuotaDriver that it uses.  There's existing code in QuotaEngine startup *tries* to look a config for which quota_driver to use, instantiates it, and will use it for all the rest of the quota calls17:42
mc_nairsheel: but currently we instantiate the QuotaEngines immediately when is loaded -
mc_nairsheel: so at that point, I *think* the QuotaEngine is always saying "What driver should I use?", but it's always the default config value because CONF hasn't been loaded yet17:44
*** eharney has joined #openstack-cinder17:45
*** alonma has quit IRC17:45
sheelmc_nair: may be its ok to set some default value for time being17:48
*** alonma has joined #openstack-cinder17:48
sheelmc_nair: and get this loaded from config later during quota call17:48
mc_nairsheel: yea, but I guess then we'd need to be re-loading the QuotaDriver (or at least do a check for it) on a quota call with isn't great17:49
sheelmc_nair: hmm right17:50
mc_nairsheel: but I think you're on the right track... maybe we need to change when the QuotaEngine gets initialized17:50
mc_nairsheel: I just was hoping there'd be an easy way to say "Load config, now!!" and then I could use the existing code that's in there :/17:50
*** e0ne has quit IRC17:51
sheelmc_nair: haha ...17:51
sheelmc_nair: I will check if there is something around for solving this...will ping you in case got success.17:52
*** alonma has quit IRC17:52
sheelmc_nair: will see if some code tweak is possible..17:53
openstackgerritSean McGinnis proposed openstack/cinder: WIP: Add in-tree tempest tests
*** alonma has joined #openstack-cinder17:55
mc_nairsheel: that would be awesome.  I'll poke around see if there's any simple things that would push the QuotaEngine instantiation later17:55
*** salv-orlando has joined #openstack-cinder17:55
sheelmc_nair: yup..17:55
SwansonRe replication v2.1 : After failover how do I know I'm in a failover state?  If I am failed over but I have non replicated volumes should I try to use them on the failed backend?17:56
*** chhavi has quit IRC17:56
*** gouthamr has joined #openstack-cinder17:59
*** alonma has quit IRC17:59
*** yuriy_n17 has quit IRC17:59
*** chhavi has joined #openstack-cinder18:00
*** haomaiwang has quit IRC18:01
*** 77CAAFNN9 has joined #openstack-cinder18:01
*** alonma has joined #openstack-cinder18:02
*** Kimmo_ has joined #openstack-cinder18:03
openstackgerritApoorva Deshpande proposed openstack/cinder: Tintri image direct clone
*** crose has quit IRC18:06
*** alonma has quit IRC18:07
*** vgridnev has quit IRC18:07
*** alonma has joined #openstack-cinder18:08
*** mvk has quit IRC18:09
*** lpetrut has quit IRC18:11
*** alonma has quit IRC18:13
*** dansmith is now known as superdan18:16
mtaninojgriffith_away: If you have a chance, could you give your opinion for LIO target problem?
*** jkraj has joined #openstack-cinder18:22
*** stevemar has joined #openstack-cinder18:25
*** liverpooler has joined #openstack-cinder18:27
*** liverpoo1er has joined #openstack-cinder18:28
*** dims is now known as dimsum__18:30
*** shyama has quit IRC18:30
*** ociuhandu has quit IRC18:36
*** sgotliv has quit IRC18:43
*** angela-s has joined #openstack-cinder18:44
*** geguileo has quit IRC18:45
*** xyang has quit IRC18:45
*** mylu has joined #openstack-cinder18:47
*** bardia has joined #openstack-cinder18:48
*** e0ne has joined #openstack-cinder18:50
*** xyang has joined #openstack-cinder18:51
*** mylu has quit IRC18:52
*** lpetrut has joined #openstack-cinder18:54
openstackgerritIvan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: Introduce functional tests for python-brick-cinderclient-ext
*** vgridnev has joined #openstack-cinder18:55
*** mylu has joined #openstack-cinder18:55
*** 77CAAFNN9 has quit IRC19:01
*** ociuhandu has joined #openstack-cinder19:01
*** mylu has quit IRC19:01
*** haomaiwang has joined #openstack-cinder19:01
hemnahuh ok19:03
hemnaso it looks like nova doesn't use os-brick for nfs19:03
* hemna has a sad19:03
*** mylu has joined #openstack-cinder19:03
patrickeasthemna: that would make way too much sense19:04
*** chhavi has quit IRC19:06
*** vgridnev has quit IRC19:06
hemnaand...the cinder nfs driver has it's own copy_volume_to_image19:07
hemnathat doesn't use os-brick19:07
hemnaso....the brick remotefs connector isn't used.19:07
hemnaI have no way to test the remotefs privsep code :(!!!19:08
patrickeastwell if no one is using it.... does it need to work? :P19:08
openstackgerritAnthony Lee proposed openstack/os-brick: Fix output returned from get_all_available_volumes
hemnaI wonder19:09
*** alonma has joined #openstack-cinder19:09
hemnaif I can use e0ne's attach w/o nova patch to cinderclient19:09
e0nehemna: you can19:09
e0nehemna: that's why I'm using umount directly in my code:(19:09
hemnaI think that's the only way to excercize this19:10
hemnaand there is no CI for it19:10
e0nehemna: now, I'm feeling OK to modify disconnect_volume for NFS connector19:10
hemnathat guy19:10
hemnaI wanted to get that tested19:10
hemnato land19:10
hemnaas a blueprint of how to migrate iSCSI and FC19:11
e0neCI will be once we merge
e0neI'm not sure that we need functional tests for os-brick too19:11
hemnae0ne, do you have the url for your cinderclient patch handy?19:11
hemnamaybe I can try that real quick on my same VM that I have nfs setup19:11
e0neit's a bit hacky, so still in WIP19:12
e0neI have to use rootwrap or privsep to remove hacks19:12
hemnaok I have the repo cloned and that patch pulled19:13
hemnapip install . ?19:13
hemnais that sufficient for cinderclient to get it ?19:13
e0neyes, you need "pip install ." for the extension19:14
*** nkrinner has quit IRC19:14
hemnacinder local-attach ....19:14
e0nethe lastes patch doesn't support nfs19:14
e0nepatch #8 does:
*** alonma has quit IRC19:15
eharneyis any of this brick connector code related to NFS accounting for it using files that aren't raw-formatted?19:17
*** alonma has joined #openstack-cinder19:17
hemnaeharney, I can't find where the remotefs connector is used19:17
hemnathe cinder nfs driver doesn't seem to use it19:17
hemnaand nova doesn't seem to use it.19:17
e0neI use it in brickclient19:18
hemnathat doesn't use it.19:18
eharneywhy is this a wtf, this code has been around forever19:19
*** stevemar has quit IRC19:20
hemnadoesn't use it.19:20
hemnaso....the remotefs connector isn't used afaik19:20
*** stevemar has joined #openstack-cinder19:20
e0nebecause this copy_volume_to_image code in drivers is very similar, sometimes even copy-pasted19:21
hemnasome drivers have to do 'special' stuffs19:21
e0newe have to move it to manager/base driver where it's possible19:21
*** alonma has quit IRC19:21
e0nethe question:19:21
e0nehow many drivers implement copy_volume_to_image using this code: image_utils.upload_volume(context, image_service, image_meta,self.local_path(volume))?19:22
hemnaeharney, so I guess at some point we should modify the nfs/remotefs driver to use os-brick for the local volume copy operations19:22
hemnaand then migrate nova to use os-brick as well19:22
jbernardi believe migration will use that connector19:23
jbernardwhen i submitted the patch, lvm to/from nfs worked19:23
jbernardand nfs to/from ceph also worked19:23
hemnajbernard, ok19:23
hemnaI'm trying to find any way to test the os-brick privsep patch19:23
jbernardsince it calls connect_volume, i believe it would follow that code path19:23
sheelmc_nair: you there19:23
hemnawhich modifies how the remotefs connector calls the local commands19:23
hemnaok, so create an lvm volume, then migrate that to nfs ?19:24
sheelmc_nair: was looking into quota in nova19:24
jbernardhemna: yeah, that worked when i submitted the patch, i swear19:24
hemnaok let me try19:24
sheelmc_nair: there something that is different then what is being used in cinder19:24
hemnajbernard, thanks19:24
jbernardhemna: but it's not tested in the gate, so usual disclaimer applies19:24
mc_nairsheel: yea - actually just hacked something to get the QuotaDriver instantiation further down (will put up a WIP shortly)19:24
jbernardhemna: np19:24
*** alonma has joined #openstack-cinder19:24
*** salv-orlando has quit IRC19:24
*** stevemar has quit IRC19:24
mc_nairsheel: anything particularly interesting with it?  I'll take a look at it now19:24
sheelmc_nair: actually in nova, they are not intializing driver_class specifically19:25
sheelmc_nair: rather at the time of call only19:25
* hemna tries19:26
sheelmc_nair: refer
e0nejbernard: in such case, we need to get disconnect_volume implemented to umount volume19:27
hemnaFailed to schedule_migrate_volume_to_host: No valid host was found. Cannot place volume 4b5befe5-112b-4c87-a984-6209ca79ad9d on trusty-devstack@nfs19:27
hemnapuked in the scheduler19:27
jbernard--migration-policy=ondemand ?19:27
mc_nairsheel: wish I had seen that before - it's what I just ended up doing :) thanks for the help on that.  Good confirmation about that being the way to go19:27
mc_nairsheel: that seems to make sense, will continue on with that.  Thanks again for the help19:28
sheelmc_nair: yeah..its ok... just wanted to help you19:28
sheelmc_nair: always welcome19:28
*** mylu has quit IRC19:29
*** alonma has quit IRC19:29
*** rlrossit has quit IRC19:29
hemna'free_capacity_gb': 0.854492187519:30
*** mylu has joined #openstack-cinder19:31
hemnanormal volume create works19:31
hemnajust can't migrate19:31
sheeleharney: hi there19:31
e0nemigrate do not use thin provisioning19:31
sheeleharney: regarding
jbernarde0ne: sparse is passed to dd if both volumes are block devices19:32
sheeleharney : comments incorporated , also jenkins turned green19:32
sheeleharney: could you find some time to review it..19:33
jbernarde0ne: does disconnect not unmount now?19:33
jbernarde0ne: i haven't tested that in some time19:33
*** alonma has joined #openstack-cinder19:34
*** rlrossit_ has joined #openstack-cinder19:34
hemnamy volume type is basically empty but the volume_backend_name19:34
hemnaI can create19:34
hemnajust not migrate19:34
e0nejbernard: it's in my TODO list to implement it at least for nfs19:35
hemnascheduler says no host found on migrate19:35
*** stevemar has joined #openstack-cinder19:35
jbernardhemna: damn, ill add that to my todo list for investigation19:36
hemnacinder migrate 4b5befe5-112b-4c87-a984-6209ca79ad9d trusty-devstack@nfs#nfs19:36
hemnacinder-volume   |    trusty-devstack@nfs    | nova | enabled |   up19:37
*** mylu has quit IRC19:37
hemnatried not passing the pool name19:37
hemnasame result19:37
*** crose has joined #openstack-cinder19:37
jbernardhemna: oh, you cannot 'migrate' to migrate19:37
jbernardhemna: you must 'retype'19:38
hemnathe nfs free capacity is reporting 0.85G though19:38
hemnabut I can create against it19:38
hemnadoes migrate not use overprovisioning ?19:38
*** alonma has quit IRC19:38
jbernardi dont think migrate will allow the type to change19:38
jbernardso you cannot cross teh backend boundry19:38
jbernardwhich is required to move from lvm to nfs19:39
hemnaretype pukes as well19:39
*** vgridnev has joined #openstack-cinder19:39
*** krotscheck is now known as krotscheck_dcm19:39
jbernardsomething like 'cinder retype --migration-policy=ondemand <volume id> <type>'19:39
jbernard^ that's "supposed" to work19:39
sheelhemna: retype pukes if migration policy is not specified19:40
sheelhemna: may be you can try with migration policy19:40
jbernardpuke might be the wrong word, it refuses the operation because a migration is required and the default is to disallow it19:41
*** alonma has joined #openstack-cinder19:42
jbernardhemna: ah yeah, oops :)19:42
hemnathat seemed to have worked19:42
hemnac-vol errors now :P19:42
* jbernard rejoices19:42
* jbernard cries19:42
jbernardhemna: privsep failing there?19:44
jbernardhemna: is that with or without your local changes?19:44
hemnajbernard, could be, have to track it down....19:44
jbernardhemna: honestly, im extatic it made it that far19:44
hemnajbernard, hehe19:45
jbernardi may try to come up with a proposal for improved migration/retype flow for austin19:45
jbernardit's really getting crazy in there19:45
hemnaI don't understand what --migration-policy is all about19:45
hemnaand why that makes it work19:46
* hemna is confused19:46
jbernardretype without migration just changes metadata (extra specs) to satisfy the request19:46
jbernardif the type or backend are differetn, then migration is required19:46
*** mvk has joined #openstack-cinder19:46
jbernardbut the default is 'no, do not do it'19:46
*** alonma has quit IRC19:46
jbernardso you must say 'on-demand' to allow it to proceed19:47
hemnaI see, so that just forces the retype -> migration19:47
hemnaand yet cinder migrate doesn't work19:47
jbernardbut.. it's not the same flow as proper 'cinder migrate'19:47
jbernardwhich just moves a volume between hosts of the same backend19:47
smcginnisThat would be too logical. :)19:47
hemnacan we please shoot ourselves more.19:48
*** alonma has joined #openstack-cinder19:50
*** alonma has quit IRC19:55
*** mylu has joined #openstack-cinder19:56
*** jwcroppe has quit IRC19:58
*** alonma has joined #openstack-cinder19:58
*** jwcroppe has joined #openstack-cinder19:59
hemnaarg ok20:00
hemnawell, progress20:00
*** haomaiwang has quit IRC20:01
*** haomaiwang has joined #openstack-cinder20:01
*** Lee1092 has quit IRC20:02
*** vgridnev has quit IRC20:03
*** alonma has quit IRC20:03
eharneyhemna: IIRC that execute argument only really exists for unit testing20:03
eharneyhemna: so there may be a way to remove it20:04
hemnaI'm hacking the patch just to get past it20:05
hemnaat least locally20:05
*** sheel has quit IRC20:05
openstackgerritAlex Meade proposed openstack/cinder-specs: User facing error messages
*** xyang has quit IRC20:14
*** salv-orlando has joined #openstack-cinder20:24
*** stevemar has quit IRC20:26
*** stevemar has joined #openstack-cinder20:26
*** lpetrut has quit IRC20:32
*** apoorvad has quit IRC20:33
*** mylu has quit IRC20:37
*** david-lyle has joined #openstack-cinder20:39
*** mylu has joined #openstack-cinder20:40
*** mylu has quit IRC20:53
*** akerr_ has quit IRC20:55
*** david-lyle has quit IRC20:55
*** david-lyle_ has joined #openstack-cinder20:55
*** porrua has quit IRC20:56
*** apoorvad has joined #openstack-cinder20:56
*** mylu has joined #openstack-cinder20:57
*** alonma has joined #openstack-cinder20:59
*** david-lyle_ is now known as david-lyle21:00
*** tongli has quit IRC21:00
*** haomaiwang has quit IRC21:01
*** raildo is now known as raildo-afk21:01
*** mylu has quit IRC21:01
*** haomaiwang has joined #openstack-cinder21:01
*** alonma has quit IRC21:03
*** alonma has joined #openstack-cinder21:06
*** mylu has joined #openstack-cinder21:08
guitarzanhmm, does anyone know who oslo_config actually works? :)21:09
guitarzanwhen does it actually parse the config file?21:09
dimsum__guitarzan : when CONF() is called21:09
guitarzanthis for example, doesn't seem to work anymore:21:10
guitarzanCONF = config.CONF21:10
guitarzanAPI = importutils.import_class(CONF.volume_api_class)21:10
guitarzanit always returns the default value for volume_api_class now21:10
*** mylu has quit IRC21:10
*** mylu has joined #openstack-cinder21:10
*** alonma has quit IRC21:10
guitarzanit doesn't complain about the option, it just doesn't seem to read the value from cinder.conf21:11
guitarzanlater, when printing out the config options, it has the correct value21:11
*** lprice has joined #openstack-cinder21:12
dimsum__guitarzan : search for "CONF(" if you print before it's called you will get the default21:12
guitarzandimsum__: yep, grepping all of cinder for that now, thanks a lot21:12
*** alonma has joined #openstack-cinder21:12
guitarzanstriking out...21:13
guitarzanso everything imported before that main is going to not have the correct config options?21:14
dimsum__guitarzan :
guitarzanbut all of these top level import uses of CONF are just not going to work?21:15
dimsum__guitarzan : when CONF() is called it gets the command line args and loads the config files21:15
guitarzandimsum__: yeah, found it21:16
hemnaI tried to use oslo.config in my standalone scripts21:16
hemna= PITA21:16
hemnathat's what I ended up doing21:16
guitarzanhemna: ah, calling CONF at top level :)21:16
*** alonma has quit IRC21:16
hemnathat's the only way I could get it to not puke21:17
guitarzandimsum__: so is that pattern of using at top level just a broken idea?21:17
dimsum__hemna : simple example -
dimsum__let me look at yours21:17
hemnadimsum__, basically the same thing21:18
guitarzandimsum__: so this just doesn't work?
dimsum__hemna : you could add the CLI stuff to CONF as well21:18
dimsum__guitarzan : depends on when that import kicks in21:19
guitarzanI'm going to have to check and see when this broke21:19
dimsum__what's the actual error you are seeing? is it in CI?21:20
guitarzandimsum__: not an error, that option just doesn't work anymore21:20
dimsum__hemna : we are always on oslo channel :)21:20
guitarzanno one tests the alternate volume api class :)21:20
dimsum__guitarzan : ah21:20
*** e0ne has quit IRC21:20
mc_nairguitarzan: yea :/  I just ran into a similar issue with this today with the config not working for quota_driver either -
hemnadimsum__, thanks :)21:21
guitarzanmc_nair: that one should work though21:21
guitarzanmc_nair: that doesn't execute on import21:21
guitarzanoh, maybe it does21:21
mc_nairguitarzan: well we instantiate the class immediately -
guitarzanQUOTAS = VolumeTypeQuotaEngine()21:21
guitarzanCGQUOTAS = CGQuotaEngine()21:21
guitarzan at th ebottom21:21
mc_nairguitarzan: so with the help of sheel, we figured out easiest way to fix it was to do something similar to what nova does -
guitarzanI'm curious when this behavior changed21:22
mc_nairguitarzan: though not sure if that will help you for the API case or not21:22
guitarzanah, just load it on demand21:22
*** timcl has quit IRC21:22
*** boris-42 has joined #openstack-cinder21:23
mc_nairyea - then it's good to go.  Was losing my mind earlier today trying to figure out why it wouldn't pick up my config change for the driver :)21:23
guitarzanmc_nair: yeah, this is pretty maddening ;)21:24
*** alonma has joined #openstack-cinder21:24
mc_nairguitarzan: indeed :) and I guess the API one is tougher to do an on-demand load for...21:26
*** stevemar has quit IRC21:27
guitarzanyou are probably right :)21:27
*** stevemar has joined #openstack-cinder21:28
*** alonma has quit IRC21:29
*** alonma has joined #openstack-cinder21:31
*** crose has quit IRC21:31
*** stevemar_ has joined #openstack-cinder21:32
*** stevemar has quit IRC21:33
*** alonma has quit IRC21:35
*** alonma has joined #openstack-cinder21:40
*** mragupat has quit IRC21:40
*** mragupat has joined #openstack-cinder21:41
*** alonma has quit IRC21:45
mc_nairguitarzan: could we do something like this?
*** baumann has quit IRC21:46
mc_nairguitarzan: it at least failed now when I put in a made up API class :)21:46
*** xyang has joined #openstack-cinder21:47
*** jgregor has quit IRC21:47
openstackgerritRyan Rossiter proposed openstack/cinder: Add test for checking object compatibilities
hemnacan we push through a few of these specs that are just sitting there.   The code has already landed.  and
guitarzanmc_nair: that does kind of work21:49
guitarzanmc_nair: it needs to be singletonized somehow I think21:49
guitarzanso ugly though :)21:49
*** xyang1 has quit IRC21:50
mc_nairguitarzan: was it singletonized before?  I can't tell if it was just used to map to the API class that you want to instantiate21:50
mc_nairguitarzan: but indeed - I uglified it21:50
guitarzanmc_nair: that's a good point21:50
mc_nairFWIW... I really like the word "singletonized"21:51
guitarzanyou can english anything21:51
Swansonverbing weirds words.21:52
guitarzanmc_nair: I don't even now for sure if returning a class instance is what that's really supposed to do21:52
guitarzanmc_nair: ya, it's not supposed to return the class21:53
guitarzaner, the instance21:53
openstackgerritMerged openstack/cinder-specs: Add extend_volume API to connectors
openstackgerritMerged openstack/cinder-specs: Brick add new Connector volume APIs
mc_nairguitarzan: that's where I was looking where I think that change would still keep the same behavior.  But I haven't tracked down other places we may use it21:54
hemnathank you21:54
*** tpsilva has quit IRC21:56
*** diablo_rojo has quit IRC21:58
*** adrianofr has quit IRC22:00
*** haomaiwang has quit IRC22:01
*** sgotliv has joined #openstack-cinder22:01
*** haomaiwa_ has joined #openstack-cinder22:01
*** dims has joined #openstack-cinder22:04
*** dimsum__ has quit IRC22:04
openstackgerritScott DAngelo proposed openstack/cinder: cinder-api-microversions code
*** jungleboyj has quit IRC22:10
guitarzanmc_nair: it needs to be trickier, because you can't change the thing you're creating in init :)22:10
scottdaHow much to folks want api-microversions in Mitaka? Badly enough to review 3500 lines of code?22:10
guitarzanmc_nair: some __new__ shennanigans are in order I think22:10
*** gouthamr has quit IRC22:11
mc_nairguitarzan: sounds like quite a party22:13
patrickeastscottda: i'll take a stab at it this weekend22:13
patrickeastscottda: have been meaning to pull it down and play around with it... day job has really gotten in the way :(22:13
scottdapatrickeast: Thanks. Adding a new /v3 endpoint means copying a bunch of /v2 stuff and /v2 unit tests, so it's not as bad as it sounds22:14
patrickeastscottda: sweet, that makes things easier22:14
scottdapatrickeast: I need to update this, but it has some ideas for testing:
scottdaAnd I need to fix the cinderclient for the /v3 changes....22:15
mc_nairxyang: oh synchronized reviews :)22:15
xyangmc_nair: you clicked the publish button quicker:)22:16
*** dustins has quit IRC22:16
mc_nairxyang: haha - yea, it's like a draw22:17
mc_nairxyang: btw - regarding, did you want me to push up something to do the validation of the max_oversubscription_config?22:18
mc_nairprobably won't get much review love from others til we get it into a non -1 state.22:19
*** knikolla has quit IRC22:19
xyangmc_nair: as long as you work with the original author.  He marked it WIP.  I don't want a first time contributor's work wasted:)22:19
mc_nairxyang: for sure :) just wanted to make sure you hadn't started on that already.  Will work with him on Monday on that22:21
*** geguileo has joined #openstack-cinder22:22
xyangmc_nair: sure22:23
guitarzanmc_nair: using __new__ actually kind of works22:24
*** lascii is now known as alaski22:24
*** stevemar_ has quit IRC22:24
*** stevemar has joined #openstack-cinder22:25
mc_nairguitarzan: haha - kind of works is what we shoot for right? Could you share the code snippet on that? My interest is piqued22:26
guitarzanmc_nair: it's just the same as yours, only overriding __new__ instead22:28
mc_nairguitarzan: ack - I get what you're saying.  Yea that makes sense22:31
*** lprice has quit IRC22:31
guitarzandoes cells still override the compute api class?22:31
* guitarzan looks22:31
guitarzanit's kind of a first class citizen, so they might not do that anymore22:32
guitarzanah, API is a method in compute now22:33
guitarzanmc_nair: this works too:
guitarzanstolen from nova22:35
mc_nairguitarzan: oh nice22:36
mc_nairguitarzan: but that's got no character.... to clean :)22:36
guitarzanhaha, indeed22:37
*** salv-orl_ has joined #openstack-cinder22:40
*** alonma has joined #openstack-cinder22:41
*** salv-orlando has quit IRC22:43
*** cbader has quit IRC22:43
*** alonma has quit IRC22:45
*** alonma has joined #openstack-cinder22:49
*** alonma has quit IRC22:53
*** garthb_ has joined #openstack-cinder22:53
*** garthb has quit IRC22:53
*** geguileo has quit IRC22:53
*** mylu has quit IRC22:54
*** erlon has quit IRC22:56
*** cknight has quit IRC22:56
*** cknight has joined #openstack-cinder22:56
*** geguileo has joined #openstack-cinder22:58
*** xyang has quit IRC22:58
*** haomaiwa_ has quit IRC23:01
*** haomaiwang has joined #openstack-cinder23:01
*** laughterwym has joined #openstack-cinder23:01
*** mylu has joined #openstack-cinder23:02
*** laughterwym has quit IRC23:06
*** dims_ has joined #openstack-cinder23:06
*** rlrossit_ has quit IRC23:06
*** mragupat has quit IRC23:08
*** dims has quit IRC23:10
*** edmondsw has quit IRC23:13
*** IlyaG has joined #openstack-cinder23:16
*** edtubill has quit IRC23:26
*** IlyaG has quit IRC23:28
*** mylu has quit IRC23:31
*** mylu has joined #openstack-cinder23:31
*** mylu has quit IRC23:32
*** IlyaG has joined #openstack-cinder23:35
*** mgarza has quit IRC23:37
*** sgotliv has quit IRC23:37
*** chlong has joined #openstack-cinder23:38
*** IlyaG has quit IRC23:46
*** cknight has quit IRC23:46
*** geguileo has quit IRC23:49
*** mvk has quit IRC23:50
*** arch-nemesis has quit IRC23:52
*** mylu has joined #openstack-cinder23:54
*** mylu has quit IRC23:58
*** bardia has quit IRC23:58

Generated by 2.14.0 by Marius Gedminas - find it at!