*** dims_ has joined #openstack-cinder | 00:06 | |
*** IlyaG has joined #openstack-cinder | 00:08 | |
*** chlong has joined #openstack-cinder | 00:20 | |
*** primechuck has joined #openstack-cinder | 00:20 | |
*** annashen has quit IRC | 00:25 | |
*** IlyaG has quit IRC | 00:27 | |
openstackgerrit | Anish Bhatt proposed openstack/os-brick: os-brick : Add support for --interface option in iscsiadm. https://review.openstack.org/193451 | 00:32 |
---|---|---|
*** Apoorva has quit IRC | 00:35 | |
*** piyanai has quit IRC | 00:35 | |
*** piyanai has joined #openstack-cinder | 00:37 | |
*** piyanai has quit IRC | 00:37 | |
*** IlyaG has joined #openstack-cinder | 00:37 | |
*** topshare has joined #openstack-cinder | 00:44 | |
*** IlyaG has quit IRC | 00:50 | |
*** topshare has quit IRC | 00:53 | |
*** liuxg has joined #openstack-cinder | 01:00 | |
*** heyun has joined #openstack-cinder | 01:00 | |
*** topshare has joined #openstack-cinder | 01:01 | |
*** lixiaoy1 has joined #openstack-cinder | 01:05 | |
*** topshare_ has joined #openstack-cinder | 01:07 | |
*** topshare has quit IRC | 01:10 | |
liuxg | . | 01:12 |
*** changbl has joined #openstack-cinder | 01:17 | |
*** lixiaoy11 has joined #openstack-cinder | 01:20 | |
*** liuxg has quit IRC | 01:21 | |
*** lixiaoy1 has quit IRC | 01:22 | |
*** liuxg has joined #openstack-cinder | 01:24 | |
*** changbl has quit IRC | 01:25 | |
*** annashen has joined #openstack-cinder | 01:26 | |
*** davechen has joined #openstack-cinder | 01:28 | |
*** lixiaoy1 has joined #openstack-cinder | 01:29 | |
*** lixiaoy11 has quit IRC | 01:31 | |
*** davechen1 has joined #openstack-cinder | 01:31 | |
*** davechen has quit IRC | 01:33 | |
*** Apoorva has joined #openstack-cinder | 01:35 | |
*** piyanai has joined #openstack-cinder | 01:36 | |
*** changbl_ has joined #openstack-cinder | 01:36 | |
*** davechen1 is now known as davechen | 01:38 | |
*** changbl has joined #openstack-cinder | 01:38 | |
*** lan has joined #openstack-cinder | 01:39 | |
*** davechen1 has joined #openstack-cinder | 01:40 | |
*** Apoorva has quit IRC | 01:40 | |
*** zhenguo has joined #openstack-cinder | 01:42 | |
*** davechen has quit IRC | 01:43 | |
*** changbl_ has quit IRC | 01:43 | |
*** IlyaG has joined #openstack-cinder | 01:44 | |
*** changbl has quit IRC | 01:49 | |
*** lixiaoy11 has joined #openstack-cinder | 01:49 | |
*** topshare_ has quit IRC | 01:50 | |
*** topshare has joined #openstack-cinder | 01:50 | |
*** lixiaoy1 has quit IRC | 01:51 | |
*** gouthamr_ has quit IRC | 01:57 | |
*** gouthamr has joined #openstack-cinder | 01:58 | |
*** lixiaoy1 has joined #openstack-cinder | 02:12 | |
*** dims_ has quit IRC | 02:14 | |
*** lixiaoy11 has quit IRC | 02:14 | |
*** changbl has joined #openstack-cinder | 02:17 | |
*** afazekas has quit IRC | 02:17 | |
*** annashen has quit IRC | 02:17 | |
*** afazekas has joined #openstack-cinder | 02:18 | |
openstackgerrit | Li Yingjun proposed openstack/cinder-specs: Add ability to update volume type public spec https://review.openstack.org/203462 | 02:18 |
*** topshare has quit IRC | 02:21 | |
*** topshare has joined #openstack-cinder | 02:22 | |
*** BharatK has joined #openstack-cinder | 02:22 | |
*** dims_ has joined #openstack-cinder | 02:29 | |
*** liuxg has quit IRC | 02:38 | |
*** IlyaG has quit IRC | 02:43 | |
*** Lee1092 has joined #openstack-cinder | 02:43 | |
*** liuxg has joined #openstack-cinder | 02:44 | |
*** topshare has quit IRC | 02:45 | |
*** kkeuning has joined #openstack-cinder | 02:46 | |
*** primechuck has quit IRC | 02:46 | |
*** topshare has joined #openstack-cinder | 02:47 | |
*** kkeuning has quit IRC | 02:47 | |
*** kkeuning has joined #openstack-cinder | 02:48 | |
*** hakimo has joined #openstack-cinder | 02:52 | |
*** topshare has quit IRC | 02:53 | |
*** Apoorva has joined #openstack-cinder | 02:53 | |
*** hakimo_ has quit IRC | 02:55 | |
openstackgerrit | Xi Yang proposed openstack/cinder: Provide snap copy feature in EMC VNX Cinder driver https://review.openstack.org/184733 | 02:57 |
openstackgerrit | wanghao proposed openstack/cinder: Add updated_at into response of querying cinder resource detail https://review.openstack.org/202029 | 03:02 |
*** tobe has joined #openstack-cinder | 03:06 | |
*** dims_ has quit IRC | 03:08 | |
openstackgerrit | xing-yang proposed openstack/cinder: Clone CG https://review.openstack.org/203473 | 03:23 |
*** gouthamr_ has joined #openstack-cinder | 03:26 | |
*** gouthamr has quit IRC | 03:28 | |
*** gouthamr_ has quit IRC | 03:32 | |
*** topshare has joined #openstack-cinder | 03:35 | |
*** vilobhmm has joined #openstack-cinder | 03:36 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 03:37 |
*** topshare has quit IRC | 03:39 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 03:42 |
*** coolsvap|away is now known as coolsvap | 03:46 | |
openstackgerrit | xing-yang proposed openstack/cinder: Add multiple pools support to VMAX driver https://review.openstack.org/189990 | 03:48 |
openstackgerrit | xing-yang proposed openstack/cinder: EMC VMAX Modify CG https://review.openstack.org/194829 | 03:54 |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 03:56 |
openstackgerrit | xing-yang proposed openstack/cinder: EMC VMAX Create CG from CG Snapshot https://review.openstack.org/195280 | 04:01 |
*** tobe has quit IRC | 04:02 | |
*** tobe has joined #openstack-cinder | 04:05 | |
*** heyun has quit IRC | 04:07 | |
*** heyun has joined #openstack-cinder | 04:07 | |
*** annashen has joined #openstack-cinder | 04:08 | |
*** dims_ has joined #openstack-cinder | 04:10 | |
*** cdelatte has joined #openstack-cinder | 04:11 | |
*** delattec has joined #openstack-cinder | 04:11 | |
*** tobe has quit IRC | 04:13 | |
*** dims_ has quit IRC | 04:15 | |
*** tobe has joined #openstack-cinder | 04:16 | |
*** changbl has quit IRC | 04:19 | |
*** annashen has quit IRC | 04:19 | |
*** annashen has joined #openstack-cinder | 04:20 | |
*** shyama has joined #openstack-cinder | 04:28 | |
*** changbl has joined #openstack-cinder | 04:33 | |
*** topshare has joined #openstack-cinder | 04:34 | |
*** annashen has quit IRC | 04:36 | |
*** kkeuning has quit IRC | 04:38 | |
*** tobe has quit IRC | 04:41 | |
*** piyanai has quit IRC | 04:43 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 04:44 |
*** Apoorva has quit IRC | 04:51 | |
*** briancurtin has quit IRC | 04:58 | |
*** ameade has quit IRC | 04:59 | |
*** briancurtin has joined #openstack-cinder | 04:59 | |
*** rhefner has quit IRC | 04:59 | |
*** rhefner has joined #openstack-cinder | 05:00 | |
*** ctracey has quit IRC | 05:00 | |
*** ameade has joined #openstack-cinder | 05:01 | |
*** thingee has quit IRC | 05:02 | |
*** ctracey has joined #openstack-cinder | 05:03 | |
*** rhefner has quit IRC | 05:04 | |
*** thingee has joined #openstack-cinder | 05:05 | |
*** rhefner has joined #openstack-cinder | 05:07 | |
*** annashen has joined #openstack-cinder | 05:10 | |
openstackgerrit | LisaLi proposed openstack/python-cinderclient: Show backup and volume info in backup_restore https://review.openstack.org/199846 | 05:20 |
*** annashen has quit IRC | 05:20 | |
*** deepakcs has joined #openstack-cinder | 05:22 | |
*** chlong has quit IRC | 05:27 | |
lixiaoy1 | Any cores help to review this patch: https://review.openstack.org/#/c/198993/ | 05:28 |
lixiaoy1 | This is related to backup restore in ceph | 05:28 |
*** dims_ has joined #openstack-cinder | 05:31 | |
*** DuncanT has quit IRC | 05:35 | |
*** annashen has joined #openstack-cinder | 05:35 | |
*** Lee1092 has quit IRC | 05:35 | |
*** zhenguo has quit IRC | 05:35 | |
*** rex_lee has quit IRC | 05:35 | |
*** serverascode has quit IRC | 05:35 | |
*** h00327910_ has quit IRC | 05:36 | |
*** scottda has quit IRC | 05:36 | |
*** ameade has quit IRC | 05:36 | |
*** briancurtin has quit IRC | 05:36 | |
*** ctracey has quit IRC | 05:36 | |
*** annashen has quit IRC | 05:36 | |
*** dims_ has quit IRC | 05:36 | |
*** rhefner has quit IRC | 05:36 | |
*** serverascode has joined #openstack-cinder | 05:39 | |
*** chlong has joined #openstack-cinder | 05:40 | |
*** DuncanT has joined #openstack-cinder | 05:40 | |
*** ameade has joined #openstack-cinder | 05:40 | |
*** Lee1092 has joined #openstack-cinder | 05:41 | |
*** vlaza has joined #openstack-cinder | 05:43 | |
*** annashen has joined #openstack-cinder | 05:43 | |
*** scottda has joined #openstack-cinder | 05:43 | |
*** briancurtin has joined #openstack-cinder | 05:44 | |
*** rhefner has joined #openstack-cinder | 05:44 | |
*** h00327910_ has joined #openstack-cinder | 05:48 | |
*** zhenguo has joined #openstack-cinder | 05:49 | |
*** rex_lee has joined #openstack-cinder | 05:49 | |
*** annashen has quit IRC | 05:50 | |
*** topshare has quit IRC | 05:51 | |
*** ctracey has joined #openstack-cinder | 05:51 | |
*** annashen has joined #openstack-cinder | 05:54 | |
*** nkrinner has joined #openstack-cinder | 05:55 | |
*** geguileo has joined #openstack-cinder | 05:57 | |
*** annashen has quit IRC | 06:02 | |
*** vlaza has quit IRC | 06:07 | |
*** alonmarx has quit IRC | 06:12 | |
*** e0ne has joined #openstack-cinder | 06:13 | |
*** alonmarx has joined #openstack-cinder | 06:13 | |
*** heyun has quit IRC | 06:13 | |
*** Tross has quit IRC | 06:13 | |
*** Tross has joined #openstack-cinder | 06:14 | |
*** heyun has joined #openstack-cinder | 06:14 | |
*** topshare has joined #openstack-cinder | 06:16 | |
*** e0ne has quit IRC | 06:17 | |
*** chlong has quit IRC | 06:20 | |
*** e0ne has joined #openstack-cinder | 06:21 | |
*** zhenguo has quit IRC | 06:23 | |
*** vlaza has joined #openstack-cinder | 06:27 | |
*** e0ne has quit IRC | 06:29 | |
*** e0ne has joined #openstack-cinder | 06:33 | |
*** _afazekas has quit IRC | 06:33 | |
*** chlong has joined #openstack-cinder | 06:36 | |
*** boris-42 has joined #openstack-cinder | 06:38 | |
*** tobe has joined #openstack-cinder | 06:42 | |
*** vilobhmm has quit IRC | 06:43 | |
*** ronis has joined #openstack-cinder | 06:48 | |
*** geguileor has joined #openstack-cinder | 06:49 | |
*** geguileo has quit IRC | 06:53 | |
*** e0ne has quit IRC | 06:56 | |
*** zhenguo has joined #openstack-cinder | 06:58 | |
*** e0ne has joined #openstack-cinder | 07:00 | |
*** IanGovett has joined #openstack-cinder | 07:07 | |
*** afazekas_ has joined #openstack-cinder | 07:09 | |
*** agarciam1 has quit IRC | 07:11 | |
*** dims_ has joined #openstack-cinder | 07:19 | |
*** e0ne has quit IRC | 07:22 | |
*** abehl has joined #openstack-cinder | 07:25 | |
*** dims_ has quit IRC | 07:25 | |
*** e0ne has joined #openstack-cinder | 07:26 | |
*** BharatK_ has joined #openstack-cinder | 07:28 | |
*** e0ne has quit IRC | 07:29 | |
*** e0ne has joined #openstack-cinder | 07:29 | |
*** BharatK has quit IRC | 07:30 | |
*** e0ne has quit IRC | 07:30 | |
*** ctracey has quit IRC | 07:31 | |
*** Lee1092 has quit IRC | 07:31 | |
*** sweston has quit IRC | 07:31 | |
*** scottda has quit IRC | 07:31 | |
*** rex_lee has quit IRC | 07:31 | |
*** zhenguo has quit IRC | 07:31 | |
*** zhiyan has quit IRC | 07:31 | |
*** cppforlife_ has quit IRC | 07:31 | |
*** ameade has quit IRC | 07:31 | |
*** thingee has quit IRC | 07:31 | |
*** DuncanT has quit IRC | 07:31 | |
*** serverascode has quit IRC | 07:31 | |
*** topshare has quit IRC | 07:33 | |
*** topshare has joined #openstack-cinder | 07:33 | |
*** chlong has quit IRC | 07:34 | |
*** belmoreira has joined #openstack-cinder | 07:36 | |
*** pcaruana has quit IRC | 07:42 | |
*** Lee1092 has joined #openstack-cinder | 07:44 | |
*** ameade has joined #openstack-cinder | 07:48 | |
*** scottda has joined #openstack-cinder | 07:48 | |
*** zhiyan has joined #openstack-cinder | 07:48 | |
*** sweston has joined #openstack-cinder | 07:49 | |
*** topshare has quit IRC | 07:51 | |
*** thingee has joined #openstack-cinder | 07:51 | |
*** zhenguo has joined #openstack-cinder | 07:53 | |
*** cppforlife_ has joined #openstack-cinder | 07:53 | |
*** jistr has joined #openstack-cinder | 07:54 | |
*** ctracey has joined #openstack-cinder | 07:54 | |
*** geguileo has joined #openstack-cinder | 07:55 | |
*** geguileor has quit IRC | 07:57 | |
*** serverascode has joined #openstack-cinder | 07:57 | |
*** topshare has joined #openstack-cinder | 08:01 | |
*** rex_lee has joined #openstack-cinder | 08:01 | |
*** topshare has quit IRC | 08:02 | |
*** bluex-pl has joined #openstack-cinder | 08:06 | |
*** geguileor has joined #openstack-cinder | 08:07 | |
*** bluex-pl has quit IRC | 08:07 | |
*** sgotliv has joined #openstack-cinder | 08:07 | |
*** bluex-pl has joined #openstack-cinder | 08:07 | |
*** DuncanT has joined #openstack-cinder | 08:09 | |
*** akrog has joined #openstack-cinder | 08:09 | |
*** zhenguo has quit IRC | 08:10 | |
*** geguileo has quit IRC | 08:10 | |
*** geguileor has quit IRC | 08:11 | |
*** geguileo has joined #openstack-cinder | 08:13 | |
*** akrog has quit IRC | 08:16 | |
*** rushiagr_away is now known as rushiagr | 08:17 | |
*** zhenguo has joined #openstack-cinder | 08:19 | |
*** dalgaaf has joined #openstack-cinder | 08:22 | |
*** shyama has quit IRC | 08:22 | |
*** shyama has joined #openstack-cinder | 08:22 | |
*** Zhongjun has joined #openstack-cinder | 08:23 | |
*** aarefiev has joined #openstack-cinder | 08:23 | |
*** jordanP has joined #openstack-cinder | 08:26 | |
*** belmoreira has quit IRC | 08:30 | |
*** pradipta has joined #openstack-cinder | 08:30 | |
*** jordanP has quit IRC | 08:31 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add support for file I/O volume migration https://review.openstack.org/187270 | 08:32 |
*** pcaruana has joined #openstack-cinder | 08:43 | |
*** jordanP has joined #openstack-cinder | 08:45 | |
*** topshare has joined #openstack-cinder | 08:54 | |
*** e0ne has joined #openstack-cinder | 08:55 | |
*** mdbooth has joined #openstack-cinder | 08:56 | |
*** topshare has quit IRC | 09:00 | |
*** topshare has joined #openstack-cinder | 09:00 | |
*** geguileor has joined #openstack-cinder | 09:01 | |
*** geguileo has quit IRC | 09:02 | |
*** akrog has joined #openstack-cinder | 09:02 | |
*** geguileor has quit IRC | 09:05 | |
*** dims_ has joined #openstack-cinder | 09:08 | |
openstackgerrit | wanghao proposed openstack/cinder: Improve performance of listing detail of volumes which created from image https://review.openstack.org/203530 | 09:13 |
*** dims_ has quit IRC | 09:14 | |
*** topshare has quit IRC | 09:17 | |
*** rushiagr is now known as rushiagr_away | 09:18 | |
*** topshare has joined #openstack-cinder | 09:21 | |
*** geguileo has joined #openstack-cinder | 09:23 | |
*** akrog has quit IRC | 09:27 | |
*** rushiagr_away is now known as rushiagr | 09:30 | |
*** yrabl has quit IRC | 09:33 | |
*** yrabl has joined #openstack-cinder | 09:34 | |
*** akrog has joined #openstack-cinder | 09:35 | |
*** shyama has quit IRC | 09:40 | |
*** aix has quit IRC | 09:41 | |
*** shyama has joined #openstack-cinder | 09:42 | |
*** akrog has quit IRC | 09:43 | |
*** geguileo has quit IRC | 09:46 | |
*** belmoreira has joined #openstack-cinder | 09:48 | |
*** lixiaoy1 has quit IRC | 09:51 | |
*** davechen1 has left #openstack-cinder | 09:54 | |
*** dims_ has joined #openstack-cinder | 09:54 | |
*** agarciam has joined #openstack-cinder | 09:55 | |
*** bluex-pl has quit IRC | 10:00 | |
*** bluex-pl has joined #openstack-cinder | 10:00 | |
*** s0nea has quit IRC | 10:08 | |
*** s0nea has joined #openstack-cinder | 10:10 | |
*** heyun has quit IRC | 10:14 | |
*** lcurtis has joined #openstack-cinder | 10:15 | |
*** belmoreira has quit IRC | 10:16 | |
*** hakimo has quit IRC | 10:17 | |
*** hakimo has joined #openstack-cinder | 10:20 | |
*** deepakcs has quit IRC | 10:22 | |
*** jaypipes has joined #openstack-cinder | 10:34 | |
*** lcurtis has quit IRC | 10:35 | |
*** geguileo has joined #openstack-cinder | 10:38 | |
*** tobe has quit IRC | 10:40 | |
*** piyanai has joined #openstack-cinder | 10:41 | |
*** lcurtis has joined #openstack-cinder | 10:48 | |
*** jasondotstar has joined #openstack-cinder | 10:50 | |
*** tobe has joined #openstack-cinder | 10:52 | |
*** lcurtis has quit IRC | 10:54 | |
*** topshare has quit IRC | 10:57 | |
*** liuxg has quit IRC | 10:59 | |
*** albertom has quit IRC | 11:04 | |
*** ig0r_ has joined #openstack-cinder | 11:08 | |
openstackgerrit | Stefan Amann proposed openstack/os-brick: Perform port_rescan on s390x platforms https://review.openstack.org/203026 | 11:09 |
*** albertom has joined #openstack-cinder | 11:11 | |
*** coolsvap is now known as coolsvap|away | 11:15 | |
*** BharatK has joined #openstack-cinder | 11:16 | |
*** BharatK_ has quit IRC | 11:19 | |
*** rakesh_mishra has joined #openstack-cinder | 11:19 | |
*** vlaza has left #openstack-cinder | 11:20 | |
*** boris-42 has quit IRC | 11:22 | |
openstackgerrit | Marc Koderer proposed openstack/cinder: Move CloneableVD to common functions https://review.openstack.org/203593 | 11:29 |
mkoderer | DuncanT: any idea who's using/testing BlockDeviceDriver? If we would remove it we can merge 4 VD subclasses to BaseVD | 11:33 |
e0ne | mkoderer: hi. BlockDeviceDriver is used by sahara team | 11:34 |
mkoderer | e0ne: ah ok.. good to know :) | 11:34 |
e0ne | mkoderer: we don't want to remove it. i'm ready to fix/extend it if needed | 11:34 |
mkoderer | e0ne: cool.. Let's wait what will be the feedback about my propsal on the ML. If we move on with it we should definitly extend it to make more functions common for all drivers | 11:36 |
mkoderer | jgriffith: ping | 11:36 |
e0ne | mkoderer: ok. please, let me know if any help with this driver is needed | 11:37 |
mkoderer | e0ne: sure thx :) | 11:37 |
e0ne | mkoderer: :) | 11:37 |
e0ne | mkoderer: fyi, we're still working on ci for it | 11:38 |
*** liuxg has joined #openstack-cinder | 11:39 | |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add smartX support for Huawei volume driver https://review.openstack.org/188732 | 11:40 |
*** piyanai has quit IRC | 11:43 | |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 11:44 |
*** piyanai has joined #openstack-cinder | 11:44 | |
*** geguileo has quit IRC | 11:47 | |
*** Zhongjun has quit IRC | 11:48 | |
liuxg | e0ne: can you have a look at https://review.openstack.org/#/c/188240/ | 11:50 |
liuxg | e0ne: when you have time :) | 11:50 |
*** ganso_ has joined #openstack-cinder | 11:51 | |
*** vlaza has joined #openstack-cinder | 11:55 | |
*** julim has joined #openstack-cinder | 11:56 | |
*** sgotliv has quit IRC | 11:57 | |
*** bluex-pl has quit IRC | 11:57 | |
*** bluex-pl has joined #openstack-cinder | 11:58 | |
*** aix has joined #openstack-cinder | 12:00 | |
*** piyanai has quit IRC | 12:04 | |
openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Adding delete wait loop for CloudByte ISCSI volumes. https://review.openstack.org/201059 | 12:05 |
*** ig0r_ has quit IRC | 12:05 | |
*** changbl has quit IRC | 12:09 | |
*** delatte has joined #openstack-cinder | 12:14 | |
*** Liu has joined #openstack-cinder | 12:15 | |
*** BharatK has quit IRC | 12:15 | |
*** cdelatte has quit IRC | 12:15 | |
*** liuxg has quit IRC | 12:16 | |
*** markvoelker has joined #openstack-cinder | 12:16 | |
*** delattec has quit IRC | 12:17 | |
*** bswartz has quit IRC | 12:19 | |
mkoderer | tbarron: ups.. grep didn't work properly :) thx for the review | 12:19 |
mkoderer | tbarron: for netapp it's used in drivers/netapp/eseries/fc_driver.py and drivers/netapp/eseries/iscsi_driver.py correct? | 12:20 |
*** timcl has joined #openstack-cinder | 12:23 | |
*** zhenguo has quit IRC | 12:23 | |
*** laughterwym has joined #openstack-cinder | 12:24 | |
*** pradipta has quit IRC | 12:28 | |
*** edmondsw has joined #openstack-cinder | 12:29 | |
*** BharatK has joined #openstack-cinder | 12:31 | |
*** kkeuning has joined #openstack-cinder | 12:33 | |
*** timcl has joined #openstack-cinder | 12:33 | |
*** kjnelson has joined #openstack-cinder | 12:34 | |
*** tobe has quit IRC | 12:34 | |
*** chlong has joined #openstack-cinder | 12:35 | |
*** kkeuning has quit IRC | 12:35 | |
*** jungleboyj has quit IRC | 12:36 | |
*** kkeuning has joined #openstack-cinder | 12:36 | |
*** alonmarx has quit IRC | 12:36 | |
*** kkeuning has quit IRC | 12:37 | |
*** pawels has joined #openstack-cinder | 12:37 | |
*** kkeuning has joined #openstack-cinder | 12:38 | |
openstackgerrit | Edwin Wang proposed openstack/cinder: Add I/T mapping check in terminate_connection https://review.openstack.org/203616 | 12:38 |
*** sgotliv has joined #openstack-cinder | 12:38 | |
*** ronis has quit IRC | 12:38 | |
*** kkeuning has quit IRC | 12:39 | |
*** kkeuning has joined #openstack-cinder | 12:39 | |
*** edmondsw has quit IRC | 12:40 | |
*** mdenny has joined #openstack-cinder | 12:40 | |
*** Zhongjun has joined #openstack-cinder | 12:41 | |
*** bluex-pl has quit IRC | 12:45 | |
*** bluex-pl has joined #openstack-cinder | 12:45 | |
*** rushil has joined #openstack-cinder | 12:48 | |
*** agarciam has quit IRC | 12:49 | |
*** ronis has joined #openstack-cinder | 12:51 | |
*** bswartz has joined #openstack-cinder | 12:53 | |
*** erlon has joined #openstack-cinder | 12:53 | |
*** alonmarx has joined #openstack-cinder | 12:56 | |
*** ronis has quit IRC | 12:59 | |
*** eharney has joined #openstack-cinder | 12:59 | |
*** dguryanov|2 has quit IRC | 12:59 | |
*** Adriano__ has joined #openstack-cinder | 13:02 | |
*** thangp has joined #openstack-cinder | 13:02 | |
*** dustins has joined #openstack-cinder | 13:02 | |
*** agarciam has joined #openstack-cinder | 13:07 | |
*** changbl has joined #openstack-cinder | 13:09 | |
*** aagrawal has joined #openstack-cinder | 13:10 | |
*** agarciam has quit IRC | 13:11 | |
*** agarciam has joined #openstack-cinder | 13:12 | |
*** aagrawal has quit IRC | 13:17 | |
*** edmondsw has joined #openstack-cinder | 13:18 | |
*** primechuck has joined #openstack-cinder | 13:21 | |
openstackgerrit | Edwin Wang proposed openstack/cinder: Add I/T mapping check in terminate_connection https://review.openstack.org/203616 | 13:22 |
*** akshai has joined #openstack-cinder | 13:24 | |
*** akerr has joined #openstack-cinder | 13:27 | |
*** boris-42 has joined #openstack-cinder | 13:28 | |
*** annegentl_ has joined #openstack-cinder | 13:30 | |
*** BharatK has quit IRC | 13:34 | |
*** dims_ has quit IRC | 13:35 | |
*** gouthamr has joined #openstack-cinder | 13:35 | |
*** pawels has quit IRC | 13:36 | |
*** Yogi1 has joined #openstack-cinder | 13:37 | |
*** pawels has joined #openstack-cinder | 13:37 | |
*** agarciam1 has joined #openstack-cinder | 13:38 | |
*** agarciam1 has quit IRC | 13:38 | |
*** agarciam1 has joined #openstack-cinder | 13:38 | |
*** vincent_hou_ has joined #openstack-cinder | 13:38 | |
*** xyang1 has joined #openstack-cinder | 13:40 | |
*** piyanai has joined #openstack-cinder | 13:41 | |
*** agarciam has quit IRC | 13:41 | |
*** jdandrea has joined #openstack-cinder | 13:42 | |
*** krtaylor has quit IRC | 13:46 | |
*** annegentl_ has quit IRC | 13:48 | |
openstackgerrit | Ph. Marek proposed openstack/cinder: Rename a constant to a better name, uppercase-only. https://review.openstack.org/203646 | 13:48 |
*** agarciam1 has quit IRC | 13:49 | |
flip214 | uhh.... is there some good reason why the CI for DRBD wouldn't be run in the last few days? http://graphite.openstack.org/render/?width=600&height=344&_salt=1434709688.361&from=-6days&xFormat=%25b%25d-%25T&title=DRBD%20Cinder%2FDevstack%20stats&colorList=red%2Cgreen%2Cblue&target=stats_counts.zuul.pipeline.check.job.check-tempest-dsvm-full-drbd-devstack-nv.FAILURE&target=stats_counts.zuul.pipeline.check.job.check-tempest-dsvm-full-drbd-devsta | 13:49 |
*** vlaza has quit IRC | 13:56 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Filter cgsnapshots data on the DB side https://review.openstack.org/197163 | 13:57 |
*** agarciam has joined #openstack-cinder | 13:58 | |
*** krtaylor has joined #openstack-cinder | 13:59 | |
*** mriedem has joined #openstack-cinder | 14:00 | |
openstackgerrit | Ph. Marek proposed openstack/cinder: Define a separate prefix for snapshots https://review.openstack.org/203657 | 14:03 |
*** abehl has quit IRC | 14:03 | |
*** breitz has joined #openstack-cinder | 14:05 | |
tbarron | mkoderer: yes | 14:06 |
tbarron | mkoderer: http://paste.openstack.org/show/391762/ | 14:06 |
mkoderer | tbarron: yep yep on it :) | 14:07 |
tbarron | mkoderer: np. I think the simplification is a good idea. Thanks. | 14:07 |
*** kkeuning has quit IRC | 14:07 | |
mkoderer | tbarron: if you have a chance to have a look to the goolge spreadsheet. I hope my script wasn't buggy | 14:11 |
*** annegentl_ has joined #openstack-cinder | 14:11 | |
mkoderer | https://docs.google.com/spreadsheets/d/1L_GuUCs-NMVbhbOj8Jtt8vjMQ23zhJ1yagSH4zSKWEw/edit?usp=sharing | 14:11 |
mkoderer | tbarron: just from the netapp side.. not sure if one of the drivers in the list are acually abstract ones | 14:11 |
tbarron | ok, looking | 14:11 |
openstackgerrit | Eric Harney proposed openstack/cinder: NFS snapshots https://review.openstack.org/147186 | 14:13 |
*** bnemec has joined #openstack-cinder | 14:15 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder-specs: Checking the existence of volume https://review.openstack.org/174823 | 14:17 |
*** haomaiwang has quit IRC | 14:21 | |
*** yuriy_n17 has joined #openstack-cinder | 14:21 | |
*** pawels has quit IRC | 14:21 | |
*** haomaiwang has joined #openstack-cinder | 14:21 | |
*** mylu has joined #openstack-cinder | 14:24 | |
*** dims_ has joined #openstack-cinder | 14:25 | |
*** dims_ has quit IRC | 14:25 | |
*** dims_ has joined #openstack-cinder | 14:26 | |
*** alonmarx_ has joined #openstack-cinder | 14:35 | |
*** thingee_ has joined #openstack-cinder | 14:36 | |
*** rex_lee_ has joined #openstack-cinder | 14:38 | |
*** ctracey_ has joined #openstack-cinder | 14:38 | |
*** jordan_ has joined #openstack-cinder | 14:38 | |
*** sweston_ has joined #openstack-cinder | 14:39 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Add more Rally scenarios to run https://review.openstack.org/194180 | 14:39 |
*** liewegas_ has joined #openstack-cinder | 14:39 | |
*** kfox1111_ has joined #openstack-cinder | 14:39 | |
*** uberjay_ has joined #openstack-cinder | 14:39 | |
*** winston-1 has joined #openstack-cinder | 14:39 | |
*** sdague_ has joined #openstack-cinder | 14:40 | |
*** ronis has joined #openstack-cinder | 14:40 | |
openstackgerrit | Richard Hedlind proposed openstack/cinder: Revert "Remove X-IO volume driver" https://review.openstack.org/203681 | 14:41 |
*** powerbsd has joined #openstack-cinder | 14:41 | |
*** albertom has quit IRC | 14:42 | |
*** ctracey has quit IRC | 14:42 | |
*** powerbsd is now known as albertom | 14:42 | |
*** swamireddy_ has quit IRC | 14:42 | |
*** mriedem has quit IRC | 14:42 | |
*** rex_lee has quit IRC | 14:42 | |
*** thingee has quit IRC | 14:42 | |
*** sweston has quit IRC | 14:42 | |
*** tbarron has quit IRC | 14:42 | |
*** uberjay has quit IRC | 14:42 | |
*** sdague has quit IRC | 14:42 | |
*** briancline has quit IRC | 14:42 | |
*** alonmarx has quit IRC | 14:42 | |
*** jordanP has quit IRC | 14:42 | |
*** kambiz has quit IRC | 14:42 | |
*** winston-d has quit IRC | 14:42 | |
*** jamielennox has quit IRC | 14:42 | |
*** kfox1111 has quit IRC | 14:42 | |
*** krtaylor has quit IRC | 14:42 | |
*** IanGovett has quit IRC | 14:42 | |
*** liewegas has quit IRC | 14:42 | |
*** rlucio_ has quit IRC | 14:42 | |
*** Swanson has quit IRC | 14:42 | |
*** tbarron has joined #openstack-cinder | 14:42 | |
*** briancline has joined #openstack-cinder | 14:42 | |
*** kambiz has joined #openstack-cinder | 14:42 | |
*** krtaylor has joined #openstack-cinder | 14:42 | |
*** sdague_ is now known as sdague | 14:42 | |
*** Swanson has joined #openstack-cinder | 14:42 | |
*** rex_lee_ is now known as rex_lee | 14:42 | |
*** jamielennox has joined #openstack-cinder | 14:42 | |
*** cbader has joined #openstack-cinder | 14:43 | |
*** swamireddy has joined #openstack-cinder | 14:43 | |
*** sweston_ is now known as sweston | 14:43 | |
*** thingee_ is now known as thingee | 14:43 | |
*** ctracey_ is now known as ctracey | 14:43 | |
*** agarciam has quit IRC | 14:43 | |
*** IanGovett has joined #openstack-cinder | 14:44 | |
*** rlucio has joined #openstack-cinder | 14:46 | |
*** mriedem has joined #openstack-cinder | 14:46 | |
*** garthb has joined #openstack-cinder | 14:49 | |
*** tsekiyama has joined #openstack-cinder | 14:50 | |
*** jdurgin1 has joined #openstack-cinder | 14:52 | |
*** mriedem has quit IRC | 14:57 | |
jgriffith | mkoderer: what's up? | 14:59 |
*** nkrinner has quit IRC | 14:59 | |
*** vincent_hou_ has quit IRC | 15:02 | |
*** shausy has joined #openstack-cinder | 15:04 | |
*** chhavi has joined #openstack-cinder | 15:04 | |
*** rakesh_mishra has quit IRC | 15:05 | |
mkoderer | jgriffith: damn, just find a bug in my spredsheet tool... anyway.. what do you think about a stepwise aproach instead of moving everything back to VolumeDriver? | 15:06 |
jgriffith | mkoderer: not sure, what do you have in mind? | 15:06 |
jgriffith | mkoderer: also not sure anybody is really on the same page as me with this whole thing anyway :) | 15:07 |
jgriffith | mkoderer: I've received some "hate" mail :) | 15:07 |
mkoderer | jgriffith: ok let's stop with all this emotional stuff and find a good solution :) | 15:08 |
*** diemt has joined #openstack-cinder | 15:08 | |
mkoderer | jgriffith: so I think we have functions that are already implemented nearly everywhere | 15:08 |
thingee | the proposal seems fine to me. lets move on | 15:08 |
jgriffith | mkoderer: ok... sorry | 15:09 |
jgriffith | mkoderer: still not following... what do you have in mind more specifically? | 15:10 |
mkoderer | jgriffith: for instance CloneableImageVD | 15:10 |
mkoderer | jgriffith: clone_image is implemented everywhere (at least I didn't find any driver that didn't implement it) | 15:11 |
mkoderer | so we move it to BaseVS | 15:11 |
mkoderer | BaseVD | 15:11 |
*** kjnelson has quit IRC | 15:11 | |
thingee | I can't for the life of me figure out why unit tests are failing because of open not being called anymore https://github.com/openstack/cinder/blob/master/cinder/tests/unit/targets/test_tgt_driver.py#L214 | 15:12 |
jgriffith | mkoderer: so you mean a combination of sorts between what's in place currently and the one-class approach? So like one subclass with all the requireds in it? | 15:12 |
thingee | this seems right according to http://www.voidspace.org.uk/python/mock/helpers.html#mock.mock_open | 15:12 |
*** shausy has quit IRC | 15:12 | |
jgriffith | thingee: mox/mock library update is proving painful :( | 15:12 |
mkoderer | jgriffith: yeah, instead of "removing" it nearly completely we do it stepwise | 15:12 |
jgriffith | thingee: I looked briefly on Saturday but couldn't find a clear idea of what was going on | 15:13 |
*** shausy has joined #openstack-cinder | 15:13 | |
*** jistr is now known as jistr|mtg | 15:13 | |
jgriffith | mkoderer: I'm certainly open to any compromise. Still not completely sure I follow what you have in mind. | 15:13 |
mkoderer | jgriffith: e.g. https://review.openstack.org/#/c/203593/ | 15:13 |
jgriffith | mkoderer: one thing I did mention in the comments somewhere was maybe being a bit more proper with the structure to begin with (a true abstract class) | 15:14 |
* jgriffith is looking | 15:14 | |
*** dannywilson has joined #openstack-cinder | 15:14 | |
mkoderer | jgriffith: it's acutally just deleting CloneableVD and moving it to BaseVD.. which makes it madatory for all drivers | 15:15 |
jgriffith | mkoderer: oh, sure.. but I'm not sure how that's different than what I proposed? | 15:15 |
jgriffith | mkoderer: removal of the method-specific classes and putting the methods in the base class? | 15:16 |
jgriffith | mkoderer: so are you thinking a structure like: Driver-->BaseVD where all the required/abc decorated methods are in the BaseVD class? | 15:16 |
mkoderer | jgriffith: mh, you moved all to BaseVD.. but there are acutal feautre sets that don't belong to Base | 15:17 |
jgriffith | mkoderer: well, I got rid of BaseVD altogether :) but yeah... I moved everything to VolumeDriver which is the base; same thing i guess | 15:17 |
jgriffith | mkoderer: but I'm confused on the part about "they don't belong there"? | 15:18 |
mkoderer | jgriffith: I have in mind to have a large BaseVD but also some "optionals" like ConsistencyGroupVD which is not mandatory for all drivers | 15:18 |
*** Zhongjun has quit IRC | 15:18 | |
thingee | mkoderer: I think jgriffith is wondering why can't we just have them part of the base, but don't decorate them | 15:19 |
thingee | jgriffith: back to mock_open http://paste.openstack.org/show/391921/ | 15:19 |
thingee | jgriffith: even the example from documentation fails | 15:19 |
thingee | jgriffith: I'll check to see if there's not already an open bug | 15:19 |
jgriffith | thingee: Yeah, I think that's where I'm confused but not entirely sure I see the difference in what mkoderer is proposing vs what we already have | 15:19 |
*** jwcroppe_ is now known as jwcroppe | 15:19 | |
jgriffith | thingee: WTF... that's weird | 15:20 |
*** dims_ has quit IRC | 15:20 | |
jgriffith | thingee: so I was unable to repro locally.. maybe I need to try again with a fresh env | 15:20 |
*** agarciam has joined #openstack-cinder | 15:20 | |
thingee | jgriffith: you definitely need a fresh env | 15:20 |
jgriffith | thingee: I guess that should have been a given to make sure I pull the lib updates :( | 15:21 |
dhellmann | jgriffith, thingee : I was going to run the os-brick release, if the timing is good for you | 15:21 |
thingee | dhellmann: that's fine, but I will be MIA until possibly later this evening | 15:22 |
jgriffith | dhellmann: It's probably ok... but it might be worth us figuring out what the deep issue we currently have is all about before having another variable in the mix | 15:22 |
Swanson | We're changing this back to VolumeDriver immediately after I moved evertything to the assorted VDs?!?!? | 15:22 |
dhellmann | thingee: should I wait, or is there someone else from the team who can address issues? | 15:22 |
dhellmann | jgriffith: ok, if you're working another issue I can definitely wait | 15:22 |
jgriffith | Swanson: we're discussing it | 15:23 |
*** mriedem has joined #openstack-cinder | 15:23 | |
thingee | jgriffith: deep issue? | 15:23 |
jgriffith | thingee: so one other thing... some of those patches over the week-end I rebased and they were "ok" | 15:23 |
jgriffith | thingee: and for example xyang1 's patch this morning was "ok" | 15:23 |
jgriffith | and Gaurangs | 15:24 |
jgriffith | etc | 15:24 |
xyang1 | jgriffith: Should it fail too? | 15:24 |
* thingee is confused | 15:24 | |
*** harlowja_at_home has joined #openstack-cinder | 15:24 | |
*** jistr|mtg is now known as jistr | 15:24 | |
*** nikeshm has joined #openstack-cinder | 15:24 | |
thingee | jgriffith: hold up for a sec. What issue are you talking about with brick? | 15:24 |
jgriffith | xyang1: no, makes me think funky rebase issues | 15:24 |
jgriffith | or something | 15:24 |
jgriffith | thingee: No, no! No issues with Brick | 15:24 |
thingee | dhellmann: ok, what jgriffith said was unrelated to brick :) | 15:25 |
mkoderer | jgriffith: thingee: ok... let me push a spec review about it... and jgriffith reviews the hell out of it? | 15:25 |
thingee | whew | 15:25 |
jgriffith | thingee: I was just saying until we fully understand what's up with the 27/34 tests right now might not want to introduce another change? | 15:25 |
xyang1 | jgriffith: The concurrency flag exposed some problems in our unit tests | 15:25 |
thingee | jgriffith: I see | 15:25 |
dhellmann | thingee, jgriffith : ok, so should I go ahead? :-) | 15:25 |
jgriffith | dhellmann: I'll defer to thingee :) | 15:26 |
jgriffith | dhellmann: i think it's fine and won't hurt | 15:26 |
jgriffith | famous last words | 15:26 |
thingee | dhellmann: it'll be fine | 15:26 |
dhellmann | ok, tagging... | 15:27 |
jgriffith | thingee: so I looked at a couple of these Friday | 15:27 |
jgriffith | thingee: rebase didn't seem to help... but what I didn't understand was why I couldn't reproduce locally | 15:28 |
jgriffith | thingee: I'm trying again with: https://review.openstack.org/#/c/189614/ | 15:28 |
jgriffith | thingee: on Friday I was convinced that any of these that were uploaded back in June just needed a rebase... but that didn't seem to work on the few that I tried | 15:29 |
*** hemnafk is now known as hemna | 15:30 | |
thingee | dhellmann: are you familiar with this? http://paste.openstack.org/show/391921/ | 15:30 |
thingee | dhellmann: at this point, mock's own examples fail | 15:30 |
dhellmann | thingee: :-( did you point it out to lifeless? | 15:30 |
dhellmann | thingee: os-brick 0.3.0 released | 15:30 |
*** mtanino has joined #openstack-cinder | 15:31 | |
thingee | dhellmann: just pinged him on #openstack-dev about it, and thanks | 15:31 |
dhellmann | thingee: have you tried mock 1.2.0? that's the latest release | 15:31 |
jgriffith | thingee: haha... ok, the one I "picked" to look at is actually failing because of the changes made in the patch :( | 15:31 |
thingee | jgriffith: mind getting me up to speed on this rebase issue? | 15:32 |
hemna | mornin | 15:32 |
jgriffith | thingee: so rebase is fixing the bulk of them | 15:32 |
jgriffith | thingee: there were two problems last week | 15:32 |
openstackgerrit | Andrey Pavlov proposed openstack/cinder: Fix doc string definitions https://review.openstack.org/193124 | 15:32 |
jgriffith | thingee: 1. Ceph gate was borked | 15:32 |
jgriffith | 2. mock/mox lib updates | 15:32 |
*** kjnelson has joined #openstack-cinder | 15:32 | |
mriedem | thingee: want to drop the -1 on this now? https://review.openstack.org/#/c/203193/ | 15:33 |
jgriffith | The Ceph/Glance issues have been fixed up, and the pinning/lib updates for mox and mock both finally propagated through via global-requirements update | 15:33 |
*** piyanai has quit IRC | 15:33 | |
jgriffith | thingee: I did a number of rechecks on stuff that failed the ceph tests in the gate.. that all worked out fine | 15:34 |
hemna | crud | 15:34 |
hemna | I was hoping that I could have gotten https://review.openstack.org/#/c/202826/ into 0.3.0 | 15:34 |
jgriffith | thingee: I'm looking now and it would appear that recent patches are running "ok" | 15:34 |
*** dims_ has joined #openstack-cinder | 15:34 | |
Swanson | jgriffith: The one thing I like about the assorted VDs is that they kind of tell you what you need to implement for a feature. So whatever you do keeping that would be marvy. | 15:35 |
*** marzif has joined #openstack-cinder | 15:35 | |
thingee | mriedem: still need to update the upper constraints | 15:35 |
openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Fix Consistency Group issues https://review.openstack.org/201685 | 15:35 |
mriedem | thingee: oh right, that's still new to me | 15:35 |
*** yuriy_n17 has quit IRC | 15:35 | |
openstackgerrit | Andrey Pavlov proposed openstack/cinder: Fix doc string definitions https://review.openstack.org/193124 | 15:36 |
mriedem | hemna: so, can we move forward with your os-brick integration change in nova w/o that FC re-discover devices change and then do that later? | 15:37 |
jgriffith | thingee: the only one that I can't quite figure out is: http://logs.openstack.org/52/198952/6/check/gate-cinder-python27/03e151b/console.html.gz#_2015-07-17_09_45_59_997 | 15:37 |
mriedem | hemna: it's a bug fix but it doesn't seem like it needs to hold up the integratoin | 15:37 |
*** piyanai has joined #openstack-cinder | 15:37 | |
*** dims__ has joined #openstack-cinder | 15:38 | |
*** harlowja_at_home has quit IRC | 15:38 | |
mriedem | jgriffith: is there a mock_open in that test_create_iscsi_target_content test? | 15:39 |
*** amick has joined #openstack-cinder | 15:40 | |
thingee | mriedem: yes | 15:40 |
*** dims__ has quit IRC | 15:40 | |
*** dims_ has quit IRC | 15:40 | |
thingee | mriedem: https://github.com/openstack/cinder/blob/master/cinder/tests/unit/targets/test_tgt_driver.py#L197 | 15:40 |
mriedem | we've been seeing lots of weirdness with mock_open since mock 1.1 | 15:40 |
thingee | mriedem: although I verified mock's own doc example fails http://paste.openstack.org/show/391921/ | 15:40 |
hemna | mriedem, I think FC detaches might complain w/o that patch | 15:40 |
*** dims_ has joined #openstack-cinder | 15:41 | |
hemna | because of the path not existing in the connection_info (from the BDM) during live migration | 15:41 |
mriedem | yeah, some things changed with mock_open in 1.1 and we had to workaround those, then our workarounds broke in mock 1.1.4, and the change in mock was reverted on friday for mock 1.2 | 15:41 |
mriedem | our workarounds for mock 1.1 https://review.openstack.org/#/c/200500/ | 15:41 |
thingee | mriedem: I'm about to test 1.2.0, but I'm assuming I could search nova's unit tests for example work around with mock_open? | 15:42 |
thingee | mriedem: oh thanks | 15:42 |
mriedem | we had to skip one test using mock_open | 15:42 |
hemna | so thingee how much trouble was it to push os-brick 0.3.0 ? | 15:42 |
mriedem | lifeless will be at the nova meetup this week so we might be able to sort some of this out with him then | 15:42 |
*** kmartin has joined #openstack-cinder | 15:43 | |
thingee | hemna: there wasn't much trouble, but there was a recent change in process of how to request it with the release team | 15:43 |
thingee | hemna: so delayed things | 15:43 |
mriedem | hemna: https://review.openstack.org/#/c/203194/ | 15:43 |
hemna | ok, so the release team has to do the work ? | 15:43 |
mriedem | ^trivial | 15:43 |
thingee | by the time I made that switch to the new process, it was friday so we had to delay | 15:43 |
thingee | hemna: they do the tags, but you have to request it via repo now | 15:44 |
mriedem | hemna: if you could get https://review.openstack.org/#/c/202826/ in then you could release 0.3.1 | 15:44 |
thingee | hemna: see http://lists.openstack.org/pipermail/openstack-dev/2015-July/069603.html | 15:44 |
hemna | ok, so if I want a 0.3.1 released, then I push up another patch just like that one? | 15:44 |
hemna | mriedem, yah that's what I'm thinking. | 15:44 |
hemna | thingee, thanks | 15:44 |
*** dims_ has quit IRC | 15:45 | |
hemna | ah that's very cool. | 15:45 |
hemna | (re: mailing list) | 15:45 |
jgriffith | thingee: so back to the unit tests.... | 15:47 |
*** harlowja_at_home has joined #openstack-cinder | 15:47 | |
thingee | jgriffith: thanks for the update of getting me up to speed. I think I was somewhat aware of these issues | 15:48 |
thingee | jgriffith: I'm trying 1.2.0 right now per dhellmann | 15:48 |
jgriffith | thingee: ok... cool! | 15:49 |
*** agarciam has quit IRC | 15:49 | |
*** agarciam has joined #openstack-cinder | 15:49 | |
thingee | jgriffith, dhellmann: it works! | 15:49 |
thingee | 1.2.0 | 15:49 |
jgriffith | sweet | 15:49 |
thingee | mriedem: ^ | 15:50 |
j_king | Trying to add a simple URL verification step to fix https://bugs.launchpad.net/cinder/+bug/1287134 but finding that the NFS driver doesn't actually use NFS urls and relies on some protocol defined by brick? is that about right? | 15:51 |
openstack | Launchpad bug 1287134 in Cinder "C-vol crashes when driver class is changed after some volumes exist on that backend" [Low,In progress] - Assigned to j_king (james-agentultra) | 15:51 |
mriedem | nice, yeah, mock 1.1.4 totally regressed our use of mock_open | 15:51 |
*** mgarza_ has joined #openstack-cinder | 15:51 | |
j_king | wondering how to properly check the provider_location value | 15:52 |
*** mylu has quit IRC | 15:53 | |
*** mylu has joined #openstack-cinder | 15:54 | |
j_king | is there someone in charge of maintaining the NFS driver that I could poke? | 15:55 |
*** jungleboyj has joined #openstack-cinder | 15:55 | |
*** amick is now known as help | 15:56 | |
*** help is now known as Guest56954 | 15:56 | |
eharney | j_king: the NFS provider_location format is defined by the driver | 15:57 |
*** afazekas_ has quit IRC | 15:57 | |
*** Guest56954 has left #openstack-cinder | 15:57 | |
*** laughterwym has quit IRC | 15:57 | |
j_king | eharney: i got the sense of that. | 15:57 |
j_king | eharney: ty | 15:57 |
eharney | j_king: i can maybe help, not sure what you want to do exactly | 15:57 |
smcginnis | thingee, jgriffith: Would love your thoughts on this: https://review.openstack.org/#/c/203158/ | 15:58 |
j_king | eharney: well the TB in that ticket points to a good location to check that the provider_location of a volume actually came from an NFS driver. | 15:58 |
smcginnis | thingee, jgriffith: Both having it in tree, then whether to run it. ;) | 15:59 |
j_king | but maybe there's a more suitable place in base class if this is a formal protocol defined by each implementation | 15:59 |
j_king | i was just going to fix the NFS driver case. :) | 15:59 |
eharney | j_king: i don't think it's format really, but NFS (and other remotefs drivers) do the same thing | 15:59 |
eharney | formal really* | 16:00 |
eharney | bbiab | 16:00 |
j_king | basically if you create an iSCSCI LVM volume and switch to the NFS driver it will try and mount it b/c it doesn't check the provider_location | 16:00 |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Efficient volume copy for generic volume migration https://review.openstack.org/183701 | 16:00 |
asselin_ | nikeshm, I think your node was deleted during the test | 16:01 |
*** amickus has joined #openstack-cinder | 16:01 | |
asselin_ | nikeshm, but I have to go now....will be back later | 16:01 |
nikeshm | thingee: i moved all CIs on cinder patches after testing on sandbox | 16:01 |
nikeshm | asselin_ sure thanks | 16:01 |
nikeshm | smcginnis: hey how was weekend | 16:03 |
smcginnis | nikeshm: Hey. Not too bad. Storm knocked out power Friday night and I didn't get it back until about 12 hours ago, but it wasn't so bad. | 16:05 |
*** leeantho has joined #openstack-cinder | 16:05 | |
nikeshm | smcginnis: ohh, i am on a vacation for 1 week, working from home | 16:06 |
smcginnis | nikeshm: Nice! | 16:06 |
smcginnis | nikeshm: Except for the working on vacation bit. :) | 16:07 |
Swanson | smcginnis: Ian spent Friday's storms in a tent out at Eaux Claires. Fun story if you run across him. | 16:07 |
mdbooth | Quick question: how does cinder feel if you stand up 2 api services connected to the same db? | 16:07 |
smcginnis | Swanson: Hah, that's right. | 16:07 |
thingee | dhellmann, jgriffith: so yeah 1.2.0 fixes it. I can no longer reproduce the issue in a fresh venv with cinder | 16:07 |
mdbooth | Is it happy with that? | 16:07 |
smcginnis | bbl | 16:07 |
nikeshm | smcginnis: yes have to monitor CIs, can not make thingee unhappy :) | 16:08 |
nikeshm | smcginnis: after testing on sandbox, i have moved them to cinder patches, will be in touch with infra to improve it further | 16:09 |
*** mylu has quit IRC | 16:11 | |
thingee | hemna: do we need another release? | 16:13 |
thingee | of brick | 16:13 |
hemna | thingee, yah | 16:13 |
hemna | thingee, https://review.openstack.org/#/c/202826/ | 16:13 |
hemna | that guy | 16:13 |
hemna | I was testing it quite a bit on friday and needed that guy to land, but I was too late to get it to land this morning. | 16:13 |
thingee | hemna: Ok, I'll wait for that patch, and propose something later today | 16:14 |
hemna | thingee, https://review.openstack.org/#/c/201643/ that guy looks like something to make the releases easier? pbr building the changelog automagically ? | 16:14 |
thingee | hemna: I need to test that, or have someone verify it | 16:15 |
hemna | thingee, yah I'm not sure how to verify that one | 16:15 |
hemna | I'm guessing dhellmann knows how to. | 16:15 |
dhellmann | thingee, hemna : see the built docs from the job (i.e., http://docs-draft.openstack.org/43/201643/2/check/gate-os-brick-docs/e71b395//doc/build/html/changelog.html ) | 16:17 |
*** jdurgin1 has quit IRC | 16:17 | |
dhellmann | it's wrong right now, but if you recheck it it should pick up the 0.3.0 tag | 16:18 |
hemna | dhellmann, ok, thanks, I'll recheck that patch now. | 16:18 |
*** e0ne has quit IRC | 16:18 | |
*** _cjones_ has joined #openstack-cinder | 16:22 | |
*** julim has quit IRC | 16:22 | |
*** mylu has joined #openstack-cinder | 16:22 | |
*** jwcroppe has quit IRC | 16:23 | |
*** Apoorva has joined #openstack-cinder | 16:23 | |
*** jwcroppe has joined #openstack-cinder | 16:23 | |
openstackgerrit | Rajini Ram proposed openstack/cinder: Eqlx Driver: Revised the ssh code to use common utils https://review.openstack.org/203720 | 16:24 |
*** jistr has quit IRC | 16:24 | |
*** salv-orlando has joined #openstack-cinder | 16:26 | |
thingee | smcginnis, jungleboyj in case you missed it, unit tests failing yesterday because of mock issues are resolved with the recent 1.2.0 release | 16:27 |
*** sgotliv has quit IRC | 16:27 | |
*** mylu has quit IRC | 16:28 | |
*** jwcroppe has quit IRC | 16:28 | |
jungleboyj | thingee: Thanks, I saw they started working. Now seeing another failure that I am starting to investigate. | 16:28 |
*** evilrob has joined #openstack-cinder | 16:30 | |
*** jordan_ has quit IRC | 16:32 | |
*** annashen has joined #openstack-cinder | 16:33 | |
*** shausy has quit IRC | 16:34 | |
*** mylu has joined #openstack-cinder | 16:35 | |
thingee | jungleboyj: just ran it, didn't hit any failures | 16:36 |
*** Yogi1 has quit IRC | 16:37 | |
*** vokt has joined #openstack-cinder | 16:38 | |
*** vokt has quit IRC | 16:38 | |
*** vokt has joined #openstack-cinder | 16:38 | |
*** vokt has quit IRC | 16:39 | |
*** vokt has joined #openstack-cinder | 16:39 | |
*** annegentl_ has quit IRC | 16:40 | |
jungleboyj | Yeah, same here. Not sure why our automatic job failed. *scratching head* | 16:40 |
*** jwcroppe has joined #openstack-cinder | 16:43 | |
*** mriedem is now known as mriedem_away | 16:43 | |
smcginnis | thingee: Thanks! | 16:46 |
*** diogogmt has joined #openstack-cinder | 16:47 | |
*** sghanekar_ has joined #openstack-cinder | 16:49 | |
*** harlowja_at_home has quit IRC | 16:50 | |
*** daneyon has joined #openstack-cinder | 16:50 | |
*** asselin has joined #openstack-cinder | 16:51 | |
*** annashen has quit IRC | 16:51 | |
scottda | mdbooth: Yes, you can setup multiple cinder-api nodes in an AZ (same DB). I've got 10 cinder-api per AZ. | 16:55 |
*** timcl1 has joined #openstack-cinder | 16:55 | |
*** rhe00_ is now known as rhedlind | 16:56 | |
*** annashen has joined #openstack-cinder | 16:56 | |
*** timcl has quit IRC | 16:58 | |
*** annashen has quit IRC | 17:01 | |
*** timcl has joined #openstack-cinder | 17:04 | |
*** annashen has joined #openstack-cinder | 17:04 | |
*** dims_ has joined #openstack-cinder | 17:05 | |
*** daneyon has quit IRC | 17:06 | |
*** daneyon_ has joined #openstack-cinder | 17:06 | |
openstackgerrit | Rajini Ram proposed openstack/cinder: Eqlx Driver: Revised the ssh code to use common utils https://review.openstack.org/203720 | 17:07 |
*** jordanP has joined #openstack-cinder | 17:12 | |
*** e0ne has joined #openstack-cinder | 17:15 | |
*** openstackgerrit has quit IRC | 17:16 | |
*** openstackgerrit has joined #openstack-cinder | 17:17 | |
*** akerr is now known as akerr_away | 17:19 | |
*** akerr_away has quit IRC | 17:19 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add connector object to create_export https://review.openstack.org/202808 | 17:20 |
*** jwcroppe has quit IRC | 17:21 | |
*** jwcroppe has joined #openstack-cinder | 17:21 | |
hemna | jungleboyj, looks like the IBM XIV CI is instantly failing currently | 17:23 |
hemna | jungleboyj, https://review.openstack.org/#/c/202808/ failed in 1 second after I updated that patch. | 17:23 |
*** akerr has joined #openstack-cinder | 17:23 | |
jungleboyj | hemna: Yeah, I know. I have been talking to them about that. | 17:23 |
* jungleboyj is not happy at the moment. | 17:24 | |
hemna | since that patch touches the XIV driver, I'd like it to report :) | 17:24 |
hemna | ok cool thanks. | 17:24 |
*** vilobhmm has joined #openstack-cinder | 17:24 | |
*** patrickeast has joined #openstack-cinder | 17:24 | |
jungleboyj | Let me send them another note. | 17:24 |
*** piyanai has quit IRC | 17:25 | |
*** sghanekar_ has quit IRC | 17:26 | |
*** jwcroppe has quit IRC | 17:26 | |
*** changbl has quit IRC | 17:33 | |
*** sghanekar_ has joined #openstack-cinder | 17:34 | |
vilobhmm | smcginnis, jungleboyj, jgriffith : If i can request you to have a look at this one whenever you are free https://review.openstack.org/#/c/163641/ so that I don't end up in the rebase cycle again. | 17:35 |
*** earlephilhower has joined #openstack-cinder | 17:39 | |
*** nzrf has joined #openstack-cinder | 17:39 | |
eharney | https://review.openstack.org/#/c/201241/ just needs a final +W | 17:40 |
openstackgerrit | Eric Brown proposed openstack/cinder: Add bandit for security static analysis testing https://review.openstack.org/179568 | 17:43 |
hemna | eharney, I'll test that locally here. running tox -epy27 now | 17:43 |
eharney | hemna: thanks | 17:44 |
*** annegentl_ has joined #openstack-cinder | 17:44 | |
*** aix has quit IRC | 17:46 | |
*** piyanai has joined #openstack-cinder | 17:48 | |
*** IanGovett has quit IRC | 17:52 | |
*** dims_ has quit IRC | 17:53 | |
*** piyanai has quit IRC | 17:55 | |
*** annegentl_ has quit IRC | 17:56 | |
*** dims_ has joined #openstack-cinder | 17:56 | |
*** annegentl_ has joined #openstack-cinder | 17:56 | |
hemna | eharney, looks good. | 17:57 |
thingee | jungleboyj: I've already disabled the CI | 17:58 |
thingee | jungleboyj: Alon five days ago said it was being worked on | 17:58 |
*** dims_ has quit IRC | 17:58 | |
openstackgerrit | Patrick East proposed openstack/cinder: Prevent missing Purity hosts from raising errors https://review.openstack.org/201393 | 17:58 |
thingee | jungleboyj: this is on the third party list | 17:58 |
jungleboyj | thingee: Ok, I thought people were still seeing it voting. | 17:58 |
Swanson | gate-tempest-dsvm-full-ceph: seems to be having issues. | 17:58 |
*** albertom has quit IRC | 17:58 | |
jungleboyj | thingee: Sorry about that. I am not happy with these teams right now. Not getting focus on the CI issues. | 17:59 |
thingee | jungleboyj: let me know if people need motivation. ;) | 17:59 |
patrickeast | you guys talking about the xiv ci? | 17:59 |
thingee | patrickeast: yes | 17:59 |
patrickeast | the one thats been voting failures? http://ec2-54-67-102-119.us-west-1.compute.amazonaws.com:5000/?project=openstack%2Fcinder&user=&timeframe=24&start=&end=&page_size=5000 | 17:59 |
patrickeast | https://review.openstack.org/#/c/201393/4 | 18:00 |
*** piyanai has joined #openstack-cinder | 18:00 | |
hemna | patrickeast, I saw it fail in 1s not long ago | 18:00 |
jungleboyj | thingee: Will do. | 18:00 |
*** pcaruana has quit IRC | 18:00 | |
patrickeast | i was confused when he said it was already disabled | 18:00 |
*** timcl has quit IRC | 18:00 | |
patrickeast | since its definitely still doing stuff | 18:00 |
hemna | patrickeast, it's trying, but instantly failing | 18:00 |
Swanson | "FAILURE in 1s" | 18:01 |
hemna | well, if you are going to fail, might as well fails quick! | 18:01 |
patrickeast | haha | 18:02 |
*** Yogi1 has joined #openstack-cinder | 18:03 | |
thingee | patrickeast, hemna, jungleboyj I ping anteaya on infra about it. I asked for that and Microsoft CI to be disabled | 18:05 |
*** albertom has joined #openstack-cinder | 18:05 | |
thingee | alright disappearing for a bit | 18:09 |
*** primechu_ has joined #openstack-cinder | 18:09 | |
thingee | bbl tonight | 18:09 |
hemna | patrickeast, can you take a look at the Pure CI failures? | 18:09 |
hemna | patrickeast, http://ec2-54-67-51-189.us-west-1.compute.amazonaws.com/43/201643/2/check/PureFCDriver-tempest-dsvm-volume-multipath-brick/e4b4ed4/console.html.gz | 18:10 |
*** primechuck has quit IRC | 18:10 | |
patrickeast | hemna: yea i’ll take a look… i’m trying to figure out why it seems to have stopped working over the weekend | 18:10 |
patrickeast | hemna: nodepool doesn’t seem to be releasing ip addresses :( | 18:10 |
patrickeast | asselin: ^ | 18:10 |
hemna | patrickeast, ok cool. I just want to make sure we aren't missing something and | 18:10 |
*** jasondotstar has quit IRC | 18:10 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Split VolumeTestCase into separate classes https://review.openstack.org/201242 | 18:11 |
openstackgerrit | Rajini Ram proposed openstack/cinder: Eqlx Driver: Revised the ssh code to use common utils https://review.openstack.org/203720 | 18:12 |
*** timcl has joined #openstack-cinder | 18:15 | |
*** timcl has quit IRC | 18:16 | |
*** s0nea has quit IRC | 18:18 | |
*** annashen has quit IRC | 18:18 | |
*** Yogi1 has quit IRC | 18:19 | |
*** lpetrut has joined #openstack-cinder | 18:21 | |
*** s0nea has joined #openstack-cinder | 18:21 | |
*** Liu has quit IRC | 18:23 | |
*** markstur has joined #openstack-cinder | 18:24 | |
*** Liu has joined #openstack-cinder | 18:24 | |
openstackgerrit | Rajini Ram proposed openstack/cinder: Eqlx Driver: Revised the ssh code to use common utils https://review.openstack.org/203720 | 18:25 |
patrickeast | hemna: hmm looks like the cisco zone manager blew up in one of pure’s recent failures http://ec2-54-67-51-189.us-west-1.compute.amazonaws.com/65/203265/1/check/PureFCDriver-tempest-dsvm-volume-multipath-brick/0425ab6/logs/screen-c-vol.txt.gz?level=TRACE | 18:25 |
patrickeast | havent seen that before | 18:25 |
patrickeast | u'Configuration update aborted: another request for config change is already in progress\n' | 18:26 |
*** Yogi1 has joined #openstack-cinder | 18:26 | |
patrickeast | does that mean parallel requests to modify the zones are not allowed? | 18:26 |
*** daneyon has joined #openstack-cinder | 18:26 | |
openstackgerrit | Rajini Ram proposed openstack/cinder: Eqlx Driver: Revised the ssh code to use common utils https://review.openstack.org/203720 | 18:28 |
*** markstur has left #openstack-cinder | 18:28 | |
*** daneyon_ has quit IRC | 18:29 | |
*** timcl has joined #openstack-cinder | 18:30 | |
*** piyanai has quit IRC | 18:30 | |
openstackgerrit | Rajini Ram proposed openstack/cinder: Eqlx Driver: Revised the ssh code to use common utils https://review.openstack.org/203720 | 18:30 |
*** sgotliv has joined #openstack-cinder | 18:31 | |
*** piyanai has joined #openstack-cinder | 18:32 | |
openstackgerrit | Rajini Ram proposed openstack/cinder: Eqlx Driver: Revised the ssh code to use common utils https://review.openstack.org/203720 | 18:32 |
*** piyanai has quit IRC | 18:33 | |
*** mylu has quit IRC | 18:34 | |
*** piyanai has joined #openstack-cinder | 18:34 | |
*** Lee1092 has quit IRC | 18:35 | |
*** piyanai has quit IRC | 18:36 | |
*** angela-s has joined #openstack-cinder | 18:36 | |
*** piyanai has joined #openstack-cinder | 18:37 | |
*** piyanai has quit IRC | 18:37 | |
*** mylu has joined #openstack-cinder | 18:37 | |
*** shyama has quit IRC | 18:39 | |
*** agarciam has quit IRC | 18:40 | |
*** agarciam has joined #openstack-cinder | 18:40 | |
openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Efficient image download and upload for Glance Cinder backend https://review.openstack.org/201754 | 18:40 |
*** piyanai has joined #openstack-cinder | 18:44 | |
*** nikeshm has quit IRC | 18:44 | |
openstackgerrit | Raunak Kumar proposed openstack/cinder: Add extra spec capability for Nimble Cinder Driver https://review.openstack.org/198853 | 18:45 |
*** gman-tx_ has quit IRC | 18:51 | |
jungleboyj | asselin: You there? | 18:52 |
*** gman-tx has joined #openstack-cinder | 18:53 | |
*** jwcroppe has joined #openstack-cinder | 18:53 | |
*** raunak has joined #openstack-cinder | 18:53 | |
hemna | jungleboyj, he's not at his desk at the moment. | 18:53 |
jungleboyj | Ok. Thanks. | 18:54 |
jungleboyj | Is he coming to FC? | 18:54 |
hemna | jungleboyj, no | 18:57 |
*** lpetrut has quit IRC | 18:58 | |
asselin | jungleboyj, hi | 18:59 |
*** DericHorn-HP has joined #openstack-cinder | 19:00 | |
*** garthb_ has joined #openstack-cinder | 19:01 | |
*** garthb has quit IRC | 19:02 | |
*** akerr has quit IRC | 19:02 | |
*** piyanai has quit IRC | 19:02 | |
*** rushiagr is now known as rushiagr_away | 19:03 | |
openstackgerrit | Dmitry Guryanov proposed openstack/os-brick: remotefs: add virtuozzo storage support https://review.openstack.org/188805 | 19:04 |
*** akerr has joined #openstack-cinder | 19:04 | |
*** vilobhmm has quit IRC | 19:04 | |
*** piyanai has joined #openstack-cinder | 19:06 | |
*** chhavi has quit IRC | 19:06 | |
*** dims_ has joined #openstack-cinder | 19:10 | |
openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix concurrent attaches on HNAS iSCSI driver https://review.openstack.org/203780 | 19:10 |
*** DericHorn-HP has quit IRC | 19:15 | |
*** vilobhmm has joined #openstack-cinder | 19:16 | |
*** raunak has quit IRC | 19:17 | |
*** annashen has joined #openstack-cinder | 19:19 | |
*** piyanai has quit IRC | 19:21 | |
*** annashen has quit IRC | 19:24 | |
*** vilobhmm has quit IRC | 19:24 | |
*** piyanai has joined #openstack-cinder | 19:24 | |
*** timcl has quit IRC | 19:27 | |
*** vilobhmm has joined #openstack-cinder | 19:27 | |
jungleboyj | asselin: Hey, so question for you about http://specs.openstack.org/openstack-infra/infra-specs/specs/openstackci.html | 19:29 |
asselin | sure | 19:29 |
jungleboyj | So, what kind of timeline are we looking at for that becoming available? | 19:30 |
vilobhmm | jgriffith, thingee, hemna : this one already has couple of +1 and +2 and needs approval…if you can have a look https://review.openstack.org/#/c/163641/...don;t want to end up in re-base cycle. | 19:31 |
*** changbl has joined #openstack-cinder | 19:31 | |
asselin | jungleboyj, depends on how much help there is to review & get things pushed through | 19:31 |
asselin | jungleboyj, almost there https://storyboard.openstack.org/#!/story/2000101 | 19:32 |
*** piyanai has quit IRC | 19:33 | |
*** timcl has joined #openstack-cinder | 19:34 | |
*** miand has joined #openstack-cinder | 19:35 | |
jungleboyj | So, the XIV guys are wondering if they should wait for that to be available before fixing their issues. | 19:35 |
*** miand has quit IRC | 19:35 | |
asselin | jungleboyj, what issues? | 19:36 |
asselin | jungleboyj, I've been migrating towards that as we go | 19:36 |
jungleboyj | I don't know the details. | 19:36 |
*** DericHorn-HP has joined #openstack-cinder | 19:37 | |
jungleboyj | They were saying they wanted to wait for your approach to land rather than fixing the old code they have. | 19:37 |
Swanson | hemna: create export with connector question. I presme the fczm_utils.AddFCZone decorator moves to create_export from initialize_connection. Nothing belongs on initialize_connection, right? | 19:38 |
openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Efficient image download and upload for Glance Cinder backend https://review.openstack.org/201754 | 19:38 |
*** salv-orlando has quit IRC | 19:38 | |
*** skylerberg has joined #openstack-cinder | 19:42 | |
*** piyanai has joined #openstack-cinder | 19:43 | |
Swanson | hemna: sorry, that decorator can't move. | 19:43 |
Swanson | hemna: and I'm guessing it is probably fine where it is.. | 19:44 |
*** annashen has joined #openstack-cinder | 19:46 | |
jungleboyj | asselin: I guess I need to better understand what their current problems are any why they are wanting to wait for your new unified approach to be available. | 19:55 |
asselin | jungleboyj, I would rather it be called "our" new unified approach. Their help would be much wlecomed. | 19:56 |
jungleboyj | Ah, so have them pull it down and start using it. Help work the bugs out? | 19:56 |
vilobhmm | xyang1: ping | 19:57 |
asselin | yes & review patches | 19:57 |
asselin | https://etherpad.openstack.org/p/common-ci-sprint | 19:57 |
xyang1 | vilobhmm: Hi | 19:58 |
*** DericHorn-HP has quit IRC | 19:59 | |
vilobhmm | xyang1 : since you are our ConsistencyGroup expert thought it would be nice if you can have a look at https://review.openstack.org/#/c/163641/. This has already lots of +1 and +2 and needs to be approved…can you please have a look ? | 20:02 |
openstackgerrit | John Griffith proposed openstack/cinder: Use a single base class for the VolumeDriver https://review.openstack.org/201812 | 20:02 |
*** gouthamr has quit IRC | 20:02 | |
openstackgerrit | John Griffith proposed openstack/cinder: Use a single base class for the VolumeDriver https://review.openstack.org/201812 | 20:03 |
*** gouthamr has joined #openstack-cinder | 20:04 | |
Swanson | jgriffith: "full proof" | 20:04 |
Swanson | ? | 20:04 |
jgriffith | Swanson: ? | 20:04 |
Swanson | From your commit message. | 20:04 |
jgriffith | Swanson: ok? | 20:04 |
Swanson | Perhaps that is a phrase I've never heard. | 20:05 |
jgriffith | Swanson: what... did I spell som | 20:05 |
jgriffith | Ohhh | 20:05 |
jgriffith | Swanson: "full proof" as in "guaranteed to always work" | 20:05 |
jgriffith | or maybe the proper term is "fool proof" :) | 20:05 |
Swanson | jgriffith: I'm more familiar with the latter. | 20:06 |
jgriffith | Swanson: yeah http://grammarist.com/usage/foolproof/ | 20:06 |
jgriffith | OpenStack improving spelling and grammar of Software commit messages everywhere! | 20:06 |
*** salv-orlando has joined #openstack-cinder | 20:06 | |
jgriffith | Swanson: I'm kinda surprised you couldn't discern what was meant by that | 20:06 |
xyang1 | vilobhmm: I can take a look. I'd like to give a try | 20:06 |
vilobhmm | xyang1: sure thanks :) | 20:07 |
jgriffith | Swanson: it's not like I wrote "specked asdfasdf weeks" instead of "foolproof" | 20:07 |
Swanson | jgriffith: :) I wasn't entirely sure if that was different than fool proof or not. | 20:07 |
jgriffith | Swanson: I think it's just my somewhat horrid spelling :) | 20:07 |
jgriffith | Happy to fix it | 20:07 |
Swanson | I will continue reviewing the patch. | 20:08 |
jgriffith | foolproof just seems so politically incorrect :) | 20:08 |
jgriffith | Not nice to call out "fools" :) | 20:08 |
openstackgerrit | John Griffith proposed openstack/cinder: Use a single base class for the VolumeDriver https://review.openstack.org/201812 | 20:09 |
*** mylu has quit IRC | 20:10 | |
Swanson | thanks! | 20:10 |
jgriffith | Swanson: sure, thanks for pointing it out. | 20:10 |
skylerberg | Is anyone else having trouble with their CI for cinder? Mine started failing yesterday. http://openstack-ci.tintri.com/tintri/refs-changes-58-202858-1/ | 20:12 |
Swanson | skylerberg: We only run if something passes gate. Since cinder has largely been choking on the ceph test in gate today I don't think we are running much. | 20:14 |
jgriffith | skylerberg: well... I haven't | 20:14 |
*** mylu has joined #openstack-cinder | 20:14 | |
jgriffith | skylerberg: you're clearly getting an error in the driver trying to refrieve the snapshot: http://openstack-ci.tintri.com/tintri/refs-changes-58-202858-1/logs/screen-c-vol.log.txt | 20:15 |
jgriffith | 2015-07-18 23:00:28.674 | 20:15 |
jgriffith | Swanson: +1 | 20:15 |
jgriffith | Swanson: I should probably force a run here now though to make sure | 20:15 |
*** krtaylor has quit IRC | 20:16 | |
skylerberg | Swanson: I am also only running if Jenkins gives a +1. | 20:17 |
skylerberg | jgriffith: Thanks for taking a look | 20:17 |
*** krtaylor has joined #openstack-cinder | 20:18 | |
*** annashen has quit IRC | 20:18 | |
jgriffith | skylerberg: sure, hope it helps... | 20:19 |
xyang1 | jgriffith: Hi, can you take a look of this and see if I've addressed your comments? https://review.openstack.org/#/c/193937/ | 20:20 |
openstackgerrit | Ph. Marek proposed openstack/cinder: DRBD: Define a separate prefix for snapshots https://review.openstack.org/203657 | 20:24 |
Swanson | jgriffith: What is the point of remove_export? I've figured out what create_export is for now that hemna is adding the connector to it but not so much remove_export. | 20:28 |
hemna | Swanson, yah I'm not sure about remove_export. terminate_connection seems to be place to nuke the exports from what I can tell | 20:29 |
skylerberg | jgriffith, Swanson: It looks like the issue is probably on our backend. I have a full database. | 20:30 |
jgriffith | Swanson: https://github.com/openstack/cinder/blob/master/cinder/volume/targets/iscsi.py#L230 | 20:30 |
jgriffith | Swanson: That "is" the purpose | 20:31 |
*** timcl has quit IRC | 20:33 | |
jgriffith | Swanson: hemna FTR, it used to be that a target was created upon volume creation. And it stayed in existence until delete volume | 20:35 |
*** jwcroppe has quit IRC | 20:35 | |
jgriffith | Swanson: hemna that's not how we do it any longer, so there's some room for clean up there I think | 20:36 |
hemna | ok that makes sense why I'm seeing remove_export in delete_volume | 20:36 |
*** jwcroppe has joined #openstack-cinder | 20:36 | |
jgriffith | hemna: yeah | 20:36 |
Swanson | jgriffith: ah. Very helpful. Thanks! | 20:38 |
*** jwcroppe has quit IRC | 20:40 | |
*** jwcroppe has joined #openstack-cinder | 20:40 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add connector object to create_export https://review.openstack.org/202808 | 20:43 |
*** changbl has quit IRC | 20:46 | |
*** annashen has joined #openstack-cinder | 20:46 | |
openstackgerrit | Rajini Ram proposed openstack/cinder: Eqlx Driver: Revised the ssh code to use common utils https://review.openstack.org/203720 | 20:47 |
*** salv-orlando has quit IRC | 20:47 | |
openstackgerrit | Kendall Nelson proposed openstack/cinder: Enable cinder-manage to delete old services from the database https://review.openstack.org/197613 | 20:49 |
openstackgerrit | John Griffith proposed openstack/cinder: Remove logging statements from migrations https://review.openstack.org/191983 | 20:53 |
*** dustins has quit IRC | 20:54 | |
*** akerr has quit IRC | 20:55 | |
*** bswartz has quit IRC | 20:55 | |
openstackgerrit | Rajini Ram proposed openstack/cinder: Eqlx Driver: Revised the ssh code to use common utils https://review.openstack.org/203720 | 20:56 |
*** changbl has joined #openstack-cinder | 20:58 | |
*** thangp has quit IRC | 21:06 | |
*** salv-orlando has joined #openstack-cinder | 21:09 | |
*** annashen has quit IRC | 21:11 | |
*** Yogi1 has quit IRC | 21:16 | |
*** mylu has quit IRC | 21:17 | |
*** dims_ has quit IRC | 21:22 | |
*** ronis has quit IRC | 21:22 | |
*** annashen has joined #openstack-cinder | 21:22 | |
openstackgerrit | John Griffith proposed openstack/cinder: Use a single base class for the VolumeDriver https://review.openstack.org/201812 | 21:31 |
*** jordanP has quit IRC | 21:39 | |
*** bluex-pl has quit IRC | 21:40 | |
*** IanGovett has joined #openstack-cinder | 21:41 | |
patrickeast | hemna: hey quick question about the create_export change, is the intent for iscsi that the CHAP stuff should be setup and saved in the provider_auth field there and not done in initialize_connection? | 21:41 |
patrickeast | or does it matter? | 21:41 |
jgriffith | hemna: pleas say no, pleas say no.... | 21:42 |
hemna | well the result of create_export turns into a model update | 21:42 |
patrickeast | lol | 21:42 |
hemna | currently in the volume manager | 21:42 |
* patrickeast is still confused by what is actually supposed to be done there | 21:43 | |
patrickeast | right | 21:43 |
*** changbl has quit IRC | 21:43 | |
patrickeast | and some drivers setup chap info and put it in the provider_auth field | 21:43 |
patrickeast | on the volume | 21:43 |
patrickeast | some, like mine dont | 21:43 |
Swanson | I was planning on returning nothing or an exception in create_export. | 21:43 |
patrickeast | but with the connector passed in now i *could* | 21:43 |
hemna | yah, I don't think ours does | 21:43 |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Adds framework for get_capabilities() feature https://review.openstack.org/201243 | 21:43 |
jgriffith | hemna: patrickeast I'm confused... create_export already returns a model_update | 21:43 |
hemna | yah it already does a model update | 21:43 |
jgriffith | hemna: patrickeast https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L1081 | 21:43 |
hemna | so, I suppose you could if you had to | 21:44 |
patrickeast | so sounds like it doesn’t matter | 21:44 |
hemna | jgriffith, yah that's what I was trying to say earlier. | 21:44 |
hemna | but most aren't aware of it, as they simply don't use create_export today | 21:44 |
patrickeast | i bring it up because if it is expected to we might need to pass the initiator data into it too | 21:44 |
patrickeast | like i did for initialize_connection | 21:44 |
Swanson | I think lvm and solidfire use it. | 21:44 |
jgriffith | hemna: k... sorry, I was confused; thought you said: <hemna> currently in the volume manager | 21:44 |
hemna | we don't even use provider_auth at all in our 3par iscsi driver fwiw. | 21:45 |
jgriffith | hemna: oops... that was gibberish; You said "well the result of create_export turns into a model update" | 21:45 |
jgriffith | hemna: but it already is a model_update which as Swanson points out a number of us rely on already | 21:46 |
hemna | yah | 21:46 |
hemna | yup | 21:46 |
jgriffith | we just don't need initiator info to build that | 21:46 |
hemna | I had no intentions of changing that at all | 21:46 |
jgriffith | patrickeast: so the idea isn't for that to go away, or change... just allow passing in initiator data for those that need it; hemna is that accurate? | 21:47 |
hemna | simply just passing in connector to the create_export, so folks can do the target creation and exporting. | 21:47 |
hemna | jgriffith, +1 | 21:47 |
hemna | exactly | 21:47 |
jgriffith | hemna: perfect :) thanks. Just wanted to make sure we were still on the same page | 21:47 |
patrickeast | right, yea, but what exactly is intended by exporting the target? with this change i can do it in either place and it isn’t any different… which is why i’m confused as to what exactly should or shouldn’t be done in each method | 21:48 |
*** e0ne has quit IRC | 21:48 | |
*** e0ne has joined #openstack-cinder | 21:48 | |
jgriffith | patrickeast: so yeah, that's kinda weird. My suggestion last week was modify just the manager code and driver.initialize_connection | 21:48 |
Swanson | I'm moving mapping of volumes on my backend to create_export and the gathering of information about that to initialize_connection. | 21:49 |
jgriffith | patrickeast: but honestly I don't know if that's actually any better. The thought being just modify manager to always call initialize_con and finish things up there | 21:49 |
patrickeast | is the expectation that they are not always called together? | 21:49 |
Swanson | Alternately I could just leave everything in initialize_connection and add protection against remapping volumes in there. | 21:49 |
jgriffith | patrickeast: add the option to update model_info from that call if desired/needed | 21:49 |
patrickeast | yea that makes sense | 21:50 |
hemna | patrickeast, so from my perspective, this change is to get most drivers to switch to using create_export as it's intended today. which is creating the export target on the array. | 21:50 |
hemna | our drivers can't do the target creation on the array today, because it needs the connector. which is why we do it in initialize_connection, which causes problems at live migration time. | 21:50 |
patrickeast | hemna: ok i’m with you so far, my driver has the same thing | 21:51 |
hemna | nova calls initialize_connection multiple times for the same volume attachment during live migration, just to fetch the connection information. | 21:51 |
jgriffith | hemna: so that's weird though, because before we used to require target creation on "volume create" like I was talking about before. And then the initialize connection call would do a model_update with the final bits that were needed at actual connect time | 21:51 |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Efficient volume copy for generic volume migration https://review.openstack.org/183701 | 21:52 |
hemna | so this change is to 1) allow drivers to use create_export for what it's intended, and 2) eventually get those same drivers to do nothing but collect the connection info inside of initialize_connection, so live migration works for them. | 21:52 |
patrickeast | hemna: so if the driver can handle initialize_connection being called multiple times for the same combo initiator+volume then it doesn’t matter where we do stuff? | 21:52 |
Swanson | I suspect the problem is backends like Dell's where you can't create an actual target until you have the connection information. | 21:52 |
patrickeast | as long as the connection info returned from initialize_connection is always correct | 21:52 |
patrickeast | right? | 21:52 |
*** IanGovett has quit IRC | 21:53 | |
hemna | jgriffith, we can't create an exported target until we know who we are exporting to. (re: needs connector) | 21:53 |
jgriffith | hemna: why not do "model_update, conn_info = self.driver.initialize_connection(volume...." | 21:53 |
Swanson | patrickeast: yes. | 21:53 |
*** IanGovett has joined #openstack-cinder | 21:53 | |
hemna | Swanson, yup. that's our issue as well. and from the looks of my research, 28 drivers have the same problem. | 21:53 |
jgriffith | hemna: Yeah, I get what you're saying (partially), but I'm saying you can "cheat" around that fairly easy | 21:53 |
jgriffith | and keep the behavior of all the calls consistent (if you wanted to) | 21:54 |
hemna | sure | 21:54 |
hemna | I think if we added the connector, then create_export would have a better meaning, and folks would/could use it as intended. | 21:54 |
hemna | fwiw | 21:55 |
jgriffith | hemna: yeah.. maybe so | 21:55 |
hemna | there seems to be a lot of confusion about it now | 21:55 |
jgriffith | hemna: sigh | 21:55 |
hemna | 28 drivers are doing it 'wrong' today, in initialize_connection | 21:55 |
hemna | my drivers included | 21:55 |
hemna | and of those 28 drivers, most aren't doing any checks to see if the volume is already exported or not | 21:56 |
Swanson | hemna: I think it makes it usable to the rest of us and keeps me from having to repeatedly check whether the volume has been mapped in initialize_connection. So there is benefit. | 21:56 |
mtanino | only lvm driver... | 21:56 |
jgriffith | hemna: that's still less than half :) | 21:56 |
hemna | which causes lots of 'fun' after live migration. | 21:56 |
*** jungleboyj has quit IRC | 21:56 | |
hemna | how many drivers do we have now? | 21:56 |
hemna | I thought it was ~ 40 ? | 21:56 |
jgriffith | hemna: actually not sure currently. We were at 52 a while ago, but some were removed etc. Would be interesting to actually get a count | 21:57 |
hemna | yah that'd be nice to see | 21:57 |
hemna | I guess I was way off! | 21:57 |
jgriffith | hemna: I dunno, you might be right on | 21:58 |
jgriffith | I haven't tried to figure it out in a while | 21:58 |
Swanson | Jgriffith: hemna, Net is that it is a lot of drivers and those who do not want to change what they are doing do not have to. | 21:58 |
eharney | look at the ABC thread on openstack-dev. his list shows 80-something drivers | 21:58 |
jgriffith | eharney: That's an awful lot of drivers!!! | 21:59 |
Swanson | It is all irrelevant. Nothing is ever going to get through gate again. | 22:00 |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder-specs: Add Connector to create_export https://review.openstack.org/202765 | 22:00 |
jgriffith | Swanson: hmm.. now what happened? | 22:00 |
*** kjnelson has quit IRC | 22:00 | |
Swanson | jgriffith: ceph has been failing all day. | 22:00 |
hemna | 80 | 22:00 |
hemna | goosh | 22:00 |
jgriffith | Swanson: Oh, the Ceph job is broke AGAIN | 22:01 |
hemna | Swanson, yah I noticed ceph started puking on my patches. :( | 22:01 |
jgriffith | It was broken most of last week, fixed over the week-end, now broke again | 22:01 |
jgriffith | Kinda wish it wouldn't have been promoted to voting :( | 22:01 |
*** akerr has joined #openstack-cinder | 22:01 | |
mtanino | So much unit test failed today... | 22:02 |
*** vilobhmm has quit IRC | 22:02 | |
Swanson | jgriffith: I popped up the last dozen jobs and sorting through them it looks like they all failed on the same thing. | 22:02 |
jgriffith | mtanino: yeah, that's another fun thing with mock/mox | 22:02 |
jgriffith | mtanino: but thingee noted that 1.2 update seems to have fixed that up | 22:02 |
mtanino | jgriffith: It's nice | 22:03 |
*** IanGovett has quit IRC | 22:05 | |
openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: Nova volume-detach --force requires API https://review.openstack.org/187305 | 22:05 |
eharney | i'm seeing a ceph failure tied to https://bugs.launchpad.net/openstack-ci/+bug/1311066 ? | 22:07 |
openstack | Launchpad bug 1311066 in OpenStack-Gate "Some nodes allocated in node pool are very very slow" [High,Confirmed] | 22:07 |
jgriffith | jaypipes: OMG.. HTF did I manage to do that! Thanks for pointing it out. Teach me to try and "send a quick email before running out the door" | 22:07 |
eharney | "nodes are slow" sounds kind of unfortunate | 22:07 |
jgriffith | eharney: there was an issue with glance interaction last week as well IIRC | 22:08 |
jgriffith | eharney: https://bugs.launchpad.net/glance/+bug/1475811 | 22:09 |
openstack | Launchpad bug 1475811 in Glance "gate-tempest-dsvm-full-ceph fails to upload image with "TypeError: an integer is required"" [Critical,Fix released] - Assigned to Matt Riedemann (mriedem) | 22:09 |
eharney | jgriffith: the one i'm looking at now is cinder-backup | 22:10 |
jgriffith | eharney: sure, I was just pointing out the one that was fixed recently | 22:10 |
jgriffith | eharney: sorry, thought you were asking about it | 22:11 |
eharney | jgriffith: yeah, i was, just wondering if there were bugs more specific than "slow" :) | 22:11 |
jgriffith | Oh... Swanson rebase should fix those I think | 22:11 |
jgriffith | eharney: that one I have no idea... that's not very useful IMO (slow) | 22:12 |
eharney | jgriffith: is "volume-0000-0000-0000-0000" something we do? | 22:15 |
eharney | that's not even a uuid | 22:15 |
jgriffith | eharney: that's no good | 22:15 |
jgriffith | eharney: I could see a unit test doing it | 22:15 |
*** changbl has joined #openstack-cinder | 22:15 | |
eharney | jgriffith: this is in tempest... http://logs.openstack.org/24/193124/10/check/gate-tempest-dsvm-full-ceph/1ac4c83/logs/screen-c-bak.txt.gz | 22:15 |
jgriffith | eharney: hmm... the backup api has something there | 22:16 |
jgriffith | eharney: Wow.. that's pretty awful | 22:16 |
jgriffith | sorry, not being "emotional" | 22:16 |
eharney | something's weird, but i dunno that i know the code enough to pinpoint it very quickly | 22:17 |
jgriffith | but setting a charges with a default volume_id of 0000-0000-0000-0000 doesn't seem too smart | 22:17 |
*** mylu has joined #openstack-cinder | 22:17 | |
eharney | oh... the backup api does that | 22:17 |
eharney | yeah, that seems... not right | 22:17 |
jgriffith | eharney: see if this is related: https://github.com/openstack/cinder/blob/master/cinder/backup/api.py#L379 | 22:17 |
jgriffith | sigh | 22:18 |
*** ganso_ has quit IRC | 22:18 | |
jgriffith | eharney: IMO that should be "None" and the populated on the import appropriately | 22:19 |
*** edmondsw has quit IRC | 22:19 | |
eharney | jgriffith: yeah, i would think so | 22:19 |
jgriffith | s/the/then/ | 22:19 |
eharney | so, it kind of seems like this failure is actually a bug | 22:21 |
jgriffith | https://bugs.launchpad.net/cinder/+bug/1476416 | 22:21 |
openstack | Launchpad bug 1476416 in Cinder "backup.api:mport_record sets volume_id to '0000-0000-0000-0000'" [Undecided,New] | 22:21 |
jgriffith | eharney: You mean that cep failure? | 22:21 |
eharney | jgriffith: yeah | 22:22 |
eharney | jgriffith: good, i was going to do that :) | 22:22 |
jgriffith | eharney: In that case "Yay" | 22:22 |
jgriffith | eharney: which log file are you looking at? | 22:23 |
jgriffith | eharney: i.e.; which tempest run | 22:23 |
*** akerr has quit IRC | 22:23 | |
eharney | jgriffith: https://review.openstack.org/#/c/193124/ | 22:23 |
jgriffith | eharney: the one I picked was a failure to deploy | 22:23 |
eharney | http://logs.openstack.org/24/193124/10/check/gate-tempest-dsvm-full-ceph/1ac4c83/ | 22:23 |
eharney | jgriffith: this is the one that triggered on the "slow" bug, probably need a more specific query | 22:24 |
jgriffith | ahh, got ya | 22:24 |
*** e0ne has quit IRC | 22:25 | |
jgriffith | eharney: yeah, that's lame: http://logs.openstack.org/24/193124/10/check/gate-tempest-dsvm-full-ceph/1ac4c83/logs/screen-c-bak.txt.gz#_2015-07-20_20_17_53_706 | 22:27 |
*** primechu_ has quit IRC | 22:27 | |
harlowja | jgriffith can u stop being so subjective please | 22:27 |
harlowja | lol | 22:27 |
*** changbl has quit IRC | 22:27 | |
jgriffith | harlowja: good lord!!! | 22:27 |
harlowja | :) | 22:27 |
eharney | i agree here, it is lame :) | 22:27 |
* jgriffith is the biggest idiot on the planet!!! | 22:27 | |
harlowja | thats subjective | 22:27 |
* harlowja i thought i was the biggest idiot on the planet!!! | 22:28 | |
harlowja | lol | 22:28 |
jgriffith | harlowja: nope, try less hard | 22:28 |
harlowja | :) | 22:28 |
jgriffith | I wear the crown!! | 22:28 |
* eharney really needs to figure out how to work elastic recheck | 22:28 | |
harlowja | jgriffith guess i need to take lessons from u | 22:28 |
jgriffith | eharney: what are you trying to do... maybe I can help? | 22:29 |
eharney | jgriffith: just query for this "error opening image volume-0000-0000-0000-0000.backup." | 22:29 |
eharney | in logstash | 22:29 |
jgriffith | should be able to just put it in quotes | 22:29 |
eharney | maybe this is just the first time it happened and it isn't indexed yet | 22:30 |
jgriffith | yeah, I didn't get a hit on it | 22:30 |
jgriffith | eharney: https://github.com/openstack/tempest/commit/18c1d0eba54f1d6e8f4df4ec76c95efe472f2b14 | 22:31 |
jgriffith | eharney: looks like that just came online | 22:32 |
jgriffith | eharney: Ya know, you'd think that the tempest changes should be gated too :) | 22:33 |
eharney | haha | 22:33 |
mtreinish | jgriffith: yeah that test was merged this morning | 22:33 |
mtreinish | I just saw some failures from it | 22:33 |
jgriffith | mtreinish: since cep is voting maybe we should make a ceph job for tempest? | 22:34 |
mtreinish | jgriffith: I'm hesitant to do that, because we're already at ~30 jobs on a tempest commit | 22:34 |
mtreinish | and the backend really shouldn't matter from the api perspective | 22:35 |
jgriffith | mtreinish: yeah, just brought that up in gerrit :( | 22:35 |
jgriffith | mtreinish: then I'd propose we should discuss whether that job should be voting or not maybe? | 22:35 |
mtreinish | but if you think it's necessary I guess we can do it | 22:35 |
jgriffith | mtreinish: well.. not necessarily | 22:35 |
mtreinish | it's just not feesible to co-gate everything on tempest | 22:35 |
jgriffith | mtreinish: just a bummer to get a pile up for stuff like this | 22:35 |
mtreinish | we have to pick and choose | 22:35 |
jgriffith | mtreinish: the thing about Ceph is it's VERY unique compared to other drivers | 22:36 |
mtreinish | heh, compared to gluster :) | 22:36 |
jgriffith | mtreinish: but yeah, I understand about having to pick/choose | 22:36 |
jgriffith | mtreinish: just kinda thinking out loud | 22:36 |
*** vilobhmm has joined #openstack-cinder | 22:37 | |
mtreinish | jgriffith: fwiw, I think we have ceph in tempest experimental | 22:38 |
mtreinish | I was just lax and didn't run it before +Aing that test | 22:38 |
*** topshare has joined #openstack-cinder | 22:39 | |
*** changbl has joined #openstack-cinder | 22:40 | |
*** gouthamr has quit IRC | 22:41 | |
*** earlephilhower has quit IRC | 22:42 | |
eharney | jgriffith: https://review.openstack.org/#/c/203845/ | 22:43 |
mtreinish | eharney: is that backup export import stuff an optional feature which depends on the cinder configuration | 22:44 |
*** garthb_ has quit IRC | 22:44 | |
*** garthb_ has joined #openstack-cinder | 22:44 | |
mtreinish | or is it something that is expected on every cinder deployment? | 22:44 |
eharney | mtreinish: i think it's there for any deployment using the cinder-backup service | 22:45 |
mtreinish | eharney: ok, if it were optional I would have suggested adding a config flag to tempest for that feature. But in this case I guess ceph is just broke | 22:45 |
*** topshare has quit IRC | 22:47 | |
*** earlephilhower has joined #openstack-cinder | 22:49 | |
*** topshare has joined #openstack-cinder | 22:50 | |
*** chlong has quit IRC | 22:54 | |
*** markus_z has joined #openstack-cinder | 23:03 | |
*** eharney has quit IRC | 23:04 | |
*** jwcroppe has quit IRC | 23:07 | |
*** annashen has quit IRC | 23:09 | |
*** markus_z has quit IRC | 23:12 | |
jaypipes | jgriffith: lol, no worries duder :) | 23:12 |
*** mylu has quit IRC | 23:14 | |
*** changbl has quit IRC | 23:15 | |
*** mriedem_away has quit IRC | 23:19 | |
*** bitblt has joined #openstack-cinder | 23:20 | |
*** jwcroppe has joined #openstack-cinder | 23:22 | |
openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: Nova volume-detach --force requires API https://review.openstack.org/187305 | 23:23 |
*** bswartz has joined #openstack-cinder | 23:23 | |
*** jungleboyj has joined #openstack-cinder | 23:24 | |
*** jwcroppe has quit IRC | 23:27 | |
*** changbl has joined #openstack-cinder | 23:29 | |
*** angela-s has quit IRC | 23:31 | |
*** jungleboyj has quit IRC | 23:35 | |
*** gouthamr has joined #openstack-cinder | 23:36 | |
*** mylu has joined #openstack-cinder | 23:37 | |
*** changbl has quit IRC | 23:37 | |
*** changbl has joined #openstack-cinder | 23:38 | |
*** IlyaG has joined #openstack-cinder | 23:38 | |
*** IlyaG has quit IRC | 23:39 | |
*** jwcroppe has joined #openstack-cinder | 23:39 | |
*** LiuXinguo has joined #openstack-cinder | 23:40 | |
*** s0nea has quit IRC | 23:42 | |
*** Liu has quit IRC | 23:43 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: WIP: Add new Connector APIs for path validation https://review.openstack.org/199764 | 23:44 |
*** s0nea has joined #openstack-cinder | 23:44 | |
*** mdenny has quit IRC | 23:45 | |
*** raunak has joined #openstack-cinder | 23:46 | |
*** mgarza_ has quit IRC | 23:47 | |
jgriffith | mtreinish: LOL... yeah; good point (re Gluster) | 23:47 |
jgriffith | mtreinish: eharney: Can't remember if we have a mechanism to skip this only for the specific configuration? https://review.openstack.org/#/c/203845/1 | 23:49 |
jgriffith | mtreinish: or if that's even worth considering here or not | 23:49 |
mtreinish | jgriffith: that's probably how I would do the skip. It should be through the test runner, if the env is just broken because of the driver | 23:50 |
jgriffith | mtreinish: cool | 23:52 |
jgriffith | mtreinish: do we have cases where we do that in test runner today? | 23:52 |
mtreinish | the cells job does that | 23:52 |
jgriffith | mtreinish: righty ho | 23:53 |
mtreinish | well they have a file in the nova repo to define the skip regex | 23:53 |
mtreinish | but it's the same basic mechanism | 23:53 |
jgriffith | mtreinish: yeah, looking for a template to submit it :) | 23:53 |
*** primechuck has joined #openstack-cinder | 23:54 | |
mtreinish | jgriffith: heh, gluster also has to do something like that IIRC :) | 23:55 |
jgriffith | mtreinish: oh yeah.. I forgot about that. | 23:55 |
*** diogogmt has quit IRC | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!