| *** logan2 has quit IRC | 00:01 | |
| *** topshare has quit IRC | 00:07 | |
| *** logan2 has joined #openstack-cinder | 00:08 | |
| *** Apoorva has quit IRC | 00:13 | |
| openstackgerrit | Kenji Yasui proposed openstack/cinder: Cleanup unused method fake_get_target https://review.openstack.org/200378 | 00:32 |
|---|---|---|
| *** markvoelker has quit IRC | 00:32 | |
| *** smoriya has joined #openstack-cinder | 00:36 | |
| *** dims has joined #openstack-cinder | 00:42 | |
| *** dims has quit IRC | 00:44 | |
| *** salv-orlando has joined #openstack-cinder | 00:49 | |
| *** zhenguo has joined #openstack-cinder | 00:49 | |
| *** openstackgerrit has quit IRC | 00:51 | |
| *** openstackgerrit has joined #openstack-cinder | 00:52 | |
| *** topshare has joined #openstack-cinder | 00:53 | |
| *** salv-orlando has quit IRC | 00:54 | |
| *** heyun has joined #openstack-cinder | 01:00 | |
| *** laughterwym has joined #openstack-cinder | 01:05 | |
| *** ericksonsantos has quit IRC | 01:09 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:10 | |
| *** lixiaoy11 has joined #openstack-cinder | 01:15 | |
| *** lixiaoy1 has quit IRC | 01:17 | |
| *** dims has joined #openstack-cinder | 01:24 | |
| *** dims_ has joined #openstack-cinder | 01:28 | |
| *** dims has quit IRC | 01:31 | |
| *** davechen has joined #openstack-cinder | 01:38 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: Storwize_svc_npiv_compatibility_mode default value change. https://review.openstack.org/198692 | 01:41 |
| openstackgerrit | Seiji Aguchi proposed openstack/cinder: Remove resource lock operation for HBSD https://review.openstack.org/185003 | 01:43 |
| *** xiangfeiz_ has joined #openstack-cinder | 01:47 | |
| *** rushil has joined #openstack-cinder | 01:54 | |
| *** vilobhmm1 has quit IRC | 02:01 | |
| *** vilobhmm has joined #openstack-cinder | 02:02 | |
| *** salv-orlando has joined #openstack-cinder | 02:03 | |
| *** salv-orlando has quit IRC | 02:08 | |
| *** DCWilliams_VA has joined #openstack-cinder | 02:08 | |
| *** DCWilliams_VA has quit IRC | 02:08 | |
| *** salv-orlando has joined #openstack-cinder | 02:08 | |
| *** salv-orlando has quit IRC | 02:10 | |
| *** jseiler has quit IRC | 02:13 | |
| *** jseiler has joined #openstack-cinder | 02:13 | |
| *** harlowja_at_home has joined #openstack-cinder | 02:25 | |
| *** tobe has joined #openstack-cinder | 02:29 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 02:31 |
| *** Apoorva has joined #openstack-cinder | 02:35 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Add Cinder internal tenant support https://review.openstack.org/199352 | 02:41 |
| openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 02:41 |
| *** vincent_hou has joined #openstack-cinder | 02:44 | |
| vincent_hou | patrickeast: Hi. | 02:45 |
| patrickeast | vincent_hou: hey | 02:45 |
| *** daneyon has joined #openstack-cinder | 02:46 | |
| patrickeast | vincent_hou: oh, just saw your comments on that internal tenant review | 02:46 |
| vincent_hou | I have one question for the patch https://review.openstack.org/#/c/199352/ | 02:46 |
| vincent_hou | That is quite important for me. :-) | 02:46 |
| *** kkeuning has joined #openstack-cinder | 02:47 | |
| patrickeast | vincent_hou: yep, so if they are none, the call to get the request context will return none | 02:47 |
| vincent_hou | Also invisible to cinder list? | 02:47 |
| patrickeast | vincent_hou: the idea being that whatever code needs it can decide if that is an error state or whatever they need to do to recover | 02:47 |
| patrickeast | vincent_hou: yep, they only show up if you are looking at the admin view | 02:48 |
| vincent_hou | Is it possible to be invisible to the admin? | 02:48 |
| patrickeast | vincent_hou: uh, probably, but this was intentional so that the admin can see them | 02:49 |
| vincent_hou | I am doing the migration stuffs, I do not want to show the redundant dest volume during migration. | 02:49 |
| patrickeast | vincent_hou: right, a normal user wont see it, the admin will see one, but you can set its description to something like ‘migration target’ | 02:50 |
| patrickeast | so they know what it is | 02:50 |
| patrickeast | and if something bad happens they can see them and delete them or whatever they need | 02:50 |
| *** mutoulbj has joined #openstack-cinder | 02:50 | |
| patrickeast | they wont need special apis or to look in the db | 02:50 |
| vincent_hou | Hmm. | 02:51 |
| *** kkeuning has quit IRC | 02:52 | |
| vincent_hou | With the current code, the normal cannot see the dest volume during migration, but the admin can. | 02:52 |
| vincent_hou | I am wondering if I need to switch to use the internal tenant. | 02:52 |
| *** mutoulbj has quit IRC | 02:53 | |
| *** mutoulbj has joined #openstack-cinder | 02:53 | |
| patrickeast | vincent_hou: hmm it wouldnt change that very much then, although it does allow you to do stuff with the internal tenants quota if you did want those tracked | 02:53 |
| patrickeast | vincent_hou: not sure if that really is important for migration though | 02:53 |
| vincent_hou | For consistency issue, I guess I should use the internal tenant, though nothing change from the UI perspective. | 02:53 |
| patrickeast | vincent_hou: i suppose it makes it easy to tell which one is the ‘real’ volume if one is owned by the internal tenant though | 02:54 |
| vincent_hou | yes. | 02:55 |
| vincent_hou | After this patch is merged, I am gonna change the migration accordingly. | 02:57 |
| *** kkeuning has joined #openstack-cinder | 02:58 | |
| patrickeast | awesome | 02:58 |
| *** kkeuning has quit IRC | 02:58 | |
| *** kkeuning has joined #openstack-cinder | 02:59 | |
| *** kkeuning has quit IRC | 03:00 | |
| *** kkeuning has joined #openstack-cinder | 03:00 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Add Cinder internal tenant support https://review.openstack.org/199352 | 03:01 |
| openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 03:01 |
| *** kkeuning has quit IRC | 03:01 | |
| *** tobe has quit IRC | 03:01 | |
| *** topshare has quit IRC | 03:02 | |
| *** kkeuning has joined #openstack-cinder | 03:02 | |
| *** tobe has joined #openstack-cinder | 03:03 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Add Cinder internal tenant support https://review.openstack.org/199352 | 03:03 |
| openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 03:03 |
| *** kkeuning has quit IRC | 03:04 | |
| *** kkeuning has joined #openstack-cinder | 03:05 | |
| *** vilobhmm has quit IRC | 03:05 | |
| *** kkeuning has quit IRC | 03:06 | |
| *** vilobhmm has joined #openstack-cinder | 03:06 | |
| *** kkeuning has joined #openstack-cinder | 03:07 | |
| *** et10man has joined #openstack-cinder | 03:07 | |
| *** kkeuning has quit IRC | 03:08 | |
| *** bnemec has quit IRC | 03:08 | |
| *** bnemec has joined #openstack-cinder | 03:10 | |
| *** topshare has joined #openstack-cinder | 03:21 | |
| *** topshare has quit IRC | 03:23 | |
| *** daneyon has quit IRC | 03:25 | |
| *** pck has joined #openstack-cinder | 03:25 | |
| *** pck_ has quit IRC | 03:26 | |
| *** rushil has quit IRC | 03:37 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Remove unused parameter in PureFCDriver _connect https://review.openstack.org/200981 | 03:45 |
| *** xiangfeiz_ has quit IRC | 03:47 | |
| *** dims_ has quit IRC | 03:48 | |
| *** coolsvap|away is now known as coolsvap | 03:50 | |
| *** tobe has quit IRC | 03:55 | |
| *** tobe has joined #openstack-cinder | 03:56 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/188240 | 03:56 |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver https://review.openstack.org/188360 | 03:58 |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add multiple pools support for Huawei driver https://review.openstack.org/188251 | 04:00 |
| *** salv-orlando has joined #openstack-cinder | 04:02 | |
| *** mutoulbj has quit IRC | 04:05 | |
| *** heyun has quit IRC | 04:06 | |
| *** sgotliv has joined #openstack-cinder | 04:06 | |
| *** harlowja_at_home has quit IRC | 04:09 | |
| *** salv-orlando has quit IRC | 04:10 | |
| *** salv-orlando has joined #openstack-cinder | 04:13 | |
| *** salv-orlando has quit IRC | 04:18 | |
| *** topshare has joined #openstack-cinder | 04:23 | |
| *** Apoorva has quit IRC | 04:30 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 04:33 |
| *** leeantho has joined #openstack-cinder | 04:35 | |
| *** bnemec has quit IRC | 04:36 | |
| *** BharatK has joined #openstack-cinder | 04:38 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 04:39 |
| openstackgerrit | Patrick East proposed openstack/cinder: Remove unused parameter in PureFCDriver _connect https://review.openstack.org/200981 | 04:45 |
| *** dims has joined #openstack-cinder | 04:48 | |
| *** haomaiwa_ has quit IRC | 04:54 | |
| *** haomaiwang has joined #openstack-cinder | 04:54 | |
| *** dims has quit IRC | 04:55 | |
| *** pcaruana has quit IRC | 05:08 | |
| *** leeantho has quit IRC | 05:10 | |
| *** salv-orlando has joined #openstack-cinder | 05:16 | |
| *** Longgeek has quit IRC | 05:18 | |
| *** Longgeek has joined #openstack-cinder | 05:20 | |
| *** krtaylor has quit IRC | 05:24 | |
| *** salv-orlando has quit IRC | 05:24 | |
| *** vincent_hou_ has joined #openstack-cinder | 05:24 | |
| *** salv-orlando has joined #openstack-cinder | 05:25 | |
| *** smoriya has quit IRC | 05:25 | |
| *** vincent_hou has quit IRC | 05:27 | |
| *** vincent_hou_ is now known as vincent_hou | 05:27 | |
| *** BharatK has quit IRC | 05:28 | |
| *** salv-orlando has quit IRC | 05:29 | |
| *** Apoorva has joined #openstack-cinder | 05:30 | |
| *** shausy has joined #openstack-cinder | 05:33 | |
| *** krtaylor has joined #openstack-cinder | 05:37 | |
| *** BharatK has joined #openstack-cinder | 05:40 | |
| *** deepakcs has joined #openstack-cinder | 05:41 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 05:45 |
| *** BharatK has quit IRC | 05:48 | |
| *** shyama has joined #openstack-cinder | 05:48 | |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Show backup and volume info in output of backup_restore command https://review.openstack.org/199846 | 05:53 |
| *** BharatK has joined #openstack-cinder | 05:56 | |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Show backup and volume info in output of backup_restore command https://review.openstack.org/199846 | 05:57 |
| *** nkrinner has joined #openstack-cinder | 06:04 | |
| *** agarciam has joined #openstack-cinder | 06:09 | |
| *** rwsu has joined #openstack-cinder | 06:14 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: Storwize_svc_npiv_compatibility_mode default value change https://review.openstack.org/198692 | 06:16 |
| *** rwsu has quit IRC | 06:16 | |
| *** ianbrown has quit IRC | 06:30 | |
| *** Apoorva has quit IRC | 06:34 | |
| *** raunak has joined #openstack-cinder | 06:35 | |
| *** vincent_hou_ has joined #openstack-cinder | 06:39 | |
| *** sgotliv has quit IRC | 06:39 | |
| *** lixiaoy11 has left #openstack-cinder | 06:39 | |
| *** lixiaoy11 has joined #openstack-cinder | 06:40 | |
| *** vincent_hou has quit IRC | 06:41 | |
| *** vincent_hou_ is now known as vincent_hou | 06:41 | |
| *** pcaruana has joined #openstack-cinder | 06:42 | |
| *** salv-orlando has joined #openstack-cinder | 06:43 | |
| *** salv-orlando has quit IRC | 06:46 | |
| *** abehl has joined #openstack-cinder | 06:47 | |
| *** afazekas has quit IRC | 06:50 | |
| *** salv-orlando has joined #openstack-cinder | 06:51 | |
| *** raunak has quit IRC | 06:51 | |
| *** salv-orlando has quit IRC | 06:56 | |
| *** vilobhmm has quit IRC | 06:59 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO support for iscsi discovery auth https://review.openstack.org/189358 | 07:04 |
| *** markus_z has joined #openstack-cinder | 07:09 | |
| *** vincent_hou has quit IRC | 07:16 | |
| *** jwcroppe has joined #openstack-cinder | 07:27 | |
| *** markus_z has quit IRC | 07:30 | |
| *** coolsvap is now known as coolsvap|afk | 07:32 | |
| *** chlong-weekend is now known as chlong-afk | 07:33 | |
| *** ams_ has joined #openstack-cinder | 07:33 | |
| *** markus_z has joined #openstack-cinder | 07:33 | |
| ams_ | hi, i add cinder services to my openstack, when i create a new volume, the status of it stuck in creating mode. what is wrong for it? please help me | 07:35 |
| *** afazekas has joined #openstack-cinder | 07:36 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 07:38 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 07:38 |
| *** ianbrown has joined #openstack-cinder | 07:41 | |
| *** salv-orlando has joined #openstack-cinder | 07:48 | |
| *** coolsvap|afk is now known as coolsvap | 07:56 | |
| openstackgerrit | Kuo-tung Kao proposed openstack/cinder: cinder list fails with 'name' sort key https://review.openstack.org/198614 | 07:56 |
| *** laughterwym_ has joined #openstack-cinder | 07:59 | |
| *** laughterwym__ has joined #openstack-cinder | 08:00 | |
| *** lixiaoy1 has joined #openstack-cinder | 08:01 | |
| *** lixiaoy11 has quit IRC | 08:01 | |
| *** jistr has joined #openstack-cinder | 08:02 | |
| *** laughterwym has quit IRC | 08:02 | |
| *** laughterwym_ has quit IRC | 08:03 | |
| *** sgotliv has joined #openstack-cinder | 08:08 | |
| *** ndipanov has joined #openstack-cinder | 08:10 | |
| *** pschaef has joined #openstack-cinder | 08:13 | |
| *** pschaef has quit IRC | 08:13 | |
| *** mutoulbj has joined #openstack-cinder | 08:18 | |
| *** ianbrown has quit IRC | 08:25 | |
| *** heyun has joined #openstack-cinder | 08:26 | |
| *** xiangfeiz_ has joined #openstack-cinder | 08:36 | |
| *** lpetrut has joined #openstack-cinder | 08:36 | |
| xiangfeiz_ | ams_: have you checked cinder logs? | 08:38 |
| *** wanghao has joined #openstack-cinder | 08:41 | |
| *** jamielennox has quit IRC | 08:46 | |
| *** laughterwym__ has quit IRC | 08:47 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/188240 | 08:53 |
| *** lixiaoy1 has quit IRC | 08:53 | |
| *** lixiaoy1 has joined #openstack-cinder | 08:53 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver https://review.openstack.org/188360 | 08:55 |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add iSCSI multipath support for Huawei driver https://review.openstack.org/188365 | 08:57 |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add multiple pools support for Huawei driver https://review.openstack.org/188251 | 09:00 |
| *** ociuhandu has joined #openstack-cinder | 09:00 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Support SmartX for Huawei volume driver https://review.openstack.org/188732 | 09:04 |
| *** aix has joined #openstack-cinder | 09:08 | |
| *** kbyrne has quit IRC | 09:10 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Show backup and volume info in backup_restore https://review.openstack.org/199876 | 09:12 |
| *** kbyrne has joined #openstack-cinder | 09:15 | |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Show backup and volume info in output of backup_restore command https://review.openstack.org/199846 | 09:17 |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Show backup and volume info in backup_restore https://review.openstack.org/199846 | 09:19 |
| *** e0ne has joined #openstack-cinder | 09:19 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 09:23 |
| openstackgerrit | LisaLi proposed openstack/cinder: Fix restore point if backup base is diff-format in ceph https://review.openstack.org/198993 | 09:25 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 09:25 |
| *** dtynan has left #openstack-cinder | 09:28 | |
| *** davechen has left #openstack-cinder | 09:36 | |
| *** e0ne is now known as e0ne_ | 09:36 | |
| *** lan__ is now known as lan | 09:38 | |
| *** jamielennox has joined #openstack-cinder | 09:46 | |
| openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Adding delete wait loop for CloudByte ISCSI volumes. https://review.openstack.org/201059 | 09:48 |
| *** lixiaoy1 has quit IRC | 09:58 | |
| *** dims has joined #openstack-cinder | 09:58 | |
| *** lpetrut has quit IRC | 09:59 | |
| *** alexpilotti has joined #openstack-cinder | 10:00 | |
| *** e0ne_ is now known as e0ne | 10:05 | |
| *** marzif has joined #openstack-cinder | 10:07 | |
| *** xiangfeiz_ has quit IRC | 10:10 | |
| *** kjelly has quit IRC | 10:11 | |
| *** Lee1092 has joined #openstack-cinder | 10:13 | |
| *** heyun has quit IRC | 10:17 | |
| *** IanGovett has joined #openstack-cinder | 10:20 | |
| *** laughterwym has joined #openstack-cinder | 10:21 | |
| *** coolsvap is now known as coolsvap|away | 10:25 | |
| *** coolsvap|away is now known as coolsvap | 10:28 | |
| *** ronis has joined #openstack-cinder | 10:32 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add entry create and cast tasks to manage workflow https://review.openstack.org/139071 | 10:42 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix lvm manage existing volume https://review.openstack.org/156939 | 10:42 |
| *** coolsvap is now known as coolsvap|away | 10:43 | |
| *** e0ne is now known as e0ne_ | 10:43 | |
| *** e0ne_ is now known as e0ne | 10:43 | |
| *** aix has quit IRC | 10:44 | |
| *** wanghao has quit IRC | 10:47 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: WIP:Implement cinder API WSGI application to run under Apache/Nginx https://review.openstack.org/200551 | 10:50 |
| *** rushiagr_away is now known as rushiagr | 10:52 | |
| *** cdelatte has joined #openstack-cinder | 10:56 | |
| *** e0ne is now known as e0ne_ | 11:01 | |
| openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Adding delete wait loop for CloudByte ISCSI volumes. https://review.openstack.org/201059 | 11:03 |
| *** belmoreira has joined #openstack-cinder | 11:06 | |
| openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Adding delete wait loop for CloudByte ISCSI volumes. https://review.openstack.org/201059 | 11:08 |
| *** aix has joined #openstack-cinder | 11:11 | |
| *** e0ne_ is now known as e0ne | 11:14 | |
| *** sgotliv has quit IRC | 11:28 | |
| *** ankit has joined #openstack-cinder | 11:28 | |
| *** jaypipes has joined #openstack-cinder | 11:29 | |
| *** tobe has quit IRC | 11:36 | |
| *** rushiagr is now known as rushiagr_away | 11:41 | |
| *** coolsvap|away is now known as coolsvap | 11:44 | |
| *** Tross1 has quit IRC | 11:58 | |
| *** sgotliv has joined #openstack-cinder | 12:05 | |
| *** topshare has quit IRC | 12:06 | |
| *** yuriy_n17 has joined #openstack-cinder | 12:06 | |
| *** sgotliv has quit IRC | 12:07 | |
| *** sgotliv has joined #openstack-cinder | 12:07 | |
| *** timcl has joined #openstack-cinder | 12:07 | |
| *** lpetrut has joined #openstack-cinder | 12:12 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO support for iscsi discovery auth https://review.openstack.org/189358 | 12:12 |
| *** thangp has joined #openstack-cinder | 12:14 | |
| *** akerr has joined #openstack-cinder | 12:16 | |
| openstackgerrit | Duncan Thomas proposed openstack/cinder: Add support for backups to xiv_ds8k driver https://review.openstack.org/118298 | 12:19 |
| *** rushiagr_away is now known as rushiagr | 12:25 | |
| *** ganso_ has joined #openstack-cinder | 12:25 | |
| *** kjnelson has joined #openstack-cinder | 12:27 | |
| *** bswartz has quit IRC | 12:27 | |
| *** rushiagr is now known as rushiagr_away | 12:29 | |
| *** topshare has joined #openstack-cinder | 12:31 | |
| *** nikeshm has joined #openstack-cinder | 12:31 | |
| *** e0ne is now known as e0ne_ | 12:33 | |
| *** edmondsw has joined #openstack-cinder | 12:36 | |
| *** ams_ has quit IRC | 12:38 | |
| *** lpetrut has quit IRC | 12:39 | |
| *** mtecer has joined #openstack-cinder | 12:39 | |
| *** e0ne_ is now known as e0ne | 12:46 | |
| *** sgotliv has quit IRC | 12:47 | |
| *** deepakcs has quit IRC | 12:48 | |
| *** mtecer has quit IRC | 12:52 | |
| *** sgotliv has joined #openstack-cinder | 12:53 | |
| *** dims has quit IRC | 12:55 | |
| *** lpetrut has joined #openstack-cinder | 12:57 | |
| *** Guest33195 has joined #openstack-cinder | 12:57 | |
| *** Guest33195 is now known as dims_ | 12:58 | |
| *** shyama is now known as shyama_ | 13:00 | |
| *** julim has joined #openstack-cinder | 13:01 | |
| *** bswartz has joined #openstack-cinder | 13:02 | |
| *** julim_ has joined #openstack-cinder | 13:04 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Add more Rally scenarios to run https://review.openstack.org/194180 | 13:05 |
| *** julim has quit IRC | 13:07 | |
| *** dustins has joined #openstack-cinder | 13:11 | |
| *** ohno13 has joined #openstack-cinder | 13:16 | |
| *** lpetrut has quit IRC | 13:18 | |
| *** belmoreira has quit IRC | 13:20 | |
| *** primechuck has joined #openstack-cinder | 13:22 | |
| *** crose has joined #openstack-cinder | 13:24 | |
| *** edmondsw has quit IRC | 13:25 | |
| *** sgotliv has quit IRC | 13:28 | |
| *** edmondsw has joined #openstack-cinder | 13:32 | |
| *** ericksonsantos has joined #openstack-cinder | 13:33 | |
| *** mutoulbj has quit IRC | 13:35 | |
| *** breitz has joined #openstack-cinder | 13:35 | |
| *** mutoulbj has joined #openstack-cinder | 13:36 | |
| *** Yogi1 has joined #openstack-cinder | 13:37 | |
| *** jungleboyj has quit IRC | 13:39 | |
| *** lpetrut has joined #openstack-cinder | 13:41 | |
| *** eharney has joined #openstack-cinder | 13:42 | |
| *** annegentle has joined #openstack-cinder | 13:43 | |
| *** rushil has joined #openstack-cinder | 13:43 | |
| *** mriedem has joined #openstack-cinder | 13:44 | |
| *** shausy has quit IRC | 13:46 | |
| *** mdenny has joined #openstack-cinder | 13:46 | |
| *** Tross has joined #openstack-cinder | 13:46 | |
| *** rushil has quit IRC | 13:48 | |
| *** jdandrea has joined #openstack-cinder | 13:50 | |
| openstackgerrit | Duncan Thomas proposed openstack/python-cinderclient: Fix bypass_url option, required by session object https://review.openstack.org/145493 | 13:51 |
| *** nikeshm has quit IRC | 13:55 | |
| *** superdan is now known as dansmith | 13:55 | |
| *** shakamunyi has joined #openstack-cinder | 13:59 | |
| *** barra204 has joined #openstack-cinder | 14:00 | |
| jgriffith | eharney: Hey, if you have a minute; I posted a couple comments in https://review.openstack.org/#/c/104653/ | 14:02 |
| jgriffith | eharney: might not be valid, just thoughts/ideas, wanted to see what you thought | 14:03 |
| eharney | jgriffith: i'll take a look (probably in a little bit) | 14:03 |
| *** mtanino has joined #openstack-cinder | 14:03 | |
| *** annegentle has quit IRC | 14:05 | |
| *** xyang has joined #openstack-cinder | 14:05 | |
| DuncanT | jgriffith: I'm fine with adding 'linear', but changing the meaning of 'default' is slightly problematic for upgrading existing systems, unless we keep Eric's 'are there already LVs' check / dance in | 14:16 |
| *** lpabon has joined #openstack-cinder | 14:16 | |
| *** xyang has quit IRC | 14:17 | |
| jgriffith | Yeah, so by no means would I suggest removing his check | 14:17 |
| jgriffith | So upgrades *shouldn't* be problematic at all. What I was asking about was leaving pretty much exactly what Eric coded up, but flipping around/minimizing the config options | 14:19 |
| jgriffith | so rather than "default, thin, auto" where default-->linear; you have "default, linear" where default --> thin | 14:20 |
| jgriffith | and if the node can't do thin default does the right thing and reverts to linear | 14:20 |
| jgriffith | and we NEVER convert anything that already exists | 14:20 |
| *** xyang has joined #openstack-cinder | 14:21 | |
| eharney | the general idea was to switch to "auto" as the default at some point in the future | 14:21 |
| eharney | if we add another option i would really not want to call it "linear", but "thick" instead | 14:21 |
| jgriffith | eharney: I gathered that, but just figured why introduce a new option temporarily rather than just holding off on that piece | 14:21 |
| eharney | hmm, good point | 14:21 |
| eharney | given that thin is widely supported now we probably could just change the meaning of "default" to do what "auto" does here? | 14:22 |
| jgriffith | eharney: yeah, that's fine. Also I'm not saying we should necessarily introduce anything new until we're ready for a new default | 14:22 |
| jgriffith | eharney: yes, exactly! | 14:22 |
| eharney | i was just aiming for something more gradual i guess, but no strong reason for that | 14:22 |
| yuriy_n17 | eharney: Hi! Please review https://review.openstack.org/#/c/174823/ | 14:23 |
| jgriffith | eharney: Yeah, my first reaction was to agree with that (gradual intro), but then I started thinking about it and didn't know if it was more confusing than helpful | 14:23 |
| eharney | jgriffith: i think just adding "thick" and changing "default" to this new behavior sounds good | 14:23 |
| *** jungleboyj has joined #openstack-cinder | 14:23 | |
| jgriffith | eharney: so I'd like to see the gate job running for a week, then just rip off the bandaid and make default do exactly what you propose in auto | 14:23 |
| eharney | jgriffith: yeah the experimental job merged over the weekend | 14:24 |
| eharney | i haven't gone and found the results yet | 14:24 |
| jgriffith | yeah, I've been patienty watching :) | 14:24 |
| jgriffith | or... impatiently more like :) | 14:24 |
| *** barra204 has quit IRC | 14:25 | |
| *** shakamunyi has quit IRC | 14:25 | |
| jgriffith | eharney: it still needs to propogate *somewhere*, things have changed so much that I'm not necessarily crystal clear on the flow anymore :( | 14:25 |
| *** whydidyoustealmy has joined #openstack-cinder | 14:25 | |
| eharney | jgriffith: what needs to? | 14:26 |
| *** shakamunyi has joined #openstack-cinder | 14:26 | |
| jgriffith | so the job merged but I don't see it "running" against anything yet | 14:26 |
| eharney | yeah, i was just trying to sort that out too... maybe it's awaiting a jenkins restart or something | 14:26 |
| jgriffith | yeah, or node-pool? | 14:27 |
| eharney | the jobs exist in jenkins but all have no runs | 14:27 |
| jgriffith | I dunno, that whole thing has become rather advanced now | 14:27 |
| *** ankit has quit IRC | 14:28 | |
| jgriffith | anyway... was curious what you thought of that. I'm fine if you want to just proceed as is, just seemed like an unnecessary intermediate step to add another config option | 14:28 |
| eharney | jgriffith: i'm up for just changing default, sounds good to me | 14:29 |
| eharney | jgriffith: err, wait | 14:29 |
| eharney | now i remember why i didn't do that | 14:29 |
| eharney | :) | 14:29 |
| eharney | if we go to thin by default we also need to default the max_overprovision_ratio to 1.0 instead of 2.0 | 14:30 |
| jgriffith | eharney: hurumph | 14:30 |
| eharney | because currently opting into thin also = opting into people filling up your VG and running out of space | 14:30 |
| jgriffith | eharney: although we could code that up to be dynamic as well no? | 14:31 |
| jgriffith | eharney: I mean, that might be a bad idea, but possible | 14:31 |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO Volume driver Consistency Groups support https://review.openstack.org/189135 | 14:31 |
| jgriffith | eharney: I just don't want to go and change it again in a month :) | 14:31 |
| *** markvoelker has joined #openstack-cinder | 14:33 | |
| jgriffith | eharney: but maybe I'm being too picky | 14:33 |
| *** julim_ has quit IRC | 14:33 | |
| eharney | jgriffith: well, i'm trying to figure out the exact path here now | 14:33 |
| *** whydidyoustealmy has quit IRC | 14:34 | |
| *** julim has joined #openstack-cinder | 14:34 | |
| *** shakamunyi has quit IRC | 14:34 | |
| * eharney is confused trying to find the thin prov options in his code base again | 14:35 | |
| *** annegentle has joined #openstack-cinder | 14:35 | |
| eharney | jgriffith: so... the default over subscription ratio is actually 20 | 14:36 |
| eharney | which is way scarier than 2, which i thought it was | 14:36 |
| jgriffith | eharney: :) | 14:36 |
| eharney | that just sounds like asking for trouble :\ | 14:37 |
| eharney | to default to that for LVM anyway | 14:37 |
| jgriffith | eharney: we can certainly think about it and change it | 14:37 |
| jgriffith | eharney: but it's awesome for dev/test :) | 14:38 |
| eharney | surely :) | 14:38 |
| *** abehl has quit IRC | 14:39 | |
| *** rhagarty_ has quit IRC | 14:39 | |
| *** rhagarty has quit IRC | 14:39 | |
| *** markvoelker_ has joined #openstack-cinder | 14:42 | |
| *** markvoelker has quit IRC | 14:44 | |
| *** tsekiyama has joined #openstack-cinder | 14:44 | |
| *** sgotliv has joined #openstack-cinder | 14:47 | |
| *** IanGovett has quit IRC | 14:48 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Non-disruptive backup https://review.openstack.org/193937 | 14:48 |
| *** nkrinner is now known as nkrinner_afk | 14:48 | |
| *** alexpilotti has quit IRC | 14:54 | |
| *** alexpilotti has joined #openstack-cinder | 14:55 | |
| *** alexpilotti has quit IRC | 14:56 | |
| *** alexpilotti has joined #openstack-cinder | 14:57 | |
| *** rhagarty has joined #openstack-cinder | 14:58 | |
| *** rhagarty_ has joined #openstack-cinder | 14:58 | |
| *** MikePar has joined #openstack-cinder | 14:59 | |
| *** cbader has joined #openstack-cinder | 14:59 | |
| *** alexpilotti has quit IRC | 14:59 | |
| *** diemt has quit IRC | 15:00 | |
| *** diemt has joined #openstack-cinder | 15:01 | |
| *** alexpilotti has joined #openstack-cinder | 15:01 | |
| MikePar | I'm looking for some guidance on getting Ceph going with OS and Cinder. I've followed through the Ceph docs on getting going with OS, and I have Glance working, but The 'volumes' section of my UI still says it can't find any volumes. | 15:02 |
| *** sgotliv has quit IRC | 15:02 | |
| *** alexpilotti has quit IRC | 15:03 | |
| *** alexpilotti has joined #openstack-cinder | 15:05 | |
| *** daneyon has joined #openstack-cinder | 15:05 | |
| *** rushil has joined #openstack-cinder | 15:06 | |
| *** sgotliv has joined #openstack-cinder | 15:09 | |
| *** cbader has quit IRC | 15:10 | |
| *** topshare has quit IRC | 15:10 | |
| *** alexpilotti has quit IRC | 15:11 | |
| yuriy_n17 | eharney: Hi! Please review https://review.openstack.org/#/c/174823/ when you have time. | 15:13 |
| *** afazekas has quit IRC | 15:15 | |
| *** topshare has joined #openstack-cinder | 15:18 | |
| *** rwsu has joined #openstack-cinder | 15:19 | |
| *** harlowja_at_home has joined #openstack-cinder | 15:19 | |
| *** markvoelker has joined #openstack-cinder | 15:20 | |
| *** markvoelker_ has quit IRC | 15:22 | |
| *** topshare has quit IRC | 15:23 | |
| *** markvoelker has quit IRC | 15:23 | |
| *** lpetrut has quit IRC | 15:24 | |
| *** BharatK has quit IRC | 15:25 | |
| openstackgerrit | Peter Penchev proposed openstack/cinder: StorPool: clean up the last uses of str.format() https://review.openstack.org/190982 | 15:28 |
| *** sgotliv has quit IRC | 15:30 | |
| *** jdurgin has joined #openstack-cinder | 15:30 | |
| *** garthb has joined #openstack-cinder | 15:31 | |
| *** garthb_ has joined #openstack-cinder | 15:31 | |
| *** laughterwym has quit IRC | 15:33 | |
| *** leeantho has joined #openstack-cinder | 15:33 | |
| *** lpetrut has joined #openstack-cinder | 15:33 | |
| *** yuriy_n17 has quit IRC | 15:34 | |
| *** akshai has joined #openstack-cinder | 15:35 | |
| *** akshai has quit IRC | 15:35 | |
| *** akshai has joined #openstack-cinder | 15:36 | |
| *** alonmarx has quit IRC | 15:36 | |
| *** mordred has quit IRC | 15:36 | |
| *** mitz_ has quit IRC | 15:36 | |
| *** mkoderer has quit IRC | 15:36 | |
| *** kambiz has quit IRC | 15:36 | |
| *** dulek has quit IRC | 15:36 | |
| *** kragniz has quit IRC | 15:36 | |
| *** Zanatoz has quit IRC | 15:36 | |
| *** cburgess has quit IRC | 15:36 | |
| *** leseb_ has quit IRC | 15:36 | |
| *** xek has quit IRC | 15:36 | |
| *** tristanC has quit IRC | 15:36 | |
| *** rmk has quit IRC | 15:36 | |
| *** DuncanT has quit IRC | 15:36 | |
| *** kragniz has joined #openstack-cinder | 15:37 | |
| *** Zanatoz has joined #openstack-cinder | 15:37 | |
| *** tristanC has joined #openstack-cinder | 15:37 | |
| *** kambiz has joined #openstack-cinder | 15:37 | |
| *** mkoderer has joined #openstack-cinder | 15:37 | |
| *** leseb has joined #openstack-cinder | 15:37 | |
| *** dulek has joined #openstack-cinder | 15:37 | |
| *** mitz has joined #openstack-cinder | 15:37 | |
| *** cburgess has joined #openstack-cinder | 15:37 | |
| *** rmk has joined #openstack-cinder | 15:37 | |
| *** xek has joined #openstack-cinder | 15:37 | |
| *** alonmarx has joined #openstack-cinder | 15:37 | |
| *** mordred has joined #openstack-cinder | 15:37 | |
| *** pcaruana has quit IRC | 15:38 | |
| openstackgerrit | Peter Penchev proposed openstack/cinder: StorPool: clean up the last uses of str.format() https://review.openstack.org/190982 | 15:39 |
| *** daneyon_ has joined #openstack-cinder | 15:41 | |
| *** ronis has quit IRC | 15:42 | |
| *** daneyon has quit IRC | 15:44 | |
| *** jwcroppe has quit IRC | 15:48 | |
| *** jwcroppe has joined #openstack-cinder | 15:48 | |
| *** morganfainberg has quit IRC | 15:51 | |
| *** morganfainberg has joined #openstack-cinder | 15:52 | |
| *** rushil has quit IRC | 15:52 | |
| *** jwcroppe has quit IRC | 15:53 | |
| *** jdurgin has quit IRC | 15:54 | |
| *** garthb_ has quit IRC | 15:55 | |
| *** garthb has quit IRC | 15:55 | |
| *** DuncanT has joined #openstack-cinder | 15:57 | |
| *** garthb has joined #openstack-cinder | 16:00 | |
| *** thangp has quit IRC | 16:01 | |
| *** annasort has joined #openstack-cinder | 16:06 | |
| *** dalgaaf has quit IRC | 16:08 | |
| *** Tross has quit IRC | 16:13 | |
| *** zhenguo has quit IRC | 16:13 | |
| *** jwcroppe has joined #openstack-cinder | 16:13 | |
| Swanson | thingee: Does something like this https://review.openstack.org/#/c/199631/ require a blue print? Some question in the review. | 16:15 |
| *** Apoorva_ has joined #openstack-cinder | 16:16 | |
| *** Yogi1 has quit IRC | 16:18 | |
| *** Guest55 has joined #openstack-cinder | 16:18 | |
| *** pv_ has joined #openstack-cinder | 16:19 | |
| *** IanGovett has joined #openstack-cinder | 16:20 | |
| pv_ | quick question, how does the format for a fibre channel connector look? or does it not matter? i see on http://docs.openstack.org/developer/cinder/devref/volume.html that the connector dictionary format is specified for an iscsi interface | 16:20 |
| *** diogogmt has joined #openstack-cinder | 16:20 | |
| *** hodos has joined #openstack-cinder | 16:21 | |
| *** e0ne has quit IRC | 16:21 | |
| thingee | Swanson: nah I'm fine without it | 16:22 |
| thingee | Swanson: I really should document what I find acceptable and present it to the meeting to see if others agree | 16:22 |
| *** mgarza_ has joined #openstack-cinder | 16:22 | |
| thingee | Believe Nova has done this already | 16:22 |
| *** shakamunyi has joined #openstack-cinder | 16:25 | |
| *** whydidyoustealmy has joined #openstack-cinder | 16:25 | |
| *** Yogi1 has joined #openstack-cinder | 16:27 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tests: test_volume mock conversion https://review.openstack.org/201241 | 16:28 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Split VolumeTestCase into separate classes https://review.openstack.org/201242 | 16:28 |
| Swanson | thingee: Thanks for the clarification! | 16:28 |
| *** edmondsw has quit IRC | 16:29 | |
| *** IanGovett has quit IRC | 16:30 | |
| *** _cjones_ has joined #openstack-cinder | 16:31 | |
| *** lcurtis has joined #openstack-cinder | 16:35 | |
| *** hemnafk is now known as hemna | 16:35 | |
| *** jwcroppe has quit IRC | 16:35 | |
| *** harlowja_at_home has quit IRC | 16:36 | |
| *** jwcroppe has joined #openstack-cinder | 16:36 | |
| *** abehl has joined #openstack-cinder | 16:37 | |
| *** whydidyoustealmy has quit IRC | 16:37 | |
| *** shakamunyi has quit IRC | 16:37 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: POC: Adding get_capabilities() feature https://review.openstack.org/201243 | 16:37 |
| *** vokt has joined #openstack-cinder | 16:39 | |
| *** jwcroppe has quit IRC | 16:40 | |
| *** Guest55 has quit IRC | 16:43 | |
| *** Guest55 has joined #openstack-cinder | 16:43 | |
| openstackgerrit | xing-yang proposed openstack/cinder: WIP: Attach snapshot - driver only https://review.openstack.org/201249 | 16:46 |
| *** patrickeast has quit IRC | 16:49 | |
| *** Tross has joined #openstack-cinder | 16:49 | |
| *** alexpilotti has joined #openstack-cinder | 16:51 | |
| *** shakamunyi has joined #openstack-cinder | 16:52 | |
| *** whydidyoustealmy has joined #openstack-cinder | 16:52 | |
| *** dannywilson has joined #openstack-cinder | 16:57 | |
| *** annashen has joined #openstack-cinder | 17:01 | |
| *** alexpilotti has quit IRC | 17:02 | |
| *** ronis has joined #openstack-cinder | 17:02 | |
| *** dannywilson has quit IRC | 17:02 | |
| *** dannywilson has joined #openstack-cinder | 17:03 | |
| *** lpetrut has quit IRC | 17:03 | |
| *** patrickeast has joined #openstack-cinder | 17:05 | |
| *** shyama_ is now known as shyama | 17:05 | |
| *** annegentle has quit IRC | 17:05 | |
| *** vilobhmm has joined #openstack-cinder | 17:05 | |
| *** annegentle has joined #openstack-cinder | 17:06 | |
| *** pradipta has joined #openstack-cinder | 17:07 | |
| *** alexpilotti has joined #openstack-cinder | 17:08 | |
| *** shakamunyi has quit IRC | 17:09 | |
| *** whydidyoustealmy has quit IRC | 17:09 | |
| *** sgotliv has joined #openstack-cinder | 17:09 | |
| *** shakamunyi has joined #openstack-cinder | 17:09 | |
| *** lpabon has quit IRC | 17:12 | |
| *** annashen has quit IRC | 17:12 | |
| openstackgerrit | xing-yang proposed openstack/cinder: WIP: Attach snapshot - driver only https://review.openstack.org/201249 | 17:16 |
| *** pradipta has quit IRC | 17:19 | |
| *** rushil has joined #openstack-cinder | 17:19 | |
| openstackgerrit | Sean Chen proposed openstack/cinder: Tintri snapshot id https://review.openstack.org/198829 | 17:19 |
| *** annashen has joined #openstack-cinder | 17:20 | |
| *** erlon has joined #openstack-cinder | 17:23 | |
| xyang | jgriffith: ping | 17:23 |
| jgriffith | xyang: hey | 17:23 |
| xyang | jgriffith: can you take a look of this? https://review.openstack.org/#/c/201249/ | 17:23 |
| *** delattec has joined #openstack-cinder | 17:23 | |
| xyang | jgriffith: I made some changes to the attach snapshot patch and limit it to driver only | 17:24 |
| Guest55 | xyang sure | 17:24 |
| Guest55 | bahhhahahahahahaha | 17:24 |
| jgriffith | xyang: sure | 17:24 |
| xyang | jgriffith: I thought you are disappearing again:) | 17:25 |
| * jgriffith loves screen sessions | 17:25 | |
| jgriffith | No disappearing this time.... just a left over screen session from when things were wonky | 17:25 |
| xyang | jgriffith: ok:) | 17:25 |
| *** IanGovett has joined #openstack-cinder | 17:26 | |
| jgriffith | xyang: anyway... yes, I'll have a look here after lunch | 17:26 |
| *** cdelatte has quit IRC | 17:27 | |
| xyang | jgriffith: sure, thanks! | 17:27 |
| *** Longgeek has quit IRC | 17:29 | |
| *** lpetrut has joined #openstack-cinder | 17:31 | |
| *** jistr has quit IRC | 17:31 | |
| hemna | smcginnis, I got a failing running tox -epy34 on this guy https://review.openstack.org/#/c/197723 | 17:33 |
| hemna | smcginnis, I'm running the full suite of tox to see if that 'fixes' it. | 17:33 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add multiple pools support to VMAX driver https://review.openstack.org/189990 | 17:34 |
| *** lpetrut has quit IRC | 17:35 | |
| *** abehl has quit IRC | 17:35 | |
| *** jgriffith is now known as lurker | 17:35 | |
| *** Lee1092 has quit IRC | 17:35 | |
| *** Guest55 is now known as jgriffith | 17:35 | |
| hemna | smcginnis, ok so the full suite has 4 errors here locally. | 17:36 |
| *** jdurgin1 has quit IRC | 17:37 | |
| *** jdurgin has joined #openstack-cinder | 17:37 | |
| *** alexpilotti has quit IRC | 17:37 | |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 17:37 |
| *** chlong-afk has quit IRC | 17:38 | |
| hemna | smcginnis, http://paste.openstack.org/show/371879/ | 17:39 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Remove useless logging from unit tests https://review.openstack.org/197723 | 17:39 |
| *** markus_z has quit IRC | 17:44 | |
| *** alexpilotti has joined #openstack-cinder | 17:44 | |
| patrickeast | hemna: oh hey, fwiw i saw that error on the pure drivers and wiping out my venv and letting it build again fixed it | 17:45 |
| hemna | patrickeast, I'll try wiping out my .tox and see if that fixes it. | 17:45 |
| smcginnis | hemna: Hmm, that's new. | 17:46 |
| *** ociuhandu has quit IRC | 17:46 | |
| hemna | rerunning | 17:46 |
| hemna | trying tox -epy27 first | 17:46 |
| smcginnis | patrickeast: I've seen this on another patch. Any idea why the pure test is failing? http://paste.openstack.org/show/371879/ | 17:46 |
| smcginnis | These seem to be since the upgrade of mock. | 17:46 |
| patrickeast | its something to do with the new mock | 17:46 |
| patrickeast | yea | 17:46 |
| patrickeast | there was a patch to fix it, which landed, but i think there is still something kind of unhappy going on with it | 17:47 |
| patrickeast | maybe just an older version of some lib in the venv breaking it though, its fine on gate | 17:47 |
| smcginnis | Well, hopefully some rechecks help. | 17:48 |
| *** jwcroppe has joined #openstack-cinder | 17:51 | |
| hemna | patrickeast, smcginnis ok same failures with a fresh .tox | 17:52 |
| patrickeast | hmm | 17:52 |
| * patrickeast is running locally | 17:52 | |
| *** edmondsw has joined #openstack-cinder | 17:53 | |
| smcginnis | I've seen the other ones before too. I'm going to try running against master and see if it's even working there now. | 17:53 |
| hemna | smcginnis, so I made sure my master was up to date, then git review -d 197723, rm -rf .tox, tox -epy27 | 17:54 |
| hemna | so, dunno | 17:54 |
| *** alexpilotti has quit IRC | 17:54 | |
| smcginnis | hemna: I mean not even run against my patch. Just run against master now that mock is updated and see if it's broken there now. | 17:55 |
| hemna | oh ok, I can try that as well | 17:55 |
| smcginnis | Mine's running now. Should know soon. | 17:55 |
| *** MikePar has left #openstack-cinder | 17:57 | |
| *** Tross1 has joined #openstack-cinder | 17:57 | |
| *** Tross has quit IRC | 17:58 | |
| hemna | tox -epy34 passed | 17:58 |
| hemna | against master | 17:58 |
| smcginnis | Yeah, mine too. | 17:58 |
| smcginnis | Rebase merge issue? | 17:58 |
| hemna | trying the patch again | 17:58 |
| smcginnis | I'll check after I get back from this next meeting. Thanks for pointing it out. | 17:58 |
| hemna | smcginnis, ok so tox -epy34 fails against the patch locally. passes against master. | 18:02 |
| *** Yogi1 has quit IRC | 18:04 | |
| *** rushil has quit IRC | 18:04 | |
| *** rushil has joined #openstack-cinder | 18:05 | |
| *** garthb has quit IRC | 18:05 | |
| *** primechuck has quit IRC | 18:06 | |
| *** rushil has quit IRC | 18:08 | |
| *** primechuck has joined #openstack-cinder | 18:08 | |
| *** akshai has quit IRC | 18:09 | |
| openstackgerrit | David Medberry proposed openstack/cinder: Removing archaic references https://review.openstack.org/193791 | 18:09 |
| *** akshai has joined #openstack-cinder | 18:09 | |
| *** aix has quit IRC | 18:11 | |
| *** shyama is now known as shyama_ | 18:15 | |
| *** shyama_ is now known as shyama | 18:15 | |
| *** shyama is now known as shyama_ | 18:15 | |
| *** shyama_ is now known as shyama | 18:16 | |
| *** shyama is now known as shyama_ | 18:16 | |
| *** shyama_ is now known as shyama | 18:22 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix update_migrate_volume() exception handling https://review.openstack.org/198731 | 18:25 |
| Swanson | Does specifying the consistency group id on volume create actually add the volume to the consistency group with that id? | 18:27 |
| patrickeast | Swanson: yea | 18:28 |
| *** rwsu has quit IRC | 18:29 | |
| *** Yogi1 has joined #openstack-cinder | 18:29 | |
| Swanson | patrickeast: Lies. Am I supposed to be doing that on volume create or should my update_cg_volumes be getting called? | 18:29 |
| patrickeast | Swanson: volume create, so like for my driver we do https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/pure.py#L119 | 18:31 |
| patrickeast | although now that we have update as an api available maybe we should change the code to use that instead… | 18:31 |
| patrickeast | xyang: ^ | 18:32 |
| patrickeast | xyang: that is how its supposed to work, right? | 18:32 |
| xyang | patrickeast ? | 18:32 |
| patrickeast | xyang: it should be handled in create_volume and add the new one to the cg | 18:32 |
| xyang | patrickeast: yes | 18:32 |
| Swanson | patrickeast: All of the create volumes, I presume get that. | 18:33 |
| Swanson | Surprised update isn't called. | 18:33 |
| *** delattec has quit IRC | 18:33 | |
| xyang | Swanson: what is not called? | 18:33 |
| patrickeast | update is relatively new, the initial cg stuff didn’t have it | 18:34 |
| xyang | Swanson: the common framework will only change db entry | 18:34 |
| Swanson | update_cg_volumes | 18:34 |
| *** delattec has joined #openstack-cinder | 18:34 | |
| xyang | Swanson: you driver should add that volume to the group on your array | 18:34 |
| Swanson | xyang: Yep. Adding it to my patch now. | 18:34 |
| Swanson | patrickeast: xyang Thanks for the clarification! | 18:35 |
| xyang | Swanson: Did I miss that, I thought you did | 18:35 |
| patrickeast | haha yea i remember looking at the review and didn’t notice that either | 18:35 |
| xyang | Swanson: wait a second, are you asking create volume or update CG? | 18:35 |
| Swanson | xyang: on create volume | 18:36 |
| *** shakamunyi has quit IRC | 18:36 | |
| xyang | Swanson: when you create volume, you should add volume to your CG in the driver code as well | 18:36 |
| xyang | Swanson: db entry will be taken care by the common code | 18:37 |
| Swanson | xyang: Adding that now. | 18:37 |
| xyang | Swanson: update CG is different because volume is already created | 18:37 |
| xyang | Swanson: for update CG you need to add volume to the CG, but you shouldn't be in create volume | 18:38 |
| Swanson | xyang: I had assumed that it would be called after create volume. Of course if I had tested that I'd have known it wasn't. | 18:38 |
| hemna | xyang, smcginnis jungleboyj patrickeast https://review.openstack.org/#/c/195350/ | 18:46 |
| hemna | I believe that patch helps fix nova live migration problems for FC volumes. | 18:47 |
| smcginnis | hemna: Nice. Got a tab open for it. | 18:47 |
| hemna | since nova seems to toss data that exists in the BDM and refetches the connection_info from cinder, multiple times. | 18:47 |
| *** shakamunyi has joined #openstack-cinder | 18:47 | |
| smcginnis | Lovely | 18:47 |
| hemna | yup | 18:47 |
| xyang | hema: great! | 18:48 |
| hemna | my next task is to document which Cinder drivers aren't careful in their initialize_connection calls. | 18:48 |
| hemna | xyang, you might have noticed a comment in the XtremIO review related to this. | 18:48 |
| xyang | hemna: ya, I saw that, thanks | 18:49 |
| smcginnis | hemna: FWIW, I pulled down and rebased and that logging patch passes locally. We'll see what Jenkins says once that finally goes through again. | 18:51 |
| hemna | smcginnis, ok cool. | 18:52 |
| *** mutoulbj has quit IRC | 18:53 | |
| *** ekarlso has quit IRC | 18:56 | |
| *** ekarlso has joined #openstack-cinder | 18:58 | |
| *** xyang has quit IRC | 19:00 | |
| *** e0ne has joined #openstack-cinder | 19:02 | |
| *** julim has quit IRC | 19:03 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: [WIP] Switch to oslo-config-generator https://review.openstack.org/200567 | 19:04 |
| *** nikeshm has joined #openstack-cinder | 19:06 | |
| *** julim has joined #openstack-cinder | 19:06 | |
| *** bkopilov has quit IRC | 19:07 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: [WIP] Switch to oslo-config-generator https://review.openstack.org/200567 | 19:07 |
| *** e0ne is now known as e0ne_ | 19:08 | |
| *** dannywilson has quit IRC | 19:08 | |
| *** bkopilov has joined #openstack-cinder | 19:11 | |
| *** shyama has quit IRC | 19:12 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: [WIP] Switch to oslo-config-generator https://review.openstack.org/200567 | 19:12 |
| mtreinish | thingee: you know those cinderclient functional tests you're discussing on the ML were always voting until I ripped them out of tempest | 19:15 |
| mtreinish | and pretty much every other project that did that migration just made them voting from the start | 19:15 |
| mtreinish | I'm not sure why cinderclient got stuck in experimental | 19:16 |
| thingee | mtreinish: I do now :) | 19:16 |
| thingee | mtreinish: well call me a fool and reply to the ML :) | 19:16 |
| mtreinish | hehe | 19:16 |
| patrickeast | hemna: smcginnis: i think i might see what causes that error in the pure unit tests, the helper method is overwriting the mocks side_effect here https://github.com/openstack/cinder/blob/master/cinder/tests/unit/test_pure.py#L212 and one of the mocks (the one that returns the host object thing that its puking on) needs to have its side effect preserved | 19:17 |
| patrickeast | i’m testing out a fix locally | 19:17 |
| patrickeast | but am still confused why its only sometimes happening | 19:18 |
| hemna | patrickeast, sweet | 19:18 |
| hemna | patrickeast, it's 100% here | 19:18 |
| patrickeast | hemna: haha i can’t even repro it now, at least not with run_tests.sh | 19:19 |
| hemna | tox! | 19:19 |
| hemna | :) | 19:19 |
| patrickeast | hemna: here try this out on your system http://paste.openstack.org/show/372075/ | 19:19 |
| patrickeast | very small change | 19:19 |
| * patrickeast is trying tox | 19:20 | |
| *** julim has quit IRC | 19:20 | |
| *** annashen has quit IRC | 19:20 | |
| *** annashen has joined #openstack-cinder | 19:21 | |
| patrickeast | huh… does tox not work on osx anymore? http://paste.openstack.org/show/372078/ | 19:21 |
| smcginnis | patrickeast: Cool. Could be concurrency. If you run it --concurrency=1 it might happen in the right order, but if you run --concurrency=7 like jenkins it could hit a window. | 19:21 |
| *** e0ne_ is now known as e0ne | 19:22 | |
| patrickeast | smcginnis: ahh good point | 19:22 |
| *** angela-s has joined #openstack-cinder | 19:22 | |
| eharney | patrickeast: that failure related to python versions in the reqs was a bug in one of the tools | 19:22 |
| *** julim has joined #openstack-cinder | 19:23 | |
| *** annashen has quit IRC | 19:23 | |
| *** annashen has joined #openstack-cinder | 19:23 | |
| patrickeast | eharney: yea looks like the pip in my tox env doesn’t like that syntax : / | 19:23 |
| *** bnemec has joined #openstack-cinder | 19:24 | |
| eharney | patrickeast: need a current version of pip and pbr i think | 19:25 |
| patrickeast | looks like upgrading my tox install helped | 19:28 |
| *** earlephilhower has joined #openstack-cinder | 19:29 | |
| e0ne | thingee, mtreinish: thanks for the comments. afair, this job was experimental because it was not stable:( | 19:30 |
| mtreinish | e0ne: I'm pretty sure it was just because jgriffith forgot to push the follow up patch to make it voting | 19:31 |
| mtreinish | the tests there are so limited that I doubt stability was really an issue | 19:32 |
| e0ne | mtreinish: i mean not tests but cinderclient job for them | 19:32 |
| mtreinish | hmm, not sure it's just deploys devstack like other jobs | 19:34 |
| e0ne | :) | 19:34 |
| *** rushil has joined #openstack-cinder | 19:36 | |
| *** _cjones_ has quit IRC | 19:37 | |
| *** _cjones_ has joined #openstack-cinder | 19:37 | |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 19:41 |
| *** lpetrut has joined #openstack-cinder | 19:45 | |
| *** edmondsw has quit IRC | 19:46 | |
| smcginnis | thingee: Just noticed this: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/datera.py#L36 | 19:47 |
| smcginnis | thingee: Might be time? | 19:47 |
| thingee | smcginnis: yes, thank you | 19:47 |
| smcginnis | np | 19:47 |
| *** rushil has quit IRC | 19:49 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Add more Rally scenarios to run https://review.openstack.org/194180 | 19:52 |
| e0ne | zuul: Queue lengths: 295 events :( | 19:53 |
| smcginnis | e0ne: Yeah, not looking too happy. :[ | 19:54 |
| openstackgerrit | Eric Harney proposed openstack/cinder: LVM Thin Provisioning auto-detect https://review.openstack.org/104653 | 19:54 |
| *** edmondsw has joined #openstack-cinder | 19:54 | |
| *** rwsu has joined #openstack-cinder | 19:55 | |
| jgriffith | mtreinish e0ne uh-oh, what didn't I finish? | 19:56 |
| mtreinish | the cli testing migration :) | 19:56 |
| jgriffith | mtreinish Hmm... thought I landed those | 19:57 |
| jgriffith | BAHHH | 19:57 |
| *** xyang has joined #openstack-cinder | 19:58 | |
| mtreinish | jgriffith: well most of it did land, just the last step to switch the job from experimental to voting | 20:03 |
| *** e0ne is now known as e0ne_ | 20:04 | |
| *** e0ne_ is now known as e0ne | 20:04 | |
| jgriffith | mtreinish: yeah, my bad :( | 20:04 |
| *** diemt has quit IRC | 20:05 | |
| *** rushil has joined #openstack-cinder | 20:06 | |
| *** patrickeast has quit IRC | 20:07 | |
| *** patrickeast has joined #openstack-cinder | 20:08 | |
| *** rwsu has quit IRC | 20:16 | |
| *** lifeless has quit IRC | 20:20 | |
| openstackgerrit | Peter Penchev proposed openstack/cinder: StorPool: clean up the last uses of str.format() https://review.openstack.org/190982 | 20:20 |
| *** raunak has joined #openstack-cinder | 20:21 | |
| *** lifeless has joined #openstack-cinder | 20:21 | |
| *** jgriffith_ has joined #openstack-cinder | 20:23 | |
| *** jgriffith has quit IRC | 20:24 | |
| *** Guest55 has joined #openstack-cinder | 20:25 | |
| *** Guest55 has quit IRC | 20:25 | |
| *** garthb has joined #openstack-cinder | 20:25 | |
| *** jgriffith_ is now known as jgriffith | 20:26 | |
| earlephilhower | hemna: ping | 20:27 |
| hemna | hey | 20:27 |
| earlephilhower | hemna, howdy. Any plans for making a new tag 0.3.0 for the os_brick pip library w/new L1 commits? | 20:27 |
| Swanson | What guest number will jgriffith be next? Gonna start an office pool. | 20:27 |
| earlephilhower | I think Devstack or Nova req's changed over last week's vacation, overriding my "git clone/install" from the git head. All my CI's Nova tests die on missing HGSTConnector. Will try to hack around it, but a more "legit" solution would be nice. | 20:27 |
| hemna | earlephilhower, I think thingee was working on a new drop last week. not sure what happened w/ it. I think the plan normally is to ship on the milestones. | 20:28 |
| *** rwsu has joined #openstack-cinder | 20:29 | |
| *** nikeshm has quit IRC | 20:30 | |
| *** Yogi1 has quit IRC | 20:30 | |
| *** jungleboyj has quit IRC | 20:30 | |
| earlephilhower | hemna, thanks. Dumb question, but how do milestones relate to L1/L2/etc.? I'm not quite clear on the terminology differences. | 20:31 |
| *** annegentle has quit IRC | 20:32 | |
| *** lurker has quit IRC | 20:33 | |
| *** lpetrut has quit IRC | 20:33 | |
| hemna | those are milestones :) | 20:35 |
| jgriffith | earlephilhower: Those "are" the milestones. i.e. L1, L2, L3... milestone-1, 2, 3 | 20:35 |
| jgriffith | :) | 20:35 |
| scottda | Where's a good place to put a diagram (i.e .png) to link to from an etherpad? | 20:37 |
| *** angela-s has quit IRC | 20:37 | |
| *** xiangfeiz has quit IRC | 20:37 | |
| *** earlephilhower has quit IRC | 20:38 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tests: test_volume mock conversion https://review.openstack.org/201241 | 20:38 |
| jgriffith | scottda: imager maybe? | 20:38 |
| jgriffith | uhhh... "imgur" | 20:38 |
| jgriffith | auto-correct | 20:38 |
| *** pv_ has quit IRC | 20:38 | |
| scottda | that'll work. Thanks | 20:39 |
| scottda | It'd be nice to have this in etherpad, but .... | 20:39 |
| *** rushil has quit IRC | 20:40 | |
| *** rushil has joined #openstack-cinder | 20:40 | |
| *** annashen has quit IRC | 20:41 | |
| *** earlephilhower has joined #openstack-cinder | 20:42 | |
| earlephilhower | Thanks, guys. The first day back in the office is always a bit rough. :) | 20:42 |
| *** rushil has quit IRC | 20:46 | |
| thingee | hemna: waiting on https://review.openstack.org/#/c/199766/ | 20:49 |
| thingee | earlephilhower: ^ | 20:50 |
| hemna | thingee, oh! :P | 20:50 |
| *** crose has quit IRC | 20:51 | |
| *** crose has joined #openstack-cinder | 20:52 | |
| erlon | thingee: jgriffith Hi Mike, John, can you merge this? it only needs one more +2 | 20:52 |
| *** angela-s has joined #openstack-cinder | 20:52 | |
| *** bswartz has quit IRC | 20:53 | |
| jgriffith | erlon: merge what? | 20:55 |
| erlon | jgriffith: sorry, https://review.openstack.org/#/c/197686/ | 20:55 |
| erlon | jgriffith: forgot you are not reading my mind :) | 20:55 |
| jgriffith | erlon: don't be sure I'm not... may have just been asking for the benefit of others :) | 20:56 |
| erlon | jgriffith: haha | 20:56 |
| earlephilhower | thingee, hemna: Wow, that's service! :) | 20:57 |
| *** delatte has joined #openstack-cinder | 20:57 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 20:58 |
| *** delattec has quit IRC | 21:00 | |
| *** dustins has quit IRC | 21:00 | |
| *** annashen has joined #openstack-cinder | 21:01 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Preserve mock side_effect’s in test_pure https://review.openstack.org/201334 | 21:01 |
| erlon | jgriffith: thanks! | 21:03 |
| *** annegentle has joined #openstack-cinder | 21:04 | |
| *** delattec has joined #openstack-cinder | 21:05 | |
| jgriffith | erlon: welcome | 21:05 |
| *** annegentle has quit IRC | 21:06 | |
| *** akerr has quit IRC | 21:07 | |
| *** annegentle has joined #openstack-cinder | 21:07 | |
| *** delatte has quit IRC | 21:08 | |
| *** ronis has quit IRC | 21:08 | |
| hodos | hi guys I have a problem started to appear recently with my nfs driver: tempest fails on test_encrypted_cinder_volumes_ | 21:08 |
| openstackgerrit | Patrick East proposed openstack/cinder: Preserve mock side_effect’s in test_pure https://review.openstack.org/201334 | 21:08 |
| hodos | while iscsi passes them | 21:09 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 21:09 |
| hodos | does any1 experiance this? | 21:09 |
| jgriffith | hodos: A change was recently merged to fix a silent bug | 21:09 |
| jgriffith | hodos: looking for patch... | 21:09 |
| jgriffith | hodos: check this... might be related: https://goo.gl/DyIC7B | 21:10 |
| hodos | jgriffith: thanks! | 21:10 |
| *** xyang has quit IRC | 21:11 | |
| hodos | jgriffith: i think it's too old, or maybe these tests have been intoduced just recently to tempest? | 21:14 |
| *** breitz has quit IRC | 21:15 | |
| hodos | jgriffith: http://140.174.232.106/refs-changes-05-201005-4/ns_nfs/logs/screen-n-cpu.log.txt here first Traceback shows that encryptor.attach_volume fails | 21:17 |
| *** hodos has quit IRC | 21:17 | |
| *** dannywilson has joined #openstack-cinder | 21:17 | |
| *** breitz has joined #openstack-cinder | 21:17 | |
| jgriffith | hodos: Yes, the tests were recently *fixed up* as well | 21:17 |
| *** hodos has joined #openstack-cinder | 21:22 | |
| *** jungleboyj has joined #openstack-cinder | 21:23 | |
| *** annegent_ has joined #openstack-cinder | 21:26 | |
| *** coolsvap has quit IRC | 21:28 | |
| *** annegentle has quit IRC | 21:29 | |
| *** mriedem is now known as mriedem_away | 21:31 | |
| *** mdenny has quit IRC | 21:32 | |
| *** kjnelson has quit IRC | 21:33 | |
| *** coolsvap|away has joined #openstack-cinder | 21:33 | |
| openstackgerrit | Raunak Kumar proposed openstack/cinder: Add extra spec capability for Nimble Cinder Driver https://review.openstack.org/198853 | 21:33 |
| *** dims___ has joined #openstack-cinder | 21:34 | |
| *** e0ne has quit IRC | 21:35 | |
| *** dims_ has quit IRC | 21:35 | |
| *** eharney has quit IRC | 21:36 | |
| *** crose has quit IRC | 21:40 | |
| *** mdenny has joined #openstack-cinder | 21:42 | |
| *** julim has quit IRC | 21:46 | |
| *** evilrob has joined #openstack-cinder | 21:48 | |
| *** albertom has quit IRC | 21:58 | |
| *** akerr has joined #openstack-cinder | 22:01 | |
| *** akerr_ has joined #openstack-cinder | 22:02 | |
| *** albertom has joined #openstack-cinder | 22:04 | |
| *** akerr has quit IRC | 22:05 | |
| *** ianbrown has joined #openstack-cinder | 22:08 | |
| *** jungleboyj has quit IRC | 22:12 | |
| *** wyatt88 has joined #openstack-cinder | 22:13 | |
| *** primechuck has quit IRC | 22:15 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: POC: Adding get_capabilities() feature https://review.openstack.org/201243 | 22:17 |
| *** alexpilotti has joined #openstack-cinder | 22:19 | |
| *** edmondsw has quit IRC | 22:20 | |
| *** markvoelker has joined #openstack-cinder | 22:27 | |
| *** markvoelker has quit IRC | 22:27 | |
| *** markvoelker has joined #openstack-cinder | 22:27 | |
| *** bnemec has quit IRC | 22:28 | |
| *** dannywilson has quit IRC | 22:29 | |
| *** dannywilson has joined #openstack-cinder | 22:30 | |
| *** dims_ has joined #openstack-cinder | 22:32 | |
| *** xyang1 has joined #openstack-cinder | 22:33 | |
| *** dims___ has quit IRC | 22:35 | |
| *** shakamunyi has quit IRC | 22:45 | |
| *** dims__ has joined #openstack-cinder | 22:46 | |
| *** dims_ has quit IRC | 22:49 | |
| *** lcurtis has quit IRC | 22:51 | |
| *** rwsu has quit IRC | 22:52 | |
| *** jwcroppe has quit IRC | 22:52 | |
| *** jwcroppe has joined #openstack-cinder | 22:53 | |
| patrickeast | uh oh, are the unit tests broken for stable/kilo? i see failures on https://review.openstack.org/#/c/201299/ that dont have anything to do with the change being added : ( | 22:55 |
| wyatt88 | patrickeast: yes | 22:55 |
| jgriffith | patrickeast: stable unit tests are messed up because of the mock lib update | 22:55 |
| patrickeast | wyatt88: is there a bug/patch already for it? | 22:55 |
| patrickeast | ahh ok | 22:55 |
| jgriffith | wyatt88: patrickeast change to pin has gone into global | 22:56 |
| jgriffith | but not in stable yet | 22:56 |
| jgriffith | I had a patch proposed but it fails to build the test env for the comparison | 22:56 |
| jgriffith | I dunno what's up | 22:56 |
| *** dims__ has quit IRC | 22:56 | |
| *** jwcroppe has quit IRC | 22:57 | |
| jgriffith | patrickeast: https://goo.gl/mKoIq8 | 22:57 |
| jgriffith | It finally passed, so need to merge it | 22:58 |
| *** sgotliv has quit IRC | 22:58 | |
| patrickeast | sweet | 22:58 |
| jgriffith | Looks like I need to fix up the commit message | 22:58 |
| thingee | jgriffith: we should revisit your question here or... https://review.openstack.org/#/c/153865/ | 22:58 |
| jgriffith | thingee: patrickeast can you guys merge this one: https://review.openstack.org/#/c/200747/ | 22:59 |
| jgriffith | I updated the commit message as per the issue eric pointed out | 22:59 |
| jgriffith | thingee: looking.... | 22:59 |
| jgriffith | thingee: oh.. that. I stand by my original comment | 23:00 |
| jgriffith | thingee: If you are asking me to downgrade my -2 to a -1 I can do that, but I think it's the complete wrong direction to be going | 23:00 |
| jgriffith | pandora's box | 23:00 |
| jgriffith | don't we offer enough in tree choices already? | 23:01 |
| jgriffith | partticularly vendr | 23:02 |
| thingee | jgriffith: not arguing against your comment, just saying I want to revisit this | 23:02 |
| *** akshai has quit IRC | 23:02 | |
| jgriffith | vendor even :) | 23:02 |
| thingee | jgriffith: also wrt your review, have you seen https://github.com/openstack/cinder/blob/master/test-requirements.txt#L12 | 23:02 |
| thingee | I've been confused by all this myself | 23:02 |
| jgriffith | thingee: yeah, this all started with a perceived problem for 2.6 only | 23:03 |
| jgriffith | thingee: then it turned out it wasn't just that "one" problem | 23:03 |
| thingee | jgriffith: is there already a review in global-requirements? | 23:03 |
| jgriffith | thingee: https://github.com/openstack/requirements/commit/407e3b2d028a1311e74b18f64a57963369f1bf02#diff-d7d5c6fa7118ea10d88f3afeaef4da77 | 23:04 |
| jgriffith | and.... | 23:04 |
| jgriffith | https://github.com/openstack/requirements/commit/7cf739c4f85ce41634475cdf494cd0e8353c008d#diff-d7d5c6fa7118ea10d88f3afeaef4da77 | 23:04 |
| jgriffith | The first being master, the second being stable/kilo | 23:04 |
| jgriffith | thingee: stable/kilo being the one I was trying to land last week | 23:05 |
| jgriffith | thingee: so we can pass unit tests. IMO reworking the unit tests in stable isn't the right approach. Better to pin the lib | 23:05 |
| jgriffith | thingee: thus the patch to update req's. Although at this point I would imagine it's on it's way through the automated process FWIW | 23:06 |
| jgriffith | thingee: so when you say "revisit" the ibm backup connector patch..... what's your thought there? | 23:06 |
| *** diogogmt has quit IRC | 23:08 | |
| thingee | jgriffith: For us to discuss. This has the dependency on your out of tree spec. I'm not sure if we can take on getting that in this release, and I don't think it would be fair to block someone because of that. | 23:08 |
| thingee | jgriffith: oh I totally missed this was for stable/kilo | 23:09 |
| *** akerr_ has quit IRC | 23:10 | |
| jgriffith | thingee: wait... I'm confused; are you still talking about the spec here: https://review.openstack.org/#/c/153865/ | 23:10 |
| thingee | you and I are talking about the mock requirements change and the backup driver out tree. you know, like we do :) | 23:11 |
| thingee | jgriffith: | 23:11 |
| jgriffith | thingee: :) | 23:11 |
| thingee | jgriffith: so first, I'm with this mock change in stable/kilo | 23:11 |
| jgriffith | k | 23:11 |
| *** mgarza_ has quit IRC | 23:12 | |
| thingee | jgriffith: second, the blocking the backup driver from liberty, I'm unsure of because of out of tree not existing yet in Cinder. | 23:12 |
| jgriffith | thingee: well, my original -2 was because it was way late in the cycle | 23:13 |
| jgriffith | thingee: as far as bringing it back/revisiting | 23:13 |
| thingee | jgriffith: who knows, ronen probably forgot about it at this point | 23:14 |
| jgriffith | thingee: my comments about out of tree had nothing to do with my proposal to remove things from tree. I was stating that using plugin appliances that bridge between Cinder and a backup device (which is what ProtecTier does | 23:14 |
| jgriffith | doesn't belong | 23:14 |
| jgriffith | In other words, isn't it enough that we have every device option under the sun, now we want gateways? | 23:14 |
| thingee | jgriffith: ah that's interesting. I was not aware that's what it does | 23:15 |
| jgriffith | If IBM wants a gateway device supported then they can do that AFAIC, but I see no reason it should be added to tree | 23:15 |
| jgriffith | thingee: We have to draw the line somewhere IMHO, and if we're not willing to do it with drivers, at least maybe we can do it with "gateways" | 23:16 |
| thingee | jgriffith: yup, no disagreements with this | 23:17 |
| *** ganso_ has quit IRC | 23:18 | |
| *** chlong-afk has joined #openstack-cinder | 23:18 | |
| jgriffith | thingee: I updated the comment to clarify | 23:19 |
| *** mdbooth has quit IRC | 23:19 | |
| jgriffith | thingee: as I think a few people may not have captured that detail | 23:19 |
| *** hodos has quit IRC | 23:24 | |
| *** mdbooth has joined #openstack-cinder | 23:25 | |
| *** topshare has joined #openstack-cinder | 23:28 | |
| *** bswartz has joined #openstack-cinder | 23:35 | |
| *** chlong-afk has quit IRC | 23:36 | |
| *** dims_ has joined #openstack-cinder | 23:42 | |
| *** topshare has quit IRC | 23:52 | |
| *** annashen has quit IRC | 23:54 | |
| *** laughterwym has joined #openstack-cinder | 23:54 | |
| *** vokt has quit IRC | 23:54 | |
| *** jwcroppe has joined #openstack-cinder | 23:55 | |
| *** laughterwym_ has joined #openstack-cinder | 23:56 | |
| *** _cjones_ has quit IRC | 23:56 | |
| *** ianbrown has quit IRC | 23:57 | |
| *** laughterwym has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!