| *** gouthamr has quit IRC | 00:01 | |
| *** gouthamr has joined #openstack-cinder | 00:01 | |
| *** dannywilson has quit IRC | 00:02 | |
| *** mylu has quit IRC | 00:02 | |
| *** mylu has joined #openstack-cinder | 00:03 | |
| *** DericHorn-HP has joined #openstack-cinder | 00:07 | |
| *** dims_ has joined #openstack-cinder | 00:10 | |
| *** annashen has joined #openstack-cinder | 00:10 | |
| *** garthb_ has quit IRC | 00:11 | |
| *** annashen has quit IRC | 00:14 | |
| *** amickus has quit IRC | 00:15 | |
| *** chlong has joined #openstack-cinder | 00:22 | |
| *** vokt has quit IRC | 00:22 | |
| patrickeast | ugh thats confusing, just noticed in volume/manager.py a handful of methods take a parameter called ‘context’ but there is an import named ‘context’ too | 00:25 |
|---|---|---|
| patrickeast | and apparently thats no problemo | 00:25 |
| jgriffith | patrickeast: :) | 00:27 |
| jgriffith | patrickeast: it's all in scoping/usage | 00:27 |
| patrickeast | jgriffith: haha yea, and now I know about it less of a problem | 00:27 |
| jgriffith | patrickeast: :) | 00:27 |
| jgriffith | patrickeast: it is confusing that we pass around a variable named "context" that is an object form the "context" lib :) | 00:28 |
| patrickeast | hah | 00:28 |
| patrickeast | jgriffith: its a good name though, since it is the context | 00:29 |
| patrickeast | jgriffith: although one of them should probably be request_context | 00:29 |
| jgriffith | patrickeast: haha... yeah, not sure what else we'd do with it :) | 00:29 |
| jgriffith | patrickeast: hey... at least it's a context object and not something else :) | 00:29 |
| patrickeast | hahaha | 00:29 |
| patrickeast | jgriffith: very true | 00:29 |
| *** tsekiyama has quit IRC | 00:30 | |
| *** DericHorn-HP has quit IRC | 00:33 | |
| *** smoriya has joined #openstack-cinder | 00:33 | |
| *** piyanai has quit IRC | 00:33 | |
| *** DericHorn-HP has joined #openstack-cinder | 00:35 | |
| *** DericHorn-HP has quit IRC | 00:35 | |
| jgriffith | mtreinish: Actually that's exactly what eharney had here: https://review.openstack.org/#/c/203845/1 My bad | 00:36 |
| jgriffith | mtreinish: LGTM | 00:36 |
| *** Apoorva has quit IRC | 00:50 | |
| *** leeantho has quit IRC | 00:52 | |
| *** heyun has joined #openstack-cinder | 00:53 | |
| *** _cjones_ has quit IRC | 00:55 | |
| *** salv-orlando has quit IRC | 00:57 | |
| *** zhenguo has joined #openstack-cinder | 00:58 | |
| *** mylu has quit IRC | 00:58 | |
| *** salv-orlando has joined #openstack-cinder | 01:00 | |
| *** mylu has joined #openstack-cinder | 01:00 | |
| *** eharney has joined #openstack-cinder | 01:01 | |
| openstackgerrit | Zhenguo Niu proposed openstack/python-cinderclient: Remove H302 https://review.openstack.org/202364 | 01:04 |
| *** lixiaoy11 has joined #openstack-cinder | 01:04 | |
| *** salv-orlando has quit IRC | 01:05 | |
| *** primechuck has quit IRC | 01:06 | |
| *** annashen has joined #openstack-cinder | 01:11 | |
| *** mtanino has quit IRC | 01:12 | |
| openstackgerrit | abhiram moturi proposed openstack/cinder: Enhance unit tests for zfssa drivers https://review.openstack.org/203895 | 01:15 |
| *** annashen has quit IRC | 01:15 | |
| *** Apoorva has joined #openstack-cinder | 01:17 | |
| *** skylerberg has quit IRC | 01:19 | |
| *** mylu has quit IRC | 01:20 | |
| *** haomaiwang has quit IRC | 01:24 | |
| *** bitblt has quit IRC | 01:29 | |
| *** diogogmt has joined #openstack-cinder | 01:30 | |
| *** Apoorva has quit IRC | 01:32 | |
| *** raunak has quit IRC | 01:34 | |
| *** mylu has joined #openstack-cinder | 01:39 | |
| *** mylu has quit IRC | 01:40 | |
| *** jdandrea has quit IRC | 01:42 | |
| *** mylu has joined #openstack-cinder | 01:44 | |
| *** Apoorva has joined #openstack-cinder | 01:44 | |
| *** jasondotstar has joined #openstack-cinder | 01:45 | |
| *** Apoorva has quit IRC | 01:46 | |
| *** mylu has quit IRC | 01:47 | |
| *** mylu has joined #openstack-cinder | 01:47 | |
| *** vilobhmm1 has joined #openstack-cinder | 01:52 | |
| *** davechen has joined #openstack-cinder | 01:53 | |
| *** vilobhmm has quit IRC | 01:53 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver https://review.openstack.org/188360 | 01:58 |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add smartX support for Huawei volume driver https://review.openstack.org/188732 | 01:59 |
| *** vilobhmm1 has quit IRC | 01:59 | |
| *** davechen1 has joined #openstack-cinder | 02:00 | |
| openstackgerrit | Li Yingjun proposed openstack/cinder-specs: Add ability to update volume type public spec https://review.openstack.org/203462 | 02:01 |
| *** primechuck has joined #openstack-cinder | 02:01 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 02:01 |
| *** davechen has quit IRC | 02:02 | |
| *** sghanekar_ has quit IRC | 02:02 | |
| *** Lee1092 has joined #openstack-cinder | 02:03 | |
| openstackgerrit | wanghao proposed openstack/cinder: Improve performance listing detail for volumes created from image https://review.openstack.org/203530 | 02:06 |
| *** Zhongjun_ has joined #openstack-cinder | 02:07 | |
| *** xyang1 has quit IRC | 02:08 | |
| *** xyang1 has joined #openstack-cinder | 02:09 | |
| *** xyang1 has quit IRC | 02:11 | |
| *** vilobhmm has joined #openstack-cinder | 02:11 | |
| *** vilobhmm has quit IRC | 02:12 | |
| *** vilobhmm has joined #openstack-cinder | 02:13 | |
| *** heyun has quit IRC | 02:18 | |
| *** Zhongjun_ is now known as Zhongjun | 02:28 | |
| *** vilobhmm has quit IRC | 02:30 | |
| *** gouthamr has quit IRC | 02:34 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 02:42 |
| openstackgerrit | xing-yang proposed openstack/cinder-specs: Non Disruptive Backup https://review.openstack.org/186897 | 02:42 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 02:45 |
| *** patrickeast has quit IRC | 02:47 | |
| *** jasondotstar has quit IRC | 02:47 | |
| *** annashen has joined #openstack-cinder | 02:51 | |
| *** hakimo_ has joined #openstack-cinder | 02:52 | |
| *** hakimo has quit IRC | 02:55 | |
| *** annashen has quit IRC | 02:55 | |
| *** mylu has quit IRC | 02:58 | |
| *** vilobhmm has joined #openstack-cinder | 03:02 | |
| *** dims_ has quit IRC | 03:09 | |
| *** dims_ has joined #openstack-cinder | 03:09 | |
| *** mylu has joined #openstack-cinder | 03:13 | |
| *** dims_ has quit IRC | 03:27 | |
| *** dims_ has joined #openstack-cinder | 03:28 | |
| *** diogogmt has quit IRC | 03:30 | |
| *** dims_ has quit IRC | 03:34 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 03:34 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 03:41 |
| *** akshai has quit IRC | 03:44 | |
| *** vilobhmm has quit IRC | 03:44 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: Add class for dog command executor https://review.openstack.org/194532 | 03:48 |
| *** rakesh_mishra has joined #openstack-cinder | 03:48 | |
| *** garthb has joined #openstack-cinder | 03:51 | |
| *** annegent_ has joined #openstack-cinder | 04:04 | |
| *** dims_ has joined #openstack-cinder | 04:06 | |
| *** mylu has quit IRC | 04:06 | |
| *** annegentl_ has quit IRC | 04:07 | |
| *** mylu has joined #openstack-cinder | 04:07 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 04:08 |
| *** dims_ has quit IRC | 04:10 | |
| *** rakesh_mishra has quit IRC | 04:11 | |
| *** dims_ has joined #openstack-cinder | 04:13 | |
| *** davechen1 has quit IRC | 04:13 | |
| *** BharatK has joined #openstack-cinder | 04:13 | |
| *** davechen has joined #openstack-cinder | 04:14 | |
| *** garthb has quit IRC | 04:15 | |
| *** dims_ has quit IRC | 04:18 | |
| *** lixiaoy11 has quit IRC | 04:21 | |
| *** mylu has quit IRC | 04:22 | |
| *** coolsvap|away is now known as coolsvap | 04:24 | |
| *** Apoorva has joined #openstack-cinder | 04:25 | |
| *** dims_ has joined #openstack-cinder | 04:27 | |
| *** lixiaoy1 has joined #openstack-cinder | 04:32 | |
| *** dims_ has quit IRC | 04:35 | |
| *** lixiaoy11 has joined #openstack-cinder | 04:37 | |
| *** lixiaoy1 has quit IRC | 04:39 | |
| *** erlon has quit IRC | 04:40 | |
| *** sgotliv has quit IRC | 04:44 | |
| *** shyama has joined #openstack-cinder | 04:45 | |
| openstackgerrit | Li Yingjun proposed openstack/python-cinderclient: Fixes table when there are multiline in result data https://review.openstack.org/203919 | 04:54 |
| *** dims_ has joined #openstack-cinder | 05:02 | |
| *** dims_ has quit IRC | 05:06 | |
| *** bnemec has quit IRC | 05:07 | |
| *** chhavi has joined #openstack-cinder | 05:11 | |
| *** deepakcs has joined #openstack-cinder | 05:16 | |
| *** dims_ has joined #openstack-cinder | 05:22 | |
| *** pradipta has joined #openstack-cinder | 05:22 | |
| *** agarciam has quit IRC | 05:26 | |
| flip214 | hemna: ping | 05:26 |
| *** dims_ has quit IRC | 05:26 | |
| *** nkrinner has joined #openstack-cinder | 05:27 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 05:27 |
| *** shyama has quit IRC | 05:34 | |
| *** shyama has joined #openstack-cinder | 05:35 | |
| *** dims_ has joined #openstack-cinder | 05:36 | |
| *** bswartz has quit IRC | 05:37 | |
| *** markvoelker has quit IRC | 05:38 | |
| *** shyama has quit IRC | 05:40 | |
| *** dims_ has quit IRC | 05:41 | |
| *** vlaza has joined #openstack-cinder | 05:41 | |
| *** ronis has joined #openstack-cinder | 05:41 | |
| *** shyama has joined #openstack-cinder | 05:41 | |
| *** heyun has joined #openstack-cinder | 05:43 | |
| *** davechen has quit IRC | 05:44 | |
| *** markvoelker has joined #openstack-cinder | 05:44 | |
| *** shyama has quit IRC | 05:45 | |
| *** davechen has joined #openstack-cinder | 05:45 | |
| *** Apoorva has quit IRC | 05:47 | |
| *** sgotliv has joined #openstack-cinder | 05:48 | |
| *** Zhongjun has quit IRC | 05:48 | |
| *** bswartz has joined #openstack-cinder | 05:49 | |
| openstackgerrit | Li Yingjun proposed openstack/python-cinderclient: Fixes table when there are multiline in result data https://review.openstack.org/203919 | 05:49 |
| *** IanGovett has joined #openstack-cinder | 05:52 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO support for iscsi discovery auth https://review.openstack.org/189358 | 05:53 |
| *** primechuck has quit IRC | 05:59 | |
| *** IanGovett has quit IRC | 06:00 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO support for iscsi discovery auth https://review.openstack.org/189358 | 06:01 |
| *** deepakcs has quit IRC | 06:04 | |
| *** BharatK has quit IRC | 06:04 | |
| *** deepakcs has joined #openstack-cinder | 06:04 | |
| *** pots_ has quit IRC | 06:07 | |
| *** pots has quit IRC | 06:07 | |
| *** pradipta has quit IRC | 06:08 | |
| *** pots has joined #openstack-cinder | 06:13 | |
| *** BharatK has joined #openstack-cinder | 06:16 | |
| *** BharatK_ has joined #openstack-cinder | 06:21 | |
| *** BharatK has quit IRC | 06:21 | |
| *** dims_ has joined #openstack-cinder | 06:21 | |
| openstackgerrit | huangtianhua proposed openstack/python-cinderclient: Raise correct message for url version check https://review.openstack.org/123670 | 06:21 |
| *** dims_ has quit IRC | 06:25 | |
| *** BharatK_ has quit IRC | 06:28 | |
| *** BharatK has joined #openstack-cinder | 06:29 | |
| *** shyama has joined #openstack-cinder | 06:33 | |
| *** salv-orlando has joined #openstack-cinder | 06:52 | |
| *** geguileo has joined #openstack-cinder | 06:55 | |
| *** dims_ has joined #openstack-cinder | 06:56 | |
| *** dims_ has quit IRC | 07:00 | |
| *** geguileo has quit IRC | 07:03 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 07:07 |
| *** abehl has joined #openstack-cinder | 07:14 | |
| *** geguileo has joined #openstack-cinder | 07:16 | |
| *** IanGovett has joined #openstack-cinder | 07:19 | |
| *** s0nea has quit IRC | 07:20 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: Move volume.api test to correct location https://review.openstack.org/198636 | 07:22 |
| *** s0nea has joined #openstack-cinder | 07:22 | |
| *** topshare has quit IRC | 07:23 | |
| *** ankit_ag has joined #openstack-cinder | 07:24 | |
| openstackgerrit | Li Yingjun proposed openstack/python-cinderclient: Fixes table when there are multiline in result data https://review.openstack.org/203919 | 07:24 |
| *** afazekas_ has joined #openstack-cinder | 07:24 | |
| *** topshare has joined #openstack-cinder | 07:24 | |
| *** yrabl is now known as yrabl_afk | 07:26 | |
| *** salv-orlando has quit IRC | 07:28 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/203952 | 07:33 |
| *** dims_ has joined #openstack-cinder | 07:37 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix 033 add encryption unique key migration https://review.openstack.org/165403 | 07:40 |
| *** dims_ has quit IRC | 07:41 | |
| *** belmoreira has joined #openstack-cinder | 07:42 | |
| *** dalgaaf has quit IRC | 07:48 | |
| *** jordanP has joined #openstack-cinder | 07:50 | |
| *** belmoreira has quit IRC | 07:52 | |
| *** zhenguo has quit IRC | 07:53 | |
| *** zhenguo has joined #openstack-cinder | 07:53 | |
| *** jistr has joined #openstack-cinder | 07:55 | |
| openstackgerrit | chenying proposed openstack/cinder: Publishing service capabilities after volume is sucessfully cteated https://review.openstack.org/203961 | 07:58 |
| *** laughterwym has joined #openstack-cinder | 08:02 | |
| *** chlong has quit IRC | 08:04 | |
| *** topshare has quit IRC | 08:05 | |
| *** topshare has joined #openstack-cinder | 08:05 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver https://review.openstack.org/188360 | 08:07 |
| *** lennyb_ has joined #openstack-cinder | 08:08 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 08:09 |
| *** topshare has quit IRC | 08:09 | |
| *** topshare has joined #openstack-cinder | 08:09 | |
| *** topshare has quit IRC | 08:10 | |
| *** topshare has joined #openstack-cinder | 08:10 | |
| *** LiuXinguo has quit IRC | 08:13 | |
| *** salv-orlando has joined #openstack-cinder | 08:25 | |
| *** topshare has quit IRC | 08:29 | |
| *** bluex-pl has joined #openstack-cinder | 08:29 | |
| *** bluex-pl has quit IRC | 08:29 | |
| *** bluex-pl has joined #openstack-cinder | 08:30 | |
| *** dims_ has joined #openstack-cinder | 08:31 | |
| openstackgerrit | chenying proposed openstack/cinder: Publishing service capabilities after volume is sucessfully cteated https://review.openstack.org/203961 | 08:32 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 08:32 |
| *** laughterwym has quit IRC | 08:34 | |
| *** geguileo has quit IRC | 08:34 | |
| *** topshare has joined #openstack-cinder | 08:35 | |
| *** dims_ has quit IRC | 08:36 | |
| *** 20WABQBTM has joined #openstack-cinder | 08:37 | |
| *** 20WABQBTM has quit IRC | 08:37 | |
| *** haomaiwang has joined #openstack-cinder | 08:37 | |
| *** geguileo has joined #openstack-cinder | 08:38 | |
| *** topshare has quit IRC | 08:38 | |
| *** topshare has joined #openstack-cinder | 08:42 | |
| *** aix has joined #openstack-cinder | 08:43 | |
| *** jaypipes has quit IRC | 08:46 | |
| *** dims_ has joined #openstack-cinder | 08:46 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix 033 add encryption unique key migration https://review.openstack.org/165403 | 08:47 |
| *** chenying has joined #openstack-cinder | 08:50 | |
| *** dims_ has quit IRC | 08:51 | |
| *** xiangxinyong has joined #openstack-cinder | 08:53 | |
| *** shyama has quit IRC | 08:54 | |
| *** aix has quit IRC | 08:58 | |
| *** geguileo has quit IRC | 08:58 | |
| *** geguileo has joined #openstack-cinder | 08:59 | |
| *** topshare has quit IRC | 09:07 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/203952 | 09:07 |
| *** piyanai has joined #openstack-cinder | 09:08 | |
| *** e0ne has joined #openstack-cinder | 09:15 | |
| *** xiangxinyong has left #openstack-cinder | 09:17 | |
| *** topshare has joined #openstack-cinder | 09:22 | |
| *** topshare has quit IRC | 09:26 | |
| *** dims_ has joined #openstack-cinder | 09:26 | |
| *** aix has joined #openstack-cinder | 09:28 | |
| *** amick has joined #openstack-cinder | 09:29 | |
| *** dims_ has quit IRC | 09:31 | |
| *** dims_ has joined #openstack-cinder | 09:36 | |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/cinder: Cinder part of volume snapshot improvements https://review.openstack.org/172808 | 09:39 |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/cinder: db: Add support for snapshot_admin_metadata https://review.openstack.org/203979 | 09:39 |
| *** dims_ has quit IRC | 09:40 | |
| *** piyanai has quit IRC | 09:40 | |
| *** lpetrut has joined #openstack-cinder | 09:42 | |
| *** lpetrut has quit IRC | 09:47 | |
| *** davechen has left #openstack-cinder | 09:47 | |
| *** lixiaoy11 has quit IRC | 09:51 | |
| *** amick has left #openstack-cinder | 09:55 | |
| *** amickus has joined #openstack-cinder | 09:58 | |
| *** boris-42 has quit IRC | 10:02 | |
| *** geguileo has quit IRC | 10:03 | |
| *** geguileor has joined #openstack-cinder | 10:03 | |
| *** aix has quit IRC | 10:04 | |
| *** vlaza has quit IRC | 10:06 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Fix use of urllib for py3 compatibility in Huawei driver https://review.openstack.org/203996 | 10:06 |
| *** Liu has joined #openstack-cinder | 10:06 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 10:13 |
| *** geguileo has joined #openstack-cinder | 10:15 | |
| *** aix has joined #openstack-cinder | 10:15 | |
| *** asselin has quit IRC | 10:15 | |
| *** dims_ has joined #openstack-cinder | 10:16 | |
| *** geguileo has quit IRC | 10:17 | |
| *** akrog has joined #openstack-cinder | 10:17 | |
| *** geguileor has quit IRC | 10:17 | |
| *** geguileo has joined #openstack-cinder | 10:18 | |
| *** geguileo is now known as Guest78809 | 10:19 | |
| *** akrog has quit IRC | 10:20 | |
| *** laughterwym has joined #openstack-cinder | 10:28 | |
| *** chenying has quit IRC | 10:34 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Remove unused arguments from c-vol's create_volume https://review.openstack.org/199590 | 10:36 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Refactoring of manager's create_volume flow https://review.openstack.org/193167 | 10:36 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Refactoring of manager's create_volume flow https://review.openstack.org/193167 | 10:39 |
| *** _afazekas has joined #openstack-cinder | 10:41 | |
| *** afazekas has quit IRC | 10:45 | |
| *** timcl has joined #openstack-cinder | 10:53 | |
| *** heyun has quit IRC | 10:55 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/203952 | 11:00 |
| *** lpetrut has joined #openstack-cinder | 11:05 | |
| *** coolsvap is now known as coolsvap|away | 11:12 | |
| *** vlaza has joined #openstack-cinder | 11:28 | |
| deepakcs | Hi all, is anyone also seeing the ceph CI job hitting timeout for the volume_backup_export_import testcase ? | 11:30 |
| deepakcs | http://logs.openstack.org/79/203979/1/check/gate-tempest-dsvm-full-ceph/e290eca/console.html | 11:30 |
| *** chhavi has quit IRC | 11:34 | |
| *** deepakcs has quit IRC | 11:37 | |
| *** gouthamr has joined #openstack-cinder | 11:39 | |
| *** salv-orlando has quit IRC | 11:40 | |
| *** omykhal has joined #openstack-cinder | 11:41 | |
| *** topshare has joined #openstack-cinder | 11:41 | |
| *** topshare has quit IRC | 11:43 | |
| *** deepakcs has joined #openstack-cinder | 11:44 | |
| omykhal | hi, 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-xxxxxx | 11:45 |
| omykhal | it should be like this - lvremove --config "activation { retry_deactivation = 1} devices { ignore_suspended_devices = 1}" -f cindervg-thin/volume-xxxxxxx | 11:46 |
| omykhal | cinder Kilo, lvm2-2.02.115-3.el7_1.1.x86_64 | 11:46 |
| omykhal | how to fix it in cinder? | 11:47 |
| *** Guest78809 has quit IRC | 11:58 | |
| *** Guest78809 has joined #openstack-cinder | 11:59 | |
| *** Maike has joined #openstack-cinder | 12:01 | |
| *** salv-orlando has joined #openstack-cinder | 12:01 | |
| *** dims_ has quit IRC | 12:09 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Add I/T mapping check for IBM FlashSystem https://review.openstack.org/203616 | 12:09 |
| *** jungleboyj has joined #openstack-cinder | 12:10 | |
| *** Guest78809 has quit IRC | 12:18 | |
| *** chlong has joined #openstack-cinder | 12:18 | |
| *** ganso_ has joined #openstack-cinder | 12:20 | |
| *** Miouge has joined #openstack-cinder | 12:23 | |
| *** dalgaaf has joined #openstack-cinder | 12:23 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/203952 | 12:23 |
| *** bswartz has quit IRC | 12:31 | |
| *** chhavi has joined #openstack-cinder | 12:35 | |
| *** piyanai has joined #openstack-cinder | 12:51 | |
| *** akerr has joined #openstack-cinder | 12:55 | |
| *** smoriya has quit IRC | 12:57 | |
| *** BharatK has quit IRC | 12:58 | |
| *** piyanai has quit IRC | 13:00 | |
| *** kjnelson has joined #openstack-cinder | 13:02 | |
| *** annegentl_ has joined #openstack-cinder | 13:04 | |
| *** annegent_ has quit IRC | 13:06 | |
| *** Maike has quit IRC | 13:08 | |
| *** dustins has joined #openstack-cinder | 13:09 | |
| *** jdandrea has joined #openstack-cinder | 13:11 | |
| *** jwcroppe has quit IRC | 13:13 | |
| *** jwcroppe has joined #openstack-cinder | 13:13 | |
| *** markvoelker has quit IRC | 13:17 | |
| openstackgerrit | Ph. Marek proposed openstack/cinder: DRBD: Rename a constant to a better name, uppercase-only. https://review.openstack.org/203646 | 13:17 |
| *** jwcroppe has quit IRC | 13:18 | |
| *** timcl has quit IRC | 13:19 | |
| *** deepakcs has quit IRC | 13:19 | |
| *** piyanai has joined #openstack-cinder | 13:19 | |
| *** julim has joined #openstack-cinder | 13:19 | |
| *** simondodsley has joined #openstack-cinder | 13:20 | |
| *** ankit_ag has quit IRC | 13:22 | |
| *** mylu has joined #openstack-cinder | 13:24 | |
| *** Yogi1 has joined #openstack-cinder | 13:25 | |
| *** thangp has joined #openstack-cinder | 13:26 | |
| *** Yogi1 has quit IRC | 13:30 | |
| *** xyang1 has joined #openstack-cinder | 13:33 | |
| openstackgerrit | Yaguang Tang proposed openstack/cinder: Remove simple scheduler which is deprecated since Juno https://review.openstack.org/202858 | 13:35 |
| *** timcl has joined #openstack-cinder | 13:37 | |
| *** annegentl_ has quit IRC | 13:39 | |
| *** mylu has quit IRC | 13:39 | |
| *** mdenny has joined #openstack-cinder | 13:44 | |
| *** changbl has quit IRC | 13:44 | |
| smcginnis | omykhal: Still around? | 13:46 |
| *** mriedem has joined #openstack-cinder | 13:46 | |
| smcginnis | omykhal: Looks like the line "'config', LVM_CONFIG" would need to be changed to "'config "%s"' % LVM_CONFIG". | 13:47 |
| smcginnis | omykhal: In brick/local_dev/lvm.py line 668. | 13:47 |
| smcginnis | omykhal: https://github.com/openstack/cinder/blob/master/cinder/brick/local_dev/lvm.py#L668 | 13:47 |
| *** piyanai has quit IRC | 13:48 | |
| smcginnis | omykhal: 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-cinder | 13:48 | |
| *** edmondsw has joined #openstack-cinder | 13:49 | |
| eharney | it seems unlikely that --config and all the opts should be in the same arg? | 13:50 |
| smcginnis | eharney: Unless everything in the {} should be passed in as the config value? Don't really know. | 13:50 |
| *** abehl has quit IRC | 13:51 | |
| eharney | i'm not too sure either, but what's there now looks sensible to me | 13:51 |
| eharney | and equivalent to what omykhal said it should be above? | 13:52 |
| smcginnis | eharney: I would be kind of surprised if it's been there and no one noticed it was a problem. | 13:52 |
| smcginnis | That line was last changed June of 2014. | 13:52 |
| *** mylu has joined #openstack-cinder | 13:53 | |
| smcginnis | I 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-cinder | 13:53 | |
| eharney | it may be confusion that the quotes aren't printed in the log, because they aren't needed there | 13:53 |
| smcginnis | eharney: Oh, good point! | 13:54 |
| *** diogogmt has joined #openstack-cinder | 13:54 | |
| smcginnis | Wait, no, we don't quote it, so it's not that they just aren't being logged. | 13:55 |
| eharney | we don't have to quote it, we aren't running it in a shell | 13:55 |
| eharney | it's passed as a single arg | 13:56 |
| *** mriedem has left #openstack-cinder | 13:57 | |
| *** jungleboyj has quit IRC | 13:58 | |
| *** piyanai has joined #openstack-cinder | 13:58 | |
| *** markus_z has joined #openstack-cinder | 13:59 | |
| *** vlaza has quit IRC | 14:02 | |
| *** crose has joined #openstack-cinder | 14:02 | |
| *** amickus has quit IRC | 14:04 | |
| *** breitz has quit IRC | 14:06 | |
| *** breitz has joined #openstack-cinder | 14:06 | |
| jgriffith | omykhal: 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 |
| jgriffith | omykhal: 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 |
| jgriffith | or "manually" I guess | 14:07 |
| *** mriedem has joined #openstack-cinder | 14:10 | |
| *** diogogmt has quit IRC | 14:10 | |
| omykhal | this 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-619a89463766 | 14:12 |
| omykhal | oslo_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-cinder | 14:12 | |
| *** jasondotstar has joined #openstack-cinder | 14:13 | |
| *** avishay has joined #openstack-cinder | 14:15 | |
| *** topshare has joined #openstack-cinder | 14:16 | |
| *** bswartz has joined #openstack-cinder | 14:17 | |
| *** lpetrut has quit IRC | 14:18 | |
| jgriffith | omykhal: ahh... thank you for the clarification | 14:18 |
| jgriffith | omykhal: 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 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add drivers list generator https://review.openstack.org/192751 | 14:19 |
| jgriffith | omykhal: although it sounds like smcginnis was aleady interested in looking at it as well. I'm not sure that config variable is the culprit still | 14:20 |
| *** mtanino has joined #openstack-cinder | 14:22 | |
| aarefiev | jgriffith: hi, I've addressed your comments https://review.openstack.org/#/c/192751/11 , please look when you have time | 14:22 |
| jgriffith | aarefiev: cool... but what about the other two issues: | 14:23 |
| jgriffith | 1. Cinder has to be loaded on the system you run it on | 14:23 |
| jgriffith | 2. Listing the parent/base modules as drivers in the resultant list | 14:24 |
| *** diogogmt has joined #openstack-cinder | 14:24 | |
| aarefiev | jgriffith: hm 1) I thought you are talking about tox env, | 14:25 |
| aarefiev | jgriffith: It is how genconfig works | 14:25 |
| *** laughterwym has quit IRC | 14:26 | |
| jgriffith | aarefiev: well, just copying what I posted in the comment into the tox.ini was the easy part | 14:26 |
| eharney | mtreinish: i think i want a regex like this? http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/devstack-gate.yaml#n2268 | 14:26 |
| mtreinish | eharney: yeah that looks like the right thing | 14:26 |
| jgriffith | aarefiev: 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-cinder | 14:26 | |
| mtreinish | eharney: fwiw I tested the one I left in the review comment with a testr list-tests, but just double check it :) | 14:26 |
| eharney | mtreinish: cool, thanks | 14:26 |
| eharney | mtreinish: yeah, glad to see that, i actually didn't know how to test it | 14:27 |
| jgriffith | aarefiev: http://paste.openstack.org/show/395905/ | 14:27 |
| *** topshare has quit IRC | 14:27 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient https://review.openstack.org/197643 | 14:27 |
| mtreinish | eharney: yeah using a regex with testr list-tests is something that for whatever reason no many people know about | 14:28 |
| aarefiev | jgriffith: hm it's strange, could you try on clean tox venv, please? | 14:29 |
| jgriffith | aarefiev: that was a clean tox env :) | 14:29 |
| aarefiev | jgriffith: it works for me | 14:29 |
| jgriffith | aarefiev: Oh? hmmm | 14:30 |
| jgriffith | aarefiev: are you running a devstack machine? | 14:30 |
| jgriffith | aarefiev: or on your laptop/workstation without devstack/cinder installed? | 14:30 |
| *** david-lyle has quit IRC | 14:30 | |
| aarefiev | jgriffith: no, it should get modules from venv | 14:30 |
| aarefiev | jgriffith: let me check | 14:30 |
| jgriffith | aarefiev: yes, I know it "should" :) | 14:31 |
| jgriffith | aarefiev: and I haven't looked at the duplicate huawei option yet | 14:31 |
| *** Yogi1 has joined #openstack-cinder | 14:35 | |
| jgriffith | Looks like not testing genconfig has bitten us once again | 14:36 |
| jgriffith | aarefiev: so it's unrelated to your work, but the config declarations are broken again. Huawei and IBM so far, maybe others | 14:37 |
| *** diogogmt has quit IRC | 14:37 | |
| *** annegentle has joined #openstack-cinder | 14:37 | |
| jgriffith | haha... novaclient | 14:37 |
| aarefiev | jgriffith: print full path to modules /home/openstack/cinder/cinder/volume/drivers/block_device.pyc | 14:38 |
| aarefiev | it generates from tree | 14:38 |
| jgriffith | aarefiev: huh? | 14:39 |
| *** nkrinner has quit IRC | 14:39 | |
| aarefiev | jgriffith: /home/openstack/cinder/cinder/volume/drivers/block_device.pyc - it is full path to one of the module | 14:40 |
| aarefiev | /home/openstack/cinder/ - cinder source | 14:40 |
| *** piyanai has quit IRC | 14:42 | |
| jgriffith | aarefiev: hmm... interesting | 14:43 |
| *** lpabon has joined #openstack-cinder | 14:44 | |
| aarefiev | jgriffith: about parent/base, you mean it shouldn't be in list, right? | 14:45 |
| *** Miouge has quit IRC | 14:45 | |
| aarefiev | jgriffith: for example common | 14:46 |
| aarefiev | jgriffith: baseVD<-common<-dr1 , baseVD<-common<-dr2 | 14:47 |
| jgriffith | aarefiev: so on that topic, correct | 14:47 |
| jgriffith | aarefiev: I'm still trying to understand why I can't run this on a system with just the tox env | 14:48 |
| aarefiev | jgriffith: I'll try on clean env | 14:49 |
| jgriffith | aarefiev: I got it working | 14:49 |
| *** Miouge has joined #openstack-cinder | 14:49 | |
| jgriffith | aarefiev: I changed the file perms to 755 and just call it bare | 14:49 |
| *** lpetrut has joined #openstack-cinder | 14:50 | |
| jgriffith | aarefiev: I'm unclear on what that matters... trying to reproduce again | 14:50 |
| jgriffith | aarefiev: hmm.. that's weird | 14:50 |
| *** krtaylor has quit IRC | 14:50 | |
| jgriffith | aarefiev: if I just cherry pick in your patch it barfs | 14:51 |
| jgriffith | aarefiev: but if I duplicate it EXACTLY in my master branch it works fine | 14:51 |
| jgriffith | aarefiev: not cherry-pick... review -d | 14:51 |
| jgriffith | Oh... wait | 14:51 |
| *** Miouge has quit IRC | 14:51 | |
| jgriffith | aarefiev: hmm... ok, well that's weird | 14:52 |
| jgriffith | aarefiev: Ok... so that seems to be working fine now | 14:53 |
| jgriffith | aarefiev: back to the other issue, with the base classes | 14:53 |
| *** Miouge has joined #openstack-cinder | 14:53 | |
| aarefiev | jgriffith: hmm strange | 14:53 |
| jgriffith | aarefiev: yeah, it's really odd | 14:54 |
| aarefiev | jgriffith: yes about base classes | 14:54 |
| jgriffith | Oh... ya know what I think it might be... | 14:54 |
| jgriffith | my pip-cache | 14:54 |
| jgriffith | So I pulled master, and ran tox on it. That would update my cache | 14:54 |
| jgriffith | for what gets installed when I run your patch | 14:55 |
| *** Miouge has quit IRC | 14:55 | |
| jgriffith | aarefiev: wonder when the last time you rebased your patch was? | 14:55 |
| jgriffith | aarefiev: anyway... let's figure out what to do about the base class stuff | 14:56 |
| jgriffith | aarefiev: 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 |
| aarefiev | jgriffith: I don't remember ) | 14:56 |
| jgriffith | aarefiev: That PurBaseVolumeDriver is an example of what it would be cool if we were smart enough to ignore | 14:56 |
| aarefiev | jgriffith: I don't see how to make it cool | 14:57 |
| jgriffith | aarefiev: :) | 14:57 |
| *** jasonsb has joined #openstack-cinder | 14:58 | |
| jgriffith | aarefiev: ok, well maybe to start, we just do it the brute force way | 14:58 |
| jgriffith | aarefiev: generate the list and keep a list of base classes that we pop out of the list after the run | 14:59 |
| aarefiev | jgriffith: I think we can't identificate it automatically | 14:59 |
| jgriffith | aarefiev: or even put a special character to denote they're not an "actual" driver | 14:59 |
| jgriffith | aarefiev: 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 patch | 15:00 |
| *** jwcroppe has joined #openstack-cinder | 15:00 | |
| aarefiev | jgriffith: I mean now, maybe abc abstract classes | 15:00 |
| jgriffith | aarefiev: yeah, we really need to clean up our driver structure a bit | 15:01 |
| jgriffith | aarefiev: it's kinda messy | 15:01 |
| jgriffith | aarefiev: ok... well so in that case; your patch looks good :) | 15:01 |
| aarefiev | jgriffith: agree on that | 15:01 |
| jgriffith | aarefiev: thanks for adding that to tox.ini for me | 15:01 |
| *** timcl has quit IRC | 15:02 | |
| aarefiev | I mean about clean up )) | 15:02 |
| *** Yogi11 has joined #openstack-cinder | 15:02 | |
| jgriffith | aarefiev: I'll leave it up to you if you want to rebase it and change perms to call the tool directly | 15:02 |
| *** krtaylor has joined #openstack-cinder | 15:02 | |
| *** Yogi1 has quit IRC | 15:02 | |
| jgriffith | aarefiev: LOL... yeah, although I'm sure you also agree that your patch is good :) | 15:02 |
| aarefiev | jgriffith: I'll rebase, what do you mean change perms to call the tool directly? | 15:03 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add drivers list generator https://review.openstack.org/192751 | 15:05 |
| jgriffith | aarefiev: I modified your patch, so the fox.ini call is just: commands = {toxinidir}/tools/generate_driver_list.py | 15:05 |
| jgriffith | aarefiev: so I made the perms on the script 755... but that is REALLY NOT important | 15:06 |
| jgriffith | aarefiev: it doesn't matter at all | 15:06 |
| aarefiev | jgriffith: ok, I've rebased | 15:07 |
| jgriffith | aarefiev: ok, +2/A | 15:07 |
| jgriffith | aarefiev: it had the required number of votes prior to my request and the rebase | 15:07 |
| jgriffith | aarefiev: thanks a lot for putting that together | 15:07 |
| *** afazekas_ has quit IRC | 15:08 | |
| *** diemt has quit IRC | 15:08 | |
| aarefiev | jgriffith: thank you, I'll think about filtering base classes | 15:08 |
| *** diemt has joined #openstack-cinder | 15:09 | |
| *** mriedem has quit IRC | 15:09 | |
| *** edmondsw has quit IRC | 15:09 | |
| *** chlong has quit IRC | 15:09 | |
| *** IanGovett has quit IRC | 15:09 | |
| *** eharney has quit IRC | 15:09 | |
| *** swamireddy has quit IRC | 15:09 | |
| *** jamielennox has quit IRC | 15:09 | |
| *** kambiz has quit IRC | 15:09 | |
| *** briancline has quit IRC | 15:09 | |
| *** tbarron has quit IRC | 15:09 | |
| *** diogogmt has joined #openstack-cinder | 15:13 | |
| *** edmondsw has joined #openstack-cinder | 15:16 | |
| *** crose has quit IRC | 15:16 | |
| *** chlong has joined #openstack-cinder | 15:16 | |
| *** eharney has joined #openstack-cinder | 15:19 | |
| *** eharney is now known as Guest8511 | 15:19 | |
| *** dannywilson has joined #openstack-cinder | 15:19 | |
| *** Guest8511 is now known as eharney | 15:20 | |
| *** eharney has quit IRC | 15:20 | |
| *** eharney has joined #openstack-cinder | 15:20 | |
| *** dannywilson has quit IRC | 15:21 | |
| *** dannywilson has joined #openstack-cinder | 15:21 | |
| *** yuriy_n17 has joined #openstack-cinder | 15:22 | |
| yuriy_n17 | jgriffith: Hi! Please review https://review.openstack.org/#/c/203054/. It fixes incorrect version for hacking package on stable/kilo | 15:23 |
| *** afazekas has joined #openstack-cinder | 15:25 | |
| *** rushil has quit IRC | 15:26 | |
| *** jdurgin1 has joined #openstack-cinder | 15:31 | |
| jgriffith | yuriy_n17: If you could, would you mind putting a reference to the parent in the commit message: https://github.com/openstack/requirements/commit/4568bfb6c841130b2edb3202d796b1fe05aec70f | 15:31 |
| jgriffith | yuriy_n17: just something like "updates from global-requirements change: <short-sha>" | 15:32 |
| *** kambiz has joined #openstack-cinder | 15:33 | |
| *** bluex-pl has quit IRC | 15:33 | |
| yuriy_n17 | jgriffith: Thanks a lot | 15:34 |
| *** rushil has joined #openstack-cinder | 15:34 | |
| *** primechuck has quit IRC | 15:35 | |
| *** david-lyle has joined #openstack-cinder | 15:38 | |
| *** ronis has quit IRC | 15:40 | |
| *** changbl has joined #openstack-cinder | 15:41 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO support for iscsi discovery auth https://review.openstack.org/189358 | 15:43 |
| *** timcl has joined #openstack-cinder | 15:43 | |
| *** Yogi11 has quit IRC | 15:44 | |
| yuriy_n17 | jgriffith: Hi! Please review https://review.openstack.org/#/c/203054/. Thanks. | 15:48 |
| *** Yogi1 has joined #openstack-cinder | 15:50 | |
| *** jwitko has joined #openstack-cinder | 15:51 | |
| *** vokt has joined #openstack-cinder | 15:51 | |
| jwitko | Hey 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 root | 15:52 |
| jwitko | url and receive a 302.) My cinder conf can be seen here: http://pastebin.com/raw.php?i=hWWQDRQy | 15:52 |
| jwitko | Can anyone help me out? | 15:52 |
| *** Yogi1 has quit IRC | 15:53 | |
| dustins | jwitko: Having a look now | 15:54 |
| jwitko | ty dustins | 15:54 |
| dustins | jwitko: No problem! | 15:54 |
| *** NobodyCam has joined #openstack-cinder | 15:54 | |
| dustins | jwitko: Is this with master branch or on some other release? | 15:55 |
| *** vokt has quit IRC | 15:56 | |
| *** jasonsb has quit IRC | 15:56 | |
| jwitko | dustins, Kilo. I'm using Openstack Ansible Deployment for deployment | 15:56 |
| *** vokt has joined #openstack-cinder | 15:56 | |
| akerr | jwitko: dustins: want to move to #openstack-netapp so we don't spam all of cinder? | 15:56 |
| dustins | akerr: But I was having so much fun! :) | 15:57 |
| dustins | Yeah, let's move over there | 15:57 |
| e0ne | do we have any plans about multibackend tests on gates? | 15:58 |
| *** annegentle has quit IRC | 15:59 | |
| *** jungleboyj has joined #openstack-cinder | 16:00 | |
| jgriffith | e0ne: we have it as an option, IIRC there was a brief period where we were doing it | 16:01 |
| e0ne | jgriffith: how can i start such jobs? | 16:02 |
| jgriffith | e0ne: everything is in place in devstack | 16:02 |
| jgriffith | e0ne: 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 value | 16:02 |
| *** leeantho has joined #openstack-cinder | 16:03 | |
| e0ne | jgriffith: i know that devstack has such option | 16:03 |
| e0ne | jgriffith: i would like to have it on gates by default | 16:04 |
| *** jistr has quit IRC | 16:04 | |
| *** ociuhandu_ has joined #openstack-cinder | 16:04 | |
| jgriffith | e0ne: sure... curious; why? | 16:04 |
| e0ne | jgriffith: i'm hitting with the last ceph backup issue | 16:04 |
| *** avishay has quit IRC | 16:04 | |
| jgriffith | e0ne: IMHO that's sort of a "specialized" config option, it doesn't really bother me much if it's not tested | 16:04 |
| *** crose has joined #openstack-cinder | 16:05 | |
| e0ne | jgriffith: we've got incremental backup tested only | 16:05 |
| jgriffith | e0ne: well, that's just a case of something we never tested in the past that we just turned on | 16:05 |
| e0ne | jgriffith: full backup is not tested | 16:05 |
| jgriffith | e0ne: yeah, and given how long full backups take, please don't :) | 16:05 |
| e0ne | jgriffith: and looks like ist's broken since kilo | 16:05 |
| e0ne | jgriffith: hope, i'm wrong | 16:06 |
| jgriffith | e0ne: yeah, I don't think the gate should be a total crutch though | 16:06 |
| e0ne | jgriffith: full backup is not much slower than secure delete | 16:06 |
| jgriffith | e0ne: I think folks interested in backup and Ceph maintainers should do at least "some" testing :) | 16:06 |
| jgriffith | e0ne: and we don't do secure delete in the gate any longer either | 16:07 |
| jgriffith | e0ne: I'm open minded about it of course. | 16:07 |
| e0ne | jgriffith: oh.. it's good :) | 16:08 |
| smcginnis | Anyone know the current situation with the ceph gate failures? | 16:08 |
| jgriffith | e0ne: if people are interested in making it happen that's cool | 16:08 |
| jgriffith | smcginnis: yes | 16:08 |
| jgriffith | smcginnis: a new test was turned on in tempest that does import backups | 16:08 |
| *** pots has left #openstack-cinder | 16:08 | |
| *** ociuhandu_ has quit IRC | 16:08 | |
| jgriffith | smcginnis: the ceph driver is not working for that case | 16:08 |
| smcginnis | jgriffith: I haven't had time to look back through the log. Is someone working on clearing that up? | 16:08 |
| jgriffith | smcginnis: eharney has submitted a temporary skip of that test for ceph-dsvm | 16:09 |
| smcginnis | jgriffith: OK, awesome. Thanks! | 16:09 |
| eharney | it just merged a minute ago | 16:09 |
| jgriffith | smcginnis: https://bugs.launchpad.net/cinder/+bug/1476735 | 16:09 |
| openstack | Launchpad bug 1476735 in Cinder "check failure in gate-tempest-dsvm-full-ceph; failure to import backup" [High,Confirmed] | 16:09 |
| smcginnis | eharney: Thanks for that. | 16:09 |
| jgriffith | smcginnis: I updated that a bit and added some extra info | 16:09 |
| jgriffith | eharney: awesome! Thanks for putting that together so quick yesterday | 16:10 |
| e0ne | jgriffith: it's a hard desicion: more quick tests or a full test suit to run on gates | 16:10 |
| jgriffith | e0ne: meh... like I said, I think that things like backup suffer from a lack of real interest/use | 16:10 |
| e0ne | jgriffith: agree. i don't remember any customer that was asking about it:( | 16:11 |
| jgriffith | e0ne: anyway.. my point was just that the gate shouldn't be the ONLY place things are tested IMHO | 16:12 |
| *** jdurgin1 has quit IRC | 16:12 | |
| jgriffith | e0ne: although if I/We ever get anything going with cinder/tests/functional that would be a good place for this sort of thing maybe | 16:12 |
| *** _cjones_ has joined #openstack-cinder | 16:13 | |
| *** _cjones_ has quit IRC | 16:13 | |
| *** _cjones_ has joined #openstack-cinder | 16:14 | |
| Swanson | smcginnis: waiting for the puppet apply of that patch. | 16:14 |
| e0ne | jgriffith: yes, i'll be happy to have functional tests too | 16:14 |
| xyang1 | jgriffith: Does the functional test use a real backend? | 16:15 |
| e0ne | jgriffith: we've got a lot of features/options whis are tested only by unit tests :( | 16:15 |
| jgriffith | xyang1: the would yes | 16:15 |
| jgriffith | xyang1: right now they don't exist :) | 16:15 |
| jgriffith | xyang1: I need to put some stuff together to get the ball rolling | 16:16 |
| xyang1 | jgriffith: The backup was actually tested by LVM and I assume swift backup driver | 16:16 |
| jgriffith | xyang1: but don't want to start something else that I won't finish :) | 16:16 |
| jgriffith | xyang1: yes... it is | 16:16 |
| xyang1 | jgriffith: :) | 16:16 |
| jgriffith | xyang1: and that's all good | 16:16 |
| jgriffith | xyang1: so the issue is that the import backup feature hasn't been tested in the gate; it was just added as a tempest test | 16:17 |
| jgriffith | xyang1: LVM works fine, ceph however found issues with it | 16:17 |
| xyang1 | jgriffith: Ok, ceph is different | 16:17 |
| jgriffith | xyang1: yes, very much so | 16:17 |
| *** lpabon has quit IRC | 16:17 | |
| jgriffith | xyang1: BTW https://review.openstack.org/#/c/201812/10/cinder/volume/driver.py | 16:18 |
| jgriffith | xyang1: I agree about the inconsistency | 16:18 |
| jgriffith | xyang1: I would actually prefer to make all of them raise NotImplemented if you have no objection | 16:18 |
| xyang1 | jgriffith: no problem | 16:19 |
| jgriffith | xyang1: cool | 16:19 |
| *** _cjones_ has quit IRC | 16:20 | |
| *** asselin_ has quit IRC | 16:20 | |
| *** yuriy_n17 has quit IRC | 16:21 | |
| *** _cjones_ has joined #openstack-cinder | 16:24 | |
| *** mgarza has joined #openstack-cinder | 16:25 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add smartX support for Huawei volume driver https://review.openstack.org/188732 | 16:26 |
| * e0ne bb in ~3 hours | 16:27 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 16:31 |
| *** krtaylor has quit IRC | 16:32 | |
| *** asselin has joined #openstack-cinder | 16:34 | |
| *** david-lyle has quit IRC | 16:34 | |
| *** Apoorva has joined #openstack-cinder | 16:36 | |
| *** jwang has quit IRC | 16:40 | |
| *** amick has joined #openstack-cinder | 16:40 | |
| *** sghanekar_ has joined #openstack-cinder | 16:43 | |
| *** e0ne has quit IRC | 16:45 | |
| *** annashen has joined #openstack-cinder | 16:47 | |
| *** annashen has quit IRC | 16:47 | |
| *** annashen has joined #openstack-cinder | 16:47 | |
| *** david-lyle has joined #openstack-cinder | 16:48 | |
| *** lpetrut has quit IRC | 16:50 | |
| *** tbarron has joined #openstack-cinder | 16:51 | |
| *** krtaylor has joined #openstack-cinder | 16:52 | |
| *** patrickeast has joined #openstack-cinder | 16:54 | |
| *** jasondotstar has quit IRC | 16:56 | |
| *** julim_ has joined #openstack-cinder | 17:03 | |
| *** dansmith has quit IRC | 17:05 | |
| *** dansmith has joined #openstack-cinder | 17:05 | |
| *** julim has quit IRC | 17:05 | |
| *** dansmith has quit IRC | 17:05 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Fix Consistency Group issues https://review.openstack.org/201685 | 17:06 |
| *** dansmith has joined #openstack-cinder | 17:06 | |
| *** jordanP has quit IRC | 17:09 | |
| jwitko | Hey 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 8776 | 17:11 |
| jwitko | anyone have any ideas? | 17:11 |
| *** jasondotstar has joined #openstack-cinder | 17:16 | |
| *** gman-tx has quit IRC | 17:17 | |
| *** lpabon has joined #openstack-cinder | 17:17 | |
| *** gman-tx has joined #openstack-cinder | 17:17 | |
| *** raunak has joined #openstack-cinder | 17:18 | |
| *** ronis has joined #openstack-cinder | 17:18 | |
| jwitko | i do see the python process has two open ports, 44837 44836 | 17:18 |
| *** vilobhmm has joined #openstack-cinder | 17:22 | |
| *** gman-tx has quit IRC | 17:22 | |
| openstackgerrit | Raunak Kumar proposed openstack/cinder: Add extra spec capability for Nimble Cinder Driver https://review.openstack.org/198853 | 17:23 |
| *** aix has quit IRC | 17:23 | |
| *** avishay has joined #openstack-cinder | 17:23 | |
| jwitko | sorry those are outgoing connections | 17:24 |
| *** tbarron has quit IRC | 17:26 | |
| *** piyanai has joined #openstack-cinder | 17:27 | |
| *** avishay has quit IRC | 17:28 | |
| *** mylu has quit IRC | 17:28 | |
| *** bswartz has quit IRC | 17:29 | |
| *** mylu has joined #openstack-cinder | 17:30 | |
| *** patrickeast_ has joined #openstack-cinder | 17:31 | |
| *** tbarron has joined #openstack-cinder | 17:31 | |
| *** mylu has quit IRC | 17:32 | |
| *** mylu has joined #openstack-cinder | 17:32 | |
| *** zhenguo has quit IRC | 17:33 | |
| *** Guest78809 has joined #openstack-cinder | 17:34 | |
| *** annegentle has joined #openstack-cinder | 17:36 | |
| *** patrickeast_ has quit IRC | 17:38 | |
| *** annegent_ has joined #openstack-cinder | 17:39 | |
| *** annegentle has quit IRC | 17:41 | |
| *** piyanai has quit IRC | 17:47 | |
| openstackgerrit | Rajini Ram proposed openstack/cinder: Eqlx Driver: Revised the ssh code to use common utils https://review.openstack.org/203720 | 17:47 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder-specs: Brick add new Connector volume APIs https://review.openstack.org/204180 | 17:48 |
| *** bitblt has joined #openstack-cinder | 17:48 | |
| *** mylu has quit IRC | 17:50 | |
| *** bitblt has quit IRC | 17:51 | |
| *** bitblt has joined #openstack-cinder | 17:51 | |
| *** annegent_ has quit IRC | 17:51 | |
| *** jasondotstar has quit IRC | 17:52 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Add new Connector APIs for path validation https://review.openstack.org/199764 | 17:53 |
| *** markvoelker has joined #openstack-cinder | 17:55 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Fix Consistency Group issues https://review.openstack.org/201685 | 17:55 |
| *** bitblt has quit IRC | 17:58 | |
| *** bitblt has joined #openstack-cinder | 17:58 | |
| *** mylu has joined #openstack-cinder | 17:59 | |
| *** erlon has joined #openstack-cinder | 18:01 | |
| *** piyanai has joined #openstack-cinder | 18:02 | |
| *** annegentle has joined #openstack-cinder | 18:02 | |
| *** jaypipes has joined #openstack-cinder | 18:02 | |
| *** boris-42 has joined #openstack-cinder | 18:02 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Enable cinder-manage to remove old services from the database https://review.openstack.org/197613 | 18:03 |
| *** mylu has quit IRC | 18:06 | |
| *** lpabon has quit IRC | 18:08 | |
| *** mylu has joined #openstack-cinder | 18:09 | |
| openstackgerrit | Anthony Lee proposed openstack/os-brick: WIP: Add new Connector APIs for path validation https://review.openstack.org/199764 | 18:09 |
| openstackgerrit | Anthony Lee proposed openstack/os-brick: Adding a tool that allows querying of LUNs https://review.openstack.org/204191 | 18:09 |
| *** diemt has quit IRC | 18:10 | |
| *** angela-s has joined #openstack-cinder | 18:12 | |
| *** skylerberg has joined #openstack-cinder | 18:13 | |
| *** tbarron_ has joined #openstack-cinder | 18:14 | |
| *** lpetrut has joined #openstack-cinder | 18:14 | |
| *** jamielennox has joined #openstack-cinder | 18:15 | |
| *** jamielennox is now known as jamielennox|away | 18:16 | |
| *** david-lyle has quit IRC | 18:21 | |
| *** jamielennox|away is now known as jamielennox | 18:23 | |
| *** swamireddy has joined #openstack-cinder | 18:23 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder-specs: Brick add new Connector volume APIs https://review.openstack.org/204180 | 18:27 |
| *** bswartz has joined #openstack-cinder | 18:30 | |
| *** annegentle has quit IRC | 18:30 | |
| openstackgerrit | Ryan Lucio proposed openstack/cinder: VMEM v7000: external-head storage pool and volume type selection https://review.openstack.org/195773 | 18:35 |
| jgriffith | hemna: 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 |
| jgriffith | hemna: That sounded to me like a "service" which is fine, just was trying to understand | 18:36 |
| jgriffith | hemna: as opposed to "cinder check-paths <volume-uuid>" --> brick.initiator:foo.get_path_info(<volume-uuid>) | 18:37 |
| *** piyanai has quit IRC | 18:37 | |
| *** diogogmt_ has joined #openstack-cinder | 18:38 | |
| *** diogogmt has quit IRC | 18:39 | |
| *** diogogmt_ is now known as diogogmt | 18:39 | |
| jgriffith | hemna: 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 though | 18:40 |
| *** piyanai has joined #openstack-cinder | 18:42 | |
| *** piyanai has quit IRC | 18:42 | |
| hemna | hey | 18:42 |
| hemna | so, I'm not exactly sure where the 'right' place is for that flask script to live to be honest. | 18:43 |
| jgriffith | hemna: yeah... I'm not sure I really understand it, so that's why I asked | 18:43 |
| hemna | I don't want brick to be a service | 18:43 |
| jgriffith | hemna: it seems like that requires implementing a server, manager, api and the whole deal | 18:43 |
| hemna | it's a library | 18:43 |
| *** amick has quit IRC | 18:44 | |
| jgriffith | hemna: ok, cool... I didn't think so but thought I should ask. | 18:44 |
| hemna | but we do need some way of verifying on remote systems that paths have correctly gone away (re: live migration, rebuild) testing. | 18:44 |
| jgriffith | hemna: I mean, it might be fine if it were... but anyway. | 18:44 |
| jgriffith | hemna: sure, that's what I said in the comment; I totally think that's a great idea | 18:44 |
| hemna | the flask thing is just something we are chewing on and I put it in the spec to try and start the discussion and get feedback | 18:44 |
| jgriffith | hemna: got ya | 18:45 |
| hemna | my initial plan was to put it in a tools dir in os-brick | 18:45 |
| jgriffith | hemna: so you were saying brick already has all this info... where is that in brick? | 18:45 |
| *** mylu has quit IRC | 18:45 | |
| jgriffith | hemna: 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 |
| hemna | so that 'info' is usually internally available inside each of the connector's connect_volume | 18:46 |
| hemna | and then thrown away | 18:46 |
| hemna | after connect_volume is done | 18:46 |
| jgriffith | hemna: what if we held on to it and allowed it to be queried? | 18:46 |
| hemna | my patch basically refactors connect_volume and makes the paths available as a method call, which connect_volume also uses. | 18:46 |
| jgriffith | hemna: just in memory | 18:46 |
| hemna | my patch actually makes those method calls on each of the connectors | 18:47 |
| hemna | the flask script was just a mechanism to make it remotely available | 18:47 |
| jgriffith | hemna: wait.. which method calls? | 18:47 |
| jgriffith | hemna: Oh... you already have a patch up that implements all of this? | 18:47 |
| hemna | the get_volume_paths(), get_all_available_volumes. | 18:47 |
| hemna | yes | 18:47 |
| hemna | I already have a patch up that does all of this | 18:47 |
| jgriffith | hemna: Oh, I'm sorry.. I didn't see it | 18:48 |
| hemna | and leeantho has a patch for the flask script. | 18:48 |
| hemna | it's been a WIP for a while | 18:48 |
| jgriffith | hemna: I'll go look instead of asking stupid questions | 18:48 |
| hemna | https://review.openstack.org/#/c/199764/ | 18:48 |
| hemna | hrmm, I thought I removed 'WIP' from the title, but it's fine that it's in there until the spec is approved anwyway | 18:48 |
| jgriffith | hemna: yah, so this seems pretty straight forward to just make a cinder-api call and a nova api call that can access this info | 18:49 |
| hemna | even if it's just for testing purpose? | 18:49 |
| hemna | disable via policy I guess ? | 18:49 |
| jgriffith | hemna: well, I don't see why it wouldn't be useful for "other" internal cleanup and such | 18:50 |
| jgriffith | hemna: and perhaps admin policy even if you wanted to go crazy with it :) | 18:50 |
| hemna | ok, I hadn't thought of that | 18:50 |
| jgriffith | hemna: but it looks like you only implemented the filesystem shares side of it | 18:51 |
| jgriffith | hemna: 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 node | 18:51 |
| hemna | I don't know what to do for rbd | 18:51 |
| *** mylu has joined #openstack-cinder | 18:51 | |
| jgriffith | hemna: me neither :) But I don't see iSCSI either? | 18:52 |
| openstackgerrit | Curt Bruns proposed openstack/cinder: Add deactivate step to extend_lv https://review.openstack.org/198813 | 18:52 |
| hemna | I have it for the ISCSIConnector | 18:52 |
| hemna | get_volume_paths | 18:52 |
| *** jasondotstar has joined #openstack-cinder | 18:52 | |
| *** mylu has quit IRC | 18:52 | |
| hemna | the parent class implements the get_all_available_volumes() | 18:52 |
| jgriffith | hemna: https://review.openstack.org/#/c/199764/5/os_brick/initiator/connector.py | 18:53 |
| jgriffith | pass ? | 18:53 |
| jgriffith | DERP | 18:53 |
| jgriffith | I see it | 18:53 |
| jgriffith | LOL | 18:53 |
| hemna | :) | 18:53 |
| hemna | I'm not in a big hurry to land this, we can talk about it at the mid cycle | 18:54 |
| hemna | I'm trying to think of adding something to our 3rd party CI to show it being used | 18:54 |
| hemna | as a separate standalone test for the time being. | 18:54 |
| hemna | I'd actually like to see if all of our tests are not leaving behind orphaned paths. :P | 18:55 |
| *** s0nea has quit IRC | 18:55 | |
| jgriffith | hemna: they are FWIW | 18:56 |
| hemna | :( | 18:56 |
| *** lpetrut has quit IRC | 18:56 | |
| jgriffith | hemna: well.. not "all" of them but there are some left behind after every tempest run | 18:56 |
| hemna | hopefully, with this, we can pinpoint which tests actually leave the paths behind | 18:56 |
| hemna | and those tests, should be marked as failure | 18:57 |
| jgriffith | hemna: yeah, I was thinking you'd just add a call to this in the cleanup | 18:57 |
| *** s0nea has joined #openstack-cinder | 18:57 | |
| jgriffith | hemna: Oh.. it just uses the old iscsiadm | 18:59 |
| *** agarciam has joined #openstack-cinder | 18:59 | |
| hemna | yah | 18:59 |
| jgriffith | hemna: 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 |
| hemna | the connect_volume code first builds a list of possible paths that might show up, and then loops looking for one of them. | 19:00 |
| hemna | I just moved that code out and made it available to call basically | 19:00 |
| *** chenying has joined #openstack-cinder | 19:00 | |
| jgriffith | hemna: it's kinda tricky no? I mean, the check will basically create a new connection if the target is reachable I think. | 19:01 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: WIP: Add new Connector APIs for path validation https://review.openstack.org/199764 | 19:01 |
| hemna | that connection should already be there, if the volume is attached | 19:01 |
| hemna | if it's not attached..... | 19:01 |
| *** lpetrut has joined #openstack-cinder | 19:01 | |
| hemna | if 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 work | 19:02 |
| hemna | review is very welcomed though! :) | 19:02 |
| jgriffith | hemna: nahh... I wasn't saying that at all (not use iscsiadm) | 19:03 |
| hemna | I hadn't thought about a non-existing connection (not attached volume) | 19:03 |
| *** breitz has quit IRC | 19:03 | |
| *** mylu has joined #openstack-cinder | 19:03 | |
| *** e0ne has joined #openstack-cinder | 19:04 | |
| hemna | I guess iscsciadm -m session dumps a list of existing sessions | 19:05 |
| jgriffith | hemna: 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 on | 19:05 |
| hemna | can probably parse that output and see if there is one for the target_portals in the connection_info passed in. | 19:06 |
| hemna | yah, agreed. | 19:06 |
| e0ne | thingee: 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 |
| jgriffith | hemna: yes, you'd need to separate out the session call from that method | 19:06 |
| e0ne | thingee: there're stats in a commit message | 19:06 |
| *** jwang has joined #openstack-cinder | 19:06 | |
| hemna | I'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 query | 19:06 |
| hemna | ok cool. thanks! | 19:06 |
| jgriffith | hemna: sure | 19:07 |
| *** breitz has joined #openstack-cinder | 19:08 | |
| e0ne | jgriffith: do you have any update with API for ironic integration/attach w/o nova? | 19:12 |
| e0ne | jgriffith: i'm goint to implement cli tool for it this week | 19:13 |
| jgriffith | e0ne: crud... I need to write that real quick | 19:13 |
| jgriffith | e0ne: we still need to sync up on that | 19:13 |
| e0ne | jgriffith: any working poc will be useful | 19:13 |
| jgriffith | e0ne: make sure I understand what you're expecting from the consumer side | 19:13 |
| jgriffith | e0ne: I was just going to expose an API call that gets the connection info.. that's all | 19:13 |
| e0ne | jgriffith: sure. can we do it tomorrow? /cc hemna | 19:13 |
| jgriffith | e0ne: sounds good | 19:14 |
| hemna | sweet | 19:14 |
| jgriffith | e0ne: I'll try to have some code ready | 19:14 |
| *** amit213 has quit IRC | 19:14 | |
| e0ne | jgriffith: great! | 19:14 |
| *** amit213 has joined #openstack-cinder | 19:14 | |
| jgriffith | e0ne: hemna I was basically going to just expose a version of intiialize_connection | 19:15 |
| *** Liu has quit IRC | 19:15 | |
| *** chenying has quit IRC | 19:15 | |
| jgriffith | e0ne: hemna or even just modify policy so it can be used as is currently without anything else needed per-say | 19:15 |
| hemna | does that include the reserve_volume call internally? | 19:15 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Fix legacy bug, init_conn bug and REST API bug. https://review.openstack.org/204217 | 19:16 |
| hemna | ok lunch...... | 19:16 |
| jgriffith | hemna: yeah... a "glob" of the two | 19:17 |
| jgriffith | hemna: and lunch is a great idea | 19:17 |
| *** breitz has quit IRC | 19:17 | |
| *** jasondotstar has quit IRC | 19:18 | |
| *** breitz has joined #openstack-cinder | 19:18 | |
| *** mylu has quit IRC | 19:20 | |
| *** chhavi has quit IRC | 19:20 | |
| *** mylu has joined #openstack-cinder | 19:21 | |
| *** dustins_ has joined #openstack-cinder | 19:21 | |
| *** mylu has quit IRC | 19:22 | |
| *** lpetrut has quit IRC | 19:23 | |
| *** Yogi1 has joined #openstack-cinder | 19:23 | |
| *** mylu has joined #openstack-cinder | 19:24 | |
| *** dustins has quit IRC | 19:25 | |
| *** annegentle has joined #openstack-cinder | 19:25 | |
| vilobhmm | ericsonsantos : ping | 19:28 |
| vilobhmm | ericksonsantos : ^^ | 19:28 |
| *** david-lyle has joined #openstack-cinder | 19:34 | |
| *** annegentle has quit IRC | 19:36 | |
| *** annegent_ has joined #openstack-cinder | 19:37 | |
| *** piyanai has joined #openstack-cinder | 19:38 | |
| *** mylu has quit IRC | 19:45 | |
| *** mylu has joined #openstack-cinder | 19:47 | |
| *** sghanekar_ has quit IRC | 19:49 | |
| *** tsekiyama has joined #openstack-cinder | 19:52 | |
| *** edtubill has joined #openstack-cinder | 19:52 | |
| *** piyanai has quit IRC | 19:55 | |
| *** piyanai has joined #openstack-cinder | 19:55 | |
| *** bitblt has quit IRC | 19:58 | |
| thingee | hemna: preparing os-brick 0.3.1 | 19:59 |
| openstackgerrit | xing-yang proposed openstack/python-cinderclient: CLI: Non-disruptive backup https://review.openstack.org/204232 | 20:00 |
| edmondsw | jungleboyj, please look at the changes I made per your comments in https://review.openstack.org/#/c/203237/ | 20:00 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Fix legacy bug, init_conn bug and REST API bug. https://review.openstack.org/204217 | 20:00 |
| *** e0ne has quit IRC | 20:02 | |
| *** mylu has quit IRC | 20:03 | |
| *** Lee1092 has quit IRC | 20:05 | |
| thingee | hemna: https://review.openstack.org/204236 | 20:06 |
| thingee | mtanino: going to get to the capabilities patch today, thanks for your patience | 20:07 |
| mtanino | thingee: sure. Waiting for your comments. Thank you | 20:07 |
| mtanino | thingee: seems still you are hard woking... | 20:08 |
| thingee | mtanino: ha I don't think so, but thanks | 20:08 |
| thingee | mtanino: I feel like I've been missing around reviews and IRC lately because of personal stuff. | 20:09 |
| *** skylerberg has quit IRC | 20:09 | |
| mtanino | thingee: liberty-2 is comming week. welcome back:) | 20:10 |
| *** Guest78809 has quit IRC | 20:11 | |
| *** tbarron has quit IRC | 20:13 | |
| *** tbarron_ is now known as tbarron | 20:13 | |
| *** amoturi has quit IRC | 20:15 | |
| Swanson | Check is just a hellscape lately. | 20:16 |
| ericksonsantos | vilobhmm, sorry, wasn't here | 20:16 |
| *** sghanekar_ has joined #openstack-cinder | 20:18 | |
| hemna | thingee, thanks | 20:26 |
| *** e0ne has joined #openstack-cinder | 20:27 | |
| *** s0nea has quit IRC | 20:28 | |
| *** s0nea has joined #openstack-cinder | 20:32 | |
| *** amick has joined #openstack-cinder | 20:35 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Continue conversion to volume object https://review.openstack.org/201404 | 20:36 |
| patrickeast | i 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#L1710 | 20:37 |
| patrickeast | and then if the driver did retype it then we don’t try and migrate | 20:37 |
| patrickeast | but for a backend like pure where we don’t really do anything for retypes, is it then ok to just implement retype and return true | 20:37 |
| patrickeast | ? | 20:37 |
| *** _cjones_ has quit IRC | 20:38 | |
| *** lpetrut has joined #openstack-cinder | 20:38 | |
| *** _cjones_ has joined #openstack-cinder | 20:38 | |
| smcginnis | patrickeast: I think you just don't implement it then. | 20:39 |
| smcginnis | patrickeast: That's how we had it before because we didn't have anything we could do. | 20:39 |
| patrickeast | smcginnis: i mean… it works if i dont implement it | 20:39 |
| *** piyanai has quit IRC | 20:39 | |
| patrickeast | smcginnis: but then it will do a slow migrate to make a new volume of the right type | 20:39 |
| patrickeast | smcginnis: instead of doing nothing, which would also be valid | 20:39 |
| smcginnis | patrickeast: But you don't have any other way to get to the new type, right? | 20:39 |
| patrickeast | smcginnis: well thats the thing… as far as our backend is concerned there is no difference | 20:40 |
| smcginnis | patrickeast: What is being retyped then? | 20:40 |
| patrickeast | smcginnis: we don’t have any knobs or dials that change on the volume based on type | 20:40 |
| patrickeast | smcginnis: cinder meta-data about the volume basically | 20:40 |
| smcginnis | patrickeast: Then it would have to be to another backend, even if the other backend is another Pure array. | 20:41 |
| patrickeast | smcginnis: this came up because someone was trying to change the cinder qos stuff | 20:41 |
| smcginnis | patrickeast: So the slow migration is the only way to get there. | 20:41 |
| patrickeast | which doesn’t actually change anything on our backend | 20:41 |
| smcginnis | patrickeast: Ah, QoS... | 20:41 |
| *** lpetrut has quit IRC | 20:41 | |
| patrickeast | but even if they had two types | 20:41 |
| smcginnis | patrickeast: In that case I believe you can just check that that is the only thing changing and return true. | 20:41 |
| patrickeast | like flash and tier-1 | 20:41 |
| patrickeast | or whatever | 20:41 |
| patrickeast | and the same backend fulfilled the types | 20:41 |
| patrickeast | we would just want to do a no-op | 20:42 |
| patrickeast | right? | 20:42 |
| Swanson | Yeah. If you don't want the volume to move tell cinder you did it. | 20:42 |
| smcginnis | patrickeast: 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 IRC | 20:42 | |
| smcginnis | patrickeast: Anything else you return false and cinder does the migrate. | 20:42 |
| patrickeast | but it looks like the driver is only called if the volume wiont move | 20:42 |
| patrickeast | wont* | 20:42 |
| smcginnis | patrickeast: Right, only if the retype is to the same backend. | 20:43 |
| patrickeast | smcginnis: ok, i think i got it | 20:43 |
| patrickeast | smcginnis: Swanson: thanks! | 20:43 |
| smcginnis | patrickeast: It confused me at first too. :) | 20:43 |
| *** piyanai has joined #openstack-cinder | 20:45 | |
| *** dustins_ has quit IRC | 20:45 | |
| *** changbl has quit IRC | 20:46 | |
| *** e0ne has quit IRC | 20:48 | |
| *** ronis has quit IRC | 20:49 | |
| *** amick has left #openstack-cinder | 20:53 | |
| *** daneyon has quit IRC | 20:54 | |
| *** amick has joined #openstack-cinder | 20:54 | |
| *** amick is now known as amickus | 20:55 | |
| *** julim_ has quit IRC | 20:55 | |
| *** agarciam has quit IRC | 20:56 | |
| *** daneyon has joined #openstack-cinder | 20:56 | |
| *** daneyon_ has joined #openstack-cinder | 21:00 | |
| *** Guest78809 has joined #openstack-cinder | 21:01 | |
| *** dustins has joined #openstack-cinder | 21:02 | |
| *** bswartz has quit IRC | 21:03 | |
| *** daneyon has quit IRC | 21:04 | |
| *** piyanai has quit IRC | 21:04 | |
| *** daneyon has joined #openstack-cinder | 21:04 | |
| *** simondodsley has quit IRC | 21:06 | |
| *** daneyon_ has quit IRC | 21:07 | |
| *** xyang1 has quit IRC | 21:11 | |
| *** annegent_ has quit IRC | 21:12 | |
| *** e0ne has joined #openstack-cinder | 21:14 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: POC for scalable backup https://review.openstack.org/203291 | 21:15 |
| *** e0ne has quit IRC | 21:15 | |
| *** pv_ has joined #openstack-cinder | 21:16 | |
| *** e0ne has joined #openstack-cinder | 21:16 | |
| *** dustins has quit IRC | 21:18 | |
| patrickeast | jgriffith: 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-cinder | 21:22 | |
| *** Yogi1 has quit IRC | 21:23 | |
| *** jungleboyj has quit IRC | 21:31 | |
| *** e0ne has quit IRC | 21:33 | |
| *** annegentle has joined #openstack-cinder | 21:34 | |
| *** sghanekar_ has quit IRC | 21:35 | |
| *** annegentle has quit IRC | 21:35 | |
| *** annegent_ has joined #openstack-cinder | 21:35 | |
| *** jasondot_ has joined #openstack-cinder | 21:36 | |
| *** jasondotstar has quit IRC | 21:39 | |
| jgriffith | patrickeast: well, I'm kinda stuck between what your'e doing and the stuff added to Glance now | 21:42 |
| jgriffith | patrickeast: there's an awful lot of overlap there it seems | 21:42 |
| patrickeast | jgriffith: yea i agree | 21:43 |
| patrickeast | jgriffith: i actually went back and took a hard look at whether or not we still needed a cache or anything | 21:43 |
| jgriffith | patrickeast: and? :) | 21:43 |
| patrickeast | jgriffith: and i think the answer is still yes, primarily because of the image formats | 21:43 |
| jgriffith | patrickeast: Oh? Interesting.. why is that? | 21:44 |
| patrickeast | jgriffith: even if cinder is a glance backend if its got a bunch of qcow2 images we can’t just clone em | 21:44 |
| patrickeast | jgriffith: we still need to cache raw versions | 21:44 |
| jgriffith | patrickeast: oh...sure | 21:44 |
| jgriffith | patrickeast: but I guess what I was proposing is that the cache is obsolete | 21:44 |
| patrickeast | jgriffith: and i went down the path of putting those raw ones into glance as another url | 21:44 |
| *** piyanai has joined #openstack-cinder | 21:44 | |
| jgriffith | patrickeast: if you can configure glance to use Cinder as one of 'n' backends, you use templates in Glance of the images you care about | 21:45 |
| patrickeast | jgriffith: right, but that only gives you fast volume creation from images if they are stored in glance as raw | 21:45 |
| patrickeast | jgriffith: maybe faster than right now | 21:46 |
| patrickeast | jgriffith: but not insta-clone kind of fast | 21:46 |
| patrickeast | jgriffith: unless i’m misunderstanding something | 21:46 |
| jgriffith | patrickeast: 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 |
| patrickeast | jgriffith: so keep both? | 21:46 |
| jgriffith | patrickeast: 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 one | 21:47 |
| patrickeast | jgriffith: or how would you then take one and upload it to a compute node or something for a non-cinder usage? | 21:47 |
| jgriffith | patrickeast: it just moves what you're working on a bit further put the stack | 21:47 |
| openstackgerrit | Raunak Kumar proposed openstack/cinder: Add extra spec capability for Nimble Cinder Driver https://review.openstack.org/198853 | 21:47 |
| patrickeast | jgriffith: yea i’m with you there, that works great for glance images configued as raw | 21:47 |
| jgriffith | patrickeast: right now you can't really... and frankly I don't care about that so much :) | 21:47 |
| patrickeast | jgriffith: but thats the eventual goal | 21:48 |
| jgriffith | patrickeast: maybe my idea doesn't work. I'll take another look at both patches and see what I can figure out | 21:48 |
| patrickeast | jgriffith: and the direction that code is going | 21:48 |
| patrickeast | jgriffith: so… i’m totally open to ditching the cache if its redundant or totally a subset of the other functionality | 21:49 |
| patrickeast | jgriffith: i’m just not quite seeing it yet | 21:49 |
| patrickeast | /me might be slow | 21:49 |
| *** tater_ has joined #openstack-cinder | 21: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 invalid | 21:52 |
| tater_ | Thanks for any input | 21:52 |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Enable cinder-manage to remove services https://review.openstack.org/197613 | 21:52 |
| openstackgerrit | Anthony Lee proposed openstack/os-brick: Adding a tool that allows querying of LUNs https://review.openstack.org/204191 | 21:53 |
| jgriffith | patrickeast: nah... it might not work | 21:53 |
| jgriffith | just an idea | 21:53 |
| jgriffith | patrickeast: i.e. the glance thing might not work | 21:53 |
| *** kjnelson has quit IRC | 21:53 | |
| patrickeast | jgriffith: bummer | 21:53 |
| *** piyanai has quit IRC | 21:53 | |
| patrickeast | jgriffith: i do like the idea of a single code path kind of solution | 21:53 |
| jgriffith | patrickeast: yeah, I'm mixed on that TBH | 21:54 |
| patrickeast | jgriffith: and it might be ok we just keep two copies of every image on the cinder backend | 21:54 |
| patrickeast | one raw, one original | 21:54 |
| jgriffith | patrickeast: I mean, I like single code path, but with enough flexibility for the drivers to do what they need to do | 21:54 |
| jgriffith | patrickeast: and frankly own the responsibility of making it work | 21:54 |
| patrickeast | jgriffith: true | 21:55 |
| jgriffith | patrickeast: I looked at just doing a Cinder cache like that, but it was kinda useless... you still had to do the dd to a volume | 21:55 |
| jgriffith | patrickeast: for some, that's the best you get, but for others that's not the case | 21:55 |
| jgriffith | patrickeast: which is why I ended up just doing it all in the driver; but I think your approach still has a middle ground | 21:56 |
| patrickeast | jgriffith: yea, i guess thats the hard part… everyone is going to have their own secret sauce | 21:56 |
| jgriffith | I just don't want to have duplicate stuff out there | 21:56 |
| patrickeast | jgriffith: same | 21:56 |
| patrickeast | jgriffith: i refactored my stuff to re-use as much as i could, both use the same volume creation code and all that | 21:56 |
| jgriffith | patrickeast: right... which is why I ended up just doing it by overriding that call. | 21:56 |
| jgriffith | patrickeast: Yeah, honestly I haven't looked at your patch in a while, let me look at it when I get back to my desk | 21:57 |
| jgriffith | 10 minutes or so (I hope) | 21:57 |
| patrickeast | jgriffith: sounds good | 21:58 |
| patrickeast | jgriffith: thanks again for helping me get this figured out | 21:58 |
| jgriffith | patrickeast: well... sadly I'm not sure I'm being much help at this point :( | 21:58 |
| patrickeast | jgriffith: haha, your providing a good sort of check to make sure i’m still doing sane things and not wasting time | 21:59 |
| patrickeast | jgriffith: very useful imo | 21: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 yet | 21:59 |
| patrickeast | pv_: if you have a local or internal git repo you can override CINDER_REPO and CINDER_BRANCH to pull your code for you | 22:00 |
| *** jaypipes has quit IRC | 22:00 | |
| patrickeast | pv_: or just let it use master and then go put your code in after devstack has started up | 22:00 |
| *** markus_z has quit IRC | 22:00 | |
| *** kragniz has quit IRC | 22:01 | |
| *** sghanekar_ has joined #openstack-cinder | 22:01 | |
| *** piyanai has joined #openstack-cinder | 22:01 | |
| *** edtubill has left #openstack-cinder | 22:01 | |
| *** dims_ has quit IRC | 22:01 | |
| *** kragniz has joined #openstack-cinder | 22:02 | |
| *** jwcroppe has quit IRC | 22:02 | |
| *** jwcroppe has joined #openstack-cinder | 22:03 | |
| *** pv_ has quit IRC | 22:03 | |
| *** jgriffith has quit IRC | 22:04 | |
| *** jgriffith has joined #openstack-cinder | 22:05 | |
| *** bnemec has quit IRC | 22:06 | |
| *** edmondsw has quit IRC | 22:07 | |
| *** jwcroppe has quit IRC | 22:07 | |
| *** daneyon_ has joined #openstack-cinder | 22:09 | |
| *** daneyon has quit IRC | 22:12 | |
| openstackgerrit | Anthony Lee proposed openstack/os-brick: Adding a tool that allows querying of LUNs https://review.openstack.org/204191 | 22:15 |
| *** akerr has quit IRC | 22:18 | |
| *** ganso_ has quit IRC | 22:18 | |
| patrickeast | asselin: 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 IRC | 22:19 | |
| patrickeast | asselin: or are there any patches in flight i need to snag too? | 22:19 |
| asselin | patrickeast, only one thing missing right now is the zuul smtp settings | 22:19 |
| asselin | those just merged earlier | 22:19 |
| asselin | upstream, need to push that to my master | 22:19 |
| patrickeast | ok cool | 22:20 |
| asselin | otherwise go for it. I updated all of mine last week | 22:20 |
| *** rhagarty_ has quit IRC | 22:20 | |
| patrickeast | awesome, everything working well? | 22:20 |
| patrickeast | just the other day ours decided to stop releaing floating ip addresses : / | 22:21 |
| patrickeast | i’m hoping an upgrade to the latest everything will help | 22:21 |
| *** rhagarty has joined #openstack-cinder | 22:21 | |
| *** piyanai_ has joined #openstack-cinder | 22:22 | |
| thingee | mtanino: 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 bits | 22:22 |
| thingee | mtanino: if that allows you to move forward | 22:22 |
| *** piyanai has quit IRC | 22:23 | |
| *** piyanai_ is now known as piyanai | 22:23 | |
| mtanino | thingee: thank you for the review. Please comment on the review. I will fix and post updated patch. | 22:23 |
| asselin | patrickeast, yes, but that's nodepool which is not yet part of this. | 22:23 |
| *** sgotliv has quit IRC | 22:24 | |
| patrickeast | asselin: hmm thats a good point | 22:24 |
| *** mgarza has quit IRC | 22:24 | |
| thingee | mtanino: do you have any opinion with these capabilities being reported to glance's metadata catalog? | 22:24 |
| patrickeast | asselin: oh well, upgrading anyway! | 22:24 |
| thingee | mtanino: see dulek's comment https://review.openstack.org/#/c/183947/ | 22:25 |
| *** rhagarty_ has joined #openstack-cinder | 22:25 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Adding a tool that allows querying of LUNs https://review.openstack.org/204191 | 22:26 |
| mtanino | thingee: this one? http://paste.openstack.org/show/281270/ | 22:27 |
| mtanino | thingee: I agree on his comment to use same format of Glance Metadata Catalog format. | 22:28 |
| mtanino | thingee: Enable to reuse Horizon widget is reasonable, I think | 22:28 |
| thingee | mtanino: are we able to use the format without having to store the information in glance? | 22:30 |
| thingee | mtanino: or do you think it makes sense to store it in glance? | 22:30 |
| tsekiyama | thingee: I believe dulek's suggestion is just for reusing the same Horizon code to parse the glance metadata catalog for the capabilities format | 22:31 |
| tsekiyama | thingee: it's not to store the information into Glance | 22:31 |
| thingee | tsekiyama: ah ok, wasn't sure if they were mutually exclusive | 22:32 |
| mtanino | thingee: 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 |
| asselin | patrickeast, ok merged the 1 liner for smtp settings | 22:43 |
| patrickeast | asselin: sweet | 22:44 |
| asselin | patrickeast, i'll be on vacation coming up, so let me know of any issues | 22:44 |
| patrickeast | asselin: will do, i’m just about ready to run the install script… getting some stuf straightened out with my wrapper scripts | 22:45 |
| *** mtanino has quit IRC | 22:49 | |
| *** haomaiwang has quit IRC | 22:49 | |
| *** crose has quit IRC | 22:52 | |
| *** piyanai has quit IRC | 22:53 | |
| *** angela-s has quit IRC | 22:54 | |
| *** krypton has joined #openstack-cinder | 23:04 | |
| *** pv_ has joined #openstack-cinder | 23:05 | |
| *** krypton has left #openstack-cinder | 23:06 | |
| *** krypton_ has joined #openstack-cinder | 23:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 23:12 |
| *** bswartz has joined #openstack-cinder | 23:12 | |
| *** jasondot_ has quit IRC | 23:12 | |
| patrickeast | asselin: aww, ran into an error right off the bat http://paste.openstack.org/show/397471/ | 23:14 |
| * asselin looks | 23:14 | |
| patrickeast | asselin: 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 contents | 23:14 |
| patrickeast | err https://github.com/rasselin/os-ext-testing/blob/master/puppet/modules/os_ext_testing/manifests/master.pp#L94 | 23:14 |
| asselin | patrickeast, update your install_master.sh script | 23:15 |
| asselin | should probably add a note about that | 23:15 |
| patrickeast | ohh funny | 23:15 |
| patrickeast | at some point i had my wrapper script point to the one in os-ext-testing | 23:16 |
| patrickeast | i guess i changed it back | 23:16 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/200898 | 23:16 |
| *** jasondotstar has joined #openstack-cinder | 23:17 | |
| *** diogogmt has quit IRC | 23:17 | |
| *** DericHorn-HP has joined #openstack-cinder | 23:18 | |
| patrickeast | asselin: much better now | 23:18 |
| patrickeast | thanks! | 23:18 |
| asselin | I probably should add those in for better backwards compatibility/migration | 23:19 |
| pv_ | hey so i updated the CINDER_REPO to my local git repo and CINDER_BRANCH to master | 23:19 |
| pv_ | and my stack.sh is giving me an error and exiting saying git call failed | 23:19 |
| pv_ | for git fetch origin | 23:19 |
| pv_ | any suggestions on why? | 23:19 |
| patrickeast | pv_: means it probably couldn’t clone the repo correctly | 23:20 |
| patrickeast | pv_: try manually cloning it with the same commands stack.sh is using | 23:20 |
| patrickeast | pv_: and make sure that works | 23:20 |
| patrickeast | pv_: you can see what its doing in the source here https://github.com/openstack-dev/devstack/blob/master/functions-common#L502 | 23:21 |
| *** annashen has quit IRC | 23:22 | |
| *** boris-42 has quit IRC | 23:22 | |
| *** jungleboyj has joined #openstack-cinder | 23:24 | |
| pv_ | thanks ill check it out | 23:25 |
| *** dannywilson has quit IRC | 23:26 | |
| *** david-lyle has quit IRC | 23:28 | |
| *** jungleboyj has quit IRC | 23:36 | |
| *** tater_ has quit IRC | 23:38 | |
| *** jasondotstar has quit IRC | 23:47 | |
| *** jasondotstar has joined #openstack-cinder | 23:50 | |
| *** ctracey has quit IRC | 23:53 | |
| *** topshare has joined #openstack-cinder | 23:53 | |
| *** hemna is now known as hemnafk | 23:53 | |
| *** tbarron has quit IRC | 23:54 | |
| *** ctracey has joined #openstack-cinder | 23:55 | |
| *** tsekiyam_ has joined #openstack-cinder | 23:55 | |
| *** tsekiyama has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!