Wednesday, 2016-06-01

*** cknight has joined #openstack-cinder00:12
*** dims has quit IRC00:14
*** merooney has joined #openstack-cinder00:16
*** cknight1 has joined #openstack-cinder00:16
*** cknight has quit IRC00:18
*** dims has joined #openstack-cinder00:19
*** garthb_ has quit IRC00:23
*** laughterwym has joined #openstack-cinder00:27
*** akshai has joined #openstack-cinder00:28
*** akshai_ has joined #openstack-cinder00:29
*** sdake_ has joined #openstack-cinder00:29
*** sdake has quit IRC00:30
*** akshai has quit IRC00:32
*** laughterwym has quit IRC00:33
*** laughterwym has joined #openstack-cinder00:33
openstackgerritJay Mehta proposed openstack/cinder: Fixed _create_3par_iscsi_host(), added unit test for 3PAR  https://review.openstack.org/31776300:34
*** ducttape_ has quit IRC00:35
*** ducttape_ has joined #openstack-cinder00:35
*** laughterwym has quit IRC00:36
*** laughterwym has joined #openstack-cinder00:36
*** suyash has quit IRC00:36
*** ducttape_ has quit IRC00:38
*** ducttape_ has joined #openstack-cinder00:38
*** akshai_ has quit IRC00:40
*** jwcroppe has quit IRC00:41
*** mtanino has quit IRC00:41
*** jwcroppe has joined #openstack-cinder00:50
*** cheneydc has joined #openstack-cinder00:56
*** sdake has joined #openstack-cinder00:56
*** Apoorva has quit IRC00:59
*** Julien-zte has joined #openstack-cinder00:59
*** gouthamr has joined #openstack-cinder00:59
*** sdake_ has quit IRC00:59
*** Julien-zte has quit IRC01:00
*** sdake has quit IRC01:02
*** Julien-zte has joined #openstack-cinder01:02
*** ducttape_ has quit IRC01:04
*** darosale has joined #openstack-cinder01:06
*** raunak has quit IRC01:06
*** karthikp has quit IRC01:17
*** karthikp has joined #openstack-cinder01:17
*** yangyapeng has joined #openstack-cinder01:19
*** yangyapeng has quit IRC01:20
*** DCWilliams_VA has joined #openstack-cinder01:20
*** yangyapeng has joined #openstack-cinder01:20
*** yangyapeng has quit IRC01:20
*** DCWilliams_VA is now known as Guest9945301:20
*** yangyapeng has joined #openstack-cinder01:21
*** ducttape_ has joined #openstack-cinder01:21
*** Guest99453 is now known as dcwilliams29id01:22
*** dcwilliams29id is now known as dcwilliamsva01:23
*** julim has joined #openstack-cinder01:24
*** dcwilliamsva has quit IRC01:29
*** julim has quit IRC01:32
*** EinstCrazy has joined #openstack-cinder01:33
*** ducttape_ has quit IRC01:37
*** Lee1092 has joined #openstack-cinder01:37
*** sdake has joined #openstack-cinder01:41
*** bardia has quit IRC01:43
*** sdake has quit IRC01:43
*** sdake has joined #openstack-cinder01:47
*** sdake_ has joined #openstack-cinder01:48
openstackgerritMerged openstack/cinder: Coho Data: Reconnect to cluster on broken pipe  https://review.openstack.org/31838201:50
*** sdake has quit IRC01:51
*** laughterwym has quit IRC02:00
*** laughter_ has joined #openstack-cinder02:03
*** jdurgin has quit IRC02:04
*** julim has joined #openstack-cinder02:10
*** hogepodge has quit IRC02:13
*** hogepodge has joined #openstack-cinder02:17
openstackgerrityuyafei proposed openstack/python-cinderclient: unit.test_base.BaseTest.test_eq does not work  https://review.openstack.org/32240802:19
openstackgerrityuyafei proposed openstack/python-cinderclient: unit.test_base.BaseTest.test_eq does not work  https://review.openstack.org/32240802:19
*** DCWilliams_VA has joined #openstack-cinder02:21
*** DCWilliams_VA is now known as Guest3530702:21
*** hogepodge has quit IRC02:21
*** ducttape_ has joined #openstack-cinder02:22
*** laughter_ has quit IRC02:27
*** laughterwym has joined #openstack-cinder02:28
*** laughterwym has quit IRC02:33
*** ducttape_ has quit IRC02:34
*** darosale has quit IRC02:35
*** sheel has joined #openstack-cinder02:51
*** laughterwym has joined #openstack-cinder02:51
*** Guest35307 has quit IRC03:04
*** laughterwym has quit IRC03:05
*** laughterwym has joined #openstack-cinder03:06
*** laughterwym has joined #openstack-cinder03:08
*** martyturner has joined #openstack-cinder03:09
*** ducttape_ has joined #openstack-cinder03:11
*** DCWilliams_VA has joined #openstack-cinder03:12
*** DCWilliams_VA is now known as Guest5755203:13
*** bardia has joined #openstack-cinder03:14
*** martyturner has quit IRC03:14
openstackgerritxing-yang proposed openstack/cinder: Fix backup using temp snapshot code path  https://review.openstack.org/32194303:19
*** Guest57552 has quit IRC03:23
*** ducttape_ has quit IRC03:27
*** ebalduf has joined #openstack-cinder03:29
*** houming has joined #openstack-cinder03:32
*** merooney has quit IRC03:33
openstackgerritWeiwei Gu proposed openstack/cinder: LVM+tgt:iscsi target deleted after VM live migrate  https://review.openstack.org/31455903:36
*** gouthamr has quit IRC03:37
openstackgerritxing-yang proposed openstack/cinder: model_update for temp volume or snapshot in backup  https://review.openstack.org/32229403:38
openstackgerritWeiwei Gu proposed openstack/cinder: LVM+tgt:iscsi target deleted after VM live migrate  https://review.openstack.org/31455903:41
*** bardia has quit IRC03:42
openstackgerrithaobing proposed openstack/cinder: The LVM backend Consistency Group Support  https://review.openstack.org/32364903:44
*** dave-mccowan has quit IRC03:48
*** ducttape_ has joined #openstack-cinder03:49
*** chip__ has quit IRC03:51
*** chip__ has joined #openstack-cinder03:53
*** karthikp has quit IRC03:56
*** links has joined #openstack-cinder04:00
*** sgotliv has joined #openstack-cinder04:03
*** ducttape_ has quit IRC04:04
*** jamielennox is now known as jamielennox|away04:08
*** Julien-zte has quit IRC04:09
*** martyturner has joined #openstack-cinder04:12
*** cknight1 has quit IRC04:15
*** cknight has joined #openstack-cinder04:15
openstackgerritEdwin Wang proposed openstack/cinder: ABC Metaclass for IBM FlashSystem  https://review.openstack.org/32334304:16
*** wanghao has quit IRC04:20
*** david-lyle has quit IRC04:21
*** wanghao has joined #openstack-cinder04:21
*** raunak has joined #openstack-cinder04:23
*** cknight has quit IRC04:27
*** darosale has joined #openstack-cinder04:34
*** liewegas has quit IRC04:35
*** sage has joined #openstack-cinder04:36
*** laughter_ has joined #openstack-cinder04:38
*** martyturner has quit IRC04:38
openstackgerrityuyafei proposed openstack/python-cinderclient: self.assertNotEqual does not work  https://review.openstack.org/32240804:39
*** liverpooler has quit IRC04:42
*** laughterwym has quit IRC04:42
*** pgadiya has joined #openstack-cinder04:53
*** EinstCrazy has quit IRC05:00
*** yangyapeng has quit IRC05:02
*** rcernin has quit IRC05:04
*** GB21 has joined #openstack-cinder05:04
*** ducttape_ has joined #openstack-cinder05:05
*** karthikp has joined #openstack-cinder05:05
openstackgerritParashuram Hallur proposed openstack/cinder: Implementation for CoprHD Cinder Drivers.  https://review.openstack.org/31764305:08
*** EinstCrazy has joined #openstack-cinder05:09
*** ducttape_ has quit IRC05:11
*** Poornima has joined #openstack-cinder05:11
openstackgerritLisaLi proposed openstack/cinder-specs: Retype encrypted volumes  https://review.openstack.org/24859305:13
*** karthikp has quit IRC05:19
*** ChubYann has quit IRC05:21
*** chip__ has quit IRC05:22
*** raunak has quit IRC05:22
*** chip__ has joined #openstack-cinder05:23
*** mtanino has joined #openstack-cinder05:25
*** laughter_ has quit IRC05:28
*** laughterwym has joined #openstack-cinder05:28
*** mtanino has quit IRC05:31
*** bardia has joined #openstack-cinder05:33
*** lpetrut has joined #openstack-cinder05:35
*** Julien-zte has joined #openstack-cinder05:39
*** jsheeren has joined #openstack-cinder05:40
*** rcernin has joined #openstack-cinder05:43
*** nkrinner has joined #openstack-cinder05:49
*** sdake has joined #openstack-cinder05:52
*** raunak has joined #openstack-cinder05:53
*** chip__ has quit IRC05:54
*** sdake_ has quit IRC05:55
*** chip__ has joined #openstack-cinder05:55
*** darosale has quit IRC05:58
*** chip__ has quit IRC05:59
*** mvk_ has joined #openstack-cinder06:00
*** chip__ has joined #openstack-cinder06:01
*** yangyapeng has joined #openstack-cinder06:01
*** sgotliv has quit IRC06:01
*** yangyape_ has joined #openstack-cinder06:01
*** mvk has quit IRC06:03
*** yangyapeng has quit IRC06:06
*** yangyape_ has quit IRC06:10
*** sdake has quit IRC06:15
*** belmoreira has joined #openstack-cinder06:17
openstackgerritWeiwei Gu proposed openstack/cinder: LVM+tgt:iscsi target deleted after VM live migrate  https://review.openstack.org/31455906:17
*** wanghao has quit IRC06:17
*** mjura has joined #openstack-cinder06:18
*** wanghao has joined #openstack-cinder06:19
*** markvoelker has quit IRC06:24
*** chip__ has quit IRC06:24
*** lpetrut has quit IRC06:25
*** EinstCrazy has quit IRC06:25
*** mjura has quit IRC06:27
*** bswartz has quit IRC06:27
*** vbala has joined #openstack-cinder06:29
*** hogepodge has joined #openstack-cinder06:31
*** yangyapeng has joined #openstack-cinder06:32
*** mjura has joined #openstack-cinder06:39
*** EinstCrazy has joined #openstack-cinder06:42
*** Julien-zte has quit IRC06:45
*** liverpooler has joined #openstack-cinder06:47
*** lpetrut has joined #openstack-cinder06:48
*** jwcroppe has quit IRC06:51
*** bardia has quit IRC06:52
*** savihou has joined #openstack-cinder07:02
*** tesseract has joined #openstack-cinder07:03
*** bardia has joined #openstack-cinder07:05
*** ducttape_ has joined #openstack-cinder07:07
*** ducttape_ has quit IRC07:11
*** arecknag has joined #openstack-cinder07:15
*** lpetrut has quit IRC07:17
*** bardia has quit IRC07:21
*** DCWilliams_VA has joined #openstack-cinder07:25
*** DCWilliams_VA is now known as Guest7510107:26
*** Guest75101 has quit IRC07:30
*** e0ne has joined #openstack-cinder07:32
*** yangyapeng has quit IRC07:34
*** yangyape_ has joined #openstack-cinder07:34
openstackgerrityuyafei proposed openstack/cinder: Huawei driver should return ceiling of volume size  https://review.openstack.org/32369907:48
*** andymaier has joined #openstack-cinder07:52
*** jwcroppe has joined #openstack-cinder07:56
*** raunak has quit IRC07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:01
*** xek has joined #openstack-cinder08:01
*** jwcroppe has quit IRC08:04
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
*** cristicalin has joined #openstack-cinder08:10
*** sgotliv has joined #openstack-cinder08:19
openstackgerritLisaLi proposed openstack/cinder: Handle quota exceed exception  https://review.openstack.org/28166608:22
*** mkoderer has quit IRC08:22
*** mkoderer has joined #openstack-cinder08:26
*** jordanP has joined #openstack-cinder08:27
*** jordanP has quit IRC08:27
*** jordan_ has joined #openstack-cinder08:27
*** markvoelker has joined #openstack-cinder08:37
*** yuriy_n17 has joined #openstack-cinder08:38
*** markvoelker has quit IRC08:42
*** yhayashi has quit IRC08:42
*** laughterwym has quit IRC08:44
*** laughterwym has joined #openstack-cinder08:45
*** Lallana has joined #openstack-cinder08:49
*** laughterwym has quit IRC08:50
*** lpetrut has joined #openstack-cinder09:00
*** Guest20402 has joined #openstack-cinder09:04
*** Guest20402 is now known as wilsonL09:04
*** Poornima has quit IRC09:06
*** GB21 has quit IRC09:06
*** Poornima has joined #openstack-cinder09:09
*** wN has quit IRC09:10
*** wN has joined #openstack-cinder09:11
*** wN has quit IRC09:11
*** wN has joined #openstack-cinder09:11
*** mvk_ has quit IRC09:14
*** bharathk has joined #openstack-cinder09:16
openstackgerrityuyafei proposed openstack/cinder: Add unit test case in Nimble Driver  https://review.openstack.org/32374809:17
openstackgerritWeiwei Gu proposed openstack/python-cinderclient: add reset-state API for consistency groups  https://review.openstack.org/32074809:26
openstackgerritYuriy Nesenenko proposed openstack/cinder: Fix quota-usage list information with invalid tenant-id  https://review.openstack.org/31567309:29
*** mvk has joined #openstack-cinder09:32
openstackgerrityuyafei proposed openstack/cinder: SolidFire driver should return ceiling of volume size  https://review.openstack.org/32319109:33
*** jamielennox|away is now known as jamielennox09:44
openstackgerritWeiwei Gu proposed openstack/cinder: LVM+tgt:iscsi target deleted after VM live migrate  https://review.openstack.org/31455909:44
*** cristicalin has quit IRC09:46
openstackgerritWeiwei Gu proposed openstack/cinder: LVM+tgt:iscsi target deleted after VM live migrate  https://review.openstack.org/31455909:48
*** jsheeren has quit IRC09:48
*** julim has quit IRC09:55
*** laughterwym has joined #openstack-cinder09:58
openstackgerritYuriy Nesenenko proposed openstack/cinder: Image virtual size doesn't fit to volume size  https://review.openstack.org/31115210:00
*** belmoreira has quit IRC10:00
*** yangyapeng has joined #openstack-cinder10:01
*** yangyapeng has quit IRC10:01
*** cheneydc has quit IRC10:02
*** laughterwym has quit IRC10:02
*** julim has joined #openstack-cinder10:03
*** EinstCrazy has quit IRC10:03
*** yangyape_ has quit IRC10:04
*** laughterwym has joined #openstack-cinder10:10
*** jsheeren has joined #openstack-cinder10:18
*** laughterwym has quit IRC10:22
*** laughterwym has joined #openstack-cinder10:23
geguileoaorourke: ping - https://review.openstack.org/#/c/32068610:27
*** laughterwym has quit IRC10:27
openstackgerritLucian Petrut proposed openstack/os-brick: Add Windows iSCSI connector  https://review.openstack.org/27252210:29
openstackgerritLucian Petrut proposed openstack/os-brick: Add Windows Fibre Channel connector  https://review.openstack.org/32378010:29
openstackgerritLucian Petrut proposed openstack/os-brick: Add Windows SMBFS connector  https://review.openstack.org/32378110:29
*** laughterwym has joined #openstack-cinder10:32
*** huanan has quit IRC10:34
*** huanan has joined #openstack-cinder10:35
*** laughterwym has quit IRC10:44
*** sdague has joined #openstack-cinder10:47
*** laughterwym has joined #openstack-cinder10:48
*** The_Ball has quit IRC10:56
*** The_Ball has joined #openstack-cinder10:56
*** raildo-afk is now known as raildo11:15
openstackgerritMerged openstack/cinder: Huawei replication should use auto recovery policy  https://review.openstack.org/32202511:16
*** ducttape_ has joined #openstack-cinder11:26
*** dave-mccowan has joined #openstack-cinder11:27
openstackgerritMerged openstack/cinder: Fix 3PAR cg display name issues  https://review.openstack.org/30944311:29
*** bharathk has quit IRC11:30
*** links has quit IRC11:34
*** laughterwym has quit IRC11:38
*** timcl has joined #openstack-cinder11:39
*** lgreg has joined #openstack-cinder11:41
*** diablo_rojo has joined #openstack-cinder11:44
*** links has joined #openstack-cinder11:47
*** ducttape_ has quit IRC11:53
*** laughterwym has joined #openstack-cinder11:54
*** jwcroppe has joined #openstack-cinder12:00
*** vbala has quit IRC12:00
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376612:01
*** martyturner has joined #openstack-cinder12:03
*** martyturner has quit IRC12:03
openstackgerritzhangsong proposed openstack/cinder: Add lock when create tgt target  https://review.openstack.org/32345212:04
*** jwcroppe has quit IRC12:05
*** Lee1092 has quit IRC12:13
*** lgreg has quit IRC12:13
*** lgreg has joined #openstack-cinder12:14
*** wilsonL has quit IRC12:15
*** delattec has joined #openstack-cinder12:15
*** wilsonL has joined #openstack-cinder12:15
*** huanan_L has joined #openstack-cinder12:15
*** ducttape_ has joined #openstack-cinder12:16
*** huanan has quit IRC12:17
*** openstackgerrit has quit IRC12:18
*** cdelatte has quit IRC12:18
*** openstackgerrit has joined #openstack-cinder12:19
*** ducttape_ has quit IRC12:19
*** DCWilliams_VA has joined #openstack-cinder12:19
*** DCWilliams_VA is now known as Guest2973512:19
*** Guest29735 has quit IRC12:20
*** JoseMello has joined #openstack-cinder12:21
*** tpsilva has joined #openstack-cinder12:22
*** lgreg has quit IRC12:24
*** markvoelker has joined #openstack-cinder12:24
*** ociuhandu has quit IRC12:26
*** gouthamr has joined #openstack-cinder12:27
*** lgreg has joined #openstack-cinder12:29
*** _ducttape_ has joined #openstack-cinder12:29
*** dmanju has joined #openstack-cinder12:33
*** gcb has quit IRC12:35
*** eharney has joined #openstack-cinder12:36
*** huanan_L has quit IRC12:42
*** lgreg has quit IRC12:42
*** huanan has joined #openstack-cinder12:43
*** _ducttape_ has quit IRC12:44
*** ociuhandu has joined #openstack-cinder12:47
*** akshai has joined #openstack-cinder12:49
*** delatte has joined #openstack-cinder12:52
*** dave-mccowan has quit IRC12:52
*** lgreg has joined #openstack-cinder12:53
*** delattec has quit IRC12:55
*** ducttape_ has joined #openstack-cinder12:55
*** ducttape_ has quit IRC13:00
*** bswartz has joined #openstack-cinder13:01
*** merooney has joined #openstack-cinder13:01
*** ducttape_ has joined #openstack-cinder13:02
*** dmanju has quit IRC13:05
*** houming has quit IRC13:05
*** merooney has quit IRC13:06
*** ducttape_ has quit IRC13:06
*** xyang1 has joined #openstack-cinder13:06
*** ducttape_ has joined #openstack-cinder13:07
*** dave-mccowan has joined #openstack-cinder13:07
*** sdake has joined #openstack-cinder13:10
*** rlrossit has joined #openstack-cinder13:10
*** ducttape_ has quit IRC13:11
*** ducttape_ has joined #openstack-cinder13:12
*** porrua has joined #openstack-cinder13:14
*** diablo_rojo has quit IRC13:15
*** liverpooler has quit IRC13:19
*** merooney has joined #openstack-cinder13:20
*** nobody_ has joined #openstack-cinder13:21
nobody_hello13:21
nobody_some expert on netapp cinder backend managed to create encrypted volumes?13:22
nobody_they do work but they're not encrypted xD13:22
*** ducttape_ has quit IRC13:28
tbarronnobody_: probably try #openstack-netapp13:30
nobody_ok thanks :)13:30
openstackgerritMerged openstack/cinder: Remove API races from migrate and retype  https://review.openstack.org/22144213:32
lixiaoy1nobody_: did you create encrypted volumes in Cinder? If yes, I am working on it https://blueprints.launchpad.net/cinder/+spec/improve-encrypted-volume13:32
*** crose has joined #openstack-cinder13:32
nobody_they work in cinder, encrypted and non encrypted, then i can go to the actual lvm volume on the controller and check if it's really encrypted13:33
openstackgerritLucian Petrut proposed openstack/os-brick: Add Windows Fibre Channel connector  https://review.openstack.org/32378013:33
openstackgerritLucian Petrut proposed openstack/os-brick: Add Windows SMBFS connector  https://review.openstack.org/32378113:33
nobody_the problem with the netapp volumes is they work but they aren't encrypted at all even if the volume type has an encryption set on it13:33
*** ducttape_ has joined #openstack-cinder13:34
*** ducttape_ has quit IRC13:34
*** sdake has quit IRC13:34
*** jwcroppe has joined #openstack-cinder13:35
lixiaoy1nobody_: currently only when the encrypted volume is created and attached to VM, VM writes data to the volume. The data are encrypted on the controller13:36
nobody_yes, but i did that and the strings are still readable from the controller, while in lvm volumes are not13:37
nobody_cinder_netapp backend doesn't apply any kind of encryption13:37
nobody_even if the volume is detected as encrypted13:37
nobody_i can string <route-to-volume-on-netapp> | grep "some text i wrote inside"13:38
openstackgerritYuriy Nesenenko proposed openstack/cinder: Fix quota-usage list information with invalid tenant-id  https://review.openstack.org/31567313:38
*** dustins has joined #openstack-cinder13:38
lixiaoy1nobody_: it is odd. In fact, the encryption function should not be related to backend controller13:39
lixiaoy1nobody_: what do you mean by "cinder_netapp doesn't apply any kinds of encryption"?13:40
*** ducttape_ has joined #openstack-cinder13:41
tbarronnobody_: are you using a netapp specific extra-spec key-value that is supposed to trigger encryption by the netapp backend itself?13:42
*** mriedem has joined #openstack-cinder13:42
lixiaoy1tbarron: Hey13:43
tbarronlixiaoy1: hi lisa13:43
nobody_no... we just use the key-value "volume_backend_name" = cinder_netapp which in the controller is the netapp share13:43
*** Guest39607 has joined #openstack-cinder13:43
nobody_and over that volume backend we create volumes with added encryption13:43
*** Guest39607 is now known as wilson-l13:43
tbarronnobody_: ok, just making sure13:44
tbarronnobody_: nfs? iscsi?13:44
nobody_netap ontap 7mode13:45
nobody_but yeah, the storage protocol is nfs13:45
tbarronnobody_: lixiaoy1 knows a whole lot more about encrypted volumes than I do, I just wanted to make sure it wasn't something the netapp backend is supposed to be doing for you13:45
*** eharney has quit IRC13:46
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/32306313:47
nobody_i started thinking that but no, the volume should "reach" the backend already encrypted, but for some reason no encryption is applied, and the weird thing is the volume works perfectly, just, it's readable from the controller13:47
lixiaoy1nobody_: tbarron: currently encrypted volumes have some bugs. May you give the process to create the encrypted volume?13:48
nobody_it's like... instead of failing to encrypt it, it was applying some sort of 0 encryption13:48
nobody_yes... with lvm i just create a volume type with key volume_backend_name = netapp13:48
*** edmondsw has joined #openstack-cinder13:49
lixiaoy1nobody_: can you look at encryption_key_id of the volume in db?13:49
openstackgerritRyan Liang proposed openstack/cinder: Pass volume_type_id in when deleting snapshot  https://review.openstack.org/32312513:49
nobody_then i create an encryption over that tipe with LuksEncryptor as provider, front-end, aes-xts-plain64, and keysize 51213:50
*** eharney has joined #openstack-cinder13:50
nobody_mhhh never looked in the db, i'll check it13:50
*** links has quit IRC13:52
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements  https://review.openstack.org/32307613:52
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/32171513:53
*** sdake has joined #openstack-cinder13:55
*** mtanino has joined #openstack-cinder13:55
*** ametts has joined #openstack-cinder13:58
*** pgadiya has quit IRC14:01
*** jgregor has joined #openstack-cinder14:02
*** merooney has quit IRC14:06
*** cknight has joined #openstack-cinder14:06
*** merooney has joined #openstack-cinder14:06
*** jungleboyj has joined #openstack-cinder14:07
*** akerr has joined #openstack-cinder14:07
*** xyang1 has quit IRC14:09
*** xyang1_ has joined #openstack-cinder14:09
*** yangyapeng has joined #openstack-cinder14:09
*** wilson-l has quit IRC14:11
*** akerr has quit IRC14:12
*** ducttape_ has quit IRC14:12
*** akerr has joined #openstack-cinder14:13
*** wilson_liu has joined #openstack-cinder14:14
*** jgregor has quit IRC14:15
*** zul has quit IRC14:18
*** akerr_ has joined #openstack-cinder14:19
*** rlrossit has quit IRC14:20
*** zul has joined #openstack-cinder14:21
*** akerr has quit IRC14:22
*** cknight1 has joined #openstack-cinder14:23
*** cknight has quit IRC14:25
*** diogogmt has joined #openstack-cinder14:27
*** karthikp has joined #openstack-cinder14:27
*** crose has quit IRC14:27
*** ametts has quit IRC14:28
*** jungleboyj has quit IRC14:35
*** liverpooler has joined #openstack-cinder14:36
*** lprice has joined #openstack-cinder14:39
*** ametts has joined #openstack-cinder14:42
*** jsheeren has quit IRC14:44
*** mfisch has joined #openstack-cinder14:48
*** akapil has joined #openstack-cinder14:48
*** mfisch has quit IRC14:48
*** mfisch has joined #openstack-cinder14:48
*** diogogmt has quit IRC14:48
*** lprice1 has joined #openstack-cinder14:49
*** jungleboyj has joined #openstack-cinder14:49
*** krtaylor has joined #openstack-cinder14:50
*** ducttape_ has joined #openstack-cinder14:51
*** lprice has quit IRC14:51
*** ducttape_ has quit IRC14:51
*** Poornima has quit IRC14:52
*** e0ne has quit IRC14:52
geguileodulek: ping14:52
*** arch-nemesis has joined #openstack-cinder14:53
openstackgerritMichal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls  https://review.openstack.org/25245814:53
openstackgerritMichal Dulko proposed openstack/cinder: Switch request_spec in create volume calls to ovo  https://review.openstack.org/24660014:53
*** xinli has joined #openstack-cinder14:53
openstackgerritMichal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO  https://review.openstack.org/25303314:53
openstackgerritMichal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype)  https://review.openstack.org/25730514:53
dulekgeguileo: pong14:55
geguileodulek: Thank you for the review on the api races14:55
geguileodulek: I think I agree on everything except one where I'm not so sure14:55
geguileodulek: https://review.openstack.org/#/c/259429/27/cinder/consistencygroup/api.py@72214:55
dulekgeguileo: Ah, interesting, I didn't noticed. Let me dig a little.14:56
dulek(BTW - I wonder how SQLA session would work when this object was sent through RPC)14:57
dulekBecause this will be lazy-loaded I think…14:57
*** Julien-zte has joined #openstack-cinder14:58
*** kwathore has joined #openstack-cinder14:59
*** arecknag has quit IRC14:59
openstackgerritGorka Eguileor proposed openstack/cinder: Minimize cost of DB exists decorator checks  https://review.openstack.org/31917714:59
openstackgerritGorka Eguileor proposed openstack/cinder: Improve DB volume check in LVM manage volume  https://review.openstack.org/31932314:59
geguileodulek: I don't understand what you mean...15:00
*** diogogmt has joined #openstack-cinder15:00
*** darosale has joined #openstack-cinder15:00
*** jungleboyj has quit IRC15:01
geguileodulek: This creates a new session for that query upong OVO's request15:01
dulekgeguileo: On the elevated context - this dates back to initial CG commit, so xyang1_ is probably the person to ask about the reason.15:01
*** diogogmt has quit IRC15:01
xyang1_dulek: what is the question15:02
*** jgregor has joined #openstack-cinder15:02
geguileoxyang1_: https://review.openstack.org/#/c/259429/27/cinder/consistencygroup/api.py@72215:02
geguileoxyang1_: Is necessary to elevate the context there?15:02
geguileos/Is/Is it15:02
xyang1_let me check15:02
dulekgeguileo: You can probably disregard my comment on sessions. :)15:02
geguileodulek: Ok :-)15:02
*** diablo_rojo has joined #openstack-cinder15:03
xyang1_geguileo: those requires admin context in the db api15:04
* dulek will be back in ~hour.15:04
geguileoxyang1_: Oh, then the context should already be admin and we can just use the context as it is, right_15:04
geguileo?15:04
xyang1_geguileo: it is a regular tenant15:05
dulekgeguileo: Not reallt, create cgsnap is an user call.15:05
geguileoYeah, my mind just slipped there for a moment15:05
xyang1_geguileo: if you can get admin context in a different way, that is fine too15:06
geguileoxyang1_: Thanks for the explanation  :-)15:06
xyang1_geguileo: no problem15:06
*** xiexs has joined #openstack-cinder15:06
*** akapil has quit IRC15:07
*** akapil has joined #openstack-cinder15:08
*** JoseMello has quit IRC15:09
*** rcernin has quit IRC15:09
sheelDear All core members : looking for final +2 on below patches.. please find some time to have a look on these...15:11
sheelhttps://review.openstack.org/#/c/313925/15:12
sheelhttps://review.openstack.org/#/c/316587/15:12
sheelhttps://review.openstack.org/#/c/317283/15:12
*** jungleboyj has joined #openstack-cinder15:13
*** ducttape_ has joined #openstack-cinder15:15
*** ducttape_ has quit IRC15:16
*** ducttape_ has joined #openstack-cinder15:16
*** xyang has joined #openstack-cinder15:17
openstackgerritGorka Eguileor proposed openstack/cinder: Remove API races from consistency groups  https://review.openstack.org/25942915:18
*** sgotliv has quit IRC15:18
*** tesseract has quit IRC15:18
guitarzanhemna: I think I've asked this before, but is the rootwrap conf stuff ignored if you're running as root?15:19
guitarzanhemna: I upgraded a test env that runs root, and it can't seem to find scsi_id anymore15:19
guitarzanhemna: very likely a config/env problem, I'm just not sure if I have to just add it to PATH15:19
*** aurelien__ has joined #openstack-cinder15:20
*** alkhodos_ has joined #openstack-cinder15:22
*** scottda has joined #openstack-cinder15:23
*** Lee1092 has joined #openstack-cinder15:26
*** gouthamr has quit IRC15:30
*** permalac has joined #openstack-cinder15:32
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Added support for failover_host failback  https://review.openstack.org/31006415:33
patrickeastgeguileo: ping15:33
Swansonpatrickeast, your image caching patch hasn't gone through, right?15:33
geguileopatrickeast: pong15:33
patrickeastSwanson: which one? i think the one in master landed15:33
patrickeastgeguileo: hey, question for you if you've got a min15:34
geguileopatrickeast: I've got the minute, don't know if I have the knowledge though  ;-)15:34
patrickeastgeguileo: i'm looking into removing some driver locks and trying to figure out how to do a relatively safe upsert where kinda thing15:34
xyangscottda: ping15:34
patrickeastgeguileo: so like playing around with it https://gist.github.com/patrick-east/52c8b3782ea4a578e3cac3c45d5d718215:34
scottdaxyang: Hi15:34
Swansonpatrickeast, Ah! That is probably the oddness with my test run. Thanks!15:34
patrickeastgeguileo: i can do a conditional update, but if nothing is there i need to insert a new entry15:35
patrickeastgeguileo: unsure how to safely do that though with sqlalchemy15:35
xyangscottda: we talked last time that we are not going to move CG API out of contrib folder, correct?  that means we will not bump microversion for any change there15:35
xyangscottda: it is regarding this review: https://review.openstack.org/#/c/312063/15:35
xyangscottda: it bumps microversion for a change under contrib15:35
geguileopatrickeast: Are you going to store data in the DB? I thought we were moving away from drivers updating the DB.  r:-??15:36
openstackgerritKendall Nelson proposed openstack/cinder: Add VolumeAttachStatus Enum  https://review.openstack.org/29088515:36
patrickeastgeguileo: heh yea, thats a whole different problem ;)15:36
geguileopatrickeast: Ok, then back to your problem15:36
patrickeastgeguileo: so this used to be a model update we returned with init connection15:36
patrickeastgeguileo: i've moved it so we can try and get rid of these locks15:36
*** nobody_ has quit IRC15:37
geguileopatrickeast: What kind of data are you updating?15:37
geguileopatrickeast: Or inserting?15:37
geguileo(depending of it exists or not)15:37
patrickeastgeguileo: its one of these guys https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L58815:37
scottdaxyang: Yes, I believe that's right. I'm in a meeting, I'll verify in a few minutes...15:37
*** diogogmt has joined #openstack-cinder15:37
*** lgreg has quit IRC15:37
xyangscottda: sure, thanks15:37
scottdaxyang: But, since CG's are not implemented by everyone, it should stay in contrib.15:37
xyangscottda: sure15:38
patrickeastgeguileo: basically a key-value pair... with a kind of overly complicated key (initiator+namespace+key = actual key/id for value)15:38
geguileopatrickeast: And you are checking if it exists based on the key and value?15:38
geguileopatrickeast: Or do you use the initiator?15:38
patrickeastgeguileo: uhh yes :D its a unique combo for initiator, namespace, and key15:39
geguileopatrickeast: I see that, that's why I'm asking...15:39
patrickeastgeguileo: so like host1 might have CHAP_PASSWORD key for driver foo, and for driver bar15:39
geguileopatrickeast: How is your update query?15:40
SwansonHate when I bozo a patch when the queue has 40 items and by the time I fix it the queue is up to 350.15:40
geguileopatrickeast: Because the way I'm doing it in the job distribution patches (that are not up yet) is doing the update, and if it fails15:41
patrickeastgeguileo: yep, thats what i was hoping to achieve15:41
geguileopatrickeast: Do a create, and if I get an exception (because of the unique constraint) I know that it already exists and I can continue15:41
patrickeastgeguileo: sort of try to update, and if its already set fail so that the driver can grab the new value15:41
patrickeastgeguileo: the end result being that we need to either set the value, or get the value that was set15:42
geguileopatrickeast: But then you are not trying to update and fail if exists, you are trying to insert and fail if exists, right?15:42
patrickeastgeguileo: yea, so thats where that wip change was kind of left off, i wasn't sure the best way to know if it failed because it didn't exist or because the expected value we actually care about changed15:43
scottdaxyang: I think that patch is ok not to microversion, since it is not a breaking change. But if we decide we don't microversions extensions, I think that means that we cannot make a breaking API change in an extension. What do you think?15:43
patrickeastgeguileo: right now we get back a 0 for rows changed, and it could have been either, right?15:43
scottdaxyang: What have we done in the past for extensions? Do we claim that the API will not change in a way that breaks existing deployments?15:43
xyangscottda: so my understanding is that the extension APIs cannot be versioned.  that’s why manila moved all extensions to core apis15:44
geguileoxyang: Extensions can be versioned as well, afaik15:45
sheelscottda: xyang: what about mapping with /v3 and microversioning in /v3 for this change15:45
xyanggeguileo: you can add them, but the tradition is not to do that.  Is that something we want to do15:45
scottdaxyang: geguileo Yes, I'm not sure why extensions could not be versioned.15:46
scottdaxyang: geguileo We could probably discuss this at the meeting today.15:46
xyangsure15:46
scottdaI put it on the agenda15:48
*** baumann has joined #openstack-cinder15:49
*** akapil has quit IRC15:52
xyangscottda: ok, thanks15:53
*** wilson_liu is now known as wilson-l15:53
*** rajinir has joined #openstack-cinder15:54
*** e0ne has joined #openstack-cinder15:54
*** sdake has quit IRC15:54
*** rlrossit has joined #openstack-cinder15:55
baumannrlrossit: Test15:55
*** nkrinner is now known as nkrinner_afk15:55
*** wilson-l has quit IRC15:58
*** wilson-l has joined #openstack-cinder15:58
*** wilson-l has left #openstack-cinder15:58
*** wilson has joined #openstack-cinder15:59
*** wilson is now known as Guest3920415:59
e0neis it meeting time?15:59
*** Guest39204 is now known as wilson-l15:59
*** GB21 has joined #openstack-cinder16:02
*** mjura has quit IRC16:05
*** raunak has joined #openstack-cinder16:08
*** vivekd has joined #openstack-cinder16:08
*** raunak has quit IRC16:09
*** Apoorva has joined #openstack-cinder16:10
*** akapil has joined #openstack-cinder16:11
*** leeantho has joined #openstack-cinder16:11
*** wilsonL has quit IRC16:12
*** wilson_liu has joined #openstack-cinder16:13
*** Gao has joined #openstack-cinder16:15
*** akapil has quit IRC16:16
*** zhangjn has quit IRC16:17
*** sdague has quit IRC16:18
*** savihou has quit IRC16:18
*** Lallana has quit IRC16:18
*** sdake has joined #openstack-cinder16:19
*** zhangjn has joined #openstack-cinder16:19
*** sdake has quit IRC16:23
e0nejungleboyj: hi! who it the best contact person for storwize unit tests?16:23
*** cknight1 has quit IRC16:23
jungleboyje0ne: Let me get you her e-mail address.16:24
e0nejungleboyj: thank you, Jay!16:24
jungleboyje0ne: Any time.16:25
e0nejungleboyj: in which timezone? :)16:26
jungleboyjBeijing16:26
openstackgerritMerged openstack/cinder: Add unit test case in Nimble Driver  https://review.openstack.org/32374816:27
e0ne:)16:27
openstackgerritLisaLi proposed openstack/os-brick: Copy encryptors from Nova to os-brick  https://review.openstack.org/24737216:27
*** martyturner has joined #openstack-cinder16:28
*** lgreg has joined #openstack-cinder16:28
jungleboyje0ne: lixqin@cn.ibm.com  Xiao Qin Li16:29
e0nejungleboyj: thanks!16:29
*** vivekd has quit IRC16:29
*** laughterwym has quit IRC16:31
jungleboyje0ne: Welcome!16:31
*** sdague has joined #openstack-cinder16:31
*** sdake has joined #openstack-cinder16:32
*** nilk has joined #openstack-cinder16:35
*** vivekd has joined #openstack-cinder16:35
*** jordan_ has quit IRC16:36
*** GB21 has quit IRC16:39
*** aurelien__ has quit IRC16:42
*** akapil has joined #openstack-cinder16:46
*** e0ne has quit IRC16:48
*** yuriy_n17 has quit IRC16:48
*** yuriy_n17 has joined #openstack-cinder16:48
*** wilson-l has quit IRC16:49
*** akapil has quit IRC16:50
*** bardia has joined #openstack-cinder16:54
jungleboyjsmcginnis: I sent a note to Xiao Qin about looking at the async migration patch.16:55
*** xyang has quit IRC16:56
*** GB21 has joined #openstack-cinder16:57
smcginnisjungleboyj: Cool, thank you!16:57
jungleboyjsmcginnis: Welcome.16:58
*** rlrossit has quit IRC17:00
*** baumann has quit IRC17:00
*** jgregor has quit IRC17:00
*** diablo_rojo has quit IRC17:00
*** links has joined #openstack-cinder17:02
*** xyang has joined #openstack-cinder17:02
*** merooney has quit IRC17:04
*** xyang has quit IRC17:05
*** jungleboyj has quit IRC17:06
geguileoaorourke: ping - https://review.openstack.org/#/c/32068617:10
*** adrianofr has quit IRC17:11
openstackgerritRajini Ram proposed openstack/cinder: Volume manage/unmanage support for Eqlx driver  https://review.openstack.org/31874817:12
*** adrianofr has joined #openstack-cinder17:12
*** gouthamr has joined #openstack-cinder17:13
*** zhangjn has quit IRC17:17
*** zhangjn has joined #openstack-cinder17:18
*** ChubYann has joined #openstack-cinder17:19
scottdaDuncanT: The plan for porting extensions is to port each individual extension to make review easier, then add URLs and microversion bump in one patch like this: https://github.com/openstack/manila/commit/2467ccf223559e7542349a5336379c6ef607352c17:19
scottdaDuncanT: So it will end up being many 1000s of LOC17:19
DuncanTscottda: looking. Most of the 1000s of lines are just a moved file though, right?17:21
*** jungleboyj has joined #openstack-cinder17:21
*** ociuhandu_ has joined #openstack-cinder17:22
DuncanTscottda: ok, that's messier than I'd hoped :-(17:23
scottdaDuncanT: Yeah, messier than I'd hoped as well. Here's one of the single extension ports: https://review.openstack.org/#/c/239913/617:24
DuncanTscottda: have to see how our patches come out I guess17:26
bswartzI never said it was trivial17:26
bswartzI just said it wasn't that bad17:26
bswartzscottda: in the patch you linked I think there are some random unneeded code cleanups making the change look more complex17:27
scottdabswartz: OK. It was just an example FYI for Duncan. I'm sure it won't be too terrible, just going to require a bit of focus, which I won't have for a few weeks. I'd like to get a first port ready for the mid-cycle, the rest should come easy after that's merged.17:28
*** karthikp has quit IRC17:33
*** ociuhandu has quit IRC17:35
*** mriedem has quit IRC17:36
*** links has quit IRC17:36
*** karthikp has joined #openstack-cinder17:36
*** bardia has quit IRC17:36
*** alkhodos_ has quit IRC17:37
*** xyang has joined #openstack-cinder17:37
*** xyang has quit IRC17:38
*** xyang has joined #openstack-cinder17:40
*** cknight has joined #openstack-cinder17:40
*** liverpooler has quit IRC17:46
*** mvk has quit IRC17:48
*** xyang has quit IRC17:48
*** zhangjn has quit IRC17:49
*** arch-nemesis has quit IRC17:50
*** suyash has joined #openstack-cinder17:51
*** zhangjn has joined #openstack-cinder17:51
*** timcl has quit IRC17:51
*** merooney has joined #openstack-cinder17:51
*** arch-nemesis has joined #openstack-cinder17:51
jgriffithildikov: I suppose we could be extra safe on this:  https://review.openstack.org/#/c/321095/17:53
*** GB21 has quit IRC17:53
*** ccesario has quit IRC17:53
jgriffithildikov: meh.. never mind, thinking out loud17:53
jgriffith:)17:53
*** merooney has quit IRC17:54
smcginnisjgriffith: What's your idea?17:54
*** timcl has joined #openstack-cinder17:55
hemnajgriffith, what do you mean be extra safe?17:55
jgriffithsmcginnis: well... so I suppose there *could* be a case where the controller and vol service run on separate nodes, and for whatever reason one of them is N-117:55
jgriffithsmcginnis: frankly I've never been a fan of trying to maintain that model to begin with17:55
ildikovjgriffith: hey, sorry, just got off the phone17:56
jgriffithsmcginnis: but.. if anybody is really worried about it, we could be the old calls back in db-api so if there was some mismatch things would still work17:56
smcginnisjgriffith: Ah, I see. That could be a problem.17:56
jgriffithjgriffith: I'd recommend it be a follow up regardless17:56
smcginnisjgriffith: But I agree, not sure if that's that big of a concern.17:56
jgriffithwhich I'm happy to do... just not right now :)17:56
ildikovjgriffith: yeah, that's the case I meant with my question, what you described above :)17:57
jgriffithildikov: good call then :)17:57
jgriffithildikov: I personally think rolling upgrades are not worth chasing for Cinder but that's besides the point.17:58
jgriffithildikov: smcginnis anyway... I just did what I was asked after the first round of -1's17:58
*** IanGovett has joined #openstack-cinder17:59
ildikovjgriffith: some support for rolling upgrades would be nice at a certain point17:59
jgriffithildikov: yeah, I'm working a patch now.. give me 5 minutes or so18:00
*** raunak has joined #openstack-cinder18:00
ildikovjgriffith: next time I will not let you convince me that easily :)18:02
openstackgerritSean McGinnis proposed openstack/cinder: Add driver interface checks  https://review.openstack.org/29714018:05
openstackgerritMerged openstack/os-brick: Updated from global requirements  https://review.openstack.org/32307618:05
*** bardia has joined #openstack-cinder18:05
*** bardia_ has joined #openstack-cinder18:06
*** ccesario has joined #openstack-cinder18:06
*** vivekd has quit IRC18:06
jgriffithildikov: :)18:06
*** bardia has quit IRC18:10
*** neophy has joined #openstack-cinder18:11
*** ccesario has quit IRC18:11
openstackgerritJohn Griffith proposed openstack/cinder: Preserve legacy db api attachment_get_by calls  https://review.openstack.org/32404918:12
smcginnisjgriffith: I guess there's the final answer to the question about fixing or changing the method name. ;)18:14
*** IanGovett has quit IRC18:14
hemnasmcginnis, ping18:14
smcginnishemna: hey18:15
hemnaso...18:15
hemnaI'm really confused by https://review.openstack.org/#/c/29714018:15
hemnaespecially wrt the fczm18:15
*** IanGovett has joined #openstack-cinder18:15
hemnawe already have https://github.com/openstack/cinder/blob/master/cinder/zonemanager/drivers/fc_zone_driver.py18:15
hemnathat describes what a fczm driver should implement18:15
hemnaand your patch adds another layer on top of that18:16
smcginnishemna: Yeah, there's definitely duplication there.18:16
hemnasomething about it doesn't seem right to me18:16
*** mriedem has joined #openstack-cinder18:16
smcginnishemna: My goal was to separate out interface and implementation and have it all in one location to make it easy to find and understand.18:16
hemnaeither replace the fc_zone_driver.py with your interface, or remove the interface18:17
*** bardia_ has quit IRC18:17
hemnaI don't think we should have both18:17
*** bardia has joined #openstack-cinder18:17
smcginnishemna: Yeah, I think you're right.18:17
smcginnishemna: Duplication, other than just being bad, could lead to confusion.18:18
hemnayah I was really confused by it18:18
smcginnishemna: Which is exactly one of the things I'm trying to get rid of with this.18:18
hemnaat first I thought it was a volume driver interface18:18
hemnathen I realized what you were trying to do18:18
smcginnisI wanted the driver interfaces clear for volumes, backup drivers, and fczm.18:18
smcginnisThat last one is pretty trivial, but...18:19
*** cknight has quit IRC18:19
hemnayah18:19
hemnathis has some bleed over w/ the ABC crap though18:19
hemnaI'm a little concerned that this will muddy things up more18:19
hemnaif we have ABC and this18:19
smcginnishemna: Well, one of the longer term goals is to clean up that ABC mess.18:19
hemnaI'm not sure what's "right" wrt to that18:19
smcginnisWe've talked about changing that, but so far we haven't been able to fix it.18:20
hemnathe ABC stuff has always been a disaster IMHOP18:20
hemnaIMHO18:20
smcginnisYeah, really didn't turn out the way I had hoped.18:20
smcginnisAnd I think either last year or the year before in Ft Collins we had talked about redoing it.18:20
hemnayah we had discussed it18:20
smcginnisI want to separate out the implementation (ABC) with the interface (this patch) to make that easier.18:21
hemnabut wasn't sure what was the end result other than leaving it as is :(18:21
smcginnisyeah, another one where I don't think we came to a conclusion.18:21
hemnathe problem is the ABC stuff was also put in place to enforce implementing methods18:21
smcginnisMy hope is this change will help make it easier to move ahead with that.18:21
hemnawhich is what the interface stuff is doing too.18:21
hemna:(18:21
smcginnisYep.18:21
hemnasadness18:21
*** cknight has joined #openstack-cinder18:22
openstackgerritMerged openstack/cinder: Huawei: Raise if no FC port found  https://review.openstack.org/29617618:22
smcginnisBut the way the ABC stuff was done, it made that enforcement and discoverability impossible. And probably worse than it was.18:22
hemnayup18:22
hemnaand confused a lot of people in the process18:22
smcginnisI still get confused by it. ;D18:23
*** ccesario has joined #openstack-cinder18:24
*** avishay has joined #openstack-cinder18:25
*** e0ne has joined #openstack-cinder18:26
hemnasmcginnis, also, https://review.openstack.org/#/c/272522/18:26
hemnaos-brick windows connectors patch18:26
hemnadidn't we decide in Austin that os-brick connectors are going to need CI ?18:27
hemna(if not, then they should IMHO)18:27
smcginnishemna: Yes. I agree with your comment there. Scary making major changes with no tests to make sure we don't break things.18:27
hemnayah18:28
hemnacan you pile on there please ?18:28
smcginnishemna: Sure, will do.18:28
*** lpetrut has quit IRC18:28
hemnado we have any notes from Austin about the CI requirements ?18:28
hemnajungleboyj ^^18:29
*** ducttape_ has quit IRC18:30
smcginnishemna: I think it was in one of the etherpads and in the recording.18:30
*** lpetrut has joined #openstack-cinder18:30
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Added support for failover_host failback  https://review.openstack.org/31006418:30
smcginnishemna: Mike wanted to write up something and be the contact point for that.18:30
patrickeastiirc the decision was that we wanted to add them.. but i don't recall setting any hard requirements18:30
patrickeasthttps://etherpad.openstack.org/p/cinder-newton-testingprocess18:30
patrickeastline 80ish18:30
smcginnisYeah, I think we need to give some time before we make it a hard requirement.18:31
smcginnisBut certainly those with their own connectors should really have it ASAP.18:31
*** Gao has quit IRC18:31
jgriffithildikov: smcginnis https://review.openstack.org/#/c/324049/118:31
jgriffith^^ that should cover things18:32
smcginnisjgriffith: Thanks, got a tab open.18:32
hemnapatrickeast, hrmm ok that's vague18:32
jgriffith:)18:32
*** Gao has joined #openstack-cinder18:32
patrickeasthemna: haha yea18:32
hemnaoh line 7918:32
* hemna waits for #Action18:32
hemna:P18:32
patrickeasthemna: smcginnis: IMO giving some leeway for existing ones makes sense, but seems like forcing it upon new ones would be a reasonable thing to do18:33
*** Lee1092 has quit IRC18:33
hemnapatrickeast, +118:33
hemnayah I'm really concerned about the windows connectors.18:33
hemnaabsolutely no testing in there other than unit tests.18:33
*** darosale has quit IRC18:34
*** cknight has quit IRC18:34
patrickeasthemna: yea, thats kinda scary18:34
*** cknight has joined #openstack-cinder18:34
patrickeasthemna: we would never know if something breaks them all :(18:34
jgriffithpatrickeast: if a tree falls in the forest.....18:35
patrickeasthaha yea18:35
hemnapeople use windows ?18:35
patrickeasti was just looking at that18:35
patrickeasti guess with hyper-v compute nodes?18:35
hemnapatrickeast, yah18:35
ildikovjgriffith: tnx! very minor nit in line18:36
hemnathey have a nova patch as well18:36
hemnaand it's failing the hyper-v CI :P18:36
patrickeasthah18:36
*** e0ne has quit IRC18:37
*** cknight1 has joined #openstack-cinder18:38
*** cknight has quit IRC18:39
*** mvk has joined #openstack-cinder18:39
*** lpetrut has quit IRC18:40
*** ccesario has quit IRC18:41
*** krtaylor has quit IRC18:41
*** jungleboyj has quit IRC18:42
*** ociuhandu_ has quit IRC18:42
openstackgerritJohn Griffith proposed openstack/cinder: Preserve legacy db api attachment_get_by calls  https://review.openstack.org/32404918:43
*** e0ne has joined #openstack-cinder18:44
*** ducttape_ has joined #openstack-cinder18:44
openstackgerritMerged openstack/cinder: Fix attachment_get_by_* to return entire list  https://review.openstack.org/32109518:45
dulekCan someone take a look on https://review.openstack.org/#/c/307075/ ? It's lack is blocking some objects work as well as causing random CI failures on changes that modify object versions.18:46
openstackgerritSean McGinnis proposed openstack/cinder: Add driver interface checks  https://review.openstack.org/29714018:46
*** mvk has quit IRC18:48
*** wilson_liu has quit IRC18:48
*** wilson1 has joined #openstack-cinder18:48
*** mvk has joined #openstack-cinder18:48
*** JoseMello has joined #openstack-cinder18:49
*** mvk_ has joined #openstack-cinder18:52
*** baumann has joined #openstack-cinder18:53
*** IanGovett has quit IRC18:53
dulekjgriffith: Maaan, you've scared me with https://review.openstack.org/#/c/324049 :)18:53
*** ccesario has joined #openstack-cinder18:53
*** xyang has joined #openstack-cinder18:54
dulekjgriffith: Explanation I've wrote may be not clear enough, so I'm here to try to answer your questions.18:54
jgriffithdulek: haha... so that was my original response, then I just decided it was easier to put it back in there and get on with my life18:54
*** jgregor has joined #openstack-cinder18:54
jgriffithdulek: I'll let others hash it out... take or leave it, no matter to me and I don't think it's that important :)18:54
dulekjgriffith: Basically codebase isn't shared resource. DB however is - that's why we're leaving DB *columns* behind.18:55
jgriffithdulek: I mean that "your response" was "my" original response in the first patch18:55
dulekjgriffith: But how the service accesses these - we don't care.18:55
hemnawhy do we ever really want to support older c-vol with newer c-api ?18:55
openstackgerritMerged openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/32171518:55
jgriffithhemna: it's not c-api at all18:55
* dulek digs through log and comments.18:55
hemnaI get that argument in nova patches too.  I just don't get it.18:55
hemnajgriffith, I know, just in general though18:56
*** ChubYann has quit IRC18:56
hemnamaybe I'm dumb.18:56
*** mvk has quit IRC18:56
*** xyang has quit IRC18:56
jgriffithdulek: I don't disagree with you.. see my comment in patch-set 318:57
*** jungleboyj has joined #openstack-cinder18:57
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376618:57
jungleboyjhemanthm: I was not in the meetings at the time, but I think  diablo_rojo took notes.18:57
jgriffithhemna: so in general it's certainly possible to have api, vol, sch, ctrlr on different servers18:57
hemnasure, it's possible, and it's possible to have different versions of those in the process of upgrading18:58
hemnaI just don't think it's smart18:58
hemna:P18:58
jgriffithhemna: there have been folks that want rolling upgrades (ie DuncanT ) and that would require the n - 1 and n + 1 type of thing18:58
dulekhemna: It's to make ops live easier. New c-api/c-sch<->old c-vol is super useful when you're running big number of LVM backends in your cloud (Rackspace does so I believe). It's easier to upgrade them one-by-one without taking down your whole cloud for the process.18:58
jgriffithhemna: I certainly don't disagree with you18:58
hemnayah, I get that.  I just don't think it's wise.18:58
jgriffithhemna: I'm not a proponent of it, I'm just trying to do what people ask to merge code18:59
hemnayah understood18:59
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/32306318:59
*** raunak has quit IRC18:59
hemnadulek, I get that too, I just am not sure the amount of complexity in Cinder is enough of a justification for it.19:00
dulekhemna: Trade-offs everywhere! :(19:01
hemnadulek, nova has gone down this route and it eventually leads to features never making it in place because of the upgrades complexity19:02
hemnaso it's not just a tradeoff IMHO19:02
*** porrua has quit IRC19:02
*** ebalduf_ has joined #openstack-cinder19:03
dulekSure, I understand the burden. I guess this is situation you're encountering more and more with maturing project.19:03
dulekI like the fact that at least operators feedback is positive. They are actually pushing Glance and Keystone to support the same.19:04
*** krtaylor has joined #openstack-cinder19:06
*** diablo_rojo has joined #openstack-cinder19:07
dulekhemna: Hey, and by the way - if you're encountering upgradeability problems with multiattach - ping me. I've did some tinkering on that, so it may be easier for me to design a solution.19:08
hemnaheh19:08
hemnamultiattach is a dirty word19:08
hemna:P19:08
guitarzandon't let the dream die19:09
*** ChubYann has joined #openstack-cinder19:09
*** xyang has joined #openstack-cinder19:12
*** ducttape_ has quit IRC19:12
ildikovdulek: I wish we would be at that stage already when this is the problem to deal with :)19:16
dulekildikov: :D19:17
openstackgerritIvan Kolodyazhny proposed openstack/cinder: RBD Thin Provisioning stats  https://review.openstack.org/17826219:20
*** angela-s has joined #openstack-cinder19:23
*** xyang has quit IRC19:24
*** xyang has joined #openstack-cinder19:25
*** lpetrut has joined #openstack-cinder19:26
*** ociuhandu has joined #openstack-cinder19:29
*** xyang has quit IRC19:30
*** permalac has quit IRC19:30
*** xyang has joined #openstack-cinder19:32
*** ducttape_ has joined #openstack-cinder19:33
*** andymaier has quit IRC19:34
*** e0ne has quit IRC19:35
*** e0ne has joined #openstack-cinder19:40
*** xyang has quit IRC19:40
*** sgotliv has joined #openstack-cinder19:42
*** xyang has joined #openstack-cinder19:43
*** e0ne has quit IRC19:43
*** porrua has joined #openstack-cinder19:44
*** sheel has quit IRC19:45
*** Apoorva has quit IRC19:45
*** dave-mccowan has quit IRC19:46
*** e0ne has joined #openstack-cinder19:48
*** ociuhandu_ has joined #openstack-cinder19:49
*** ociuhandu_ has quit IRC19:49
*** timcl has quit IRC19:51
*** xyang has quit IRC19:52
*** raunak has joined #openstack-cinder19:55
*** porrua has quit IRC19:56
*** Apoorva has joined #openstack-cinder19:57
*** kfarr has joined #openstack-cinder20:04
*** dave-mccowan has joined #openstack-cinder20:04
*** david-lyle has joined #openstack-cinder20:05
*** lpetrut has quit IRC20:08
*** ociuhandu has quit IRC20:08
*** e0ne has quit IRC20:09
openstackgerritAlexey Khodos proposed openstack/cinder: NexentaStor5: add extend method to NFS driver  https://review.openstack.org/29788020:11
*** chip_ has joined #openstack-cinder20:12
*** jbernard has quit IRC20:16
*** jbernard has joined #openstack-cinder20:18
*** angela-s has quit IRC20:22
*** eharney has quit IRC20:25
*** xyang has joined #openstack-cinder20:26
*** lpetrut has joined #openstack-cinder20:27
*** lgreg has quit IRC20:30
*** xyang has quit IRC20:31
*** lgreg has joined #openstack-cinder20:31
*** xyang has joined #openstack-cinder20:32
openstackgerritAlexey Khodos proposed openstack/cinder: NexentaStor5: add extend method to NFS driver  https://review.openstack.org/29788020:33
*** bharath has joined #openstack-cinder20:47
*** xinli has quit IRC20:48
*** sdake has quit IRC20:49
*** lgreg has quit IRC20:50
*** baumann has quit IRC20:50
*** david-lyle has quit IRC20:51
*** diablo_rojo has quit IRC20:51
*** neophy has quit IRC20:52
*** neophy has joined #openstack-cinder20:53
*** jungleboyj has quit IRC20:55
*** bharath has quit IRC20:56
*** jgregor has quit IRC20:57
openstackgerritxing-yang proposed openstack/cinder: Add group type and group specs  https://review.openstack.org/32016520:58
*** kfarr has quit IRC20:58
*** julim has quit IRC21:00
*** david-lyle has joined #openstack-cinder21:01
*** raildo is now known as raildo-afk21:03
*** neophy has quit IRC21:05
*** hemna is now known as hemnafk21:07
*** jungleboyj has joined #openstack-cinder21:21
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - iSCSI Multipath support  https://review.openstack.org/31706021:25
*** akshai has quit IRC21:30
*** kaisers_ has joined #openstack-cinder21:32
*** jgregor has joined #openstack-cinder21:33
*** bill_az has joined #openstack-cinder21:34
*** dustins has quit IRC21:37
*** jgregor has quit IRC21:44
*** sgotliv has quit IRC21:48
*** edmondsw has quit IRC21:50
*** sdake has joined #openstack-cinder21:50
*** mriedem has quit IRC21:54
*** chip_ has quit IRC21:55
*** avishay has quit IRC21:56
*** chip_ has joined #openstack-cinder21:57
*** akerr_ has quit IRC21:59
*** sdake_ has joined #openstack-cinder22:01
*** ducttape_ has quit IRC22:01
*** david-lyle has quit IRC22:01
*** sdake has quit IRC22:03
*** xyang has left #openstack-cinder22:04
*** cknight1 has quit IRC22:05
*** mtanino has quit IRC22:06
*** jungleboyj has quit IRC22:07
openstackgerritMerged openstack/cinder: Remove API races from consistency groups  https://review.openstack.org/25942922:07
*** martyturner has quit IRC22:09
*** xyang1_ has quit IRC22:09
*** ebalduf_ has quit IRC22:10
*** chip_ has quit IRC22:13
*** david-lyle has joined #openstack-cinder22:16
*** sdake_ has quit IRC22:21
*** karthikp has quit IRC22:22
*** lpetrut has quit IRC22:27
*** tpsilva has quit IRC22:27
*** ametts has quit IRC22:30
*** sdake has joined #openstack-cinder22:31
*** leeantho has quit IRC22:34
*** mtanino has joined #openstack-cinder22:35
*** gouthamr has quit IRC22:36
*** MrHeavy has quit IRC22:38
*** ducttape_ has joined #openstack-cinder22:48
*** jwcroppe has quit IRC22:56
*** arch-nemesis has quit IRC22:57
*** karthikp has joined #openstack-cinder22:59
*** kurtmartin has joined #openstack-cinder23:00
openstackgerritBardia Keyoumarsi proposed openstack/cinder: Coho data sanity fix in the driver unittest  https://review.openstack.org/32412723:03
*** sdague has quit IRC23:07
*** JoseMello has quit IRC23:09
*** lgreg has joined #openstack-cinder23:20
*** karthikp has quit IRC23:20
*** nug has quit IRC23:26
*** cFouts is now known as nug23:26
*** nug is now known as Guest3563823:27
*** karthikp has joined #openstack-cinder23:28
*** gnarld_ has joined #openstack-cinder23:28
*** Gao has quit IRC23:31
*** Gao has joined #openstack-cinder23:32
*** lgreg has quit IRC23:32
*** diogogmt has quit IRC23:33
*** karthikp has quit IRC23:43
*** karthikp has joined #openstack-cinder23:44
*** mtanino has quit IRC23:49
*** Julien-zte has quit IRC23:51
*** afazekas has quit IRC23:52
*** sdake_ has joined #openstack-cinder23:55
*** afazekas has joined #openstack-cinder23:56
*** sdake has quit IRC23:58
*** david-lyle has quit IRC23:59

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