patrickeast | honestly i don't even know if things work with it enabled even without that change | 00:00 |
---|---|---|
patrickeast | so maybe it isn't an issue | 00:00 |
hemna | hrmm | 00:00 |
hemna | so in a cloud I’m not sure why you’d want to enable it | 00:01 |
hemna | using the WWN for the path is far safer | 00:01 |
patrickeast | agreed | 00:01 |
hemna | as it’s guaranteed to be unique and always the same on every host | 00:01 |
hemna | hrmm | 00:02 |
patrickeast | so maybe its fine to just say as a best practice don't turn it on | 00:02 |
patrickeast | ? | 00:02 |
openstackgerrit | Alex O'Rourke proposed openstack/cinder: LeftHand: Adding Consistency Group Support https://review.openstack.org/214386 | 00:02 |
patrickeast | or could brick disable it? | 00:02 |
hemna | I think we had a bug with redhat 7 when they turned on friendly names and it broke lots of stuff already | 00:02 |
hemna | I’ll have to play with that tomorrow | 00:03 |
hemna | thanks for bringing that up :) | 00:03 |
patrickeast | np | 00:03 |
kmartin | redhat fixed the code in nova to support the user friendly names | 00:04 |
*** smoriya has joined #openstack-cinder | 00:04 | |
hemna | might be able to reverse lookup the mpath device from the WWN if it’s enabled | 00:05 |
hemna | ok guys….l8rs…soccer | 00:05 |
*** hemna has quit IRC | 00:06 | |
openstackgerrit | Sonia Ghanekar proposed openstack/cinder: Add manage/unmanage volume support for Nimble https://review.openstack.org/214392 | 00:08 |
*** lixiaoy1 has joined #openstack-cinder | 00:09 | |
*** vilobhmm1 has joined #openstack-cinder | 00:09 | |
*** gouthamr has quit IRC | 00:11 | |
*** garthb has quit IRC | 00:11 | |
*** simondodsley has joined #openstack-cinder | 00:17 | |
*** lixiaoy11 has joined #openstack-cinder | 00:28 | |
*** vilobhmm1 has quit IRC | 00:29 | |
*** tsekiyam_ has joined #openstack-cinder | 00:29 | |
*** martyturner has quit IRC | 00:29 | |
*** IlyaG has quit IRC | 00:29 | |
*** lixiaoy1 has quit IRC | 00:30 | |
*** mtanino has quit IRC | 00:31 | |
*** tsekiyama has quit IRC | 00:32 | |
*** tsekiyam_ has quit IRC | 00:34 | |
*** salv-orl_ has quit IRC | 00:34 | |
*** lixiaoy11 has left #openstack-cinder | 00:36 | |
*** lixiaoy11 has joined #openstack-cinder | 00:36 | |
*** _cjones_ has quit IRC | 00:39 | |
openstackgerrit | Alex O'Rourke proposed openstack/cinder: LeftHand: Adding Consistency Group Support https://review.openstack.org/214386 | 00:44 |
*** andreykurilin__ has quit IRC | 00:45 | |
*** dims has joined #openstack-cinder | 00:46 | |
*** dims_ has quit IRC | 00:47 | |
*** akerr has joined #openstack-cinder | 00:48 | |
*** Yogi1 has joined #openstack-cinder | 00:49 | |
*** skylerberg has quit IRC | 00:53 | |
*** dims has quit IRC | 00:56 | |
*** dims has joined #openstack-cinder | 00:56 | |
*** Yogi1 has quit IRC | 00:57 | |
*** zhonghua-lee has joined #openstack-cinder | 00:59 | |
*** lixiaoy11 has quit IRC | 01:03 | |
*** Yogi1 has joined #openstack-cinder | 01:08 | |
*** leeantho has quit IRC | 01:08 | |
*** simondodsley has quit IRC | 01:08 | |
*** zhenguo has joined #openstack-cinder | 01:09 | |
*** Yogi1 has quit IRC | 01:10 | |
*** hideme has joined #openstack-cinder | 01:11 | |
*** simondodsley has joined #openstack-cinder | 01:15 | |
*** Yogi1 has joined #openstack-cinder | 01:15 | |
*** cbader has joined #openstack-cinder | 01:15 | |
*** davechen has joined #openstack-cinder | 01:16 | |
*** asselin__ has joined #openstack-cinder | 01:20 | |
jgriffith | kmartin: ping | 01:21 |
*** asselin has quit IRC | 01:22 | |
*** julim has joined #openstack-cinder | 01:24 | |
*** asselin__ has quit IRC | 01:25 | |
*** cbader has quit IRC | 01:25 | |
*** zhongjun has joined #openstack-cinder | 01:30 | |
*** Apoorva has quit IRC | 01:34 | |
*** simondodsley has quit IRC | 01:34 | |
*** Yogi1 has quit IRC | 01:38 | |
*** Lee1092 has joined #openstack-cinder | 01:40 | |
*** akerr has quit IRC | 01:52 | |
*** RuiChen has joined #openstack-cinder | 02:04 | |
*** haomaiwang has joined #openstack-cinder | 02:05 | |
*** RuiChen has quit IRC | 02:11 | |
*** diogogmt has joined #openstack-cinder | 02:11 | |
*** baojg has joined #openstack-cinder | 02:15 | |
*** RuiChen has joined #openstack-cinder | 02:16 | |
openstackgerrit | XinXiaohui proposed openstack/cinder: Calculate virtual free capacity and notify https://review.openstack.org/206923 | 02:18 |
*** RuiChen has left #openstack-cinder | 02:18 | |
openstackgerrit | xing-yang proposed openstack/cinder: ScaleIO driver should use os-brick connector https://review.openstack.org/214420 | 02:18 |
*** zhongjun has quit IRC | 02:23 | |
*** annegentle has quit IRC | 02:25 | |
*** annegentle has joined #openstack-cinder | 02:26 | |
*** annegentle has quit IRC | 02:32 | |
*** baojg has quit IRC | 02:35 | |
*** p0rtal_ has joined #openstack-cinder | 02:36 | |
*** p0rtal has quit IRC | 02:39 | |
*** Liu has joined #openstack-cinder | 02:44 | |
*** lxg has quit IRC | 02:47 | |
*** baojg has joined #openstack-cinder | 02:48 | |
*** salv-orlando has joined #openstack-cinder | 02:49 | |
*** hakimo_ has joined #openstack-cinder | 02:52 | |
*** mylu has joined #openstack-cinder | 02:54 | |
*** hakimo has quit IRC | 02:54 | |
*** dims has quit IRC | 02:55 | |
*** tiny-hands has quit IRC | 02:59 | |
*** haomaiwang has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-cinder | 03:02 | |
*** salv-orlando has quit IRC | 03:05 | |
*** asselin has joined #openstack-cinder | 03:13 | |
*** tbarron has quit IRC | 03:15 | |
*** tiny-hands has joined #openstack-cinder | 03:16 | |
*** rodrigods has quit IRC | 03:16 | |
*** erlon has quit IRC | 03:18 | |
*** rodrigods has joined #openstack-cinder | 03:19 | |
*** tbarron has joined #openstack-cinder | 03:20 | |
*** annasort has quit IRC | 03:24 | |
*** sparr has quit IRC | 03:28 | |
*** logan2 has quit IRC | 03:31 | |
*** zhonghua-lee has quit IRC | 03:31 | |
*** sparr has joined #openstack-cinder | 03:32 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 03:35 |
openstackgerrit | Wilson Liu proposed openstack/cinder: Add hypermetro support for Huawei driver https://review.openstack.org/202023 | 03:42 |
*** diogogmt has quit IRC | 03:42 | |
*** zhongjun has joined #openstack-cinder | 03:44 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Fix way of get LUN id in Huawei driver https://review.openstack.org/209749 | 03:46 |
*** haigang has joined #openstack-cinder | 03:47 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Add volume retype support for Huawei driver https://review.openstack.org/201485 | 03:51 |
*** mylu has quit IRC | 03:54 | |
*** p0rtal has joined #openstack-cinder | 03:58 | |
*** p0rtal has quit IRC | 03:59 | |
*** julim has quit IRC | 03:59 | |
*** p0rtal has joined #openstack-cinder | 03:59 | |
*** haomaiwang has quit IRC | 04:01 | |
*** p0rtal_ has quit IRC | 04:01 | |
*** haomaiwang has joined #openstack-cinder | 04:02 | |
*** p0rtal has quit IRC | 04:04 | |
*** salv-orlando has joined #openstack-cinder | 04:04 | |
*** asselin has quit IRC | 04:06 | |
*** asselin has joined #openstack-cinder | 04:07 | |
*** tbarron has quit IRC | 04:08 | |
openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 04:10 |
*** pv has quit IRC | 04:11 | |
*** angerson has quit IRC | 04:12 | |
*** garthb has joined #openstack-cinder | 04:13 | |
*** sparr has quit IRC | 04:14 | |
*** BharatK has joined #openstack-cinder | 04:15 | |
*** tbarron has joined #openstack-cinder | 04:17 | |
*** angerson has joined #openstack-cinder | 04:18 | |
*** salv-orlando has quit IRC | 04:21 | |
*** mylu has joined #openstack-cinder | 04:24 | |
*** sparr has joined #openstack-cinder | 04:28 | |
*** shyama has joined #openstack-cinder | 04:33 | |
openstackgerrit | yuji hagiwara proposed openstack/cinder: Remove duplicate keys from dictionary https://review.openstack.org/213999 | 04:43 |
*** coolsvap|away is now known as coolsvap | 04:46 | |
*** tbarron has quit IRC | 04:48 | |
*** deepakcs has joined #openstack-cinder | 04:52 | |
*** zhongjun has quit IRC | 04:52 | |
*** tbarron has joined #openstack-cinder | 04:55 | |
*** p0rtal has joined #openstack-cinder | 04:58 | |
*** baojg has quit IRC | 04:59 | |
*** lixiaoy1 has joined #openstack-cinder | 04:59 | |
*** haomaiwang has quit IRC | 05:01 | |
*** haomaiwa_ has joined #openstack-cinder | 05:02 | |
jgriffith | patrickeast: a few questions/comments regarding your caching patch | 05:07 |
jgriffith | patrickeast: I'll try and catch up with you tomorrow | 05:07 |
*** BharatK has quit IRC | 05:08 | |
*** coolsvap is now known as coolsvap|away | 05:08 | |
*** vilobhmm1 has joined #openstack-cinder | 05:14 | |
*** mylu has quit IRC | 05:18 | |
*** garthb has quit IRC | 05:19 | |
*** boris-42 has quit IRC | 05:20 | |
*** shausy has joined #openstack-cinder | 05:23 | |
*** coolsvap|away is now known as coolsvap | 05:23 | |
*** mylu has joined #openstack-cinder | 05:25 | |
*** prometheanfire has joined #openstack-cinder | 05:28 | |
*** salv-orlando has joined #openstack-cinder | 05:28 | |
*** sgotliv_ has joined #openstack-cinder | 05:29 | |
*** mylu has quit IRC | 05:29 | |
prometheanfire | thingee: so, about a proper zfs driver for cinder... | 05:30 |
prometheanfire | thingee: you are at the midcycle (as am I), so will probably bug you tomorrow about it | 05:30 |
*** p0rtal has quit IRC | 05:34 | |
*** shausy has quit IRC | 05:36 | |
*** shausy has joined #openstack-cinder | 05:36 | |
*** BharatK has joined #openstack-cinder | 05:45 | |
*** BharatK has quit IRC | 05:51 | |
*** sgotliv_ has quit IRC | 05:52 | |
*** baojg has joined #openstack-cinder | 05:58 | |
*** haomaiwa_ has quit IRC | 06:01 | |
*** haomaiwa_ has joined #openstack-cinder | 06:02 | |
*** p0rtal has joined #openstack-cinder | 06:02 | |
*** BharatK has joined #openstack-cinder | 06:03 | |
*** lpetrut has joined #openstack-cinder | 06:05 | |
*** afazkas has joined #openstack-cinder | 06:06 | |
*** p0rtal has quit IRC | 06:07 | |
*** hw_chenying has joined #openstack-cinder | 06:08 | |
*** coolsvap is now known as coolsvap|away | 06:10 | |
*** kaisers has joined #openstack-cinder | 06:10 | |
*** vilobhmm1 has quit IRC | 06:11 | |
*** mylu has joined #openstack-cinder | 06:12 | |
*** p0rtal has joined #openstack-cinder | 06:12 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/214468 | 06:14 |
*** mylu has quit IRC | 06:16 | |
*** vlaza has joined #openstack-cinder | 06:22 | |
*** yrabl has joined #openstack-cinder | 06:27 | |
*** ankit_ag has joined #openstack-cinder | 06:28 | |
*** markus_z has joined #openstack-cinder | 06:30 | |
*** agarciam has joined #openstack-cinder | 06:36 | |
*** nkrinner has joined #openstack-cinder | 06:39 | |
*** salv-orlando has quit IRC | 06:40 | |
*** zhongjun has joined #openstack-cinder | 06:50 | |
*** p0rtal has quit IRC | 06:50 | |
*** haomaiwa_ has quit IRC | 07:01 | |
*** haomaiwang has joined #openstack-cinder | 07:01 | |
*** zhonghua-lee has joined #openstack-cinder | 07:14 | |
*** sileht has joined #openstack-cinder | 07:15 | |
*** ronis has joined #openstack-cinder | 07:16 | |
*** mylu has joined #openstack-cinder | 07:16 | |
*** ronis_ has joined #openstack-cinder | 07:18 | |
*** ronis__ has joined #openstack-cinder | 07:19 | |
*** anshul has quit IRC | 07:19 | |
*** anshul has joined #openstack-cinder | 07:20 | |
*** ronis has quit IRC | 07:21 | |
*** mylu has quit IRC | 07:21 | |
openstackgerrit | wanghao proposed openstack/cinder: Add updated_at into response of listing detail https://review.openstack.org/202029 | 07:22 |
*** lpetrut has quit IRC | 07:26 | |
*** belmoreira has joined #openstack-cinder | 07:28 | |
*** simondodsley has joined #openstack-cinder | 07:32 | |
*** agarciam has quit IRC | 07:32 | |
*** simondodsley has quit IRC | 07:36 | |
*** IanGovett has joined #openstack-cinder | 07:37 | |
*** andreykurilin__ has joined #openstack-cinder | 07:38 | |
*** ronis_ has quit IRC | 07:38 | |
*** IanGovett has quit IRC | 07:41 | |
*** salv-orlando has joined #openstack-cinder | 07:41 | |
*** IanGovett has joined #openstack-cinder | 07:44 | |
openstackgerrit | Xiangfei Zhu proposed openstack/cinder: VMware: Fix re-attach volume error for VC 5.1 https://review.openstack.org/207380 | 07:45 |
*** ronis__ has quit IRC | 07:52 | |
*** agarciam has joined #openstack-cinder | 07:53 | |
*** deepakcs has quit IRC | 07:54 | |
*** boris-42 has joined #openstack-cinder | 07:58 | |
*** zhongjun has quit IRC | 07:59 | |
*** haomaiwang has quit IRC | 08:01 | |
openstackgerrit | zhaohua proposed openstack/cinder: Huawei: fix multi REST-URLs bug https://review.openstack.org/214500 | 08:01 |
*** haomaiwang has joined #openstack-cinder | 08:02 | |
openstackgerrit | zhaohua proposed openstack/cinder: Huawei: fix multi REST-URLs bug https://review.openstack.org/214500 | 08:05 |
openstackgerrit | Lei Zhang proposed openstack/cinder: Prevent concurrent backups for same volume https://review.openstack.org/202914 | 08:06 |
*** deepakcs has joined #openstack-cinder | 08:11 | |
*** skiafa has quit IRC | 08:14 | |
*** ociuhandu has quit IRC | 08:20 | |
*** jordanP has joined #openstack-cinder | 08:20 | |
*** timbyr_ has joined #openstack-cinder | 08:21 | |
*** markvoelker has quit IRC | 08:22 | |
*** jistr has joined #openstack-cinder | 08:34 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/213891 | 08:34 |
*** lpetrut has joined #openstack-cinder | 08:39 | |
*** alexschm has joined #openstack-cinder | 08:47 | |
*** haigang has quit IRC | 08:52 | |
*** haigang has joined #openstack-cinder | 08:52 | |
*** shyama_ has joined #openstack-cinder | 08:52 | |
*** haigang has quit IRC | 08:52 | |
*** haigang has joined #openstack-cinder | 08:53 | |
*** haigang has quit IRC | 08:53 | |
*** haigang has joined #openstack-cinder | 08:53 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Use version convert methods from oslo.utils https://review.openstack.org/214516 | 08:54 |
*** aix has joined #openstack-cinder | 08:54 | |
*** shyama has quit IRC | 08:55 | |
*** shyama_ is now known as shyama | 08:55 | |
*** sgotliv has joined #openstack-cinder | 08:57 | |
*** haomaiwang has quit IRC | 09:01 | |
*** haomaiwa_ has joined #openstack-cinder | 09:02 | |
*** lixiaoy1 has quit IRC | 09:14 | |
*** ishant has joined #openstack-cinder | 09:19 | |
*** markvoelker has joined #openstack-cinder | 09:23 | |
*** bluex-pl has joined #openstack-cinder | 09:24 | |
*** bluex-pl has quit IRC | 09:24 | |
*** bluex-pl has joined #openstack-cinder | 09:25 | |
*** markvoelker has quit IRC | 09:28 | |
*** davechen has left #openstack-cinder | 09:43 | |
*** jaypipes has quit IRC | 09:46 | |
*** ociuhandu has joined #openstack-cinder | 09:54 | |
*** mylu has joined #openstack-cinder | 09:58 | |
*** haomaiwa_ has quit IRC | 10:01 | |
*** haomaiwang has joined #openstack-cinder | 10:02 | |
*** mylu has quit IRC | 10:02 | |
*** belmoreira has quit IRC | 10:08 | |
*** belmoreira has joined #openstack-cinder | 10:09 | |
*** BharatK has quit IRC | 10:11 | |
*** BharatK has joined #openstack-cinder | 10:13 | |
*** belmoreira has quit IRC | 10:14 | |
*** belmoreira has joined #openstack-cinder | 10:15 | |
*** abhi has quit IRC | 10:15 | |
*** kmartin has quit IRC | 10:16 | |
*** asselin_ has quit IRC | 10:16 | |
*** mdenny has quit IRC | 10:17 | |
*** baojg has quit IRC | 10:26 | |
*** baojg has joined #openstack-cinder | 10:27 | |
*** zhonghua-lee has quit IRC | 10:28 | |
*** gouthamr has joined #openstack-cinder | 10:31 | |
*** baojg has quit IRC | 10:31 | |
*** gouthamr has quit IRC | 10:31 | |
*** gouthamr has joined #openstack-cinder | 10:32 | |
*** hw_chenying has quit IRC | 10:35 | |
geguileo | jungleboyj: Hi, you already +2 this patch before, could you have another look please? https://review.openstack.org/#/c/211257/ | 10:47 |
*** belmoreira has quit IRC | 10:48 | |
openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 10:48 |
*** salv-orlando has quit IRC | 10:53 | |
*** dims has joined #openstack-cinder | 10:56 | |
*** ronis has joined #openstack-cinder | 10:59 | |
*** ccesario_ has quit IRC | 10:59 | |
*** gouthamr has quit IRC | 11:00 | |
*** dims_ has joined #openstack-cinder | 11:03 | |
aarefiev | DuncanT: hi, you asked about job for api under apache: https://review.openstack.org/#/c/208498/ | 11:05 |
DuncanT | aarefiev: That's great, thanks. I'll add a comment in support | 11:06 |
aarefiev | DuncanT: thanks | 11:06 |
*** dims has quit IRC | 11:07 | |
*** ishant has quit IRC | 11:10 | |
*** haomaiwang has quit IRC | 11:12 | |
*** anshul has quit IRC | 11:15 | |
*** anshul has joined #openstack-cinder | 11:16 | |
*** ociuhandu has quit IRC | 11:16 | |
*** ociuhandu has joined #openstack-cinder | 11:18 | |
*** boris-42 has quit IRC | 11:20 | |
*** markvoelker has joined #openstack-cinder | 11:24 | |
*** markvoelker has quit IRC | 11:28 | |
*** dims has joined #openstack-cinder | 11:29 | |
*** tiny-hands has quit IRC | 11:29 | |
*** cdelatte has joined #openstack-cinder | 11:31 | |
*** delattec has joined #openstack-cinder | 11:31 | |
*** dims_ has quit IRC | 11:31 | |
*** dims_ has joined #openstack-cinder | 11:31 | |
*** dims has quit IRC | 11:34 | |
*** diogogmt has joined #openstack-cinder | 11:46 | |
*** jaypipes has joined #openstack-cinder | 11:53 | |
*** mylu has joined #openstack-cinder | 11:59 | |
*** mylu has quit IRC | 12:03 | |
*** afazkas has quit IRC | 12:04 | |
*** andreykurilin__ has quit IRC | 12:05 | |
*** julim has joined #openstack-cinder | 12:10 | |
*** raildo-afk is now known as raildo | 12:14 | |
adrianofr | smcginnis: thingee: Hi, would you mind taking a look again in https://review.openstack.org/#/c/197999/ ? | 12:15 |
*** bswartz has quit IRC | 12:15 | |
adrianofr | smcginnis: thingee: It was already approved before, but we faced some merge conflicts | 12:15 |
*** belmoreira has joined #openstack-cinder | 12:19 | |
*** belmoreira has quit IRC | 12:19 | |
*** diogogmt has quit IRC | 12:20 | |
*** erlon has joined #openstack-cinder | 12:22 | |
*** deepakcs has quit IRC | 12:23 | |
*** salv-orlando has joined #openstack-cinder | 12:25 | |
*** angerson has quit IRC | 12:26 | |
*** angerson has joined #openstack-cinder | 12:26 | |
*** chlong has joined #openstack-cinder | 12:27 | |
*** edmondsw has joined #openstack-cinder | 12:27 | |
DuncanT | adrianofr: +2/A again | 12:29 |
adrianofr | DuncanT: Thanks Duncan :) | 12:30 |
*** salv-orl_ has joined #openstack-cinder | 12:32 | |
*** salv-orlando has quit IRC | 12:33 | |
*** kjnelson has quit IRC | 12:33 | |
*** yuriy_n17 has joined #openstack-cinder | 12:35 | |
*** smoriya has quit IRC | 12:36 | |
*** logan2 has joined #openstack-cinder | 12:39 | |
*** haomaiwang has joined #openstack-cinder | 12:39 | |
*** haigang has quit IRC | 12:39 | |
*** kjnelson has joined #openstack-cinder | 12:40 | |
*** markvoelker has joined #openstack-cinder | 12:40 | |
*** julim has quit IRC | 12:41 | |
*** julim has joined #openstack-cinder | 12:42 | |
*** porrua has joined #openstack-cinder | 12:43 | |
*** akerr has joined #openstack-cinder | 12:44 | |
*** martyturner has joined #openstack-cinder | 12:44 | |
*** martyturner has quit IRC | 12:45 | |
*** eharney has joined #openstack-cinder | 12:45 | |
*** BharatK has quit IRC | 12:52 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Add volume retype support for Huawei driver https://review.openstack.org/201485 | 12:54 |
openstackgerrit | Anton Arefiev proposed openstack/cinder: WIP:Add rootwrap daemon mode support https://review.openstack.org/149677 | 12:56 |
openstackgerrit | Anton Arefiev proposed openstack/cinder: WIP: Switch to use execute from cinder utils https://review.openstack.org/214595 | 12:56 |
openstackgerrit | Wilson Liu proposed openstack/cinder: Add hypermetro support for Huawei driver https://review.openstack.org/202023 | 12:59 |
*** xyang1 has joined #openstack-cinder | 13:00 | |
*** tiny-hands has joined #openstack-cinder | 13:01 | |
*** bill_az has joined #openstack-cinder | 13:01 | |
*** martyturner has joined #openstack-cinder | 13:03 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 13:04 |
*** gouthamr has joined #openstack-cinder | 13:06 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Fix way of get LUN id in Huawei driver https://review.openstack.org/209749 | 13:07 |
*** yrabl_ has quit IRC | 13:07 | |
*** yrabl has quit IRC | 13:08 | |
*** yrabl_ has joined #openstack-cinder | 13:08 | |
*** yrabl_ is now known as yrabl | 13:08 | |
*** yrabl has quit IRC | 13:08 | |
*** yrabl has joined #openstack-cinder | 13:09 | |
*** BharatK has joined #openstack-cinder | 13:09 | |
eharney | jungleboyj: can you visit https://review.openstack.org/#/c/211257/ again? (previously +2'd) | 13:13 |
*** annegentle has joined #openstack-cinder | 13:13 | |
jungleboyj | eharney: Can do. | 13:14 |
*** esker has joined #openstack-cinder | 13:14 | |
*** BharatK has quit IRC | 13:14 | |
eharney | jungleboyj: thanks | 13:15 |
*** JoseMello has joined #openstack-cinder | 13:16 | |
*** andreykurilin__ has joined #openstack-cinder | 13:16 | |
*** BharatK has joined #openstack-cinder | 13:16 | |
*** dustins has joined #openstack-cinder | 13:19 | |
*** lcurtis has joined #openstack-cinder | 13:22 | |
*** jgregor has joined #openstack-cinder | 13:23 | |
*** dims_ has quit IRC | 13:29 | |
*** dims has joined #openstack-cinder | 13:29 | |
*** mriedem has joined #openstack-cinder | 13:33 | |
*** annegentle has quit IRC | 13:37 | |
*** Yogi1 has joined #openstack-cinder | 13:38 | |
*** diogogmt has joined #openstack-cinder | 13:43 | |
openstackgerrit | Silvan Kaiser proposed openstack/cinder: Correct the order of AssertEquals params in Quobyte tests https://review.openstack.org/214613 | 13:43 |
openstackgerrit | Silvan Kaiser proposed openstack/cinder: Corrects the order of AssertEquals params in Quobyte tests https://review.openstack.org/214613 | 13:44 |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Adds the random option to cinder retry function https://review.openstack.org/214317 | 13:44 |
*** sgotliv has quit IRC | 13:44 | |
*** sgotliv has joined #openstack-cinder | 13:45 | |
*** Yogi11 has joined #openstack-cinder | 13:46 | |
*** diogogmt has quit IRC | 13:47 | |
*** Yogi1 has quit IRC | 13:48 | |
*** Yogi11 has quit IRC | 13:51 | |
*** jungleboyj has quit IRC | 13:57 | |
*** mriedem is now known as mriedem_meeting | 13:59 | |
yuriy_n17 | smcginnis: Hi. Please review https://review.openstack.org/#/c/211253/ | 13:59 |
openstackgerrit | Peter Wang proposed openstack/cinder: Support efficient non-disruptive volume backup in VNX https://review.openstack.org/214626 | 13:59 |
openstackgerrit | Peter Wang proposed openstack/cinder: Support efficient non-disruptive volume backup in VNX https://review.openstack.org/214626 | 14:01 |
*** ankit_ag has quit IRC | 14:03 | |
*** bswartz has joined #openstack-cinder | 14:05 | |
openstackgerrit | Peter Wang proposed openstack/cinder: Support efficient non-disruptive volume backup in VNX https://review.openstack.org/214626 | 14:05 |
yuriy_n17 | smcginnis: Thanks a lot. | 14:06 |
smcginnis | yuriy_n17: No problem. Thanks for adding that. | 14:06 |
*** kbyrne has quit IRC | 14:07 | |
*** timbyr_ has quit IRC | 14:07 | |
yuriy_n17 | smcginnis: what about to review https://review.openstack.org/#/c/207469/ | 14:08 |
smcginnis | yuriy_n17: Sure. I need to catch up on some other things first, but I have it open in a tab now. | 14:08 |
yuriy_n17 | smcginnis: ok. Thanks for advance. | 14:09 |
*** thangp has joined #openstack-cinder | 14:10 | |
*** kbyrne has joined #openstack-cinder | 14:11 | |
*** boris-42 has joined #openstack-cinder | 14:15 | |
*** mtanino has joined #openstack-cinder | 14:18 | |
*** edmondsw has quit IRC | 14:22 | |
*** vlaza has quit IRC | 14:27 | |
*** kjnelson has quit IRC | 14:29 | |
*** deepakcs has joined #openstack-cinder | 14:33 | |
*** deepakcs has quit IRC | 14:34 | |
*** edmondsw has joined #openstack-cinder | 14:37 | |
*** crose has joined #openstack-cinder | 14:37 | |
*** mriedem_meeting is now known as mriedem | 14:37 | |
openstackgerrit | John Griffith proposed openstack/cinder: Cinder replication V2 https://review.openstack.org/200269 | 14:38 |
jgriffith | yuriy_n17: minor nit on that patch | 14:40 |
jgriffith | yuriy_n17: when we did V2 one of the things that people said they would like was to s/display_name/name/ | 14:41 |
jgriffith | yuriy_n17: seems like we could just use name right from the start with he backup info and just avoid the whole display_name debate altogtether? | 14:42 |
jgriffith | yuriy_n17: either that or somebody will come along and add it later :( | 14:42 |
smcginnis | jgriffith: Do we need to support both name and display_name, or just get rid of name all together? | 14:42 |
*** alexschm has quit IRC | 14:42 | |
jgriffith | smcginnis: the other way around, get rid of display_name altogether | 14:42 |
smcginnis | Oh, right. | 14:43 |
jgriffith | smcginnis: yuriy_n17 Like many things we've been kinda non comital about it, but maybe we shouldn't be | 14:43 |
openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Service object https://review.openstack.org/160417 | 14:44 |
smcginnis | jgriffith: I agree. If that's what we're leaning toward then we should just commit to it. | 14:44 |
openstackgerrit | Alex O'Rourke proposed openstack/cinder: LeftHand: Adding Consistency Group Support https://review.openstack.org/214386 | 14:44 |
*** annegentle has joined #openstack-cinder | 14:46 | |
*** annegentle has quit IRC | 14:47 | |
*** annegentle has joined #openstack-cinder | 14:47 | |
*** mylu has joined #openstack-cinder | 14:49 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' for cinder backup-list https://review.openstack.org/207469 | 14:53 |
*** diogogmt has joined #openstack-cinder | 14:54 | |
yuriy_n17 | jgriffith: Hi. Please review https://review.openstack.org/#/c/207469/ | 14:55 |
smcginnis | yuriy_n17: Your commit message needs to be updated now. | 14:56 |
jgriffith | yuriy_n17: one question from me :) | 14:57 |
jgriffith | yuriy_n17: Why are we using an environment variable for ALL_TENANTS? | 14:57 |
jgriffith | yuriy_n17: as opposed to it being a flag to the command? | 14:57 |
jgriffith | yuriy_n17: I mean, I realize others did that.... I'm just wondering "why"? | 14:58 |
jgriffith | yuriy_n17: not on your patch to fix it of course | 14:59 |
*** ccesario has joined #openstack-cinder | 15:00 | |
ccesario | j #openstack-dev | 15:00 |
yuriy_n17 | jgriffith: Good catch. Thanks. Just fix it. | 15:00 |
jgriffith | yuriy_n17: I don't think you need to change it in your patch | 15:00 |
jgriffith | yuriy_n17: we're doing it on other objects, I just don't know why :( | 15:00 |
*** dims has quit IRC | 15:01 | |
*** logan2 has quit IRC | 15:01 | |
jgriffith | yuriy_n17: so we have a special case that takes an environ var or the all_tenants arg | 15:01 |
*** dims has joined #openstack-cinder | 15:01 | |
jgriffith | not sure I get why it's that way is all | 15:01 |
jgriffith | yuriy_n17: FWIW I gave your patch a +2 | 15:01 |
smcginnis | yuriy_n17, jgriffith: I'd like to see the commit message updated just so there's no chance of confusion on display_name. | 15:02 |
jgriffith | smcginnis: DOH | 15:02 |
jgriffith | Yeah, good call | 15:02 |
*** BharatK has quit IRC | 15:02 | |
*** garthb has joined #openstack-cinder | 15:03 | |
yuriy_n17 | smcginnis: ok | 15:04 |
*** dims_ has joined #openstack-cinder | 15:05 | |
*** logan2 has joined #openstack-cinder | 15:05 | |
*** dims has quit IRC | 15:06 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' for cinder backup-list https://review.openstack.org/207469 | 15:06 |
jgriffith | yuriy_n17: Thanks! smcginnis Ship It!!! | 15:07 |
jgriffith | :) | 15:07 |
smcginnis | ;) | 15:07 |
*** zul has joined #openstack-cinder | 15:07 | |
yuriy_n17 | jgriffith, smcginnis: Thanks a lot for your comments. | 15:08 |
*** hemnafk is now known as hemna | 15:11 | |
*** dannywilson has joined #openstack-cinder | 15:13 | |
*** dannywilson has quit IRC | 15:14 | |
eharney | geguileo: re: comments on https://review.openstack.org/#/c/183701/ -- that does look dangerous | 15:14 |
*** dannywilson has joined #openstack-cinder | 15:14 | |
eharney | geguileo: since the patch before it needs a change, if mtanino isn't around to fix that quickly, i'd suggest you fix it and re-up the patch and i'll add a +2 back to it | 15:15 |
geguileo | eharney: I'm not sure if I'm right, because I haven't looked into capabilities... | 15:15 |
mtanino | eharney: Thank you for the review. I will fix quickly. | 15:15 |
eharney | geguileo: i'm not either, but, i think so | 15:15 |
eharney | mtanino: ok great | 15:15 |
geguileo | mtanino: ping - https://review.openstack.org/#/c/183701/14/cinder/volume/driver.py,cm | 15:15 |
mtanino | geguileo: your comment is right. | 15:15 |
mtanino | geguileo: I wil fix it asap | 15:16 |
geguileo | mtanino: Thanks for the quick response!! | 15:16 |
*** coolsvap|away is now known as coolsvap | 15:18 | |
thrawn01 | cinderclient.tests.unit.test_shell.ShellTest.test_password_prompted test is failing on master, anyone else see this? I'm using python-keystoneclient==1.6.0 | 15:18 |
thrawn01 | did someone forget to bump the keystoneclient required version? | 15:18 |
eharney | mtanino: you probably saw, but i put notes on the dependent patch as well | 15:19 |
*** ronis has quit IRC | 15:19 | |
*** dims_ has quit IRC | 15:19 | |
*** dims has joined #openstack-cinder | 15:19 | |
mtanino | eharney: you mean typo for thin_provisioning? | 15:19 |
eharney | mtanino: yes | 15:19 |
mtanino | eharney: yes, I will fix these two patches at once. thanks, | 15:20 |
*** coolsvap is now known as coolsvap|away | 15:22 | |
*** daneyon has joined #openstack-cinder | 15:22 | |
*** mdenny has joined #openstack-cinder | 15:22 | |
*** p0rtal has joined #openstack-cinder | 15:23 | |
*** zul has quit IRC | 15:23 | |
*** p0rtal has quit IRC | 15:23 | |
*** p0rtal has joined #openstack-cinder | 15:24 | |
*** BharatK has joined #openstack-cinder | 15:24 | |
*** daneyon_ has quit IRC | 15:26 | |
smcginnis | Anyone else seeing their CI tests starting to take a whole lot longer to run? | 15:29 |
*** ebalduf has joined #openstack-cinder | 15:29 | |
*** kjnelson has joined #openstack-cinder | 15:29 | |
*** xyang has joined #openstack-cinder | 15:30 | |
geguileo | smcginnis: I was getting that same feeling | 15:30 |
*** bswartz has quit IRC | 15:31 | |
smcginnis | geguileo: At least it's not just me then. :) | 15:32 |
geguileo | :-) | 15:32 |
*** mylu has quit IRC | 15:34 | |
*** jgregor has quit IRC | 15:36 | |
*** jgregor1 has joined #openstack-cinder | 15:36 | |
*** bswartz has joined #openstack-cinder | 15:37 | |
thrawn01 | LOL, the python-cinderclient unit tests are failing because my ISP does DNS intercepts. | 15:38 |
* thrawn01 facepalms | 15:38 | |
*** tsekiyama has joined #openstack-cinder | 15:38 | |
eharney | thrawn01: if your ISP's DNS behavior breaks the tests, there's probably a bug in the tests | 15:39 |
patrickeast | jgriffith: hey, just saw your comments on the image cache review, i'm not sure what the issue is causing the image to not work.. but i think we might actually be able to take that check out | 15:40 |
*** haomaiwang has quit IRC | 15:40 | |
patrickeast | jgriffith: it also doesn't need virtual_size anymore, tsekiyama came up with a way to solve it | 15:40 |
patrickeast | jgriffith: the latest patch should work on any image | 15:41 |
*** nkrinner has quit IRC | 15:41 | |
jgriffith | patrickeast: Ahhh | 15:41 |
thrawn01 | eharney: yeah, apparently keystone is actually making requests http://no.where/v2.0 in the tests, the requests should instead get a mocked response, instead of actually trying to do a DNS lookup on no.where | 15:41 |
jgriffith | patrickeast: I wrote some code last night that breaks the image-utils process in two, then uses the temp file size | 15:41 |
eharney | thrawn01: well, that's not good | 15:41 |
jgriffith | patrickeast: is that what tsekiyama did? | 15:41 |
patrickeast | jgriffith: haha, pretty much the same thing we download the image early and use the temp file | 15:42 |
thrawn01 | eharney: I'll fix it | 15:42 |
jgriffith | patrickeast: also, any thoughts on the permission issues from glance? | 15:42 |
jgriffith | patrickeast: LOL | 15:42 |
jgriffith | ok | 15:42 |
jgriffith | well here I thought I was smart | 15:42 |
jgriffith | well... I still am, just tsekiyama is smart too :) | 15:42 |
patrickeast | jgriffith: exactly | 15:42 |
patrickeast | jgriffith: so for the permissions thing | 15:42 |
patrickeast | jgriffith: i don't know that we actually need to do that check anymore | 15:42 |
patrickeast | jgriffith: early on the internal tenant actually would do glance interactions and things making that a requirement | 15:43 |
jgriffith | patrickeast: yeah we could just use the original tenants context for the fetch | 15:43 |
patrickeast | jgriffith: but now it just copies the volume under the hood and it doesn't really matter who owns it | 15:43 |
jgriffith | patrickeast: but I am curious why that fails on a public image? | 15:43 |
*** jungleboyj has joined #openstack-cinder | 15:44 | |
patrickeast | jgriffith: honestly... i dunno, more than likely a bug introduced in the last couple of patchsets or something | 15:44 |
patrickeast | worked on my machine :p | 15:44 |
jgriffith | haha | 15:44 |
*** dustins has quit IRC | 15:44 | |
patrickeast | ill take that check out and play around with it today | 15:44 |
patrickeast | it *should* work better | 15:44 |
jgriffith | patrickeast: well if you've got those changes in your queue I'll wait to test it | 15:44 |
jgriffith | patrickeast: ok, thanks | 15:45 |
*** adurbin_ has joined #openstack-cinder | 15:45 | |
jgriffith | patrickeast: by the way, would love reviews on the replication patch so I can get that done | 15:45 |
*** zul has joined #openstack-cinder | 15:45 | |
patrickeast | jgriffith: ok cool, i'll take a look this morning | 15:46 |
jgriffith | patrickeast: tsekiyama hmm... that's different than what I was thinking of.... | 15:48 |
jgriffith | patrickeast: tsekiyama https://review.openstack.org/#/c/213346/1/cinder/image/image_utils.py | 15:48 |
*** lpetrut has quit IRC | 15:48 | |
jgriffith | patrickeast: tsekiyama I'm not sure I see how it solves the problem... I could just be missing it though | 15:49 |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Efficient volume copy for generic volume migration https://review.openstack.org/183701 | 15:49 |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Adds framework for get_capabilities() feature https://review.openstack.org/201243 | 15:49 |
*** dims has quit IRC | 15:49 | |
*** kjnelson is now known as diablo_rojo | 15:49 | |
mtanino | eharney: geguileo sorry for waiting ^^ | 15:49 |
jgriffith | patrickeast: tsekiyama because we already create the volume before the fetch_to_volume_format so it's too late | 15:49 |
* geguileo looking at patch | 15:49 | |
*** dims has joined #openstack-cinder | 15:49 | |
patrickeast | jgriffith: :D thats the trick, look at what is in the create_volume flow in the image cache patch now | 15:50 |
jgriffith | patrickeast: tsekiyama I had thought of refactoring that method into two parts; and get the image info before creating the volume | 15:50 |
patrickeast | jgriffith: we do the temporary image download first | 15:50 |
patrickeast | jgriffith: yea i think its similar to what you are describing, i would guess the split up is different | 15:50 |
jgriffith | patrickeast: hmm... ok I think I see | 15:51 |
patrickeast | jgriffith: so ~line 739 of https://review.openstack.org/#/c/195795/41/cinder/volume/flows/manager/create_volume.py,cm | 15:51 |
patrickeast | jgriffith: any fetch calls inside that with clause will have access to the temporary image | 15:52 |
jgriffith | patrickeast: the problem is this is then additive behavior as opposed to say just modifying the create-from-image all the way around | 15:52 |
jgriffith | patrickeast: I need to step through it, maybe i'm missing something | 15:52 |
*** dims_ has joined #openstack-cinder | 15:53 | |
*** mylu has joined #openstack-cinder | 15:53 | |
openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 15:53 |
patrickeast | jgriffith: gotcha, well i'm open to doing it in whatever way works best and causes the least amount of grief | 15:53 |
jgriffith | patrickeast: ahhh.... ok, he's got it there | 15:53 |
jgriffith | patrickeast: or *you* have it there | 15:54 |
*** dims has quit IRC | 15:54 | |
*** zul has quit IRC | 15:55 | |
*** scohen1 has joined #openstack-cinder | 15:56 | |
openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 15:57 |
*** takedakn has joined #openstack-cinder | 15:57 | |
mtanino | geguileo: thanks. I will add test case on that path. | 15:59 |
geguileo | mtanino: Awesome! | 16:00 |
*** _cjones_ has joined #openstack-cinder | 16:00 | |
*** cebruns_ is now known as cebruns | 16:01 | |
*** daneyon_ has joined #openstack-cinder | 16:05 | |
*** cbader has joined #openstack-cinder | 16:05 | |
*** Yogi1 has joined #openstack-cinder | 16:07 | |
*** Apoorva has joined #openstack-cinder | 16:07 | |
*** daneyon has quit IRC | 16:08 | |
*** scohen1 has quit IRC | 16:09 | |
*** dustins has joined #openstack-cinder | 16:09 | |
*** martyturner has quit IRC | 16:10 | |
*** kmartin has joined #openstack-cinder | 16:11 | |
*** p0rtal has quit IRC | 16:12 | |
*** gouthamr has quit IRC | 16:13 | |
*** dflorea_ has joined #openstack-cinder | 16:13 | |
*** scohen1 has joined #openstack-cinder | 16:13 | |
*** scohen1 has left #openstack-cinder | 16:13 | |
*** dflorea_ has quit IRC | 16:13 | |
*** mylu has quit IRC | 16:13 | |
*** dflorea has joined #openstack-cinder | 16:14 | |
*** mylu has joined #openstack-cinder | 16:14 | |
*** leeantho has joined #openstack-cinder | 16:15 | |
*** ronis has joined #openstack-cinder | 16:19 | |
*** yuriy_n17 has quit IRC | 16:20 | |
*** takedakn has quit IRC | 16:20 | |
*** lpetrut has joined #openstack-cinder | 16:21 | |
*** bluex-pl has quit IRC | 16:22 | |
*** agarciam has quit IRC | 16:27 | |
*** xyang has quit IRC | 16:28 | |
*** jistr has quit IRC | 16:30 | |
*** jordanP has quit IRC | 16:30 | |
*** HoloIRCUser3 has joined #openstack-cinder | 16:30 | |
*** vilobhmm1 has joined #openstack-cinder | 16:30 | |
thingee | jgriffith: thanks for doing the meeting | 16:32 |
jgriffith | thingee: NP | 16:32 |
jgriffith | kmartin: did you see my questions around https://bugs.launchpad.net/cinder/+bug/1486286? | 16:35 |
openstack | Launchpad bug 1486286 in Cinder "'qos_specs' does not get populated by Cinder during scheduler volume placement" [Undecided,Confirmed] - Assigned to Anthony Lee (anthony-mic-lee) | 16:35 |
*** jgregor1 has quit IRC | 16:36 | |
*** jgregor has joined #openstack-cinder | 16:36 | |
thingee | geguileo: you can now sign up for stuff ;) https://etherpad.openstack.org/p/cinder-liberty-3-reviews | 16:37 |
thingee | smcginnis: I asked yesterday if you at all tested the volume migration status update work. | 16:38 |
thingee | smcginnis: since you gave a +2 | 16:38 |
geguileo | thingee: Ok, I've reviewed 3 or 4 of those already | 16:38 |
geguileo | thingee: I'll start writing my name next to them | 16:38 |
thingee | geguileo: feel free to changing your vote +2 or +2/a if you feel comfortable | 16:38 |
geguileo | thingee: OK | 16:39 |
*** dims_ has quit IRC | 16:39 | |
openstackgerrit | Derrick Wippler proposed openstack/python-cinderclient: Fixed test_password_prompted https://review.openstack.org/214704 | 16:39 |
*** dims has joined #openstack-cinder | 16:39 | |
thrawn01 | eharney: https://review.openstack.org/#/c/214704/ | 16:40 |
*** zul has joined #openstack-cinder | 16:40 | |
geguileo | thingee: I just checked and my 4 reviews are -1 }XD | 16:41 |
eharney | thrawn01: cool, thanks | 16:42 |
thingee | geguileo: nice! | 16:42 |
thingee | jungleboyj: hey hoss | 16:43 |
thingee | jungleboyj: https://bugs.launchpad.net/cinder/+bug/1455028 | 16:44 |
openstack | Launchpad bug 1455028 in Cinder "cinder cannot start because of: oslo_config.cfg.DuplicateOptError: duplicate option: fatal_exception_format_errors" [Medium,Triaged] | 16:44 |
thingee | diablo_rojo: ^ | 16:45 |
*** prometheanfire has left #openstack-cinder | 16:46 | |
*** Yogi1 has quit IRC | 16:46 | |
*** zul has quit IRC | 16:47 | |
*** zul has joined #openstack-cinder | 16:49 | |
mgagne | thingee: is there a way to query through the API the backend capacity/usage? | 16:51 |
*** martyturner has joined #openstack-cinder | 16:51 | |
*** earlephilhower has quit IRC | 16:51 | |
mgagne | thingee: I know a lot of states are stored in memory but last time I checked, I didn't find a way to extract or query them | 16:51 |
smcginnis | thingee: Sorry, missed that. | 16:51 |
smcginnis | thingee: I had done some migrations with no errors. | 16:51 |
*** dustins has quit IRC | 16:52 | |
smcginnis | thingee: Not sure in depth testing, but looked OK and code looked fine. | 16:52 |
smcginnis | thingee: My test rig is a little messed up right now, otherwise I would run it through again. | 16:52 |
thingee | smcginnis: great! | 16:52 |
*** garthb has quit IRC | 16:52 | |
thingee | mgagne: https://github.com/openstack/cinder/blob/master/cinder/api/contrib/scheduler_stats.py | 16:53 |
thingee | mgagne: trying to figure out the endpoint for this :P | 16:53 |
mgagne | thingee: first time I hear about that one. will take a look | 16:53 |
diablo_rojo | thingee: Jay and I were looking at that yesterday. We ran into a similar error and got a patch pushed up for that, and are still looking into this one | 16:53 |
mriedem | jungleboyj: neither of the bugs tied to this have a severity https://review.openstack.org/#/c/210219/ but could you gauge whether or not you think that would be good to have on stable? | 16:53 |
thingee | diablo_rojo: thank you thank you | 16:53 |
thingee | mgagne: yay, I'm useful | 16:54 |
mgagne | :D | 16:54 |
diablo_rojo | thingee: no problem :) | 16:54 |
*** gouthamr has joined #openstack-cinder | 16:54 | |
*** lpetrut has quit IRC | 16:54 | |
mgagne | thingee: we were talking about CMDB and potentially usage/capacity tracking. I think that's one thing that openstack could improve a lot (IMO). Be it through API calls or well documented database schema/views. | 16:55 |
*** dustins has joined #openstack-cinder | 16:56 | |
mgagne | thingee: will take a look at what you sent next week and come back if I have any comments or suggestions for improvements. | 16:56 |
*** dflorea has quit IRC | 16:57 | |
thingee | mgagne: is this in the thought that the operator does not have access to the DB directly? | 16:57 |
thingee | smcginnis: thank you | 16:58 |
mgagne | thingee: we (I) do have access, it's just not that easy to reverse engineer sometimes. Might not be the case for cinder (thank god) but still. our BI team isn't that familiar with openstack and its concepts | 16:59 |
*** kvidvans has quit IRC | 16:59 | |
*** dflorea has joined #openstack-cinder | 16:59 | |
shausy | Im trying volume retype with 3par as backend, im getting message as successfully sceheduled for retype but retype is not going thru | 17:00 |
shausy | which log I should refer ? | 17:00 |
shausy | how to see its successful or not ? | 17:00 |
thingee | mgagne: got it. let me know if Cinder is missing anything else | 17:00 |
mgagne | thingee: sure. ty | 17:00 |
*** lpetrut has joined #openstack-cinder | 17:02 | |
*** rhagarty has quit IRC | 17:03 | |
*** sseverson has quit IRC | 17:03 | |
*** rhagarty_ has quit IRC | 17:04 | |
*** leeantho has quit IRC | 17:04 | |
*** cbader has quit IRC | 17:04 | |
*** cbader has joined #openstack-cinder | 17:04 | |
*** xyang has joined #openstack-cinder | 17:04 | |
*** sseverson has joined #openstack-cinder | 17:04 | |
*** rhagarty_ has joined #openstack-cinder | 17:04 | |
*** leeantho has joined #openstack-cinder | 17:04 | |
*** rhagarty has joined #openstack-cinder | 17:04 | |
*** resker has joined #openstack-cinder | 17:04 | |
thingee | patrickeast: huge thanks on testing https://review.openstack.org/#/c/144590/ | 17:07 |
patrickeast | np | 17:07 |
*** kmartin has quit IRC | 17:08 | |
*** kmartin has joined #openstack-cinder | 17:08 | |
*** esker has quit IRC | 17:08 | |
*** sgotliv has quit IRC | 17:09 | |
*** cbader has quit IRC | 17:10 | |
*** rhagarty has quit IRC | 17:10 | |
*** cbader has joined #openstack-cinder | 17:10 | |
*** rhagarty has joined #openstack-cinder | 17:10 | |
kmartin | jgriffith, yeah, reading you comments on the bug now. The evaluator schedule and weigher can take advantage of the qos_specs if they were available. Anthony is working on a response to the bug now | 17:11 |
*** asselin_ has joined #openstack-cinder | 17:11 | |
jgriffith | kmartin: well, what I'm saying is that's how it was intended to work | 17:12 |
*** skylerberg has joined #openstack-cinder | 17:12 | |
jgriffith | kmartin: we never intended to filter on the keys inside of the qos-spec itself that I'm aware of | 17:12 |
jgriffith | kmartin: for that you can just use defined keys in your extra-specs | 17:13 |
*** dflorea_ has joined #openstack-cinder | 17:13 | |
*** dflorea has quit IRC | 17:13 | |
jgriffith | see what I mean? The other problem is those aren't "capabilities" like "true/false, enabled/disabled" | 17:13 |
*** markstur has quit IRC | 17:13 | |
kmartin | jgriffith, the point of adding the qos_specs was to put the qos related key values in it. | 17:14 |
jgriffith | kmartin: so you'd have to report every possible minIOP setting for example for it to filter on it | 17:14 |
*** markus_z has quit IRC | 17:14 | |
jgriffith | kmartin: right and assign it to a type | 17:14 |
jgriffith | kmartin: and easily do things like *change* the qos or the qos assigned to a type | 17:14 |
jgriffith | kmartin: it was just a little cleaner than having a huge extra-specs entry | 17:15 |
kmartin | jgriffith, yes and we are doing that, but we would like to take advantage of the keys and values to create filter and goodness functions for the eval filter | 17:16 |
*** jgregor has quit IRC | 17:17 | |
*** ndipanov has joined #openstack-cinder | 17:17 | |
jgriffith | kmartin: ok, but I guess I was saying we never intended those keys to be filtered. But if you have a proposal that's cool. | 17:17 |
*** ndipanov_ has joined #openstack-cinder | 17:17 | |
openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 17:17 |
*** ndipanov_ has quit IRC | 17:18 | |
jgriffith | kmartin: should make sure winston-d has a look at it | 17:18 |
*** p0rtal has joined #openstack-cinder | 17:18 | |
kmartin | jgriffith, agree... Anthony will put some examples in the defect and add Winston to the review | 17:19 |
*** markstur has joined #openstack-cinder | 17:19 | |
*** shausy has quit IRC | 17:19 | |
*** shausy has joined #openstack-cinder | 17:20 | |
kmartin | leeantho, ^^ | 17:20 |
jgriffith | leeantho: kmartin cool... IMO it's not a bug and it shouldn't do what you're proposing but I could be completely wrong so it would be good to see more detail and have others take a look at it | 17:21 |
openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Cinder Nested Quota Driver https://review.openstack.org/205369 | 17:21 |
*** resker has quit IRC | 17:22 | |
*** zul has quit IRC | 17:23 | |
*** diablo_rojo has quit IRC | 17:32 | |
openstackgerrit | Sonia Ghanekar proposed openstack/cinder: Add manage/unmanage volume support for Nimble https://review.openstack.org/214392 | 17:34 |
*** mylu has quit IRC | 17:34 | |
*** krtaylor has quit IRC | 17:37 | |
*** kjnelson has joined #openstack-cinder | 17:43 | |
*** p0rtal_ has joined #openstack-cinder | 17:44 | |
*** xyang has quit IRC | 17:45 | |
*** edtubill has joined #openstack-cinder | 17:46 | |
*** p0rtal has quit IRC | 17:47 | |
openstackgerrit | John Griffith proposed openstack/cinder: Filter out extra-specs from type get for non-admin https://review.openstack.org/214731 | 17:47 |
patrickeast | jgriffith: hey, looking at the replication review, were you planning to add in the 'replication_devices' config option to the default driver configs or is it going to be up to each backend to have their own flavor of it? | 17:47 |
jgriffith | patrickeast: ahh.. yeah, I need to add that. Thanks! | 17:48 |
jgriffith | patrickeast: I was going to leave it up to drivers, but it would be good to standardize it | 17:48 |
patrickeast | jgriffith: ok cool, yea i think at least that one could/should be standaridized | 17:48 |
*** jordanP has joined #openstack-cinder | 17:49 | |
patrickeast | jgriffith: and somewhat related, would we enforce a standard format for the key/value descriptions on that and shared parsing or whatever | 17:49 |
patrickeast | jgriffith: or should dannywilson start reading the olso config docs ;) | 17:49 |
jgriffith | patrickeast: LOL... let me get a good example of that written up | 17:50 |
*** dflorea_ has quit IRC | 17:50 | |
jgriffith | patrickeast: I'll dot that next | 17:50 |
* dannywilson sighs in relief ;) | 17:50 | |
jgriffith | patrickeast: I'm curious on your opinion on replica-count for first pass? | 17:50 |
jgriffith | jungleboyj: ^^ you as well | 17:50 |
jgriffith | I had it in there to start, but thought to simplify first pass just limit it to 1 | 17:50 |
patrickeast | jgriffith: i still think it should just be something specified on a volume type | 17:51 |
dannywilson | jgriffith: +1 to leave it out for first pass and limit to 1 | 17:51 |
patrickeast | jgriffith: if it isn't specified its up to the driver to do their own thing | 17:51 |
*** breitz has quit IRC | 17:51 | |
jgriffith | patrickeast: well.. yeah, but you need to have a different structure for the config option to specify multiple targets | 17:52 |
jgriffith | patrickeast: see what I mean? | 17:52 |
patrickeast | jgriffith: ohh yea | 17:52 |
*** breitz has joined #openstack-cinder | 17:52 | |
*** edtubill has left #openstack-cinder | 17:52 | |
jgriffith | patrickeast: I'm in agreement on leaving the specification to the type i.e.: replica_count=100000 | 17:52 |
jgriffith | :) | 17:52 |
jordanP | thingee, hi. I've fixed my CI issue in https://review.openstack.org/#/c/200610/ (Rework Scality SOFS driver to use RemoteFS class), if you have a chance to have a look again, that would be great. sorry to bother you again, but as L-3 is approaching I start to feel the pressure :) | 17:52 |
patrickeast | jgriffith: i'm ok with multiple targets, the example you had with like 'backend':[key:value,etc]' would work | 17:52 |
jgriffith | patrickeast: but then I need 1000 target entries in the config file :) | 17:52 |
jgriffith | kk | 17:52 |
patrickeast | jgriffith: haha yea | 17:52 |
*** anshul has quit IRC | 17:53 | |
patrickeast | then again as a first pass maybe its not a big deal | 17:53 |
dannywilson | jgriffith: to iron out the issues it might be better to start with single target? hmm | 17:54 |
*** ronis_ has joined #openstack-cinder | 17:54 | |
jgriffith | dannywilson: yeah... although I was just thinking that might be bad, because then the syntax for the config entry changes | 17:54 |
*** afazkas has joined #openstack-cinder | 17:54 | |
jgriffith | dannywilson: I'lll look at it this afternoon and ping ya | 17:55 |
jgriffith | meanwhile.. meeting :( | 17:55 |
*** aix has quit IRC | 17:55 | |
dannywilson | jgriffith: thanks | 17:55 |
*** ronis has quit IRC | 17:55 | |
*** dflorea has joined #openstack-cinder | 17:56 | |
*** kvidvans has joined #openstack-cinder | 17:56 | |
*** angela-s has joined #openstack-cinder | 17:58 | |
*** dflorea has quit IRC | 18:00 | |
*** dflorea has joined #openstack-cinder | 18:01 | |
*** dflorea has quit IRC | 18:03 | |
*** annegentle has quit IRC | 18:04 | |
jungleboyj | jgriffith: Going to look at that this afternoon. | 18:04 |
jungleboyj | So you are ready for comments on there? | 18:04 |
leeantho | jgriffith, I tried to clarify this bug a little more here now: https://bugs.launchpad.net/cinder/+bug/1486286 . when you have time take a look again at the new comment | 18:06 |
openstack | Launchpad bug 1486286 in Cinder "'qos_specs' does not get populated by Cinder during scheduler volume placement" [Undecided,Confirmed] - Assigned to Anthony Lee (anthony-mic-lee) | 18:06 |
*** rushil has joined #openstack-cinder | 18:07 | |
*** BharatK has quit IRC | 18:08 | |
*** jordanP has quit IRC | 18:08 | |
*** krtaylor has joined #openstack-cinder | 18:13 | |
*** shausy has quit IRC | 18:16 | |
*** danieru has quit IRC | 18:26 | |
*** ebalduf has quit IRC | 18:29 | |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port test_nfs to Python 3 https://review.openstack.org/214384 | 18:30 |
*** anshul has joined #openstack-cinder | 18:30 | |
openstackgerrit | xing-yang proposed openstack/python-cinderclient: CLI: Non-disruptive backup https://review.openstack.org/204232 | 18:31 |
*** bapalm has quit IRC | 18:31 | |
*** kmartin has quit IRC | 18:32 | |
*** bapalm has joined #openstack-cinder | 18:32 | |
*** jwang has quit IRC | 18:34 | |
openstackgerrit | xing-yang proposed openstack/python-cinderclient: CLI: Non-disruptive backup https://review.openstack.org/204232 | 18:34 |
*** jms has joined #openstack-cinder | 18:35 | |
xyang1 | eharney: Hi, I updated the help text. Can you check to see if it is clear now? Thanks | 18:35 |
*** martyturner has quit IRC | 18:36 | |
jms | So ... Crazy question. Is it possible to have a single Cinder deployment work with multiple Openstack deployments (Each with their own Keystone instance)? | 18:36 |
*** eharney has quit IRC | 18:37 | |
*** martyturner has joined #openstack-cinder | 18:38 | |
jgriffith | leeantho: thanks! | 18:39 |
jgriffith | leeantho: I responded as well :) | 18:39 |
*** eharney has joined #openstack-cinder | 18:39 | |
*** jwang has joined #openstack-cinder | 18:40 | |
jgriffith | leeantho: I will butt out for now; but I think there's a misunderstanding either between me completely understanding what you're getting at OR your interpretation of how the QoS specs should work in conjunction with he filter scheduler. | 18:40 |
jgriffith | leeantho: At the end of the day I guess if it doesn't break compatibility I don't care. And I'm fairly certain nobody else is going to really have much of an opinion; so I'm kind of at the mercy of you and your team at this point :) | 18:41 |
jungleboyj | mriedem: Seems to me that that is a relatively high bug. I nominated it for Kilo. You want me to cherry-pick it? | 18:45 |
*** esker has joined #openstack-cinder | 18:46 | |
jungleboyj | kjnelson: Thanks for tackling thingee's question! | 18:46 |
*** xyang has joined #openstack-cinder | 18:48 | |
guitarzan | jms: that indeed is a crazy question! :D | 18:49 |
guitarzan | jms: I don't think you would be able to auth against both | 18:49 |
guitarzan | jms: maybe this is a job for the federated identity people? | 18:49 |
*** zul has joined #openstack-cinder | 18:50 | |
mriedem | jungleboyj: doesn't matter to me who proposes it | 18:54 |
mriedem | it's not a clean cherry pick, i already tried via the gerrit ui | 18:54 |
jungleboyj | mriedem: I can tackle it then. I have another cherry-pick to do this afternoon as well. | 18:55 |
mriedem | jungleboyj: plus all those horizon panels to write | 18:55 |
mriedem | for multi-attach volume migration | 18:55 |
jungleboyj | mriedem: :-p | 18:56 |
mriedem | did i just blow some minds? | 18:56 |
* jungleboyj looks at kjnelson | 18:56 | |
*** kmartin has joined #openstack-cinder | 18:56 | |
*** zhonghua-lee has joined #openstack-cinder | 19:00 | |
*** zhongjun has joined #openstack-cinder | 19:00 | |
*** rushil has quit IRC | 19:01 | |
kjnelson | jungleboyj: are you looking at me? | 19:02 |
* thangp thinks of that famous line in Casblanca - "Here's looking at you kid." | 19:03 | |
*** dflorea has joined #openstack-cinder | 19:03 | |
jungleboyj | thangp: +! | 19:03 |
jungleboyj | kjnelson: What thangp said | 19:03 |
*** rushil has joined #openstack-cinder | 19:03 | |
*** lpetrut has quit IRC | 19:03 | |
kjnelson | jungleboyj: oh Lord. | 19:04 |
*** lpetrut has joined #openstack-cinder | 19:04 | |
smcginnis | kjnelson: He has you calling him that now, huh? | 19:04 |
jungleboyj | smcginnis: ;-) | 19:05 |
* jungleboyj is watching over my shoulder. She is going to hurt me. | 19:05 | |
kjnelson | smcginnis: Life just got very confusing all of a sudden. | 19:05 |
kjnelson | jungleboyj: Yep. Here I come.. | 19:05 |
*** dflorea has quit IRC | 19:06 | |
*** jgregor has joined #openstack-cinder | 19:06 | |
* jungleboyj Just got punched. | 19:06 | |
*** p0rtal has joined #openstack-cinder | 19:06 | |
*** dflorea has joined #openstack-cinder | 19:06 | |
jungleboyj | smcginnis: Thanks for that. | 19:06 |
smcginnis | jungleboyj: I'm so good I can get you from miles away. :) | 19:07 |
jungleboyj | smcginnis: No comment. | 19:07 |
kjnelson | smcginnis: Watch out. I'll get you in October. | 19:07 |
smcginnis | hehe | 19:07 |
* jungleboyj Thinks you two are colluding against me! | 19:07 | |
smcginnis | Mwahahaha | 19:08 |
*** ociuhandu has quit IRC | 19:08 | |
kjnelson | So my threats mean nothing? | 19:08 |
*** zhongjun has quit IRC | 19:08 | |
jungleboyj | smcginnis: Watch yourself. Don't poke the diablo. | 19:08 |
smcginnis | kjnelson: No, I just have until October to come up with a plan. ;) | 19:09 |
kjnelson | smcginnis: I will bring the fire. | 19:09 |
*** p0rtal_ has quit IRC | 19:09 | |
smcginnis | kjnelson: Then I'll bring the cider. | 19:09 |
smcginnis | :P | 19:09 |
jungleboyj | :-) It is like we never left Colorado. | 19:10 |
kjnelson | smcginnis: Thats fireball ;) I am talking about the ginger rage I refer to as "the fire" | 19:10 |
jungleboyj | kjnelson: What have we done to smcginnis ? | 19:10 |
jungleboyj | OMG. | 19:10 |
smcginnis | jungleboyj: Better question is - what is she going to do to us. | 19:10 |
*** Yogi1 has joined #openstack-cinder | 19:11 | |
kjnelson | jungleboyj: We brought him down to our level ;) | 19:11 |
jungleboyj | smcginnis: Indeed... | 19:11 |
jungleboyj | :-) | 19:11 |
kjnelson | Wouldn't you like to know ;) | 19:11 |
*** danieru has joined #openstack-cinder | 19:12 | |
jungleboyj | smcginnis: Just say no. It is safer. ;) | 19:12 |
*** annegentle has joined #openstack-cinder | 19:13 | |
* smcginnis hides | 19:13 | |
*** lpetrut has quit IRC | 19:13 | |
*** afazkas has quit IRC | 19:13 | |
*** afazekas has joined #openstack-cinder | 19:14 | |
*** ebalduf has joined #openstack-cinder | 19:14 | |
jungleboyj | Too bad we don't have fun? | 19:14 |
*** Liu has quit IRC | 19:15 | |
*** esker has quit IRC | 19:16 | |
kjnelson | I love that ^ will be around forever | 19:16 |
*** zhonghua-lee has quit IRC | 19:16 | |
jungleboyj | ? | 19:16 |
kjnelson | The conversation will be imortalized...right? | 19:17 |
jungleboyj | kjnelson: Oh, yes ... | 19:17 |
jungleboyj | That will be fun to explain to my kids some day. | 19:17 |
*** anshul has quit IRC | 19:18 | |
erlon | jgriffith: ping | 19:20 |
erlon | hemna: ping | 19:20 |
*** lennyb has quit IRC | 19:20 | |
jgriffith | erlon: hey | 19:20 |
erlon | jgriffith: hey, question, in you storage is it possible to add som LUNs and make then to appear only to specific initiators? | 19:21 |
erlon | *some | 19:21 |
jgriffith | erlon: yes, access groups | 19:22 |
*** eharney has quit IRC | 19:22 | |
jgriffith | erlon: so we allow the use of either CHAP or AccessGroups | 19:22 |
jgriffith | actually I think even both for the really paranoid :) | 19:22 |
jgriffith | erlon: you still having CI issues? | 19:23 |
*** Lee1092 has quit IRC | 19:23 | |
*** Yogi1 has left #openstack-cinder | 19:23 | |
erlon | jgriffith: yes, Im investigating the staled devices on multinode setups | 19:23 |
erlon | jgriffith: when the initiator autenticates, it scans all LUNs, and all of then appear in the node | 19:24 |
jgriffith | erlon: are they coming from the cinder services or Nova? Or do you know | 19:24 |
erlon | jgriffith: both | 19:24 |
jgriffith | erlon: common thread there would be brick then I guess | 19:24 |
erlon | jgriffith: so, if you have 4 computes, the LUNs of the target will apperar in all | 19:24 |
jgriffith | erlon: Whaaaa | 19:25 |
jgriffith | that shouldn't be | 19:25 |
jgriffith | erlon: do you use AG's or Chap? | 19:25 |
erlon | jgriffith: yep, and what happens when you remove the LUN from the system? | 19:25 |
erlon | jgriffith: we don't have AG, only CHAP/not CHAP | 19:26 |
jgriffith | Well that sounds craptacular | 19:26 |
*** eharney has joined #openstack-cinder | 19:26 | |
jgriffith | erlon: so Chap if set on a volume per volume basis should solve that for you | 19:26 |
erlon | jgriffith: I actually only tested without CHAP but, the behaviour with CHAP should be the same | 19:26 |
jgriffith | erlon: but if it's set global that's tricky | 19:26 |
jgriffith | erlon: well... the scan won't show devices if they don't authenticate | 19:26 |
jgriffith | i.e. if you don't have the creds file | 19:27 |
erlon | jgriffith: i think so, but my storage does not provide AG | 19:27 |
erlon | jgriffith: I though about removing the additional devices right after I scan, so, what lefts after a connect is only the newly added volume | 19:28 |
jms | guitarzan: Thanks. That's about the answer I was expecting. | 19:28 |
erlon | jgriffith: which is what it should happen using CHAP/AG's or no security | 19:29 |
openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Better exception handling in init_conn https://review.openstack.org/214316 | 19:30 |
*** Yogi1 has joined #openstack-cinder | 19:31 | |
*** Yogi1 has left #openstack-cinder | 19:31 | |
jgriffith | erlon: so what I'm saying is I tested that and don't have that problem | 19:31 |
jgriffith | erlon: BUT I'm using Chap | 19:31 |
jgriffith | erlon: I can futz with some things later and convert it to no chap and see what happens | 19:32 |
jgriffith | erlon: maybe I can reproduce your issue | 19:32 |
erlon | jgriffith: I'll try whith CHAP too | 19:32 |
jgriffith | erlon: although..... | 19:32 |
jgriffith | erlon: hmmm; I'll take another look later. One thing about that is when I did it I only had one volume for that account attached to one node | 19:33 |
jgriffith | the other one was a different account and different credentials | 19:33 |
jgriffith | but still... it' build the security file off of the in (or at least it used to) | 19:33 |
jgriffith | so that shouldn't matter | 19:33 |
jms | guitarzan: In a similar thought ... If we have a large OS instance, and are provisioning out smaller OS instances ... Would being able to attach a peice of storage (iscsi/iser, etc..) to a provisioned OS node be a workaround... or would that blow up somewhere else along the lines since it's still going through different OS instances? | 19:33 |
erlon | jgriffith: but if I understood the problem correctly, that won't make difference. As long as a host 2 can see all LUNs of a target, including the one it is not using, you will have problems to remove | 19:34 |
jgriffith | hemna: has rewritten almost all of that code at this point I think so he's going to know it MUCH better than me problably | 19:34 |
jgriffith | erlon: Oh... are you not specifying the lun in your provider fields? | 19:34 |
jms | guitarzan: *provisioned OS cinder node | 19:35 |
guitarzan | jms: I'm not sure I can actually picture what all that means | 19:35 |
erlon | jgriffith: yes, but brick only seems to use that to build the /dev/disk/... path | 19:35 |
jgriffith | jms: I do that all the time | 19:35 |
erlon | jgriffith: and then use the path to remove the device | 19:36 |
jgriffith | jms: cinder create, attach to instance running cinder and use as backing store for LVM | 19:36 |
jms | jgriffith: Can the storage be resized in this case? | 19:37 |
jgriffith | jms: hmm.... you would probably want to add disks and do vgextend for that | 19:37 |
jgriffith | jms: as opposed to extending the disk you have | 19:37 |
jgriffith | jms: BTW, I'm in no way recommending this for production use without testing :) | 19:38 |
jgriffith | jms: I run test systems like this all the time, but those are test systems... so... ya know take it for what it's worth | 19:38 |
jgriffith | jms: why don't you just deploy another cinder node? | 19:38 |
jgriffith | jms: and what backend are you using by the way? | 19:39 |
jgriffith | jms: depending on what you're using you might be able to do trickier things | 19:39 |
jms | jgriffith: Because ... these are *customer* OS clouds | 19:39 |
jgriffith | jms: ? | 19:39 |
jms | jgriffith: Or the idea is for them to be out of our control :/ | 19:39 |
openstackgerrit | Sonia Ghanekar proposed openstack/cinder: Add manage/unmanage volume support for Nimble https://review.openstack.org/214392 | 19:39 |
jgriffith | jms: oh.. in that case heck yeah; use a cinder volume for them to build their own cinder node :) | 19:40 |
*** kmartin has quit IRC | 19:40 | |
jgriffith | That's kinda what they want anyway | 19:40 |
jgriffith | jms: but it sucks to be you if/when things go bad :( | 19:40 |
jms | jgriffith: The idea is to use ZFS as the backend | 19:40 |
jgriffith | jms: alright... well then I'm out | 19:41 |
jms | jgriffith: lol ... you don't say ;) | 19:41 |
jgriffith | jms: you're getting all sorts of exotic on me | 19:41 |
jgriffith | jms: you could just skip cinder and give them ZFS shares | 19:41 |
jgriffith | :) | 19:41 |
jms | jgriffith: No .. because the storage must be controlled by cinder ... because ... that's what those in a higher pay grade then me said | 19:42 |
jgriffith | jms: lol... I hear ya; even if it's not the best answer, it's the answer you have | 19:43 |
jms | Thanks | 19:43 |
*** ndipanov has quit IRC | 19:43 | |
jgriffith | jms: so you've got an interesting issue there... i think my idea around building the tenant a cinder-service backed by your parent clouds cinder-volume would work out descent | 19:44 |
jgriffith | jms: given they don't want you to have control of anything that seems to make the most sense | 19:44 |
jgriffith | jms: and there may be a clever way to use ZFS driver as well here | 19:45 |
jgriffith | jms: but that I don't know | 19:45 |
*** e0ne has joined #openstack-cinder | 19:45 | |
jgriffith | jungleboyj: triaging merged bugs :) | 19:45 |
*** afazekas has quit IRC | 19:46 | |
*** raildo is now known as raildo-afk | 19:47 | |
jgriffith | jungleboyj: there's about a billion more of them waiting for you out there :) | 19:47 |
jms | jgriffith: Yeah ... that was one of the workarounds we had came up with if we couldn't to a single cinder service with auth to multiple keystone services ... | 19:47 |
*** e0ne has quit IRC | 19:47 | |
jgriffith | jms: although...... | 19:48 |
jgriffith | jms: you should hop over to keystone channel... I think that might actually be doable | 19:48 |
jgriffith | jms: because keystone just registers the cinder endpoint | 19:48 |
jms | So ... setup multiple endpoints then... | 19:48 |
jgriffith | jms: I'm not sure you'd even need to do that | 19:48 |
jgriffith | jms: you just go to the keystone service you're setting up and point it to cinder endpoint | 19:49 |
jgriffith | jms: because it's not that you want cinder to talk to multiple keystones, you want multiple keystones to one cinder service | 19:49 |
jms | ... Ohhhhh.... | 19:49 |
*** esker has joined #openstack-cinder | 19:50 | |
jgriffith | jms: those keystone services don't know squat about each other in this case | 19:50 |
jgriffith | jms: they just see it as "their" endpoint and token | 19:50 |
*** belmoreira has joined #openstack-cinder | 19:50 | |
jgriffith | jms: in theory seems doable | 19:51 |
*** ociuhandu has joined #openstack-cinder | 19:51 | |
*** ronis_ has quit IRC | 19:51 | |
jgriffith | jms: you should go try it and let me know :) | 19:51 |
jms | jgriffith: Yes that does seem cleaner | 19:51 |
jgriffith | jms: otherwise I'm going to try it next week just out of curiousity | 19:51 |
jms | jgriffith: It's going to take a bit for us to do that here ... just trying to get things planned out | 19:52 |
jms | jgriffith: Thanks much on the info though. I didn't think about changing the endpoint on the provisioned side to point back. Should work as long as they don't change network requirements. | 19:53 |
*** jgravel has joined #openstack-cinder | 19:54 | |
*** jgravel has left #openstack-cinder | 19:54 | |
*** esker has quit IRC | 19:54 | |
*** esker has joined #openstack-cinder | 19:55 | |
geguileo | jungleboyj: Thanks for the review :-) | 19:56 |
*** alejandrito has joined #openstack-cinder | 19:59 | |
*** annegentle has quit IRC | 19:59 | |
*** alejandrito_ has joined #openstack-cinder | 20:00 | |
*** alejandrito has quit IRC | 20:00 | |
*** e0ne has joined #openstack-cinder | 20:02 | |
*** ebalduf_ has joined #openstack-cinder | 20:07 | |
openstackgerrit | Joel Coffman proposed openstack/cinder: Clean up volume_type_update method https://review.openstack.org/213829 | 20:11 |
*** p0rtal_ has joined #openstack-cinder | 20:13 | |
openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 20:14 |
openstackgerrit | Danny Wilson proposed openstack/cinder-specs: Update for ability of backend to report discard/trim/unmap https://review.openstack.org/211583 | 20:14 |
patrickeast | any python3 gurus around? im running into a failure from using MagicMock(spec=file) like so http://logs.openstack.org/95/195795/41/check/gate-cinder-python34/6db3ee6/console.html#_2015-08-19_04_21_14_547 | 20:15 |
*** p0rtal has quit IRC | 20:15 | |
patrickeast | i'm trying to figure out how to do that in a compatible way.. the only reference i see is in mock.py https://github.com/testing-cabal/mock/blob/master/mock/mock.py#L2483-L2495 | 20:15 |
patrickeast | which is seriously gross | 20:15 |
*** rushil has quit IRC | 20:16 | |
*** e0ne has quit IRC | 20:18 | |
*** alejandrito_ has quit IRC | 20:18 | |
*** ebalduf has quit IRC | 20:19 | |
*** lpetrut has joined #openstack-cinder | 20:20 | |
*** _cjones_ has quit IRC | 20:21 | |
*** r-daneel has joined #openstack-cinder | 20:24 | |
*** jms has left #openstack-cinder | 20:24 | |
*** julim has quit IRC | 20:25 | |
openstackgerrit | Kedar Vidvans proposed openstack/cinder: Assisted volume migration for Oracle ZFSSA drivers https://review.openstack.org/210608 | 20:26 |
*** xyang has left #openstack-cinder | 20:28 | |
*** e0ne has joined #openstack-cinder | 20:29 | |
*** akerr has quit IRC | 20:35 | |
*** thangp has quit IRC | 20:35 | |
*** e0ne has quit IRC | 20:35 | |
*** e0ne has joined #openstack-cinder | 20:36 | |
*** kmartin has joined #openstack-cinder | 20:42 | |
*** ebalduf has joined #openstack-cinder | 20:43 | |
*** belmoreira has quit IRC | 20:44 | |
*** annegentle has joined #openstack-cinder | 20:45 | |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Efficient volume copy for generic volume migration https://review.openstack.org/183701 | 20:46 |
*** IanGovett has quit IRC | 20:46 | |
*** lpetrut has quit IRC | 20:46 | |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Allow specified backend capabilities to be retrieved https://review.openstack.org/212861 | 20:47 |
jungleboyj | geguileo: You are welcome! | 20:48 |
*** dustins has quit IRC | 20:50 | |
*** mylu has joined #openstack-cinder | 20:51 | |
*** annegentle has quit IRC | 20:53 | |
*** jungleboyj has quit IRC | 20:53 | |
*** annegentle has joined #openstack-cinder | 20:54 | |
*** edtubill has joined #openstack-cinder | 20:54 | |
erlon | patrickeast: hey | 20:55 |
patrickeast | erlon: hi | 20:55 |
erlon | patrickeast: have finished the implementation of image caching for Pure? | 20:55 |
patrickeast | erlon: yea https://review.openstack.org/#/c/195795/ | 20:56 |
patrickeast | erlon: that one supersedes it | 20:57 |
erlon | patrickeast: Goodness, 50% of the work was the commit message lol | 20:57 |
patrickeast | lol | 20:57 |
*** bswartz has quit IRC | 20:57 | |
*** esker has quit IRC | 20:58 | |
erlon | patrickeast: hmm, so, the implementation is generic only? no need to addons in drivers? | 20:59 |
patrickeast | erlon: yep, everyone gets it fo free | 21:00 |
*** zul has quit IRC | 21:00 | |
erlon | patrickeast: hmm, thats great! | 21:00 |
openstackgerrit | Diem Tran proposed openstack/cinder: Local cache feature of Oracle ZFSSA drivers https://review.openstack.org/212155 | 21:00 |
erlon | smcginnis: around? | 21:02 |
*** rushil has joined #openstack-cinder | 21:09 | |
openstackgerrit | John Griffith proposed openstack/cinder: Parse out SolidFire account from api response https://review.openstack.org/214797 | 21:11 |
*** _cjones_ has joined #openstack-cinder | 21:12 | |
*** rushil_ has joined #openstack-cinder | 21:13 | |
*** p0rtal has joined #openstack-cinder | 21:13 | |
*** e0ne has quit IRC | 21:15 | |
*** rushil_ has quit IRC | 21:16 | |
*** rushil has quit IRC | 21:16 | |
*** gouthamr has quit IRC | 21:16 | |
*** rushil has joined #openstack-cinder | 21:16 | |
*** p0rtal_ has quit IRC | 21:17 | |
*** krtaylor has quit IRC | 21:18 | |
*** e0ne has joined #openstack-cinder | 21:19 | |
openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 21:19 |
*** porrua has quit IRC | 21:22 | |
openstackgerrit | Derrick Wippler proposed openstack/python-cinderclient: No longer ignores CINDER_SERVICE_NAME https://review.openstack.org/214802 | 21:26 |
*** ndipanov has joined #openstack-cinder | 21:26 | |
smcginnis | erlon: Back now. Gotta run soon though. What's up? | 21:26 |
erlon | smcginnis: you were waiting for the CI run the tests on this patch: https://review.openstack.org/#/c/214317/ | 21:27 |
erlon | smcginnis: have just finished | 21:27 |
smcginnis | erlon: Cool, thanks for pointing it out. | 21:28 |
smcginnis | erlon: I'll take another look. | 21:28 |
erlon | smcginnis: np, thanks! | 21:28 |
*** e0ne has quit IRC | 21:28 | |
smcginnis | erlon: Done | 21:28 |
erlon | smcginnis: awesome! | 21:29 |
thrawn01 | eharney: are you around? | 21:29 |
thrawn01 | eharney: in reference to https://review.openstack.org/#/c/214704/1/cinderclient/tests/unit/test_shell.py | 21:30 |
thrawn01 | eharney: don't we want our tests to fail if keystone changes? tests should let us know what an upstream library breaks our stuff. | 21:30 |
thrawn01 | when* | 21:31 |
*** martyturner has quit IRC | 21:31 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Add rootwrap filters https://review.openstack.org/207553 | 21:33 |
*** p0rtal_ has joined #openstack-cinder | 21:36 | |
*** p0rtal has quit IRC | 21:39 | |
eharney | thrawn01: not really, other libraries don't guarantee how their internals work, just the interfaces we use | 21:39 |
eharney | thrawn01: hooking the requests library inside of keystoneclient involves details that aren't really cinder's business to worry about | 21:40 |
openstackgerrit | Goutham Pacha Ravi proposed openstack/cinder: Add parameter to filter pools for NetApp driver https://review.openstack.org/210588 | 21:40 |
thrawn01 | eharney: so, your point is, maybe one day keystone doesn't use requests internally | 21:40 |
*** jgregor has quit IRC | 21:41 | |
eharney | thrawn01: or adds a different request call before that one, and we hook the wrong one, then the test is doing something unintended, etc | 21:41 |
thrawn01 | eharney: aight, I'll change it, I'm gonna have to mock out multiple keystone calls instead of just one. which is why I changed it to mock out the requests call. I killed multiple calls with one stone | 21:42 |
thrawn01 | but i see ur point | 21:43 |
*** ebalduf has quit IRC | 21:43 | |
eharney | thrawn01: hmm, hopefully it's not too bad, i can help tomorrow if needed | 21:44 |
jgriffith | thrawn01: we've been bit pretty badly in the past calling internal lib methods; granted they were private methods so it was obvious, but still same risk probably | 21:48 |
*** nikeshm has quit IRC | 21:48 | |
jgriffith | thrawn01: on the bright side though, the unit tests all blew up right away :) | 21:49 |
jgriffith | making it rather obvious | 21:49 |
thrawn01 | jgriffith: I guess my concern is, if I mock out the keystone responses, then keystone changes, my tests wont fail because I mocked the responses. | 21:51 |
jgriffith | thrawn01: yeah, I kinda liked that approach TBH | 21:51 |
jgriffith | thrawn01: but on the other hand, don't we kinda want them to? | 21:51 |
*** sgotliv has joined #openstack-cinder | 21:51 | |
jgriffith | thrawn01: I mean if something changes that drastically in the keystone interface we probably won't all sorts of things to start puking | 21:52 |
eharney | jgriffith: i think so, since that would be breaking the library API | 21:52 |
jgriffith | thrawn01: anyway, I don't feel too strongly either way... there's pros/cons to each appraoch | 21:52 |
jgriffith | eharney: +1 | 21:52 |
jgriffith | eharney: the bad thing is that's not exactly unheard of :( | 21:53 |
jgriffith | but I think it's perfect acceptable to NOT expect those API interfaces to change | 21:53 |
eharney | jgriffith: i think the positive spin there is, we're agile :) | 21:53 |
jgriffith | eharney: LOL | 21:53 |
jgriffith | I like that | 21:53 |
*** dflorea has quit IRC | 21:54 | |
*** mriedem has quit IRC | 21:57 | |
*** kjnelson has quit IRC | 21:59 | |
*** krtaylor has joined #openstack-cinder | 22:00 | |
*** rushil_ has joined #openstack-cinder | 22:01 | |
*** crose has quit IRC | 22:03 | |
*** esker has joined #openstack-cinder | 22:03 | |
*** rushil has quit IRC | 22:04 | |
*** resker has joined #openstack-cinder | 22:07 | |
*** tiny-hands has quit IRC | 22:07 | |
*** resker has quit IRC | 22:08 | |
*** JoseMello has quit IRC | 22:09 | |
*** esker has quit IRC | 22:09 | |
*** sgotliv has quit IRC | 22:13 | |
*** annegentle has quit IRC | 22:17 | |
*** edtubill has quit IRC | 22:20 | |
openstackgerrit | Derrick Wippler proposed openstack/python-cinderclient: Fixed test_password_prompted https://review.openstack.org/214704 | 22:21 |
*** esker has joined #openstack-cinder | 22:21 | |
*** akerr has joined #openstack-cinder | 22:22 | |
openstackgerrit | Derrick Wippler proposed openstack/python-cinderclient: Fixed test_password_prompted https://review.openstack.org/214704 | 22:24 |
openstackgerrit | Derrick Wippler proposed openstack/python-cinderclient: Fixed test_password_prompted https://review.openstack.org/214704 | 22:25 |
thrawn01 | maybe I got it right this time ^ lol. many typo's | 22:26 |
hemna | patrickeast, ping | 22:32 |
patrickeast | hemna: yo | 22:32 |
hemna | patrickeast, so I just turned on friendly names | 22:33 |
hemna | and it looks like we might get lucky | 22:33 |
patrickeast | oh yea? | 22:33 |
hemna | multipath creates a /dev/disk/by-id/dm-uuid-mpath-<WWN> ->> ../../dm-N | 22:33 |
patrickeast | awesome! | 22:34 |
hemna | yah, so if /dev/mapper/<WWN> doesn't exist | 22:34 |
hemna | try /dev/disk/by-id/dm-uuid-mpath-<WWN> | 22:34 |
patrickeast | that makes sense, i had wondered how the system kept track of them without using the WWN | 22:34 |
hemna | then if that's not there, then mpath isn't running or it's blacklisted the array | 22:34 |
hemna | in which case /dev/disk/by-id/scsi-<WWN> will exist or the original discovered /dev/disk/by-path/<long ass file name here> | 22:35 |
hemna | so I'll see if I can update my patch to do those 3 tests | 22:35 |
hemna | which will be WAY faster than running multipath -ll <path> and then parsing out text | 22:36 |
patrickeast | yea definitely, will be a great improvement | 22:36 |
hemna | at least this is what I see on Ubuntu | 22:36 |
hemna | redhate might be different.... | 22:36 |
* hemna ducks | 22:36 | |
patrickeast | haha | 22:36 |
*** akerr has quit IRC | 22:36 | |
*** bswartz has joined #openstack-cinder | 22:36 | |
patrickeast | i mean, if it works on ubuntu + devstack its fine for shipping, right? | 22:37 |
hemna | ship that sh!t!! | 22:37 |
hemna | ok so everyone, start doing CI with redhat, fedora, Suse, Ubuntu.....go! | 22:37 |
hemna | easy peasy | 22:37 |
patrickeast | mhmm mhmm | 22:38 |
hemna | as easy as building an airplane. | 22:38 |
anish | just need some paper | 22:39 |
patrickeast | building the airplane is easy... making it fly... http://media.giphy.com/media/3oEdvcDQ8ce8xxdV2o/giphy.gif | 22:40 |
hemna | lol | 22:41 |
hemna | yah baby! | 22:41 |
anish | flugtag <3 | 22:41 |
hemna | ship it! | 22:41 |
anish | I prefer this one https://www.youtube.com/watch?v=auVvoxFv_Fo | 22:41 |
patrickeast | haha | 22:42 |
hemna | glad they had that long runup. | 22:42 |
anish | clearly a metaphor for devstack | 22:43 |
hemna | ok guys, off to soccer...l8s | 22:45 |
*** r-daneel has quit IRC | 22:49 | |
*** hemna is now known as hemnafk | 22:49 | |
*** mylu has quit IRC | 22:52 | |
*** mylu has joined #openstack-cinder | 22:53 | |
*** esker has quit IRC | 22:54 | |
*** esker has joined #openstack-cinder | 22:54 | |
*** bnemec has quit IRC | 22:56 | |
*** mylu_ has joined #openstack-cinder | 22:56 | |
*** mylu has quit IRC | 22:57 | |
*** mylu_ has quit IRC | 22:59 | |
*** mylu has joined #openstack-cinder | 22:59 | |
*** bill_az_ has joined #openstack-cinder | 23:00 | |
*** edtubill has joined #openstack-cinder | 23:00 | |
*** edtubill has left #openstack-cinder | 23:00 | |
*** bill_az has quit IRC | 23:01 | |
*** dannywil_ has joined #openstack-cinder | 23:01 | |
*** diogogmt has quit IRC | 23:03 | |
*** mylu_ has joined #openstack-cinder | 23:03 | |
*** dannywilson has quit IRC | 23:03 | |
*** mylu has quit IRC | 23:04 | |
*** lcurtis has quit IRC | 23:04 | |
*** vincent_hou has joined #openstack-cinder | 23:04 | |
*** mylu_ has quit IRC | 23:05 | |
*** mylu has joined #openstack-cinder | 23:06 | |
*** albertom is now known as albertom-afk | 23:06 | |
*** zul has joined #openstack-cinder | 23:07 | |
*** eharney has quit IRC | 23:08 | |
*** rushil_ has quit IRC | 23:10 | |
openstackgerrit | Danny Wilson proposed openstack/cinder: Cinder replication V2 https://review.openstack.org/200269 | 23:10 |
openstackgerrit | Danny Wilson proposed openstack/cinder: WIP - Replication V2 for Pure Storage https://review.openstack.org/214825 | 23:10 |
*** ebalduf has joined #openstack-cinder | 23:10 | |
*** mylu has quit IRC | 23:10 | |
*** rushil has joined #openstack-cinder | 23:10 | |
openstackgerrit | Anthony Lee proposed openstack/cinder: Fix QoS keys not being available to scheduler filter and weighers https://review.openstack.org/214826 | 23:11 |
*** zul has quit IRC | 23:12 | |
patrickeast | jgriffith: uhh i think danny might have accidently the latest replication patchset | 23:12 |
thingee | jgriffith: https://review.openstack.org/#/c/205622/ | 23:12 |
*** mylu has joined #openstack-cinder | 23:14 | |
openstackgerrit | Patrick East proposed openstack/cinder: WIP - Replication V2 for Pure Storage https://review.openstack.org/214825 | 23:16 |
*** mylu has quit IRC | 23:17 | |
*** mylu has joined #openstack-cinder | 23:17 | |
*** angela-s has quit IRC | 23:20 | |
*** bnemec has joined #openstack-cinder | 23:20 | |
patrickeast | hmm i updated that branch to be based on patchset 8 (like it should have been) and updated the review, but it didn't put the base replication one back to where it should be | 23:23 |
dannywil_ | ugh | 23:24 |
*** NightKhaos has joined #openstack-cinder | 23:26 | |
*** mylu has quit IRC | 23:26 | |
*** mylu has joined #openstack-cinder | 23:27 | |
*** chlong has quit IRC | 23:28 | |
NightKhaos | So, I just figured out how to online extend volumes with Libvirt and Ceph. I'm wondering if we can realistically build this into Cinder or Nova? | 23:31 |
*** dims_ has joined #openstack-cinder | 23:33 | |
*** dims has quit IRC | 23:36 | |
*** dims_ has quit IRC | 23:39 | |
*** dims has joined #openstack-cinder | 23:39 | |
*** zul has joined #openstack-cinder | 23:41 | |
*** vincent_hou has quit IRC | 23:42 | |
*** bnemec has quit IRC | 23:43 | |
*** dims has quit IRC | 23:44 | |
*** ganso_ has quit IRC | 23:45 | |
*** bnemec has joined #openstack-cinder | 23:45 | |
*** ndipanov has quit IRC | 23:45 | |
*** mylu has quit IRC | 23:50 | |
*** mylu_ has joined #openstack-cinder | 23:53 | |
*** edmondsw has quit IRC | 23:56 | |
*** erlon has quit IRC | 23:58 | |
*** bnemec has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!