Friday, 2017-06-30

*** Apoorva has quit IRC00:01
*** Apoorva has joined #openstack-cinder00:02
*** Apoorva_ has quit IRC00:02
*** Apoorva has quit IRC00:04
*** yongwc has joined #openstack-cinder00:13
openstackgerritMerged openstack/cinder master: SMBFS: enhance volume cloning  https://review.openstack.org/47274700:15
*** Julien-zte has joined #openstack-cinder00:28
*** zhurong has joined #openstack-cinder00:33
*** dhinesh has quit IRC00:42
*** junbo has quit IRC00:45
*** markstur_ has quit IRC00:46
*** markstur has joined #openstack-cinder00:47
*** junbo has joined #openstack-cinder00:49
*** iceyao has joined #openstack-cinder00:56
*** johnthetubaguy has quit IRC00:56
*** johnthetubaguy has joined #openstack-cinder00:57
*** yongwc has quit IRC00:58
*** iceyao has quit IRC01:01
openstackgerritTommyLike proposed openstack/cinder master: Add ability to extend 'in-use' volume  https://review.openstack.org/45428701:09
*** iceyao has joined #openstack-cinder01:17
*** sdague has quit IRC01:18
*** iceyao has quit IRC01:22
*** iceyao has joined #openstack-cinder01:24
openstackgerritYingzhe Zeng proposed openstack/cinder master: Add 'LUNType' configuration verification for Dorado array  https://review.openstack.org/45236201:27
*** iceyao has quit IRC01:28
*** Julien-zte has quit IRC01:33
*** iceyao has joined #openstack-cinder01:35
*** Julien-zte has joined #openstack-cinder01:36
*** rods has quit IRC01:40
*** zhaochao has quit IRC01:41
*** markstur has quit IRC01:42
*** Shunli has joined #openstack-cinder01:42
*** zhangguoqing has joined #openstack-cinder01:43
*** lhx__ has joined #openstack-cinder01:44
*** zsli_ has joined #openstack-cinder01:44
*** zsli__ has joined #openstack-cinder01:46
*** Shunli has quit IRC01:47
*** kaisers_ has joined #openstack-cinder01:48
*** zsli_ has quit IRC01:49
*** zhaochao has joined #openstack-cinder01:49
*** kaisers__ has quit IRC01:50
*** zhurong has quit IRC01:50
*** rods has joined #openstack-cinder01:53
*** lhx__ has quit IRC01:54
*** lhx__ has joined #openstack-cinder01:54
*** lucasxu has joined #openstack-cinder01:54
*** s-shiono has joined #openstack-cinder01:58
*** tsuzuki has joined #openstack-cinder02:02
*** markstur has joined #openstack-cinder02:02
*** linlan has quit IRC02:08
*** zhurong has joined #openstack-cinder02:10
*** markstur has quit IRC02:11
openstackgerritwangxiyuan proposed openstack/cinder master: Support metadata for backup resource-1  https://review.openstack.org/47154102:12
openstackgerritwangxiyuan proposed openstack/cinder master: Use OVO for test_backup  https://review.openstack.org/47250602:12
*** diablo_rojo has joined #openstack-cinder02:20
*** markstur has joined #openstack-cinder02:48
*** enriquetaso_ has quit IRC02:52
*** markstur has quit IRC02:53
*** salv-orlando has joined #openstack-cinder03:02
openstackgerritVivek Soni proposed openstack/cinder master: Volume type with encryption comparison  https://review.openstack.org/47788803:06
*** salv-orlando has quit IRC03:07
*** diablo_rojo has quit IRC03:07
*** vivsoni has joined #openstack-cinder03:13
*** wanghao has joined #openstack-cinder03:15
openstackgerritjunbo.li proposed openstack/python-cinderclient master: Add support for Unicode value filters  https://review.openstack.org/47912203:20
*** gouthamr has quit IRC03:25
*** iceyao has quit IRC03:33
*** iceyao has joined #openstack-cinder03:34
*** lhx__ has quit IRC03:35
*** psachin has joined #openstack-cinder03:44
*** Julien-zte has quit IRC03:44
*** iceyao has quit IRC03:55
*** jwcroppe has joined #openstack-cinder03:58
*** iceyao has joined #openstack-cinder03:58
*** iceyao has quit IRC04:03
*** mtanino_ has quit IRC04:03
*** salv-orlando has joined #openstack-cinder04:03
*** salv-orlando has quit IRC04:08
openstackgerritjunbo.li proposed openstack/python-cinderclient master: Fix support for Unicode value filters  https://review.openstack.org/47912204:10
openstackgerritbhavani proposed openstack/cinder-specs master: Remove pbr warnerrors in favor of sphinx check  https://review.openstack.org/47887504:23
*** adisky__ has joined #openstack-cinder04:26
*** arnewiebalck_ has joined #openstack-cinder04:27
openstackgerritbhavani proposed openstack/cinder-specs master: Remove pbr warnerrors in favor of sphinx check  https://review.openstack.org/47887504:28
*** lucasxu has quit IRC04:30
openstackgerritbhavani proposed openstack/cinder-specs master: Remove pbr warnerrors in favor of sphinx check  https://review.openstack.org/47887504:34
*** e0ne has joined #openstack-cinder04:51
*** Poornima has joined #openstack-cinder04:51
*** e0ne has quit IRC04:58
*** rha has quit IRC05:02
*** salv-orlando has joined #openstack-cinder05:04
*** zengyingzhe has quit IRC05:08
*** zengyingzhe_ has joined #openstack-cinder05:08
*** salv-orlando has quit IRC05:08
openstackgerritRyan Liang proposed openstack/cinder master: Unity: temp snapshot for backup deleted twice  https://review.openstack.org/47914805:11
*** Poornima has quit IRC05:13
*** salv-orlando has joined #openstack-cinder05:15
*** edmondsw_ has joined #openstack-cinder05:17
*** ChubYann has quit IRC05:18
*** edmondsw has quit IRC05:19
openstackgerritMerged openstack/cinder master: Mock time.sleep for VMAXRestTest.test_remove_vol_from_sg_failed  https://review.openstack.org/47376805:21
openstackgerritjiamin proposed openstack/cinder master: DS8K: add replication consistency group support  https://review.openstack.org/46587705:22
*** iceyao has joined #openstack-cinder05:22
*** lpetrut has joined #openstack-cinder05:31
*** Julien-zte has joined #openstack-cinder05:34
*** markstur has joined #openstack-cinder05:40
*** markstur has quit IRC05:44
*** rcernin has joined #openstack-cinder05:45
*** pgadiya has joined #openstack-cinder05:48
*** links has quit IRC06:07
*** salv-orlando has quit IRC06:21
*** Poornima has joined #openstack-cinder06:21
*** salv-orlando has joined #openstack-cinder06:21
*** links has joined #openstack-cinder06:21
*** Poornima has quit IRC06:21
*** Poornima has joined #openstack-cinder06:22
*** Poornima has quit IRC06:22
*** Poornima has joined #openstack-cinder06:22
*** salv-orlando has quit IRC06:26
*** salv-orlando has joined #openstack-cinder06:28
*** zhurong has quit IRC06:28
*** Julien-zte has quit IRC06:29
*** sticker has quit IRC06:31
*** Poornima has quit IRC06:33
*** lpetrut has quit IRC06:35
*** s-shiono has quit IRC06:38
*** pcaruana has joined #openstack-cinder06:45
*** crushil_ has joined #openstack-cinder06:48
openstackgerritTommyLike proposed openstack/cinder master: Add ability to extend 'in-use' volume  https://review.openstack.org/45428706:54
*** zhurong has joined #openstack-cinder07:06
*** tesseract has joined #openstack-cinder07:14
*** arnewiebalck_ has quit IRC07:23
*** markstur has joined #openstack-cinder07:28
*** markstur has quit IRC07:33
*** lpetrut has joined #openstack-cinder07:35
*** gouthamr has joined #openstack-cinder07:39
*** crushil_ has quit IRC07:42
*** wanghao has quit IRC07:42
*** lpetrut has quit IRC07:42
*** lhx__ has joined #openstack-cinder07:42
*** wanghao has joined #openstack-cinder07:43
*** dims has quit IRC07:45
*** dims has joined #openstack-cinder07:48
*** lhx__ has quit IRC07:49
*** lhx__ has joined #openstack-cinder07:49
*** e0ne has joined #openstack-cinder07:49
*** boris-42__ has joined #openstack-cinder07:55
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:02
*** vivsoni has quit IRC08:07
*** vivsoni has joined #openstack-cinder08:07
*** salv-orl_ has joined #openstack-cinder08:19
*** Poornima has joined #openstack-cinder08:22
*** salv-orlando has quit IRC08:23
openstackgerritCheng Yong Zhao proposed openstack/cinder master: Add gmcv support in SVC driver  https://review.openstack.org/46119108:23
*** salv-orl_ has quit IRC08:24
*** mtanino__ has joined #openstack-cinder08:31
*** mtanino__ has quit IRC08:31
*** Poornima has quit IRC08:32
*** rha has joined #openstack-cinder08:39
*** rha has joined #openstack-cinder08:39
*** Poornima has joined #openstack-cinder08:40
*** markstur has joined #openstack-cinder08:40
openstackgerritzhangdaolong proposed openstack/os-brick master: the ioctx should be close when open rbd pool failed  https://review.openstack.org/44411408:45
*** markstur has quit IRC08:45
*** dims has quit IRC08:45
*** dims has joined #openstack-cinder08:47
*** Poornima has quit IRC08:53
*** tsuzuki has quit IRC08:59
openstackgerritLucian Petrut proposed openstack/cinder master: Windows iSCSI: use configured iSCSI portals  https://review.openstack.org/46465909:09
openstackgerritLucian Petrut proposed openstack/cinder master: Allow extension of volumes with snapshots for SMBfs driver  https://review.openstack.org/46290209:12
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Add ability to specify backup driver via class name  https://review.openstack.org/47747209:12
*** markstur has joined #openstack-cinder09:16
*** markstur has quit IRC09:21
*** wanghao_ has joined #openstack-cinder09:26
*** lpetrut has joined #openstack-cinder09:27
*** wanghao has quit IRC09:30
*** zsli__ has quit IRC09:30
*** wanghao_ has quit IRC09:31
*** vivsoni has quit IRC09:36
*** vivsoni has joined #openstack-cinder09:36
*** links has quit IRC09:44
*** arnewiebalck_ has joined #openstack-cinder09:45
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: (WIP) Add cinder backup service initialize check  https://review.openstack.org/44651809:51
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Add ability to specify backup driver via class name  https://review.openstack.org/47747209:52
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Replication, replacing SMI-S with REST  https://review.openstack.org/46602709:52
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Live Migration, replacing SMI-S with REST  https://review.openstack.org/47227909:55
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - implement Generic Volume Group feature  https://review.openstack.org/47454609:57
*** dalgaaf has quit IRC09:58
*** v12aml has quit IRC09:59
*** dalgaaf has joined #openstack-cinder10:00
*** iceyao has quit IRC10:00
*** v12aml has joined #openstack-cinder10:00
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - bootable volume errors  https://review.openstack.org/47267410:01
*** links has joined #openstack-cinder10:01
*** iceyao has joined #openstack-cinder10:03
*** Julien-zte has joined #openstack-cinder10:03
*** asbishop has joined #openstack-cinder10:06
*** iceyao has quit IRC10:07
*** vivsoni has quit IRC10:08
*** vivsoni has joined #openstack-cinder10:08
*** vivsoni has quit IRC10:12
*** vivsoni has joined #openstack-cinder10:12
*** boris-42__ has quit IRC10:13
*** Julien-zte has quit IRC10:15
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Do not review! Test LIO job fix  https://review.openstack.org/47925210:28
*** markstur has joined #openstack-cinder10:29
*** markstur has quit IRC10:33
*** zhurong has quit IRC10:37
*** stakeda has quit IRC10:39
*** arnewiebalck_ has quit IRC10:40
*** Poornima has joined #openstack-cinder10:45
*** iceyao has joined #openstack-cinder10:49
*** gkadam has joined #openstack-cinder10:50
*** arnewiebalck_ has joined #openstack-cinder10:56
*** bjolo has quit IRC10:58
*** bjolo has joined #openstack-cinder10:58
*** markstur has joined #openstack-cinder11:05
*** arnewiebalck_ has quit IRC11:08
*** markstur has quit IRC11:10
*** chenying has quit IRC11:11
*** chenying has joined #openstack-cinder11:12
*** reveller_ has joined #openstack-cinder11:21
*** reveller has quit IRC11:21
*** reveller_ is now known as reveller11:21
*** rtjure has joined #openstack-cinder11:24
*** gouthamr has quit IRC11:26
*** zhangguoqing has quit IRC11:29
*** psachin has quit IRC11:38
*** gkadam has quit IRC11:49
*** zhangguoqing has joined #openstack-cinder11:50
*** iceyao_ has joined #openstack-cinder11:50
*** iceyao has quit IRC11:51
*** vivsoni has quit IRC11:55
*** vivsoni has joined #openstack-cinder11:55
*** vivsoni_ has joined #openstack-cinder12:05
*** vivsoni has quit IRC12:06
*** liverpooler has joined #openstack-cinder12:13
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements  https://review.openstack.org/47791812:13
*** markstur has joined #openstack-cinder12:17
*** pgadiya has quit IRC12:19
*** markstur has quit IRC12:22
openstackgerritOpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext master: Updated from global requirements  https://review.openstack.org/47807012:24
*** Poornima has quit IRC12:28
*** salv-orlando has joined #openstack-cinder12:50
*** markstur has joined #openstack-cinder12:53
*** salv-orlando has quit IRC12:56
openstackgerritEric Harney proposed openstack/os-brick master: Don't use ignoreskipactivation for thin LVM  https://review.openstack.org/47929412:57
openstackgerritEric Harney proposed openstack/cinder master: LVM: Don't initialize VG with lvm_type='auto'  https://review.openstack.org/47929512:58
openstackgerritEric Harney proposed openstack/cinder master: Don't use ignoreskipactivation for thin LVM  https://review.openstack.org/47929612:58
*** markstur has quit IRC12:58
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: (WIP) Add cinder backup service initialize check  https://review.openstack.org/44651812:59
*** lucasxu has joined #openstack-cinder13:04
*** gcb has quit IRC13:07
openstackgerritMerged openstack/cinder master: import installation guide pages from openstack-manuals  https://review.openstack.org/47748913:08
*** Dinesh_Bhor has quit IRC13:09
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Compression, replacing SMI-S with REST  https://review.openstack.org/46019113:10
*** Julien-zte has joined #openstack-cinder13:11
*** lhx__ has quit IRC13:12
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Replication, replacing SMI-S with REST  https://review.openstack.org/46602713:13
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Live Migration, replacing SMI-S with REST  https://review.openstack.org/47227913:17
*** links has quit IRC13:20
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - implement Generic Volume Group feature  https://review.openstack.org/47454613:22
*** gkadam has joined #openstack-cinder13:31
*** salv-orlando has joined #openstack-cinder13:34
*** felipemonteiro has joined #openstack-cinder13:37
*** felipemonteiro_ has joined #openstack-cinder13:41
*** jmlowe has quit IRC13:41
*** salv-orlando has quit IRC13:43
*** felipemonteiro has quit IRC13:44
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - QoS, replacing SMI-S with REST  https://review.openstack.org/45675913:44
*** mriedem has joined #openstack-cinder13:45
*** lhx_ has joined #openstack-cinder13:50
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements  https://review.openstack.org/47791813:52
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Compression, replacing SMI-S with REST  https://review.openstack.org/46019113:55
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Replication, replacing SMI-S with REST  https://review.openstack.org/46602713:59
*** jmlowe has joined #openstack-cinder14:02
*** mriedem has quit IRC14:05
*** markstur has joined #openstack-cinder14:05
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Live Migration, replacing SMI-S with REST  https://review.openstack.org/47227914:06
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - implement Generic Volume Group feature  https://review.openstack.org/47454614:08
*** zhangguoqing has quit IRC14:09
*** markstur has quit IRC14:09
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - bootable volume errors  https://review.openstack.org/47267414:16
danpawlikhemna, jungleboyj, smcginnis, jbernard: Hello. I fixed pep8 errors and I hope I push last time the patch for this PS https://review.openstack.org/#/c/456672/14:21
danpawlikas always: if you have time please check  ^^14:21
danpawlikthanks!14:21
*** iceyao_ has quit IRC14:25
*** gouthamr has joined #openstack-cinder14:27
*** markstur has joined #openstack-cinder14:41
*** markstur has quit IRC14:46
*** iceyao has joined #openstack-cinder14:51
*** iceyao has quit IRC14:56
*** markstur has joined #openstack-cinder15:02
*** dansmith is now known as superdan15:04
*** rcernin has quit IRC15:11
*** zhaochao has quit IRC15:12
jungleboyjdanpawlik: ok, will take a look soon.15:13
danpawlikjungleboyj: thx :)15:25
*** lpetrut has quit IRC15:28
*** Julien-zte has quit IRC15:36
*** Julien-zte has joined #openstack-cinder15:36
*** e0ne has quit IRC15:39
*** Apoorva has joined #openstack-cinder15:55
*** diablo_rojo has joined #openstack-cinder16:02
*** mdavidson has quit IRC16:03
*** Apoorva_ has joined #openstack-cinder16:03
*** Apoorva has quit IRC16:03
*** nikeshm has joined #openstack-cinder16:07
*** kaisers_ has quit IRC16:09
*** kaisers_ has joined #openstack-cinder16:09
*** tesseract has quit IRC16:11
*** Apoorva_ has quit IRC16:13
*** kaisers_ has quit IRC16:13
*** Apoorva has joined #openstack-cinder16:14
*** jistr is now known as jistr|off16:21
*** dillaman has quit IRC16:32
*** kaisers_ has joined #openstack-cinder16:32
*** kaisers_ has quit IRC16:37
nikeshmhi16:44
nikeshmi am getting this "tempest.lib.exceptions.SSHTimeout: Connection to the 172.24.5.4 via SSH timed out"16:45
nikeshmin my CI16:45
nikeshmi saw some other CIs also gettung this errro16:45
*** kaisers_ has joined #openstack-cinder16:47
patrickeastnikeshm: could be a lot of things, just means it failed to ssh into some instance that tempest tried to boot and verify it was running16:49
patrickeastfor whatever reason it timed out reaching it16:49
openstackgerritChris M proposed openstack/cinder master: Fix exception in dothill iSCSI driver's terminate_connection  https://review.openstack.org/47856316:51
*** kaisers_ has quit IRC16:51
openstackgerritPatrick East proposed openstack/python-brick-cinderclient-ext master: Don't override brick's execute function  https://review.openstack.org/47937116:56
openstackgerritPatrick East proposed openstack/python-brick-cinderclient-ext master: Fix to allow multipathing with attach/detach  https://review.openstack.org/47937216:56
openstackgerritPatrick East proposed openstack/python-brick-cinderclient-ext master: Actually do the detach with the new attach/detach flow  https://review.openstack.org/47937316:56
patrickeastjgriffith: hemna ^16:56
patrickeastworks now... i think16:57
*** gkadam has quit IRC16:57
ildikovpatrickeast: nice, tnx :)16:58
*** lbragstad has quit IRC17:06
nikeshmpatrickeast: i am seeing this issue in many CIs, so curious to know any bug or issue17:07
*** lhx_ has quit IRC17:14
jgriffithnikeshm a lot of our CI's have been failing the last several days; I believe it's related to some of the placement changes that landed17:21
jgriffithI could be wrong, but quick glance last night that's what it sort of seemed like17:22
nikeshmjust correcting myself, i saw ssh timeout issue in few CIs  :)17:25
openstackgerritChris M proposed openstack/cinder master: Convert dothill driver to use distributed lock manager  https://review.openstack.org/47906917:27
*** dhinesh has joined #openstack-cinder17:31
openstackgerritEric Harney proposed openstack/cinder master: LVM: Don't initialize VG with lvm_type='auto'  https://review.openstack.org/47929517:35
openstackgerritEric Harney proposed openstack/cinder master: Don't use ignoreskipactivation for thin LVM  https://review.openstack.org/47929617:35
*** kaisers_ has joined #openstack-cinder17:42
*** pcaruana has quit IRC17:42
jgriffithanybody around that might be able to help with some policy/context issues I'm having?17:44
jgriffithsmcginnis ^^17:44
jgriffitheharney maybe?17:44
*** kaisers_ has quit IRC17:46
*** jwcroppe_ has joined #openstack-cinder17:46
*** jwcroppe has quit IRC17:49
openstackgerritPatrick East proposed openstack/python-brick-cinderclient-ext master: Don't override brick's execute function  https://review.openstack.org/47937117:49
openstackgerritPatrick East proposed openstack/python-brick-cinderclient-ext master: Actually do the detach with the new attach/detach flow  https://review.openstack.org/47937317:49
openstackgerritPatrick East proposed openstack/python-brick-cinderclient-ext master: Fix to allow multipathing with attach/detach  https://review.openstack.org/47937217:49
gouthamrkinda weird that 10-12 tests, all in tempest.api.compute are failing. other Volume attach tests go through fine17:53
gouthamrnikeshm if you manage to zero down on the issue, let us know... figuring out what changed.. the tests got an update 7 days ago.. but the failures are more recent...17:54
*** ChubYann has joined #openstack-cinder17:56
*** markstur has quit IRC18:18
openstackgerritPatrick East proposed openstack/cinder master: Fix image cache db query when limits are enabled  https://review.openstack.org/47939118:22
*** lpetrut has joined #openstack-cinder18:25
*** iceyao has joined #openstack-cinder18:46
*** kaisers_ has joined #openstack-cinder18:48
*** kaisers_ has quit IRC18:50
*** iceyao has quit IRC18:50
*** Apoorva_ has joined #openstack-cinder19:10
*** adisky__ has quit IRC19:11
*** Apoorva has quit IRC19:14
*** Apoorva_ has quit IRC19:15
openstackgerritChris M proposed openstack/cinder master: Remove dothill driver  https://review.openstack.org/47901819:16
*** e0ne has joined #openstack-cinder19:17
*** nikeshm has quit IRC19:21
*** rha has quit IRC19:24
*** rods has quit IRC19:24
*** john5223_ has quit IRC19:24
*** e0ne has quit IRC19:26
*** rods has joined #openstack-cinder19:27
*** mmotiani has joined #openstack-cinder19:27
*** rha has joined #openstack-cinder19:28
*** rha has quit IRC19:28
*** rha has joined #openstack-cinder19:28
openstackgerritMatt Smith (_alastor_) proposed openstack/cinder master: Datera 2.4.0 driver update  https://review.openstack.org/47354119:34
jgriffithpatrickeast thank you for removing the executor (or at least making it possible)19:39
*** e0ne has joined #openstack-cinder19:40
*** jmlowe has quit IRC19:46
*** iceyao has joined #openstack-cinder19:49
*** e0ne has quit IRC19:51
patrickeastjgriffith: heh, yea, that "feature" of os-brick is pretty broken now19:53
patrickeastused to be so that you could override it with some custom non-oslo thing, but its not really possible now that we use a special one that has some non-standard parameter requirements19:53
jgriffithyeah19:53
patrickeastjgriffith: ildikov: was there a fix somewhere for this? http://paste.openstack.org/show/614217/19:53
jgriffithit's always been kinda wonky for me though anyway19:54
*** iceyao has quit IRC19:54
patrickeastjgriffith: same, especially with all the privsep magic19:54
ildikovpatrickeast: smcginnis released brick with hemna's fix19:54
jgriffithpatrickeast huh, how did you make that happen?19:54
patrickeast:o19:54
patrickeasti just tried to attach a volume to an instance19:55
jgriffithoh.. yeah, I add "LIBS_FROM_GIT=os-brick" to my devstack just be default now19:55
patrickeasti am using tip of tree os-brick19:55
patrickeastfor all the multipath goodness19:55
patrickeastmebbe i need to refresh though if its a recent fix19:55
ildikovpatrickeast: I don't think it made it to global-requirements19:55
jgriffithcould be you found a new issue :(19:56
ildikovpatrickeast: https://review.openstack.org/#/c/478768/19:56
ildikovthat one should contain the target_lun fix19:57
ildikovhmm wait, that's a KeyError19:57
ildikovthat's something else19:57
ildikovdo you have the latest from everything?19:57
patrickeasthttps://www.irccloud.com/pastebin/r0QBRvuH/19:57
patrickeastwhich matches up with what got tagged at 1.1419:57
ildikovpatrickeast: brick casts target_lun to int19:58
patrickeastmmm maybe i did something dumb and didn't restart n-cpu after updating it though... one sec19:58
jgriffithpatrickeast nah, not brick related I don't think19:58
ildikovthe KeyError you see is something else than the error I had in mind first19:58
patrickeastoh19:58
jgriffithI don't think that *thing* exists in the attachment object19:58
* patrickeast actually reads scrollback19:58
patrickeastoh19:59
jgriffithdon't know if it's missing from object builder, view-builder etc etc19:59
jgriffiththat's a custom thing isn't it?19:59
ildikovthe current attach PoC casts target_lun too to int so you should have that one work with older brick too19:59
patrickeastso back in the day i think there was some debate about whether or not multipath devices should return "target_lun" *and* "target_luns"19:59
jgriffithahhh19:59
patrickeastiirc we only return the plural (list of luns)19:59
patrickeastlemme check19:59
SwansonOn the 4th be sure to keep the martian children in your hearts.  https://www.mediamatters.org/video/2017/06/29/alex-jones-guest-children-have-been-kidnapped-and-sent-slave-colony-mars/21710920:00
ildikovpatrickeast: what are you trying to do that had this error?20:01
patrickeastildikov: just a nova volume-attach20:02
patrickeasthave a volume and a running ephemeral vm20:02
patrickeastwant to connect em20:02
ildikovthat should work20:02
ildikov:/20:03
patrickeastwell... this might be the first ever attempt with multipathing20:04
patrickeastso good news maybe, i didn't waste all this time setting up an env to test it :D20:04
ildikovoh right, that can easily happen20:05
ildikovI've surely never tried it with multipathing :)20:06
*** jbrc has joined #openstack-cinder20:07
ildikovhow does it work with the current attach flow?20:08
*** liverpooler has quit IRC20:10
patrickeastildikov: drivers have the option to return a list of targets/portals/luns like so https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/pure.py#L178220:11
patrickeastildikov: and os-brick will use the lists to do the attachment, in non-mulitipath it can try discovery on each one until it finds a "live" one (sort of a half-way highly available multipath thing, but without multipathd and device-mapper involved)20:12
patrickeastildikov: or for multipath it connects to all of them at the same time20:12
ildikovpatrickeast: and is it fully hidden from Nova?20:12
patrickeastildikov: i dunno about hidden, but nova didn't look too carefully at the connection info given back from cinder before20:14
patrickeastildikov: was just part of the data blob20:14
ildikovpatrickeast: the error you copied in, Nova looks for target_lun only20:14
ildikovpatrickeast: not target_lun or target_luns20:14
patrickeastyea, thats the problem, it needs to look for either one20:15
patrickeastalthough i'm not sure why it needs to know which lun it is20:15
jgriffithpatrickeast yeah, but I can't figure out how that worked before :)20:15
* patrickeast is pulling up the nova code to poke around20:15
ildikovpatrickeast: ah, because of the int cast20:15
ildikovpatrickeast: and brick does the same in the latest version I believe20:16
patrickeastafaik it used to be back in the day nova got the reply from cinder, took the dict and shoveled it down the hole to os-brick20:16
patrickeastno inspecting the fields20:16
patrickeastso yea, if we are casting and playing with things in there we need to be more careful20:16
ildikovpatrickeast: but when we did the versioned or whatever object thing in Cinder for the new flow connection_info got converted into String's...20:17
ildikovpatrickeast: that's why we're looking into target_lun now as it has to be int and basic attach failed with that20:17
patrickeastdunno20:17
ildikovpatrickeast: in Nova it's just a temporary fix for the gate20:17
patrickeastwhy does it need to be an int?20:17
patrickeastafaik in brick we only use it as a string anyway20:18
ildikovpatrickeast: it's fixed in brick, the patch I copied earlier, but I think that doesn't look for target_luns either20:18
ildikovpatrickeast: no idea, I only remember the error that something somewhere cried it's not int... :S20:18
patrickeastgot a link to the bug or change for it?20:19
ildikovpatrickeast: https://review.openstack.org/#/c/468108/220:20
patrickeastunrelated, what in the world is a devstack-dsvm-updown-* job?20:20
ildikovlet me not guess :D20:21
patrickeastoh, so this maybe wasn't a thing before because afaik we didn't use ovo's until now :D20:22
ildikovyep20:22
ildikovovo was an "awesome" idea... :D20:22
patrickeastweird though that brick would care20:22
patrickeasthaha20:22
patrickeastso good20:22
patrickeastfixed everything, like taskflow20:22
ildikov:)20:23
patrickeastok so, its looking like if i go comment out that line in nova that does the conversion this should go away20:26
patrickeastsince brick has the fix in it to auto-convert things20:27
ildikovI added only as the brick release wasn't out yet and I wanted to see what's failing on the gate in tempest20:27
ildikovso from Nova it'll go20:27
patrickeastsweet, easy fix :D20:28
ildikovI hope you won't find any bigger problem than this :D20:28
patrickeastso far so good, pulling that line out and it seems to work ok20:29
*** vivsoni_ has quit IRC20:30
*** mvk has quit IRC20:38
*** lpetrut has quit IRC20:41
patrickeastyea with that change to both the connect and disconnect methods it seems to be working fine for the basic workflow (attach/detach)20:42
patrickeasti'll test more stuff out later... gotta run for the day20:42
ildikovawesome!20:43
ildikovthanks!20:43
openstackgerritChris M proposed openstack/cinder master: Convert dothill driver to use distributed lock manager  https://review.openstack.org/47906920:46
*** lucasxu has quit IRC20:47
*** mvk has joined #openstack-cinder20:49
*** iceyao has joined #openstack-cinder20:51
*** iceyao has quit IRC20:56
*** markstur has joined #openstack-cinder20:58
*** asbishop has quit IRC21:08
*** iceyao has joined #openstack-cinder21:12
*** edmondsw_ has quit IRC21:14
*** iceyao has quit IRC21:17
*** iceyao has joined #openstack-cinder21:33
*** iceyao has quit IRC21:37
*** felipemonteiro_ has quit IRC21:48
*** superdan is now known as dansmith21:55
*** iceyao has joined #openstack-cinder21:56
*** john5223_ has joined #openstack-cinder22:00
*** iceyao has quit IRC22:01
*** jbrc has quit IRC22:14
openstackgerritxing-yang proposed openstack/cinder master: Tiramisu: Add groups param to failover_host  https://review.openstack.org/47768822:18
*** iceyao has joined #openstack-cinder22:37
*** iceyao has quit IRC22:41
*** iceyao has joined #openstack-cinder22:42
*** iceyao has quit IRC22:47
*** iceyao has joined #openstack-cinder23:09
*** iceyao has quit IRC23:14
*** wanghao has joined #openstack-cinder23:14
*** iceyao has joined #openstack-cinder23:27
*** iceyao has quit IRC23:31
openstackgerritChris M proposed openstack/cinder master: Remove dothill driver  https://review.openstack.org/47901823:37
*** dhinesh has quit IRC23:42
*** iceyao has joined #openstack-cinder23:49
*** iceyao has quit IRC23:53

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