Tuesday, 2016-06-14

*** ihrachys has joined #openstack-cinder00:01
*** enriquetaso has quit IRC00:03
*** yangyapeng has quit IRC00:05
*** Julien-zte has joined #openstack-cinder00:10
*** anteaya has joined #openstack-cinder00:11
*** ihrachys has quit IRC00:17
*** diogogmt has quit IRC00:18
*** iceyao has joined #openstack-cinder00:20
*** ircuser-1 has quit IRC00:21
*** garthb has quit IRC00:22
*** jwcroppe_ has quit IRC00:23
*** suyash has quit IRC00:27
*** mtanin___ has joined #openstack-cinder00:30
*** ircuser-1 has joined #openstack-cinder00:30
*** mtanino has quit IRC00:31
*** jwcroppe has joined #openstack-cinder00:33
*** laughterwym has joined #openstack-cinder00:39
*** yamada-h has joined #openstack-cinder00:40
*** cknight has joined #openstack-cinder00:40
*** bardia has joined #openstack-cinder00:42
*** yamada-h has quit IRC00:44
*** iceyao has quit IRC00:56
*** cheneydc has joined #openstack-cinder00:56
*** laughterwym has quit IRC00:59
*** laughterwym has joined #openstack-cinder01:00
jay-mehtaCould anyone please review https://review.openstack.org/#/c/319511/ . CI has passed and is at +101:03
*** zul has joined #openstack-cinder01:04
*** Apoorva_ has quit IRC01:04
*** karthikp_ has joined #openstack-cinder01:05
*** yangyapeng has joined #openstack-cinder01:15
*** EinstCrazy has joined #openstack-cinder01:33
*** Lee1092 has joined #openstack-cinder01:35
*** laughterwym has quit IRC01:40
*** zul has quit IRC01:41
*** laughterwym has joined #openstack-cinder01:41
*** yamada-h has joined #openstack-cinder01:43
openstackgerritLisaLi proposed openstack/cinder: Create encrypted volumes from images  https://review.openstack.org/21656701:45
*** raunak has quit IRC01:46
*** yamada-h has quit IRC01:47
*** laughterwym has quit IRC01:50
*** laughterwym has joined #openstack-cinder01:51
openstackgerritLisaLi proposed openstack/cinder: Handle quota exceed exception  https://review.openstack.org/28166601:51
*** EinstCra_ has joined #openstack-cinder01:51
*** laughter_ has joined #openstack-cinder01:51
*** zul has joined #openstack-cinder01:54
*** laughte__ has joined #openstack-cinder01:54
*** laughterwym has quit IRC01:55
*** EinstCrazy has quit IRC01:55
*** laughter_ has quit IRC01:56
*** EinstCra_ has quit IRC01:58
*** EinstCrazy has joined #openstack-cinder01:59
*** EinstCrazy has quit IRC02:01
*** EinstCrazy has joined #openstack-cinder02:03
*** laughte__ has quit IRC02:04
*** cknight has quit IRC02:05
*** cknight has joined #openstack-cinder02:05
*** erlon has quit IRC02:06
openstackgerritwuyuting proposed openstack/cinder: Tegile driver doesn't honor create from snapshot volume size  https://review.openstack.org/32924302:09
*** mfisch has quit IRC02:10
*** jungleboyj has joined #openstack-cinder02:11
*** jseiler__ has joined #openstack-cinder02:15
*** laughterwym has joined #openstack-cinder02:15
*** mtanino has joined #openstack-cinder02:16
openstackgerritLisaLi proposed openstack/cinder-specs: Retype encrypted volumes  https://review.openstack.org/24859302:18
*** jseiler_ has quit IRC02:18
*** mtanin___ has quit IRC02:18
*** laughterwym has quit IRC02:19
*** suyash has joined #openstack-cinder02:19
*** EinstCrazy has quit IRC02:20
*** Swanson has quit IRC02:22
*** Swanson has joined #openstack-cinder02:22
*** EinstCrazy has joined #openstack-cinder02:29
*** julim has joined #openstack-cinder02:29
*** cknight has quit IRC02:30
yhayashiSorry to interrupt you. Could anyone reveiw https://review.openstack.org/#/c/297657/?  This patch has been in no review for a long time although CI passed at the end of Apr. and in May....02:31
*** dave-mccowan has quit IRC02:32
*** diablo_rojo has joined #openstack-cinder02:35
*** mtanino has quit IRC02:37
*** gouthamr_ has quit IRC02:38
*** laughterwym has joined #openstack-cinder02:43
*** EinstCrazy has quit IRC02:45
*** EinstCrazy has joined #openstack-cinder02:45
*** raunak has joined #openstack-cinder02:56
*** EinstCrazy has quit IRC03:01
*** adisky has joined #openstack-cinder03:02
*** EinstCrazy has joined #openstack-cinder03:04
*** links has joined #openstack-cinder03:08
*** openstackgerrit has quit IRC03:11
*** openstackgerrit has joined #openstack-cinder03:12
*** iceyao has joined #openstack-cinder03:12
*** laughterwym has quit IRC03:17
*** laughterwym has joined #openstack-cinder03:17
*** laughterwym has quit IRC03:22
*** diablo_rojo has quit IRC03:23
*** jungleboyj has quit IRC03:26
*** karthikp_ has quit IRC03:34
*** karthikp_ has joined #openstack-cinder03:34
*** raunak has quit IRC03:39
*** asselin_ has quit IRC03:40
*** nag_ has joined #openstack-cinder03:40
*** iceyao has quit IRC03:41
*** raunak has joined #openstack-cinder03:41
*** ociuhandu has quit IRC03:45
*** ociuhandu has joined #openstack-cinder03:49
*** ducttape_ has quit IRC03:50
*** laughterwym has joined #openstack-cinder03:53
*** laughter_ has joined #openstack-cinder03:55
*** laughterwym has quit IRC03:58
*** yamada-h has joined #openstack-cinder04:03
*** ociuhandu has quit IRC04:05
*** Poornima has joined #openstack-cinder04:07
*** huyang has quit IRC04:08
*** wanghao has quit IRC04:10
*** wanghao has joined #openstack-cinder04:11
openstackgerritYaoZheng proposed openstack/cinder: image_size_m should get ceiling of image  https://review.openstack.org/32926204:11
*** raunak has quit IRC04:12
*** markvoelker has quit IRC04:16
*** sgotliv has joined #openstack-cinder04:20
*** sgotliv has quit IRC04:20
*** garthb has joined #openstack-cinder04:27
*** raunak has joined #openstack-cinder04:33
*** Poornima has quit IRC04:43
*** Poornima has joined #openstack-cinder04:43
*** yamada-h has quit IRC04:53
*** yamada-h has joined #openstack-cinder04:54
*** zhenguo_ has joined #openstack-cinder04:54
*** yamada-h has quit IRC04:59
*** sheel has joined #openstack-cinder05:05
*** yamada-h has joined #openstack-cinder05:05
*** pgadiya has joined #openstack-cinder05:06
*** lpetrut has joined #openstack-cinder05:09
*** sgotliv has joined #openstack-cinder05:10
*** laughterwym has joined #openstack-cinder05:10
*** bhavik has joined #openstack-cinder05:12
*** laughter_ has quit IRC05:14
*** markvoelker has joined #openstack-cinder05:17
*** salv-orl_ has quit IRC05:17
*** salv-orlando has joined #openstack-cinder05:18
*** itzdilip has joined #openstack-cinder05:20
*** markvoelker has quit IRC05:22
*** karthikp_ has quit IRC05:24
*** avishay has joined #openstack-cinder05:29
gushemna/smcginnis/others: hrm, I just read the scrollback.05:29
*** salv-orl_ has joined #openstack-cinder05:29
gusSo the issue is that you need to do upgrade testing without being able to do any sort of upgrade to anything other than os_brick?  That sounds awkward.05:29
*** ChubYann has quit IRC05:29
*** iceyao has joined #openstack-cinder05:30
gusMy initial response would have been: just add the right helper_command to nova.conf in some phase before the os_brick grenade upgrade is attempted.05:31
*** salv-orlando has quit IRC05:31
openstackgerritXuepeng Ji proposed openstack/cinder-specs: Support image signature verification  https://review.openstack.org/32870805:34
gusI could fix the world (rootwrap and/or some library like oslo.config) so that oslo.privsep can determine the name of the rootwrap executable, and then change the default to rootwrap (rather than plain sudo) - but if you can't upgrade any nova conf files (like rootwrap filters) then I don't think that will actually help.05:34
*** salv-orl_ has quit IRC05:34
*** salv-orlando has joined #openstack-cinder05:34
*** GB21 has joined #openstack-cinder05:39
*** EinstCrazy has quit IRC05:40
*** raunak has quit IRC05:40
openstackgerritKushal Wathore proposed openstack/cinder: Add unit tests in cinder component for 3PAR driver  https://review.openstack.org/31537605:41
*** garthb has quit IRC05:47
*** savihou has joined #openstack-cinder05:49
openstackgerritPeter Wang proposed openstack/cinder: VNX: New Cinder driver in Newton  https://review.openstack.org/31242705:52
*** GB21 has quit IRC05:53
*** GB21 has joined #openstack-cinder05:53
*** laughterwym has quit IRC05:55
*** laughterwym has joined #openstack-cinder05:56
*** arecknag has joined #openstack-cinder05:59
*** liverpooler has quit IRC06:00
*** liverpooler has joined #openstack-cinder06:01
*** savihou has quit IRC06:06
*** rcernin has joined #openstack-cinder06:12
*** salv-orlando has quit IRC06:13
*** salv-orlando has joined #openstack-cinder06:14
*** GB21 has quit IRC06:15
*** Poornima has quit IRC06:17
*** suyash has quit IRC06:17
*** jbernard has quit IRC06:17
*** jbernard has joined #openstack-cinder06:19
openstackgerritxing-yang proposed openstack/cinder: Add generic volume groups  https://review.openstack.org/32245906:21
*** GB21 has joined #openstack-cinder06:31
*** Poornima has joined #openstack-cinder06:32
*** EinstCrazy has joined #openstack-cinder06:32
*** bardia has joined #openstack-cinder06:33
openstackgerritYaoZheng proposed openstack/cinder: image_size_m should get ceiling of image  https://review.openstack.org/32926206:36
*** jwcroppe has quit IRC06:38
pocketprotectoranyone using scality here06:39
*** EinstCrazy has quit IRC06:39
*** ihrachys has joined #openstack-cinder06:41
*** EinstCrazy has joined #openstack-cinder06:41
*** EinstCrazy has quit IRC06:44
*** EinstCrazy has joined #openstack-cinder06:44
*** jwcroppe has joined #openstack-cinder06:47
*** openstackgerrit has quit IRC06:48
*** openstackgerrit has joined #openstack-cinder06:49
*** ociuhandu has joined #openstack-cinder06:50
*** ducttape_ has joined #openstack-cinder06:51
*** tesseract has joined #openstack-cinder06:56
*** ducttape_ has quit IRC06:56
*** EinstCrazy has quit IRC06:59
*** EinstCrazy has joined #openstack-cinder07:00
*** akapil has joined #openstack-cinder07:00
*** akapil has quit IRC07:01
dulekxinli: Use classmethod instead of remotable_classmethod now. You could find it out checking out git log to see what got in recently: https://review.openstack.org/#/c/312929/07:03
*** akapil has joined #openstack-cinder07:05
*** bardia has quit IRC07:06
*** markvoelker has joined #openstack-cinder07:18
*** iceyao has quit IRC07:19
*** pece has joined #openstack-cinder07:21
*** markvoelker has quit IRC07:22
*** pece has quit IRC07:32
*** EinstCrazy has quit IRC07:37
*** EinstCrazy has joined #openstack-cinder07:40
*** lpetrut has quit IRC07:42
*** salv-orlando has quit IRC07:44
*** salv-orlando has joined #openstack-cinder07:44
*** EinstCrazy has quit IRC07:45
*** EinstCrazy has joined #openstack-cinder07:45
*** GB21 has quit IRC07:47
*** EinstCrazy has quit IRC07:49
*** EinstCrazy has joined #openstack-cinder07:50
*** EinstCrazy has quit IRC07:55
*** akapil has quit IRC07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:00
DuncanTyhayashi: One question on your patch, otherwise looks good to me08:01
*** EinstCrazy has joined #openstack-cinder08:01
*** e0ne has joined #openstack-cinder08:05
*** faizy has joined #openstack-cinder08:08
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
*** akapil has joined #openstack-cinder08:16
*** GB21 has joined #openstack-cinder08:18
*** faizy_ has joined #openstack-cinder08:21
*** faizy has quit IRC08:22
*** mvk has quit IRC08:24
*** pgadiya has quit IRC08:25
*** itzdilip has quit IRC08:25
*** nag_ has quit IRC08:25
*** nikeshm has quit IRC08:25
*** pgadiya has joined #openstack-cinder08:26
openstackgerritReedip proposed openstack/python-cinderclient: Fix argument order for assertEqual to (expected, observed)  https://review.openstack.org/26400608:27
*** akapil has quit IRC08:27
*** akapil has joined #openstack-cinder08:28
*** pece has joined #openstack-cinder08:28
*** laughterwym has quit IRC08:29
*** akapil_ has joined #openstack-cinder08:31
*** laughterwym has joined #openstack-cinder08:32
*** akapil_ has quit IRC08:34
*** akapil_ has joined #openstack-cinder08:35
*** akapil has quit IRC08:35
*** laughterwym has quit IRC08:37
*** akapil has joined #openstack-cinder08:43
*** Julien-zte has quit IRC08:44
*** akapil_ has quit IRC08:46
*** mvk has joined #openstack-cinder08:51
*** EinstCrazy has quit IRC08:52
*** ducttape_ has joined #openstack-cinder08:52
*** EinstCrazy has joined #openstack-cinder08:54
openstackgerritYuriy Nesenenko proposed openstack/cinder: Deleting volume metadata keys with a single request  https://review.openstack.org/32839908:55
yhayashiDuncanT: Thanks! I reply it for your comment. I think there is no problem with the ordering. thanks!08:55
DuncanTyhayashi: Great, I'll get back to it shortly.08:57
*** Mech422__ has joined #openstack-cinder08:57
yhayashiDuncanT: thanks!08:57
*** ducttape_ has quit IRC08:58
Mech422__Morning - anyone know if you still have to run cinder type-create if you want to use a default_volume_type?08:58
Mech422__I'm getting unknown volume type errors on my last rebuild ?08:58
Mech422__(cinder.conf has default_volume_type set to the same value as enable_driver)08:59
dulekMech422__: backend_name != volume_type09:02
dulekMech422__: You should create just a single volume_type with correct extra_specs pointing to a backend.09:03
dulekMech422__: And then set it as default.09:03
*** vgridnev has quit IRC09:03
Mech422__dulek: I'm running Kolla - so its doing the setup ... it doesn't appear to do a type-create in any of the playbooks09:04
Mech422__dulek: it just has the enabled_ and default_ options in cinder.conf09:04
dulekMech422__: If it sets default_type in cinder.conf, but doesn't create it, then that's a bug in Kolla IMO>09:05
Mech422__dulek: ok - thats what I'm asking - the create is still required ?09:05
Mech422__dulek: (I'm asking because the installation guides never mention having to do a type create...)09:05
dulekMech422__: Yes, I think so. I'm not much surprised, I've seen inconsistencies in configs provided by Kolla before. :)09:06
Mech422__dulek: ok - I can give them a heads up...09:06
Mech422__dulek: if I just comment out the default_volume_type in cinder.conf and restart cinder - should it work then ? (I wanna have some testing to show kolla guys..)09:07
dulekMech422__: Yup, it should create volumes without volume type, which will probably work.09:08
Mech422__dulek: awesome - thanks !09:08
dulekMech422__: But I still recommend having a volume type.09:08
Mech422__dulek: heh - I'll work on a lil patch if this fixes it09:08
Mech422__dulek: the really freaky part is, when I was rebuilding last week - it worked ?!?09:08
Mech422__dulek: and there haven't been any kolla checkins to the cinder stuff09:09
Mech422__dulek: so I wanted to ask and make sure I'm not crazy :-P09:09
*** lpetrut has joined #openstack-cinder09:10
*** EinstCrazy has quit IRC09:10
*** EinstCrazy has joined #openstack-cinder09:10
dulekMech422__: You're building from Cinder master? It could happen we've changed something, but didn't noticed. Or noticed and it's described in release notes.09:11
Mech422__dulek: yeah - I'm tracking kolla master - which I think tracks master on all the openstack projects ?09:12
dulekMech422__: I don't really know…09:13
Mech422__dulek: well - you got me to the next step - so thank you very much :-)09:13
*** lprice1 has quit IRC09:13
*** sticker has joined #openstack-cinder09:14
*** jordanP has joined #openstack-cinder09:14
*** lprice has joined #openstack-cinder09:15
dulekHappy to help. :)09:15
*** akapil has quit IRC09:17
*** EinstCrazy has quit IRC09:18
openstackgerritSzymon Borkowski proposed openstack/cinder: Convert backup_device to OVO  https://review.openstack.org/32933209:18
*** EinstCrazy has joined #openstack-cinder09:18
*** akapil has joined #openstack-cinder09:20
*** GB21 has quit IRC09:25
*** akapil_ has joined #openstack-cinder09:27
*** Mech422__ is now known as Mech42209:27
*** akapil_ has quit IRC09:27
*** akapil_ has joined #openstack-cinder09:27
openstackgerrithuanan proposed openstack/cinder: Huawei: Support backup snapshot  https://review.openstack.org/32933609:30
*** akapil has quit IRC09:30
*** EinstCrazy has quit IRC09:33
*** EinstCrazy has joined #openstack-cinder09:33
*** yamada-h has quit IRC09:35
*** pgadiya has quit IRC09:36
*** EinstCrazy has quit IRC09:41
*** EinstCrazy has joined #openstack-cinder09:41
openstackgerrithuanan proposed openstack/cinder: Huawei: Support backup snapshot  https://review.openstack.org/32933609:43
*** pgadiya has joined #openstack-cinder09:43
*** tyrola has joined #openstack-cinder09:44
*** EinstCrazy has quit IRC09:45
*** EinstCrazy has joined #openstack-cinder09:46
*** mvk has quit IRC09:50
*** ociuhandu has quit IRC09:51
*** mvk has joined #openstack-cinder09:51
*** caoshufeng has joined #openstack-cinder09:52
*** GB21 has joined #openstack-cinder09:52
*** EinstCrazy has quit IRC09:53
*** EinstCrazy has joined #openstack-cinder09:53
openstackgerritwanghao proposed openstack/cinder: Volume update API misses metadata checking  https://review.openstack.org/32935209:54
*** cheneydc has quit IRC10:01
*** EinstCrazy has quit IRC10:03
*** EinstCrazy has joined #openstack-cinder10:03
*** caoshufeng has quit IRC10:04
Mech422dulek: gah - just rebuild/redeployed from source, and it suddenly works ?!?!10:06
*** links has quit IRC10:07
Mech422dulek: wow - I got the same exception in cinder-api, but this time the vm spawned ?10:12
*** ociuhandu has joined #openstack-cinder10:13
Mech422dulek: log snippet if you'd care to look: http://pastebin.com/F6AqYiNk10:15
*** links has joined #openstack-cinder10:21
*** yhayashi has quit IRC10:21
*** EinstCrazy has quit IRC10:25
*** pgadiya has quit IRC10:26
*** yangyape_ has joined #openstack-cinder10:26
*** pgadiya has joined #openstack-cinder10:26
*** yangyapeng has quit IRC10:30
*** yangyape_ has quit IRC10:30
*** akapil_ has quit IRC10:31
*** sdague has joined #openstack-cinder10:31
*** akapil has joined #openstack-cinder10:32
*** EinstCrazy has joined #openstack-cinder10:32
*** akapil has quit IRC10:34
*** yamada-h has joined #openstack-cinder10:36
*** EinstCrazy has quit IRC10:36
*** salv-orlando has quit IRC10:38
*** salv-orlando has joined #openstack-cinder10:39
*** yamada-h has quit IRC10:41
*** EinstCrazy has joined #openstack-cinder10:45
*** jwcroppe has quit IRC10:51
*** jwcroppe has joined #openstack-cinder10:51
*** ducttape_ has joined #openstack-cinder10:54
*** salv-orlando has quit IRC10:58
*** bhavik has quit IRC10:58
*** salv-orlando has joined #openstack-cinder10:58
*** ducttape_ has quit IRC10:59
dulekMech422: I don't know why VM got spawned, but this is definitely a bug in Kolla - it should create a volume type and not use backend name there.11:04
*** akapil has joined #openstack-cinder11:08
*** permalac has quit IRC11:09
Mech422dulek: yeah - I think whats happening is cinder is being smart and falling back to default type None..11:13
Mech422dulek: and the actual 'hang the VM' problem is somewhere else11:13
dulekMech422: Could be!11:13
Mech422dulek: when I list the image - the volume type is ''11:13
dulekMech422: image? You mean - volume?11:13
*** jwcroppe has quit IRC11:14
Mech422err..yeah - sorry11:14
Mech422'cinder list' gives Volume type of '-'11:14
Mech422so now I'm trying to figure out what else can be causing the problem...11:15
Mech422and I opened a bug against Kolla to either remove the default_type or do a proper create11:15
*** bhavik has joined #openstack-cinder11:15
*** sheel has quit IRC11:15
*** liverpooler has quit IRC11:17
*** alyson_ has joined #openstack-cinder11:17
*** liverpooler has joined #openstack-cinder11:17
Mech422this is sorta freaking me out as it seems like a race - sometimes I deploy the cluster and it works...other times it just spins on 'allocating block device' in Horizon11:17
dulekMech422: I think now it's time to move to #kolla and talk with the team there. I'm not very experienced in how kolla works.11:21
Mech422dulek: oh yeah - I'm waiting for them to wake up :-)11:22
*** EinstCrazy has quit IRC11:22
Mech422dulek: I appreciate your help - don't expect you to solve it for me :-)11:22
*** jwcroppe has joined #openstack-cinder11:23
*** ociuhandu has quit IRC11:23
*** ociuhandu has joined #openstack-cinder11:23
*** ociuhandu has quit IRC11:26
*** ociuhandu has joined #openstack-cinder11:26
*** salv-orl_ has joined #openstack-cinder11:28
*** salv-orlando has quit IRC11:32
*** akapil has quit IRC11:37
*** akapil has joined #openstack-cinder11:38
*** akapil has quit IRC11:42
*** akapil has joined #openstack-cinder11:43
openstackgerritxiexs proposed openstack/cinder: Make examples consistent with actul API  https://review.openstack.org/32685211:50
*** laughterwym has joined #openstack-cinder11:52
*** raildo-afk is now known as raildo11:54
*** timcl has joined #openstack-cinder11:54
*** ducttape_ has joined #openstack-cinder11:54
*** laughterwym has quit IRC11:57
*** winston-d_ has quit IRC11:57
*** tpsilva has joined #openstack-cinder11:57
*** bhavik has quit IRC11:58
*** takedakn has joined #openstack-cinder11:58
*** yangyapeng has joined #openstack-cinder12:01
openstackgerritYuriy Nesenenko proposed openstack/cinder: Deleting volume metadata keys with a single request  https://review.openstack.org/32839912:01
*** ducttape_ has quit IRC12:03
*** yamada-h has joined #openstack-cinder12:06
*** avishay has quit IRC12:06
*** akapil has quit IRC12:07
*** faizy_ has quit IRC12:07
*** akapil has joined #openstack-cinder12:08
openstackgerrithuanan proposed openstack/cinder: Huawei: Support backup snapshot  https://review.openstack.org/32933612:08
*** yamada-h has quit IRC12:10
*** markvoelker has joined #openstack-cinder12:13
*** bhavik has joined #openstack-cinder12:13
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file  https://review.openstack.org/29407612:16
*** pauloewerton has joined #openstack-cinder12:16
*** adrianofr has joined #openstack-cinder12:19
adrianofrscottda: Hey Scott :) . I've addressed your comments on https://review.openstack.org/#/c/28546912:23
adrianofrscottda: Thanks for looking at it12:23
openstackgerritxing-yang proposed openstack/cinder: Add group type and group specs  https://review.openstack.org/32016512:23
*** edleafe has left #openstack-cinder12:28
*** kaisers1 has quit IRC12:29
*** salv-orl_ has quit IRC12:33
*** salv-orlando has joined #openstack-cinder12:34
openstackgerritMerged openstack/cinder: Backup and restore broken in zfssaiscsi driver  https://review.openstack.org/32846512:34
openstackgerritMerged openstack/cinder: Cleanup volume_type_projects  https://review.openstack.org/32885212:37
*** nikeshm has joined #openstack-cinder12:38
openstackgerritYuriy Nesenenko proposed openstack/cinder: Deleting volume metadata keys with a single request  https://review.openstack.org/32839912:46
openstackgerritxing-yang proposed openstack/cinder: Add generic volume groups  https://review.openstack.org/32245912:46
*** pgadiya has quit IRC12:48
*** GB21 has quit IRC12:51
*** takedakn has quit IRC12:53
openstackgerritMerged openstack/cinder: Set QoS on cloned volumes  https://review.openstack.org/32562612:57
openstackgerritxing-yang proposed openstack/cinder: WIP: Add group snapshots  https://review.openstack.org/32805212:59
*** salv-orlando has quit IRC13:00
*** salv-orlando has joined #openstack-cinder13:00
*** akerr has joined #openstack-cinder13:01
*** xyang1 has joined #openstack-cinder13:04
*** akapil has quit IRC13:08
*** laughterwym has joined #openstack-cinder13:09
*** akapil has joined #openstack-cinder13:12
*** diogogmt has joined #openstack-cinder13:13
nikeshmhi13:14
nikeshmfollowing http://docs.openstack.org/admin-guide/blockstorage-driver-filter-weighing.html for add filter_function code13:15
nikeshmi want to restrict the volume creation if number of volumes exceed 250 on our array13:16
*** asselin_ has joined #openstack-cinder13:16
nikeshmusing this in code13:17
nikeshmhttp://paste.openstack.org/show/515982/13:17
nikeshmnot sure what string to give in cinder.conf for filter_function13:18
*** akapil has quit IRC13:18
*** akapil has joined #openstack-cinder13:18
nikeshm'capabilities.total_voumes < 250' is not working13:18
nikeshm'capabilities.total_voumes < 5 is not working13:19
nikeshmit is allowing to create more volumes13:19
*** dave-mccowan has joined #openstack-cinder13:20
*** akapil has quit IRC13:21
*** akapil has joined #openstack-cinder13:21
*** gouthamr has joined #openstack-cinder13:21
openstackgerritMerged openstack/cinder: Updated from global requirements  https://review.openstack.org/32873813:26
openstackgerritTomaz Vieira proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages  https://review.openstack.org/31078513:27
*** crose has joined #openstack-cinder13:27
*** porrua has joined #openstack-cinder13:27
*** diogogmt has quit IRC13:28
*** srinivasd has joined #openstack-cinder13:29
srinivasdHi.i am using fc pass through in ubuntu 14.04.4 for our FC driver in in CI using this link https://github.com/openstack/third-party-ci-tools/blob/master/provisioning_scripts/fibre_channel/invoke_fc_passthrough.sh but it is failing at the commnad virsh attach-device giving error as  http://paste.openstack.org/show/515937/13:29
*** ericksonsantos has joined #openstack-cinder13:29
*** faiz89 has joined #openstack-cinder13:30
*** dustins has joined #openstack-cinder13:32
*** Srinu has joined #openstack-cinder13:32
*** mriedem_away is now known as mriedem13:33
*** akapil has quit IRC13:34
*** erlon has joined #openstack-cinder13:35
openstackgerritYuriy Nesenenko proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request  https://review.openstack.org/30044313:36
*** akapil has joined #openstack-cinder13:37
*** akapil has quit IRC13:38
*** akapil has joined #openstack-cinder13:39
*** cknight has joined #openstack-cinder13:39
*** edmondsw has joined #openstack-cinder13:41
*** akapil_ has joined #openstack-cinder13:41
*** mtanino has joined #openstack-cinder13:43
*** faizy_ has joined #openstack-cinder13:44
*** darosale has joined #openstack-cinder13:44
*** akapil has quit IRC13:44
*** eharney has joined #openstack-cinder13:45
scottdaIf a volume is part of a consistencyGroup and the volume is in error, and you take a CG snapshot, should the CG snapshot fail?13:45
Srinuhai.i am using fc pass through in ubuntu 14.04.4 for our FC driver in in CI using this link https://github.com/openstack/third-party-ci-tools/blob/master/provisioning_scripts/fibre_channel/invoke_fc_passthrough.sh but it is failing at the commnad virsh attach-device giving error as  http://paste.openstack.org/show/515937/13:46
*** fifieldt has joined #openstack-cinder13:47
*** iceyao has joined #openstack-cinder13:49
*** diablo_rojo has joined #openstack-cinder13:49
xyang1scottda: it should.  did you see it succeed?13:49
scottdaxyang1: Yes, On mitaka13:50
scottdaxyang1: Thanks for the reply. I'll repro on Devstack and file a bug.13:50
xyang1scottda: sure, thanks13:50
*** EinstCrazy has joined #openstack-cinder13:50
*** lprice has quit IRC13:51
Srinuxyang1: hi.i am using fc pass through in ubuntu 14.04.4 for our FC driver in in CI using this link https://github.com/openstack/third-party-ci-tools/blob/master/provisioning_scripts/fibre_channel/invoke_fc_passthrough.sh but it is failing at the commnad virsh attach-device giving error as  http://paste.openstack.org/show/515937/13:51
openstackgerritxiexs proposed openstack/cinder: Fix a wrong key for the volume create API  https://review.openstack.org/32945513:52
*** bhavik has quit IRC13:52
xyang1Srinu: not sure why.  hemnafk may know13:53
Srinuhemnafk: Hi Please help me. i am using fc pass through in ubuntu 14.04.4 for our FC driver in in CI using this link https://github.com/openstack/third-party-ci-tools/blob/master/provisioning_scripts/fibre_channel/invoke_fc_passthrough.sh but it is failing at the commnad virsh attach-device giving error as  http://paste.openstack.org/show/515937/13:55
SrinuDid anyone face the same issue.13:56
*** cdelatte has joined #openstack-cinder13:56
asselin_Srinu, hemnafk isn't around right now. Stay logged in and perhaps he'll respond later today.13:56
Srinuasselin_: Ok Thank you.13:57
*** vnogin has joined #openstack-cinder14:01
*** Julien-zte has joined #openstack-cinder14:02
*** Julien-z_ has joined #openstack-cinder14:03
*** pgadiya has joined #openstack-cinder14:04
*** pgadiya has quit IRC14:05
*** iceyao has quit IRC14:06
*** Julien-zte has quit IRC14:07
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376614:08
*** ducttape_ has joined #openstack-cinder14:12
*** akerr has quit IRC14:12
*** ducttape_ has quit IRC14:12
*** ducttape_ has joined #openstack-cinder14:13
*** faiz89 has quit IRC14:13
*** links has quit IRC14:14
*** faiz89 has joined #openstack-cinder14:16
*** faizy_ has quit IRC14:17
*** rlrossit has joined #openstack-cinder14:19
*** EinstCrazy has quit IRC14:21
*** zhonghua-lee has quit IRC14:21
*** zhonghua-lee has joined #openstack-cinder14:22
*** bill_az has joined #openstack-cinder14:22
*** akerr has joined #openstack-cinder14:22
*** timcl has quit IRC14:24
*** lprice has joined #openstack-cinder14:26
*** Nag_ has joined #openstack-cinder14:28
*** adisky has quit IRC14:30
*** mriedem1 has joined #openstack-cinder14:31
*** mriedem has quit IRC14:33
*** kaisers has joined #openstack-cinder14:33
*** avishay has joined #openstack-cinder14:36
*** jwcroppe_ has joined #openstack-cinder14:39
*** arecknag has quit IRC14:41
*** jwcroppe has quit IRC14:41
*** jwcroppe_ has quit IRC14:45
*** eharney has quit IRC14:45
*** jwcroppe_ has joined #openstack-cinder14:47
*** julim has quit IRC14:47
srinivasdjwcroppe_ : Hi.i am using fc pass through in ubuntu 14.04.4 for our FC driver in in CI using this link https://github.com/openstack/third-party-ci-tools/blob/master/provisioning_scripts/fibre_channel/invoke_fc_passthrough.sh but it is failing at the commnad virsh attach-device giving error as  http://paste.openstack.org/show/515937/14:47
*** reduxio has joined #openstack-cinder14:49
reduxioHello. I'm developing a cinder driver, and followed the openstack documentation on third-parti CI as far as i understood it. I'm at the point that i want to try and run the tests using the sandbox.. there's no 'ON' switch.. can anyone help?14:50
*** jwcroppe has joined #openstack-cinder14:50
*** jwcroppe_ has quit IRC14:52
*** bill_az has quit IRC14:54
*** akapil_ has quit IRC14:55
*** crose has quit IRC14:56
*** eharney has joined #openstack-cinder14:56
*** xinli has joined #openstack-cinder14:56
*** akapil has joined #openstack-cinder14:57
*** timcl has joined #openstack-cinder14:59
*** jwcroppe has quit IRC15:00
*** jwcroppe has joined #openstack-cinder15:01
*** jwcroppe_ has joined #openstack-cinder15:01
Swansonreduxio, You want to ask on #openstack-infra15:03
*** jwcroppe has quit IRC15:05
*** faiz89 has quit IRC15:05
*** garthb has joined #openstack-cinder15:05
reduxioSwanson: thanks. i will15:07
*** avishay has quit IRC15:07
*** akapil has quit IRC15:07
openstackgerritxiexs proposed openstack/cinder: Add proper scheduler_hint key to dev-ref  https://review.openstack.org/32945515:08
*** geguileo_phone has joined #openstack-cinder15:08
*** fifieldt has quit IRC15:08
*** frontrunner has joined #openstack-cinder15:09
*** akapil has joined #openstack-cinder15:10
Srinuasselin_: Hi15:11
*** rcernin has quit IRC15:11
*** jwcroppe has joined #openstack-cinder15:11
asselin_Srinu, hello15:11
*** karthikp_ has joined #openstack-cinder15:12
*** enriquetaso has joined #openstack-cinder15:12
Srinuasselin_: i am getting this  error: Failed to attach device from /tmp/tmp.eTlOXNIKfi_fcoe.xml error: internal error: unable to execute QEMU command 'device_add': Device initialization failed.  Attach result: 115:12
Srinuasselin_: when i run the virsh attach-device command15:12
*** jwcroppe_ has quit IRC15:13
*** akapil_ has joined #openstack-cinder15:14
*** julim has joined #openstack-cinder15:14
asselin_Srinu, I don't know. Didn't you just ask me in -infra?15:14
*** pece has quit IRC15:15
*** akapil has quit IRC15:17
*** bardia has joined #openstack-cinder15:18
*** akapil_ has quit IRC15:18
scottdaxyang1: I filed a bug for cgshapshot succeeds with volume in error: https://bugs.launchpad.net/cinder/+bug/159245115:21
openstackLaunchpad bug 1592451 in Cinder "consistency snapshot creation passed even if the volume is in error state" [Undecided,New]15:21
scottdaxyang1: I realize I cannot repro in devstack because lvm doesn't have cgs, but we saw this in our lab with a 3par. I'll try to find a 3par person to have a look, but maybe you've someone who could verify.15:22
*** jungleboyj has joined #openstack-cinder15:25
*** salv-orlando has quit IRC15:25
*** salv-orlando has joined #openstack-cinder15:26
*** Julien-z_ has quit IRC15:28
*** faiz89 has joined #openstack-cinder15:30
*** salv-orlando has quit IRC15:30
*** salv-orlando has joined #openstack-cinder15:30
*** tongli has joined #openstack-cinder15:31
*** faiz89 has quit IRC15:32
*** Poornima has quit IRC15:33
*** liverpooler has quit IRC15:34
openstackgerritTom Swanson proposed openstack/cinder: WIP: Dell SC: Use Live Volume for replication  https://review.openstack.org/32953115:35
*** permalac has joined #openstack-cinder15:38
openstackgerritMerged openstack/python-cinderclient: Fix argument order for assertEqual to (expected, observed)  https://review.openstack.org/26400615:39
openstackgerritxing-yang proposed openstack/cinder: Add generic volume groups  https://review.openstack.org/32245915:42
*** delattec has joined #openstack-cinder15:44
*** jdillaman has quit IRC15:44
*** cdelatte has quit IRC15:47
*** bardia_ has joined #openstack-cinder15:51
*** bardia has quit IRC15:51
*** bardia_ has quit IRC15:51
*** delattec has quit IRC15:51
*** delattec has joined #openstack-cinder15:53
*** jwcroppe_ has joined #openstack-cinder15:54
*** jwcroppe has quit IRC15:55
*** gcb has quit IRC15:55
*** timcl has quit IRC15:55
*** gcb has joined #openstack-cinder15:56
xyang1scottda: you can use cinder.tests.unit.fake_driver.FakeGateDriver15:57
scottdaxyang1: Thanks. You mean to set FakeGateDriver as a backend in devstack?15:58
xyang1scottda: yes, just replace lvm15:58
*** laughterwym has quit IRC15:58
scottdaxyang1: Great, thanks15:58
xyang1scottda: it is the same as lvm except it implemented cg functions for testing purpose15:59
*** laughterwym has joined #openstack-cinder15:59
*** e0ne has quit IRC16:00
DuncanTsmcginnis: Ping re priv.sep and upgrade... it doesn't look like the sudo rules for priv.sep are incompatible with those normally required, is that right?16:01
DuncanTsmcginnis: (bug https://bugs.launchpad.net/os-brick/+bug/1592043)16:01
openstackLaunchpad bug 1592043 in os-brick "os-brick 1.4.0 increases volume setup failure rates" [Undecided,Confirmed]16:01
*** hemnafk is now known as hemna16:02
*** laughterwym has quit IRC16:03
openstackgerritAleksey Ruban proposed openstack/cinder: Replaced nas_ip option and added HA support  https://review.openstack.org/31934916:05
*** gouthamr has quit IRC16:05
*** raunak has joined #openstack-cinder16:05
*** srinivasd has quit IRC16:06
*** leeantho has joined #openstack-cinder16:06
hemnagus ping16:09
*** raunak has quit IRC16:10
*** xyang1 has quit IRC16:10
openstackgerritxing-yang proposed openstack/cinder: Add group type and group specs  https://review.openstack.org/32016516:11
*** raunak has joined #openstack-cinder16:12
*** xyang1 has joined #openstack-cinder16:13
*** rlrossit has quit IRC16:13
*** GB21 has joined #openstack-cinder16:14
*** mriedem has joined #openstack-cinder16:15
*** permalac has quit IRC16:16
*** Apoorva has joined #openstack-cinder16:16
*** bluex has quit IRC16:16
*** Apoorva has quit IRC16:17
*** rlrossit has joined #openstack-cinder16:17
*** rlrossit has quit IRC16:17
*** rlrossit has joined #openstack-cinder16:17
*** bluex has joined #openstack-cinder16:18
*** Apoorva has joined #openstack-cinder16:18
*** mriedem1 has quit IRC16:18
*** yangyapeng has quit IRC16:19
nikeshmhi16:19
*** yangyapeng has joined #openstack-cinder16:19
nikeshmhi, how to write filter_function strings, some examples using capabilities.total_volumes, some examples using stats.16:21
*** Nag_ has quit IRC16:22
smcginnisDuncanT: hemna knows better, but I believe the issue is the sudo rules get deleted before privsep starts.16:24
smcginnisDuncanT: Sean D wrote up a good summary on the ML.16:24
hemnasmcginnis, DuncanT I'm working on an os-brick patch16:24
hemnaI'm about to test it16:24
smcginnishemna: Awesome. Did you see the ML discussion?16:24
hemnano I haven't yet.16:25
DuncanThemna: Can it be fixed by adding some new (priv-sep specific) rules to sudoers before starting the upgrade though?16:25
hemnaI just got in a few minutes ago and am pounding out some code to try and get this fixed16:25
smcginnishemna: Might be good to take a look. It's not just a cinder issue.16:25
hemnaDuncanT, there are multiple ways to fix this16:25
hemnathe problem is the rules for running grenade tests16:25
hemnathose rules prohibit from changing any config files during the upgrade16:25
smcginnisDuncanT: The problem is, according to the grenade rules that's not allowed.16:25
smcginnisYeah, what he said. :)16:26
hemnare: rootwrap filters, nova.conf, cinder.conf, sudoers.d, etc.16:26
hemnayou can't touch ANY of those16:26
hemna:(16:26
hemnathat's a whole other discussion how anything is every added16:26
hemnabut whatevs16:26
*** faiz89 has joined #openstack-cinder16:27
*** timcl has joined #openstack-cinder16:27
DuncanTWell, the grenade rules might be broken though. Would changing the sudoers file before the upgrade (so both the old and new stuff works) solve the problem?16:27
*** gouthamr has joined #openstack-cinder16:27
smcginnisDuncanT: I think so.16:27
hemnaadding an entry in the sudoers.d would fix it16:27
hemnaI've validated that yesterday16:27
hemnayou'll have to talk to sdague about that :)16:27
hemnaI'm trying another approach16:27
DuncanThemna: IT looked like it would. Thanks.16:28
sdagueDuncanT: right, that's option #116:28
sdaguedon't support CD for this upgrade, explain what the admin has to do to modify configs before they upgrade16:28
DuncanTsdague: why is it not CD as long as the steps don't break anything currently running? Saying no config changes ever is very limiting16:29
*** ChubYann has joined #openstack-cinder16:29
DuncanTsdague: It isn't a backwards incompatible config change16:29
*** huanan_L has quit IRC16:30
DuncanT(not that I've any objection to adding fallback code with warnings if that is possible)16:30
*** huanan has joined #openstack-cinder16:31
sdagueIt is limitting, but if you are machine CDing daily, manual steps are no good. That conversation happened a long time ago.16:32
DuncanTsdague: Hmmmm, so you'd prefer a fallback with deprecation warnings?16:33
DuncanTsdague: Using the usual full cycle deprecation I guess16:33
sdagueDuncanT well, if this was a one time thing, I'd say take the hit. But the fact that we're going to hit this with every privsep library... it feels like we need a better pattern16:34
sdagueos-vif is about to run into the same issue16:34
*** nkrinner is now known as nkrinner_afk16:34
smcginnissdague: I agree. I think this is a bigger issue than just working around it in os-brick.16:34
DuncanTsdague: I can see how you can do a fallback, but damn it's messy code16:34
sdaguethe mailing list is probably the best place to discuss this all, I tried to frame the bug there and steps forward16:34
smcginnis+116:35
DuncanTThe fallback will have to be done per callsite too.16:35
sdagueand open up for people exploring options there16:35
DuncanTThe mailing list is a very slow feedback loop though... we should certainly summarise any discussions and detail possible solutions there, but brainstorming is easier and quicker on IRC16:36
sdaguewell, except you also need folks like gus in the loop16:36
sdagueso IRC ends up being lots of repetition and coming to concensus without enough folks involved, which makes all the consensus no good16:37
sdaguethis is complicated enough, it's ok to take it slow and make sure everyone's good with eventual solution16:37
*** geguileo_phone has quit IRC16:37
*** faiz89 has quit IRC16:38
*** tongli has quit IRC16:39
*** geguileo_phone has joined #openstack-cinder16:41
*** chris_morrell has joined #openstack-cinder16:41
*** Srinivas has joined #openstack-cinder16:42
*** tesseract has quit IRC16:43
*** faiz89 has joined #openstack-cinder16:43
*** e0ne has joined #openstack-cinder16:43
SrinivasHi. I am using fc passthrough in ubuntu 14.04.4 in devstack for ci. But it is giving me an error as..failed to attach device and device intialization failed16:44
patrickeastSrinivas: oh hey, sry for the late reply, saw your message last night and forgot to get back to you16:44
*** e0ne has quit IRC16:44
SrinivasHemnafk : can you help me16:45
patrickeastSrinivas: make sure you you iommu enabled16:45
dulekxinli: You weren't online, so I'll repeat. :)16:45
dulekxinli: Use classmethod instead of remotable_classmethod now. You could find it out checking out git log to see what got in recently: https://review.openstack.org/#/c/312929/16:45
patrickeastSrinivas: thats usually what caused that for me16:45
hemnaSrinivas, I can't at the moment.16:46
SrinivasPatrickeast :  i enabled iommu16:46
hemnaSrinivas, I'm trying to fix a grenade issue16:46
openstackgerritxing-yang proposed openstack/cinder: Add generic volume groups  https://review.openstack.org/32245916:47
*** mvk has quit IRC16:47
patrickeastSrinivas: what i would recommend is going through instructions like this https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/5/html/Virtualization/chap-Virtualization-PCI_passthrough.html those are from redhat... but its pretty much the same for ubuntu hypervisors too16:48
patrickeastSrinivas: make sure all the pre-reqs are setup, and that you can dettach the device OK16:48
*** wanghao has quit IRC16:49
*** salv-orlando has quit IRC16:49
*** wanghao has joined #openstack-cinder16:49
*** suyash has joined #openstack-cinder16:50
*** salv-orlando has joined #openstack-cinder16:50
patrickeastSrinivas: also, if its not too late for your CI setup, I found that using https://wiki.openstack.org/wiki/Pci_passthrough works more consistently than the fc passthrough script in the third party tools repo16:51
patrickeasti switched my ci to use that feature and things are working better16:51
patrickeastthe main reason i think is because if passthrough fails nodepool kills the node (since the instance goes into error state) instead of the jenkins job failing16:51
*** jordanP has quit IRC16:53
SrinivasPatrickeast: i can do pci attach and detach in the script in third party ci in invoke fc passthrough16:54
*** timcl has quit IRC16:54
hemnaarg16:54
asselin_patrickeast, that's a nice link.16:54
hemnaalmost have it16:54
asselin_Srinivas, look at that link patrickeast suggested. It's definitely a superior way to get pci passthrough working.16:55
hemnafriggin rootwrap16:55
*** timcl has joined #openstack-cinder16:56
patrickeastasselin_: yea i remember seeing it a long time ago and being unable to get it working, but I tried again recently and its been pretty sweet16:57
patrickeastasselin_: it looks like the code in nova pretty much does the same thing that the passthrough script does, but with some smarter error handling16:57
asselin_no doubt16:58
*** garthb_ has joined #openstack-cinder16:59
*** garthb has quit IRC17:00
SrinivasPatrickeast: i am getting an error while running virsh attach-device17:02
Srinivaserror : qemuMonitorJSONCheckError:354 : internal error: unable to execute QEMU command 'device_add': Device initialization failed.17:03
*** lpetrut has quit IRC17:05
Srinivas Patrickeast: error as unable to execute qemu command device add device intialization failed17:07
*** chris_morrell has quit IRC17:09
*** garthb_ has quit IRC17:10
*** garthb_ has joined #openstack-cinder17:10
patrickeastSrinivas: ok, understood17:12
*** e0ne has joined #openstack-cinder17:12
openstackgerritMatt Smith proposed openstack/cinder: Fixing regression introduced in review 326685  https://review.openstack.org/32957417:12
patrickeastSrinivas: can you check what driver is being used for the device when you run the attach-device command? and show the output of something like 'dmesg | grep -i iommu'17:13
SrinivasPatrickeast: how can i overcome that issue17:14
*** wanghao has quit IRC17:14
SrinivasPatrickeast:  iam using libvirt driver17:15
*** wanghao has joined #openstack-cinder17:16
*** ducttape_ has quit IRC17:17
*** chris_morrell has joined #openstack-cinder17:17
patrickeastSrinivas: not the nova driver, the actual pci driver17:18
*** mvk has joined #openstack-cinder17:18
patrickeastSrinivas: look at the output of something like lspci -vvv17:18
patrickeastSrinivas: it should be something like vfio-pci17:19
*** geguileo_phone has quit IRC17:20
*** yangyapeng has quit IRC17:20
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: WIP fix grenade upgrade tests  https://review.openstack.org/32958617:20
hemnasdague, smcginnis jgriffith ^^17:20
*** yangyapeng has joined #openstack-cinder17:20
hemnano unit tests, but I just got that working and thought I'd throw it up there17:21
hemnaand see if folks agree if that's the right way to go17:21
hemnaDuncanT, ^^17:21
SrinivasPatrickeast: it is showing  vfio-pci17:22
*** lixiaoy1 has quit IRC17:22
patrickeastSrinivas: ok good, so now check your syslog/messages and qemu log when you do the attach-device and see if there is any additional output to hint at why it failed17:23
patrickeastSrinivas: also please paste the output of 'dmesg | grep -e DMAR -e IOMMU'17:23
patrickeastSrinivas: unless you are on an AMD CPU, in which case its different..17:24
*** lixiaoy1 has joined #openstack-cinder17:24
*** salv-orlando has quit IRC17:26
*** salv-orlando has joined #openstack-cinder17:26
SrinivasPatrickeast: cpu is intel xenon not the amd cpu17:27
patrickeastSrinivas: ok, and the output of that command?17:27
*** ducttape_ has joined #openstack-cinder17:28
*** geguileo_phone has joined #openstack-cinder17:28
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: WIP fix grenade upgrade tests  https://review.openstack.org/32958617:29
*** salv-orl_ has joined #openstack-cinder17:29
*** The_Ball has quit IRC17:31
asselin_anyone know where the cinder run <ci name> recheck is documented?17:32
*** salv-orlando has quit IRC17:32
smcginnisasselin_: Some (not all) are documented here: https://wiki.openstack.org/wiki/ThirdPartySystems17:33
smcginnisasselin_: We decided in a meeting that it should be "run-[CI Name]", but not everyone has adopted that yet.17:33
asselin_smcginnis, is it documented on any wiki page? I remember the meeting decision....17:34
smcginnisasselin_: Found it... https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#How_do_I_trigger_my_CI_to_rerun_on_gerrit_comments.3F17:35
asselin_cool thanks!17:35
smcginnisasselin_: Any time17:35
*** timcl1 has joined #openstack-cinder17:36
*** jwcroppe has joined #openstack-cinder17:37
*** timcl has quit IRC17:37
*** mriedem1 has joined #openstack-cinder17:38
*** jwcroppe_ has quit IRC17:39
*** Srinivas has quit IRC17:40
*** mriedem has quit IRC17:40
*** JoseMello has joined #openstack-cinder17:40
*** e0ne has quit IRC17:41
*** jdillaman has joined #openstack-cinder17:41
*** jwcroppe_ has joined #openstack-cinder17:41
*** mriedem1 is now known as mriedem17:42
*** jwcroppe has quit IRC17:42
*** The_Ball has joined #openstack-cinder17:44
*** asselin_ has quit IRC17:44
*** ociuhandu has quit IRC17:45
*** kfarr has joined #openstack-cinder17:45
*** ihrachys has quit IRC17:46
*** mtanino has quit IRC17:51
*** jwcroppe has joined #openstack-cinder17:53
*** GB21 has quit IRC17:53
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring  https://review.openstack.org/28546917:53
*** faiz89 has quit IRC17:55
*** jwcroppe_ has quit IRC17:55
*** faiz89 has joined #openstack-cinder17:57
*** tyrola has quit IRC18:02
*** bardia has joined #openstack-cinder18:03
*** faiz89 has quit IRC18:06
*** mtanino has joined #openstack-cinder18:08
*** salv-orl_ has quit IRC18:10
*** salv-orlando has joined #openstack-cinder18:11
*** salv-orlando has quit IRC18:11
*** salv-orlando has joined #openstack-cinder18:12
*** raunak has quit IRC18:12
*** chris_morrell has quit IRC18:13
*** mvk_ has joined #openstack-cinder18:14
*** geguileo_phone has quit IRC18:16
*** geguileo_phone has joined #openstack-cinder18:16
*** mvk has quit IRC18:17
*** zhongjun_ has quit IRC18:21
*** zhongjun_ has joined #openstack-cinder18:21
*** raunak has joined #openstack-cinder18:24
*** lpetrut has joined #openstack-cinder18:26
*** raunak has quit IRC18:26
*** raunak has joined #openstack-cinder18:26
xinligeguileor: one question regarding your feature, https://github.com/openstack/cinder/commit/504b466295f3d97307216095f450ff842179631918:28
*** harlowja has quit IRC18:30
xinligeguileor: After removing the decorator @remotable and @remotable_classmethod, one of my method get_by_id in now code test_qos.py failed due to not find the method. Any idea?18:30
*** rlrossit has quit IRC18:31
*** harlowja has joined #openstack-cinder18:32
*** bardia has quit IRC18:32
smcginnisxinli: Did you change that to just @classmethod?18:33
xinlismcginnis: yes, I modified all the method which marked with @remote-classmethod to @classmethod, the basic tests passed all18:34
*** e0ne has joined #openstack-cinder18:35
xinlismcginnis: I compare both test_backup.py and test_qos.py, they are the same in the test_get_by_id(), and I also compare ../objects/backup.py and ../objects/qos_specs.py, both have inherited the get_by_id from base.PersistenObject, but that method has been modified to @classmethod.18:37
xinlismcginnis: the error message : http://paste.openstack.org/show/516049/18:38
*** geguileo_phone has quit IRC18:43
*** salv-orlando has quit IRC18:45
*** akerr has quit IRC18:46
*** sdague has quit IRC18:46
*** salv-orlando has joined #openstack-cinder18:46
*** lifeless has quit IRC18:46
*** lifeless has joined #openstack-cinder18:47
*** geguileo_phone has joined #openstack-cinder18:52
*** e0ne has quit IRC18:53
*** geguileo_phone has quit IRC18:57
*** geguileo_phone has joined #openstack-cinder18:58
*** darosale has quit IRC19:01
*** akapil has joined #openstack-cinder19:03
*** raunak has quit IRC19:06
*** sdake has quit IRC19:06
*** sdake has joined #openstack-cinder19:07
*** akapil has quit IRC19:08
openstackgerritXinli Guan proposed openstack/cinder: Move QualityOfServiceSpecs to use VersionedObject  https://review.openstack.org/25198919:11
xinlismcginnis: I have fixed the code and posted in the Gerrit.19:13
openstackgerritMatt Smith proposed openstack/cinder: Netmask needs corersion to string  https://review.openstack.org/32957419:14
*** akerr has joined #openstack-cinder19:17
*** wN has quit IRC19:21
*** jwcroppe has quit IRC19:22
*** wN has joined #openstack-cinder19:22
*** wN has quit IRC19:22
*** wN has joined #openstack-cinder19:22
*** rlrossit has joined #openstack-cinder19:29
*** ociuhandu has joined #openstack-cinder19:29
*** mriedem has quit IRC19:31
*** mriedem has joined #openstack-cinder19:34
*** faiz89 has joined #openstack-cinder19:34
openstackgerritAlyson proposed openstack/cinder: HNAS: Add support for consistency groups in iSCSI driver  https://review.openstack.org/32704319:34
*** tyrola has joined #openstack-cinder19:40
*** Lee1092 has quit IRC19:43
*** salv-orlando has quit IRC19:43
openstackgerritMerged openstack/cinder: Tests: Use mock for notifier tests  https://review.openstack.org/31705519:44
*** salv-orlando has joined #openstack-cinder19:44
*** delattec has quit IRC19:45
*** harlowja_ has joined #openstack-cinder19:45
openstackgerritMerged openstack/cinder: Tests: Remove notification_driver config  https://review.openstack.org/32906419:46
adrianofreharney: dulek: I've addressed your comments on https://review.openstack.org/#/c/294076/ . Please, validate it when you have some time. Thanks :)19:48
*** lprice has quit IRC19:49
*** harlowja has quit IRC19:49
openstackgerritMerged openstack/cinder: image_size_m should get ceiling of image  https://review.openstack.org/32926219:49
*** raunak has joined #openstack-cinder19:50
*** tyrola has quit IRC19:52
*** sdague has joined #openstack-cinder19:56
*** porrua has quit IRC19:57
*** e0ne has joined #openstack-cinder19:58
scottdaIf you are running multi-backend in your 3rd party CI, please have a look at https://review.openstack.org/#/c/329178/2/devstack-vm-gate.sh20:00
scottdaPlease let me know who you are setting up your multi-backend, and if that patch will break you.20:00
scottda"let me know who you are setting up..." -> "let me know how you are setting up..."20:02
*** tyrola has joined #openstack-cinder20:02
*** ducttape_ has quit IRC20:07
openstackgerritxing-yang proposed openstack/cinder: Add group type and group specs  https://review.openstack.org/32016520:07
openstackgerritxing-yang proposed openstack/cinder: WIP: Add group snapshots  https://review.openstack.org/32805220:07
openstackgerritxing-yang proposed openstack/cinder: Add generic volume groups  https://review.openstack.org/32245920:07
*** akerr has quit IRC20:07
*** tyrola has quit IRC20:09
*** cdelatte has joined #openstack-cinder20:12
*** rlrossit has quit IRC20:12
*** jgregor has joined #openstack-cinder20:14
*** ducttape_ has joined #openstack-cinder20:16
openstackgerritTom Swanson proposed openstack/cinder: WIP: Dell SC: Use Live Volume for replication  https://review.openstack.org/32953120:16
*** ducttape_ has quit IRC20:16
*** ducttape_ has joined #openstack-cinder20:16
*** geguileo_phone has quit IRC20:18
*** geguileo_phone has joined #openstack-cinder20:26
*** diogogmt has joined #openstack-cinder20:27
*** mriedem1 has joined #openstack-cinder20:27
*** timcl1 has quit IRC20:27
*** e0ne has quit IRC20:29
*** jgregor has quit IRC20:30
*** mriedem has quit IRC20:30
*** andrei____ has joined #openstack-cinder20:30
andrei____hello20:30
andrei____is the deadline for the new driver contribution be the 13 of july?20:31
andrei____thank you20:31
*** mriedem1 is now known as mriedem20:32
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file  https://review.openstack.org/29407620:34
andrei____deadline for Newton20:35
andrei____?20:35
*** rlrossit has joined #openstack-cinder20:36
*** gouthamr has quit IRC20:36
Swansonandrei____, http://releases.openstack.org/newton/schedule.html20:36
smcginnisandrei____: Basically midnight US west coast on 12th of July.20:37
*** geguileo_phone has quit IRC20:39
andrei____thank you.20:39
*** mkrcmari__ has joined #openstack-cinder20:39
andrei____Is it possible to be able to contribute new driver in time?20:39
andrei____I am about code review20:40
Swansonandrei____, Do you have a CI?20:40
*** jungleboyj has quit IRC20:41
_alastor_andrei____: From my experience, if you have a CI set up you can get through the Gerrit code review process in 2-5 days depending on how persistent and responsive you are20:41
andrei____CI like environment for testing?20:41
andrei____http://docs.openstack.org/infra/system-config/test-infra-requirements.html like this one?20:42
_alastor_andrei____: A CI reporting here: ci-watch.tintri.com/project?project=cinder&time=24+hours20:42
*** geguileo_phone has joined #openstack-cinder20:42
_alastor_andrei____: You can read more here: https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers20:42
smcginnisandrei____: And here https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver20:42
*** mvk_ has quit IRC20:43
andrei____thank you, I am coming here from this pages20:43
*** diogogmt has quit IRC20:43
*** jgregor has joined #openstack-cinder20:44
andrei____can you please give me a link of how to setup CI?20:45
*** mvk_ has joined #openstack-cinder20:45
*** dustins has quit IRC20:45
openstackgerritMerged openstack/cinder: Windows SMBFS: fix VHD/x resize  https://review.openstack.org/32900020:45
_alastor_andrei____: http://www.joinfu.com/2014/02/setting-up-an-external-openstack-testing-system/20:46
smcginnisandrei____: That ones a bit out of date.20:46
smcginnisBut that will give you the background on it.20:46
andrei____\(^-^)/20:46
andrei____thank you20:46
smcginnisandrei____: More info linked from the earlier links above - http://docs.openstack.org/infra/system-config/third_party.html20:46
_alastor_smcginnis: It's the one currently listed in the third-party driver wiki20:46
_alastor_smcginnis: Is there a more up to date one?20:47
*** raildo is now known as raildo-afk20:47
*** cknight has quit IRC20:47
smcginnis_alastor_: Yeah... somewhere. :)20:47
*** diogogmt has joined #openstack-cinder20:48
*** mkrcmari__ has quit IRC20:48
*** edmondsw has quit IRC20:52
*** kfarr has quit IRC20:53
andrei____thank you again!20:54
openstackgerritxing-yang proposed openstack/python-cinderclient: Add generic volume groups  https://review.openstack.org/32262720:58
openstackgerritxing-yang proposed openstack/python-cinderclient: Add group types and group specs  https://review.openstack.org/32015720:58
*** mvk has joined #openstack-cinder20:58
*** akshai has joined #openstack-cinder20:59
*** mvk_ has quit IRC20:59
*** eharney has quit IRC21:01
*** crose has joined #openstack-cinder21:02
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/32967721:04
*** faiz89 has quit IRC21:05
*** e0ne has joined #openstack-cinder21:06
*** faiz89 has joined #openstack-cinder21:08
*** gouthamr has joined #openstack-cinder21:08
openstackgerritOpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext: Updated from global requirements  https://review.openstack.org/32968021:10
*** pauloewerton has quit IRC21:11
*** julim has quit IRC21:12
*** geguileo_phone has quit IRC21:14
*** crose has quit IRC21:15
jbernarddid we get rid of the clone_image() call?  i see implementations for it in some drivers, but I don't see anywhere in the manager that calls this function21:15
smcginnisreduxio: ping21:16
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Fix improperly configured privsep  https://review.openstack.org/32958621:17
hemnalets hope that does it21:17
asselinandrei____, here's the most recent link: http://docs.openstack.org/infra/openstackci/third_party_ci.html21:18
e0nejbernard: AFAIR, we call it from create volume flow21:18
asselin_alastor_ that link is a good read, but many parts are  outdated. http://docs.openstack.org/infra/openstackci/third_party_ci.html provides updated instructinos21:18
e0nejbernard: cinder/volume/flows/manager/create_volume.py21:19
jbernarde0ne: ahh, you're right21:19
_alastor_asselin: Thanks, I'll update the links on the Cinder wiki21:19
jbernarde0ne: i failed to see taht, thanks21:19
e0nejbernard: np21:19
e0nejbernard: I spent about 20 minutes to find it first time21:20
e0nelooks like I was useful today and I can try to sleep :)21:20
e0nebb tomorrow21:21
jbernarde0ne: lol, cheers21:21
e0ne:)21:21
openstackgerritxing-yang proposed openstack/cinder: WIP: Add group snapshots  https://review.openstack.org/32805221:21
e0nejbernard: btw, do you have any update with multipath issue?21:22
jbernarde0ne: i haven't had a chance yet, ive been busy on other tasks, but it's in my queue (if that helps)21:23
jbernarde0ne: you found a workaround?21:23
e0nejbernard: yes, I've stopped multipathd and it's working now21:24
e0nejbernard: but we need to clarify where is the issue: multipath-utils or rbd21:24
e0nejbernard: I didn't find any docs about multipath and rbd map21:25
e0ne:(21:25
jbernarde0ne: bummer, an opportunity to create some perhaps ;)21:26
e0ne:)21:26
e0nejbernard: added to my todo list:)21:26
jbernarde0ne: your workaround cant be deployed, i would assume; if another backend support multipath - then stopping the daemon would be... not good21:27
e0nejbernard: there is another workaround21:27
e0nejbernard: add /dev/rbd to ignore list for multipathd21:27
e0neit also works21:27
jbernarde0ne: ahh, interesting21:27
hemnasmcginnis, https://review.openstack.org/#/c/297140/21:28
hemnaso....are we going to go through each of the driver CI and make sure this patch isn't breaking each one?  There are a bunch of failures.21:28
e0nejbernard: http://paste.openstack.org/show/516064/21:29
*** leeantho has quit IRC21:29
smcginnishemna: I'll take a look at those. I believe they passed in past revs.21:31
e0nejbernard: but I still hope to get it configurable w/o hacks with multipathd21:31
smcginnishemna: Theoretically shouldn't be able to break anything, but decorators are a bit tricky sometimes when combined.21:32
smcginnishemna: Yeah, all appear to be CI or infrastructure errors or similar non-related issues.21:34
smcginnishemna: At least the ones that actually have their logs accessible. :/21:34
hemnadoes it make sense to break that patch up ?21:35
*** faiz89 has quit IRC21:35
hemnathis is one of those cases where I'd like to see all of the touched driver CIs passing before approving21:35
smcginnishemna: I did want to do that, but not sure how to easily do so.21:35
hemnaI figured it might be painful21:35
smcginnisI probably could. Some wouldn't show to much value, but as long as it's clear it's working toward a bigger solution I think that should be good.21:36
hemnaI like the idea and the patch21:36
smcginnisAwesome21:36
hemnaI'm not sure what the value of the ABC stuff would be after that21:36
smcginnishemna: Yeah, that would be on the list then to rework what we have right now with the ABC stuff.21:37
smcginnishemna: I know we've talked a lot in the past about redoinng that.21:37
smcginnishemna: Hopefully this would make that easier to refactor to something more useful.21:37
hemnayup21:37
*** jwcroppe has joined #openstack-cinder21:38
*** jungleboyj has joined #openstack-cinder21:39
*** cdelatte has quit IRC21:39
*** geguileo_phone has joined #openstack-cinder21:40
*** jgregor has quit IRC21:41
*** e0ne has quit IRC21:42
*** andrei____ has quit IRC21:48
*** sileht has quit IRC21:51
*** lprice has joined #openstack-cinder21:51
*** sileht has joined #openstack-cinder21:52
*** frontrunner has quit IRC21:53
*** gouthamr_ has joined #openstack-cinder21:58
smcginnisjgriffith: :)21:59
*** lprice1 has joined #openstack-cinder22:00
*** gouthamr has quit IRC22:01
*** lprice has quit IRC22:01
*** xinli has quit IRC22:06
*** rlrossit has quit IRC22:10
*** mriedem is now known as mriedem_away22:12
gussdague/hemna: So how urgent is solving the privsep upgrade thing (wrt release cycle plans)?22:14
*** akshai has quit IRC22:16
gus(just juggling how I slot this in with other work)22:16
*** akshai has joined #openstack-cinder22:17
smcginnisgus: As it is now, we can't include privsep usage in Newton until we sort out how upgrades are supposed to work.22:17
smcginnisgus: We had to block the os-brick release that included it because of grenade test failures.22:18
smcginnisgus: hemna has a patch up for os-brick. I'd love to have your thoughts on that.22:18
smcginnisgus: https://review.openstack.org/#/c/329586/22:18
gussmcginnis: ack.  I'll have a look at it right after I get some breakfast.22:19
smcginnisgus: Haha, definitely breakfast is higher priority. ;)22:19
gussmcginnis: and I should be able to code up the (3) approach suggested on the thread today - but it will require a new oslo.privsep release + global-requirements + review round trip through nova/cinder.22:20
*** jwcroppe has quit IRC22:20
smcginnisgus: Awesome. That may be the best answer given others using privsep.22:21
*** openstackstatus has quit IRC22:25
*** openstack has joined #openstack-cinder22:28
*** rcernin has joined #openstack-cinder22:28
*** catintheroof has quit IRC22:30
*** jungleboyj has quit IRC22:30
*** mtanino has quit IRC22:38
*** mtanino has joined #openstack-cinder22:43
*** ducttape_ has quit IRC22:43
hemnaback22:45
hemnagus ok cool.   take a look at my patch if you can, maybe we can coordinate the fixes22:46
hemnaas it is, my patch works for os-brick in the short term22:46
hemnawe could push out a 1.4.1 with this and then change it later once we have a new API in privsep22:47
hemnasmcginnis, also, shouldn't we add more tests to os-brick patches?22:51
hemnasay....grenade-dsvm ?22:52
hemna:)22:52
*** leeantho has joined #openstack-cinder22:52
*** leeantho_ has joined #openstack-cinder22:52
*** gouthamr_ has quit IRC22:54
*** gouthamr has joined #openstack-cinder22:55
*** tpsilva has quit IRC22:57
*** mvk_ has joined #openstack-cinder22:58
*** laughterwym has joined #openstack-cinder22:59
*** JoseMello has quit IRC22:59
*** mvk has quit IRC23:01
*** leeantho_ has quit IRC23:02
*** laughterwym has quit IRC23:03
*** wN has quit IRC23:07
*** alyson_ has quit IRC23:08
*** wN has joined #openstack-cinder23:10
*** wN has quit IRC23:10
*** wN has joined #openstack-cinder23:10
*** Vasanthi has joined #openstack-cinder23:11
*** salv-orlando has quit IRC23:14
VasanthiHello - I am working on a branch for a bug and have already submitted some patches to my original review. Before I fixed the previous round of review, I merged from master to bring in the latest changes form master into my branch and contnued with the fixes suggested in teh review. Now my commit comment says this "    Merge branch 'master' of https://git.openstack.org/openstack/cinder into bp/vmem-7000-series-iscsi-drive23:14
*** salv-orlando has joined #openstack-cinder23:14
Vasanthiand modify the comment to reflect the original change and send it out for review?23:15
*** wN has quit IRC23:15
*** Vasanthi has quit IRC23:16
*** Vasanthi has joined #openstack-cinder23:16
VasanthiHello - I am working on a branch for a bug and have already submitted some patches to my original review. Before I fixed the previous round of review, I merged from master to bring in the latest changes form master into my branch and contnued with the fixes suggested in teh review. Now my commit comment says this "    Merge branch 'master' of https://git.openstack.org/openstack/cinder into bp/vmem-7000-series-iscsi-drive23:16
Vasanthiand modify the comment to reflect the original change and send it out for review?23:17
*** karthikp_ has quit IRC23:18
*** wN has joined #openstack-cinder23:19
*** wN has quit IRC23:19
*** wN has joined #openstack-cinder23:19
*** lpetrut has quit IRC23:22
*** akshai has quit IRC23:23
openstackgerritMerged openstack/cinder: Fixed _create_3par_iscsi_host(), added unit test for 3PAR  https://review.openstack.org/31776323:23
*** salv-orl_ has joined #openstack-cinder23:29
*** yamada-h has joined #openstack-cinder23:30
*** salv-orlando has quit IRC23:32
*** sdake has quit IRC23:32
*** sdake has joined #openstack-cinder23:34
*** yamada-h has quit IRC23:34
*** rcernin has quit IRC23:40
*** asselin_ has joined #openstack-cinder23:40
*** asselin_ has quit IRC23:45
*** yhayashi has joined #openstack-cinder23:52
*** sdague has quit IRC23:53
*** cknight has joined #openstack-cinder23:53
*** xyang1 has quit IRC23:56

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