Tuesday, 2015-07-21

*** gouthamr has quit IRC00:01
*** gouthamr has joined #openstack-cinder00:01
*** dannywilson has quit IRC00:02
*** mylu has quit IRC00:02
*** mylu has joined #openstack-cinder00:03
*** DericHorn-HP has joined #openstack-cinder00:07
*** dims_ has joined #openstack-cinder00:10
*** annashen has joined #openstack-cinder00:10
*** garthb_ has quit IRC00:11
*** annashen has quit IRC00:14
*** amickus has quit IRC00:15
*** chlong has joined #openstack-cinder00:22
*** vokt has quit IRC00:22
patrickeastugh thats confusing, just noticed in volume/manager.py a handful of methods take a parameter called ‘context’ but there is an import named ‘context’ too00:25
patrickeastand apparently thats no problemo00:25
jgriffithpatrickeast: :)00:27
jgriffithpatrickeast: it's all in scoping/usage00:27
patrickeastjgriffith: haha yea, and now I know about it less of a problem00:27
jgriffithpatrickeast: :)00:27
jgriffithpatrickeast: it is confusing that we pass around a variable named "context" that is an object form the "context" lib :)00:28
patrickeastjgriffith: its a good name though, since it is the context00:29
patrickeastjgriffith: although one of them should probably be request_context00:29
jgriffithpatrickeast: haha... yeah, not sure what else we'd do with it :)00:29
jgriffithpatrickeast: hey... at least it's a context object and not something else :)00:29
patrickeastjgriffith: very true00:29
*** tsekiyama has quit IRC00:30
*** DericHorn-HP has quit IRC00:33
*** smoriya has joined #openstack-cinder00:33
*** piyanai has quit IRC00:33
*** DericHorn-HP has joined #openstack-cinder00:35
*** DericHorn-HP has quit IRC00:35
jgriffithmtreinish: Actually that's exactly what eharney had here: https://review.openstack.org/#/c/203845/1  My bad00:36
jgriffithmtreinish: LGTM00:36
*** Apoorva has quit IRC00:50
*** leeantho has quit IRC00:52
*** heyun has joined #openstack-cinder00:53
*** _cjones_ has quit IRC00:55
*** salv-orlando has quit IRC00:57
*** zhenguo has joined #openstack-cinder00:58
*** mylu has quit IRC00:58
*** salv-orlando has joined #openstack-cinder01:00
*** mylu has joined #openstack-cinder01:00
*** eharney has joined #openstack-cinder01:01
openstackgerritZhenguo Niu proposed openstack/python-cinderclient: Remove H302  https://review.openstack.org/20236401:04
*** lixiaoy11 has joined #openstack-cinder01:04
*** salv-orlando has quit IRC01:05
*** primechuck has quit IRC01:06
*** annashen has joined #openstack-cinder01:11
*** mtanino has quit IRC01:12
openstackgerritabhiram moturi proposed openstack/cinder: Enhance unit tests for zfssa drivers  https://review.openstack.org/20389501:15
*** annashen has quit IRC01:15
*** Apoorva has joined #openstack-cinder01:17
*** skylerberg has quit IRC01:19
*** mylu has quit IRC01:20
*** haomaiwang has quit IRC01:24
*** bitblt has quit IRC01:29
*** diogogmt has joined #openstack-cinder01:30
*** Apoorva has quit IRC01:32
*** raunak has quit IRC01:34
*** mylu has joined #openstack-cinder01:39
*** mylu has quit IRC01:40
*** jdandrea has quit IRC01:42
*** mylu has joined #openstack-cinder01:44
*** Apoorva has joined #openstack-cinder01:44
*** jasondotstar has joined #openstack-cinder01:45
*** Apoorva has quit IRC01:46
*** mylu has quit IRC01:47
*** mylu has joined #openstack-cinder01:47
*** vilobhmm1 has joined #openstack-cinder01:52
*** davechen has joined #openstack-cinder01:53
*** vilobhmm has quit IRC01:53
openstackgerritLiu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver  https://review.openstack.org/18836001:58
openstackgerritLiu Xinguo proposed openstack/cinder: Add smartX support for Huawei volume driver  https://review.openstack.org/18873201:59
*** vilobhmm1 has quit IRC01:59
*** davechen1 has joined #openstack-cinder02:00
openstackgerritLi Yingjun proposed openstack/cinder-specs: Add ability to update volume type public spec  https://review.openstack.org/20346202:01
*** primechuck has joined #openstack-cinder02:01
openstackgerritLiu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver  https://review.openstack.org/20140602:01
*** davechen has quit IRC02:02
*** sghanekar_ has quit IRC02:02
*** Lee1092 has joined #openstack-cinder02:03
openstackgerritwanghao proposed openstack/cinder: Improve performance listing detail for volumes created from image  https://review.openstack.org/20353002:06
*** Zhongjun_ has joined #openstack-cinder02:07
*** xyang1 has quit IRC02:08
*** xyang1 has joined #openstack-cinder02:09
*** xyang1 has quit IRC02:11
*** vilobhmm has joined #openstack-cinder02:11
*** vilobhmm has quit IRC02:12
*** vilobhmm has joined #openstack-cinder02:13
*** heyun has quit IRC02:18
*** Zhongjun_ is now known as Zhongjun02:28
*** vilobhmm has quit IRC02:30
*** gouthamr has quit IRC02:34
openstackgerritPatrick East proposed openstack/cinder: Generic image-volume cache  https://review.openstack.org/19579502:42
openstackgerritxing-yang proposed openstack/cinder-specs: Non Disruptive Backup  https://review.openstack.org/18689702:42
openstackgerritVincent Hou proposed openstack/cinder: Volume status management during migration  https://review.openstack.org/18631202:45
*** patrickeast has quit IRC02:47
*** jasondotstar has quit IRC02:47
*** annashen has joined #openstack-cinder02:51
*** hakimo_ has joined #openstack-cinder02:52
*** hakimo has quit IRC02:55
*** annashen has quit IRC02:55
*** mylu has quit IRC02:58
*** vilobhmm has joined #openstack-cinder03:02
*** dims_ has quit IRC03:09
*** dims_ has joined #openstack-cinder03:09
*** mylu has joined #openstack-cinder03:13
*** dims_ has quit IRC03:27
*** dims_ has joined #openstack-cinder03:28
*** diogogmt has quit IRC03:30
*** dims_ has quit IRC03:34
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/18961403:34
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/18961403:41
*** akshai has quit IRC03:44
*** vilobhmm has quit IRC03:44
openstackgerritTeruaki Ishizaki proposed openstack/cinder: Sheepdog: Add class for dog command executor  https://review.openstack.org/19453203:48
*** rakesh_mishra has joined #openstack-cinder03:48
*** garthb has joined #openstack-cinder03:51
*** annegent_ has joined #openstack-cinder04:04
*** dims_ has joined #openstack-cinder04:06
*** mylu has quit IRC04:06
*** annegentl_ has quit IRC04:07
*** mylu has joined #openstack-cinder04:07
openstackgerritVincent Hou proposed openstack/cinder: Volume status management during migration  https://review.openstack.org/18631204:08
*** dims_ has quit IRC04:10
*** rakesh_mishra has quit IRC04:11
*** dims_ has joined #openstack-cinder04:13
*** davechen1 has quit IRC04:13
*** BharatK has joined #openstack-cinder04:13
*** davechen has joined #openstack-cinder04:14
*** garthb has quit IRC04:15
*** dims_ has quit IRC04:18
*** lixiaoy11 has quit IRC04:21
*** mylu has quit IRC04:22
*** coolsvap|away is now known as coolsvap04:24
*** Apoorva has joined #openstack-cinder04:25
*** dims_ has joined #openstack-cinder04:27
*** lixiaoy1 has joined #openstack-cinder04:32
*** dims_ has quit IRC04:35
*** lixiaoy11 has joined #openstack-cinder04:37
*** lixiaoy1 has quit IRC04:39
*** erlon has quit IRC04:40
*** sgotliv has quit IRC04:44
*** shyama has joined #openstack-cinder04:45
openstackgerritLi Yingjun proposed openstack/python-cinderclient: Fixes table when there are multiline in result data  https://review.openstack.org/20391904:54
*** dims_ has joined #openstack-cinder05:02
*** dims_ has quit IRC05:06
*** bnemec has quit IRC05:07
*** chhavi has joined #openstack-cinder05:11
*** deepakcs has joined #openstack-cinder05:16
*** dims_ has joined #openstack-cinder05:22
*** pradipta has joined #openstack-cinder05:22
*** agarciam has quit IRC05:26
flip214hemna: ping05:26
*** dims_ has quit IRC05:26
*** nkrinner has joined #openstack-cinder05:27
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/18961405:27
*** shyama has quit IRC05:34
*** shyama has joined #openstack-cinder05:35
*** dims_ has joined #openstack-cinder05:36
*** bswartz has quit IRC05:37
*** markvoelker has quit IRC05:38
*** shyama has quit IRC05:40
*** dims_ has quit IRC05:41
*** vlaza has joined #openstack-cinder05:41
*** ronis has joined #openstack-cinder05:41
*** shyama has joined #openstack-cinder05:41
*** heyun has joined #openstack-cinder05:43
*** davechen has quit IRC05:44
*** markvoelker has joined #openstack-cinder05:44
*** shyama has quit IRC05:45
*** davechen has joined #openstack-cinder05:45
*** Apoorva has quit IRC05:47
*** sgotliv has joined #openstack-cinder05:48
*** Zhongjun has quit IRC05:48
*** bswartz has joined #openstack-cinder05:49
openstackgerritLi Yingjun proposed openstack/python-cinderclient: Fixes table when there are multiline in result data  https://review.openstack.org/20391905:49
*** IanGovett has joined #openstack-cinder05:52
openstackgerritShay Halsband proposed openstack/cinder: XtremIO support for iscsi discovery auth  https://review.openstack.org/18935805:53
*** primechuck has quit IRC05:59
*** IanGovett has quit IRC06:00
openstackgerritShay Halsband proposed openstack/cinder: XtremIO support for iscsi discovery auth  https://review.openstack.org/18935806:01
*** deepakcs has quit IRC06:04
*** BharatK has quit IRC06:04
*** deepakcs has joined #openstack-cinder06:04
*** pots_ has quit IRC06:07
*** pots has quit IRC06:07
*** pradipta has quit IRC06:08
*** pots has joined #openstack-cinder06:13
*** BharatK has joined #openstack-cinder06:16
*** BharatK_ has joined #openstack-cinder06:21
*** BharatK has quit IRC06:21
*** dims_ has joined #openstack-cinder06:21
openstackgerrithuangtianhua proposed openstack/python-cinderclient: Raise correct message for url version check  https://review.openstack.org/12367006:21
*** dims_ has quit IRC06:25
*** BharatK_ has quit IRC06:28
*** BharatK has joined #openstack-cinder06:29
*** shyama has joined #openstack-cinder06:33
*** salv-orlando has joined #openstack-cinder06:52
*** geguileo has joined #openstack-cinder06:55
*** dims_ has joined #openstack-cinder06:56
*** dims_ has quit IRC07:00
*** geguileo has quit IRC07:03
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/18961407:07
*** abehl has joined #openstack-cinder07:14
*** geguileo has joined #openstack-cinder07:16
*** IanGovett has joined #openstack-cinder07:19
*** s0nea has quit IRC07:20
openstackgerritVipin Balachandran proposed openstack/cinder: Move volume.api test to correct location  https://review.openstack.org/19863607:22
*** s0nea has joined #openstack-cinder07:22
*** topshare has quit IRC07:23
*** ankit_ag has joined #openstack-cinder07:24
openstackgerritLi Yingjun proposed openstack/python-cinderclient: Fixes table when there are multiline in result data  https://review.openstack.org/20391907:24
*** afazekas_ has joined #openstack-cinder07:24
*** topshare has joined #openstack-cinder07:24
*** yrabl is now known as yrabl_afk07:26
*** salv-orlando has quit IRC07:28
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/20395207:33
*** dims_ has joined #openstack-cinder07:37
openstackgerritAnton Arefiev proposed openstack/cinder: Fix 033 add encryption unique key migration  https://review.openstack.org/16540307:40
*** dims_ has quit IRC07:41
*** belmoreira has joined #openstack-cinder07:42
*** dalgaaf has quit IRC07:48
*** jordanP has joined #openstack-cinder07:50
*** belmoreira has quit IRC07:52
*** zhenguo has quit IRC07:53
*** zhenguo has joined #openstack-cinder07:53
*** jistr has joined #openstack-cinder07:55
openstackgerritchenying proposed openstack/cinder: Publishing service capabilities after volume is sucessfully cteated  https://review.openstack.org/20396107:58
*** laughterwym has joined #openstack-cinder08:02
*** chlong has quit IRC08:04
*** topshare has quit IRC08:05
*** topshare has joined #openstack-cinder08:05
openstackgerritLiu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver  https://review.openstack.org/18836008:07
*** lennyb_ has joined #openstack-cinder08:08
openstackgerritLiu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver  https://review.openstack.org/20140608:09
*** topshare has quit IRC08:09
*** topshare has joined #openstack-cinder08:09
*** topshare has quit IRC08:10
*** topshare has joined #openstack-cinder08:10
*** LiuXinguo has quit IRC08:13
*** salv-orlando has joined #openstack-cinder08:25
*** topshare has quit IRC08:29
*** bluex-pl has joined #openstack-cinder08:29
*** bluex-pl has quit IRC08:29
*** bluex-pl has joined #openstack-cinder08:30
*** dims_ has joined #openstack-cinder08:31
openstackgerritchenying proposed openstack/cinder: Publishing service capabilities after volume is sucessfully cteated  https://review.openstack.org/20396108:32
openstackgerritMichal Dulko proposed openstack/cinder: ConsistencyGroup Object  https://review.openstack.org/16364108:32
*** laughterwym has quit IRC08:34
*** geguileo has quit IRC08:34
*** topshare has joined #openstack-cinder08:35
*** dims_ has quit IRC08:36
*** 20WABQBTM has joined #openstack-cinder08:37
*** 20WABQBTM has quit IRC08:37
*** haomaiwang has joined #openstack-cinder08:37
*** geguileo has joined #openstack-cinder08:38
*** topshare has quit IRC08:38
*** topshare has joined #openstack-cinder08:42
*** aix has joined #openstack-cinder08:43
*** jaypipes has quit IRC08:46
*** dims_ has joined #openstack-cinder08:46
openstackgerritAnton Arefiev proposed openstack/cinder: Fix 033 add encryption unique key migration  https://review.openstack.org/16540308:47
*** chenying has joined #openstack-cinder08:50
*** dims_ has quit IRC08:51
*** xiangxinyong has joined #openstack-cinder08:53
*** shyama has quit IRC08:54
*** aix has quit IRC08:58
*** geguileo has quit IRC08:58
*** geguileo has joined #openstack-cinder08:59
*** topshare has quit IRC09:07
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/20395209:07
*** piyanai has joined #openstack-cinder09:08
*** e0ne has joined #openstack-cinder09:15
*** xiangxinyong has left #openstack-cinder09:17
*** topshare has joined #openstack-cinder09:22
*** topshare has quit IRC09:26
*** dims_ has joined #openstack-cinder09:26
*** aix has joined #openstack-cinder09:28
*** amick has joined #openstack-cinder09:29
*** dims_ has quit IRC09:31
*** dims_ has joined #openstack-cinder09:36
openstackgerritDeepak C Shetty (deepakcs) proposed openstack/cinder: Cinder part of volume snapshot improvements  https://review.openstack.org/17280809:39
openstackgerritDeepak C Shetty (deepakcs) proposed openstack/cinder: db: Add support for snapshot_admin_metadata  https://review.openstack.org/20397909:39
*** dims_ has quit IRC09:40
*** piyanai has quit IRC09:40
*** lpetrut has joined #openstack-cinder09:42
*** lpetrut has quit IRC09:47
*** davechen has left #openstack-cinder09:47
*** lixiaoy11 has quit IRC09:51
*** amick has left #openstack-cinder09:55
*** amickus has joined #openstack-cinder09:58
*** boris-42 has quit IRC10:02
*** geguileo has quit IRC10:03
*** geguileor has joined #openstack-cinder10:03
*** aix has quit IRC10:04
*** vlaza has quit IRC10:06
openstackgerritLiu Xinguo proposed openstack/cinder: Fix use of urllib for py3 compatibility in Huawei driver  https://review.openstack.org/20399610:06
*** Liu has joined #openstack-cinder10:06
openstackgerritLiu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver  https://review.openstack.org/20140610:13
*** geguileo has joined #openstack-cinder10:15
*** aix has joined #openstack-cinder10:15
*** asselin has quit IRC10:15
*** dims_ has joined #openstack-cinder10:16
*** geguileo has quit IRC10:17
*** akrog has joined #openstack-cinder10:17
*** geguileor has quit IRC10:17
*** geguileo has joined #openstack-cinder10:18
*** geguileo is now known as Guest7880910:19
*** akrog has quit IRC10:20
*** laughterwym has joined #openstack-cinder10:28
*** chenying has quit IRC10:34
openstackgerritMichal Dulko proposed openstack/cinder: Remove unused arguments from c-vol's create_volume  https://review.openstack.org/19959010:36
openstackgerritMichal Dulko proposed openstack/cinder: Refactoring of manager's create_volume flow  https://review.openstack.org/19316710:36
openstackgerritMichal Dulko proposed openstack/cinder: Refactoring of manager's create_volume flow  https://review.openstack.org/19316710:39
*** _afazekas has joined #openstack-cinder10:41
*** afazekas has quit IRC10:45
*** timcl has joined #openstack-cinder10:53
*** heyun has quit IRC10:55
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/20395211:00
*** lpetrut has joined #openstack-cinder11:05
*** coolsvap is now known as coolsvap|away11:12
*** vlaza has joined #openstack-cinder11:28
deepakcsHi all, is anyone also seeing the ceph CI job hitting timeout for the volume_backup_export_import testcase ?11:30
*** chhavi has quit IRC11:34
*** deepakcs has quit IRC11:37
*** gouthamr has joined #openstack-cinder11:39
*** salv-orlando has quit IRC11:40
*** omykhal has joined #openstack-cinder11:41
*** topshare has joined #openstack-cinder11:41
*** topshare has quit IRC11:43
*** deepakcs has joined #openstack-cinder11:44
omykhalhi, could somebody help me, please. I faced with a problem during cinder delete. This command (from volume.log) doesn`t work  - cinder-rootwrap /etc/cinder/rootwrap.conf lvremove --config activation { retry_deactivation = 1} devices { ignore_suspended_devices = 1} -f cindervg-thin/volume-xxxxxx11:45
omykhalit should be like this -  lvremove --config "activation { retry_deactivation = 1} devices { ignore_suspended_devices = 1}" -f cindervg-thin/volume-xxxxxxx11:46
omykhalcinder Kilo, lvm2-2.02.115-3.el7_1.1.x86_6411:46
omykhalhow to fix it in cinder?11:47
*** Guest78809 has quit IRC11:58
*** Guest78809 has joined #openstack-cinder11:59
*** Maike has joined #openstack-cinder12:01
*** salv-orlando has joined #openstack-cinder12:01
*** dims_ has quit IRC12:09
openstackgerritEdwin Wang proposed openstack/cinder: Add I/T mapping check for IBM FlashSystem  https://review.openstack.org/20361612:09
*** jungleboyj has joined #openstack-cinder12:10
*** Guest78809 has quit IRC12:18
*** chlong has joined #openstack-cinder12:18
*** ganso_ has joined #openstack-cinder12:20
*** Miouge has joined #openstack-cinder12:23
*** dalgaaf has joined #openstack-cinder12:23
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/20395212:23
*** bswartz has quit IRC12:31
*** chhavi has joined #openstack-cinder12:35
*** piyanai has joined #openstack-cinder12:51
*** akerr has joined #openstack-cinder12:55
*** smoriya has quit IRC12:57
*** BharatK has quit IRC12:58
*** piyanai has quit IRC13:00
*** kjnelson has joined #openstack-cinder13:02
*** annegentl_ has joined #openstack-cinder13:04
*** annegent_ has quit IRC13:06
*** Maike has quit IRC13:08
*** dustins has joined #openstack-cinder13:09
*** jdandrea has joined #openstack-cinder13:11
*** jwcroppe has quit IRC13:13
*** jwcroppe has joined #openstack-cinder13:13
*** markvoelker has quit IRC13:17
openstackgerritPh. Marek proposed openstack/cinder: DRBD: Rename a constant to a better name, uppercase-only.  https://review.openstack.org/20364613:17
*** jwcroppe has quit IRC13:18
*** timcl has quit IRC13:19
*** deepakcs has quit IRC13:19
*** piyanai has joined #openstack-cinder13:19
*** julim has joined #openstack-cinder13:19
*** simondodsley has joined #openstack-cinder13:20
*** ankit_ag has quit IRC13:22
*** mylu has joined #openstack-cinder13:24
*** Yogi1 has joined #openstack-cinder13:25
*** thangp has joined #openstack-cinder13:26
*** Yogi1 has quit IRC13:30
*** xyang1 has joined #openstack-cinder13:33
openstackgerritYaguang Tang proposed openstack/cinder: Remove simple scheduler which is deprecated since Juno  https://review.openstack.org/20285813:35
*** timcl has joined #openstack-cinder13:37
*** annegentl_ has quit IRC13:39
*** mylu has quit IRC13:39
*** mdenny has joined #openstack-cinder13:44
*** changbl has quit IRC13:44
smcginnisomykhal: Still around?13:46
*** mriedem has joined #openstack-cinder13:46
smcginnisomykhal: Looks like the line "'config', LVM_CONFIG" would need to be changed to "'config "%s"' % LVM_CONFIG".13:47
smcginnisomykhal: In brick/local_dev/lvm.py line 668.13:47
smcginnisomykhal: https://github.com/openstack/cinder/blob/master/cinder/brick/local_dev/lvm.py#L66813:47
*** piyanai has quit IRC13:48
smcginnisomykhal: I'm not too up on the LVM functionality, but if you confirm that's an issue I can push up a patch to fix it.13:48
*** bnemec has joined #openstack-cinder13:48
*** edmondsw has joined #openstack-cinder13:49
eharneyit seems unlikely that --config and all the opts should be in the same arg?13:50
smcginniseharney: Unless everything in the {} should be passed in as the config value? Don't really know.13:50
*** abehl has quit IRC13:51
eharneyi'm not too sure either, but what's there now looks sensible to me13:51
eharneyand equivalent to what omykhal said it should be above?13:52
smcginniseharney: I would be kind of surprised if it's been there and no one noticed it was a problem.13:52
smcginnisThat line was last changed June of 2014.13:52
*** mylu has joined #openstack-cinder13:53
smcginnisI suppose it wouldn't be a record for a bug not being noticed, but I would still be surprised.13:53
*** dims_ has joined #openstack-cinder13:53
eharneyit may be confusion that the quotes aren't printed in the log, because they aren't needed there13:53
smcginniseharney: Oh, good point!13:54
*** diogogmt has joined #openstack-cinder13:54
smcginnisWait, no, we don't quote it, so it's not that they just aren't being logged.13:55
eharneywe don't have to quote it, we aren't running it in a shell13:55
eharneyit's passed as a single arg13:56
*** mriedem has left #openstack-cinder13:57
*** jungleboyj has quit IRC13:58
*** piyanai has joined #openstack-cinder13:58
*** markus_z has joined #openstack-cinder13:59
*** vlaza has quit IRC14:02
*** crose has joined #openstack-cinder14:02
*** amickus has quit IRC14:04
*** breitz has quit IRC14:06
*** breitz has joined #openstack-cinder14:06
jgriffithomykhal: eharney smcginnis Would be good to see some more detail on why that's believed to be failing.  We haven't had any issues with it for the past year or so and the expansion should be fine in the execute method.14:06
jgriffithomykhal: what Operating System/Version are you using?  And are you actually seeing an error in Cinder, or are you trying to copy/paste out the call and run it locally?14:07
jgriffithor "manually" I guess14:07
*** mriedem has joined #openstack-cinder14:10
*** diogogmt has quit IRC14:10
omykhalthis is RHEL 7, there was update from icehouse to kilo. When I try to use cinder delete <volume>, an error in volume.log appears : oslo_messaging.rpc.dispatcher Command: sudo cinder-rootwrap /etc/cinder/rootwrap.conf lvremove --config activation { retry_deactivation = 1} devices { ignore_suspended_devices = 1} -f cindervg-thin/volume-13775c44-7fce-4f58-a474-619a8946376614:12
omykhaloslo_messaging.rpc.dispatcher Stderr: u'  device-mapper: message ioctl on  failed: Invalid argument\n  Failed to resume cindervg-thin-pool.\n  Failed to update pool cindervg-thin/cindervg-thin-pool.\n'14:12
*** amoturi has joined #openstack-cinder14:12
*** jasondotstar has joined #openstack-cinder14:13
*** avishay has joined #openstack-cinder14:15
*** topshare has joined #openstack-cinder14:16
*** bswartz has joined #openstack-cinder14:17
*** lpetrut has quit IRC14:18
jgriffithomykhal: ahh... thank you for the clarification14:18
jgriffithomykhal: If you pastebin the full trace and log I'll log a bug and we can get it fixed up.  Unless you were interested in working on it yourself?14:19
openstackgerritAnton Arefiev proposed openstack/cinder: Add drivers list generator  https://review.openstack.org/19275114:19
jgriffithomykhal: although it sounds like smcginnis was aleady interested in looking at it as well.  I'm not sure that config variable is the culprit still14:20
*** mtanino has joined #openstack-cinder14:22
aarefievjgriffith: hi, I've addressed your comments https://review.openstack.org/#/c/192751/11 , please look when you have time14:22
jgriffithaarefiev: cool... but what about the other two issues:14:23
jgriffith1. Cinder has to be loaded on the system you run it on14:23
jgriffith2. Listing the parent/base modules as drivers in the resultant list14:24
*** diogogmt has joined #openstack-cinder14:24
aarefievjgriffith: hm 1) I thought you are talking about tox env,14:25
aarefievjgriffith: It is how genconfig works14:25
*** laughterwym has quit IRC14:26
jgriffithaarefiev: well, just copying what I posted in the comment into the tox.ini was the easy part14:26
eharneymtreinish: i think i want a regex like this?   http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/devstack-gate.yaml#n226814:26
mtreinisheharney: yeah that looks like the right thing14:26
jgriffithaarefiev: perhaps you should try and run that on a system that doesn't have Cinder installed yourself. (as well as other folks)14:26
*** primechuck has joined #openstack-cinder14:26
mtreinisheharney: fwiw I tested the one I left in the review comment with a testr list-tests, but just double check it :)14:26
eharneymtreinish: cool, thanks14:26
eharneymtreinish: yeah, glad to see that, i actually didn't know how to test it14:27
jgriffithaarefiev: http://paste.openstack.org/show/395905/14:27
*** topshare has quit IRC14:27
openstackgerritKyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient  https://review.openstack.org/19764314:27
mtreinisheharney: yeah using a regex with testr list-tests is something that for whatever reason no many people know about14:28
aarefievjgriffith: hm it's strange, could you try on clean tox venv, please?14:29
jgriffithaarefiev: that was a clean tox env :)14:29
aarefievjgriffith: it works for me14:29
jgriffithaarefiev: Oh?  hmmm14:30
jgriffithaarefiev: are you running a devstack machine?14:30
jgriffithaarefiev: or on your laptop/workstation without devstack/cinder installed?14:30
*** david-lyle has quit IRC14:30
aarefievjgriffith: no, it should get modules from venv14:30
aarefievjgriffith: let me check14:30
jgriffithaarefiev: yes, I know it "should" :)14:31
jgriffithaarefiev: and I haven't looked at the duplicate huawei option yet14:31
*** Yogi1 has joined #openstack-cinder14:35
jgriffithLooks like not testing genconfig has bitten us once again14:36
jgriffithaarefiev: so it's unrelated to your work, but the config declarations are broken again.  Huawei and IBM so far, maybe others14:37
*** diogogmt has quit IRC14:37
*** annegentle has joined #openstack-cinder14:37
jgriffithhaha... novaclient14:37
aarefievjgriffith: print full path to modules /home/openstack/cinder/cinder/volume/drivers/block_device.pyc14:38
aarefievit generates from tree14:38
jgriffithaarefiev: huh?14:39
*** nkrinner has quit IRC14:39
aarefievjgriffith: /home/openstack/cinder/cinder/volume/drivers/block_device.pyc - it is full path to one of the module14:40
aarefiev/home/openstack/cinder/ - cinder source14:40
*** piyanai has quit IRC14:42
jgriffithaarefiev: hmm... interesting14:43
*** lpabon has joined #openstack-cinder14:44
aarefievjgriffith: about parent/base, you mean it shouldn't be in list, right?14:45
*** Miouge has quit IRC14:45
aarefievjgriffith: for example common14:46
aarefievjgriffith: baseVD<-common<-dr1 , baseVD<-common<-dr214:47
jgriffithaarefiev: so on that topic, correct14:47
jgriffithaarefiev: I'm still trying to understand why I can't run this on a system with just the tox env14:48
aarefievjgriffith: I'll try on clean env14:49
jgriffithaarefiev: I got it working14:49
*** Miouge has joined #openstack-cinder14:49
jgriffithaarefiev: I changed the file perms to 755 and just call it bare14:49
*** lpetrut has joined #openstack-cinder14:50
jgriffithaarefiev: I'm unclear on what that matters...  trying to reproduce again14:50
jgriffithaarefiev: hmm.. that's weird14:50
*** krtaylor has quit IRC14:50
jgriffithaarefiev: if I just cherry pick in your patch it barfs14:51
jgriffithaarefiev: but if I duplicate it EXACTLY in my master branch it works fine14:51
jgriffithaarefiev: not cherry-pick... review -d14:51
jgriffithOh... wait14:51
*** Miouge has quit IRC14:51
jgriffithaarefiev: hmm... ok, well that's weird14:52
jgriffithaarefiev: Ok... so that seems to be working fine now14:53
jgriffithaarefiev: back to the other issue, with the base classes14:53
*** Miouge has joined #openstack-cinder14:53
aarefievjgriffith: hmm strange14:53
jgriffithaarefiev: yeah, it's really odd14:54
aarefievjgriffith: yes about base classes14:54
jgriffithOh... ya know what I think it might be...14:54
jgriffithmy pip-cache14:54
jgriffithSo I pulled master, and ran tox on it.  That would update my cache14:54
jgriffithfor what gets installed when I run your patch14:55
*** Miouge has quit IRC14:55
jgriffithaarefiev: wonder when the last time you rebased your patch was?14:55
jgriffithaarefiev: anyway... let's figure out what to do about the base class stuff14:56
jgriffithaarefiev: for example: 'cinder.volume.drivers.pure.PureBaseVolumeDriver',14:56
jgriffith 'cinder.volume.drivers.pure.PureFCDriver',14:56
jgriffith 'cinder.volume.drivers.pure.PureISCSIDriver',14:56
aarefievjgriffith: I don't remember )14:56
jgriffithaarefiev: That PurBaseVolumeDriver is an example of what it would be cool if we were smart enough to ignore14:56
aarefievjgriffith: I don't see how to make it cool14:57
jgriffithaarefiev: :)14:57
*** jasonsb has joined #openstack-cinder14:58
jgriffithaarefiev: ok, well maybe to start, we just do it the brute force way14:58
jgriffithaarefiev: generate the list and keep a list of base classes that we pop out of the list after the run14:59
aarefievjgriffith: I think we can't identificate it automatically14:59
jgriffithaarefiev: or even put a special character to denote they're not an "actual" driver14:59
jgriffithaarefiev: I think we can make it automatic but we might need to add some helpers in the code to make it work, which I won't ask for you to do in the first iteration of the patch15:00
*** jwcroppe has joined #openstack-cinder15:00
aarefievjgriffith: I mean now, maybe abc abstract classes15:00
jgriffithaarefiev: yeah, we really need to clean up our driver structure a bit15:01
jgriffithaarefiev: it's kinda messy15:01
jgriffithaarefiev: ok... well so in that case; your patch looks good :)15:01
aarefievjgriffith: agree on that15:01
jgriffithaarefiev: thanks for adding that to tox.ini for me15:01
*** timcl has quit IRC15:02
aarefievI mean about clean up ))15:02
*** Yogi11 has joined #openstack-cinder15:02
jgriffithaarefiev: I'll leave it up to you if you want to rebase it and change perms to call the tool directly15:02
*** krtaylor has joined #openstack-cinder15:02
*** Yogi1 has quit IRC15:02
jgriffithaarefiev: LOL... yeah,  although I'm sure you also agree that your patch is good :)15:02
aarefievjgriffith: I'll rebase, what do you mean change perms to call the tool directly?15:03
openstackgerritAnton Arefiev proposed openstack/cinder: Add drivers list generator  https://review.openstack.org/19275115:05
jgriffithaarefiev: I modified your patch, so the fox.ini call is just:  commands = {toxinidir}/tools/generate_driver_list.py15:05
jgriffithaarefiev: so I made the perms on the script 755... but that is REALLY NOT important15:06
jgriffithaarefiev: it doesn't matter at all15:06
aarefievjgriffith: ok, I've rebased15:07
jgriffithaarefiev: ok, +2/A15:07
jgriffithaarefiev: it had the required number of votes prior to my request and the rebase15:07
jgriffithaarefiev: thanks a lot for putting that together15:07
*** afazekas_ has quit IRC15:08
*** diemt has quit IRC15:08
aarefievjgriffith: thank you, I'll think about filtering base classes15:08
*** diemt has joined #openstack-cinder15:09
*** mriedem has quit IRC15:09
*** edmondsw has quit IRC15:09
*** chlong has quit IRC15:09
*** IanGovett has quit IRC15:09
*** eharney has quit IRC15:09
*** swamireddy has quit IRC15:09
*** jamielennox has quit IRC15:09
*** kambiz has quit IRC15:09
*** briancline has quit IRC15:09
*** tbarron has quit IRC15:09
*** diogogmt has joined #openstack-cinder15:13
*** edmondsw has joined #openstack-cinder15:16
*** crose has quit IRC15:16
*** chlong has joined #openstack-cinder15:16
*** eharney has joined #openstack-cinder15:19
*** eharney is now known as Guest851115:19
*** dannywilson has joined #openstack-cinder15:19
*** Guest8511 is now known as eharney15:20
*** eharney has quit IRC15:20
*** eharney has joined #openstack-cinder15:20
*** dannywilson has quit IRC15:21
*** dannywilson has joined #openstack-cinder15:21
*** yuriy_n17 has joined #openstack-cinder15:22
yuriy_n17jgriffith: Hi! Please review https://review.openstack.org/#/c/203054/. It fixes incorrect version for hacking package on stable/kilo15:23
*** afazekas has joined #openstack-cinder15:25
*** rushil has quit IRC15:26
*** jdurgin1 has joined #openstack-cinder15:31
jgriffithyuriy_n17: If you could, would you mind putting a reference to the parent in the commit message: https://github.com/openstack/requirements/commit/4568bfb6c841130b2edb3202d796b1fe05aec70f15:31
jgriffithyuriy_n17: just something like "updates from global-requirements change: <short-sha>"15:32
*** kambiz has joined #openstack-cinder15:33
*** bluex-pl has quit IRC15:33
yuriy_n17jgriffith: Thanks a lot15:34
*** rushil has joined #openstack-cinder15:34
*** primechuck has quit IRC15:35
*** david-lyle has joined #openstack-cinder15:38
*** ronis has quit IRC15:40
*** changbl has joined #openstack-cinder15:41
openstackgerritShay Halsband proposed openstack/cinder: XtremIO support for iscsi discovery auth  https://review.openstack.org/18935815:43
*** timcl has joined #openstack-cinder15:43
*** Yogi11 has quit IRC15:44
yuriy_n17jgriffith: Hi! Please review https://review.openstack.org/#/c/203054/. Thanks.15:48
*** Yogi1 has joined #openstack-cinder15:50
*** jwitko has joined #openstack-cinder15:51
*** vokt has joined #openstack-cinder15:51
jwitkoHey Guys, When starting cinder I am getting an error with the initialization of the netapp driver.  It is complaining of failed basic auth.   http://pastebin.com/raw.php?i=vBSCxtBQ.   However I have double-checked the credentials in the cinder.conf and can SSH using them without an issue (unsure of how to validate on netapp for HTTP basic auth.  I can provide the basic auth to root15:52
jwitkourl and receive a 302.) My cinder conf can be seen here:  http://pastebin.com/raw.php?i=hWWQDRQy15:52
jwitkoCan anyone help me out?15:52
*** Yogi1 has quit IRC15:53
dustinsjwitko: Having a look now15:54
jwitkoty dustins15:54
dustinsjwitko: No problem!15:54
*** NobodyCam has joined #openstack-cinder15:54
dustinsjwitko: Is this with master branch or on some other release?15:55
*** vokt has quit IRC15:56
*** jasonsb has quit IRC15:56
jwitkodustins, Kilo.  I'm using Openstack Ansible Deployment  for deployment15:56
*** vokt has joined #openstack-cinder15:56
akerrjwitko: dustins: want to move to #openstack-netapp so we don't spam all of cinder?15:56
dustinsakerr: But I was having so much fun! :)15:57
dustinsYeah, let's move over there15:57
e0nedo we have any plans about multibackend tests on gates?15:58
*** annegentle has quit IRC15:59
*** jungleboyj has joined #openstack-cinder16:00
jgriffithe0ne: we have it as an option, IIRC there was a brief period where we were doing it16:01
e0nejgriffith: how can i start such jobs?16:02
jgriffithe0ne: everything is in place in devstack16:02
jgriffithe0ne: just use the enabled_backends directives in your conf file.  I'm not certain where we landed on having it enabled in gate etc; or if there's a ton of value16:02
*** leeantho has joined #openstack-cinder16:03
e0nejgriffith: i know that devstack has such option16:03
e0nejgriffith: i would like to have it on gates by default16:04
*** jistr has quit IRC16:04
*** ociuhandu_ has joined #openstack-cinder16:04
jgriffithe0ne: sure... curious; why?16:04
e0nejgriffith: i'm hitting with the last ceph backup issue16:04
*** avishay has quit IRC16:04
jgriffithe0ne: IMHO that's sort of a "specialized" config option, it doesn't really bother me much if it's not tested16:04
*** crose has joined #openstack-cinder16:05
e0nejgriffith: we've got incremental backup tested only16:05
jgriffithe0ne: well, that's just a case of something we never tested in the past that we just turned on16:05
e0nejgriffith: full backup is not tested16:05
jgriffithe0ne: yeah, and given how long full backups take, please don't :)16:05
e0nejgriffith: and looks like ist's broken since kilo16:05
e0nejgriffith: hope, i'm wrong16:06
jgriffithe0ne: yeah, I don't think the gate should be a total crutch though16:06
e0nejgriffith: full backup is not much slower than secure delete16:06
jgriffithe0ne: I think folks interested in backup and Ceph maintainers should do at least "some" testing :)16:06
jgriffithe0ne: and we don't do secure delete in the gate any longer either16:07
jgriffithe0ne: I'm open minded about it of course.16:07
e0nejgriffith: oh.. it's good :)16:08
smcginnisAnyone know the current situation with the ceph gate failures?16:08
jgriffithe0ne: if people are interested in making it happen that's cool16:08
jgriffithsmcginnis: yes16:08
jgriffithsmcginnis: a new test was turned on in tempest that does import backups16:08
*** pots has left #openstack-cinder16:08
*** ociuhandu_ has quit IRC16:08
jgriffithsmcginnis: the ceph driver is not working for that case16:08
smcginnisjgriffith: I haven't had time to look back through the log. Is someone working on clearing that up?16:08
jgriffithsmcginnis: eharney has submitted a temporary skip of that test for ceph-dsvm16:09
smcginnisjgriffith: OK, awesome. Thanks!16:09
eharneyit just merged a minute ago16:09
jgriffithsmcginnis: https://bugs.launchpad.net/cinder/+bug/147673516:09
openstackLaunchpad bug 1476735 in Cinder "check failure in gate-tempest-dsvm-full-ceph; failure to import backup" [High,Confirmed]16:09
smcginniseharney: Thanks for that.16:09
jgriffithsmcginnis: I updated that a bit and added some extra info16:09
jgriffitheharney: awesome!  Thanks for putting that together so quick yesterday16:10
e0nejgriffith: it's a hard desicion: more quick tests  or a full test suit to run on gates16:10
jgriffithe0ne: meh... like I said, I think that things like backup suffer from a lack of real interest/use16:10
e0nejgriffith: agree. i don't remember any customer that was asking about it:(16:11
jgriffithe0ne: anyway.. my point was just that the gate shouldn't be the ONLY place things are tested IMHO16:12
*** jdurgin1 has quit IRC16:12
jgriffithe0ne: although if I/We ever get anything going with cinder/tests/functional that would be a good place for this sort of thing maybe16:12
*** _cjones_ has joined #openstack-cinder16:13
*** _cjones_ has quit IRC16:13
*** _cjones_ has joined #openstack-cinder16:14
Swansonsmcginnis: waiting for the puppet apply of that patch.16:14
e0nejgriffith: yes, i'll be happy to have functional tests too16:14
xyang1jgriffith: Does the functional test use a real backend?16:15
e0nejgriffith: we've got a lot of features/options whis are tested only by unit tests :(16:15
jgriffithxyang1: the would yes16:15
jgriffithxyang1: right now they don't exist :)16:15
jgriffithxyang1: I need to put some stuff together to get the ball rolling16:16
xyang1jgriffith: The backup was actually tested by LVM and I assume swift backup driver16:16
jgriffithxyang1: but don't want to start something else that I won't finish :)16:16
jgriffithxyang1: yes... it is16:16
xyang1jgriffith: :)16:16
jgriffithxyang1: and that's all good16:16
jgriffithxyang1: so the issue is that the import backup feature hasn't been tested in the gate; it was just added as a tempest test16:17
jgriffithxyang1: LVM works fine, ceph however found issues with it16:17
xyang1jgriffith: Ok, ceph is different16:17
jgriffithxyang1: yes, very much so16:17
*** lpabon has quit IRC16:17
jgriffithxyang1: BTW https://review.openstack.org/#/c/201812/10/cinder/volume/driver.py16:18
jgriffithxyang1: I agree about the inconsistency16:18
jgriffithxyang1: I would actually prefer to make all of them raise NotImplemented if you have no objection16:18
xyang1jgriffith: no problem16:19
jgriffithxyang1: cool16:19
*** _cjones_ has quit IRC16:20
*** asselin_ has quit IRC16:20
*** yuriy_n17 has quit IRC16:21
*** _cjones_ has joined #openstack-cinder16:24
*** mgarza has joined #openstack-cinder16:25
openstackgerritLiu Xinguo proposed openstack/cinder: Add smartX support for Huawei volume driver  https://review.openstack.org/18873216:26
* e0ne bb in ~3 hours16:27
openstackgerritLiu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver  https://review.openstack.org/20140616:31
*** krtaylor has quit IRC16:32
*** asselin has joined #openstack-cinder16:34
*** david-lyle has quit IRC16:34
*** Apoorva has joined #openstack-cinder16:36
*** jwang has quit IRC16:40
*** amick has joined #openstack-cinder16:40
*** sghanekar_ has joined #openstack-cinder16:43
*** e0ne has quit IRC16:45
*** annashen has joined #openstack-cinder16:47
*** annashen has quit IRC16:47
*** annashen has joined #openstack-cinder16:47
*** david-lyle has joined #openstack-cinder16:48
*** lpetrut has quit IRC16:50
*** tbarron has joined #openstack-cinder16:51
*** krtaylor has joined #openstack-cinder16:52
*** patrickeast has joined #openstack-cinder16:54
*** jasondotstar has quit IRC16:56
*** julim_ has joined #openstack-cinder17:03
*** dansmith has quit IRC17:05
*** dansmith has joined #openstack-cinder17:05
*** julim has quit IRC17:05
*** dansmith has quit IRC17:05
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Fix Consistency Group issues  https://review.openstack.org/20168517:06
*** dansmith has joined #openstack-cinder17:06
*** jordanP has quit IRC17:09
jwitkoHey guys I have cinder-volume and cinder-scheduler running with no noticable issues in the logs however it does not seem to be listening on the default port 877617:11
jwitkoanyone have any ideas?17:11
*** jasondotstar has joined #openstack-cinder17:16
*** gman-tx has quit IRC17:17
*** lpabon has joined #openstack-cinder17:17
*** gman-tx has joined #openstack-cinder17:17
*** raunak has joined #openstack-cinder17:18
*** ronis has joined #openstack-cinder17:18
jwitkoi do see the python process has two open ports,  44837 4483617:18
*** vilobhmm has joined #openstack-cinder17:22
*** gman-tx has quit IRC17:22
openstackgerritRaunak Kumar proposed openstack/cinder: Add extra spec capability for Nimble Cinder Driver  https://review.openstack.org/19885317:23
*** aix has quit IRC17:23
*** avishay has joined #openstack-cinder17:23
jwitkosorry those are outgoing connections17:24
*** tbarron has quit IRC17:26
*** piyanai has joined #openstack-cinder17:27
*** avishay has quit IRC17:28
*** mylu has quit IRC17:28
*** bswartz has quit IRC17:29
*** mylu has joined #openstack-cinder17:30
*** patrickeast_ has joined #openstack-cinder17:31
*** tbarron has joined #openstack-cinder17:31
*** mylu has quit IRC17:32
*** mylu has joined #openstack-cinder17:32
*** zhenguo has quit IRC17:33
*** Guest78809 has joined #openstack-cinder17:34
*** annegentle has joined #openstack-cinder17:36
*** patrickeast_ has quit IRC17:38
*** annegent_ has joined #openstack-cinder17:39
*** annegentle has quit IRC17:41
*** piyanai has quit IRC17:47
openstackgerritRajini Ram proposed openstack/cinder: Eqlx Driver: Revised the ssh code to use common utils  https://review.openstack.org/20372017:47
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder-specs: Brick add new Connector volume APIs  https://review.openstack.org/20418017:48
*** bitblt has joined #openstack-cinder17:48
*** mylu has quit IRC17:50
*** bitblt has quit IRC17:51
*** bitblt has joined #openstack-cinder17:51
*** annegent_ has quit IRC17:51
*** jasondotstar has quit IRC17:52
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Add new Connector APIs for path validation  https://review.openstack.org/19976417:53
*** markvoelker has joined #openstack-cinder17:55
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Fix Consistency Group issues  https://review.openstack.org/20168517:55
*** bitblt has quit IRC17:58
*** bitblt has joined #openstack-cinder17:58
*** mylu has joined #openstack-cinder17:59
*** erlon has joined #openstack-cinder18:01
*** piyanai has joined #openstack-cinder18:02
*** annegentle has joined #openstack-cinder18:02
*** jaypipes has joined #openstack-cinder18:02
*** boris-42 has joined #openstack-cinder18:02
openstackgerritKendall Nelson proposed openstack/cinder: Enable cinder-manage to remove old services from the database  https://review.openstack.org/19761318:03
*** mylu has quit IRC18:06
*** lpabon has quit IRC18:08
*** mylu has joined #openstack-cinder18:09
openstackgerritAnthony Lee proposed openstack/os-brick: WIP: Add new Connector APIs for path validation  https://review.openstack.org/19976418:09
openstackgerritAnthony Lee proposed openstack/os-brick: Adding a tool that allows querying of LUNs  https://review.openstack.org/20419118:09
*** diemt has quit IRC18:10
*** angela-s has joined #openstack-cinder18:12
*** skylerberg has joined #openstack-cinder18:13
*** tbarron_ has joined #openstack-cinder18:14
*** lpetrut has joined #openstack-cinder18:14
*** jamielennox has joined #openstack-cinder18:15
*** jamielennox is now known as jamielennox|away18:16
*** david-lyle has quit IRC18:21
*** jamielennox|away is now known as jamielennox18:23
*** swamireddy has joined #openstack-cinder18:23
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder-specs: Brick add new Connector volume APIs  https://review.openstack.org/20418018:27
*** bswartz has joined #openstack-cinder18:30
*** annegentle has quit IRC18:30
openstackgerritRyan Lucio proposed openstack/cinder: VMEM v7000: external-head storage pool and volume type selection  https://review.openstack.org/19577318:35
jgriffithhemna: the reason I was asking was that usually "flask-script" means you have the full framework of a flask server, shell and the web app itself.18:36
jgriffithhemna: That sounded to me like a "service" which is fine, just was trying to understand18:36
jgriffithhemna: as opposed to "cinder check-paths <volume-uuid>"  --> brick.initiator:foo.get_path_info(<volume-uuid>)18:37
*** piyanai has quit IRC18:37
*** diogogmt_ has joined #openstack-cinder18:38
*** diogogmt has quit IRC18:39
*** diogogmt_ is now known as diogogmt18:39
jgriffithhemna: and more appropriately I guess that gets added as a Cinder call that can be made from the Nova side as well.  I'm not clear on what info in Brick already has this info though18:40
*** piyanai has joined #openstack-cinder18:42
*** piyanai has quit IRC18:42
hemnaso, I'm not exactly sure where the 'right' place is for that flask script to live to be honest.18:43
jgriffithhemna: yeah... I'm not sure I really understand it, so that's why I asked18:43
hemnaI don't want brick to be a service18:43
jgriffithhemna: it seems like that requires implementing a server, manager, api and the whole deal18:43
hemnait's a library18:43
*** amick has quit IRC18:44
jgriffithhemna: ok, cool... I didn't think so but thought I should ask.18:44
hemnabut we do need some way of verifying on remote systems that paths have correctly gone away (re: live migration, rebuild) testing.18:44
jgriffithhemna: I mean, it might be fine if it were... but anyway.18:44
jgriffithhemna: sure, that's what I said in the comment; I totally think that's a great idea18:44
hemnathe flask thing is just something we are chewing on and I put it in the spec to try and start the discussion and get feedback18:44
jgriffithhemna: got ya18:45
hemnamy initial plan was to put it in a tools dir in os-brick18:45
jgriffithhemna: so you were saying brick already has all this info... where is that in brick?18:45
*** mylu has quit IRC18:45
jgriffithhemna: so are you opposed to the idea of actually making it a method in the lib that could be called by its consumer?18:46
hemnaso that 'info' is usually internally available inside each of the connector's connect_volume18:46
hemnaand then thrown away18:46
hemnaafter connect_volume is done18:46
jgriffithhemna: what if we held on to it and allowed it to be queried?18:46
hemnamy patch basically refactors connect_volume and makes the paths available as a method call, which connect_volume also uses.18:46
jgriffithhemna: just in memory18:46
hemnamy patch actually makes those method calls on each of the connectors18:47
hemnathe flask script was just a mechanism to make it remotely available18:47
jgriffithhemna: wait.. which method calls?18:47
jgriffithhemna: Oh... you already have a patch up that implements all of this?18:47
hemnathe get_volume_paths(), get_all_available_volumes.18:47
hemnaI already have a patch up that does all of this18:47
jgriffithhemna: Oh, I'm sorry.. I didn't see it18:48
hemnaand leeantho has a patch for the flask script.18:48
hemnait's been a WIP for a while18:48
jgriffithhemna: I'll go look instead of asking stupid questions18:48
hemnahrmm, I thought I removed 'WIP' from the title, but it's fine that it's in there until the spec is approved anwyway18:48
jgriffithhemna: yah, so this seems pretty straight forward to just make a cinder-api call and a nova api call that can access this info18:49
hemnaeven if it's just for testing purpose?18:49
hemnadisable via policy I guess ?18:49
jgriffithhemna: well, I don't see why it wouldn't be useful for "other" internal cleanup and such18:50
jgriffithhemna: and perhaps admin policy even if you wanted to go crazy with it :)18:50
hemnaok, I hadn't thought of that18:50
jgriffithhemna: but it looks like you only implemented the filesystem shares side of it18:51
jgriffithhemna: if this is just a simple iscsiadm call then you can even just put a direct call in the tempest test that runs on the node18:51
hemnaI don't know what to do for rbd18:51
*** mylu has joined #openstack-cinder18:51
jgriffithhemna: me neither :)  But I don't see iSCSI either?18:52
openstackgerritCurt Bruns proposed openstack/cinder: Add deactivate step to extend_lv  https://review.openstack.org/19881318:52
hemnaI have it for the ISCSIConnector18:52
*** jasondotstar has joined #openstack-cinder18:52
*** mylu has quit IRC18:52
hemnathe parent class implements the get_all_available_volumes()18:52
jgriffithhemna: https://review.openstack.org/#/c/199764/5/os_brick/initiator/connector.py18:53
jgriffithpass ?18:53
jgriffithI see it18:53
hemnaI'm not in a big hurry to land this, we can talk about it at the mid cycle18:54
hemnaI'm trying to think of adding something to our 3rd party CI to show it being used18:54
hemnaas a separate standalone test for the time being.18:54
hemnaI'd actually like to see if all of our tests are not leaving behind orphaned paths. :P18:55
*** s0nea has quit IRC18:55
jgriffithhemna: they are FWIW18:56
*** lpetrut has quit IRC18:56
jgriffithhemna: well.. not "all" of them but there are some left behind after every tempest run18:56
hemnahopefully, with this, we can pinpoint which tests actually leave the paths behind18:56
hemnaand those tests, should be marked as failure18:57
jgriffithhemna: yeah, I was thinking you'd just add a call to this in the cleanup18:57
*** s0nea has joined #openstack-cinder18:57
jgriffithhemna: Oh.. it just uses the old iscsiadm18:59
*** agarciam has joined #openstack-cinder18:59
jgriffithhemna: so it checks if the target is there... where does it actually try and nuke the connection if the target doesn't respond?18:59
hemnathe connect_volume code first builds a list of possible paths that might show up, and then loops looking for one of them.19:00
hemnaI just moved that code out and made it available to call basically19:00
*** chenying has joined #openstack-cinder19:00
jgriffithhemna: it's kinda tricky no?  I mean, the check will basically create a new connection if the target is reachable I think.19:01
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: WIP: Add new Connector APIs for path validation  https://review.openstack.org/19976419:01
hemnathat connection should already be there, if the volume is attached19:01
hemnaif it's not attached.....19:01
*** lpetrut has joined #openstack-cinder19:01
hemnaif it's not attached, then this call shouldn't be made.  not sure how to check if the volume is locally attached w/o doing iscsiadm work19:02
hemnareview is very welcomed though! :)19:02
jgriffithhemna: nahh... I wasn't saying that at all (not use iscsiadm)19:03
hemnaI hadn't thought about a non-existing connection (not attached volume)19:03
*** breitz has quit IRC19:03
*** mylu has joined #openstack-cinder19:03
*** e0ne has joined #openstack-cinder19:04
hemnaI guess iscsciadm -m session dumps a list of existing sessions19:05
jgriffithhemna: well... my point was you can't really use "_connect_to_iscsi_portal" to check orphaned connections, because it's going to create connections for the targets you call it on19:05
hemnacan probably parse that output and see if there is one for the target_portals in the connection_info passed in.19:06
hemnayah, agreed.19:06
e0nethingee: hi Mike. should i bring topic 'Make gate-rally-dsvm-cinder job voting for Cinder patches' again into the meeting or you can +1 on patch (https://review.openstack.org/#/c/203680/).19:06
jgriffithhemna: yes, you'd need to separate out the session call from that method19:06
e0nethingee: there're stats in a commit message19:06
*** jwang has joined #openstack-cinder19:06
hemnaI'll have to first make sure there is an existing session to a target_portal, and then build a list of connected target_portals to query19:06
hemnaok cool.  thanks!19:06
jgriffithhemna: sure19:07
*** breitz has joined #openstack-cinder19:08
e0nejgriffith: do you have any update with API for ironic integration/attach w/o nova?19:12
e0nejgriffith: i'm goint to implement cli tool for it this week19:13
jgriffithe0ne: crud... I need to write that real quick19:13
jgriffithe0ne: we still need to sync up on that19:13
e0nejgriffith: any working poc will be useful19:13
jgriffithe0ne: make sure I understand what you're expecting from the consumer side19:13
jgriffithe0ne: I was just going to expose an API call that gets the connection info.. that's all19:13
e0nejgriffith: sure. can we do it tomorrow? /cc hemna19:13
jgriffithe0ne:  sounds good19:14
jgriffithe0ne: I'll try to have some code ready19:14
*** amit213 has quit IRC19:14
e0nejgriffith: great!19:14
*** amit213 has joined #openstack-cinder19:14
jgriffithe0ne: hemna I was basically going to just expose a version of intiialize_connection19:15
*** Liu has quit IRC19:15
*** chenying has quit IRC19:15
jgriffithe0ne: hemna or even just modify policy so it can be used as is currently without anything else needed per-say19:15
hemnadoes that include the reserve_volume call internally?19:15
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Fix legacy bug, init_conn bug and REST API bug.  https://review.openstack.org/20421719:16
hemnaok lunch......19:16
jgriffithhemna: yeah... a "glob" of the two19:17
jgriffithhemna: and lunch is a great idea19:17
*** breitz has quit IRC19:17
*** jasondotstar has quit IRC19:18
*** breitz has joined #openstack-cinder19:18
*** mylu has quit IRC19:20
*** chhavi has quit IRC19:20
*** mylu has joined #openstack-cinder19:21
*** dustins_ has joined #openstack-cinder19:21
*** mylu has quit IRC19:22
*** lpetrut has quit IRC19:23
*** Yogi1 has joined #openstack-cinder19:23
*** mylu has joined #openstack-cinder19:24
*** dustins has quit IRC19:25
*** annegentle has joined #openstack-cinder19:25
vilobhmmericsonsantos : ping19:28
vilobhmmericksonsantos : ^^19:28
*** david-lyle has joined #openstack-cinder19:34
*** annegentle has quit IRC19:36
*** annegent_ has joined #openstack-cinder19:37
*** piyanai has joined #openstack-cinder19:38
*** mylu has quit IRC19:45
*** mylu has joined #openstack-cinder19:47
*** sghanekar_ has quit IRC19:49
*** tsekiyama has joined #openstack-cinder19:52
*** edtubill has joined #openstack-cinder19:52
*** piyanai has quit IRC19:55
*** piyanai has joined #openstack-cinder19:55
*** bitblt has quit IRC19:58
thingeehemna: preparing os-brick 0.3.119:59
openstackgerritxing-yang proposed openstack/python-cinderclient: CLI: Non-disruptive backup  https://review.openstack.org/20423220:00
edmondswjungleboyj, please look at the changes I made per your comments in https://review.openstack.org/#/c/203237/20:00
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Fix legacy bug, init_conn bug and REST API bug.  https://review.openstack.org/20421720:00
*** e0ne has quit IRC20:02
*** mylu has quit IRC20:03
*** Lee1092 has quit IRC20:05
thingeehemna: https://review.openstack.org/20423620:06
thingeemtanino: going to get to the capabilities patch today, thanks for your patience20:07
mtaninothingee: sure. Waiting for your comments. Thank you20:07
mtaninothingee: seems still you are hard woking...20:08
thingeemtanino: ha I don't think so, but thanks20:08
thingeemtanino: I feel like I've been missing around reviews and IRC lately because of personal stuff.20:09
*** skylerberg has quit IRC20:09
mtaninothingee: liberty-2 is comming week. welcome back:)20:10
*** Guest78809 has quit IRC20:11
*** tbarron has quit IRC20:13
*** tbarron_ is now known as tbarron20:13
*** amoturi has quit IRC20:15
SwansonCheck is just a hellscape lately.20:16
ericksonsantosvilobhmm, sorry, wasn't here20:16
*** sghanekar_ has joined #openstack-cinder20:18
hemnathingee, thanks20:26
*** e0ne has joined #openstack-cinder20:27
*** s0nea has quit IRC20:28
*** s0nea has joined #openstack-cinder20:32
*** amick has joined #openstack-cinder20:35
openstackgerritThang Pham proposed openstack/cinder: Continue conversion to volume object  https://review.openstack.org/20140420:36
patrickeasti have a question about retyping…. it looks like if the backends are not going to change it will call the drivers retype method https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L171020:37
patrickeastand then if the driver did retype it then we don’t try and migrate20:37
patrickeastbut for a backend like pure where we don’t really do anything for retypes, is it then ok to just implement retype and return true20:37
*** _cjones_ has quit IRC20:38
*** lpetrut has joined #openstack-cinder20:38
*** _cjones_ has joined #openstack-cinder20:38
smcginnispatrickeast: I think you just don't implement it then.20:39
smcginnispatrickeast: That's how we had it before because we didn't have anything we could do.20:39
patrickeastsmcginnis: i mean… it works if i dont implement it20:39
*** piyanai has quit IRC20:39
patrickeastsmcginnis: but then it will do a slow migrate to make a new volume of the right type20:39
patrickeastsmcginnis: instead of doing nothing, which would also be valid20:39
smcginnispatrickeast: But you don't have any other way to get to the new type, right?20:39
patrickeastsmcginnis: well thats the thing… as far as our backend is concerned there is no difference20:40
smcginnispatrickeast: What is being retyped then?20:40
patrickeastsmcginnis: we don’t have any knobs or dials that change on the volume based on type20:40
patrickeastsmcginnis: cinder meta-data about the volume basically20:40
smcginnispatrickeast: Then it would have to be to another backend, even if the other backend is another Pure array.20:41
patrickeastsmcginnis: this came up because someone was trying to change the cinder qos stuff20:41
smcginnispatrickeast: So the slow migration is the only way to get there.20:41
patrickeastwhich doesn’t actually change anything on our backend20:41
smcginnispatrickeast: Ah, QoS...20:41
*** lpetrut has quit IRC20:41
patrickeastbut even if they had two types20:41
smcginnispatrickeast: In that case I believe you can just check that that is the only thing changing and return true.20:41
patrickeastlike flash and tier-120:41
patrickeastor whatever20:41
patrickeastand the same backend fulfilled the types20:41
patrickeastwe would just want to do a no-op20:42
SwansonYeah.  If you don't want the volume to move tell cinder you did it.20:42
smcginnispatrickeast: As long as it is something your driver can take care of with the same backend I think you just update the metadata and return true.20:42
*** thangp has quit IRC20:42
smcginnispatrickeast: Anything else you return false and cinder does the migrate.20:42
patrickeastbut it looks like the driver is only called if the volume wiont move20:42
smcginnispatrickeast: Right, only if the retype is to the same backend.20:43
patrickeastsmcginnis: ok, i think i got it20:43
patrickeastsmcginnis: Swanson: thanks!20:43
smcginnispatrickeast: It confused me at first too. :)20:43
*** piyanai has joined #openstack-cinder20:45
*** dustins_ has quit IRC20:45
*** changbl has quit IRC20:46
*** e0ne has quit IRC20:48
*** ronis has quit IRC20:49
*** amick has left #openstack-cinder20:53
*** daneyon has quit IRC20:54
*** amick has joined #openstack-cinder20:54
*** amick is now known as amickus20:55
*** julim_ has quit IRC20:55
*** agarciam has quit IRC20:56
*** daneyon has joined #openstack-cinder20:56
*** daneyon_ has joined #openstack-cinder21:00
*** Guest78809 has joined #openstack-cinder21:01
*** dustins has joined #openstack-cinder21:02
*** bswartz has quit IRC21:03
*** daneyon has quit IRC21:04
*** piyanai has quit IRC21:04
*** daneyon has joined #openstack-cinder21:04
*** simondodsley has quit IRC21:06
*** daneyon_ has quit IRC21:07
*** xyang1 has quit IRC21:11
*** annegent_ has quit IRC21:12
*** e0ne has joined #openstack-cinder21:14
openstackgerritTom Barron proposed openstack/cinder: POC for scalable backup  https://review.openstack.org/20329121:15
*** e0ne has quit IRC21:15
*** pv_ has joined #openstack-cinder21:16
*** e0ne has joined #openstack-cinder21:16
*** dustins has quit IRC21:18
patrickeastjgriffith: hey, have you had a chance to look at the cache stuff anymore and/or my comments on the internal tenant stuff?21:21
*** jasondotstar has joined #openstack-cinder21:22
*** Yogi1 has quit IRC21:23
*** jungleboyj has quit IRC21:31
*** e0ne has quit IRC21:33
*** annegentle has joined #openstack-cinder21:34
*** sghanekar_ has quit IRC21:35
*** annegentle has quit IRC21:35
*** annegent_ has joined #openstack-cinder21:35
*** jasondot_ has joined #openstack-cinder21:36
*** jasondotstar has quit IRC21:39
jgriffithpatrickeast: well, I'm kinda stuck between what your'e doing and the stuff added to Glance now21:42
jgriffithpatrickeast: there's an awful lot of overlap there it seems21:42
patrickeastjgriffith: yea i agree21:43
patrickeastjgriffith: i actually went back and took a hard look at whether or not we still needed a cache or anything21:43
jgriffithpatrickeast: and? :)21:43
patrickeastjgriffith: and i think the answer is still yes, primarily because of the image formats21:43
jgriffithpatrickeast: Oh?  Interesting.. why is that?21:44
patrickeastjgriffith: even if cinder is a glance backend if its got a bunch of qcow2 images we can’t just clone em21:44
patrickeastjgriffith: we still need to cache raw versions21:44
jgriffithpatrickeast: oh...sure21:44
jgriffithpatrickeast: but I guess what I was proposing is that the cache is obsolete21:44
patrickeastjgriffith: and i went down the path of putting those raw ones into glance as another url21:44
*** piyanai has joined #openstack-cinder21:44
jgriffithpatrickeast: if you can configure glance to use Cinder as one of 'n' backends, you use templates in Glance of the images you care about21:45
patrickeastjgriffith: right, but that only gives you fast volume creation from images if they are stored in glance as raw21:45
patrickeastjgriffith: maybe faster than right now21:46
patrickeastjgriffith: but not insta-clone kind of fast21:46
patrickeastjgriffith: unless i’m misunderstanding something21:46
jgriffithpatrickeast: so what I was thinking was an adaptation... that if configured, we always create the raw image on a Cinder volume in Glance....21:46
patrickeastjgriffith: so keep both?21:46
jgriffithpatrickeast: then when a request to download an image comes from Cinder that is to download to a volume, we just auto-magically use the raw one21:47
patrickeastjgriffith: or how would you then take one and upload it to a compute node or something for a non-cinder usage?21:47
jgriffithpatrickeast: it just moves what you're working on a bit further put the stack21:47
openstackgerritRaunak Kumar proposed openstack/cinder: Add extra spec capability for Nimble Cinder Driver  https://review.openstack.org/19885321:47
patrickeastjgriffith: yea i’m with you there, that works great for glance images configued as raw21:47
jgriffithpatrickeast: right now you can't really... and frankly I don't care about that so much :)21:47
patrickeastjgriffith: but thats the eventual goal21:48
jgriffithpatrickeast: maybe my idea doesn't work.  I'll take another look at both patches and see what I can figure out21:48
patrickeastjgriffith: and the direction that code is going21:48
patrickeastjgriffith: so… i’m totally open to ditching the cache if its redundant or totally a subset of the other functionality21:49
patrickeastjgriffith: i’m just not quite seeing it yet21:49
patrickeast/me might be slow21:49
*** tater_ has joined #openstack-cinder21:49
tater_Hello All, I've got an issue where the rabbitmq queue is filling up from cinder_scheduler_ fannout. The queue never empties. I'm unable to crate an instance as create from image (new volume)21:51
tater_I see this in the cinder logs: Availability zone 'myzonename' is invalid21:52
tater_Thanks for any input21:52
openstackgerritKendall Nelson proposed openstack/cinder: Enable cinder-manage to remove services  https://review.openstack.org/19761321:52
openstackgerritAnthony Lee proposed openstack/os-brick: Adding a tool that allows querying of LUNs  https://review.openstack.org/20419121:53
jgriffithpatrickeast: nah... it might not work21:53
jgriffithjust an idea21:53
jgriffithpatrickeast: i.e. the glance thing might not work21:53
*** kjnelson has quit IRC21:53
patrickeastjgriffith: bummer21:53
*** piyanai has quit IRC21:53
patrickeastjgriffith: i do like the idea of a single code path kind of solution21:53
jgriffithpatrickeast: yeah, I'm mixed on that TBH21:54
patrickeastjgriffith: and it might be ok we just keep two copies of every image on the cinder backend21:54
patrickeastone raw, one original21:54
jgriffithpatrickeast: I mean, I like single code path, but with enough flexibility for the drivers to do what they need to do21:54
jgriffithpatrickeast: and frankly own the responsibility of making it work21:54
patrickeastjgriffith: true21:55
jgriffithpatrickeast: I looked at just doing a Cinder cache like that, but it was kinda useless... you still had to do the dd to a volume21:55
jgriffithpatrickeast: for some, that's the best you get, but for others that's not the case21:55
jgriffithpatrickeast: which is why I ended up just doing it all in the driver; but I think your approach still has a middle ground21:56
patrickeastjgriffith: yea, i guess thats the hard part… everyone is going to have their own secret sauce21:56
jgriffithI just don't want to have duplicate stuff out there21:56
patrickeastjgriffith: same21:56
patrickeastjgriffith: i refactored my stuff to re-use as much as i could, both use the same volume creation code and all that21:56
jgriffithpatrickeast: right... which is why I ended up just doing it by overriding that call.21:56
jgriffithpatrickeast: Yeah, honestly I haven't looked at your patch in a while, let me look at it when I get back to my desk21:57
jgriffith10 minutes or so (I hope)21:57
patrickeastjgriffith: sounds good21:58
patrickeastjgriffith: thanks again for helping me get this figured out21:58
jgriffithpatrickeast: well... sadly I'm not sure I'm being much help at this point :(21:58
patrickeastjgriffith: haha, your providing a good sort of check to make sure i’m still doing sane things and not wasting time21:59
patrickeastjgriffith: very useful imo21:59
pv_hey quick question, what should I set the CINDER_BRANCH field as in my local.conf for devstack? my driver code I'm trying to test isn't a part of any official branch yet21:59
patrickeastpv_: if you have a local or internal git repo you can override CINDER_REPO and CINDER_BRANCH to pull your code for you22:00
*** jaypipes has quit IRC22:00
patrickeastpv_: or just let it use master and then go put your code in after devstack has started up22:00
*** markus_z has quit IRC22:00
*** kragniz has quit IRC22:01
*** sghanekar_ has joined #openstack-cinder22:01
*** piyanai has joined #openstack-cinder22:01
*** edtubill has left #openstack-cinder22:01
*** dims_ has quit IRC22:01
*** kragniz has joined #openstack-cinder22:02
*** jwcroppe has quit IRC22:02
*** jwcroppe has joined #openstack-cinder22:03
*** pv_ has quit IRC22:03
*** jgriffith has quit IRC22:04
*** jgriffith has joined #openstack-cinder22:05
*** bnemec has quit IRC22:06
*** edmondsw has quit IRC22:07
*** jwcroppe has quit IRC22:07
*** daneyon_ has joined #openstack-cinder22:09
*** daneyon has quit IRC22:12
openstackgerritAnthony Lee proposed openstack/os-brick: Adding a tool that allows querying of LUNs  https://review.openstack.org/20419122:15
*** akerr has quit IRC22:18
*** ganso_ has quit IRC22:18
patrickeastasselin: hey, i’m going to switch my ci to use the latest stuff from your repo/upstream puppet modules, just got my new project-config created… just wanted to check with you, as-is it should be working, right?22:18
*** rhagarty has quit IRC22:19
patrickeastasselin: or are there any patches in flight i need to snag too?22:19
asselinpatrickeast, only one thing missing right now is the zuul smtp settings22:19
asselinthose just merged earlier22:19
asselinupstream, need to push that to my master22:19
patrickeastok cool22:20
asselinotherwise go for it. I updated all of mine last week22:20
*** rhagarty_ has quit IRC22:20
patrickeastawesome, everything working well?22:20
patrickeastjust the other day ours decided to stop releaing floating ip addresses : /22:21
patrickeasti’m hoping an upgrade to the latest everything will help22:21
*** rhagarty has joined #openstack-cinder22:21
*** piyanai_ has joined #openstack-cinder22:22
thingeemtanino: I think there are some additional things I would like to see with your get capabilities patch. I think we can agree on the API though that you have in place with the driver class though and RPC related bits22:22
thingeemtanino: if that allows you to move forward22:22
*** piyanai has quit IRC22:23
*** piyanai_ is now known as piyanai22:23
mtaninothingee: thank you for the review. Please comment on the review. I will fix and post updated patch.22:23
asselinpatrickeast, yes, but that's nodepool which is not yet part of this.22:23
*** sgotliv has quit IRC22:24
patrickeastasselin: hmm thats a good point22:24
*** mgarza has quit IRC22:24
thingeemtanino: do you have any opinion with these capabilities being reported to glance's metadata catalog?22:24
patrickeastasselin: oh well, upgrading anyway!22:24
thingeemtanino: see dulek's comment https://review.openstack.org/#/c/183947/22:25
*** rhagarty_ has joined #openstack-cinder22:25
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Adding a tool that allows querying of LUNs  https://review.openstack.org/20419122:26
mtaninothingee: this one? http://paste.openstack.org/show/281270/22:27
mtaninothingee: I agree on his comment to use same format of Glance Metadata Catalog format.22:28
mtaninothingee: Enable to reuse Horizon widget is reasonable, I think22:28
thingeemtanino: are we able to use the format without having to store the information in glance?22:30
thingeemtanino: or do you think it makes sense to store it in glance?22:30
tsekiyamathingee: I believe dulek's suggestion is just for reusing the same Horizon code to parse the glance metadata catalog for the capabilities format22:31
tsekiyamathingee: it's not to store the information into Glance22:31
thingeetsekiyama: ah ok, wasn't sure if they were mutually exclusive22:32
mtaninothingee: In my patchset, capabilities is still blank dictionary. If we decide the specification, I will post follow up patch for LVM and NFS as a reference implementation.22:36
asselinpatrickeast, ok merged the 1 liner for smtp settings22:43
patrickeastasselin: sweet22:44
asselinpatrickeast, i'll be on vacation coming up, so let me know of any issues22:44
patrickeastasselin: will do, i’m just about ready to run the install script… getting some stuf straightened out with my wrapper scripts22:45
*** mtanino has quit IRC22:49
*** haomaiwang has quit IRC22:49
*** crose has quit IRC22:52
*** piyanai has quit IRC22:53
*** angela-s has quit IRC22:54
*** krypton has joined #openstack-cinder23:04
*** pv_ has joined #openstack-cinder23:05
*** krypton has left #openstack-cinder23:06
*** krypton_ has joined #openstack-cinder23:10
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/20133623:12
*** bswartz has joined #openstack-cinder23:12
*** jasondot_ has quit IRC23:12
patrickeastasselin: aww, ran into an error right off the bat http://paste.openstack.org/show/397471/23:14
* asselin looks23:14
patrickeastasselin: i’m not sure i understand what puppet is trying to say, line 94 of master.pp is http://paste.openstack.org/show/397471/ and mentions nothing about ssl cert contents23:14
patrickeasterr https://github.com/rasselin/os-ext-testing/blob/master/puppet/modules/os_ext_testing/manifests/master.pp#L9423:14
asselinpatrickeast, update your install_master.sh script23:15
asselinshould probably add a note about that23:15
patrickeastohh funny23:15
patrickeastat some point i had my wrapper script point to the one in os-ext-testing23:16
patrickeasti guess i changed it back23:16
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/20089823:16
*** jasondotstar has joined #openstack-cinder23:17
*** diogogmt has quit IRC23:17
*** DericHorn-HP has joined #openstack-cinder23:18
patrickeastasselin: much better now23:18
asselinI probably should add those in for better backwards compatibility/migration23:19
pv_hey so i updated the CINDER_REPO to my local git repo and CINDER_BRANCH to master23:19
pv_and my stack.sh is giving me an error and exiting saying git call failed23:19
pv_for git fetch origin23:19
pv_any suggestions on why?23:19
patrickeastpv_: means it probably couldn’t clone the repo correctly23:20
patrickeastpv_: try manually cloning it with the same commands stack.sh is using23:20
patrickeastpv_: and make sure that works23:20
patrickeastpv_: you can see what its doing in the source here https://github.com/openstack-dev/devstack/blob/master/functions-common#L50223:21
*** annashen has quit IRC23:22
*** boris-42 has quit IRC23:22
*** jungleboyj has joined #openstack-cinder23:24
pv_thanks ill check it out23:25
*** dannywilson has quit IRC23:26
*** david-lyle has quit IRC23:28
*** jungleboyj has quit IRC23:36
*** tater_ has quit IRC23:38
*** jasondotstar has quit IRC23:47
*** jasondotstar has joined #openstack-cinder23:50
*** ctracey has quit IRC23:53
*** topshare has joined #openstack-cinder23:53
*** hemna is now known as hemnafk23:53
*** tbarron has quit IRC23:54
*** ctracey has joined #openstack-cinder23:55
*** tsekiyam_ has joined #openstack-cinder23:55
*** tsekiyama has quit IRC23:58

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