Tuesday, 2016-05-17

*** leeantho has quit IRC00:02
*** yhayashi has joined #openstack-cinder00:03
*** yangyapeng has quit IRC00:04
*** suyash has quit IRC00:05
*** IlyaG has joined #openstack-cinder00:15
*** laughterwym has joined #openstack-cinder00:18
*** laughterwym has joined #openstack-cinder00:19
*** IlyaG has quit IRC00:23
*** xingyang has joined #openstack-cinder00:24
*** xingyang has quit IRC00:24
*** sdake_ has joined #openstack-cinder00:27
*** mtanino_ has joined #openstack-cinder00:27
*** mtanino has quit IRC00:29
*** sdake has quit IRC00:30
*** bill_az has quit IRC00:33
*** bardia has quit IRC00:36
*** jamielennox|away is now known as jamielennox00:37
*** diogogmt has joined #openstack-cinder00:42
*** ducttape_ has joined #openstack-cinder00:43
*** ircuser-1 has quit IRC00:46
*** mriedem has joined #openstack-cinder00:47
*** IlyaG has joined #openstack-cinder00:54
*** EinstCrazy has joined #openstack-cinder00:54
*** cheneydc has joined #openstack-cinder00:55
*** alkhodos has quit IRC01:05
*** raunak has quit IRC01:13
*** akerr has quit IRC01:13
*** yangyapeng has joined #openstack-cinder01:14
*** jwcroppe has joined #openstack-cinder01:15
*** yangyape_ has joined #openstack-cinder01:16
*** yangyape_ has quit IRC01:16
*** yangyapeng has quit IRC01:16
*** yangyapeng has joined #openstack-cinder01:17
*** yangyapeng has quit IRC01:17
*** yangyapeng has joined #openstack-cinder01:17
*** Apoorva_ has joined #openstack-cinder01:27
*** krtaylor has joined #openstack-cinder01:28
*** Lee1092 has joined #openstack-cinder01:28
*** krtaylor has quit IRC01:30
*** Apoorva has quit IRC01:31
*** Apoorva_ has quit IRC01:32
*** jamielennox is now known as jamielennox|away01:59
*** mtanino has joined #openstack-cinder02:00
*** Julien-zte has joined #openstack-cinder02:01
*** mtanino_ has quit IRC02:02
*** mtanino has quit IRC02:06
*** ducttape_ has quit IRC02:08
*** houming has joined #openstack-cinder02:11
*** raunak has joined #openstack-cinder02:23
*** iceyao has joined #openstack-cinder02:24
*** mriedem has quit IRC02:28
*** iceyao_ has joined #openstack-cinder02:29
*** Julien-zte has quit IRC02:29
smcginnisjgriffith: Glad the conversation got around to stats reporting.02:31
smcginnisjgriffith: That's where I was going with it.02:31
smcginnisjgriffith: config option causes problems for multibackend.02:31
smcginnisjgriffith: So I think we need the driver to report something like "allocation_unit_size: 8" or something to report its size in increments.02:32
*** iceyao has quit IRC02:32
smcginnisOr maybe that and minimum_allocation_size to report the smallest size it supports, which may or may not be the increments from there on.02:32
jgriffithsmcginnis: yeah, that seems like a reasonable way to go to me, the only real sane answer anyway02:36
*** enriquetaso has quit IRC02:42
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/31634302:55
*** duobei has joined #openstack-cinder02:59
*** itlinux has quit IRC03:09
*** EinstCra_ has joined #openstack-cinder03:10
*** EinstCrazy has quit IRC03:14
*** alekseyruban has quit IRC03:18
*** wxy has joined #openstack-cinder03:21
*** ducttape_ has joined #openstack-cinder03:24
*** raunak has quit IRC03:27
*** links has joined #openstack-cinder03:31
*** sdake has joined #openstack-cinder03:36
*** itlinux has joined #openstack-cinder03:38
*** sdake_ has quit IRC03:38
*** itlinux has quit IRC03:39
*** itzdilip has quit IRC03:41
*** gouthamr_ has quit IRC03:46
*** duobei has quit IRC03:52
*** gcb has quit IRC03:53
*** sgotliv_ has joined #openstack-cinder03:54
*** jamielennox|away is now known as jamielennox03:55
*** sgotliv_ has quit IRC03:57
*** sgotliv has joined #openstack-cinder03:57
*** bill_az has joined #openstack-cinder03:58
*** laughterwym has quit IRC03:59
*** ducttape_ has quit IRC04:00
*** laughterwym has joined #openstack-cinder04:05
*** gcb has joined #openstack-cinder04:06
*** sheel has joined #openstack-cinder04:06
*** duobei has joined #openstack-cinder04:08
*** salv-orlando has joined #openstack-cinder04:15
*** salv-orl_ has quit IRC04:18
*** sdake_ has joined #openstack-cinder04:20
*** sdake has quit IRC04:20
*** laughter_ has joined #openstack-cinder04:20
*** duobei has quit IRC04:22
openstackgerritvenkatamahesh proposed openstack/python-cinderclient: Update the home-page with developer documentation  https://review.openstack.org/31723204:22
*** laughterwym has quit IRC04:23
*** itzdilip has joined #openstack-cinder04:23
*** delatte has quit IRC04:27
*** harlowja_at_home has joined #openstack-cinder04:29
*** lpetrut has joined #openstack-cinder04:37
*** dave-mccowan has quit IRC04:37
*** IlyaG_ has joined #openstack-cinder04:50
*** IlyaG has quit IRC04:54
*** GB21 has joined #openstack-cinder04:56
*** sdake_ has quit IRC04:56
*** asselin has joined #openstack-cinder05:03
*** GB21 has quit IRC05:04
*** avishay has joined #openstack-cinder05:05
*** asselin__ has quit IRC05:06
*** e0ne has joined #openstack-cinder05:09
*** GB21 has joined #openstack-cinder05:09
*** e0ne has quit IRC05:11
*** shausy has joined #openstack-cinder05:14
*** savihou has joined #openstack-cinder05:19
*** harlowja_at_home has quit IRC05:20
*** lpetrut has quit IRC05:21
*** lpetrut has joined #openstack-cinder05:22
*** laughter_ has quit IRC05:23
*** laughterwym has joined #openstack-cinder05:23
*** nkrinner has joined #openstack-cinder05:26
*** ChubYann has quit IRC05:28
*** GB21 has quit IRC05:31
*** gcb has quit IRC05:36
*** lpetrut has quit IRC05:44
*** iceyao_ has quit IRC05:46
*** iceyao has joined #openstack-cinder05:46
*** duobei has joined #openstack-cinder05:49
*** gcb has joined #openstack-cinder05:50
*** mjura has joined #openstack-cinder05:51
*** iceyao_ has joined #openstack-cinder05:53
*** iceyao has quit IRC05:55
*** savihou has quit IRC05:58
*** mjura has quit IRC05:59
*** mjura has joined #openstack-cinder05:59
*** rcernin has joined #openstack-cinder06:03
*** liverpooler has quit IRC06:04
*** liverpooler has joined #openstack-cinder06:04
*** daneyon has quit IRC06:08
*** savihou has joined #openstack-cinder06:11
*** rcernin has quit IRC06:18
*** savihou has quit IRC06:18
*** rcernin has joined #openstack-cinder06:21
*** savihou has joined #openstack-cinder06:22
*** bill_az has quit IRC06:26
openstackgerritRyan Liang proposed openstack/cinder: VNX: New Cinder driver in Newton  https://review.openstack.org/31242706:28
*** cdelatte has joined #openstack-cinder06:30
*** jamielennox is now known as jamielennox|away06:41
*** ircuser-1 has joined #openstack-cinder06:45
*** belmoreira has joined #openstack-cinder06:53
*** arecknag has joined #openstack-cinder06:54
*** duobei has quit IRC06:57
*** duobei has joined #openstack-cinder07:05
*** ozialien10 has quit IRC07:07
openstackgerritSheel Rana proposed openstack/cinder: migrate to os-api-ref  https://review.openstack.org/31728307:23
*** duobei has quit IRC07:26
*** iceyao_ has quit IRC07:27
*** sayalilunkad has joined #openstack-cinder07:28
*** gcb has quit IRC07:29
*** sayalilunkad has quit IRC07:36
*** sayalilunkad has joined #openstack-cinder07:36
openstackgerritSheel Rana proposed openstack/cinder: Updated volume, version, limit, metadata API-ref  https://review.openstack.org/31658707:39
openstackgerritSheel Rana proposed openstack/cinder: migrate to os-api-ref  https://review.openstack.org/31728307:40
*** gcb has joined #openstack-cinder07:41
DuncanTsmcginnis: I think it is a very sane answer to say 'the cinder API works in 1 gb increments and backends that don't support it are not suitable of use with cinder' - it might not be an answer we eventually go with, but it should certainly be an acknowledged option on the table. Whenever we put weirdness like this in our API it has a cost forever, and some of07:42
DuncanTthose costs are undoubtedly not worth it07:42
*** savihou has quit IRC07:48
*** savihou has joined #openstack-cinder07:48
*** ozialien10 has joined #openstack-cinder07:52
openstackgerritBin Zhou proposed openstack/cinder: add function call chain  https://review.openstack.org/31729507:52
*** iceyao has joined #openstack-cinder07:53
*** cloudpuppy has joined #openstack-cinder07:54
*** ozialien10 has quit IRC07:57
*** ozialien10 has joined #openstack-cinder07:58
*** duobei has joined #openstack-cinder07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:00
*** liverpooler has quit IRC08:01
*** liverpooler has joined #openstack-cinder08:01
*** divab has quit IRC08:06
*** iceyao_ has joined #openstack-cinder08:07
lixiaoy1DuncanT: Could you kindly review https://review.openstack.org/#/c/247577?08:07
*** iceyao has quit IRC08:08
*** savihou has quit IRC08:10
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
*** savihou has joined #openstack-cinder08:10
*** cristicalin has joined #openstack-cinder08:13
*** caspinol has joined #openstack-cinder08:15
*** jistr has joined #openstack-cinder08:19
openstackgerritMatan Sabag proposed openstack/cinder: Manage/unmanage snapshot in ScaleIO driver Add support for manage/unmanage snapshot in the ScaleIO driver.  https://review.openstack.org/31674208:20
openstackgerritMatan Sabag proposed openstack/cinder: Manage/unmanage snapshot in ScaleIO driver  https://review.openstack.org/31674208:21
openstackgerritMatan Sabag proposed openstack/cinder: ScaleIO actual volume size model update  https://review.openstack.org/31681108:26
*** e0ne has joined #openstack-cinder08:30
*** cristicalin has quit IRC08:31
*** cristicalin has joined #openstack-cinder08:31
*** mvk has joined #openstack-cinder08:33
*** laughterwym has quit IRC08:43
*** cristicalin has quit IRC08:44
*** laughterwym has joined #openstack-cinder08:44
*** laughterwym has quit IRC08:46
*** laughterwym has joined #openstack-cinder08:46
openstackgerritLisaLi proposed openstack/cinder-specs: Retype encrypted volumes  https://review.openstack.org/24859308:46
*** savihou has quit IRC08:47
*** savihou has joined #openstack-cinder08:48
*** laughterwym has quit IRC08:51
*** salv-orlando has quit IRC08:54
*** caspinol has quit IRC09:00
*** yhayashi has quit IRC09:01
*** caspinol has joined #openstack-cinder09:08
*** jordanP has joined #openstack-cinder09:10
*** caspinol has quit IRC09:11
*** cloudpuppy has quit IRC09:32
*** cloudpuppy has joined #openstack-cinder09:32
*** cloudpuppy has quit IRC09:33
*** cloudpuppy has joined #openstack-cinder09:33
*** cloudpuppy has quit IRC09:34
*** cloudpuppy has joined #openstack-cinder09:34
*** dave-mccowan has joined #openstack-cinder09:35
*** mvk has quit IRC09:35
*** mvk has joined #openstack-cinder09:37
yuriy_n17dulek: Hi! Very sorry for disturbing. Please review https://review.openstack.org/#/c/315673/ whenever you have free time.09:43
*** dobson has quit IRC09:56
*** cheneydc has quit IRC10:01
*** laughterwym has joined #openstack-cinder10:01
*** duobei has quit IRC10:02
*** duobei has joined #openstack-cinder10:03
*** laughterwym has quit IRC10:06
*** caspinol has joined #openstack-cinder10:07
*** caspinol has quit IRC10:09
openstackgerritSheel Rana proposed openstack/cinder: migrate to os-api-ref  https://review.openstack.org/31728310:12
*** salv-orl_ has joined #openstack-cinder10:13
*** salv-orl_ has quit IRC10:13
*** salv-orl_ has joined #openstack-cinder10:13
*** iceyao_ has quit IRC10:13
*** sdague has joined #openstack-cinder10:18
*** EinstCra_ has quit IRC10:25
*** e0ne has quit IRC10:25
*** dobson has joined #openstack-cinder10:26
*** e0ne has joined #openstack-cinder10:26
*** yangyapeng has quit IRC10:29
*** duobei has quit IRC10:39
*** dobson has quit IRC10:43
*** lpetrut has joined #openstack-cinder10:48
*** jordanP has quit IRC10:49
*** jwcroppe has quit IRC10:51
*** gcb has quit IRC10:51
*** jwcroppe has joined #openstack-cinder10:53
*** jamielennox|away is now known as jamielennox10:53
*** dobson has joined #openstack-cinder10:55
*** jwcroppe_ has joined #openstack-cinder10:56
*** jwcroppe has quit IRC10:59
yuriy_n17geguileo: Hi! Very sorry for disturbing. Please review https://review.openstack.org/#/c/315673/ whenever you have free time.11:16
*** gcb has joined #openstack-cinder11:18
*** jistr has quit IRC11:24
*** ccesario has quit IRC11:32
*** salv-orl_ has quit IRC11:32
*** diogogmt has quit IRC11:35
*** salv-orlando has joined #openstack-cinder11:35
*** diogogmt has joined #openstack-cinder11:37
openstackgerritAvishay Traeger proposed openstack/cinder: List manageable volumes and snapshots  https://review.openstack.org/28529611:41
openstackgerritAvishay Traeger proposed openstack/cinder: Map volume/snapshot manage extensions to v3  https://review.openstack.org/30732411:41
avishayDuncanT: added "DocImpact" ^^11:42
*** caspinol has joined #openstack-cinder11:47
*** dave-mccowan has quit IRC11:47
*** vnogin has quit IRC11:47
*** caspinol has quit IRC11:48
*** diogogmt has quit IRC11:54
*** caspinol has joined #openstack-cinder11:55
*** laughterwym has joined #openstack-cinder11:55
*** laughterwym has quit IRC11:57
*** jistr has joined #openstack-cinder12:00
*** gouthamr has joined #openstack-cinder12:02
*** julim has joined #openstack-cinder12:05
*** caspinol has quit IRC12:05
*** laughterwym has joined #openstack-cinder12:06
*** duobei has joined #openstack-cinder12:07
*** ducttape_ has joined #openstack-cinder12:09
*** houming has quit IRC12:10
*** dave-mccowan has joined #openstack-cinder12:13
openstackgerritzhangguoqing proposed openstack/cinder: Violin: resize volume if cloned lun is larger  https://review.openstack.org/29509412:14
dulekguitarzan: I've just read your talk with geguileo. Basically we've incremented RPC versions and made managers to accept both of them in Mitaka RC1.12:15
*** duobei has quit IRC12:15
dulekguitarzan: a0686c4641f0a22182b3b7ee3d0c7b70d00fd9c4 is the place we've did that.12:15
*** raildo-afk is now known as raildo12:15
dulekguitarzan: Now we're dropping that compat code. Current N's master has scheduler compat code removed.12:16
dulekguitarzan: So to do L->N upgrade you need to go through this commit (actually through something between that and d0bddb19dc508265274a1193c7d5c1d19e35e441 where we're removing first compatibility code), and then to master.12:17
*** JoseMello has joined #openstack-cinder12:22
*** krtaylor has joined #openstack-cinder12:25
*** ducttape_ has quit IRC12:25
*** timcl has joined #openstack-cinder12:33
*** adrianofr has joined #openstack-cinder12:34
openstackgerritTomaz Vieira proposed openstack/cinder: Improve hitachi hnas volume drivers log messages  https://review.openstack.org/31078512:36
openstackgerritMerged openstack/cinder: Manage existing volume with invalid host  https://review.openstack.org/31448212:37
*** ccesario has joined #openstack-cinder12:39
*** gcb has quit IRC12:40
*** EinstCrazy has joined #openstack-cinder12:45
*** sheel has quit IRC12:45
*** edmondsw has joined #openstack-cinder12:48
*** salv-orlando has quit IRC12:49
*** gcb has joined #openstack-cinder12:53
*** salv-orlando has joined #openstack-cinder12:55
openstackgerritCedric Brandily proposed openstack/python-cinderclient: Support --os-key option  https://review.openstack.org/30024313:02
*** xyang1 has joined #openstack-cinder13:02
*** Michael-ZTE has joined #openstack-cinder13:05
*** Michael-ZTE has quit IRC13:06
*** eharney has joined #openstack-cinder13:07
guitarzandulek: it's no problem, I just didn't understand what was going on13:08
*** cknight has joined #openstack-cinder13:09
*** rlrossit has joined #openstack-cinder13:10
openstackgerritAvishay Traeger proposed openstack/cinder: List manageable volumes and snapshots  https://review.openstack.org/28529613:11
openstackgerritAvishay Traeger proposed openstack/cinder: Map volume/snapshot manage extensions to v3  https://review.openstack.org/30732413:11
avishayDuncanT: and a rebase... ^^13:11
*** tpsilva has joined #openstack-cinder13:12
*** yangyapeng has joined #openstack-cinder13:12
*** mriedem has joined #openstack-cinder13:14
*** ducttape_ has joined #openstack-cinder13:15
openstackgerritMerged openstack/cinder: Use example.com in unit tests  https://review.openstack.org/31513113:15
openstackgerritMerged openstack/cinder: Whitelist 'rm' in api-ref tox environment  https://review.openstack.org/31636313:16
*** akerr has joined #openstack-cinder13:17
*** jordanP has joined #openstack-cinder13:18
*** ducttape_ has quit IRC13:23
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/31634313:26
*** diogogmt has joined #openstack-cinder13:26
*** avishay has quit IRC13:26
openstackgerritWilson Liu proposed openstack/cinder-specs: Migrate volume between backends in an async way  https://review.openstack.org/31285313:27
openstackgerritScott DAngelo proposed openstack/cinder: Make api_vicroversion_dev more explicit  https://review.openstack.org/31746013:27
openstackgerritScott DAngelo proposed openstack/cinder: Make api_microversion_dev more explicit  https://review.openstack.org/31746013:28
*** porrua has joined #openstack-cinder13:29
*** bill_az has joined #openstack-cinder13:29
*** openstackgerrit has quit IRC13:32
*** diogogmt has quit IRC13:33
*** openstackgerrit has joined #openstack-cinder13:34
openstackgerritWilson Liu proposed openstack/cinder-specs: Migrate volume between backends in an async way  https://review.openstack.org/31285313:35
*** links has quit IRC13:36
*** cknight1 has joined #openstack-cinder13:38
*** cknight has quit IRC13:40
*** shausy has quit IRC13:45
*** crose has joined #openstack-cinder13:45
*** takedakn has joined #openstack-cinder13:45
*** dustins has joined #openstack-cinder13:49
openstackgerritSean McGinnis proposed openstack/cinder: Add sample config file to cinder docs  https://review.openstack.org/31422013:52
*** mjura has quit IRC13:53
*** jgregor has joined #openstack-cinder13:54
*** crose has quit IRC13:54
*** ametts has joined #openstack-cinder13:58
*** iceyao has joined #openstack-cinder13:59
*** mpjetta has quit IRC14:01
*** ducttape_ has joined #openstack-cinder14:02
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/31634314:02
*** jordanP has quit IRC14:03
*** med_ has joined #openstack-cinder14:04
*** med_ has quit IRC14:04
*** med_ has joined #openstack-cinder14:04
*** Julien-zte has joined #openstack-cinder14:05
*** caspinol has joined #openstack-cinder14:05
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements  https://review.openstack.org/31749314:08
*** avishay has joined #openstack-cinder14:09
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/31718614:09
*** kfarr has joined #openstack-cinder14:14
*** fnordahl has quit IRC14:17
*** fnordahl has joined #openstack-cinder14:17
kfarrHi all, if I'd like to add a spec to the meeting agenda for the 18th, do I just add it to the wiki, or do I need to get it informally approved here first?14:17
smcginniskfarr: You can just add it to the agenda. No permission required. :)14:17
kfarrThanks smcginnis!14:18
smcginniskfarr: No problem, thanks for checking.14:18
*** jordanP has joined #openstack-cinder14:24
ildikovsmcginnis: hi14:28
ildikovsmcginnis: I had an epic fail with the Cinder-Nova meeting schedule you might already saw on the mailing list14:29
ildikovsmcginnis: I suggested this channel as a fall back only for today14:29
*** baumann has joined #openstack-cinder14:29
ildikovsmcginnis: do you have any concern regarding this idea?14:30
*** mtanino has joined #openstack-cinder14:30
smcginnisildikov: Sure, we can try that.14:34
smcginnisildikov: I think it should work out OK.14:35
*** markvoelker has joined #openstack-cinder14:35
*** markvoelker has quit IRC14:35
ildikovsmcginnis: cool, thanks for confirmation14:35
*** diablo_rojo has joined #openstack-cinder14:36
*** markvoelker has joined #openstack-cinder14:36
*** karthikp has joined #openstack-cinder14:37
smcginnisildikov: No problem. Thanks for pulling it together. Meeting timing can be tricky.14:38
*** markvoelker has quit IRC14:39
*** markvoelker has joined #openstack-cinder14:39
scottdaildikov: What's the official time of the meeting? 1700 UTC or 1800 UTC?14:42
ildikovscottda: 1700UTC14:42
ildikovscottda: I put up the question whether it should be 1800UTC on Tuesdays or 1700UTC on Mondays14:43
scottdaildikov: OK, thanks. I believe the latest email in the thread had a typo (which was 1800)...14:43
*** Julien-zte has quit IRC14:45
*** Julien-zte has joined #openstack-cinder14:45
openstackgerritAlex Meade proposed openstack/python-cinderclient: Add pagination support for Messages  https://review.openstack.org/31576914:45
ildikovscottda: I sent out a follow up mail to the smaller group to find another slot that works for the meeting channels as well14:46
ildikovscottda: I wasn't sure how much Monday 1700UTC would be popular in the US, so I wanted ask around before have that option as a fall back from next week14:46
scottdaildikov: OK. Thanks for all the work arranging this14:47
ildikovscottda: np, I'm sorry I annoy you guys so much with this :(14:48
*** markvoelker has quit IRC14:51
*** lprice has joined #openstack-cinder14:51
openstackgerritSheel Rana proposed openstack/cinder: migrate to os-api-ref  https://review.openstack.org/31728314:52
*** markvoelker has joined #openstack-cinder14:53
*** xinli has joined #openstack-cinder14:55
*** dramakri has quit IRC14:55
openstackgerritSheel Rana proposed openstack/cinder: migrate to os-api-ref  https://review.openstack.org/31728314:56
*** diogogmt has joined #openstack-cinder14:56
*** zhonghua-lee has quit IRC14:56
smcginnisildikov: Just to make sure I have things straight - we're talking meeting today in 1 hour?14:57
*** dramakri has joined #openstack-cinder14:57
ildikovsmcginnis: 1700UTC is two hours from now14:57
*** markvoelker has quit IRC14:57
*** iceyao has quit IRC14:57
*** markvoelker has joined #openstack-cinder14:58
ildikovsmcginnis: although it must be a magical time as many people asked me last week as well when we start exactly :)14:58
*** IlyaG_ has quit IRC14:58
*** takedakn has quit IRC14:59
*** salv-orlando has quit IRC14:59
smcginnisildikov: Thanks! :)14:59
*** salv-orlando has joined #openstack-cinder15:00
*** salv-orlando has quit IRC15:00
*** salv-orlando has joined #openstack-cinder15:00
*** salv-orlando has quit IRC15:01
*** salv-orlando has joined #openstack-cinder15:02
openstackgerritAlex Meade proposed openstack/python-cinderclient: Add v3 user messages  https://review.openstack.org/29905215:03
openstackgerritAlex Meade proposed openstack/python-cinderclient: Add pagination support for Messages  https://review.openstack.org/31576915:03
*** Julien-zte has quit IRC15:04
ameadescottda: ^^ thanks15:04
*** laughterwym has quit IRC15:04
*** markvoelker has quit IRC15:04
*** laughterwym has joined #openstack-cinder15:04
scottdaameade: I don't see the changes in PS#4. Is there another patch coming?15:07
ameadescottda: i made the exact changes in the prior patch so it just shows as a rebase15:07
*** sheel has joined #openstack-cinder15:09
ameadescottda: in the patch it depends on i mean15:09
sheelHello All, could someone please review on https://review.openstack.org/#/c/316587/15:10
ameadesheel: I'll check that out soon, even though i dont have +2 powers15:10
sheelameade:   +1 attracts +2...15:11
sheelameade:  so it will help for sure... ;015:11
ameadesheel: oh I was going to -1 it15:11
ameadesheel: just kidding :P15:11
sheelameade:  haha....15:11
*** yangyapeng has quit IRC15:12
buhmandumb question: I thought I read in the austin summit notes that cinderclient was deprecated--is actively adding new features/etc.. still allowed?15:14
sheelbuhman:  no its still not  deprecated... AFAIK its ok to add features in V2...15:16
SwansonI thought they were de-deprecating it.15:16
sheelSwanson:  :)15:17
sheelbuhman: you can go ahead..its still alive for this release at least...15:18
buhmanso, what I've really wanted to do is refactor *client's extension architecture to be more flexible; when I asked about making a generic oslo.apiclientextensions or similar in #openstack-oslo, I was discouraged15:18
scottdaameade: I'm still confused... Your first patch looks like it's picked up the not-yet-merged change: https://review.openstack.org/#/c/299052/12/cinderclient/base.py15:19
scottdaameade: And your second patch doesn't have a Depends-On for first patch (It has Depends-On for server-side code).15:19
sheelbuhman:  I think that is fair enough... may be smcginnis  can through some more light on it...15:20
ameadescottda: whats the net yet merged change ur referring to?15:25
ameadei made those changes so that the api_versions.wraps decorator works properly15:26
ameadethe second patch's parent is the first patch15:26
scottdaameade: https://review.openstack.org/#/c/309283/15:26
*** nkrinner is now known as nkrinner_afk15:26
ameadehaha awesome15:27
ameadei'll just rebase onto that15:27
ameadei didnt even know about that15:27
scottdaameade: That explains everything! Maybe just put a Depends-On for that 303283?15:27
*** belmoreira has quit IRC15:28
openstackgerritAlex Meade proposed openstack/python-cinderclient: Add v3 user messages  https://review.openstack.org/29905215:29
openstackgerritAlex Meade proposed openstack/python-cinderclient: Add pagination support for Messages  https://review.openstack.org/31576915:29
*** caspinol has quit IRC15:31
*** leeantho has joined #openstack-cinder15:32
*** sticker has quit IRC15:32
*** alkhodos has joined #openstack-cinder15:32
mriedemildikov: so we're doing the nova/cinder meeting at 1800 UTC in this channel today?15:33
*** lprice1 has joined #openstack-cinder15:34
*** lprice has quit IRC15:34
*** enriquetaso has joined #openstack-cinder15:35
ildikovmriedem: the plan for today is/was to keep the 1700UTC that I announced on the ML, but have it here as a fall back15:36
savihouGents, apologies for nudging (again.. :-( )  - will you guys be able to review https://review.openstack.org/#/c/263026/ ?15:36
mriedemah ok15:36
mriedemworks for me15:36
ildikovmriedem: and in the meantime find another slot that would work for the small group and the IRC channels as well15:36
ildikovmriedem: that's why I sent out the follow up mail15:36
ildikovmriedem: sorry for the confusion15:37
*** jistr has quit IRC15:38
*** akshai has joined #openstack-cinder15:38
*** mvk has quit IRC15:39
openstackgerritXinli Guan proposed openstack/cinder: IBM FlashSystem: Cleanup host resource leaking  https://review.openstack.org/30745115:45
*** daneyon has joined #openstack-cinder15:48
*** asselin__ has joined #openstack-cinder15:52
*** caspinol has joined #openstack-cinder15:53
*** earlephilhower has joined #openstack-cinder15:53
*** kfarr has quit IRC15:53
*** daneyon_ has joined #openstack-cinder15:54
*** daneyon has quit IRC15:55
*** daneyon has joined #openstack-cinder15:55
*** daneyon_ has quit IRC15:55
*** asselin has quit IRC15:56
*** lprice1 has quit IRC16:00
*** savihou has quit IRC16:02
*** lprice has joined #openstack-cinder16:05
smcginnisbuhman, sheel: Sorry, had stepped away. Yes, new functionality can and should still be added to python-cinderclient.16:07
smcginnisbuhman, sheel: Long term goal is to have osc support all the same functionality. Once it does, then we can visit the idea of deprecating cinderclient cli.16:07
*** sdague has quit IRC16:07
buhmansmcginnis: hmm, so how does adding new functionality to cinderclient get osc closer to having feature parity?16:10
smcginnisbuhman: It doesn't. It actually makes it harder.16:11
smcginnisbuhman: But we need to continue to support end users with CLI access until we can switch.16:11
smcginnisbuhman: So there is a lot of work making sure whatever we add to cinderclient also gets added to osc.16:11
*** laughterwym has quit IRC16:15
*** laughterwym has joined #openstack-cinder16:16
ameadeif i'm not mistaken, horizon uses cinderclient16:17
david-lyleameade: yes, but not via CLI16:17
ameadedavid-lyle: yeah, but the CLI layer in cinderclient is near trivial16:18
david-lyleameade: sure, just deprecating CLI is not problematic for Horizon in and of itself16:19
ameadeoh we're only deprecating the CLI part? does OSC use cinderclient libraries?16:20
smcginnisameade: Yes.16:22
smcginnisameade: The deprecation talk is only for the CLI part.16:22
*** caspinol has quit IRC16:22
smcginnisameade: The client library part would remain and I believe is what is ultimately used by osc and others.16:22
ameadeso we have to add a feature to cinderclient in order to get it in osc anyways, k cool i'm caught up16:23
*** jordanP has quit IRC16:24
*** laughterwym has quit IRC16:24
smcginnisameade: Yep, pretty much. :)16:26
*** Apoorva has joined #openstack-cinder16:27
*** daneyon has quit IRC16:28
*** e0ne has quit IRC16:28
sheelsmcginnis:  thanks for giving thoughts...16:28
*** daneyon has joined #openstack-cinder16:29
*** yuriy_n17 has quit IRC16:31
*** xinli has quit IRC16:38
*** diablo_rojo has quit IRC16:38
*** raunak has joined #openstack-cinder16:40
*** diablo_rojo has joined #openstack-cinder16:41
jgriffithpatrickeast: ping16:41
*** kfarr has joined #openstack-cinder16:45
patrickeastjgriffith: pong16:49
jgriffithpatrickeast: not sure if you've seen my drama lately :)16:50
openstackLaunchpad bug 1582806 in Cinder "virtual_size property in image no longer usable for caching on SF driver" [High,Triaged] - Assigned to John Griffith (john-griffith)16:50
jgriffithpatrickeast: so the virtual_size property is R/O16:50
patrickeastUh oh,  no I've missed it (have been out the last week)16:51
jgriffithpatrickeast: in other words, users/admins can't set that any longer (I dunno wtf it's for then)16:51
jgriffithSo my local caching broke after a customer upgraded... and I think the general mechanism is now f'd too16:51
jgriffithI'm working a patch right now and was going to just change the name to "min_volsz" as that's a custom property16:51
patrickeastI'll have to double check, but I think we fixed that part of the generic cache, it doesn't look for that property from glance16:51
jgriffithpatrickeast: oh... even better16:52
jgriffithif that's the case then life is ok16:52
jgriffithhmm... it grabs it in taskflow create16:52
patrickeastWe download the image first and look at the IMG info output then decide on the size16:52
openstackgerritSheel Rana proposed openstack/cinder: Add pagination support to messages  https://review.openstack.org/31311316:52
patrickeastYep yep16:53
patrickeastThat's the one16:53
*** daneyon has quit IRC16:57
*** bill_az has quit IRC16:58
*** bill_az has joined #openstack-cinder16:59
*** daneyon has joined #openstack-cinder16:59
ildikovping scottda ildikov DuncanT ameade cFouts johnthetubaguy e0ne jgriffith andrearosa hemna mriedem smcginnis17:00
scottdaildikov: Hi17:00
ildikovso as suggested on the ML earlier I would like to hijack the Cinder channel for a quick sync17:01
* ameade lurking17:01
mriedemoh it's here17:01
ildikovjgriffith: around?17:02
jgriffithildikov: hi there17:02
ildikovjgriffith: I hoped we can start with your plans on initialize_connection17:02
ildikovjgriffith: any progress with that item?17:03
jgriffithildikov: sure.17:03
jgriffithildikov: yes.. that was pretty easy with the exception of our crazy unit tests :(17:03
jgriffithI plan to get that fixed up17:03
jgriffithOne thing that is troubling for me though....17:03
jgriffithon volume.manager.attach we've gotten ourselves in a pretty bad state17:04
jgriffithwe have a number of arguments that are "optional" which is kinda bad IMO17:04
*** bill_az has quit IRC17:04
jgriffithand we don't really document in the docstrings what and why17:04
jgriffithwe don't even do something like set a default=None to signify things17:05
jgriffithI'd like to propose we fix that now while we're at this17:05
jgriffithIn other words, I'd like to add a follow up patch to what I'm working on that just eliminates the whole "optional" arguments thing17:05
*** jgregor has quit IRC17:05
jgriffithThat would mean a dependent patch to Nova.  So the alternative here....17:06
jgriffithRewrite the flow using "new names" and then we don't have version problems, compatability issues etc17:06
jgriffithdeprecate the old calls17:06
scottdaIf the changes are significant enough, I say re-write with new names.17:07
mriedemnova is only calling the api17:07
hemnaos-attach ?17:07
mriedemthe cinder api is eventually calling the volume manager17:07
*** diablo_rojo has quit IRC17:08
mriedemso any changes there are rpc changes in cinder17:08
mriedemnot exposed to nova17:08
jgriffithmriedem: I can hide them :)17:08
jgriffithmriedem: the only thing is I'd like to submit a change to Nova for the current release that just sets standard args for attach17:08
jgriffithmriedem: no more "maybe I'll get this, maybe I'll get that"17:09
johnthetubaguyso there is stuff we should be sending?17:09
johnthetubaguydo you have an example?17:09
*** baumann has quit IRC17:09
hemnajgriffith, that stuffs ?17:09
jgriffithjohnthetubaguy: yeah, that would've been useful eh :)17:09
*** baumann has joined #openstack-cinder17:09
jgriffithhemna: yep17:09
johnthetubaguyI am wondering also if sending that stuff to old versions would be OK? I guess its fine as it was optional before?17:09
jgriffithjohnthetubaguy: yes17:09
johnthetubaguyso thats doesn't seem too bad17:10
hemnaI think the older versions would just ignore em17:10
jgriffithjohnthetubaguy: so my plan was to put the logic in the new calls to grab the old call if need be17:10
jgriffithall handled on Cinder's side17:10
johnthetubaguyso old nova talking to new cinder needs care17:10
johnthetubaguybut its an edge case17:10
jgriffithjohnthetubaguy: yes, but I would address that case on the Cinder side17:10
jgriffithso my change would be compatible either way17:11
jgriffithjohnthetubaguy: if it's an old nova then it just uses the existing calls (that I'll deprecate)17:11
*** earlephilhower has quit IRC17:11
jgriffithjohnthetubaguy: if it's the new format and I just route it to the appropriate method in the manager17:11
johnthetubaguysounds like its (1) get nova to send stuff it should be sending (2) tidy cinder code so its clear: a) what is optional, b) what is optional because we did silly things in the past, c) whats always been required17:11
*** vasanthi has quit IRC17:12
jgriffithkind of the other way around actually.... always goes to the new, and it decides what to do17:12
johnthetubaguyhmm, so new nova, old cinder? you eventually drop that API?17:12
jgriffithjohnthetubaguy: you got it17:12
jgriffithjohnthetubaguy: after another release at least17:12
jgriffithjohnthetubaguy: cuz if you're trying to do n-2 you're hosed anyway most likely17:12
johnthetubaguyand new Nova, old cinder? has to know about both cases?17:12
jgriffithjohnthetubaguy: yup, that one is covered as well17:13
*** marcin12345 has quit IRC17:13
johnthetubaguyhow is it covered? I am missing something I think17:13
jgriffithjohnthetubaguy: since the volume_actions module uses a kwargs we have some safety there17:13
johnthetubaguyso I am confused17:14
jgriffithjohnthetubaguy: so even if new nova talks to old cinder and sends something in the body that old cinder doesn't understand it's just going to ignore it anyway17:14
smcginnisSo we stay flexible with being able to parse kwargs, but in normal cases everything is explicitly passed in.17:14
johnthetubaguythere is a new format?17:14
jgriffithsmcginnis: correct17:14
jgriffithjohnthetubaguy: not a new format, just make the args required instead of "optional"17:14
jgriffithjohnthetubaguy: I don't explain code well, it may be best if I just finish the proposed patch and throw it out for everyone to check17:15
jgriffithjohnthetubaguy: if it gets barfed on that's ok17:15
ildikovwill we actually pass in new arguments or we just pass everything from now on that's expected to be there?17:15
johnthetubaguyOK, so that breaks old nova talking to new cinder right?17:15
jgriffithjohnthetubaguy: nope, I can address that17:15
ildikovas if nothing is fundamentally new than new Nova old Cinder should work17:15
johnthetubaguyso then they are not required still?17:15
jgriffithjohnthetubaguy: I'm saying I've thought of both cases old-nova:new-cinder and new-nova:old-cinder and it works17:15
jgriffithjohnthetubaguy: well... yeah, I see your point17:16
johnthetubaguylets back up a sec...17:16
jgriffithjohnthetubaguy: so technically YES, you could not change Nova and things would still work17:16
johnthetubaguyI am cool with nova starting to fill in some optional params it didn't before, that works17:16
jgriffithjohnthetubaguy: it's not as bad as it sounds I promise :)17:16
johnthetubaguyits the required on cinder side, or new format stuff that sounds dangerous17:16
jgriffithjohnthetubaguy: and actually at least in all of the iSCSI cases I checked Nova is giving everything I want consistently.  I just want to add one more thing for multi-attach :)17:17
johnthetubaguythats not the hostname right?17:17
jgriffithjohnthetubaguy: either that or initiator17:18
jgriffithso here's what we have at intialize-connection time:17:18
jgriffithjohnthetubaguy: sorry.. I said initiator; I mean either host or ip17:19
jgriffithinitiator might be better due to multipath17:19
jgriffithbut I'm not sure yet17:19
jgriffithjohnthetubaguy: basically we have MORE than enough info in the connector17:20
jgriffithjohnthetubaguy: including duplicate info IMO17:20
johnthetubaguymy worry is live-migrate17:20
hemnathe FC case adds 2 more items wwpns, wwnns17:20
hemnawhich are lists of wwns17:20
jgriffithwell... just host and ip17:20
jgriffithhemna: yeah, I noticed that17:20
hemna(in the connector)17:20
jgriffithjohnthetubaguy: why's that?17:20
johnthetubaguyif you need that, we need to start updating that across live-migrate, somehow17:20
openstackgerritSheel Rana proposed openstack/cinder: migrate to os-api-ref  https://review.openstack.org/31728317:21
jgriffithjohnthetubaguy: I mean, what's missing on the live-migration piece?17:21
johnthetubaguywe haven't got the neturon version of this straight yet17:21
johnthetubaguyjgriffith: so we would need to work out when to update the host during the live-migrate, so it doesn't break things17:21
jgriffithjohnthetubaguy: oh... got ya17:21
johnthetubaguyyou basically have a connection on both hosts for a while17:22
jgriffithjohnthetubaguy: so currently I believe we reinitialize, that would cover it I believe17:22
ildikovjgriffith: if we remove the second call17:22
*** earlephilhower has joined #openstack-cinder17:22
ildikovjgriffith: to initialize_connection I mean17:22
hemnayah, this is part of the effort to eliminate the need for the 2nd call to refetch the connection info that nova lost17:23
johnthetubaguyright, so if we have matched pairs of initialize / destroy(?) we are in a better place17:23
hemnanova had the connection_info at the start of the live migration process and writes over it in the bdm.17:23
johnthetubaguyand each of the initialize calls passes the correct host, so we might be good17:23
*** daneyon has quit IRC17:23
ildikovhemna: correct17:23
mriedemhemna: btw, i know why the live migration job is busted there, it's unrelated17:24
hemnathat was a WIP to 'save' the original connection_info, to avoid the 2nd call.17:24
jgriffithjohnthetubaguy: ildikov so what I'm getting at is that as long as I have the connector object and knew what to do at attach time I can cover those cases17:24
hemnamriedem, ok cool17:24
jgriffithhemna: you don't even need to do that if i have the initiator or host info at attach17:24
jgriffithhemna: I'll already know what is connected where17:24
jgriffithhemna: then *we* can determine what to do or not to do when initialize_connection is called17:24
hemnaI like the idea of not having nova call cinder a 2nd time.  just saves a useless round trip to Cinder...fwiw.17:24
jgriffithhemna: yeah, I'm fine with that17:25
johnthetubaguyjgriffith: if we set the host on initiailize_connection, then we are good, I guess?17:25
smcginnishemna: +117:25
jgriffithhemna: I've just been trying to minimize impact on Nova API interactions17:25
jgriffithhemna: I'd certainly be down with following up with something like that17:25
hemnajgriffith, ok cool.   lets figure out what's best.17:25
jgriffithjohnthetubaguy: yeah17:25
hemnaI kinda think it's a bug in nova that it calls cinder a 2nd time right now.17:25
hemnabut I'm open to whatevs17:25
johnthetubaguyhemna: yeah, that extra call is totally a bug17:26
jgriffithhemna: nah, I don't disagree with you.  I've viewed some of that as a secondary effort though17:26
ildikovhemna: I think we should remove that 2nd call, it's very confusing there...17:26
hemnajgriffith, but I think having cinder be smart in init_connection is also good in the case where it's not nova calling cinder.17:26
johnthetubaguyI mean, I wish we didn't need that information when calling cinder, but thats getting ahead of ourselves17:26
jgriffithjohnthetubaguy: :)17:26
mriedemso passing the connector to attach, that's going to store the connector in the bdm attachment table?17:26
jgriffithdon't forget, I'm trying to also solve the "old-nova:new-cinder" case17:27
jgriffithso I'm going to have that logic initially regardless17:27
jgriffithmriedem: you'll have to talk with hemna on that17:27
jgriffithmriedem: my proposal is not to do that, at least not right now17:27
mriedem"so what I'm getting at is that as long as I have the connector object and knew what to do at attach time I can cover those cases"17:27
mriedemi thought it came from ^17:27
mriedemor is the connector stored when we call init_connection?17:28
jgriffithmriedem: as long as I have the connector at initialize_connection, and then eithter, host, IP or initiator on the attach call we can handle all of it on the Cinder side17:28
hemnajgriffith, +117:29
mriedemok, nova passes the connector to init today17:29
mriedembut not the host/connector/ip to attach17:29
mriedembut looks like we can, it's been in the cinder api as optional17:29
jgriffithmriedem: correct17:29
mriedempoint i was trying to get to was older nova wouldn't be passing those, but it sounds like we've already covered that17:29
jgriffithmriedem: my only outstanding question was *which* of those is the best to use17:30
jgriffithmriedem: to cover FC, Ceph etc etc etc17:30
ildikovwasn't passing the host name the one created issues for us? or that was another call?17:30
jgriffithmriedem: as well as multi-path17:30
scottdaildikov: We had originally tried passing both host and Instance_id17:30
jgriffithildikov: yes, *host* was the one that you and hemna added17:30
mriedemjgriffith: nova has the connector when it calls attach, it just doesn't pass it, and the os-attach api doesn't take connector17:31
hemnaildikov, I think jgriffith is going to pull the hostname out of the connector though17:31
jgriffithildikov: I'm not sure that's the right one though17:31
hemnaas that's always been there.17:31
scottdaildikov: Passing both caused problems with older cinder17:31
ildikovyeah, I remember the check for that17:31
jgriffithhemna: it's there but it's not used... that's my point about the optional stuff :)17:31
johnthetubaguyscottda: it was stuff older cinder didn't accept, that time, I guess?17:32
scottdajohnthetubaguy: yes17:32
mriedemhow old? when was https://github.com/openstack/cinder/blob/master/cinder/api/contrib/volume_actions.py#L58-L66 added?17:32
hemnafor whatever reason when the host was added to the API it was enforced that it had to be either host or instance_uuid17:32
jgriffithjohnthetubaguy: correct17:32
hemnanot both.  which....seems silly17:32
jgriffithjohnthetubaguy: the way it was implemented it broke a bunch of drivers17:32
jgriffithbecause the param wasn't added to them17:32
ildikovhemna: correct, we can fix that now with an API microversion though if needed17:33
mriedemah https://github.com/openstack/cinder/commit/728f9838d2ca44b66fc240cac211fcae4b28641617:33
hemnaildikov, we could, but I would rather see what jgriffith has in mind and see if we can fix all this shit once and for all :)17:33
johnthetubaguywith microversion, Nova can detect what it needs to send, based on the detected cinder version, older system just keep using their older microversion17:33
*** IlyaG has joined #openstack-cinder17:34
*** lpetrut has quit IRC17:34
smcginnisjgriffith: Any idea how far along until we can take a look at a WIP patch?17:34
smcginnisI think that may help looking at a little code at this point.17:34
jgriffithsmcginnis: end of day today or tomorrow morning?17:34
ildikovhemna: I'm not against, I just don't want more CIs broken this time :)17:34
smcginnisjgriffith: Oh, awesome!17:35
jgriffithsmcginnis: or course I always say "today or tomorrow morning" :)17:35
jgriffithbut honestly I need to get this done and move on to other things17:36
hemnajgriffith, anything I can help with ?17:36
jgriffithhemna: you can review it for the FC pieces :)17:36
jgriffithhemna: I'll likely need some help on that part since I can't test it17:37
hemnaok :)  Just let me know if you need anything else.17:37
jgriffithwill do17:37
ildikovjgriffith: it's fine in an iterative way as well, I mean just upload what you have and then the next patches later, we're all eager to chime in :)17:37
smcginnisWe need to make sure jbernard or one of those types looks at it too.17:37
*** salv-orl_ has joined #openstack-cinder17:37
jgriffithsmcginnis: +1  jbernard and also eharney17:37
scottdasmcginnis: Hopefully you mean "Ceph type"17:37
smcginnisscottda: Haha, yep. :)17:37
jgriffithsmcginnis: smcginnis ceph, nfs and FC are the ones I'll want to get others to look at more closely17:38
smcginnisjgriffith: +1 Let's not get an iSCSI only solution if we can help it.17:38
hemnawait, people use non iSCSI ?17:39
smcginnisNot if jgriffith could help it. :P17:40
jgriffithsmcginnis: untrue17:40
jgriffithsmcginnis: RBD is cool17:40
*** salv-orlando has quit IRC17:40
hemnarm -rf cinder/zonemanager ......17:40
jgriffiththat's about it though :)17:40
openstackgerritParashuram Hallur proposed openstack/cinder: Implements : Blueprint CoprHD Cinder Drivers - https://blueprints.launchpad.net/cinder/+spec/coprhd-cinder-drivers  https://review.openstack.org/31764317:40
ildikovanything else to this item? :)17:41
*** avishay has quit IRC17:43
ildikovmriedem: johnthetubaguy: did you have a chance to look at these: https://review.openstack.org/#/c/312773/ , https://review.openstack.org/#/c/315789/ ?17:43
johnthetubaguyI did look at the first one, not properly looked at the second one17:44
johnthetubaguythere is a todo for me right there17:44
ildikovjohnthetubaguy: cool, thanks17:44
johnthetubaguywe should make sure we get the bugs marked with the correct priority, etc17:44
mriedemildikov: i haven't yet17:44
ildikovjohnthetubaguy: basically it would be good to see whether the direction is good or not, etc.17:45
johnthetubaguyadded a quick question on the second one17:45
ildikovjohnthetubaguy: can you check that?17:45
jgriffithhemna: ahh... cool17:46
ildikovjohnthetubaguy: I mean the prios, it can only be set by cores17:46
hemnajohnthetubaguy, I think other places were calling check_attach for some reason outside of the normal volume attach process17:46
hemnaI didn't understand those17:46
hemnahence, leaving it and the small refactor17:47
hemnaI can try finding those other cases and ask about those fwiw.17:47
ildikovhemna: I can try to check those17:47
*** e0ne has joined #openstack-cinder17:47
johnthetubaguyhemna: ildikov: I remember we spoke about them at some point, but I don't remember what they were now...17:48
*** ducttape_ has quit IRC17:48
ildikovhemna: I spent some time with check_attach during the last cycle, although I have to admit I did not understand everything :)17:48
*** earlephilhower has quit IRC17:48
ildikovhemna: we can both take a look and then sync up on the ones in question17:49
johnthetubaguyso the remove check_attach seems sound, just want to dig a little into the other use cases, just in case17:49
hemnasounds good17:49
ildikovhemna: it should not be that many cases17:49
ildikovjohnthetubaguy: cool!17:49
johnthetubaguythe saving of the connection_info seems too hacky, but I am failing to suggest a nice alternative right now, I also want to check what other drivers are doing, as it feels odd this is libvirt specific17:50
hemnajohnthetubaguy, yah it does seem hacky17:50
hemnaI just threw that up there as a starting point for discussion, hoping to get nova cores to look at it and provide some direction.17:51
mriedemhas there been any progress on volume migration / swap volume testing?17:51
ildikovscottda: ^^17:51
*** mtanino has quit IRC17:51
scottdaI've got it all working for retype-with-migration, but I need to get a patch up for the config changes17:51
johnthetubaguyhemna: ack17:51
scottdaWe've had Tempest tests for this in-tree for a while, but I'm not sure why multi-backend is not configured to have them run.17:52
mriedemscottda: patch in tempest?17:52
*** ChubYann has joined #openstack-cinder17:53
mriedemand last week you said devstack removed multi backend support?17:53
scottdamriedem: No, they just changed the way you do it.17:53
mriedemor are you going to do this in the multinode job?17:53
scottdamriedem: I saw the patch to remove ENABLE_MULTIBACKEND and was confused.17:53
scottdaI want to add multi-node job, there's a patch up for this, but the dev has moved on and I'll pick it up.17:54
*** ducttape_ has joined #openstack-cinder17:55
scottdaThat does not work ATM ^^^17:57
*** jgregor has joined #openstack-cinder17:57
scottdaDoes anyone know how to change owners on a patch?17:57
*** rlrossit has quit IRC17:57
*** mpjetta has joined #openstack-cinder17:57
mriedemscottda: you don't17:58
mriedembut you can contribute to it17:58
scottdaOK, but I need to be able to WIP it, so maybe best to have someone with the powers abandon it, and I'll push up a new one.17:58
ildikovscottda: you can only change Author, but that will not fix the issues you have with it :(17:58
mriedemscottda: just prefix the commit title with WIP:17:59
mriedemwhile you're working on it17:59
scottdaAlright, that's just details. I'll be back to work on this and keep the tests moving forward.17:59
mriedemit already has a -217:59
ildikovscottda: if you have a new one up can you add the link to the etherpad? I will add the current for now18:00
scottdasure, that works.18:00
* mriedem jumps to another meeting18:00
jgriffithscottda: you should use the existing one and be a co-author18:00
jgriffithscottda: you're completely enabled to be a comitter and mark as wip etc18:01
scottdajgriffith: ok, I'll do it that way then, thanks18:01
*** bardia has joined #openstack-cinder18:01
*** akshai has quit IRC18:01
*** bill_az has joined #openstack-cinder18:01
jgriffithscottda: unless you're actually going to rewrite everything.  If you're basing it on what somebody else started they should get credit18:02
*** bill_az has quit IRC18:02
scottdajgriffith: Yes, no problem with that. I think the existing patch mostly works, something is currently broken but that seems to be a regression.18:02
*** timcl1 has joined #openstack-cinder18:03
*** akshai has joined #openstack-cinder18:03
jgriffithscottda: darn bugs :)18:03
scottdaYeah, if it weren't for those.....18:03
smcginnisscottda: Thanks for picking that one up. I thing we need that coverage regardless.18:04
scottdasmcginnis: Yeah, everywhere I look we need coverage. But this also helps with testing Nova swap_volume, so double-good.18:04
ildikovhemna: scottda smcginnis johnthetubaguy jgriffith mriedem: regarding the new meeting slot, would Monday 1700UTC work for everyone?18:05
ildikovTuesday seems colliding for a few18:05
jgriffithildikov: sure why not :)18:05
johnthetubaguythat works for me18:05
hemnafine with me18:05
*** timcl has quit IRC18:05
scottdaWorks for me18:05
mriedemildikov: i'm on vacation next week18:05
smcginnisildikov: That would work for me.18:05
ildikovmriedem: that I know, I meant in general18:06
smcginnismriedem: You can still connect in. Come on, where's your commitment? :P18:06
mriedemkeep in mind monday 5/30 is a US holiday18:06
* johnthetubaguy goes to make dinner18:06
ildikovmriedem: I already asked johnthetubaguy to participate as you'll be out18:06
mriedemotherwise monday at 1700 works for me18:06
ildikovmriedem: ok, I officially give up meeting organization...18:07
*** rlrossit has joined #openstack-cinder18:07
ildikovok, then let's go with Mondays in general and we will have a backup slot for 5/3018:07
ildikovmriedem: thanks for the heads up!18:08
smcginnisildikov: Sounds like a plan.18:08
ildikovmriedem: and also have a nice vacation18:08
*** asselin__ has quit IRC18:08
ildikovthank you all, I will register the slot for Mondays 1700UTC18:09
scottdathanks ildikov18:09
ildikovok, patch proposed, fingers crossed :)18:13
*** akshai has quit IRC18:16
*** mtanino has joined #openstack-cinder18:18
*** akshai has joined #openstack-cinder18:18
*** rcernin has quit IRC18:31
*** lpetrut has joined #openstack-cinder18:34
openstackgerritIvan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: DO NOT MERGE. Testing job  https://review.openstack.org/31766818:36
*** salv-orl_ has quit IRC18:37
*** timcl1 has quit IRC18:41
*** akshai has quit IRC18:43
*** bill_az has joined #openstack-cinder18:43
*** akerr has quit IRC18:49
*** akerr has joined #openstack-cinder18:50
*** sdague has joined #openstack-cinder18:51
*** akshai has joined #openstack-cinder18:59
*** timcl has joined #openstack-cinder19:01
*** akshai_ has joined #openstack-cinder19:02
*** timcl has quit IRC19:03
smcginnisGrumble grumble grumble gerrit grumble...19:04
*** akshai has quit IRC19:04
openstackgerritMerged openstack/cinder: Make api_microversion_dev more explicit  https://review.openstack.org/31746019:05
openstackgerritMerged openstack/cinder: Updated from global requirements  https://review.openstack.org/31634319:05
Swansonsmcginnis, living the gerrit dream?19:07
*** timcl has joined #openstack-cinder19:11
*** lpetrut has quit IRC19:14
*** lcurtis has joined #openstack-cinder19:14
*** lpetrut has joined #openstack-cinder19:16
*** Lee1092 has quit IRC19:22
*** akshai_ has quit IRC19:25
*** sdake has joined #openstack-cinder19:26
*** daneyon has joined #openstack-cinder19:30
*** sdake has quit IRC19:32
*** sdake_ has joined #openstack-cinder19:32
*** mriedem has quit IRC19:35
*** salv-orlando has joined #openstack-cinder19:35
*** mriedem has joined #openstack-cinder19:41
*** e0ne has quit IRC19:48
*** e0ne has joined #openstack-cinder19:51
*** sdake_ has quit IRC19:53
*** e0ne has quit IRC19:54
*** ccesario has quit IRC19:54
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: Cleanup for Volume Object(Part 3)  https://review.openstack.org/31769819:55
*** sdake has joined #openstack-cinder19:58
*** rcernin has joined #openstack-cinder19:58
*** e0ne has joined #openstack-cinder20:01
*** e0ne has quit IRC20:04
*** sheel has quit IRC20:15
*** lpetrut has quit IRC20:18
smcginniseharney: ping20:22
*** daneyon_ has joined #openstack-cinder20:23
*** rlrossit has quit IRC20:24
smcginniseharney: nm, was going to ask why you approved/unapproved this one: https://review.openstack.org/#/c/316348/20:24
smcginniseharney: But no CI results yet, I'm guessing.20:24
*** akerr has quit IRC20:25
*** daneyon_ has quit IRC20:25
*** akshai has joined #openstack-cinder20:25
*** akshai_ has joined #openstack-cinder20:26
*** daneyon has quit IRC20:27
*** daneyon has joined #openstack-cinder20:29
*** akshai has quit IRC20:30
*** daneyon has quit IRC20:31
*** ntpttr_ has quit IRC20:32
openstackgerritMitsuhiro Tanino proposed openstack/cinder: Skip dict conversion when no volumes are updated  https://review.openstack.org/31772820:33
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Added support for failover_host failback  https://review.openstack.org/31006420:37
*** timcl has quit IRC20:37
*** daneyon has joined #openstack-cinder20:40
*** rcernin has quit IRC20:41
*** cknight1 has quit IRC20:45
*** gouthamr has quit IRC20:46
*** daneyon has quit IRC20:47
*** raildo is now known as raildo-afk20:47
*** daneyon_ has joined #openstack-cinder20:49
dulekCan someone take look on https://review.openstack.org/#/c/297699/ ? It's pretty trivial and already got +2s.20:50
openstackgerritMichal Dulko proposed openstack/cinder: Remove support for 1.x volume RPC API  https://review.openstack.org/29770120:51
*** daneyon has joined #openstack-cinder20:52
*** ntpttr_ has joined #openstack-cinder20:53
*** daneyon_ has quit IRC20:55
*** sdake_ has joined #openstack-cinder20:59
*** salv-orlando has quit IRC20:59
*** salv-orlando has joined #openstack-cinder21:00
eharneysmcginnis: yeah, it looks pretty safe, but i figured i'd see if any showed up21:00
*** sdake has quit IRC21:00
*** julim has quit IRC21:01
*** bill_az has quit IRC21:03
smcginniseharney: Yeah, good plan.21:05
*** bill_az has joined #openstack-cinder21:05
*** porrua has quit IRC21:06
*** raunak has quit IRC21:07
*** dustins has quit IRC21:12
*** raunak has joined #openstack-cinder21:13
*** sdague has quit IRC21:14
*** ccesario has joined #openstack-cinder21:25
*** baumann has left #openstack-cinder21:26
*** jgregor has quit IRC21:28
*** jamielennox is now known as jamielennox|away21:30
ntpttr_smcginnis: when you have a minute mind taking a look at https://review.openstack.org/#/c/230622/ ?21:33
*** haplo37 has joined #openstack-cinder21:37
*** jamielennox|away is now known as jamielennox21:39
*** caspinol has joined #openstack-cinder21:39
*** ametts has quit IRC21:41
*** edmondsw has quit IRC21:45
smcginnisntpttr_: Sure, I'll take a look.21:51
*** lprice has quit IRC21:53
*** ametts has joined #openstack-cinder21:54
*** sdake_ is now known as sdake21:54
*** mriedem has quit IRC21:55
*** kongwei has joined #openstack-cinder22:00
*** ozialien10 has quit IRC22:04
*** ozialien10 has joined #openstack-cinder22:04
kongweieharney: Thank you for the code review (https://review.openstack.org/#/c/312355). We will fix ASAP.22:07
*** ametts has quit IRC22:07
*** markvoelker has joined #openstack-cinder22:14
*** ducttape_ has quit IRC22:17
*** tpsilva has quit IRC22:17
*** xyang1 has quit IRC22:19
*** markvoelker has quit IRC22:21
*** markvoelker has joined #openstack-cinder22:22
*** dansmith has quit IRC22:23
*** dansmith has joined #openstack-cinder22:23
*** JoseMello has quit IRC22:29
*** aorourke__ is now known as aorourke22:30
*** aunnam_ has joined #openstack-cinder22:33
*** kongwei has quit IRC22:38
*** karthikp has quit IRC22:42
*** sdake_ has joined #openstack-cinder22:45
openstackgerritMerged openstack/python-cinderclient: Add options when uploading images to Glance  https://review.openstack.org/23062222:46
*** sdake has quit IRC22:48
*** haplo37 has quit IRC22:49
*** sticker has joined #openstack-cinder22:49
*** alkhodos has quit IRC22:52
*** kfarr has quit IRC22:54
*** sdake_ has quit IRC22:54
*** r-daneel has joined #openstack-cinder23:01
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: 3PAR Clean up VLUN deletion on detach  https://review.openstack.org/31776123:02
*** diogogmt has quit IRC23:03
*** diogogmt has joined #openstack-cinder23:05
openstackgerritJay Mehta proposed openstack/cinder: Implemented unit test cases for HPE 3PAR driver  https://review.openstack.org/31776323:07
*** markvoelker has quit IRC23:12
*** karthikp has joined #openstack-cinder23:18
*** lcurtis has quit IRC23:19
*** salv-orlando has quit IRC23:21
*** caspinol has quit IRC23:22
*** sdake has joined #openstack-cinder23:24
*** dansmith has quit IRC23:31
*** dansmith has joined #openstack-cinder23:32
*** daneyon has quit IRC23:35
*** IlyaG has quit IRC23:50
*** EinstCrazy has quit IRC23:55
*** karthikp has quit IRC23:59

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