| *** leeantho has quit IRC | 00:03 | |
| *** Suyi_ has quit IRC | 00:04 | |
| *** rdx_matt has joined #openstack-cinder | 00:08 | |
| *** jamielennox is now known as jamielennox|away | 00:12 | |
| *** sdague has quit IRC | 00:16 | |
| *** gouthamr has joined #openstack-cinder | 00:17 | |
| *** diogogmt has quit IRC | 00:27 | |
| *** erlon has joined #openstack-cinder | 00:28 | |
| *** alonma has joined #openstack-cinder | 00:32 | |
| *** alonma has quit IRC | 00:36 | |
| *** alonma has joined #openstack-cinder | 00:39 | |
| *** rdx_matt has quit IRC | 00:41 | |
| *** laughterwym has joined #openstack-cinder | 00:44 | |
| *** laughterwym has quit IRC | 00:44 | |
| *** alonma has quit IRC | 00:44 | |
| *** laughterwym has joined #openstack-cinder | 00:44 | |
| *** laughterwym has joined #openstack-cinder | 00:45 | |
| *** sdake_ has quit IRC | 00:45 | |
| *** alonma has joined #openstack-cinder | 00:47 | |
| *** sdake has joined #openstack-cinder | 00:47 | |
| *** mtanin___ has quit IRC | 00:48 | |
| *** alonma has quit IRC | 00:51 | |
| *** ducttape_ has quit IRC | 00:54 | |
| *** ducttape_ has joined #openstack-cinder | 00:56 | |
| *** enriquetaso has quit IRC | 00:56 | |
| *** huyang has joined #openstack-cinder | 00:59 | |
| *** cknight1 has joined #openstack-cinder | 01:00 | |
| *** cknight has quit IRC | 01:02 | |
| *** sdake has quit IRC | 01:04 | |
| *** kaisers_ has joined #openstack-cinder | 01:06 | |
| *** kaisers_ has quit IRC | 01:11 | |
| *** yangyapeng has joined #openstack-cinder | 01:11 | |
| *** ducttape_ has quit IRC | 01:17 | |
| *** Apoorva has quit IRC | 01:18 | |
| *** zhongjun_ has joined #openstack-cinder | 01:29 | |
| *** TaylorHuang has joined #openstack-cinder | 01:31 | |
| *** openstackgerrit has quit IRC | 01:33 | |
| *** gouthamr_ has joined #openstack-cinder | 01:34 | |
| *** Lee1092 has joined #openstack-cinder | 01:34 | |
| *** TaylorH has quit IRC | 01:34 | |
| *** openstackgerrit has joined #openstack-cinder | 01:34 | |
| *** TaylorH has joined #openstack-cinder | 01:35 | |
| *** gouthamr has quit IRC | 01:37 | |
| *** rdx_matt has joined #openstack-cinder | 01:38 | |
| *** TaylorHuang has quit IRC | 01:38 | |
| *** Srinu_ has joined #openstack-cinder | 01:38 | |
| *** TaylorH has quit IRC | 01:39 | |
| *** jwcroppe has quit IRC | 01:42 | |
| *** yangyapeng has quit IRC | 01:44 | |
| *** yangyapeng has joined #openstack-cinder | 01:44 | |
| *** itzdilip has joined #openstack-cinder | 01:52 | |
| openstackgerrit | JiWei proposed openstack/cinder: Implement nfs manage/unmanage of a volume https://review.openstack.org/340209 | 01:55 |
|---|---|---|
| *** Srinu_ has quit IRC | 01:58 | |
| *** laughterwym has quit IRC | 02:03 | |
| *** coolsvap_ has joined #openstack-cinder | 02:04 | |
| *** laughterwym has joined #openstack-cinder | 02:04 | |
| *** julim has joined #openstack-cinder | 02:05 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Add backup update function (microversion) https://review.openstack.org/263179 | 02:08 |
| *** rdx_matt has quit IRC | 02:11 | |
| *** sdake has joined #openstack-cinder | 02:12 | |
| *** Suyi_ has joined #openstack-cinder | 02:12 | |
| *** iceyao has joined #openstack-cinder | 02:15 | |
| *** ebalduf has joined #openstack-cinder | 02:16 | |
| *** bardia has quit IRC | 02:19 | |
| *** salv-orl_ has quit IRC | 02:21 | |
| *** lixiaoy1 has quit IRC | 02:23 | |
| *** lixiaoy1 has joined #openstack-cinder | 02:24 | |
| *** Gao has joined #openstack-cinder | 02:26 | |
| *** Lallana has quit IRC | 02:26 | |
| *** yangyapeng has quit IRC | 02:30 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Move fake_constants to root test directory https://review.openstack.org/347104 | 02:31 |
| *** alonma has joined #openstack-cinder | 02:35 | |
| *** alonma has quit IRC | 02:39 | |
| *** jamielennox|away is now known as jamielennox | 02:41 | |
| *** alonma has joined #openstack-cinder | 02:41 | |
| *** ebalduf has quit IRC | 02:45 | |
| *** alonma has quit IRC | 02:46 | |
| *** ducttape_ has joined #openstack-cinder | 02:48 | |
| *** sdake has quit IRC | 02:53 | |
| *** ducttape_ has quit IRC | 02:53 | |
| *** gouthamr_ has quit IRC | 02:54 | |
| *** kaisers_ has joined #openstack-cinder | 02:55 | |
| *** iceyao has quit IRC | 02:58 | |
| *** cknight1 has quit IRC | 02:59 | |
| *** kaisers_ has quit IRC | 02:59 | |
| *** alonma has joined #openstack-cinder | 03:00 | |
| *** cknight has joined #openstack-cinder | 03:01 | |
| *** iceyao has joined #openstack-cinder | 03:01 | |
| *** alonma has quit IRC | 03:04 | |
| *** alonma has joined #openstack-cinder | 03:07 | |
| *** rdx_matt has joined #openstack-cinder | 03:09 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: Storwize: do the correct update after failover https://review.openstack.org/292570 | 03:10 |
| *** alonma has quit IRC | 03:11 | |
| *** sdake has joined #openstack-cinder | 03:13 | |
| *** cknight has quit IRC | 03:14 | |
| *** thumpba has joined #openstack-cinder | 03:15 | |
| *** thumpba has quit IRC | 03:18 | |
| *** yangyapeng has joined #openstack-cinder | 03:22 | |
| *** salv-orlando has joined #openstack-cinder | 03:22 | |
| *** erlon has quit IRC | 03:26 | |
| *** salv-orlando has quit IRC | 03:28 | |
| *** ducttape_ has joined #openstack-cinder | 03:31 | |
| *** ducttape_ has quit IRC | 03:36 | |
| *** yangyape_ has joined #openstack-cinder | 03:36 | |
| *** yangyapeng has quit IRC | 03:39 | |
| *** smile-luobin has joined #openstack-cinder | 03:39 | |
| *** rdx_matt has quit IRC | 03:41 | |
| *** enriquetaso has joined #openstack-cinder | 03:44 | |
| *** enriquetaso has quit IRC | 03:44 | |
| *** sheel has joined #openstack-cinder | 03:48 | |
| *** zhongjun_ has quit IRC | 03:49 | |
| *** abhitechie has joined #openstack-cinder | 03:51 | |
| *** harlowja_at_home has joined #openstack-cinder | 03:52 | |
| *** zhonghua has quit IRC | 03:52 | |
| *** zhonghua-lee has joined #openstack-cinder | 03:53 | |
| *** alonma has joined #openstack-cinder | 03:53 | |
| *** links has joined #openstack-cinder | 03:54 | |
| *** alonma has quit IRC | 03:58 | |
| *** lpetrut has joined #openstack-cinder | 03:59 | |
| *** thumpba has joined #openstack-cinder | 04:00 | |
| *** alonma has joined #openstack-cinder | 04:01 | |
| *** laughterwym has quit IRC | 04:02 | |
| *** alonma has quit IRC | 04:05 | |
| *** laughterwym has joined #openstack-cinder | 04:05 | |
| *** thumpba has quit IRC | 04:06 | |
| *** kaisers_ has joined #openstack-cinder | 04:07 | |
| *** kaisers_ has quit IRC | 04:07 | |
| *** alonma has joined #openstack-cinder | 04:08 | |
| *** sandanar has joined #openstack-cinder | 04:09 | |
| *** links has quit IRC | 04:09 | |
| *** laughterwym has quit IRC | 04:10 | |
| *** links has joined #openstack-cinder | 04:12 | |
| *** alonma has quit IRC | 04:12 | |
| *** thumpba has joined #openstack-cinder | 04:15 | |
| *** alonma has joined #openstack-cinder | 04:16 | |
| *** jgregor has joined #openstack-cinder | 04:19 | |
| *** alonma has quit IRC | 04:20 | |
| *** harlowja_at_home has quit IRC | 04:22 | |
| *** alonma has joined #openstack-cinder | 04:23 | |
| *** salv-orlando has joined #openstack-cinder | 04:24 | |
| *** laughterwym has joined #openstack-cinder | 04:24 | |
| *** ManishD has joined #openstack-cinder | 04:26 | |
| *** jgregor1 has joined #openstack-cinder | 04:26 | |
| *** thumpba has quit IRC | 04:27 | |
| *** jgregor has quit IRC | 04:27 | |
| *** alonma has quit IRC | 04:28 | |
| *** salv-orlando has quit IRC | 04:28 | |
| *** laughterwym has quit IRC | 04:29 | |
| *** pwm has joined #openstack-cinder | 04:29 | |
| *** pwm has quit IRC | 04:30 | |
| *** lpetrut has quit IRC | 04:30 | |
| *** jamielennox is now known as jamielennox|away | 04:30 | |
| *** pwm has joined #openstack-cinder | 04:30 | |
| *** alonma has joined #openstack-cinder | 04:31 | |
| *** bks has joined #openstack-cinder | 04:31 | |
| *** avishay has joined #openstack-cinder | 04:34 | |
| *** liverpooler has quit IRC | 04:34 | |
| *** Poornima has joined #openstack-cinder | 04:35 | |
| avishay | smcginnis: hey, just saw your review on the list manageable v3 patch. the rebase was actually trivial - just change from microversion 3.7 to 3.8. | 04:35 |
| *** alonma has quit IRC | 04:35 | |
| *** thumpba has joined #openstack-cinder | 04:36 | |
| *** alonma has joined #openstack-cinder | 04:38 | |
| *** rdx_matt has joined #openstack-cinder | 04:39 | |
| *** GB21 has joined #openstack-cinder | 04:39 | |
| *** laughterwym has joined #openstack-cinder | 04:39 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 04:40 |
| *** jgregor1 has quit IRC | 04:42 | |
| *** alonma has quit IRC | 04:42 | |
| *** sandanar has quit IRC | 04:44 | |
| *** alonma has joined #openstack-cinder | 04:46 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group type and group specs https://review.openstack.org/320165 | 04:46 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 04:50 |
| *** alonma has quit IRC | 04:50 | |
| *** jamielennox|away is now known as jamielennox | 04:51 | |
| *** alonma has joined #openstack-cinder | 04:53 | |
| openstackgerrit | xing-yang proposed openstack/cinder: WIP: Add group snapshots https://review.openstack.org/328052 | 04:57 |
| *** Poornima has quit IRC | 04:57 | |
| *** alonma has quit IRC | 04:57 | |
| *** ducttape_ has joined #openstack-cinder | 04:57 | |
| *** adisky has joined #openstack-cinder | 05:00 | |
| avishay | if any core reviewer can +A this again I would appreciate it - keep having to rebase microversion patch... https://review.openstack.org/#/c/307324/ | 05:00 |
| avishay | DuncanT smcginnis hemnafk ^ | 05:01 |
| *** ducttape_ has quit IRC | 05:02 | |
| *** GB21 has quit IRC | 05:03 | |
| *** GB21 has joined #openstack-cinder | 05:03 | |
| *** takedakn has joined #openstack-cinder | 05:04 | |
| openstackgerrit | Vijay Ladani proposed openstack/cinder: Retry delete snapshot if snapshot is busy https://review.openstack.org/342722 | 05:05 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Fix wrong usage of config opiton gpfs_images_share_mode https://review.openstack.org/334868 | 05:08 |
| *** pbandark has joined #openstack-cinder | 05:08 | |
| *** takedakn has quit IRC | 05:09 | |
| *** laughter_ has joined #openstack-cinder | 05:10 | |
| *** rdx_matt has quit IRC | 05:12 | |
| *** laughterwym has quit IRC | 05:13 | |
| *** salv-orlando has joined #openstack-cinder | 05:14 | |
| *** pwm has quit IRC | 05:15 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: test: fix wrong usage of config option in test_window https://review.openstack.org/334877 | 05:17 |
| *** maestropandy has joined #openstack-cinder | 05:21 | |
| openstackgerrit | Satish Venkatasubramanian proposed openstack/cinder: Add volumes/summary API support. https://review.openstack.org/335067 | 05:22 |
| *** GB21 has quit IRC | 05:25 | |
| *** beagles has quit IRC | 05:26 | |
| *** GB21 has joined #openstack-cinder | 05:27 | |
| *** bks has quit IRC | 05:29 | |
| *** laughter_ has quit IRC | 05:31 | |
| *** ChubYann has quit IRC | 05:31 | |
| *** laughterwym has joined #openstack-cinder | 05:31 | |
| *** pgadiya has joined #openstack-cinder | 05:34 | |
| *** avishay has quit IRC | 05:43 | |
| *** alonma has joined #openstack-cinder | 05:47 | |
| *** GB21 has quit IRC | 05:49 | |
| *** GB21 has joined #openstack-cinder | 05:51 | |
| *** alonma has quit IRC | 05:52 | |
| *** sandanar has joined #openstack-cinder | 05:53 | |
| *** EinstCrazy has joined #openstack-cinder | 05:53 | |
| *** maestropandy has quit IRC | 05:54 | |
| *** alonma has joined #openstack-cinder | 05:55 | |
| *** coolsvap_ is now known as coolsvap | 05:56 | |
| nikeshm | avishay: hi | 05:56 |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO: fix the returned value of CG actions https://review.openstack.org/346831 | 05:57 |
| *** EinstCrazy has quit IRC | 05:58 | |
| *** ducttape_ has joined #openstack-cinder | 05:58 | |
| *** alonma has quit IRC | 05:59 | |
| *** bardia has joined #openstack-cinder | 06:00 | |
| *** EinstCrazy has joined #openstack-cinder | 06:02 | |
| *** arecknag has joined #openstack-cinder | 06:02 | |
| *** nkrinner_afk is now known as nkrinner | 06:02 | |
| *** alonma has joined #openstack-cinder | 06:02 | |
| *** ducttape_ has quit IRC | 06:02 | |
| *** Suyi_ has quit IRC | 06:05 | |
| *** zhongjun_ has joined #openstack-cinder | 06:06 | |
| *** alonma has quit IRC | 06:07 | |
| *** laughterwym has quit IRC | 06:09 | |
| *** alonma has joined #openstack-cinder | 06:10 | |
| *** rdx_matt has joined #openstack-cinder | 06:10 | |
| *** EinstCra_ has joined #openstack-cinder | 06:12 | |
| *** laughter_ has joined #openstack-cinder | 06:12 | |
| *** laughte__ has joined #openstack-cinder | 06:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/346496 | 06:15 |
| *** GB21 has quit IRC | 06:15 | |
| *** alonma has quit IRC | 06:15 | |
| *** laughterwym has joined #openstack-cinder | 06:15 | |
| *** GB21 has joined #openstack-cinder | 06:16 | |
| *** EinstCrazy has quit IRC | 06:16 | |
| *** sdake has quit IRC | 06:16 | |
| *** alonma has joined #openstack-cinder | 06:17 | |
| *** laughter_ has quit IRC | 06:18 | |
| *** laughte__ has quit IRC | 06:19 | |
| *** GB21 has quit IRC | 06:21 | |
| *** alonma has quit IRC | 06:22 | |
| *** alonma has joined #openstack-cinder | 06:25 | |
| *** salv-orl_ has joined #openstack-cinder | 06:28 | |
| *** alonma has quit IRC | 06:29 | |
| *** bardia has quit IRC | 06:29 | |
| *** salv-orlando has quit IRC | 06:31 | |
| *** liverpooler has joined #openstack-cinder | 06:31 | |
| openstackgerrit | lg.yue proposed openstack/cinder: add metadata length check to volume-manage https://review.openstack.org/340727 | 06:32 |
| *** alonma has joined #openstack-cinder | 06:32 | |
| *** amitkqed has joined #openstack-cinder | 06:35 | |
| *** amitkqed has quit IRC | 06:36 | |
| *** alonma has quit IRC | 06:37 | |
| *** amitkqed has joined #openstack-cinder | 06:37 | |
| *** laughterwym has quit IRC | 06:38 | |
| *** GB21 has joined #openstack-cinder | 06:40 | |
| *** alonma has joined #openstack-cinder | 06:40 | |
| *** laughter_ has joined #openstack-cinder | 06:41 | |
| *** ManishD has quit IRC | 06:42 | |
| *** rdx_matt has quit IRC | 06:42 | |
| *** tesseract- has joined #openstack-cinder | 06:44 | |
| *** alonma has quit IRC | 06:44 | |
| *** sdake has joined #openstack-cinder | 06:45 | |
| openstackgerrit | Vijay Ladani proposed openstack/cinder: Retry delete snapshot if snapshot is busy https://review.openstack.org/347647 | 06:46 |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Add "start_version" and "end_version" support to argparse https://review.openstack.org/340129 | 06:46 |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Enhance api_microversion doc https://review.openstack.org/331447 | 06:46 |
| *** rcernin has joined #openstack-cinder | 06:47 | |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Fix useless api_version of Manager class https://review.openstack.org/309283 | 06:47 |
| *** maestropandy has joined #openstack-cinder | 06:48 | |
| *** ameade has quit IRC | 06:49 | |
| *** rhefner has quit IRC | 06:49 | |
| *** derekjhyang has quit IRC | 06:49 | |
| *** zhiyan has quit IRC | 06:49 | |
| *** briancurtin has quit IRC | 06:49 | |
| *** rhefner has joined #openstack-cinder | 06:50 | |
| *** cppforlife_ has quit IRC | 06:50 | |
| *** amit213 has quit IRC | 06:50 | |
| *** ildikov has quit IRC | 06:50 | |
| *** amit213 has joined #openstack-cinder | 06:51 | |
| *** adrianofr has quit IRC | 06:51 | |
| *** ildikov has joined #openstack-cinder | 06:51 | |
| *** derekjhyang has joined #openstack-cinder | 06:51 | |
| *** alyson_ has quit IRC | 06:52 | |
| *** ameade has joined #openstack-cinder | 06:52 | |
| *** zhiyan has joined #openstack-cinder | 06:52 | |
| *** briancurtin has joined #openstack-cinder | 06:52 | |
| *** cppforlife_ has joined #openstack-cinder | 06:52 | |
| *** thumpba has quit IRC | 06:53 | |
| *** adrianofr has joined #openstack-cinder | 06:53 | |
| *** alyson_ has joined #openstack-cinder | 06:53 | |
| *** nkrinner has quit IRC | 06:54 | |
| *** nkrinner has joined #openstack-cinder | 06:57 | |
| *** mjura has joined #openstack-cinder | 06:58 | |
| *** ducttape_ has joined #openstack-cinder | 06:59 | |
| *** sdake has quit IRC | 07:01 | |
| *** ducttape_ has quit IRC | 07:03 | |
| *** nkrinner has quit IRC | 07:05 | |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 07:06 |
| *** thumpba has joined #openstack-cinder | 07:15 | |
| ellen | Could I get some reviews on this patch please? https://review.openstack.org/#/c/344941/ | 07:16 |
| *** rdx_matt has joined #openstack-cinder | 07:17 | |
| *** nkrinner has joined #openstack-cinder | 07:18 | |
| *** laughter_ has quit IRC | 07:18 | |
| *** thumpba has quit IRC | 07:18 | |
| *** jordanP has joined #openstack-cinder | 07:19 | |
| *** jordanP has quit IRC | 07:19 | |
| *** e0ne has joined #openstack-cinder | 07:20 | |
| *** JayConroy has quit IRC | 07:22 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder: Use object instead of string when reset backup https://review.openstack.org/347665 | 07:22 |
| *** JayConroy has joined #openstack-cinder | 07:23 | |
| *** salv-orlando has joined #openstack-cinder | 07:25 | |
| *** salv-orl_ has quit IRC | 07:25 | |
| *** alee_dinner has quit IRC | 07:26 | |
| *** laughterwym has joined #openstack-cinder | 07:26 | |
| *** laughter_ has joined #openstack-cinder | 07:28 | |
| *** laughte__ has joined #openstack-cinder | 07:29 | |
| *** laughterwym has quit IRC | 07:32 | |
| *** laughte__ has quit IRC | 07:32 | |
| *** laughter_ has quit IRC | 07:33 | |
| openstackgerrit | zheng yin proposed openstack/cinder: Add hacking for assertFalse https://review.openstack.org/347669 | 07:33 |
| *** alonma has joined #openstack-cinder | 07:34 | |
| *** laughterwym has joined #openstack-cinder | 07:34 | |
| *** hemnafk has quit IRC | 07:35 | |
| *** hemnafk has joined #openstack-cinder | 07:35 | |
| *** mtreinish has quit IRC | 07:36 | |
| *** laughter_ has joined #openstack-cinder | 07:36 | |
| *** mtreinish has joined #openstack-cinder | 07:36 | |
| *** lickitysplitted has quit IRC | 07:37 | |
| *** lickitysplitted has joined #openstack-cinder | 07:37 | |
| *** maestropandy has quit IRC | 07:38 | |
| *** alonma has quit IRC | 07:39 | |
| *** laughterwym has quit IRC | 07:39 | |
| *** lennyb has quit IRC | 07:40 | |
| *** avishay has joined #openstack-cinder | 07:40 | |
| *** lennyb has joined #openstack-cinder | 07:41 | |
| *** alonma has joined #openstack-cinder | 07:41 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO: Implement update_migrated_volume https://review.openstack.org/344333 | 07:42 |
| *** takedakn has joined #openstack-cinder | 07:42 | |
| *** alonma has quit IRC | 07:45 | |
| *** e0ne has quit IRC | 07:48 | |
| *** alonma has joined #openstack-cinder | 07:49 | |
| *** alonma has quit IRC | 07:53 | |
| *** ollie_ has quit IRC | 07:55 | |
| *** GB21 has quit IRC | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| *** ducttape_ has joined #openstack-cinder | 08:00 | |
| *** akapil has joined #openstack-cinder | 08:02 | |
| *** akapil has quit IRC | 08:03 | |
| *** akapil has joined #openstack-cinder | 08:03 | |
| *** wewe0901 has joined #openstack-cinder | 08:04 | |
| *** ducttape_ has quit IRC | 08:04 | |
| *** alonma has joined #openstack-cinder | 08:05 | |
| openstackgerrit | Satish Venkatasubramanian proposed openstack/cinder: Add volumes/summary API support. https://review.openstack.org/335067 | 08:05 |
| *** guy has joined #openstack-cinder | 08:05 | |
| *** abhitechie has quit IRC | 08:07 | |
| *** abhitechie has joined #openstack-cinder | 08:07 | |
| *** akapil has quit IRC | 08:07 | |
| *** alonma has quit IRC | 08:09 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| *** salv-orlando has quit IRC | 08:10 | |
| *** jdurgin has quit IRC | 08:11 | |
| *** akapil has joined #openstack-cinder | 08:11 | |
| *** salv-orlando has joined #openstack-cinder | 08:12 | |
| *** jdurgin has joined #openstack-cinder | 08:12 | |
| *** alonma has joined #openstack-cinder | 08:12 | |
| *** hoonetorg has quit IRC | 08:13 | |
| *** karthikp_ has quit IRC | 08:14 | |
| *** hoonetorg has joined #openstack-cinder | 08:17 | |
| *** salv-orlando has quit IRC | 08:17 | |
| *** avishay has quit IRC | 08:18 | |
| *** alonma has quit IRC | 08:19 | |
| *** alonma has joined #openstack-cinder | 08:21 | |
| *** alonma has quit IRC | 08:25 | |
| *** alee_dinner has joined #openstack-cinder | 08:25 | |
| *** alonma has joined #openstack-cinder | 08:28 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fail manage operations if service is down https://review.openstack.org/345245 | 08:30 |
| *** laughter_ has quit IRC | 08:30 | |
| *** avishay has joined #openstack-cinder | 08:32 | |
| *** alonma has quit IRC | 08:33 | |
| *** takedakn1 has joined #openstack-cinder | 08:33 | |
| *** takedakn has quit IRC | 08:33 | |
| *** takedakn1 is now known as takedakn | 08:33 | |
| *** alonma has joined #openstack-cinder | 08:36 | |
| *** e0ne has joined #openstack-cinder | 08:40 | |
| *** alonma has quit IRC | 08:40 | |
| *** lixiaoy1 has quit IRC | 08:42 | |
| *** alonma has joined #openstack-cinder | 08:43 | |
| *** yangyape_ has quit IRC | 08:45 | |
| *** EinstCra_ has quit IRC | 08:45 | |
| *** iceyao has quit IRC | 08:45 | |
| *** yangyapeng has joined #openstack-cinder | 08:46 | |
| *** iceyao has joined #openstack-cinder | 08:46 | |
| *** EinstCrazy has joined #openstack-cinder | 08:46 | |
| *** zhangjn has quit IRC | 08:47 | |
| *** EinstCrazy has quit IRC | 08:47 | |
| *** EinstCrazy has joined #openstack-cinder | 08:47 | |
| *** alonma has quit IRC | 08:48 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Refactor create, save, and destroy OVO methods https://review.openstack.org/315541 | 08:49 |
| openstackgerrit | Huang Zhiteng proposed openstack/cinder: DO NOT MERGE: Implement Stochastic Weight handler https://review.openstack.org/347697 | 08:49 |
| *** savihou has joined #openstack-cinder | 08:50 | |
| *** alonma has joined #openstack-cinder | 08:51 | |
| *** zhangjn has joined #openstack-cinder | 08:52 | |
| *** GB21 has joined #openstack-cinder | 08:53 | |
| *** alonma has quit IRC | 08:55 | |
| *** bks has joined #openstack-cinder | 08:56 | |
| *** ducttape_ has joined #openstack-cinder | 09:01 | |
| *** alonma has joined #openstack-cinder | 09:01 | |
| *** ducttape_ has quit IRC | 09:05 | |
| *** alonma has quit IRC | 09:06 | |
| *** laughterwym has joined #openstack-cinder | 09:07 | |
| *** avishay has quit IRC | 09:07 | |
| *** GB21 has quit IRC | 09:07 | |
| *** alonma has joined #openstack-cinder | 09:09 | |
| openstackgerrit | zheng yin proposed openstack/cinder: Add hacking for assertFalse https://review.openstack.org/347669 | 09:09 |
| *** Adam__ has joined #openstack-cinder | 09:10 | |
| Adam__ | Morning! Does anyone know the impact on existing volumes if I were to enable thin provisioning / lvm_type=thin? Is there a risk of losing existing containers? | 09:10 |
| *** GB21 has joined #openstack-cinder | 09:11 | |
| *** laughterwym has quit IRC | 09:11 | |
| *** sticker has quit IRC | 09:13 | |
| *** alonma has quit IRC | 09:14 | |
| *** sticker has joined #openstack-cinder | 09:15 | |
| *** EinstCrazy has quit IRC | 09:17 | |
| *** avishay has joined #openstack-cinder | 09:19 | |
| *** maestropandy has joined #openstack-cinder | 09:19 | |
| *** GB21 has quit IRC | 09:20 | |
| openstackgerrit | Dinesh Bhor proposed openstack/python-cinderclient: Use self.ks_logger instead of ks_logger https://review.openstack.org/347712 | 09:21 |
| *** EinstCra_ has joined #openstack-cinder | 09:25 | |
| Dinesh_Bhor | Can someone restore this patch ? https://review.openstack.org/#/c/131182/ | 09:25 |
| Dinesh_Bhor | I want to take it ahead. | 09:25 |
| *** EinstCra_ has quit IRC | 09:27 | |
| *** iceyao has quit IRC | 09:27 | |
| *** iceyao has joined #openstack-cinder | 09:28 | |
| *** EinstCrazy has joined #openstack-cinder | 09:28 | |
| e0ne | Dinesh_Bhor: I can do it but looks like community doesn't want to have it | 09:29 |
| e0ne | Dinesh_Bhor: it makes migrations code too complicated | 09:29 |
| Dinesh_Bhor | e0ne: It will be great if you restore the patch I will push a new patch over it and discuss with community. | 09:30 |
| e0ne | Dinesh_Bhor: ok. did you see "Need to re-think it and make cross-project" comment? | 09:30 |
| e0ne | Dinesh_Bhor: patch is restrored | 09:31 |
| *** nherciu has joined #openstack-cinder | 09:31 | |
| Dinesh_Bhor | e0ne: yah ..I have seen that. We are proposing this in glance as well. Lets see what comes next. | 09:32 |
| Dinesh_Bhor | e0ne: Thank you for restoring. | 09:32 |
| *** dansmith has quit IRC | 09:33 | |
| *** yangyapeng has quit IRC | 09:34 | |
| *** dansmith has joined #openstack-cinder | 09:35 | |
| *** dansmith is now known as Guest41069 | 09:35 | |
| *** yangyapeng has joined #openstack-cinder | 09:35 | |
| openstackgerrit | Abhishek Kekane proposed openstack/cinder: Allow archiving deleted rows to shadow tables, for performance https://review.openstack.org/131182 | 09:35 |
| openstackgerrit | lakshman proposed openstack/cinder: Support manage/unmanage and extra-specs in K2 https://review.openstack.org/346540 | 09:37 |
| *** iceyao has quit IRC | 09:37 | |
| e0ne | Dinesh_Bhor: IMO, it would be great if we can do it in oslo.db | 09:37 |
| Dinesh_Bhor | e0ne: ok, I will try to implement this suggestion. | 09:38 |
| e0ne | Dinesh_Bhor: thanks | 09:39 |
| *** yangyapeng has quit IRC | 09:40 | |
| *** EinstCra_ has joined #openstack-cinder | 09:42 | |
| *** iceyao has joined #openstack-cinder | 09:42 | |
| *** yangyapeng has joined #openstack-cinder | 09:42 | |
| *** EinstCrazy has quit IRC | 09:43 | |
| nikeshm | e0ne: hi do you know how to add boolean value while setting up key value pair in volume type | 09:45 |
| nikeshm | cinder type-key xyz set xxx=True | 09:46 |
| e0ne | nikeshm: did you try True or False? | 09:46 |
| nikeshm | yes tried, but in cinder extra-specs-list, it is showing {'xxx': 'True'} | 09:47 |
| nikeshm | how can we do {'xxx': True} | 09:48 |
| *** tangchen_ has quit IRC | 09:50 | |
| *** yuelongguang has quit IRC | 09:50 | |
| openstackgerrit | Merged openstack/os-brick: Remove useless info logging in check_valid_device https://review.openstack.org/343662 | 09:53 |
| *** alee_dinner has quit IRC | 09:54 | |
| *** maestropandy has quit IRC | 09:57 | |
| *** chlong_POffice has quit IRC | 09:57 | |
| *** alee_dinner has joined #openstack-cinder | 09:59 | |
| *** alonma has joined #openstack-cinder | 10:03 | |
| *** tangchen_ has joined #openstack-cinder | 10:04 | |
| *** anshprat has quit IRC | 10:05 | |
| *** hwcomcn has joined #openstack-cinder | 10:06 | |
| *** mvk has quit IRC | 10:07 | |
| *** alonma has quit IRC | 10:08 | |
| *** alonma has joined #openstack-cinder | 10:10 | |
| *** chlong_POffice has joined #openstack-cinder | 10:11 | |
| *** yangyapeng has quit IRC | 10:12 | |
| *** EinstCra_ has quit IRC | 10:13 | |
| *** EinstCrazy has joined #openstack-cinder | 10:13 | |
| *** alonma has quit IRC | 10:15 | |
| *** alonma has joined #openstack-cinder | 10:18 | |
| *** alonma has quit IRC | 10:22 | |
| *** alonma has joined #openstack-cinder | 10:25 | |
| *** maestropandy has joined #openstack-cinder | 10:27 | |
| *** lprice1 has quit IRC | 10:29 | |
| *** lprice has joined #openstack-cinder | 10:29 | |
| *** ociuhandu has joined #openstack-cinder | 10:30 | |
| *** alonma has quit IRC | 10:30 | |
| *** alonma has joined #openstack-cinder | 10:33 | |
| *** alonma has quit IRC | 10:37 | |
| *** mvk has joined #openstack-cinder | 10:38 | |
| *** alonma has joined #openstack-cinder | 10:40 | |
| *** takedakn has quit IRC | 10:45 | |
| *** alonma has quit IRC | 10:45 | |
| *** akapil has quit IRC | 10:48 | |
| *** maestropandy has quit IRC | 10:51 | |
| winston-d | nikeshm: try 'cinder key-key xyz set xxx=<is>True' | 10:53 |
| *** akapil has joined #openstack-cinder | 11:04 | |
| *** tpsilva has joined #openstack-cinder | 11:05 | |
| *** akapil has quit IRC | 11:09 | |
| *** salv-orlando has joined #openstack-cinder | 11:09 | |
| *** gouthamr has joined #openstack-cinder | 11:14 | |
| *** akapil has joined #openstack-cinder | 11:16 | |
| *** gouthamr_ has joined #openstack-cinder | 11:21 | |
| *** gouthamr has quit IRC | 11:24 | |
| *** takedakn has joined #openstack-cinder | 11:25 | |
| openstackgerrit | Alyson proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages https://review.openstack.org/310785 | 11:25 |
| *** sdague has joined #openstack-cinder | 11:30 | |
| *** gouthamr_ is now known as gouthamr | 11:30 | |
| *** takedakn has quit IRC | 11:32 | |
| *** ociuhandu has quit IRC | 11:33 | |
| *** ociuhandu has joined #openstack-cinder | 11:35 | |
| *** cdelatte has joined #openstack-cinder | 11:48 | |
| *** abhitechie has quit IRC | 11:50 | |
| *** erlon has joined #openstack-cinder | 11:51 | |
| *** savihou has quit IRC | 11:52 | |
| *** savihou has joined #openstack-cinder | 11:53 | |
| *** takedakn has joined #openstack-cinder | 11:54 | |
| gouthamr | e0ne: hi! when you have time, could you please review https://review.openstack.org/#/c/346093 - I had your +2/+W, but it needed rebasing and minor change at the gate. | 12:00 |
| *** Lee1092 has quit IRC | 12:03 | |
| *** ducttape_ has joined #openstack-cinder | 12:03 | |
| *** ducttape_ has quit IRC | 12:05 | |
| *** ducttape_ has joined #openstack-cinder | 12:06 | |
| gouthamr | e0ne: thanks! | 12:07 |
| e0ne | gouthamr: you're welcome | 12:07 |
| *** andymaier has joined #openstack-cinder | 12:12 | |
| *** savihou has quit IRC | 12:14 | |
| *** timcl has joined #openstack-cinder | 12:14 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix 'driver does not honor clone/snapshot' bugs https://review.openstack.org/336092 | 12:15 |
| *** maestropandy has joined #openstack-cinder | 12:16 | |
| openstackgerrit | Alyson proposed openstack/cinder: HNAS: Add support for manage/unmanage snapshots in NFS driver https://review.openstack.org/333992 | 12:16 |
| *** laughter_ has joined #openstack-cinder | 12:19 | |
| *** akerr has joined #openstack-cinder | 12:21 | |
| *** chlong_POffice has quit IRC | 12:21 | |
| *** ducttape_ has quit IRC | 12:22 | |
| *** akerr_ has joined #openstack-cinder | 12:23 | |
| *** sdake has joined #openstack-cinder | 12:23 | |
| *** alee_dinner is now known as alee | 12:24 | |
| *** gouthamr has quit IRC | 12:24 | |
| *** hwcomcn has quit IRC | 12:25 | |
| *** yangyapeng has joined #openstack-cinder | 12:25 | |
| *** akerr has quit IRC | 12:26 | |
| *** alonma has joined #openstack-cinder | 12:28 | |
| *** chlong has joined #openstack-cinder | 12:29 | |
| *** e0ne has quit IRC | 12:31 | |
| *** akerr_ is now known as akerr | 12:31 | |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Revmove extra 'u' from cli output https://review.openstack.org/347804 | 12:33 |
| *** alonma has quit IRC | 12:33 | |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Revmove extra 'u' from cli output https://review.openstack.org/347804 | 12:34 |
| *** beagles has joined #openstack-cinder | 12:35 | |
| *** gouthamr has joined #openstack-cinder | 12:36 | |
| *** alonma has joined #openstack-cinder | 12:37 | |
| *** JoseMello has joined #openstack-cinder | 12:38 | |
| *** EinstCrazy has quit IRC | 12:40 | |
| *** iceyao has quit IRC | 12:41 | |
| *** alonma has quit IRC | 12:42 | |
| *** abhitechie has joined #openstack-cinder | 12:46 | |
| *** pauloewerton has joined #openstack-cinder | 12:47 | |
| *** beagles has left #openstack-cinder | 12:54 | |
| *** sheel has quit IRC | 12:56 | |
| openstackgerrit | zheng yin proposed openstack/cinder: Add hacking for assertFalse https://review.openstack.org/347669 | 13:01 |
| *** avishay has quit IRC | 13:05 | |
| *** coolsvap is now known as coolsvap_ | 13:07 | |
| *** links has quit IRC | 13:08 | |
| *** rlrossit has joined #openstack-cinder | 13:11 | |
| *** timcl has quit IRC | 13:12 | |
| *** faiz89 has joined #openstack-cinder | 13:12 | |
| *** merooney has joined #openstack-cinder | 13:14 | |
| Srinu | patrickeast: hi | 13:16 |
| *** porrua has joined #openstack-cinder | 13:17 | |
| *** edmondsw has joined #openstack-cinder | 13:17 | |
| Srinu | what is version of zuul, jenkins and nodepool in your CI configuration | 13:17 |
| *** liverpooler has quit IRC | 13:17 | |
| *** jungleboyj has joined #openstack-cinder | 13:18 | |
| *** dustins has joined #openstack-cinder | 13:21 | |
| *** avishay has joined #openstack-cinder | 13:21 | |
| *** adisky has quit IRC | 13:21 | |
| openstackgerrit | zheng yin proposed openstack/cinder: Use assertEqual() instead of assertDictEqual() https://review.openstack.org/346746 | 13:21 |
| *** akshai has joined #openstack-cinder | 13:21 | |
| *** gouthamr has quit IRC | 13:22 | |
| *** Adam__ has quit IRC | 13:22 | |
| *** gouthamr has joined #openstack-cinder | 13:23 | |
| *** pgadiya has quit IRC | 13:23 | |
| *** crose has joined #openstack-cinder | 13:24 | |
| *** xyang1 has joined #openstack-cinder | 13:24 | |
| *** gman-tx has joined #openstack-cinder | 13:25 | |
| *** timcl has joined #openstack-cinder | 13:26 | |
| gouthamr | looking for a +2/+W, anyone? :) https://review.openstack.org/#/c/346093/ | 13:27 |
| *** Yogi1 has joined #openstack-cinder | 13:27 | |
| smcginnis | jgriffith, hemnafk: I do need to request branches for the "stable-driver" branches we discussed though. | 13:29 |
| flip214 | what should I do if "tools/lintstack.head.py generate" dies with "AttributeError: 'NoneType' object has no attribute 'groups'" ? | 13:30 |
| smcginnis | panic | 13:30 |
| smcginnis | JK :) | 13:30 |
| smcginnis | flip214: Where did you see that? | 13:30 |
| *** mriedem has joined #openstack-cinder | 13:31 | |
| *** david-lyle_ has joined #openstack-cinder | 13:31 | |
| *** david-lyle_ has quit IRC | 13:31 | |
| *** ebalduf has joined #openstack-cinder | 13:33 | |
| gouthamr | smcginnis: thank you.. | 13:33 |
| smcginnis | gouthamr: np | 13:34 |
| flip214 | smcginnis: running that command locally, because of http://logs.openstack.org/70/347270/1/check/gate-cinder-pylint/7bfdd37/console.html | 13:34 |
| smcginnis | Is gerrit extremely slow for anyone else right now? | 13:34 |
| *** david-lyle has quit IRC | 13:34 | |
| *** ducttape_ has joined #openstack-cinder | 13:36 | |
| *** alonma has joined #openstack-cinder | 13:39 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Fix retype when storage policy is enabled https://review.openstack.org/337672 | 13:39 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Add volume name in vCenter to conn info https://review.openstack.org/344069 | 13:40 |
| *** abhinavtechie has joined #openstack-cinder | 13:41 | |
| *** maestropandy has left #openstack-cinder | 13:42 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX driver - Cleanup of Initiator group fails https://review.openstack.org/347838 | 13:42 |
| *** aagrawal has joined #openstack-cinder | 13:43 | |
| *** abhitechie has quit IRC | 13:43 | |
| *** yuriy_n17 has joined #openstack-cinder | 13:43 | |
| *** alonma has quit IRC | 13:43 | |
| *** abhinavtechie has quit IRC | 13:45 | |
| *** alonma has joined #openstack-cinder | 13:46 | |
| *** Guest41069 is now known as dansmith | 13:48 | |
| nikeshm | winston-d: cinder type-key xyz set xxx=<is>True -bash: is: No such file or directory | 13:48 |
| *** aagrawal has quit IRC | 13:48 | |
| smcginnis | FYI - gerrit restart coming. Brace yourselves. :) | 13:51 |
| *** alonma has quit IRC | 13:51 | |
| -openstackstatus- NOTICE: Gerrit is being restarted now to relieve performance degradation | 13:51 | |
| *** alonma has joined #openstack-cinder | 13:54 | |
| *** takedakn has quit IRC | 13:55 | |
| *** jgregor has joined #openstack-cinder | 13:56 | |
| *** Fdaisuke has joined #openstack-cinder | 13:57 | |
| *** alonma has quit IRC | 13:58 | |
| *** avishay has quit IRC | 14:00 | |
| *** alonma has joined #openstack-cinder | 14:03 | |
| *** jwcroppe has joined #openstack-cinder | 14:03 | |
| *** alonma has quit IRC | 14:08 | |
| flip214 | BTW, what's the right solution if Python 2.7 "long" doesn't exist in Python 3 any more? I saw some guides telling to have a version check with "long = int"... (ouch) | 14:10 |
| jgriffith | smcginnis: LOL | 14:10 |
| jgriffith | smcginnis: I don't think we're going to go the branch path at this time | 14:10 |
| *** alonma has joined #openstack-cinder | 14:10 | |
| jgriffith | smcginnis: unless others want to fight that battle, but I don't | 14:11 |
| bswartz | nikeshm: use quotes | 14:11 |
| *** aswadr_ has joined #openstack-cinder | 14:11 | |
| *** yuelongguang has joined #openstack-cinder | 14:11 | |
| smcginnis | jgriffith: I know the feature branch idea got shot down. | 14:11 |
| bswartz | cinder type-key xyz set "xxx=<is> True" | 14:12 |
| jgriffith | smcginnis: gaahhhh | 14:12 |
| jgriffith | sorry | 14:12 |
| smcginnis | But the other one we talked about was having stable branches that were a little more open and a little longer lived for driver bug backports. | 14:12 |
| jgriffith | didn't read close enough | 14:12 |
| jgriffith | YES please on the other branches we talked about | 14:12 |
| smcginnis | Yeah, no worries. I figured that was the case. :) | 14:12 |
| nikeshm | bswartz: cinder type-key xyz set xxx='<is>True' | 14:12 |
| smcginnis | That does seem to address some of the concern for wanting drivers out of tree. | 14:12 |
| *** bks has quit IRC | 14:13 | |
| *** avishay has joined #openstack-cinder | 14:13 | |
| jgriffith | smcginnis: well at least the ones for people that are just worried about supporting them long term | 14:13 |
| bswartz | smcginnis: who's shooting this stuff down? | 14:13 |
| nikeshm | bswartz: {'xyz': 'True'} | 14:13 |
| jgriffith | which is probably good | 14:13 |
| jgriffith | I keep trying to like gertty but I just can't get myself to do it | 14:14 |
| smcginnis | bswartz: The first thing we talked about for feature branches was whether that could be used for redesigning some of our cinder-nova interaction. | 14:14 |
| smcginnis | bswartz: But then when we synced up with nova they weren't interested in that. They just wanted to use microversions and the regular process. | 14:15 |
| *** alonma has quit IRC | 14:15 | |
| johnthetubaguy | its more we have never had a successful merge back into the tree, its probably best to just do WIP patches, unless that gets super hard for some other reason? | 14:16 |
| bswartz | smcginnis: that makes sense -- the feature branch idea only came up because we thought nova wouldn't test with unmerged code | 14:16 |
| johnthetubaguy | depends-on has made life a lot easier with these kinds of things | 14:17 |
| smcginnis | johnthetubaguy: True | 14:17 |
| jgriffith | smcginnis: bswartz FWIW I say merge the cinder code, I wrote it specifically so it could be merged without impacting anything | 14:17 |
| smcginnis | My only concern is the constant rebasing and version # changing on microversions. | 14:18 |
| jgriffith | in theory at least :) | 14:18 |
| smcginnis | jgriffith: +1 | 14:18 |
| *** cknight has joined #openstack-cinder | 14:18 | |
| *** alonma has joined #openstack-cinder | 14:18 | |
| johnthetubaguy | you can just pick a really high one, and pick the real one when you are ready to merge | 14:18 |
| bswartz | jgriffith: what if nova finds a bug after its merged and you have to change the API? | 14:18 |
| openstackgerrit | Philipp Marek proposed openstack/cinder: DRBD: Disk-options for new resources https://review.openstack.org/347267 | 14:18 |
| openstackgerrit | Philipp Marek proposed openstack/cinder: DRBD: Don't put clients everywhere https://review.openstack.org/347268 | 14:18 |
| openstackgerrit | Philipp Marek proposed openstack/cinder: DRBD: Make calculation use more bits https://review.openstack.org/347269 | 14:18 |
| openstackgerrit | Philipp Marek proposed openstack/cinder: DRBD: Use the ClientHelper class to reduce amount of code https://review.openstack.org/347270 | 14:18 |
| openstackgerrit | Philipp Marek proposed openstack/cinder: DRBD: Avoid an extra resize later on https://review.openstack.org/347271 | 14:18 |
| openstackgerrit | Philipp Marek proposed openstack/cinder: DRBD: Do batch delivery https://review.openstack.org/347272 | 14:18 |
| openstackgerrit | Philipp Marek proposed openstack/cinder: DRBD: Report an error if a snapshot has vanished https://review.openstack.org/347273 | 14:18 |
| openstackgerrit | Philipp Marek proposed openstack/cinder: DRBD: Avoid duplicated volumes from the error path https://review.openstack.org/347274 | 14:19 |
| smcginnis | Dang | 14:19 |
| *** akapil_ has joined #openstack-cinder | 14:19 | |
| smcginnis | bswartz: Version bump. | 14:19 |
| jgriffith | I'd rather merge it and iterate on it in tree than branches or anything else. johnthetubaguy yeah, I like the idea of just picking some non-existent micro-version that it'll never see. But then again I hate micro-versions... so there's that | 14:19 |
| smcginnis | :) | 14:19 |
| johnthetubaguy | jgriffith: but the alternatives are really way more terrible | 14:20 |
| jgriffith | bswartz: I think you're making this more difficult than it needs to be. This isn't a "released" thing | 14:20 |
| jgriffith | johnthetubaguy: meh... perhaps | 14:20 |
| jgriffith | johnthetubaguy: I still dislike them :) | 14:20 |
| johnthetubaguy | the experimental header is what I wanted to allow iteration in tree | 14:20 |
| jgriffith | johnthetubaguy: and that's not going to change | 14:20 |
| jgriffith | johnthetubaguy: bswartz smcginnis so this case is the one case that we felt the branch model would work well for | 14:21 |
| jgriffith | I have to admit though, the more I think about it the more I wonder "who" if anybody would be pulling that and running it other than dev teams anyway | 14:21 |
| johnthetubaguy | its just the merge back into master will be painful, and you can just checkout the WIP patches | 14:21 |
| *** akapil has quit IRC | 14:22 | |
| *** rdx_matt has quit IRC | 14:22 | |
| jgriffith | johnthetubaguy: I get it, for sure. Don't get me wrong, I'm not debating it | 14:22 |
| *** alonma has quit IRC | 14:22 | |
| jgriffith | johnthetubaguy: if I had a better feel for exactly how it would make life better that might be different | 14:22 |
| *** rdx_matt has joined #openstack-cinder | 14:22 | |
| *** rlrossit has quit IRC | 14:23 | |
| *** akapil_ has quit IRC | 14:23 | |
| johnthetubaguy | I mean if a branch is better, I am all for it, but yeah, not sure it helps here | 14:23 |
| *** lprice has quit IRC | 14:23 | |
| jgriffith | johnthetubaguy: yeah... I think we both are kinda in the same place on that. | 14:23 |
| *** andymaier has quit IRC | 14:25 | |
| *** alonma has joined #openstack-cinder | 14:25 | |
| avishay | the CI instability is killing me... at the end of a 6 hour run in the gate hit the flaky unit test bug. now did "reverify no bug" and it's now in the check queue for some reason... :-/ | 14:26 |
| smcginnis | avishay: Not sure why it gets booted to the check queue some times. Yeah - pretty painful right now. | 14:27 |
| *** liverpooler has joined #openstack-cinder | 14:27 | |
| *** rdx_matt has quit IRC | 14:27 | |
| avishay | smcginnis: bummer | 14:28 |
| *** alonma has quit IRC | 14:30 | |
| *** alonma has joined #openstack-cinder | 14:33 | |
| *** diablo_rojo has joined #openstack-cinder | 14:33 | |
| *** alee has quit IRC | 14:34 | |
| *** alonma has quit IRC | 14:38 | |
| *** cgalan has joined #openstack-cinder | 14:39 | |
| *** alonma has joined #openstack-cinder | 14:41 | |
| *** xiexs_ has quit IRC | 14:41 | |
| *** ebalduf has quit IRC | 14:42 | |
| *** pbandark has quit IRC | 14:45 | |
| *** alonma has quit IRC | 14:46 | |
| *** xinli has joined #openstack-cinder | 14:46 | |
| *** bardia has joined #openstack-cinder | 14:46 | |
| *** alonma has joined #openstack-cinder | 14:48 | |
| *** abhitechie has joined #openstack-cinder | 14:48 | |
| *** enriquetaso has joined #openstack-cinder | 14:49 | |
| *** ebalduf has joined #openstack-cinder | 14:50 | |
| *** bardia has quit IRC | 14:51 | |
| smcginnis | _alastor_: Can you give me that ci-status command line you used again. When you have a moment. | 14:51 |
| *** alonma has quit IRC | 14:53 | |
| *** lprice has joined #openstack-cinder | 14:55 | |
| Dinesh_Bhor | smcginnis: Hi, Could you please take a look at it ? https://review.openstack.org/#/c/347712/ | 14:57 |
| smcginnis | Dinesh_Bhor: Sure, got it open in a tab. | 14:58 |
| *** nkrinner has quit IRC | 14:58 | |
| *** coolsvap_ is now known as coolsvap | 14:59 | |
| _alastor_ | smcginnis: Sure here you go: third-party-ci-tools/monitoring/ci-status/ci-status.py --all -t 2 --is-reporting --failures --number-of-reports | 14:59 |
| _alastor_ | smcginnis: And here's my .gerritqueryrc | 14:59 |
| *** jistr is now known as jistr|call | 15:00 | |
| _alastor_ | smcginnis: http://pastebin.com/raw/zrh14X9T | 15:00 |
| Dinesh_Bhor | smcginnis: Thank you so much | 15:00 |
| *** nkrinner has joined #openstack-cinder | 15:00 | |
| smcginnis | _alastor_: Awesome, thanks. | 15:01 |
| _alastor_ | smcginnis: Just remember to run with just the --scrape-wiki and --force flags first to make sure your cache is updated | 15:01 |
| *** alonma has joined #openstack-cinder | 15:04 | |
| *** arecknag has quit IRC | 15:06 | |
| *** nkrinner has quit IRC | 15:08 | |
| *** alonma has quit IRC | 15:09 | |
| *** liverpooler has quit IRC | 15:10 | |
| xinli | @smcginnis: I hit one issue, after pulling the master tree, and kick off "git -e py27", the unit test hang on for 30 minutes, how can I debug the issue? http://paste.openstack.org/show/542611/ | 15:12 |
| xinli | @smcginnis: this is just a pure unit test, only test my sandbox from master tree, no any ptach code. | 15:13 |
| *** mtanino has joined #openstack-cinder | 15:14 | |
| JayConroy | :q | 15:15 |
| *** pcaruana has quit IRC | 15:17 | |
| smcginnis | xinli: There are issues that are being looked into with hanging unit tests. Just Ctrl-C and try again. | 15:17 |
| smcginnis | Yay, privsep patch is on its way! https://review.openstack.org/344450 | 15:18 |
| *** laughter_ has quit IRC | 15:18 | |
| xinli | @smcginnis: got it, thanks | 15:19 |
| *** laughterwym has joined #openstack-cinder | 15:19 | |
| *** laughterwym has quit IRC | 15:19 | |
| *** nkrinner has joined #openstack-cinder | 15:22 | |
| xinli | @smcginnis: I take a look at this one: https://review.openstack.org/#/c/344450/, does it relate to the unit test failure? | 15:22 |
| *** iceyao has joined #openstack-cinder | 15:22 | |
| *** alonma has joined #openstack-cinder | 15:23 | |
| *** rcernin has quit IRC | 15:24 | |
| *** yangyapeng has quit IRC | 15:26 | |
| smcginnis | xinli: No, that's not related. | 15:26 |
| *** GB21 has joined #openstack-cinder | 15:27 | |
| xinli | @smcginnis: now re-run the cmd: "tox -e py27" | 15:27 |
| *** alonma has quit IRC | 15:27 | |
| smcginnis | yes | 15:28 |
| *** mjura has quit IRC | 15:28 | |
| xinli | @smcginnis: this time , it passed, how can I avoid this situation? | 15:30 |
| *** alonma has joined #openstack-cinder | 15:31 | |
| *** iceyao has quit IRC | 15:32 | |
| *** yangyapeng has joined #openstack-cinder | 15:33 | |
| *** ebalduf has quit IRC | 15:34 | |
| openstackgerrit | Merged openstack/cinder: NetApp: Refactor unit tests https://review.openstack.org/346093 | 15:34 |
| *** jistr|call is now known as jistr | 15:35 | |
| *** alonma has quit IRC | 15:35 | |
| *** ntpttr__ has joined #openstack-cinder | 15:35 | |
| *** ducttape_ has quit IRC | 15:37 | |
| *** Lee1092 has joined #openstack-cinder | 15:37 | |
| *** alonma has joined #openstack-cinder | 15:38 | |
| *** karthikp_ has joined #openstack-cinder | 15:39 | |
| *** bardia has joined #openstack-cinder | 15:39 | |
| *** rdx_matt has joined #openstack-cinder | 15:40 | |
| *** rdx_matt has quit IRC | 15:40 | |
| *** rdx_matt has joined #openstack-cinder | 15:40 | |
| *** faiz89 has quit IRC | 15:42 | |
| *** alonma has quit IRC | 15:43 | |
| *** david-lyle has joined #openstack-cinder | 15:43 | |
| *** liverpooler has joined #openstack-cinder | 15:43 | |
| *** diogogmt has joined #openstack-cinder | 15:44 | |
| *** xyang has joined #openstack-cinder | 15:44 | |
| *** alonma has joined #openstack-cinder | 15:46 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Use Live Volume for replication https://review.openstack.org/329531 | 15:48 |
| *** alonma has quit IRC | 15:50 | |
| Swanson | Infra queue is in the lowish 200s today! | 15:53 |
| *** alonma has joined #openstack-cinder | 15:54 | |
| *** jwcroppe has quit IRC | 15:55 | |
| *** cgalan has quit IRC | 15:57 | |
| *** hemnafk is now known as hemna | 15:57 | |
| *** alonma has quit IRC | 15:58 | |
| bardia | can anyone find the failure in this: http://logs.openstack.org/38/346238/4/check/gate-cinder-python27-db-ubuntu-xenial/6ea9dd0/console.html? | 16:00 |
| *** leeantho has joined #openstack-cinder | 16:00 | |
| *** e0ne has joined #openstack-cinder | 16:01 | |
| *** tyrola has joined #openstack-cinder | 16:05 | |
| *** avishay has quit IRC | 16:05 | |
| *** tyrola_ has joined #openstack-cinder | 16:10 | |
| *** tyrola_ has quit IRC | 16:11 | |
| *** ducttape_ has joined #openstack-cinder | 16:12 | |
| *** jwcroppe has joined #openstack-cinder | 16:13 | |
| *** itlinux has quit IRC | 16:13 | |
| *** tyrola has quit IRC | 16:13 | |
| *** rdx_matt has quit IRC | 16:14 | |
| *** nkrinner is now known as nkrinner_afk | 16:14 | |
| *** slunkad_ has quit IRC | 16:15 | |
| *** sayalilunkad has joined #openstack-cinder | 16:17 | |
| *** cgalan has joined #openstack-cinder | 16:21 | |
| *** sayalilunkad has quit IRC | 16:25 | |
| *** Apoorva has joined #openstack-cinder | 16:25 | |
| *** crose has quit IRC | 16:26 | |
| *** david-lyle has quit IRC | 16:28 | |
| *** Thambirajan has joined #openstack-cinder | 16:32 | |
| *** Gao has quit IRC | 16:36 | |
| *** Lallana has joined #openstack-cinder | 16:37 | |
| *** sayalilunkad has joined #openstack-cinder | 16:38 | |
| *** mvk has quit IRC | 16:39 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Splitting Out Connectors from connector.py https://review.openstack.org/307974 | 16:40 |
| *** Suyi has joined #openstack-cinder | 16:45 | |
| karthikp_ | Please could somebody review my patch: https://review.openstack.org/#/c/332480/ | 16:48 |
| openstackgerrit | Jay Conroy proposed openstack/cinder: Fix volume upload-to-image for vhd disk-format https://review.openstack.org/347944 | 16:48 |
| JayConroy | Could I get reviews on my patch please? https://review.openstack.org/#/c/347944/ Thanks! :) | 16:50 |
| *** akapil has joined #openstack-cinder | 16:51 | |
| *** Apoorva_ has joined #openstack-cinder | 16:53 | |
| *** alonma has joined #openstack-cinder | 16:54 | |
| *** Apoorva has quit IRC | 16:56 | |
| *** akapil has quit IRC | 16:56 | |
| hemna | _alastor_, so what field(s) should I add ? | 16:58 |
| *** e0ne has quit IRC | 16:58 | |
| *** alonma has quit IRC | 16:58 | |
| hemna | _alastor_, so for our CI the name is "HPE Storage CI", the wiki page/name is HPE_Storage_CI, the gerrit account is 'hp-cinder-ci' | 17:00 |
| _alastor_ | hemna: So when doing Gerrit query filters, it's really useful to have the account that posts for the CI as well as the exact job name for the CI. Eg. datera-ci is my account name, datera-dsvm-full is the job name | 17:00 |
| hemna | can't we lookup the job name ? | 17:00 |
| _alastor_ | hemna: That's what I was using the wiki for | 17:00 |
| hemna | I'd rather not put a bunch of attributes in each drier | 17:00 |
| *** nherciu has quit IRC | 17:00 | |
| hemna | but just exactly the required minimum field or fields. | 17:01 |
| hemna | 1 or 2 | 17:01 |
| _alastor_ | hemna: Ideally it would just be some sort of dictionary containing the necessary keys | 17:01 |
| _alastor_ | hemna: but job name is more useful than account name because one account can post for multiple jobs | 17:01 |
| hemna | can we find everything from the gerrit account id + CI name " | 17:01 |
| hemna | like | 17:01 |
| hemna | CI_GERRIT_ID = 'hp-cinder-ci' | 17:01 |
| hemna | CI_NAME = 'HPE Storage CI' | 17:01 |
| hemna | aren't the job names defined in the infra somewhere ? | 17:02 |
| _alastor_ | hemna: Not any place that isn't as out of date as the wiki :) | 17:02 |
| smcginnis | job names can be whatever you want to call them. | 17:02 |
| hemna | smcginnis, sure, but I'm trying to find out if they can be looked up base on gerrit account id and or CI name | 17:02 |
| hemna | so we don't have to put a bunch of shit in each driver | 17:02 |
| smcginnis | Not that I'm aware of. | 17:03 |
| hemna | which also has to be maintained on the wiki, etc. | 17:03 |
| _alastor_ | hemna: I agree that they should be tracked automagically somewhere by infra, but I haven't found any place with that info yet | 17:03 |
| smcginnis | We can just have the script access the CI logs for each job and grep the c-vol log for to see which driver is loaded. :D | 17:03 |
| * smcginnis is joking if that wasn't clear | 17:04 | |
| hemna | the problem is that the job names can change | 17:04 |
| _alastor_ | smcginnis: HA, because I was having a bit of a heart attack :) | 17:04 |
| hemna | based upon the internal needs of the CI network, etc | 17:04 |
| hemna | we change our job names | 17:04 |
| hemna | because we move it to a new host, or array etc | 17:04 |
| smcginnis | Honestly though, that's probably the only consistent and accurate way to tell what any given job is testing. :/ | 17:04 |
| hemna | we test multiple arrays (different firmware, configurations etc) | 17:04 |
| smcginnis | bbl | 17:05 |
| hemna | _alastor_, so I suppose the job names can be put on the wiki | 17:06 |
| *** faiz89 has joined #openstack-cinder | 17:06 | |
| _alastor_ | hemna: That would certainly be nice | 17:06 |
| hemna | the driver has the wiki/CI name | 17:06 |
| hemna | the wiki has the gerrit account name | 17:06 |
| hemna | so can't we just put the driver CI wiki name ? | 17:07 |
| hemna | CI_NAME = 'HPE_Storage_CI" | 17:07 |
| hemna | which is the exact name of | 17:07 |
| hemna | https://wiki.openstack.org/wiki/ThirdPartySystems/<CI_NAME> | 17:07 |
| _alastor_ | hemna: That's pretty close to what I currently do, but it still requires keeping the wiki up-to-date for all drivers | 17:07 |
| hemna | we'd have 2 drivers that point to that CI_NAME | 17:08 |
| hemna | would it make sense | 17:08 |
| _alastor_ | hemna: We also discussed a bit of this at the mid-cycle. Adding the mapping to the wiki would mean I could just get everything I need from there | 17:08 |
| hemna | to have some mapping in tree | 17:08 |
| hemna | I'd rather have 1 place to keep all the shit up to date | 17:09 |
| hemna | otherwise it'll always be out of sync | 17:09 |
| *** rdx_matt has joined #openstack-cinder | 17:10 | |
| _alastor_ | hemna: Apparently adding a "Jobs" field to the wiki requires changing the template in infra | 17:10 |
| _alastor_ | hemna: which I hear is a pita | 17:10 |
| *** rdx_matt has quit IRC | 17:11 | |
| hemna | ugh | 17:11 |
| *** rdx_matt has joined #openstack-cinder | 17:11 | |
| _alastor_ | But adding it would mean I have everything I need, provided it's up-to-date | 17:11 |
| hemna | theoretically, we could have a mapping in tree | 17:12 |
| hemna | for all of the info | 17:12 |
| hemna | but...that requires updating, etc. | 17:12 |
| hemna | so you need the job names to track how often they are reporting pass/fail? | 17:12 |
| *** tesseract- has quit IRC | 17:13 | |
| *** ebalduf has joined #openstack-cinder | 17:13 | |
| _alastor_ | hemna: I need job names so I can separate results for CIs that report multiple jobs on the same account | 17:13 |
| _alastor_ | hemna: Otherwise they get lumped together and the results look weird | 17:14 |
| hemna | ok | 17:14 |
| *** links has joined #openstack-cinder | 17:15 | |
| *** sdake has quit IRC | 17:15 | |
| _alastor_ | hemna: Here's my current .gerritquerycache file, so you can see what I scrape: http://paste.openstack.org/show/542670/ | 17:15 |
| DuncanT | At least if it's in tree we can see it's updated during driver addition/removal. I prefer that to the wiki (and there's talk of infra dumping the wiki anyway) | 17:16 |
| *** sandanar has quit IRC | 17:16 | |
| _alastor_ | DuncanT: I agree it should be in tree, as long as it has comparable fields to the wiki | 17:17 |
| DuncanT | Yup. Which might come back to the yaml file. We'll know when it's out of date since the tool will flag the missing CI reports | 17:18 |
| patrickeast | +1 for in tree, if we pick a yaml or json file with required fields we can validate changes to it too | 17:18 |
| patrickeast | force all the fields to be there | 17:18 |
| * patrickeast is just catching up on irc from the meeting and ealier | 17:19 | |
| DuncanT | I can turn a gerritquerycache into yaml easily enough | 17:19 |
| _alastor_ | DuncanT: the field that's missing from the gerritquerycache is the job field, because I had no place to get it :) | 17:20 |
| *** david-lyle has joined #openstack-cinder | 17:20 | |
| DuncanT | _alastor_: Ok, I don't mind trying to fill that in by hand once | 17:21 |
| *** david-lyle has quit IRC | 17:23 | |
| *** david-lyle_ has joined #openstack-cinder | 17:23 | |
| *** david-lyle_ is now known as david-lyle | 17:24 | |
| *** david-lyle_ has joined #openstack-cinder | 17:25 | |
| erlon | _alastor_: not followed the above chatting, but have you considered to have a standard to the job names? | 17:26 |
| erlon | _alastor_: for example, the NFs CI JOB wound need to be: NfsDriver-dsvm | 17:26 |
| _alastor_ | erlon: I'm fine with a standard, but it seems like hemna's CI changes job name based on location? | 17:26 |
| patrickeast | erlon: i think trying to enforce that would be wayyyy harder than just keeping a map of what ci systems have for job names | 17:27 |
| *** ebalduf has quit IRC | 17:27 | |
| erlon | _alastor_: then would be HPE3PARFCDriver-mylocation | 17:27 |
| erlon | patrickeast: but way more reliable | 17:28 |
| patrickeast | erlon: yea, if everyone would go change their jobs names | 17:28 |
| erlon | patrickeast: we have enforced people to use CIs which has proven to be hard but possible | 17:28 |
| patrickeast | eh sort of | 17:29 |
| *** david-lyle has quit IRC | 17:29 | |
| * patrickeast might just have a skewed outlook on it | 17:29 | |
| erlon | patrickeast: we create a list based in the names, if the maintainer does not change the name, than the Driver will be show in the removal list | 17:29 |
| *** Apoorva_ has quit IRC | 17:30 | |
| patrickeast | yea, but by when? | 17:30 |
| *** Apoorva has joined #openstack-cinder | 17:30 | |
| patrickeast | if we do it now, to give fair warning it wouldn't be till like N-3 when we yank them | 17:30 |
| patrickeast | which would probably mean early O | 17:31 |
| patrickeast | maybe we do both | 17:31 |
| *** xyang has quit IRC | 17:31 | |
| patrickeast | start forcing a naming convention | 17:31 |
| patrickeast | *and* start a mapping | 17:31 |
| erlon | patrickeast: the removal policy its something to yet be defined, but I don't think that changing the jobs names is that hard | 17:31 |
| _alastor_ | +1 ideal case :) | 17:31 |
| *** ebalduf has joined #openstack-cinder | 17:31 | |
| smcginnis | We haven't even been able to get everone to adopt our consistent recheck triggers. | 17:32 |
| _alastor_ | smcginnis: Although that's way harder to verify correctness on with how flakey the CIs are | 17:32 |
| *** xyang has joined #openstack-cinder | 17:33 | |
| *** david-lyle_ has quit IRC | 17:34 | |
| *** ccesario has quit IRC | 17:34 | |
| *** david-lyle_ has joined #openstack-cinder | 17:34 | |
| smcginnis | True | 17:34 |
| erlon | smcginnis: my question is how was possible to make everyone to adopt CI, or at least have something to say its a CIs, and its not possible to change the CI names | 17:34 |
| smcginnis | Yeah | 17:35 |
| smcginnis | And really, if we start reporting drivers for removal based on name mismatches - that will get folks to fix the names pretty darn quick. | 17:35 |
| erlon | smcginnis: yeap, it work as magic, and in one whay or another, Its a way to know who its interested actively maintain theier drivers | 17:36 |
| openstackgerrit | Xinli Guan proposed openstack/cinder: Add functional tests for nested quotas https://review.openstack.org/298453 | 17:37 |
| hemna | _alastor_, ping | 17:38 |
| erlon | smcginnis: people that don't have a reporting CI will be the one that won't give feedback | 17:38 |
| hemna | so, I think what we should do is add a new field to the wiki page | 17:38 |
| hemna | and that field is job names | 17:38 |
| hemna | and it's a dictionary | 17:38 |
| smcginnis | Infra controls that template, and last time someone from Cinder tried to add something to the template they were not too happy about it. | 17:39 |
| smcginnis | But there is freeform text area at the bottom that can be used for anything outside of the template. | 17:39 |
| smcginnis | We could grep it out of there. | 17:39 |
| hemna | {'cinder': ['3par-iscsi-eos10', '3par-fc-eos10'], 'manila': ['3par-manila-eos10'], 'os-brick': ['3par-iscsi-brick-eos10']} | 17:39 |
| smcginnis | Or talk to infra about modifying the template and see if they would be open to it. | 17:39 |
| hemna | as we have jobs for multiple projects | 17:39 |
| smcginnis | hemna: Ooo, that would be nice. | 17:39 |
| openstackgerrit | lakshman proposed openstack/cinder: Support manage/unmanage and extra-specs in K2 https://review.openstack.org/346540 | 17:40 |
| hemna | and I can update the driver code to add the CI name | 17:40 |
| hemna | and modify the cinder/tools/generate_driver_list.py to output a python dict of the driver name and ci name | 17:41 |
| hemna | (optionally) | 17:41 |
| *** david-lyle_ has quit IRC | 17:42 | |
| smcginnis | hemna: Actually, there's an idea too. If/when my patch goes through, all that will be posted in our docs: http://docs-draft.openstack.org/70/346470/6/check/gate-cinder-docs-ubuntu-xenial/654a9ca//doc/build/html/drivers.html | 17:43 |
| hemna | can we get someone to update the wiki template ? | 17:43 |
| smcginnis | Though probably better to extract from the code than scrape the web page, now that I thought more than 2 seconds about it. :) | 17:43 |
| hemna | smcginnis, that looks like it's directly from the cinder/tools/generate_driver_list.py ? | 17:43 |
| smcginnis | hemna: Yep | 17:43 |
| hemna | ok | 17:43 |
| hemna | I was going to add a command line switch to that script | 17:43 |
| hemna | to output a dict instead | 17:43 |
| hemna | the dict would have everything it currently outputs but the addition to the CI_NAME attribute on the driver | 17:44 |
| smcginnis | hemna: That would be useful. | 17:44 |
| hemna | that CI_NAME is the name of the 3rd party CI wiki page | 17:44 |
| *** rdx_matt has quit IRC | 17:44 | |
| hemna | ok let me hack this up and see what it looks like. | 17:45 |
| smcginnis | We could probably have a few different output formatters: rst, dict, yaml | 17:45 |
| hemna | sure | 17:45 |
| hemna | for now I'll just hack the crap out of it and see if it's the direction we want to go | 17:45 |
| _alastor_ | hemna: If I use "hp-storage-ci" instead of "hp-cinder-ci" as your gerrit account name I show your r | 17:45 |
| _alastor_ | CI as reporting | 17:46 |
| *** sdague has quit IRC | 17:46 | |
| hemna | hp-storage-ci isn't anything though | 17:46 |
| hemna | https://wiki.openstack.org/wiki/ThirdPartySystems/HPE_Storage_CI | 17:46 |
| hemna | there is no association to hp-storage-ci anywhere afaik | 17:46 |
| _alastor_ | hemna: Weird. On reviews your CI shows up as "HPE Storage CI" | 17:47 |
| hemna | yah that's correct | 17:47 |
| hemna | that's the CI name | 17:47 |
| hemna | and the wiki page is that replacing spaces with _ | 17:47 |
| _alastor_ | hemna: http://paste.openstack.org/show/542674/ | 17:52 |
| hemna | wtf | 17:52 |
| _alastor_ | hemna: The "username" in the response is "hp-storage-ci", which maps to the account used to post the comment | 17:52 |
| hemna | leeantho, | 17:52 |
| hemna | leeantho, is our wiki page for the gerrit id out of date? | 17:53 |
| hemna | leeantho, ^^^ | 17:53 |
| hemna | _alastor_, thanks :) | 17:53 |
| _alastor_ | hemna: np. I was pretty perplexed as well :) | 17:53 |
| *** sdague has joined #openstack-cinder | 17:54 | |
| *** links has quit IRC | 17:54 | |
| smcginnis | rootwrap/privsep is merged! https://review.openstack.org/#/c/344450/ | 17:55 |
| hemna | smcginnis, w00t! | 17:55 |
| smcginnis | happy dance time | 17:55 |
| hemna | smcginnis, https://review.openstack.org/#/c/307974 | 17:55 |
| hemna | :) | 17:55 |
| * hemna is trying hard to break the world. | 17:56 | |
| hemna | :P | 17:56 |
| smcginnis | hehe | 17:56 |
| smcginnis | Dang, jobs still sitting Queued for that one. | 17:56 |
| smcginnis | I'll watch for it. | 17:56 |
| hemna | yah | 17:57 |
| hemna | is there a way to do a static class attribute method ? | 17:57 |
| smcginnis | Gr, off to another meeting. | 17:57 |
| hemna | to force folks to implement it | 17:57 |
| smcginnis | @class_method? | 17:57 |
| DuncanT | hemna: You don't need to instantiate a class to get an attribute | 17:58 |
| *** mvk has joined #openstack-cinder | 17:58 | |
| hemna | DuncanT, yah I know | 17:58 |
| DuncanT | Oh, to force it | 17:59 |
| DuncanT | Sorry | 17:59 |
| hemna | but I want to ensure that a driver implements the attribute | 17:59 |
| hemna | and is forced to | 17:59 |
| hemna | (for new drivers/refactors, etc) | 17:59 |
| *** ntpttr__ has quit IRC | 17:59 | |
| hemna | anyway, I can just add CI_NAME for now | 17:59 |
| hemna | and have the generate_drivers_tool puke out some error if they don't see that attribute | 18:00 |
| hemna | _alastor_, ok we updated our wiki to fix the gerrit account name | 18:00 |
| hemna | _alastor_, the tool should probably bitch about that | 18:00 |
| hemna | fwiw | 18:00 |
| _alastor_ | hemna: True, not sure how I'd go about it though. I could just have it complain about everything. Like the weather in norcal at the moment :) | 18:02 |
| hemna | hehe | 18:02 |
| hemna | yah it's freaking hot here | 18:02 |
| hemna | supposed to be 108F here in Roseville, CA today | 18:03 |
| patrickeast | ouch | 18:04 |
| DuncanT | We;ve dropped to under 100F to past week. Only 75% humidity too | 18:05 |
| hemna | DuncanT, get out the jackets! | 18:06 |
| patrickeast | lol | 18:06 |
| patrickeast | its pretty nice here :P | 18:06 |
| *** jamielennox is now known as jamielennox|away | 18:10 | |
| Swanson | I'm not saying the Russians have a weather control machine. But if they do I think if they could turn the weather to a nice dryish 85 that would be great. | 18:11 |
| jungleboyj | hemna: Thankfully clouds have come over so it is only 80 in my un-airconditioned house. | 18:15 |
| openstackgerrit | Merged openstack/cinder: Map volume/snapshot manage extensions to v3 https://review.openstack.org/307324 | 18:17 |
| *** ociuhandu has quit IRC | 18:21 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/cinder: [WIP] Add functional test to new API functions https://review.openstack.org/339692 | 18:24 |
| *** zul_ has joined #openstack-cinder | 18:25 | |
| *** zul has quit IRC | 18:25 | |
| hemna | http://paste.openstack.org/show/542679/ | 18:26 |
| hemna | _alastor_, smcginnis ^^ | 18:27 |
| *** abhitechie has quit IRC | 18:27 | |
| _alastor_ | hemna: Is that taken from information in tree? | 18:29 |
| jungleboyj | patrickeast: Ping. | 18:29 |
| hemna | _alastor_, yah | 18:29 |
| _alastor_ | hemna: it's missing a bunch of drivers like Datera :) | 18:30 |
| hemna | that's only a short copy/paste | 18:30 |
| _alastor_ | hemna: ah, gotcha. Well, if we can get job info and gerrit account in there we'll be golden | 18:31 |
| hemna | _alastor_, try this https://gist.github.com/ae53e085fce7ece3d9f133e761a47ec4 | 18:32 |
| hemna | https://gist.github.com/WaltHP/ae53e085fce7ece3d9f133e761a47ec4#file-driver_list-py-L333 | 18:32 |
| hemna | (currently scrubbed the description) | 18:33 |
| hemna | so that outputs a python list | 18:33 |
| hemna | which can be directly used. | 18:33 |
| _alastor_ | hemna: I'll have to do some fuzzy matching with underscores/hyphens/spaces, but it might work | 18:36 |
| hemna | what's missing ? | 18:37 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix 'driver does not honor clone/snapshot' bugs https://review.openstack.org/336092 | 18:38 |
| erlon | pots: ^~ | 18:38 |
| _alastor_ | hemna: jobname. I need that to get results for specific jobs. The driver itself isn't mentioned in the CI review posts | 18:40 |
| pots | erlon: thanks! | 18:40 |
| hemna | ok well we need a new field on the wiki page | 18:40 |
| hemna | and a dictionary | 18:40 |
| hemna | <hemna> {'cinder': ['3par-iscsi-eos10', '3par-fc-eos10'], 'manila': ['3par-manila-eos10'], 'os-brick': ['3par-iscsi-brick-eos10']} | 18:40 |
| hemna | that would be the contents of the jobs field on the wiki | 18:41 |
| _alastor_ | hemna: Yeah, something. If it's a required attribute on the driver that would work too | 18:41 |
| *** rdx_matt has joined #openstack-cinder | 18:41 | |
| hemna | we can't do that | 18:41 |
| hemna | job names change frequently | 18:41 |
| _alastor_ | hemna: Ah right, forgot | 18:41 |
| *** rdx_matt has quit IRC | 18:41 | |
| hemna | and I don't want to force people to do code reviews to update that stuffs | 18:41 |
| *** rdx_matt has joined #openstack-cinder | 18:42 | |
| *** alonma has joined #openstack-cinder | 18:42 | |
| smcginnis | hemna: I did verify - if you put a class method in the cinder.interface.volume_driver definitions it will cause the tox -e compliance tests to fail for drivers that don't have it defined. | 18:42 |
| hemna | ok | 18:42 |
| hemna | after I get all of the drivers updated with a CI_NAME attribute | 18:42 |
| smcginnis | +1 | 18:42 |
| hemna | I'll add that to the interface | 18:42 |
| hemna | :) | 18:43 |
| hemna | I have a lot of driver files to touch.... | 18:43 |
| *** ccesario has joined #openstack-cinder | 18:43 | |
| smcginnis | 96 of them, but the looks of it. | 18:43 |
| hemna | I'm going to need everyone to review this patch | 18:44 |
| *** ericksonsantos has quit IRC | 18:44 | |
| hemna | because I might get the CI_NAME wrong for any given driver | 18:44 |
| hemna | *sigh* EMC | 18:44 |
| smcginnis | hemna: Well, if we start basing CI checking off of it, we'll find out pretty quick if any are wrong. | 18:44 |
| hemna | EMCCoprHDFCDriver | 18:44 |
| hemna | wtf is that | 18:45 |
| hemna | I'm just guessing on this stuff | 18:45 |
| *** ChubYann has joined #openstack-cinder | 18:45 | |
| smcginnis | Open source CoprHD driver that they still want to have a company name on. :/ | 18:45 |
| hemna | no idea which CI is associated with it | 18:45 |
| *** e0ne has joined #openstack-cinder | 18:45 | |
| hemna | CI_NAME = "unknown" ? | 18:46 |
| hemna | no clue | 18:46 |
| hemna | xyang, ping | 18:46 |
| *** e0ne has quit IRC | 18:47 | |
| smcginnis | EMC CoprHD CI | 18:47 |
| *** alonma has quit IRC | 18:47 | |
| hemna | https://wiki.openstack.org/wiki/ThirdPartySystems ? | 18:47 |
| smcginnis | I had to track it down to here: https://review.openstack.org/#/c/342171/3 | 18:48 |
| *** smile-luobin has quit IRC | 18:48 | |
| smcginnis | That's the last patch it commented on. | 18:48 |
| *** alonma has joined #openstack-cinder | 18:48 | |
| smcginnis | That's got to get fixed. It doesn't look like it's commenting on all patches. | 18:48 |
| hemna | and it has no wiki page | 18:49 |
| *** smile-luobin has joined #openstack-cinder | 18:49 | |
| hemna | or is it this one ? https://wiki.openstack.org/wiki/ThirdPartySystems/emc-xio-ci | 18:49 |
| hemna | no idea | 18:49 |
| xyang1 | hemna: hi | 18:50 |
| hemna | what CI is associated with the coprHD drivers ? | 18:50 |
| hemna | there are 2 of them | 18:50 |
| smcginnis | hemna: No, not that one. | 18:50 |
| smcginnis | That's XtremeIO | 18:50 |
| xyang1 | let me check | 18:50 |
| smcginnis | I left comments in their forum for the wiki and the reporting. | 18:50 |
| hemna | https://github.com/openstack/cinder/tree/master/cinder/volume/drivers/coprhd | 18:50 |
| hemna | those guys | 18:50 |
| hemna | the scaleio.py and fc.py | 18:51 |
| hemna | oh and iscsi.py in there | 18:51 |
| xyang1 | emc-coprhd-ci | 18:51 |
| hemna | xyang1, and the wiki url ? | 18:51 |
| hemna | :P | 18:51 |
| xyang1 | vipr | 18:51 |
| hemna | !! | 18:51 |
| openstack | hemna: Error: "!" is not a valid command. | 18:51 |
| hemna | * S M H * | 18:51 |
| xyang1 | I created it initially for vipr, that is why | 18:51 |
| *** ntpttr__ has joined #openstack-cinder | 18:52 | |
| xyang1 | before there was coprhd | 18:52 |
| smcginnis | xyang1: All of those EMC entries should probably be updated to actually have their names. | 18:52 |
| *** avishay has joined #openstack-cinder | 18:52 | |
| xyang1 | which name is not right? | 18:52 |
| hemna | I have no idea which wiki page is associated with any EMC driver | 18:52 |
| hemna | so confusing | 18:52 |
| *** alonma has quit IRC | 18:53 | |
| smcginnis | xyang1: They should have the name of the CI so it's easy to read, not the cryptic account name. | 18:53 |
| xyang1 | emc-xxx-ci | 18:53 |
| smcginnis | EMC XXX CI | 18:53 |
| avishay | smcginnis DuncanT hemna scottda_vacation etc - thanks for your help with the list manageable patches! | 18:53 |
| smcginnis | emc-vipr-ci = EMC CoprHD CI | 18:53 |
| smcginnis | avishay: ;) | 18:53 |
| hemna | EMCCoprHDISCSIDriver | 18:53 |
| hemna | ? | 18:53 |
| xyang1 | I'll take a look | 18:54 |
| smcginnis | xyang1: Oh, wanted to point out too that the new VNX driver doesn't appear to be defining VERSION right. It doesn't show up. | 18:54 |
| xyang1 | is it | 18:54 |
| smcginnis | At least here: http://paste.openstack.org/show/542679/ | 18:54 |
| hemna | I need the wiki page name for each of the 3 drivers in the coprhd dir | 18:54 |
| smcginnis | hemna: Oh, and HPEXPFCDriver isn't either. | 18:55 |
| hemna | doh | 18:55 |
| *** faiz89 has quit IRC | 18:55 | |
| *** alonma has joined #openstack-cinder | 18:56 | |
| *** david-lyle_ has joined #openstack-cinder | 18:57 | |
| hemna | oh man | 18:57 |
| hemna | I can't believe I took this on... | 18:57 |
| hemna | lolz | 18:57 |
| hemna | so | 18:57 |
| hemna | many | 18:57 |
| hemna | drivers | 18:57 |
| hemna | pain | 18:58 |
| smcginnis | Haha! :D | 18:58 |
| hemna | I really hope this is going to help | 18:58 |
| smcginnis | hemna: I think it will. So far at least the idea seems sound. | 18:59 |
| *** ebalduf has quit IRC | 19:00 | |
| *** alonma has quit IRC | 19:00 | |
| hemna | xyang1, I'll need you to review all of the EMC driver changes once I post this | 19:01 |
| hemna | it's hard to make sense of them | 19:01 |
| *** lprice has quit IRC | 19:01 | |
| xyang | hemna: where is your change? | 19:02 |
| hemna | re: once I post this | 19:02 |
| xyang | ok | 19:02 |
| hemna | I have a ways to go | 19:02 |
| xyang | hemna: is this a code change in cinder or wiki change? | 19:02 |
| xyang | @smcginnis: they defined version here: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/emc/vnx/adapter.py#L44 | 19:02 |
| hemna | smcginnis, I'm noticing several drivers that don't have a VERSION defined in them | 19:03 |
| hemna | :( | 19:03 |
| xyang | smcginnis: should be here? | 19:03 |
| hemna | EMCVNXDriver doesn't have one | 19:03 |
| xyang | https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/emc/vnx/driver.py | 19:03 |
| hemna | heh | 19:03 |
| hemna | was just looking at that | 19:03 |
| xyang | so the version was in iSCSI and FC drivers in the past. I think that is why they added it in adapter instead of the driver file | 19:04 |
| smcginnis | xyang: Ah, that explains it. It's a class static variable, so the child classes don't end up with a driver.VERSION variable. | 19:04 |
| xyang | @smiginnis: should it be moved to driver.py? | 19:04 |
| hemna | here, let me push up a WIP of what I have | 19:04 |
| *** RageLtMan has joined #openstack-cinder | 19:04 | |
| smcginnis | xyang: Not sure what you mean. | 19:04 |
| RageLtMan | Hi folks, anyone here who commits regularly? i have found a bug and understand the remedial vector, but dont want to get involved in the mess of trying to submit changes to openstack | 19:05 |
| smcginnis | xyang: Each driver class would need it defined. | 19:05 |
| smcginnis | RageLtMan: What's the issue? | 19:05 |
| smcginnis | You can always file a big with the information and let someone else pick it up. | 19:06 |
| xyang | @smcginnis: you said you can't find the version automatically? So it should be moved from https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/emc/vnx/adapter#L44 to https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/emc/vnx/driver.py | 19:06 |
| *** alonma has joined #openstack-cinder | 19:06 | |
| RageLtMan | The bug: creating an instance from an image which has hw_disk_bus=scsi set result in Cinder recording the mountpoint of the boot volume as /dev/vda in the cinder DB's volume_attachment table | 19:06 |
| smcginnis | xyang: Yeah, I believe so. | 19:06 |
| RageLtMan | when attempting to attach a volume to the instance subsequent to the original create call, it does not record a warning or error, just puts the volume back in available state | 19:06 |
| xyang | smcginnis: vnx/driver.py has the driver definition but version is in vnx/adapter.py now. There is only a docstring entry for version in vnx/driver.py | 19:06 |
| xyang | ok | 19:07 |
| *** david-lyle_ is now known as david-lyle | 19:07 | |
| RageLtMan | if the mountpoint in the database is manually repaired to read /dev/sda then cinder attach ops work fine | 19:07 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_NAME to all drivers https://review.openstack.org/348002 | 19:07 |
| RageLtMan | so, cinder needs to properly record the mountpoint at instance creation based on the type of disk presented by the bus configured for nova | 19:07 |
| *** fifieldt has quit IRC | 19:08 | |
| smcginnis | hemna: Don't we get that from Nova? ^^ | 19:08 |
| hemna | smcginnis, yes | 19:08 |
| hemna | afaik cinder doesn't even use it anywhere | 19:08 |
| avishay | hemna: +1 | 19:09 |
| avishay | correct | 19:09 |
| smcginnis | hemna: Yeah, I didn't think so either. | 19:09 |
| RageLtMan | I quickly skimmed the relevant sources, and it appears the mountpoint param passed to volume-attach is ignored | 19:09 |
| xyang | hemna: the CI_name is the account name? | 19:09 |
| hemna | xyang, no | 19:09 |
| hemna | xyang, read the commit message :) | 19:09 |
| xyang | hemna: smcginnis just asked me to spell them out | 19:09 |
| xyang | sure | 19:09 |
| hemna | https://wiki.openstack.org/wiki/ThirdPartySystems/HPE_Storage_CI | 19:09 |
| avishay | seems weird that the behavior in nova changes according to vda or sda | 19:09 |
| *** haplo37__ has joined #openstack-cinder | 19:09 | |
| hemna | CI_NAME = "HPE_Storage_CI" | 19:09 |
| hemna | xyang, the purpose of CI_NAME is to be a lookup to the wiki page | 19:10 |
| smcginnis | xyang: So as it is right now, CI_NAME would be account name because that's how they are defined on the wiki. | 19:10 |
| *** alonma has quit IRC | 19:10 | |
| smcginnis | But normally it's the actual CI name, not the account name. | 19:10 |
| avishay | does cinderclient have support for microversions yet? i see v3 but not sure how microversions work there... | 19:10 |
| hemna | https://wiki.openstack.org/wiki/ThirdPartySystems/<CI_NAME> | 19:10 |
| hemna | that's what it is | 19:10 |
| hemna | avishay, yes I believe so | 19:10 |
| _alastor_ | So if I understand it correctly: in tree drivers --> CI_NAME --> wiki lookup --> gerrit account/jobname? | 19:10 |
| hemna | _alastor_, yah | 19:11 |
| _alastor_ | hemna: kk | 19:11 |
| hemna | _alastor_, does that work? | 19:11 |
| hemna | _alastor_, https://review.openstack.org/#/c/348002/ | 19:11 |
| xyang | smcginnis: but we can't have space there though | 19:11 |
| _alastor_ | hemna: should work just fine. As long as by the end of it I have gerrit account and jobname | 19:11 |
| hemna | you can pull that and run the cinder/tools/generate_driver_list.py | 19:11 |
| hemna | and get the output | 19:11 |
| avishay | hemna: if i want to add support for an API that is in 3.8, how do i express that? what if support for things before 3.8 weren't added? | 19:11 |
| hemna | ./generate_driver_list.py --format dict | 19:11 |
| hemna | it'll poop out a giant list of dicts. | 19:11 |
| smcginnis | xyang: Check out all the other ones. The spaces just end up _ | 19:12 |
| xyang | smcginnis: so should I change now that hemna has submitted the patch? they will go out of sync if I change:) | 19:12 |
| smcginnis | avishay: https://review.openstack.org/#/c/301941/ | 19:12 |
| hemna | avishay, I think scottda added a docs page on how to use the client w/ a specific microversion | 19:12 |
| smcginnis | avishay: Merged in April. | 19:12 |
| openstackgerrit | Justin A Wilson proposed openstack/cinder: Fixed request/response parameters in doc https://review.openstack.org/348004 | 19:13 |
| hemna | I think you have to ask for the version # at client instantiation | 19:13 |
| hemna | anyway, _alastor_, smcginnis, xyang if you can look at that CI_NAME patch for me | 19:13 |
| hemna | I have more to add | 19:13 |
| hemna | but I need to grab some lunch | 19:13 |
| _alastor_ | hemna: will do | 19:13 |
| *** alonma has joined #openstack-cinder | 19:13 | |
| smcginnis | xyang: Maybe... Would be nice to have that match the rest and have the display name and not the account name. | 19:13 |
| hemna | I just want to make sure that's the direction we want to go before I go hacking up more drivers | 19:13 |
| smcginnis | hemna: Thanks for working on that. I'll take a look soon. | 19:13 |
| smcginnis | hemna: +1 | 19:14 |
| hemna | add review comments for incorrect CI_NAME values | 19:14 |
| hemna | ok lunch..... | 19:14 |
| avishay | smcginnis hemna thanks | 19:14 |
| *** rdx_matt has quit IRC | 19:15 | |
| avishay | smcginnis: OK so I see the version in the client is still 3.0. I can't just add "list manageable" and bump it to 3.8 without adding all the changes in 3.1-3.7, right? | 19:15 |
| smcginnis | avishay: I believe it's just an argument to specify which microversion you want. | 19:16 |
| avishay | smcginnis: but if i specify 3.8 i expect 3.1-3.7 to be included as well, no? | 19:17 |
| smcginnis | CLI or as a library. CLI you just are making an individual call, so it should be able to just impact the one microversion you want. | 19:18 |
| * smcginnis is looking for client docs | 19:18 | |
| *** alonma has quit IRC | 19:18 | |
| *** fifieldt has joined #openstack-cinder | 19:19 | |
| avishay | so sometimes the client will say it is 3.0 and sometimes 3.8? | 19:19 |
| *** xinli has quit IRC | 19:20 | |
| *** IlyaG has joined #openstack-cinder | 19:20 | |
| *** coolsvap has quit IRC | 19:21 | |
| smcginnis | avishay: So 3.0 is the base for everything and the default if no other specified. | 19:24 |
| smcginnis | avishay: If I remember right, you add "--os-volume-api-version 3.8" to make that specific call to the requested microversion. | 19:25 |
| smcginnis | I haven't had need to really use it much, so a little rusty on that. | 19:25 |
| *** sdake has joined #openstack-cinder | 19:25 | |
| smcginnis | Unfortuntely Scott is on vacation right now, but he's done a lot as he was working on adding all that support. | 19:26 |
| *** xyang has quit IRC | 19:26 | |
| avishay | smcginnis: yep i did that when testing v3 APIs manually and it worked fine. i just don't understand how cinderclient is supposed to work. maybe i'll just wait until scott returns... | 19:26 |
| *** xyang has joined #openstack-cinder | 19:26 | |
| xyang | smcginnis: can you check if this is correct? https://wiki.openstack.org/wiki/ThirdPartySystems/EMC_ScaleIO_CI | 19:26 |
| smcginnis | avishay: I would consider him the authority on that at this point. ;) | 19:26 |
| avishay | smcginnis: yep he certainly helped me on the server part of it | 19:27 |
| xyang | smcginnis: I have to recreate the page, can't just rename it. want to make sure it is correct before changing all of them | 19:27 |
| smcginnis | xyang: Nice! Yeah, I think that looks better. | 19:27 |
| xyang | smcginnis: ok, thanks | 19:27 |
| smcginnis | _alastor_: Is that what you would expect scraping the wiki? ^^ | 19:27 |
| *** xinli has joined #openstack-cinder | 19:28 | |
| *** ebalduf has joined #openstack-cinder | 19:29 | |
| *** Yogi1 has quit IRC | 19:30 | |
| avishay | smcginnis: OK I see now - e0ne pushed a patch to support the change in 3.1 | 19:32 |
| avishay | will work off that | 19:33 |
| _alastor_ | smcginnis: Yes, than page has everything I need other than jobname, but we're still debating what that should look like | 19:33 |
| *** ccesario has quit IRC | 19:33 | |
| *** jungleboyj has quit IRC | 19:33 | |
| smcginnis | avishay: Excellent | 19:33 |
| smcginnis | _alastor_: Cool, thanks. | 19:33 |
| *** alonma has joined #openstack-cinder | 19:34 | |
| xinli | @xyang1: I hit this issue again, all the test paseed, but the result shows error. http://logs.openstack.org/53/298453/6/check/gate-cinder-python27-db-ubuntu-xenial/b6de4e2/console.html | 19:34 |
| xinli | @xyang1: http://paste.openstack.org/show/542686/ | 19:35 |
| Swanson | Me, too. | 19:35 |
| *** alonma has quit IRC | 19:38 | |
| *** avishay has quit IRC | 19:39 | |
| *** ebalduf has quit IRC | 19:41 | |
| hemna | smcginnis, eqlx.py is a Dell driver ? | 19:41 |
| erlon | xinli: I got that too, just recheck until it passes :/ | 19:42 |
| *** sdake has quit IRC | 19:43 | |
| xinli | erlon: thx. | 19:43 |
| *** thiagolib has joined #openstack-cinder | 19:46 | |
| Swanson | hemna, yes | 19:46 |
| hemna | ok | 19:47 |
| hemna | mv eqlx.py dell/ | 19:47 |
| hemna | :P | 19:47 |
| Swanson | I'm not having the filthy eqlx.py driver in with my code. | 19:48 |
| hemna | lolz | 19:48 |
| smcginnis | Hey now. :) | 19:49 |
| *** ebalduf has joined #openstack-cinder | 19:50 | |
| *** ntpttr- has quit IRC | 19:50 | |
| smcginnis | WOw, I think it's been so long since the last time the X-IO CI reported that the lastcomment script isn't even returning anything. | 19:50 |
| smcginnis | Or I'm screwing something up. | 19:50 |
| *** ntpttr- has joined #openstack-cinder | 19:51 | |
| smcginnis | Yeah, I think I'm screwing something up. | 19:51 |
| *** timcl has quit IRC | 19:52 | |
| hemna | Hitachi drivers also missing VERSION | 19:52 |
| *** sdake has joined #openstack-cinder | 19:53 | |
| *** jdurgin has quit IRC | 19:53 | |
| xyang | hemna: I updated EMC CI names | 19:54 |
| *** ntpttr__ has quit IRC | 19:54 | |
| hemna | xyang, thanks :) | 19:54 |
| hemna | I'm sure half of my CI_NAME entries are wrong | 19:54 |
| smcginnis | Anyone know where the "official" lastcomment script is? I'm using the one here: https://github.com/openstack/third-party-ci-tools/tree/master/monitoring/lastcomment-scoreboard | 19:54 |
| smcginnis | But doesn't seem like it's working right. | 19:54 |
| smcginnis | Or I'm really messing something up. | 19:55 |
| xyang | hemna: you were right. It's just that I changed the names on wiki to be consistent with others:) | 19:55 |
| *** lprice has joined #openstack-cinder | 19:55 | |
| smcginnis | Incidentally: https://github.com/openstack/third-party-ci-tools/blob/master/monitoring/lastcomment-scoreboard/ci.yaml | 19:55 |
| patrickeast | smcginnis: it was jogo's project iirc https://github.com/jogo/lastcomment | 19:56 |
| smcginnis | patrickeast: That's it! | 19:56 |
| hemna | IBM drivers missing VERSION | 19:57 |
| hemna | Infortrend drivers missing VERSION | 19:59 |
| *** lprice has quit IRC | 19:59 | |
| *** ccesario has joined #openstack-cinder | 20:00 | |
| smcginnis | SO this is a weird one I would like to get some input on... | 20:00 |
| *** porrua has quit IRC | 20:00 | |
| smcginnis | The DotHill drivers aren't reporting. | 20:01 |
| hemna | wtf | 20:01 |
| smcginnis | If folks were around, there was some discussion on these. | 20:01 |
| hemna | Kaminario | 20:01 |
| hemna | ? | 20:01 |
| smcginnis | They are OEMd. | 20:01 |
| smcginnis | So there IS a Lenovo driver that is basically just inherited from DotHill that is reporting CI status. | 20:01 |
| smcginnis | So indirectly it is reporting CI results for that code. | 20:01 |
| hemna | ugh | 20:02 |
| smcginnis | But at the time, when we agreed to allow three different drivers with one code base, we stated we needed reporting CI for all. | 20:02 |
| hemna | ew I see that | 20:02 |
| hemna | LenovoFCDriver is just a derivation of DotHillFCDriver | 20:02 |
| smcginnis | Yep | 20:02 |
| smcginnis | One other one too, but can't remember offhand which one. | 20:02 |
| smcginnis | Since been acquired by Seagate: http://www.theregister.co.uk/2015/08/19/seagate_scoops_dot_hill_for_us694_million/ | 20:03 |
| *** ebalduf has quit IRC | 20:03 | |
| smcginnis | For $694 million I'll run CI for them. ;) | 20:03 |
| hemna | lol | 20:03 |
| hemna | I'll do it for half that | 20:03 |
| smcginnis | Hey, stay out of my way man! :P | 20:04 |
| patrickeast | psh id do it for like 3 mil, plus T&M | 20:04 |
| hemna | free market, bitches! | 20:04 |
| smcginnis | So that's how it's going to be. | 20:04 |
| smcginnis | :D | 20:04 |
| hemna | patrickeast, cheapskate. you get what you pay for! | 20:04 |
| patrickeast | lol | 20:04 |
| patrickeast | hemna: it only has to run for 6 months at a time, then its contract renewal time XD | 20:04 |
| patrickeast | don't want it working tooo smoothly | 20:05 |
| smcginnis | Technically, it just has to run once every two months now. | 20:05 |
| hemna | so...what to do about the lenovo ci ? | 20:05 |
| patrickeast | lol | 20:05 |
| smcginnis | Doh! Did I write that out loud. | 20:05 |
| hemna | https://wiki.openstack.org/wiki/ThirdPartySystems/Vedams-LenovoStorage_FCISCSI_CI | 20:06 |
| hemna | I guess that's it ? | 20:06 |
| smcginnis | hemna: I'm thinking leaving it for now? It's obviously not tested against the Dot Hill version of the box, but the Lenovo one is just a rebranded OEM, so kinda sorta it's being tested. :/ | 20:07 |
| hemna | yah | 20:07 |
| hemna | I was trying to figure out what CI_NAME to put on it | 20:07 |
| hemna | not sure about CI_NAME for lvm.py | 20:07 |
| * hemna skips for now | 20:07 | |
| smcginnis | Jenkins? | 20:07 |
| hemna | omg | 20:07 |
| hemna | netapp/ | 20:07 |
| smcginnis | LOL | 20:08 |
| hemna | 300 drivers in there alone | 20:08 |
| hemna | I hope they are all reporting under the same name | 20:08 |
| hemna | NetApp_CI | 20:08 |
| akerr | hemna: we are | 20:08 |
| hemna | akerr, phew! thanks | 20:08 |
| *** Apoorva has quit IRC | 20:09 | |
| hemna | netapp drivers don't have VERSION either..... | 20:09 |
| akerr | don't get me started | 20:09 |
| hemna | :P | 20:10 |
| akerr | 2 years I've been asking for a version scheme | 20:10 |
| hemna | hrmm | 20:10 |
| hemna | I can create one for you :P | 20:10 |
| smcginnis | Dang, last reported 2016-03-18 20:23:25 (130 days, 23:47:11 old) | 20:10 |
| smcginnis | I'd say we're overdue for removal on that one. | 20:11 |
| hemna | whoa | 20:11 |
| smcginnis | That was still in Mitaka | 20:11 |
| *** rdx_matt has joined #openstack-cinder | 20:12 | |
| *** rdx_matt has quit IRC | 20:12 | |
| *** rdx_matt has joined #openstack-cinder | 20:12 | |
| *** merooney has quit IRC | 20:13 | |
| *** smile-luobin_ has joined #openstack-cinder | 20:14 | |
| *** smile-luobin has quit IRC | 20:14 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: quobyte volume driver should use DLM https://review.openstack.org/348020 | 20:14 |
| *** akshai has quit IRC | 20:15 | |
| *** Lallana has quit IRC | 20:17 | |
| *** cdelatte has quit IRC | 20:17 | |
| *** Lallana has joined #openstack-cinder | 20:19 | |
| *** ntpttr__ has joined #openstack-cinder | 20:20 | |
| *** ericksonsantos has joined #openstack-cinder | 20:20 | |
| *** mdenny has quit IRC | 20:20 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Remove X-IO volume driver https://review.openstack.org/348022 | 20:21 |
| *** mdenny has joined #openstack-cinder | 20:21 | |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring https://review.openstack.org/285469 | 20:23 |
| hemna | wtf | 20:24 |
| hemna | Synology driver | 20:24 |
| hemna | was added June 23rd | 20:24 |
| hemna | I don't see a wiki page | 20:24 |
| smcginnis | hemna: You're going to need a beer after all this, aren't you? | 20:24 |
| hemna | yes | 20:24 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Quobyte volume driver should use DLM https://review.openstack.org/348020 | 20:27 |
| *** chris_morrell has quit IRC | 20:27 | |
| openstackgerrit | Ben Swartzlander proposed openstack/cinder: Add stochastic weight handler to scheduler https://review.openstack.org/347010 | 20:29 |
| _alastor_ | So if ci-status is now going to use cinder/tools/generate_driver_list.py, that means it now has a dependency on cinder. The cinder repository doesn't appear to be available as a pypi package though... | 20:30 |
| *** JoseMello has quit IRC | 20:30 | |
| smcginnis | _alastor_: Hmm, yeah, that's not good. | 20:30 |
| Swanson | Added June 23rd and no Wiki page? That's the fault of our Commander in PTL. Thanks, smcginnis! | 20:30 |
| smcginnis | Swanson: Shut it! | 20:30 |
| Swanson | Gonna be a bloodbath when DuncanTrump takes over this mess. | 20:31 |
| patrickeast | lol | 20:31 |
| *** ducttape_ has quit IRC | 20:31 | |
| *** aswadr_ has quit IRC | 20:31 | |
| _alastor_ | So one option is that I move the script into Cinder and just make it Cinder-specific | 20:31 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Remove Tintri volume driver https://review.openstack.org/348026 | 20:32 |
| _alastor_ | Or I could just add to the readme that you need to clone Cinder and setup.py develop install it into your virtualenv for it to work | 20:33 |
| smcginnis | _alastor_: Adds complexity, but avoids duplication. | 20:33 |
| hemna | Swanson, lolz | 20:34 |
| smcginnis | But we are the only ones that have the ability to generate a driver list, as far as I know. | 20:34 |
| *** alonma has joined #openstack-cinder | 20:34 | |
| *** ntpttr__ has quit IRC | 20:35 | |
| *** ntpttr__ has joined #openstack-cinder | 20:36 | |
| _alastor_ | Or I could create a c-driver-ci-status service with a c-driver-ci-status-manager that will handle RPC calls to the service (Active-Active HA support pending of course) | 20:36 |
| _alastor_ | jk | 20:36 |
| _alastor_ | I don't want to have to touch oslo enough to do that | 20:37 |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file https://review.openstack.org/294076 | 20:37 |
| hemna | who is ZTE ? | 20:37 |
| erlon | mtanino: ping | 20:38 |
| mtanino | erlon: pong | 20:38 |
| mtanino | erlon: hey :) | 20:38 |
| *** alonma has quit IRC | 20:38 | |
| erlon | mtanino: Hey Mitsushiro! | 20:38 |
| mtanino | erlon: What's up? | 20:39 |
| erlon | mtanino: I know you have worked with qemu-img haven't you? | 20:39 |
| *** faiz89 has joined #openstack-cinder | 20:39 | |
| mtanino | erlon: yes, in previous. | 20:39 |
| mtanino | erlon: mu | 20:40 |
| erlon | may be you can help, or know who | 20:40 |
| erlon | I'm trying to figure out an error on Cinder NFS Driver, I trying to create/delete snapshots from a snapshot chain. There are 3 situations: | 20:40 |
| erlon | 1 - I create/deete the snapshots in Cinder (it uses qemu-img). It goes OK! I can delete and I can create snapshots. | 20:40 |
| erlon | 2 - I create/delete the snapshots, in online mode (it uses libvirt). It goes Ok, as weel. | 20:40 |
| erlon | 3 - I create the snapshots in Cinder (offline mode using qemu-img) and delete then in online mode (using libvirt), then it fails with this message: | 20:41 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Remove Tegile volume driver https://review.openstack.org/348032 | 20:41 |
| erlon | libvirtError: invalid argument: could not find image 'volume-8c58ee12-9ad0-4d71-ae29-e18399af6c8e.53a3cb87-c2fc-4e4e-988a-8ead73d753e0' in chain for '/opt/stack/data/nova/mnt/a3b4c6ddd9bf82edd4f726872be58d05/volume-8c58ee12-9ad0-4d71-ae29-e18399af6c8e | 20:41 |
| erlon | mtanino: any idea? | 20:41 |
| *** alonma has joined #openstack-cinder | 20:41 | |
| mtanino | erlon: Let me see the NFS driver. | 20:44 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_NAME to all drivers https://review.openstack.org/348002 | 20:44 |
| *** Apoorva has joined #openstack-cinder | 20:45 | |
| erlon | mtanino: this cames from the nova side: nova/virt/libvirt/driver.py | 20:45 |
| erlon | mtanino: full stack error on n-cpu: https://gist.github.com/anonymous/bb4ebb7e4344e26ca0658542a4314eb1 | 20:46 |
| *** rdx_matt has quit IRC | 20:46 | |
| *** sdake has quit IRC | 20:46 | |
| hemna | ok I think I have most of them | 20:46 |
| hemna | lvm.py is missing | 20:46 |
| *** alonma has quit IRC | 20:46 | |
| hemna | _alastor_, not sure what I should put for lvm.py | 20:46 |
| hemna | so smcginnis didn't you say I could put a CI_NAME in the interface and it'd puke on drivers that don't have it ? | 20:47 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Scality volume driver should use DLM https://review.openstack.org/348034 | 20:48 |
| _alastor_ | hemna: I dunno either. I assume we have an LVM CI? | 20:48 |
| hemna | well we do have CI for lvm | 20:48 |
| hemna | it's the standard tempest dsvm full | 20:48 |
| mtanino | erlon: step 3 means that once you take a snapshot when the volume is offline(not attached) and then attach the volume to VM, then delete a snapthot, right? | 20:48 |
| _alastor_ | Ha! Yeah, I have no idea then. I wasn't planning on the tool to be used for Jenkins stats. You can just go look at jenkins :) | 20:49 |
| hemna | _alastor_, we have a few drivers like this though | 20:49 |
| *** alonma has joined #openstack-cinder | 20:49 | |
| hemna | that are part of the standard check/gate | 20:49 |
| hemna | that don't have a thirdparty systems wiki page | 20:49 |
| erlon | mtanino: yes | 20:49 |
| *** ebalduf has joined #openstack-cinder | 20:49 | |
| hemna | would be nice if the tool worked for them as well | 20:49 |
| smcginnis | hemna: I tested with adding a class method. | 20:50 |
| hemna | gate-tempest-dsvm-full | 20:50 |
| smcginnis | hemna: Not sure if it can check that a variable is defined. | 20:50 |
| hemna | oh class method | 20:50 |
| smcginnis | hemna: Wouldn't CI_NAME = Jenkins and test name = test-dsvm-full or whatever it is? | 20:50 |
| hemna | I dunno | 20:50 |
| hemna | the tool would have to special case the Jenkins CI_NAME | 20:51 |
| _alastor_ | hmmm, I don't think the name matters as long as it matches the wiki page. Maybe just come up with a colloquial name for it like "Tempest LVM Full" or "That Shit That Always Fails For Me" | 20:51 |
| *** cdelatte has joined #openstack-cinder | 20:51 | |
| hemna | or we create a Cinder_Jenkins wiki page w/ the list of jobs ? | 20:51 |
| _alastor_ | hemna: I can always write a separate parser for that page. It's not a problem | 20:52 |
| hemna | up to you | 20:52 |
| hemna | anyone else can push a new patch against that review to make changes | 20:52 |
| _alastor_ | hemna: Better to have it all on one page if we can, so it's just a single page load | 20:52 |
| _alastor_ | hemna: The longest part of doing all of this is just loading pages to scrape the wiki | 20:53 |
| mtanino | erlon: I guess nova tries to remove online snapshot but acutually the snapshot is created via qemu-img at offline state so taht libvirt can't find any snapshot to remove, then you hit error?... | 20:53 |
| *** alonma has quit IRC | 20:54 | |
| mtanino | erlon: I don't have much knowledge around remotefs drivers online/offline snapshot, just a guess. | 20:56 |
| erlon | mtanino: hmm, I don't know, I can be that but the error message should point that that operation were not allowed | 20:56 |
| erlon | mtanino: ok, no worries just asked to see if you had heard about something like that | 20:57 |
| *** ducttape_ has joined #openstack-cinder | 20:57 | |
| erlon | mtanino: but I got to this point and have no idea how to move. From that call, and on, nova uses the libvirtlib, which is C++ | 20:58 |
| mtanino | erlon: not python... | 20:59 |
| erlon | mtanino: yep | 20:59 |
| mtanino | erlon: one question. if you detach the volume again, can you delete the snapshot?(this is offline snapshot delete) | 21:00 |
| *** xinli has quit IRC | 21:01 | |
| erlon | mtanino: yes, but if I have tried to delete it, attached, I have to set the status of the snapshot to avaiable | 21:01 |
| mtanino | erlon: Ah, I see | 21:02 |
| erlon | mtanino: it looks either a libvirt bug, or an incompatibility between qemu-img vs libvirt | 21:02 |
| hemna | _alastor_, smcginnis https://gist.github.com/WaltHP/b9b00fdf0e3cd3d59ca122841676a441 | 21:05 |
| *** ntpttr__ has quit IRC | 21:05 | |
| hemna | can see which ones are missing the CI_NAME | 21:06 |
| mtanino | erlon: I guess "create at offline state but delete at online state" is not considered or are there drivers they can this operation? | 21:07 |
| smcginnis | hemna: Nice | 21:08 |
| hemna | so, what do we do with the drbd driver | 21:09 |
| hemna | and blockdevice driver | 21:09 |
| erlon | mtanino: yes, all RemoteFS drivers that implements snapshots should do that | 21:09 |
| smcginnis | hemna: All the Jenkins tested ones? | 21:09 |
| hemna | sheepdog as well | 21:09 |
| erlon | mtanino: s/should/do/ | 21:09 |
| hemna | I put it as 'unknown' | 21:09 |
| hemna | glusterfs | 21:10 |
| *** jdurgin has joined #openstack-cinder | 21:10 | |
| *** dustins has quit IRC | 21:10 | |
| erlon | mtanino: don't know how they work though | 21:10 |
| mtanino | erlon: unfortunately me too... | 21:11 |
| *** ntpttr__ has joined #openstack-cinder | 21:11 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Remove Scality volume driver https://review.openstack.org/348042 | 21:11 |
| *** pauloewerton has quit IRC | 21:11 | |
| hemna | gate-tempest-dsvm-full-sheepdog-nv | 21:12 |
| hemna | I guess that covers the sheepdog driver | 21:12 |
| *** ebalduf has quit IRC | 21:13 | |
| hemna | _alastor_, so...... CI_NAME = "Cinder_Jenkins" ? | 21:13 |
| hemna | for lvm, glusterfs, nfs, sheepdog, etc | 21:13 |
| hemna | (even though that Cinder_Jenkins wiki page doesn't exist) | 21:14 |
| *** ntpttr__ has quit IRC | 21:14 | |
| hemna | smbfs as well | 21:14 |
| hemna | bleh | 21:14 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Smbfs volume driver should use DLM https://review.openstack.org/348044 | 21:14 |
| mtanino | erlon: I couldn't find "create at offline state but delete at online state" test case in the tempest | 21:14 |
| smcginnis | Is there a driver that matches IBM NAS CI? | 21:14 |
| smcginnis | diablo_rojo: Do you know? ^^ | 21:15 |
| hemna | smcginnis, not according to my patch :P | 21:15 |
| _alastor_ | hemna: That works for me. I could just report each of them as Cinder_Jenkins_XXXX | 21:15 |
| smcginnis | OK, didn't think so. | 21:15 |
| *** zeestrat has joined #openstack-cinder | 21:15 | |
| erlon | mtanino: yep, Im afraid that would break 'some' drivers :) | 21:16 |
| hemna | I guess when jenkins doesn't exist, we can rename them to Cinder_zuul | 21:16 |
| mtanino | erlon: ;) | 21:16 |
| *** jamielennox|away is now known as jamielennox | 21:17 | |
| erlon | mtanino: when I find out a way out here Ill add a test there just to see the what happens lol | 21:17 |
| smcginnis | hemna: But isn't the reporting CI name "Jenkins"? Not sure why we can't just use that. | 21:17 |
| mtanino | erlon: :) | 21:17 |
| _alastor_ | smcginnis: It might get confusing with other projects all sharing the 3rd party wiki | 21:18 |
| hemna | smcginnis, yah I didn't think we'd get away with creating a Jenkins page off of the 3rdPartySystems page | 21:19 |
| smcginnis | Ah, OK. I see what you're saying now. | 21:19 |
| smcginnis | You mean the world doesn't revolve around us!? | 21:19 |
| smcginnis | :D | 21:19 |
| hemna | heh | 21:19 |
| *** faiz89 has quit IRC | 21:22 | |
| smcginnis | OK, I think I have patches up for all non-compliant drivers. | 21:23 |
| smcginnis | https://review.openstack.org/#/q/topic:remove_noncompliant_driver | 21:23 |
| *** david-lyle has quit IRC | 21:25 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_NAME to all drivers https://review.openstack.org/348002 | 21:25 |
| hemna | https://gist.github.com/f335ebcc7d92308a4fa23685d053e659 ^^ | 21:25 |
| smcginnis | OK, I'm getting out of here and bringing my girls to Star Trek. So y'all tomorrow. | 21:27 |
| hemna | smcginnis, w00t | 21:28 |
| hemna | have fun | 21:28 |
| mtanino | see ya! | 21:29 |
| *** diablo_rojo has quit IRC | 21:31 | |
| hemna | whoa | 21:31 |
| hemna | wtf happened to the sheepdog.py | 21:31 |
| hemna | *sigh* | 21:32 |
| _alastor_ | yeah, just commented on it :) | 21:32 |
| hemna | l a m e | 21:32 |
| _alastor_ | ha | 21:32 |
| *** xyang has quit IRC | 21:33 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_NAME to all drivers https://review.openstack.org/348002 | 21:34 |
| hemna | that should be better | 21:34 |
| hemna | doh | 21:34 |
| hemna | borked blockbridge as well | 21:34 |
| *** xyang has joined #openstack-cinder | 21:35 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_NAME to all drivers https://review.openstack.org/348002 | 21:35 |
| * patrickeast is kinda late to the party | 21:36 | |
| hemna | party! | 21:36 |
| patrickeast | so why are we adding in strings to the wiki pages? | 21:36 |
| patrickeast | and not just the actual ci name? | 21:36 |
| hemna | the tool is going to consume the output of the generate_driver_list.py script (https://gist.github.com/WaltHP/f335ebcc7d92308a4fa23685d053e659) | 21:37 |
| patrickeast | now depending on where you look my ci goes by: "Pure_Storage_CI", "Pure Storage CI", "pure-storage-ci", "cinder-ci@purestorage.com" | 21:37 |
| hemna | then use that to associate the driver to the data that's in the wiki page | 21:37 |
| hemna | everyone is different | 21:37 |
| hemna | we are also going to put the job names in a new dictionary on the wiki page as well | 21:38 |
| hemna | Jobs {'cinder': ['3par-iscsi-eos10', '3par-fc-eos10'], 'manila': ['3par-manila-eos10'], 'os-brick': ['3par-iscsi-brick-eos10']} | 21:39 |
| patrickeast | yea, i guess i'm not a huge fan of the indirection | 21:39 |
| hemna | well, job names change all the time | 21:39 |
| hemna | at least for us | 21:39 |
| patrickeast | that string in the cinder driver basically means nothing except for how mediawiki interpreted my CI name | 21:39 |
| hemna | and the wiki is updateable outside of gerrit | 21:39 |
| hemna | there is no other way to correctly describe where to get the info (from the wiki page) | 21:39 |
| *** xyang has quit IRC | 21:40 | |
| hemna | I suppose I could rename the variable | 21:40 |
| hemna | CI_NAME -> CI_WIKI_NAME | 21:40 |
| hemna | the problem is the wiki page is the source of truth for a lot of shit | 21:40 |
| patrickeast | that would at least make it less confusing, since for almost everyone it isn't actually the CI name | 21:40 |
| hemna | and the data in the wiki page for each ci can change | 21:41 |
| xyang1 | hemna: I think that's more clear, just don't make me rename the wiki page again | 21:41 |
| patrickeast | lol | 21:41 |
| hemna | :P | 21:41 |
| * hemna pulls out my sed skillz | 21:42 | |
| xyang1 | hemna: patrickeast because you can't reallt rename them, you can only recreate them | 21:42 |
| *** rdx_matt has joined #openstack-cinder | 21:42 | |
| xyang1 | so the old ones are still there and I don't know how to get rid of them | 21:43 |
| *** rdx_matt has quit IRC | 21:43 | |
| patrickeast | yea the wiki is kind of a pain for stuff like that | 21:43 |
| *** rdx_matt has joined #openstack-cinder | 21:43 | |
| patrickeast | which is why im a little bit concerned about having it be the nexus of our information with pointers from cinder to it | 21:43 |
| patrickeast | but, if it works, it works | 21:44 |
| *** FransUrbo has joined #openstack-cinder | 21:44 | |
| FransUrbo | How do I specify what volume backend store to use when creating an instance w/ a volume? | 21:45 |
| hemna | https://gist.github.com/WaltHP/75b07e89ce1ea4d10532fb991de0dcee | 21:45 |
| hemna | that look betters ? | 21:45 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_WIKI_NAME to all drivers https://review.openstack.org/348002 | 21:46 |
| *** ketonne has joined #openstack-cinder | 21:48 | |
| xyang1 | hemna: looks better | 21:48 |
| hemna | sweet | 21:49 |
| *** alonma has joined #openstack-cinder | 21:49 | |
| *** haplo37__ has quit IRC | 21:52 | |
| *** alonma has quit IRC | 21:54 | |
| *** jgregor has quit IRC | 21:54 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Disabling online volume copy in HPE 3PAR drivers https://review.openstack.org/348060 | 22:02 |
| *** Lee1092 has quit IRC | 22:03 | |
| *** ducttape_ has quit IRC | 22:05 | |
| *** mdenny has quit IRC | 22:08 | |
| *** rfinn has joined #openstack-cinder | 22:08 | |
| *** mdenny has joined #openstack-cinder | 22:09 | |
| *** IlyaG has quit IRC | 22:11 | |
| *** enriquetaso has quit IRC | 22:12 | |
| *** mriedem has quit IRC | 22:12 | |
| *** xyang1 has quit IRC | 22:13 | |
| *** salv-orlando has quit IRC | 22:13 | |
| *** mdenny has quit IRC | 22:13 | |
| *** nmadhok has joined #openstack-cinder | 22:15 | |
| *** rdx_matt has quit IRC | 22:17 | |
| *** salv-orlando has joined #openstack-cinder | 22:17 | |
| *** hemna is now known as hemnafk | 22:18 | |
| *** mdenny has joined #openstack-cinder | 22:19 | |
| *** david-lyle has joined #openstack-cinder | 22:22 | |
| *** salv-orlando has quit IRC | 22:23 | |
| *** edmondsw has quit IRC | 22:25 | |
| *** david-lyle has quit IRC | 22:25 | |
| *** david-lyle has joined #openstack-cinder | 22:25 | |
| *** FransUrbo has quit IRC | 22:31 | |
| *** diogogmt_ has joined #openstack-cinder | 22:33 | |
| *** diogogmt has quit IRC | 22:34 | |
| *** diogogmt_ is now known as diogogmt | 22:34 | |
| *** tpsilva has quit IRC | 22:38 | |
| *** gman-tx has quit IRC | 22:40 | |
| *** jwcroppe has quit IRC | 22:40 | |
| *** jwcroppe has joined #openstack-cinder | 22:41 | |
| *** leeantho has quit IRC | 22:41 | |
| *** IlyaG has joined #openstack-cinder | 22:44 | |
| *** gouthamr has quit IRC | 22:44 | |
| *** jwcroppe has quit IRC | 22:45 | |
| *** jamielennox is now known as jamielennox|away | 22:45 | |
| *** IlyaG has quit IRC | 22:49 | |
| *** david-lyle has quit IRC | 22:50 | |
| *** david-lyle_ has joined #openstack-cinder | 22:50 | |
| *** alonma has joined #openstack-cinder | 22:50 | |
| *** alonma has quit IRC | 22:54 | |
| *** diogogmt has quit IRC | 22:55 | |
| *** diogogmt has joined #openstack-cinder | 22:56 | |
| *** alonma has joined #openstack-cinder | 22:57 | |
| *** david-lyle_ has quit IRC | 23:01 | |
| *** alonma has quit IRC | 23:02 | |
| *** xianghui has joined #openstack-cinder | 23:03 | |
| *** mriedem has joined #openstack-cinder | 23:03 | |
| *** cknight has quit IRC | 23:10 | |
| *** rdx_matt has joined #openstack-cinder | 23:12 | |
| *** rdx_matt has quit IRC | 23:13 | |
| *** rdx_matt has joined #openstack-cinder | 23:13 | |
| *** sdake has joined #openstack-cinder | 23:17 | |
| patrickeast | heh, thats a new one | 23:17 |
| patrickeast | turns out the flavor matching in nodepool uses 'in' https://github.com/openstack-infra/nodepool/blob/master/nodepool/provider_manager.py#L135 | 23:17 |
| patrickeast | so setting something up with flavor == 'devstack-node' will match a flavor called 'devstack-node-fc' in that code :( | 23:18 |
| * patrickeast has gotten lucky for the last year with the order of the list matching the right ones | 23:18 | |
| *** rfinn has quit IRC | 23:24 | |
| *** sdake has quit IRC | 23:25 | |
| *** mtanino has quit IRC | 23:37 | |
| *** ducttape_ has joined #openstack-cinder | 23:37 | |
| *** bill_az has joined #openstack-cinder | 23:38 | |
| *** chlong has quit IRC | 23:38 | |
| *** salv-orlando has joined #openstack-cinder | 23:42 | |
| *** rdx_matt has quit IRC | 23:47 | |
| *** salv-orlando has quit IRC | 23:47 | |
| *** Apoorva_ has joined #openstack-cinder | 23:49 | |
| *** Apoorva has quit IRC | 23:52 | |
| *** Apoorva_ has quit IRC | 23:53 | |
| *** diogogmt has quit IRC | 23:55 | |
| *** akerr has quit IRC | 23:58 | |
| *** akerr has joined #openstack-cinder | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!