Wednesday, 2017-11-22

*** rtjure has quit IRC00:00
*** rtjure has joined #openstack-cinder00:03
*** salv-orlando has quit IRC00:03
*** eharney has quit IRC00:14
*** itlinux has joined #openstack-cinder00:25
*** ntpttr_laptop has joined #openstack-cinder00:27
*** wanghao has joined #openstack-cinder00:31
*** wanghao has quit IRC00:31
*** moshele has joined #openstack-cinder00:31
*** wanghao has joined #openstack-cinder00:33
*** wanghao has quit IRC00:45
*** wanghao has joined #openstack-cinder00:45
*** zhangguoqing has joined #openstack-cinder00:46
*** zhangguoqing has quit IRC00:48
*** zhangguoqing has joined #openstack-cinder00:48
*** itlinux has quit IRC00:50
*** moshele has quit IRC00:50
*** Apoorva_ has quit IRC00:50
*** Apoorva has joined #openstack-cinder00:51
*** moshele has joined #openstack-cinder00:55
*** Chealion has quit IRC00:57
*** Chealion has joined #openstack-cinder00:58
*** zhangguo_ has joined #openstack-cinder00:59
*** zhangguoqing has quit IRC01:00
*** ntpttr_laptop has quit IRC01:01
*** zhangguo_ has quit IRC01:04
*** zhangguoqing has joined #openstack-cinder01:04
*** zhangguoqing has quit IRC01:05
*** zhangguoqing has joined #openstack-cinder01:05
*** moshele has quit IRC01:07
*** zhangguoqing has quit IRC01:29
*** zhangguoqing has joined #openstack-cinder01:29
*** rcernin has quit IRC01:29
*** rcernin has joined #openstack-cinder01:30
*** zhangguoqing has quit IRC01:34
*** zhangguoqing has joined #openstack-cinder01:37
*** links has joined #openstack-cinder01:44
*** itlinux has joined #openstack-cinder01:45
*** Apoorva_ has joined #openstack-cinder01:50
*** Apoorva_ has quit IRC01:50
tommylikehucinder bug list for bugsmash: https://bugs.launchpad.net/cinder/+bugs?field.tag=bugsmash01:51
*** itlinux has quit IRC01:53
*** Apoorva has quit IRC01:53
*** zhouyaguo has joined #openstack-cinder01:53
*** kukacz has quit IRC02:00
*** salv-orlando has joined #openstack-cinder02:01
*** kukacz_ has joined #openstack-cinder02:01
*** salv-orlando has quit IRC02:05
*** zhurong has joined #openstack-cinder02:15
*** zhaochao has quit IRC02:21
*** dalgaaf has quit IRC02:29
*** dalgaaf has joined #openstack-cinder02:30
*** zhaochao has joined #openstack-cinder02:30
openstackgerritMerged openstack/cinder master: Fix description for volume_type object in API Ref  https://review.openstack.org/42546102:30
*** markstur has quit IRC02:30
*** markstur has joined #openstack-cinder02:31
*** markstur has quit IRC02:31
*** stakeda has joined #openstack-cinder02:38
openstackgerritMerged openstack/cinder master: Update oslo.context RequestContext args  https://review.openstack.org/52201402:40
openstackgerritMerged openstack/cinder master: Add fixture to only emit DeprecationWarning once  https://review.openstack.org/52203602:46
openstackgerritMerged openstack/cinder master: Remove DB authorisation checking with quota API operations.  https://review.openstack.org/52171202:46
*** itlinux has joined #openstack-cinder02:47
openstackgerrityixuan zhang proposed openstack/cinder master: Storwize: add hyperswap volume support  https://review.openstack.org/49241402:49
*** markstur has joined #openstack-cinder02:58
*** lhx_ has joined #openstack-cinder03:00
openstackgerritTommyLike proposed openstack/cinder master: Support create volume from backup  https://review.openstack.org/51294903:00
*** salv-orlando has joined #openstack-cinder03:02
*** salv-orlando has quit IRC03:06
*** zhangbailin has joined #openstack-cinder03:07
*** bkopilov has quit IRC03:12
*** dhinesh has quit IRC03:14
*** zhangguoqing has quit IRC03:14
*** vivsoni has joined #openstack-cinder03:14
*** zhangguoqing has joined #openstack-cinder03:14
*** zhangbailin has quit IRC03:15
*** vivsoni__ has quit IRC03:15
openstackgerritRajat Sharma proposed openstack/python-cinderclient master: Migrate to Zuul v3  https://review.openstack.org/52172303:20
*** zhangguoqing has quit IRC03:23
*** zhangguoqing has joined #openstack-cinder03:23
*** zhangguoqing has quit IRC03:24
*** zhangguoqing has joined #openstack-cinder03:24
*** zhangguo_ has joined #openstack-cinder03:27
*** itlinux has quit IRC03:27
*** akekane has joined #openstack-cinder03:28
*** akekane is now known as abhishekk03:29
*** erlon has quit IRC03:30
*** zhangguoqing has quit IRC03:30
*** zhangguo_ has quit IRC03:31
*** zhangguoqing has joined #openstack-cinder03:34
*** dhinesh has joined #openstack-cinder03:35
openstackgerritPooja Jadhav proposed openstack/cinder master: V3 jsonschema validation: Attachments  https://review.openstack.org/51845703:56
*** itlinux has joined #openstack-cinder04:00
*** markstur has quit IRC04:01
*** salv-orlando has joined #openstack-cinder04:02
pooja_jadhavsmcginnis, tommylikehu, jungleboyj : Hi, I have rebased patch with master, could you please make workflow +1 again..?04:03
*** zhangguo_ has joined #openstack-cinder04:04
*** zhangguoqing has quit IRC04:05
*** salv-orlando has quit IRC04:07
pooja_jadhavtommylikehu: Thank you :)04:07
*** sam_nowitzki has joined #openstack-cinder04:09
openstackgerritPooja Jadhav proposed openstack/cinder master: V3 jsonschema validation: Group types  https://review.openstack.org/51964304:13
tommylikehupooja_jadhav: np04:15
openstackgerritPooja Jadhav proposed openstack/cinder master: V3 jsonschema validation: Group type specs  https://review.openstack.org/52028504:20
*** vivsoni_ has joined #openstack-cinder04:22
*** vivsoni has quit IRC04:22
*** moshele has joined #openstack-cinder04:23
*** zhangguo_ has quit IRC04:25
openstackgerritzhangbailin proposed openstack/cinder master: Update the detection mechanism of the volume under the host  https://review.openstack.org/52211004:26
openstackgerritwangzhenyu proposed openstack/cinder master: Modify the UT path of cinder volume  https://review.openstack.org/52211104:26
*** bkopilov has joined #openstack-cinder04:33
jungleboyjpooja_jadhav: will do when I get to a point where I can do some code reviews.04:36
pooja_jadhavjungleboyj: ok04:38
*** psachin has joined #openstack-cinder04:38
*** zhangguoqing has joined #openstack-cinder04:45
*** zhangguoqing has quit IRC04:45
*** zhangguoqing has joined #openstack-cinder04:46
*** zhurong has quit IRC04:48
*** lhx_ has quit IRC04:48
*** zhangguoqing has quit IRC04:51
*** lhx_ has joined #openstack-cinder04:54
*** sam_nowitzki has quit IRC04:59
*** moshele has quit IRC05:02
*** salv-orlando has joined #openstack-cinder05:03
*** uberjay has quit IRC05:04
openstackgerritzhangbailin proposed openstack/cinder master: Update the detection mechanism of the volume  https://review.openstack.org/52211005:06
*** salv-orlando has quit IRC05:07
*** uberjay has joined #openstack-cinder05:10
openstackgerritzhangbailin proposed openstack/cinder master: Update the detection mechanism of the volume  https://review.openstack.org/52211005:11
*** zhangguoqing has joined #openstack-cinder05:11
*** zhurong has joined #openstack-cinder05:12
*** pgadiya has joined #openstack-cinder05:30
*** pgadiya has quit IRC05:30
*** zhangguoqing has quit IRC05:35
*** zhangguoqing has joined #openstack-cinder05:35
*** zhangguo_ has joined #openstack-cinder05:39
*** zhangguo_ has quit IRC05:39
*** zhangguo_ has joined #openstack-cinder05:40
*** zhangguoqing has quit IRC05:40
*** moshele has joined #openstack-cinder05:49
*** AJaeger has left #openstack-cinder05:49
*** lkuchlan has joined #openstack-cinder05:51
*** zhangguoqing has joined #openstack-cinder05:51
*** zhangguo_ has quit IRC05:53
*** markvoelker has quit IRC05:53
*** zhangguo_ has joined #openstack-cinder05:59
*** zhangguoqing has quit IRC06:02
*** zhangguo_ has quit IRC06:04
*** salv-orlando has joined #openstack-cinder06:04
*** zhangguoqing has joined #openstack-cinder06:04
*** salv-orlando has quit IRC06:05
*** salv-orlando has joined #openstack-cinder06:05
openstackgerritwangzhenyu proposed openstack/cinder master: Modify the UT path of cinder volume  https://review.openstack.org/52211106:08
*** zhangguoqing has quit IRC06:09
*** zhangguoqing has joined #openstack-cinder06:09
*** gouthamr has quit IRC06:10
*** zhangguo_ has joined #openstack-cinder06:17
*** zhangguo_ has quit IRC06:17
*** zhangguo_ has joined #openstack-cinder06:17
*** zhangguoqing has quit IRC06:18
*** zhangguoqing has joined #openstack-cinder06:18
*** zhangguo_ has quit IRC06:18
*** lkuchlan has quit IRC06:18
*** lkuchlan has joined #openstack-cinder06:19
openstackgerritzhangbailin proposed openstack/cinder master: Update the detection mechanism of the volume  https://review.openstack.org/52211006:19
openstackgerritVan Hung Pham proposed openstack/python-cinderclient master: Fix to use "." to source script files  https://review.openstack.org/52212806:19
*** lpetrut has joined #openstack-cinder06:24
openstackgerritMerged openstack/cinder master: Try Fetching value from 'cluster' when raise exception  https://review.openstack.org/52145906:27
*** rtjure has quit IRC06:29
*** rtjure has joined #openstack-cinder06:32
*** openstackgerrit has quit IRC06:33
*** markvoelker has joined #openstack-cinder06:54
*** namnh has joined #openstack-cinder06:54
*** itlinux has quit IRC07:07
*** zhangguo_ has joined #openstack-cinder07:09
*** zhangguo_ has quit IRC07:09
*** zhangguoqing has quit IRC07:09
*** josecastroleon has joined #openstack-cinder07:09
*** zhangguoqing has joined #openstack-cinder07:10
*** openstackgerrit has joined #openstack-cinder07:12
openstackgerritMerged openstack/cinder master: V3 jsonschema validation: Attachments  https://review.openstack.org/51845707:12
*** zhangguo_ has joined #openstack-cinder07:15
*** zhangguoqing has quit IRC07:15
*** rcernin has quit IRC07:17
*** isaacb has joined #openstack-cinder07:23
openstackgerritwangzhenyu proposed openstack/cinder master: Modify the wrong link in document  https://review.openstack.org/52215007:23
*** zhangguoqing has joined #openstack-cinder07:33
*** zhangguoqing has quit IRC07:34
*** zhangguoqing has joined #openstack-cinder07:35
*** zhangguo_ has quit IRC07:35
*** zhangguoqing has quit IRC07:36
*** zhangguoqing has joined #openstack-cinder07:37
*** josecastroleon has quit IRC07:41
*** zhangguoqing has quit IRC07:41
openstackgerritTommyLike proposed openstack/cinder master: support create a volume from image snapshot  https://review.openstack.org/31494407:43
*** enriquetaso has quit IRC07:43
*** enriquetaso has joined #openstack-cinder07:43
*** moshele has quit IRC07:43
*** armax has quit IRC07:44
*** armax has joined #openstack-cinder07:45
*** armax has quit IRC07:45
*** armax has joined #openstack-cinder07:46
*** armax has quit IRC07:46
*** armax has joined #openstack-cinder07:46
*** armax has quit IRC07:47
*** armax has joined #openstack-cinder07:47
*** threestrands has quit IRC07:47
*** armax has quit IRC07:47
*** armax has joined #openstack-cinder07:48
*** armax has quit IRC07:48
*** pcaruana has joined #openstack-cinder07:49
*** armax has joined #openstack-cinder07:49
*** armax has quit IRC07:49
*** armax has joined #openstack-cinder07:50
*** lpetrut has quit IRC07:50
*** armax has quit IRC07:50
*** lpetrut has joined #openstack-cinder07:55
*** moshele has joined #openstack-cinder07:56
*** lkuchlan has quit IRC07:57
*** hoonetorg has quit IRC07:58
*** zhangguoqing has joined #openstack-cinder07:59
*** josecastroleon has joined #openstack-cinder08:08
*** hoonetorg has joined #openstack-cinder08:12
*** tesseract has joined #openstack-cinder08:14
*** alexchadin has joined #openstack-cinder08:18
*** gkadam has joined #openstack-cinder08:20
*** salv-orlando has quit IRC08:25
*** salv-orlando has joined #openstack-cinder08:26
*** lpetrut has quit IRC08:27
openstackgerritzhangbailin proposed openstack/cinder master: Update access control of show under hostAPI  https://review.openstack.org/52211008:29
*** salv-orlando has quit IRC08:30
*** amoralej|off is now known as amoralej08:36
*** lkuchlan has joined #openstack-cinder08:37
*** jsheeren has joined #openstack-cinder08:39
jsheerenhi all, i have a question about private volume types08:40
jsheereni created a private volume type, associated a backend etc08:40
jsheereneverything works when i create a volume and specify that new private type08:40
jsheerenif i do not specify a volume type, it sets the type to "none" and creates a volume on a seemingly random selected volume type08:41
jsheereneven if the tenant does not have access to that volume type08:41
jsheerenusing openstack ocata08:41
*** salv-orlando has joined #openstack-cinder08:42
jsheerenis there a config option that i need to set to fix that?08:43
*** lhx_ has quit IRC08:47
*** zhangguo_ has joined #openstack-cinder08:47
*** zhangguoqing has quit IRC08:49
*** lhx_ has joined #openstack-cinder08:50
jsheerenso it looks like cinder-scheduler is not respecting private volume types, as it's createing a volume on a type and backend the tenant should not have access to08:52
*** zhangguo_ has quit IRC09:09
*** zhouyaguo has quit IRC09:09
*** zhurong has quit IRC09:09
*** alexchadin has quit IRC09:26
*** alexchadin has joined #openstack-cinder09:27
*** wanghao has quit IRC09:31
*** wanghao has joined #openstack-cinder09:32
*** wanghao has quit IRC09:32
*** wanghao has joined #openstack-cinder09:33
*** wanghao has quit IRC09:33
*** wanghao has joined #openstack-cinder09:33
*** wanghao has quit IRC09:34
*** wanghao has joined #openstack-cinder09:34
*** wanghao has quit IRC09:35
*** wanghao has joined #openstack-cinder09:35
*** wanghao has quit IRC09:35
*** wanghao has joined #openstack-cinder09:36
*** wanghao has quit IRC09:36
*** wanghao has joined #openstack-cinder09:37
*** wanghao has quit IRC09:37
*** wanghao has joined #openstack-cinder09:37
*** wanghao has quit IRC09:38
*** AlexeyAbashkin has joined #openstack-cinder09:38
*** wanghao has joined #openstack-cinder09:38
*** lhx_ has quit IRC09:38
*** lhx_ has joined #openstack-cinder09:39
*** wanghao has quit IRC09:43
*** dhinesh has quit IRC09:43
*** isaacb has quit IRC09:44
*** ganso has joined #openstack-cinder09:49
*** abhishekk has quit IRC09:53
*** moshele has quit IRC09:55
*** stakeda has quit IRC09:55
*** niraj_singh has joined #openstack-cinder09:55
*** lpetrut has joined #openstack-cinder10:01
*** abhishekk has joined #openstack-cinder10:05
*** sri_ has quit IRC10:13
niraj_singhI am working on service token implementation for cinder to glance interaction. In cinder there is centralised config file. To create a session using load_session_from_conf_options method we need to provide group name as a parameter which should have config option like {'cacert','insecure','timeout','cert','key'} etc. So in cinder these option can be configured in default group because these options are in global_opt. There is no specific group10:15
niraj_singhassign to these config option. In my thinking we cannot use "DEFAULT" as a group name.10:15
niraj_singhIf we create a new group then it will have duplicate values. So how can i use these conf option to "load session from conf option". Is there a way?10:15
*** alexchadin has quit IRC10:19
*** gmann is now known as gmann_afk10:24
*** moshele has joined #openstack-cinder10:27
*** e0ne has joined #openstack-cinder10:29
*** zhonghua2 has joined #openstack-cinder10:32
*** freesky has quit IRC10:32
*** abhishekk has quit IRC10:34
*** namnh has quit IRC10:42
*** josecastroleon has quit IRC10:46
*** josecastroleon1 has joined #openstack-cinder10:46
*** lkuchlan has quit IRC10:57
*** josecastroleon1 has quit IRC11:09
*** bkopilov has quit IRC11:10
*** sdague has joined #openstack-cinder11:17
*** alexchadin has joined #openstack-cinder11:18
*** AlexeyAbashkin has quit IRC11:31
*** adisky_ has quit IRC11:32
*** AlexeyAbashkin has joined #openstack-cinder11:34
*** alexchadin has quit IRC12:18
*** alexchadin has joined #openstack-cinder12:19
*** kukacz_ is now known as kukacz12:20
tommylikehuhey jungleboyj, smcginnis : the etherpad for bugsmash in China :) https://etherpad.openstack.org/p/OpenStack-Bug-Smash-Queens-Wuhan-Bugs-List12:20
*** alexchadin has quit IRC12:21
*** hoonetorg has quit IRC12:23
*** zhaochao has quit IRC12:26
*** AlexeyAbashkin has quit IRC12:29
*** sticker has joined #openstack-cinder12:32
*** abishop has joined #openstack-cinder12:34
*** josecastroleon has joined #openstack-cinder12:41
*** AlexeyAbashkin has joined #openstack-cinder12:44
*** yangyapeng has joined #openstack-cinder12:45
*** gmann_afk is now known as gmann12:51
jsheerenso i was looking into bugs more and found: https://bugs.launchpad.net/cinder/+bug/151116012:52
openstackLaunchpad bug 1511160 in Cinder "Access to Private Volume Type Not Enforced." [Undecided,Invalid]12:52
jsheerenthis exactly describes what i'm facing12:52
*** spectr has joined #openstack-cinder12:53
*** lkuchlan has joined #openstack-cinder12:54
*** alexchadin has joined #openstack-cinder12:54
*** lhx_ has quit IRC12:58
*** bkopilov has joined #openstack-cinder12:58
*** zhangguoqing has joined #openstack-cinder13:01
*** amoralej is now known as amoralej|brb13:04
*** spectr has quit IRC13:04
*** gmann is now known as gmann_afk13:11
*** zhouyaguo has joined #openstack-cinder13:18
*** schmots has joined #openstack-cinder13:23
*** lhx_ has joined #openstack-cinder13:31
*** josecastroleon1 has joined #openstack-cinder13:39
*** josecastroleon has quit IRC13:39
*** dave-mccowan has joined #openstack-cinder13:40
*** mriedem has joined #openstack-cinder13:41
*** AlexeyAbashkin has quit IRC13:43
*** dave-mcc_ has joined #openstack-cinder13:44
*** links has quit IRC13:44
*** dave-mccowan has quit IRC13:45
*** alexchadin has quit IRC13:49
*** josecastroleon1 has quit IRC13:49
*** AlexeyAbashkin has joined #openstack-cinder13:50
*** amoralej|brb is now known as amoralej13:55
*** josecastroleon has joined #openstack-cinder14:10
openstackgerritMatthew Booth proposed openstack/os-brick master: Make close on cryptsetup volumes idempotent  https://review.openstack.org/52226314:21
*** yangyapeng has quit IRC14:24
*** zhangguoqing has quit IRC14:24
*** spectr has joined #openstack-cinder14:27
*** crushil has joined #openstack-cinder14:31
*** dpawlik is now known as danpawlik14:37
*** danpawlik has quit IRC14:38
*** danpawlik has joined #openstack-cinder14:39
*** jose-phillips has quit IRC14:46
*** jose-phillips has joined #openstack-cinder14:48
*** jose-phillips has quit IRC14:49
*** rcernin has joined #openstack-cinder14:51
*** sticker_ has joined #openstack-cinder14:53
*** sticker__ has joined #openstack-cinder14:54
openstackgerritSilvan Kaiser proposed openstack/cinder master: Adds a Cache for Volumes Created from Snapshots with Quobyte  https://review.openstack.org/50297414:56
openstackgerritSilvan Kaiser proposed openstack/cinder master: Adds Overlay Volumes Created from Snapshots to Quobyte  https://review.openstack.org/50705014:56
openstackgerritSean McGinnis proposed openstack/cinder master: Update unity tests to use our test base  https://review.openstack.org/52227114:56
*** sticker has quit IRC14:57
smcginnisOK, I am done with deprecation messages!14:57
*** sticker has joined #openstack-cinder14:57
*** sticker_ has quit IRC14:58
*** armax has joined #openstack-cinder14:58
*** sticker_ has joined #openstack-cinder14:59
*** sticker__ has quit IRC15:00
*** sticker__ has joined #openstack-cinder15:00
*** takedakn has joined #openstack-cinder15:00
*** sticker has quit IRC15:02
*** spectr has quit IRC15:03
*** sticker_ has quit IRC15:04
*** sticker__ has quit IRC15:05
*** erlon has joined #openstack-cinder15:08
*** gouthamr has joined #openstack-cinder15:12
*** josecastroleon has quit IRC15:15
*** lkuchlan has quit IRC15:15
*** sticker has joined #openstack-cinder15:16
*** rcernin has quit IRC15:18
*** yangyapeng has joined #openstack-cinder15:19
*** moshele has quit IRC15:27
*** jsheeren has quit IRC15:27
*** dhinesh has joined #openstack-cinder15:28
*** takedakn has quit IRC15:33
openstackgerritMerged openstack/cinder-specs master: Clean up required packages  https://review.openstack.org/52120615:33
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX Doc - clarification and typo fixes  https://review.openstack.org/51895615:38
*** AlexeyAbashkin has quit IRC15:49
*** psachin has quit IRC15:49
jgriffith@smcginnis don't stop now!!15:51
smcginnisjgriffith: :)15:51
smcginnisjgriffith: I think after all my patches land, the only deprecation messages left over are coming from libs we don't control.15:51
smcginnisActually, there is the db test cases: DeprecationWarning: Using class 'MySQLOpportunisticTestCase' (either directly or via inheritance) is deprecated15:53
smcginnisI guess that is something in our code yet.15:53
e0nesmcginnis: it's used in migrations tests15:55
kukaczhi, I'm experiencing issue deleting volumes which were created by migration from different vendor backend - the cinder object deletes but backend volume can't be matched by the driver (solidfire) and remains undeleted15:55
kukaczit's Ocata. I wonder if that might be a generic issue or is it more backend/driver specific15:56
smcginnise0ne: Yeah, I've seen those fixed in other projects I think. Need to look into what the recommended change would be.15:56
smcginniskukacz: Which backend are you using?15:56
kukaczsmcginnis: solidfire15:57
smcginniskukacz: I know there's someone here that can help you, but I'll wait for him to speak up. :)15:57
kukaczsmcginnis: I know too, was just shy to address so personally on first shot :-)15:58
smcginnis:)15:58
*** msno has joined #openstack-cinder15:58
*** markstur has joined #openstack-cinder16:00
*** salv-orlando has quit IRC16:02
*** salv-orlando has joined #openstack-cinder16:03
*** msno has quit IRC16:07
*** salv-orlando has quit IRC16:07
*** crushil has quit IRC16:17
jgriffithkukacz don't be shy :)16:20
jgriffithkukacz sorry, I didn't see this16:21
kukaczjgriffith: :), no worries16:21
jgriffithso can you explain a bit more about the volumes and how they were created?  You mentioned migration...16:21
kukaczsure16:21
openstackgerritSean McGinnis proposed openstack/cinder master: Use new oslo.db test cases  https://review.openstack.org/52229016:22
kukaczwe got two cinder backends - ceph and solidfire (driver version 2.0.7). I can create a volume using ceph backend and migrate it to solidfire using volume type set (openstack volume set ...)16:23
*** eharney has joined #openstack-cinder16:23
kukaczwhen I cinder delete such volume though, it gets deleted from cinder, but remains on SF16:23
openstackgerritMerged openstack/cinder master: Modify the wrong link in document  https://review.openstack.org/52215016:23
jgriffithkukacz I'll refrain from telling you what I think about cinder's migration *feature* :)16:25
jgriffithkukacz ok, and I'm assuming of course you see the volume on the SF side and have been able to access it?16:25
kukaczyes16:25
jgriffithkukacz any chance you can shoot over some c-vol logs?16:25
*** itlinux has joined #openstack-cinder16:26
kukaczit seems it is just being identified by "migration_uuid" instead of "uuid". I've added some extra debug calls into the driver to find the variable content16:27
*** sticker has quit IRC16:27
kukaczsure, I'll share16:27
kukaczwait a minute, I'll collect it into pastebin16:27
jgriffithkukacz yeah, so there's a bug in the migration code (I'm trying to find in LP) that I fixed a while back16:27
smcginnisProbably another case of issues from updating the volume ID during migrate.16:27
jgriffithThere's some issues in that code; like it will fail the last uuid swap portion of the process silently16:27
jgriffithThere's a couple of other things where it can fail silently and completely screw up the data base16:28
lpetrutjungleboyj: hi, any chance you may reconsider this -1 ? :) https://review.openstack.org/#/c/514659/ the CI has passed meanwhile, I think we had some connectivity issues.16:28
*** felipemonteiro has joined #openstack-cinder16:29
*** felipemonteiro_ has joined #openstack-cinder16:31
*** felipemonteiro has quit IRC16:34
*** crushil has joined #openstack-cinder16:35
kukaczjgriffith: here: https://gist.github.com/kukacz/26f905a3684acb2b54cf8049743f6a2c16:38
kukaczjgriffith: please let me know if you need more lines extracted from the logs16:39
*** itlinux has quit IRC16:47
*** e0ne has quit IRC16:50
*** moshele has joined #openstack-cinder16:53
*** tesseract has quit IRC16:57
*** itlinux has joined #openstack-cinder17:02
jungleboyjlpetrut:  Looking.17:03
lpetrutjungleboyj: great, thanks17:03
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - support for manage/unmanage snapshot  https://review.openstack.org/51873517:03
jungleboyjlpetrut: Done.17:04
*** salv-orlando has joined #openstack-cinder17:04
lpetrutjungleboyj: that was fast, thanks again :)17:05
jungleboyjAsk and ye shall receive ... Sometimes.  ;-)17:06
lpetrutsince we're talking about reviews, if there's any NFS like driver maintainer around: we had a request for manageable volumes support (a customer was migrating disks from a different cloud), so I've added it at the remotefs module level as a mixin: https://review.openstack.org/#/c/521158/17:08
*** salv-orlando has quit IRC17:08
lpetrutany similar driver may just inherit the mixin class (made sure that it also works out of the box for the nfs driver)17:10
*** moshele has quit IRC17:13
*** itlinux has quit IRC17:15
*** ntpttr_laptop has joined #openstack-cinder17:15
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - support for manage/unmanage snapshot  https://review.openstack.org/51873517:17
*** pcaruana has quit IRC17:18
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Incorrect stats reporting  https://review.openstack.org/51938317:18
*** zengyingzhe_ has quit IRC17:19
*** zengyingzhe_ has joined #openstack-cinder17:19
*** ntpttr_laptop has quit IRC17:24
*** zhouyaguo has quit IRC17:24
*** freesky has joined #openstack-cinder17:24
*** zhonghua2 has quit IRC17:24
*** gkadam has quit IRC17:25
dhineshon tempest cinder runs, i have failures like this "tempest.exceptions.VolumeResourceBuildErrorException: volume 702b535e-9677-46a7-96bb-c09cd52b4733 failed to build and is in ERROR status"17:25
dhineshhas anyone seen this before?17:25
smcginnisdhinesh: Looks like the volume failed to create. You would have to look in the c-vol logs to see why.17:26
smcginnisdhinesh: Usually works if you search for Traceback17:26
*** ntpttr_laptop has joined #openstack-cinder17:27
*** josecastroleon has joined #openstack-cinder17:28
*** salv-orlando has joined #openstack-cinder17:31
*** salv-orlando has quit IRC17:31
*** salv-orlando has joined #openstack-cinder17:31
dhineshsmcginnis: got it, looks like it was not able to initialize driver17:32
smcginnisThat'll do it. ;)17:32
*** AlexeyAbashkin has joined #openstack-cinder17:33
dhineshsmcginnis: any idea if tempest does 'type-create' for the volumes, before it starts the tests?17:35
smcginnisdhinesh: Devstack will create the default volume type.17:36
*** AlexeyAbashkin has quit IRC17:38
*** josecastroleon has quit IRC17:38
*** edmondsw has joined #openstack-cinder17:41
dhineshsmcginnis: in that case Devstack created "default_volume_type = lvmdriver-1" on cinder.conf  during the run, so do i need to overwrite it with my driver?17:43
*** e0ne has joined #openstack-cinder17:44
smcginnisdhinesh: You probably shouldn't have LVM set up at all. But you can set CINDER_ENABLED_BACKENDS to your backend to make sure it configures it as the default type.17:45
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - revert a volume to last snapshot  https://review.openstack.org/52065917:45
*** itlinux has joined #openstack-cinder17:48
*** crushil has quit IRC17:50
dhineshsmcginnis: ok thanks, am gonna try this out "CINDER_ENABLED_BACKENDS=mybackend:iscsi"17:50
jgriffithkukacz: I found the problem...17:53
jgriffithe0ne: do you remember this:  https://github.com/openstack/cinder/commit/a8e3220bd8e03697dd7ae1f34bf4e678558a46ed#diff-edd3b1109653835d85834f80ae3c9b2c17:53
jgriffithkukacz: e0ne so the problem is that was introduced to fix the deletion of migrating *off* of SF17:53
e0nejgriffith: I do a bit17:53
jgriffithkukacz: e0ne the result is it breaks deleting volumes *ON* to SF17:53
jgriffith:)17:53
e0nejgriffith: :(17:53
jgriffithThis is because of the stupid uuid swapping games that are played during migration17:54
jgriffithI think I can just add some logic to look both ways17:54
jgriffithanywho... I'll play with it a bit and then get a bug opened up and a fix17:55
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - revert a volume to last snapshot  https://review.openstack.org/52065917:57
*** lpetrut has quit IRC17:59
kukaczjgriffith: great that you've found it. I have not tested the opposite migration yet even17:59
jgriffithkukacz: hehe, that one works thanks to e0ne17:59
kukaczjgriffith: thanks for the fast progress!17:59
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Fix error handling and checks for generic volume groups  https://review.openstack.org/52108017:59
thingeejgriffith: jbernard replied back https://review.openstack.org/#/c/256091/17:59
jgriffithkukacz: e0ne let me know what you think of this:  https://gist.github.com/j-griffith/d53b95071bd9d7969cf92ea10b08d24818:01
jgriffiththingee: ack18:01
jgriffithjbernard: that's cool, one not/ask that's admittedly pretty nit-picky18:04
jgriffithjbernard: if you have a chance to throw in a comment in the code explaining that that would be super awesome18:04
jgriffithif not that's ok too, you're on your way to merging without me :)18:04
kukaczjgriffith: nice, it seems to me that will resolve my case as the migration id will have no chance to get into the lookup query. but I'm not a programmer, I am too far from reviewing code to John18:08
*** ntpttr_laptop has quit IRC18:13
*** ntpttr_laptop has joined #openstack-cinder18:13
*** dhinesh has quit IRC18:18
*** markstur_ has joined #openstack-cinder18:26
e0nejgriffith: I'm sorry, was afk18:26
e0nejgriffith: everything I remember, we both tested that patch on our envs18:26
*** markstur has quit IRC18:27
jgriffithe0ne: yeah, the trick is we need the one case you have for migrated from, and the second one for migrated too18:28
*** moshele has joined #openstack-cinder18:36
*** shaner has quit IRC18:36
*** lhx_ has quit IRC18:37
*** crushil has joined #openstack-cinder18:44
*** imacdonn has quit IRC18:45
*** imacdonn has joined #openstack-cinder18:45
*** shaner has joined #openstack-cinder18:48
*** dhinesh has joined #openstack-cinder18:49
*** dhinesh_ has joined #openstack-cinder18:51
*** moshele has quit IRC18:52
*** dhinesh has quit IRC18:53
*** e0ne has quit IRC19:02
*** e0ne has joined #openstack-cinder19:04
jbernardjgriffith: sure thing, will do19:09
*** e0ne has quit IRC19:15
*** e0ne has joined #openstack-cinder19:15
*** edmondsw has quit IRC19:16
openstackgerritJon Bernard proposed openstack/cinder master: RBD: support driver-assisted volume migration  https://review.openstack.org/25609119:16
jgriffiththanks jbernard19:19
*** lpetrut has joined #openstack-cinder19:19
*** schmots has quit IRC19:20
*** schmots_ has joined #openstack-cinder19:20
jbernardjgriffith: thank you, that one's been in the queue for a while ;)19:23
*** salv-orlando has quit IRC19:24
e0nejgriffith: as I understood, you just rollback my patch19:24
*** salv-orlando has joined #openstack-cinder19:24
*** schmots_ has quit IRC19:24
e0nejgriffith: I can't argue because I haven't an environment to test19:25
*** niedbalski is now known as ikslabdein19:26
*** ikslabdein is now known as gyosaboy19:26
*** gyosaboy is now known as niedbalski19:27
*** e0ne has quit IRC19:27
jgriffithe0ne: so I'm looking at the migration_completion code in the volume-object; I think this might apply to any driver that tries to rely on UUID's19:27
jgriffithI don't want to rollback your patch because that breaks the case you fixed :)19:27
*** edmondsw has joined #openstack-cinder19:28
*** salv-orlando has quit IRC19:28
*** schmots has joined #openstack-cinder19:30
*** eharney has quit IRC19:31
*** edmondsw has quit IRC19:32
openstackgerritMerged openstack/cinder master: SMBFS: add fixed image support  https://review.openstack.org/51465919:35
*** amoralej is now known as amoralej|off19:46
*** edmondsw has joined #openstack-cinder20:02
*** edmondsw has quit IRC20:06
ildikovjgriffith: ping20:19
jgriffithildikov: pong20:19
ildikovjgriffith: I added a very annoying comment to your shared_targets patch, I'm ashamed of it, but wondering what's your view still :)20:19
ildikovjgriffith: about a new mv20:20
*** gouthamr has quit IRC20:20
jgriffithildikov: yeah, no mv bump yet20:21
jgriffithildikov: you'll notice those changes don't update the response from the API *yet*20:21
jgriffithif those ever merge, I have a patch that updates the view builder and response as well as bumps the mv20:21
*** gouthamr has joined #openstack-cinder20:21
jgriffithI got tired of having to rebase mv numbers :)20:21
ildikovah ok, cool20:22
ildikovyeah, I hear ya20:22
ildikovwill try to get people to look at it next week20:22
ildikovI mean to get those patches merged20:22
ildikovBTW, did you plan to write up anything about the Cinder side of multi-attach?20:23
jgriffithI suppose I should, otherwise it doesn't look like anybody else will20:24
jgriffithwell, I know you would, but you can't do *everything*20:24
*** salv-orlando has joined #openstack-cinder20:25
*** itlinux has quit IRC20:25
ildikovI would certainly prefer not to :)20:28
ildikovI thought to look for people who could help out with things like the policy part20:29
ildikovso some text on what we would like to achieve would be nice20:29
ildikovso I can just point people there20:30
ildikovjgriffith: ^20:30
*** salv-orlando has quit IRC20:31
jgriffithildikov: sure... I can write up my take on it and then everyone can -1 it :)20:34
jgriffithIt's really pretty simple I think....20:35
jgriffithpolicy:20:35
jgriffith{allow_multi_atach: True, allow_multi_attach_rw: False}20:35
jgriffithor whatever that looks like now in policy yaml stuff20:36
ildikovallow_multi_attach_rw: False?20:36
ildikovso I think it was boot from volume where we wanted some policies20:37
ildikovor well, bootable volumes20:37
ildikovand next cycle we will look into the R/W, R/O stuff20:37
jgriffithThat's up to Nova I would think20:37
jgriffithbut if you want I guess we can add it in Cinder as well20:38
jgriffiththat doesn't make a ton of sense to me though20:38
jgriffithildikov: well you can't boot from an RO volume so your statement doesn't make sense to me20:39
ildikovthose two were separate20:40
ildikovI mean RO is after we have the basic multi-attach support20:40
ildikovand we need to/wanted to handle BFV now20:41
jgriffithildikov: you mean RW20:41
jgriffithwhatever, like I said... maybe I'm not the right person for the job :)20:41
jgriffithagain, put BFV controls in Nova IMO makes the most sense20:42
jgriffithor we can make things as complex and difficult as possible, that's always fun too :)20:42
jgriffithNova has knowledge of what hypervisor it's using and what it can/can't do etc20:42
jgriffithnot Cinder20:43
*** schmots has quit IRC20:45
ildikovI think we said something about bootable and multi-attach volumes20:45
jgriffithPeople said lots of things, they always do20:45
ildikovhence I would like to have a spec, where we can capture what we actually want in written form :)20:46
jgriffithand I said I would go ahead and do it20:46
ildikovI will save my questions for the review then :)20:47
jgriffithbut then you're going to -1 it because you won't like what I say and will want something different20:47
ildikovthank you!20:47
jgriffith:)20:47
ildikovI can ask questions without a -120:47
jgriffithhaha, I know20:48
ildikovI'm not obsessed with that :)20:48
jgriffithI just have a feeling I'm on a different page than everybody else with how this should work (again), but I'll give it a whirl20:48
ildikovat this point I think the question is who takes the time on writing up the first version20:53
jgriffithworks for me20:54
ildikovas I don't think anyone is on the same page with anyone else at this point :)20:54
jgriffithif I finish up what I'm working on now I'll see if I can't get something up this afternoon20:54
ildikovsounds good20:57
ildikovthank you!20:57
*** salv-orlando has joined #openstack-cinder21:00
*** salv-orlando has quit IRC21:00
*** salv-orlando has joined #openstack-cinder21:01
*** gouthamr has quit IRC21:03
*** abishop has quit IRC21:11
*** linkmark has joined #openstack-cinder21:13
jgriffithof course21:14
*** gouthamr has joined #openstack-cinder21:15
*** ganso has quit IRC21:28
*** jmlowe has joined #openstack-cinder21:32
*** dave-mcc_ has quit IRC21:39
*** threestrands has joined #openstack-cinder21:45
*** threestrands has quit IRC21:45
*** threestrands has joined #openstack-cinder21:45
*** jmlowe has quit IRC21:45
*** pooja_jadhav has quit IRC21:45
*** mchlumsky has quit IRC21:46
*** mchlumsky has joined #openstack-cinder21:51
*** felipemonteiro_ has quit IRC21:57
*** lpetrut has quit IRC21:57
*** bhagyashris has quit IRC22:00
*** schmots has joined #openstack-cinder22:01
*** schmots has quit IRC22:03
*** niraj_singh has quit IRC22:09
*** AlexeyAbashkin has joined #openstack-cinder22:09
*** Dinesh_Bhor has quit IRC22:11
jungleboyjjgriffith: ildikov  We talked about the boot from volume case when we were in Sydney22:12
jungleboyjhttps://etherpad.openstack.org/p/sydney-cinder-nova-cp-multi-attach22:12
jungleboyjAnswer was that we would let users shoot themselves in the foot with BFV if they wanted to.22:13
*** schmots has joined #openstack-cinder22:13
jungleboyjNicely handling R/W and RO options is future work.22:13
*** AlexeyAbashkin has quit IRC22:13
ildikovyeah, but I think we also want to have an option to turn it off with multi-attach, I might mix it up with something else this point though...22:15
*** Apoorva has joined #openstack-cinder22:16
jungleboyjildikov:  Correct.  We discussed the ability to just specify that multi-attach is not supported on a node.22:16
jgriffithildikov: jungleboyj regardless of what everybody said in Sydney, my vote is we implement just R/O with a policy to disable it altogether and that's pretty much it for the cinder side.22:17
jgriffithildikov: jungleboyj in the future as you point out R/W can be dealt with/added22:17
jungleboyjjgriffith:  I am ok with that as a starting point.22:18
jgriffithildikov: jungleboyj as far as BFV, if you really want it thwarted on the Cinder side, that's fine, but it belongs on the Nova side IMO22:18
jungleboyjjgriffith:  Totally agree.  If we are talking about just a Cinder Spec then I think it is irrelevant.22:18
jgriffithjungleboyj: agreed22:18
*** rcernin has joined #openstack-cinder22:20
*** ntpttr_laptop has quit IRC22:24
ildikovjungleboyj: jgriffith: the reason for going for RW right now is that because *no one* had an idea on how to set RO when I last asked...22:25
openstackgerritMerged openstack/cinder master: RBD: support driver-assisted volume migration  https://review.openstack.org/25609122:26
openstackgerritMichael Latchmansingh proposed openstack/cinder master: Adds DataCore Volume Drivers  https://review.openstack.org/43085822:26
jgriffithildikov: I'll mark them as R/O only for now is what I think I said22:26
jgriffithit's still going to be up to libvirt (or whoever) to figure that out IMO, unless a driver supports enforcing it on their side22:27
jgriffithanyway, my point was just that policy on the cinder side is useless22:27
jgriffithif the consumer side can't use it22:28
jungleboyjildikov: Well, the agreement was the first mount was R/W and the second R/O.22:28
*** schmots has quit IRC22:28
jgriffiththe whole multi-attach thing is dumb anyway frankly :)22:29
ildikovjungleboyj: yep, that's what I recall22:29
jungleboyjildikov:  Did they find that useless?22:29
ildikovjungleboyj: who found what useless?22:30
ildikovjgriffith: the easiest way to forget about it is implementing it at this point22:31
ildikovjgriffith: I'm sick and tired of re-discussing who not cloudy and dumb it is...22:32
*** jmlowe has joined #openstack-cinder22:32
jungleboyjildikov:  Did they find having one R/W connection with the multi-attach R/O useless?22:33
ildikovs/who/how/22:33
ildikovjungleboyj: who do you mean by they?22:33
ildikovjungleboyj: at this point I personally have no idea how to do it... :)22:34
*** schmots has joined #openstack-cinder22:36
jungleboyjildikov:  The people who said R/O attachments were useless.22:38
jungleboyjSo the problem there is you don't know how to make a subsequent attachment readlonly?22:39
ildikovjungleboyj: ask johnthetubaguy  :)22:39
*** schmots has quit IRC22:40
jungleboyjSo, whatever we can do to make things work and get something in would be good.22:46
ildikovjungleboyj: make it work as soon as we can and then polish it22:50
*** zhangguoqing has joined #openstack-cinder22:50
ildikovjungleboyj: people will have more preference once they have something to play with22:50
*** felipemonteiro has joined #openstack-cinder22:51
jungleboyjildikov:  Yes!22:52
*** felipemonteiro_ has joined #openstack-cinder22:52
*** Apoorva has quit IRC22:52
jungleboyjSo, we need to decide if we are going to let people do R/W for multi-attach or not.  If we do it comes with a lot of warnings.22:52
*** Apoorva has joined #openstack-cinder22:53
*** zhangguoqing has quit IRC22:54
*** felipemonteiro has quit IRC22:56
ildikovjungleboyj: I thought we decided that that's what we're going to do22:57
ildikovjungleboyj: and document what to be careful with, etc22:57
*** sdague has quit IRC23:03
*** AlexeyAbashkin has joined #openstack-cinder23:08
*** AlexeyAbashkin has quit IRC23:12
jungleboyjildikov:  Yep, so I am not sure what the issue is here.23:14
jungleboyjjgriffith:  ^^^ ?23:14
*** abishop has joined #openstack-cinder23:21
*** abishop has quit IRC23:23
*** lbragstad has quit IRC23:23
ildikovjungleboyj: no issues, let's have the Cinder side spec and argue there if anything's left to argue about :)23:31
*** Apoorva has quit IRC23:31
jungleboyj works for me.23:31
*** gouthamr has quit IRC23:34
*** felipemonteiro_ has quit IRC23:38
*** schmots has joined #openstack-cinder23:47
*** schmots has quit IRC23:50
*** schmots has joined #openstack-cinder23:54
*** edmondsw has joined #openstack-cinder23:54
*** schmots has quit IRC23:55
*** salv-orlando has quit IRC23:57
*** salv-orlando has joined #openstack-cinder23:58
*** edmondsw has quit IRC23:59

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