| *** sghanekar_ has quit IRC | 00:00 | |
| *** setmason has left #openstack-cinder | 00:00 | |
| *** breitz has quit IRC | 00:01 | |
| *** dims has joined #openstack-cinder | 00:03 | |
| *** bnemec has quit IRC | 00:03 | |
| *** salv-orlando has quit IRC | 00:04 | |
| *** dims has quit IRC | 00:05 | |
| *** dims has joined #openstack-cinder | 00:05 | |
| *** takedakn has quit IRC | 00:07 | |
| *** takedakn has joined #openstack-cinder | 00:08 | |
| *** vilobhmm1 has joined #openstack-cinder | 00:09 | |
| *** sghanekar has joined #openstack-cinder | 00:09 | |
| *** tsekiyam_ has joined #openstack-cinder | 00:09 | |
| *** dims has quit IRC | 00:10 | |
| *** mtanino has quit IRC | 00:12 | |
| *** gouthamr has quit IRC | 00:12 | |
| *** tsekiyama has quit IRC | 00:13 | |
| *** gouthamr has joined #openstack-cinder | 00:13 | |
| *** tsekiyam_ has quit IRC | 00:13 | |
| *** leeantho has quit IRC | 00:16 | |
| *** dims has joined #openstack-cinder | 00:17 | |
| *** takedakn has quit IRC | 00:22 | |
| *** gouthamr has quit IRC | 00:26 | |
| *** mudassirlatif has quit IRC | 00:32 | |
| *** akerr has joined #openstack-cinder | 00:33 | |
| *** garthb has quit IRC | 00:34 | |
| *** akerr_ has joined #openstack-cinder | 00:38 | |
| *** markvoelker has joined #openstack-cinder | 00:39 | |
| *** akerr has quit IRC | 00:42 | |
| *** esker has joined #openstack-cinder | 00:43 | |
| *** _cjones_ has quit IRC | 00:44 | |
| *** vilobhmm1 has quit IRC | 00:56 | |
| *** EinstCrazy has joined #openstack-cinder | 00:56 | |
| *** zhangjn has joined #openstack-cinder | 00:57 | |
| *** zhenguo has joined #openstack-cinder | 00:57 | |
| *** dannywilson has quit IRC | 01:03 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:04 | |
| *** smoriya has joined #openstack-cinder | 01:10 | |
| winston-d_zZZ | hemnafk: ping | 01:11 |
|---|---|---|
| *** winston-d_zZZ is now known as winston-d | 01:11 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from delete methods https://review.openstack.org/205835 | 01:12 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add atomic conditional updates to objects https://review.openstack.org/205834 | 01:12 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: WIP: Remove more API races https://review.openstack.org/216378 | 01:12 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Improve metadata update operations https://review.openstack.org/216376 | 01:12 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from attach and detach methods https://review.openstack.org/216377 | 01:12 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Move get_by_id to CinderObject https://review.openstack.org/218012 | 01:12 |
| *** lixiaoy11 has joined #openstack-cinder | 01:15 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 01:15 |
| *** lixiaoy1 has quit IRC | 01:17 | |
| *** akerr_ has quit IRC | 01:23 | |
| guitarzan | jlk: nice! | 01:24 |
| *** davechen has joined #openstack-cinder | 01:28 | |
| *** setmason has joined #openstack-cinder | 01:29 | |
| *** Guest62749 has joined #openstack-cinder | 01:34 | |
| lixiaoy11 | geguileo: Hi Gorka, are you still here? | 01:34 |
| *** Guest62749 has quit IRC | 01:36 | |
| *** Lee1092 has joined #openstack-cinder | 01:39 | |
| *** akerr has joined #openstack-cinder | 01:45 | |
| *** sghanekar has quit IRC | 01:47 | |
| *** wilson1 has joined #openstack-cinder | 01:50 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Replication V2 for Pure Storage https://review.openstack.org/214825 | 01:53 |
| *** hw__chenying has joined #openstack-cinder | 01:53 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 01:53 |
| *** wilson has quit IRC | 01:53 | |
| *** setmason has quit IRC | 01:58 | |
| *** p0rtal_ has quit IRC | 02:00 | |
| *** dims has quit IRC | 02:06 | |
| *** jwcroppe has joined #openstack-cinder | 02:07 | |
| *** boris-42 has quit IRC | 02:10 | |
| *** jwcroppe has quit IRC | 02:11 | |
| *** Apoorva has quit IRC | 02:19 | |
| *** haomaiwang has joined #openstack-cinder | 02:23 | |
| *** haomaiwang has quit IRC | 02:24 | |
| *** haomaiwang has joined #openstack-cinder | 02:25 | |
| *** angerson has quit IRC | 02:26 | |
| *** angerson has joined #openstack-cinder | 02:29 | |
| *** dims has joined #openstack-cinder | 02:33 | |
| *** sparr has quit IRC | 02:34 | |
| *** wilson-1 has joined #openstack-cinder | 02:34 | |
| *** wilson1 has quit IRC | 02:38 | |
| *** sparr has joined #openstack-cinder | 02:42 | |
| openstackgerrit | chenying proposed openstack/python-cinderclient: Add a period for the description string of a argument https://review.openstack.org/218066 | 02:44 |
| *** haomaiwang has quit IRC | 02:44 | |
| *** hakimo has joined #openstack-cinder | 02:52 | |
| *** hakimo_ has quit IRC | 02:54 | |
| *** haomaiwang has joined #openstack-cinder | 02:58 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:02 | |
| *** raildo-afk has quit IRC | 03:05 | |
| *** mriedem has quit IRC | 03:06 | |
| *** bill_az has quit IRC | 03:07 | |
| *** raildo-afk has joined #openstack-cinder | 03:08 | |
| *** haomaiwang has quit IRC | 03:18 | |
| *** dims has quit IRC | 03:23 | |
| *** vaidy has quit IRC | 03:27 | |
| *** daneyon has joined #openstack-cinder | 03:27 | |
| *** vaidy has joined #openstack-cinder | 03:27 | |
| *** swamireddy has quit IRC | 03:27 | |
| *** haomaiwang has joined #openstack-cinder | 03:27 | |
| *** tbarron has quit IRC | 03:28 | |
| *** daneyon_ has quit IRC | 03:28 | |
| *** mgagne has quit IRC | 03:28 | |
| *** swamireddy has joined #openstack-cinder | 03:29 | |
| *** salv-orlando has joined #openstack-cinder | 03:30 | |
| *** tbarron has joined #openstack-cinder | 03:32 | |
| *** mgagne has joined #openstack-cinder | 03:34 | |
| *** haomaiwang has quit IRC | 03:36 | |
| *** salv-orlando has quit IRC | 03:36 | |
| *** harlowja_at_home has joined #openstack-cinder | 03:40 | |
| *** akerr is now known as akerr_away | 03:42 | |
| *** subhadeep_de has joined #openstack-cinder | 03:47 | |
| *** akerr_away is now known as akerr | 03:48 | |
| *** akerr has quit IRC | 03:49 | |
| subhadeep_de | Hi Folks, I am quite new to Cinder project. Let me know what would be the best way to contribute | 03:49 |
| scottda | subhadeep_de: Have you seen this: https://wiki.openstack.org/wiki/How_To_Contribute | 03:52 |
| subhadeep_de | yes, i did few bug fixes in cinder client and nova for the cinder for volume api in Juno | 03:53 |
| openstackgerrit | Tina Tang proposed openstack/cinder: Clone cg support in VNX driver https://review.openstack.org/215530 | 03:54 |
| subhadeep_de | should I start looking at the reviews for the existing bug fixes | 03:55 |
| subhadeep_de | to get more understanding on the ccinder driver? | 03:55 |
| anish | subhadeep_de: you might find more things to fix here https://github.com/openstack/os-brick | 03:58 |
| *** jwcroppe has joined #openstack-cinder | 03:59 | |
| *** harlowja_at_home has quit IRC | 04:01 | |
| subhadeep_de | thanks anish I will look into this, is this more of discovery module on the compute node | 04:02 |
| subhadeep_de | ? | 04:02 |
| *** hemna has joined #openstack-cinder | 04:05 | |
| anish | subhadeep_de: no, it combines the iscsi code from nova/cinder into a separate library | 04:07 |
| anish | it's relatively new so probably has more things to do than regular cinder | 04:07 |
| *** links has joined #openstack-cinder | 04:09 | |
| subhadeep_de | okay, thanks I will go through the library details. does it come deployed with devstack or we need to install it seperately? | 04:10 |
| *** subhadeep_de has quit IRC | 04:14 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 04:17 |
| *** martyturner has joined #openstack-cinder | 04:18 | |
| *** baojg has joined #openstack-cinder | 04:19 | |
| *** martyturner has quit IRC | 04:22 | |
| *** hw__chenying has quit IRC | 04:23 | |
| *** haomaiwang has joined #openstack-cinder | 04:26 | |
| *** vilobhmm1 has joined #openstack-cinder | 04:27 | |
| *** frickler has quit IRC | 04:30 | |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Volume status managment for voluem migration https://review.openstack.org/189547 | 04:31 |
| openstackgerrit | Patrick East proposed openstack/cinder: Replication V2 for Pure Storage https://review.openstack.org/214825 | 04:34 |
| *** subhadeep_de has joined #openstack-cinder | 04:35 | |
| subhadeep_de | Hi anish, does it come deployed with devstack? | 04:35 |
| *** subhadeep_de has quit IRC | 04:36 | |
| *** subhadeep_de has joined #openstack-cinder | 04:37 | |
| *** BharatK has joined #openstack-cinder | 04:42 | |
| *** p0rtal has joined #openstack-cinder | 04:47 | |
| *** p0rtal has quit IRC | 04:48 | |
| *** p0rtal has joined #openstack-cinder | 04:48 | |
| *** haomaiwang has quit IRC | 04:50 | |
| *** deepakcs has joined #openstack-cinder | 04:56 | |
| *** subhadee_ has joined #openstack-cinder | 04:57 | |
| *** subhadee_ has quit IRC | 05:00 | |
| *** p0rtal has quit IRC | 05:00 | |
| *** subhadee_ has joined #openstack-cinder | 05:00 | |
| *** subhadeep_de has quit IRC | 05:01 | |
| *** subhadee_ has quit IRC | 05:02 | |
| *** theanalyst has quit IRC | 05:02 | |
| *** subhadeep_de has joined #openstack-cinder | 05:02 | |
| *** theanalyst has joined #openstack-cinder | 05:05 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Allow specified backend capabilities to be retrieved https://review.openstack.org/212861 | 05:07 |
| *** hemna has quit IRC | 05:15 | |
| *** setmason has joined #openstack-cinder | 05:15 | |
| *** setmason_ has joined #openstack-cinder | 05:18 | |
| *** setmason has quit IRC | 05:20 | |
| *** setmason_ is now known as setmason | 05:20 | |
| *** shyama has joined #openstack-cinder | 05:25 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Data transfer between encrypted volumes and images https://review.openstack.org/216567 | 05:30 |
| *** salv-orlando has joined #openstack-cinder | 05:31 | |
| *** haomaiwang has joined #openstack-cinder | 05:33 | |
| *** haomaiwang has quit IRC | 05:35 | |
| *** salv-orlando has quit IRC | 05:36 | |
| *** jwcroppe has quit IRC | 05:41 | |
| *** jwcroppe has joined #openstack-cinder | 05:42 | |
| *** skylerberg has quit IRC | 05:47 | |
| *** jwcroppe has quit IRC | 05:48 | |
| *** BharatK has quit IRC | 05:51 | |
| *** setmason has quit IRC | 05:51 | |
| *** setmason has joined #openstack-cinder | 05:52 | |
| *** agarciam has joined #openstack-cinder | 05:53 | |
| *** haomaiwang has joined #openstack-cinder | 06:02 | |
| *** BharatK has joined #openstack-cinder | 06:06 | |
| *** setmason has quit IRC | 06:17 | |
| *** salv-orlando has joined #openstack-cinder | 06:18 | |
| *** haomaiwang has quit IRC | 06:23 | |
| *** anshul has joined #openstack-cinder | 06:28 | |
| *** lixiaoy11 has quit IRC | 06:32 | |
| *** lixiaoy1 has joined #openstack-cinder | 06:32 | |
| *** baojg has quit IRC | 06:33 | |
| *** p0rtal has joined #openstack-cinder | 06:35 | |
| *** p0rtal has quit IRC | 06:35 | |
| *** p0rtal has joined #openstack-cinder | 06:35 | |
| nikeshm | hi | 06:40 |
| *** ankit_ag has joined #openstack-cinder | 06:42 | |
| *** JoeStack has joined #openstack-cinder | 06:42 | |
| *** jwcroppe has joined #openstack-cinder | 06:44 | |
| *** lixiaoy11 has joined #openstack-cinder | 06:45 | |
| *** lixiaoy1 has quit IRC | 06:47 | |
| *** nkrinner has joined #openstack-cinder | 06:48 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Raise HTTPBadRequest when invalid filters are given https://review.openstack.org/217015 | 06:50 |
| openstackgerrit | LisaLi proposed openstack/cinder: Raise HTTPBadRequest for invalid filters https://review.openstack.org/217015 | 07:00 |
| *** vilobhmm1 has quit IRC | 07:02 | |
| *** vincent_hou has joined #openstack-cinder | 07:03 | |
| *** ronis has joined #openstack-cinder | 07:05 | |
| *** p0rtal has quit IRC | 07:09 | |
| *** vincent_hou has quit IRC | 07:10 | |
| *** vincent_hou has joined #openstack-cinder | 07:11 | |
| *** stefan_amann has joined #openstack-cinder | 07:12 | |
| *** EinstCrazy has quit IRC | 07:13 | |
| *** zhangjn has quit IRC | 07:15 | |
| *** jwcroppe has quit IRC | 07:18 | |
| *** subhadee_ has joined #openstack-cinder | 07:21 | |
| *** subhadeep_de has quit IRC | 07:21 | |
| *** haomaiwang has joined #openstack-cinder | 07:38 | |
| *** bluex-pl has joined #openstack-cinder | 07:42 | |
| *** bluex-pl has quit IRC | 07:42 | |
| *** bluex-pl has joined #openstack-cinder | 07:43 | |
| openstackgerrit | XinXiaohui proposed openstack/cinder: Calculate virtual free capacity and notify https://review.openstack.org/206923 | 07:43 |
| *** sgotliv has joined #openstack-cinder | 07:44 | |
| *** haomaiwang has quit IRC | 07:44 | |
| *** haomaiwa_ has joined #openstack-cinder | 07:45 | |
| *** haomaiw__ has joined #openstack-cinder | 07:49 | |
| *** haomaiwa_ has quit IRC | 07:50 | |
| *** ndipanov has joined #openstack-cinder | 07:51 | |
| *** ndipanov has quit IRC | 07:52 | |
| *** ndipanov has joined #openstack-cinder | 07:52 | |
| *** haomaiw__ has quit IRC | 07:58 | |
| *** ronis has quit IRC | 08:02 | |
| *** leopoldj has joined #openstack-cinder | 08:04 | |
| *** shyama has quit IRC | 08:05 | |
| *** yuriy_n17 has joined #openstack-cinder | 08:08 | |
| *** sgotliv has quit IRC | 08:11 | |
| *** jistr has joined #openstack-cinder | 08:12 | |
| *** jordanP has joined #openstack-cinder | 08:13 | |
| *** hw__chenying has joined #openstack-cinder | 08:17 | |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Volume migration progress https://review.openstack.org/218145 | 08:18 |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Volume migration progress https://review.openstack.org/218145 | 08:22 |
| *** ronis has joined #openstack-cinder | 08:24 | |
| *** IanGovett has joined #openstack-cinder | 08:25 | |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Volume status managment for volume migration https://review.openstack.org/189547 | 08:26 |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Volume migration progress https://review.openstack.org/218145 | 08:26 |
| *** lpetrut has joined #openstack-cinder | 08:34 | |
| *** lpetrut has left #openstack-cinder | 08:39 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds migration abortion https://review.openstack.org/207754 | 08:40 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 08:40 |
| openstackgerrit | Takeaki Matsumoto proposed openstack/cinder: Fix url in API response to get original https://review.openstack.org/216591 | 08:40 |
| *** shyama has joined #openstack-cinder | 08:47 | |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Volume status management for volume migration https://review.openstack.org/189547 | 08:47 |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Volume migration progress https://review.openstack.org/218145 | 08:47 |
| *** amit213 has quit IRC | 08:50 | |
| *** amit213 has joined #openstack-cinder | 08:51 | |
| *** vincent_hou has quit IRC | 08:52 | |
| *** baojg has joined #openstack-cinder | 08:53 | |
| *** e0ne has joined #openstack-cinder | 08:54 | |
| *** skraynev has quit IRC | 09:00 | |
| *** e0ne has quit IRC | 09:03 | |
| *** jordanP has quit IRC | 09:07 | |
| *** jordanP has joined #openstack-cinder | 09:07 | |
| *** takedakn has joined #openstack-cinder | 09:23 | |
| *** e0ne has joined #openstack-cinder | 09:25 | |
| *** subhadeep_de has joined #openstack-cinder | 09:25 | |
| *** subhadeep_de has quit IRC | 09:26 | |
| *** subhadee_ has quit IRC | 09:26 | |
| *** subhadeep_de has joined #openstack-cinder | 09:26 | |
| *** baojg has quit IRC | 09:26 | |
| *** skraynev has joined #openstack-cinder | 09:33 | |
| *** takedakn has quit IRC | 09:33 | |
| *** haomaiwang has joined #openstack-cinder | 09:34 | |
| *** skraynev has quit IRC | 09:38 | |
| *** salv-orlando has quit IRC | 09:42 | |
| *** davechen has left #openstack-cinder | 09:43 | |
| *** skraynev has joined #openstack-cinder | 09:45 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Minor optimization https://review.openstack.org/218178 | 09:49 |
| *** rushiagr_away is now known as rushiagr | 09:53 | |
| *** lixiaoy11 has quit IRC | 09:56 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 10:02 | |
| *** bluex-pl has quit IRC | 10:06 | |
| *** bluex-pl has joined #openstack-cinder | 10:06 | |
| *** bluex-pl has quit IRC | 10:07 | |
| *** bluex-pl has joined #openstack-cinder | 10:08 | |
| *** bluex-pl has quit IRC | 10:08 | |
| *** bluex-pl has joined #openstack-cinder | 10:08 | |
| *** jwcroppe has joined #openstack-cinder | 10:22 | |
| *** lpetrut1 has joined #openstack-cinder | 10:25 | |
| *** haomaiwa_ has quit IRC | 10:27 | |
| *** yuriy_n17 has quit IRC | 10:27 | |
| *** mgagne has quit IRC | 10:27 | |
| *** swamireddy has quit IRC | 10:27 | |
| *** daneyon has quit IRC | 10:27 | |
| *** vaidy has quit IRC | 10:27 | |
| *** hakimo has quit IRC | 10:27 | |
| *** aix has quit IRC | 10:27 | |
| *** jgriffith has quit IRC | 10:27 | |
| *** jamielennox has quit IRC | 10:27 | |
| *** sayali has quit IRC | 10:27 | |
| *** d0ugal has quit IRC | 10:27 | |
| *** links has quit IRC | 10:27 | |
| *** tbarron has quit IRC | 10:27 | |
| *** markvoelker has quit IRC | 10:27 | |
| *** jwang has quit IRC | 10:27 | |
| *** Roamer` has quit IRC | 10:27 | |
| *** lprice1 has quit IRC | 10:27 | |
| *** afazekas has quit IRC | 10:27 | |
| *** leseb has quit IRC | 10:27 | |
| *** anteaya has quit IRC | 10:27 | |
| *** lpetrut1 is now known as lpetrut | 10:27 | |
| *** links has joined #openstack-cinder | 10:29 | |
| *** tbarron has joined #openstack-cinder | 10:29 | |
| *** markvoelker has joined #openstack-cinder | 10:29 | |
| *** lprice1 has joined #openstack-cinder | 10:29 | |
| *** jwang has joined #openstack-cinder | 10:29 | |
| *** Roamer` has joined #openstack-cinder | 10:29 | |
| *** afazekas has joined #openstack-cinder | 10:29 | |
| *** leseb has joined #openstack-cinder | 10:29 | |
| *** anteaya has joined #openstack-cinder | 10:29 | |
| *** lpetrut is now known as 18VAAKDGR | 10:29 | |
| *** haomaiwa_ has joined #openstack-cinder | 10:30 | |
| *** yuriy_n17 has joined #openstack-cinder | 10:30 | |
| *** mgagne has joined #openstack-cinder | 10:30 | |
| *** swamireddy has joined #openstack-cinder | 10:30 | |
| *** vaidy has joined #openstack-cinder | 10:30 | |
| *** daneyon has joined #openstack-cinder | 10:30 | |
| *** hakimo has joined #openstack-cinder | 10:30 | |
| *** aix has joined #openstack-cinder | 10:30 | |
| *** jgriffith has joined #openstack-cinder | 10:30 | |
| *** jamielennox has joined #openstack-cinder | 10:30 | |
| *** sayali has joined #openstack-cinder | 10:30 | |
| *** d0ugal has joined #openstack-cinder | 10:30 | |
| *** haomaiwa_ has quit IRC | 10:30 | |
| *** yuriy_n17 has quit IRC | 10:30 | |
| *** mgagne has quit IRC | 10:30 | |
| *** swamireddy has quit IRC | 10:30 | |
| *** daneyon has quit IRC | 10:30 | |
| *** vaidy has quit IRC | 10:30 | |
| *** hakimo has quit IRC | 10:30 | |
| *** aix has quit IRC | 10:30 | |
| *** jgriffith has quit IRC | 10:30 | |
| *** jamielennox has quit IRC | 10:30 | |
| *** sayali has quit IRC | 10:30 | |
| *** d0ugal has quit IRC | 10:30 | |
| *** yuriy_n17 has joined #openstack-cinder | 10:32 | |
| *** mgagne has joined #openstack-cinder | 10:32 | |
| *** swamireddy has joined #openstack-cinder | 10:32 | |
| *** vaidy has joined #openstack-cinder | 10:32 | |
| *** daneyon has joined #openstack-cinder | 10:32 | |
| *** hakimo has joined #openstack-cinder | 10:32 | |
| *** aix has joined #openstack-cinder | 10:32 | |
| *** jgriffith has joined #openstack-cinder | 10:32 | |
| *** jamielennox has joined #openstack-cinder | 10:32 | |
| *** sayali has joined #openstack-cinder | 10:32 | |
| *** d0ugal has joined #openstack-cinder | 10:32 | |
| *** hw__chenying has quit IRC | 10:34 | |
| *** zhenguo has quit IRC | 10:40 | |
| *** bluex-pl has quit IRC | 10:51 | |
| *** dimsum__ has joined #openstack-cinder | 10:53 | |
| *** takedakn has joined #openstack-cinder | 10:56 | |
| *** bluex-pl has joined #openstack-cinder | 11:01 | |
| *** bluex-pl has quit IRC | 11:02 | |
| *** tbarron has left #openstack-cinder | 11:03 | |
| *** tbarron has joined #openstack-cinder | 11:03 | |
| *** bluex-pl has joined #openstack-cinder | 11:03 | |
| *** bluex-pl has quit IRC | 11:04 | |
| *** bluex-pl has joined #openstack-cinder | 11:04 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 11:07 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from delete methods https://review.openstack.org/205835 | 11:13 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add atomic conditional updates to objects https://review.openstack.org/205834 | 11:13 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: WIP: Remove more API races https://review.openstack.org/216378 | 11:13 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Move get_by_id to CinderObject https://review.openstack.org/218012 | 11:13 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Improve metadata update operations https://review.openstack.org/216376 | 11:13 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from attach and detach methods https://review.openstack.org/216377 | 11:13 |
| *** aix has quit IRC | 11:14 | |
| *** takedakn has quit IRC | 11:24 | |
| *** takedakn has joined #openstack-cinder | 11:26 | |
| *** takedakn has quit IRC | 11:35 | |
| *** porrua has joined #openstack-cinder | 11:39 | |
| *** shyama has quit IRC | 11:40 | |
| *** esker has quit IRC | 11:40 | |
| *** porrua has quit IRC | 11:46 | |
| *** wilson1 has joined #openstack-cinder | 11:47 | |
| *** wilson-1 has quit IRC | 11:50 | |
| *** julim has joined #openstack-cinder | 11:51 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Remove hardcoded releases list from unit tests https://review.openstack.org/218226 | 11:52 |
| *** tpsilva has joined #openstack-cinder | 11:59 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Clone cg support in VNX driver https://review.openstack.org/215530 | 12:00 |
| *** rushiagr is now known as rushiagr_away | 12:03 | |
| *** jwcroppe has quit IRC | 12:06 | |
| *** deepakcs has quit IRC | 12:14 | |
| *** raildo-afk is now known as raildo | 12:16 | |
| *** martyturner has joined #openstack-cinder | 12:20 | |
| *** aix has joined #openstack-cinder | 12:24 | |
| *** akerr has joined #openstack-cinder | 12:28 | |
| *** salv-orlando has joined #openstack-cinder | 12:30 | |
| jordanP | geguileo, hi. Is someone working on the UT failure in VMware ? | 12:31 |
| jordanP | I am also affected by this | 12:31 |
| *** salv-orlando has quit IRC | 12:32 | |
| geguileo | jordanP: UT? | 12:33 |
| jordanP | geguileo, unit test | 12:33 |
| geguileo | jordanP: You mean the _SentinelObject issue? | 12:33 |
| jordanP | geguileo, yes | 12:34 |
| *** chlong has joined #openstack-cinder | 12:34 | |
| geguileo | jordanP: I have been looking at it at it's not an easy problem :-( | 12:34 |
| jordanP | yeah but it's blocking cinder... | 12:35 |
| geguileo | jordanP: It only happens when tests are run concurrently | 12:35 |
| geguileo | jordanP: And not always | 12:35 |
| geguileo | jordanP: It appeared on the 24th | 12:35 |
| erlon | geguileo: does it use something in CONF? | 12:35 |
| jordanP | geguileo, do you have an elasticsearch query already ? | 12:35 |
| erlon | geguileo: I have run into problems using conf before | 12:35 |
| geguileo | erlon: The problem is that a Mock suddenly is not into place | 12:36 |
| erlon | geguileo: ?? | 12:36 |
| geguileo | jordanP: I did a quick on with just the message | 12:36 |
| geguileo | s/on/one | 12:36 |
| geguileo | erlon: You run the tests with ./test_run -> Everyting OK | 12:36 |
| geguileo | erlon: You run it with tox (uses parallel execution) -> Fails 1 every N runs | 12:37 |
| erlon | geguileo: they happen with --concurrency>1? | 12:37 |
| geguileo | erlon: Yes | 12:37 |
| erlon | geguileo: hmm | 12:37 |
| geguileo | erlon: And what happens is that a method that should have been Mocked, is not mocked | 12:37 |
| eikke | geguileo: I believe test_create_backing_from_stream_optimized_file mock's builtin 'open', which is a global effect, and can indeed affect other tests under parallel execution (in a single python runtime) | 12:38 |
| geguileo | erlon: I think it is not a inter-test dependency | 12:38 |
| erlon | geguileo: hmm, ill try to reproduce here | 12:38 |
| erlon | geguileo: what is the tox command? | 12:38 |
| jordanP | this line is suspicious also: https://github.com/openstack/cinder/blob/master/cinder/tests/unit/test_vmware_vmdk.py#L1687 | 12:39 |
| jordanP | file_open is the mock of builtin open | 12:39 |
| eikke | which is used as a context manager all the time in cinder.volume.drivers.vmware.vmdk.open | 12:39 |
| geguileo | jordanP: Why suspicious? | 12:40 |
| eikke | so within the context, tmp_file will be 'the file object returned by the open call' | 12:40 |
| eikke | so if you have concurrently running tests, and another test performs 'open' (like, linecache does when looking at the backtrace), things go wrong | 12:40 |
| jordanP | geguileo, that's not how I would mock 'open' | 12:40 |
| geguileo | jordanP: Neither would I | 12:41 |
| jordanP | I would use mock.mock_open | 12:41 |
| geguileo | jordanP: But that hasn't change lately (I believe) | 12:41 |
| eikke | jordanP: that'd have the same problem here, fwiw | 12:41 |
| geguileo | jordanP: By the way, thanks for the Tempest review :-) | 12:42 |
| *** bluex-pl has quit IRC | 12:42 | |
| *** BharatK has quit IRC | 12:42 | |
| *** bluex-pl has joined #openstack-cinder | 12:43 | |
| jordanP | geguileo, np :) | 12:43 |
| jordanP | that volume backup is indeed broken | 12:43 |
| *** bluex-pl has quit IRC | 12:43 | |
| geguileo | erlon: jordanP I have to start testing migration patches that have been submitted, so I can't work more on that right now, I'll try to have a look later tonight if it isn't resolved by then | 12:43 |
| *** bluex-pl has joined #openstack-cinder | 12:44 | |
| *** gouthamr has joined #openstack-cinder | 12:44 | |
| erlon | geguileo: ok, I won't have that much time too, but ill try to give a look | 12:44 |
| geguileo | erlon: OK, thanks | 12:45 |
| *** haomaiwa_ has joined #openstack-cinder | 12:45 | |
| *** zul has quit IRC | 12:49 | |
| *** Yogi1 has joined #openstack-cinder | 12:49 | |
| *** rushiagr_away is now known as rushiagr | 12:51 | |
| *** zul has joined #openstack-cinder | 12:52 | |
| *** takedakn has joined #openstack-cinder | 12:52 | |
| *** timcl has joined #openstack-cinder | 12:53 | |
| jordanP | eikke, you said "<eikke> jordanP: that'd have the same problem here, fwiw" | 12:54 |
| jordanP | but mock_open has a readlines() method | 12:55 |
| jordanP | guys a logstash query: http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiQXR0cmlidXRlRXJyb3I6ICdfU2VudGluZWxPYmplY3QnIG9iamVjdCBoYXMgbm8gYXR0cmlidXRlICdyZWFkbGluZXMnXCIgQU5EIHRhZ3M6Y29uc29sZSIsImZpZWxkcyI6W10sIm9mZnNldCI6MCwidGltZWZyYW1lIjoiMTcyODAwIiwiZ3JhcGhtb2RlIjoiY291bnQiLCJ0aW1lIjp7InVzZXJfaW50ZXJ2YWwiOjB9LCJzdGFtcCI6MTQ0MDc2NjU3MDY2MX0= | 12:56 |
| *** takedakn has quit IRC | 12:56 | |
| jordanP | 100% failure | 12:57 |
| geguileo | jordanP: What? | 12:57 |
| *** merooney has joined #openstack-cinder | 12:57 | |
| geguileo | jordanP: Is it 100% now? | 12:57 |
| jordanP | if the message "SentinelObject' object has no attribute 'readline" is in the console log then the job fails | 12:58 |
| *** nkrinner has quit IRC | 12:58 | |
| jordanP | I am not saying cinder is 100% blocked :) | 12:58 |
| tbarron | jordanP: geguileo: eikke: https://bugs.launchpad.net/cinder/+bug/1488690 | 12:59 |
| openstack | Launchpad bug 1488690 in Cinder "test_create_backing_from_stream_optimized_file fails with "AttributeError: '_SentinelObject' object has no attribute 'readlines'"" [Undecided,In progress] - Assigned to Jon Bernard (jbernard) | 12:59 |
| jordanP | just that my query is a good marker | 12:59 |
| *** takedakn has joined #openstack-cinder | 12:59 | |
| tbarron | jordanP: geguileo: eikke: https://review.openstack.org/#/c/217882/ | 12:59 |
| geguileo | tbarron: Thanks | 12:59 |
| *** dustins has joined #openstack-cinder | 13:00 | |
| geguileo | tbarron: Although I never like skipping tests | 13:00 |
| jordanP | "nuke it from orbit" | 13:00 |
| jbernard | i dont' either, but ive really failed at getting it to fail locally | 13:00 |
| tbarron | jordanP: geguileo: eikke: also https://review.openstack.org/#/c/217936/ | 13:00 |
| *** haomaiwa_ has quit IRC | 13:01 | |
| geguileo | tbarron: Yeah, that's more like it :-) | 13:01 |
| jordanP | I am not sure | 13:01 |
| tbarron | geguileo: but it doesn't work. It was my guess too though. | 13:01 |
| jordanP | open was mocked | 13:01 |
| geguileo | Although I'll have to talk with Eric | 13:01 |
| geguileo | Because it should be the same thing | 13:01 |
| jordanP | yes | 13:01 |
| geguileo | And as far as I could tell | 13:02 |
| geguileo | The problem is not there | 13:02 |
| *** haomaiwang has joined #openstack-cinder | 13:02 | |
| jordanP | So the "skip" patch is stuck in the gate... | 13:02 |
| *** nkrinner has joined #openstack-cinder | 13:03 | |
| tbarron | geguileo: jordanP: eikke: jbernard: so what eikke was saying above sounds plausible. | 13:03 |
| erlon | geguileo: what BEs will use to test the migration? | 13:04 |
| geguileo | erlon: I was just asking myself that same question XD | 13:04 |
| geguileo | erlon: Probably LVM, NFS and Ceph | 13:04 |
| erlon | geguileo: lol | 13:04 |
| tbarron | I really don't like the way that test combines a positive and negative test case in the same test either. | 13:05 |
| eikke | I think here's the problem | 13:05 |
| erlon | geguileo: Im not familiar with Ceph, does it need a hardware? | 13:05 |
| jbernard | erlon: nope | 13:05 |
| eikke | https://github.com/openstack/cinder/commit/212aff327a36f925be3be69b9b54b946dbdd5c2a removed the use of fileutils | 13:05 |
| geguileo | erlon: It needs a cluster | 13:06 |
| eikke | before, fileutils.open was mocked, which doesnt affect 'open' | 13:06 |
| eikke | after, global 'open' is mocked | 13:06 |
| geguileo | erlon: Or I think devstack can deploy it as well (that one I haven't tried) | 13:06 |
| tbarron | well, leaving while I drive off to work. I'll catch up on the backlog when I get there. eikke: sounds like you are on it! | 13:06 |
| eikke | but now somewhere in the vmware driver (in _create_backing_from_stream_optimized_file), an exception is thrown | 13:06 |
| eikke | which is caught and re-thrown through oslo_utils' rethrow context manager | 13:06 |
| eikke | which then uses traceback & linecache, which uses open, which fails etc | 13:07 |
| erlon | geguileo: hmmm, we will probably need to test migration in our drivers as well, and I want to setup a environment with as many BE as possible | 13:07 |
| eikke | hiding the original exception | 13:07 |
| erlon | geguileo: will you use single or multi node? | 13:07 |
| geguileo | erlon: Single | 13:08 |
| eikke | jordanP: sorry, coming back to the fact mock_open() has a 'readlines' attribute: sure, but it won't return what linecache expects it to return. Really, mocking something global like 'open' isi rather bad practice, because someone in any call chain could require open to behave non-mocked, and then you're screwed | 13:09 |
| jordanP | agreed | 13:09 |
| geguileo | eikke: Yes, something like pdb.set_trace() ;-) | 13:09 |
| geguileo | XD | 13:09 |
| eikke | jordanP: remember how I advised you to mock 'as high up in the chain as possible'? ;-) | 13:10 |
| *** aix has quit IRC | 13:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/215440 | 13:12 |
| *** kbyrne has quit IRC | 13:13 | |
| jordanP | pepperidge farm remembers | 13:15 |
| *** jwcroppe has joined #openstack-cinder | 13:16 | |
| *** annegentle has joined #openstack-cinder | 13:17 | |
| openstackgerrit | Edmund Rhudy proposed openstack/cinder: Adds allow_availability_zone_fallback option to Cinder https://review.openstack.org/217857 | 13:19 |
| *** kbyrne has joined #openstack-cinder | 13:19 | |
| *** kbyrne has quit IRC | 13:20 | |
| smcginnis | e0ne, geguileo, jgriffith: This looks good to me, but would like at least one more set of eyes on it: https://review.openstack.org/#/c/201243/ | 13:21 |
| eikke | bon, added some comments to the bug report | 13:21 |
| *** kbyrne has joined #openstack-cinder | 13:21 | |
| geguileo | smcginnis: I'll have a look | 13:21 |
| geguileo | smcginnis: I reviewed previous path, so it should be quick | 13:21 |
| smcginnis | geguileo: Yeah, just would like more sanity check. Thanks! | 13:22 |
| *** asselin has joined #openstack-cinder | 13:22 | |
| e0ne | smcginnis: thanks for reminder:) | 13:23 |
| *** leopoldj has quit IRC | 13:24 | |
| *** breitz has joined #openstack-cinder | 13:28 | |
| *** links has quit IRC | 13:29 | |
| *** guest1 has joined #openstack-cinder | 13:29 | |
| *** aix has joined #openstack-cinder | 13:29 | |
| *** IanGovett has quit IRC | 13:30 | |
| geguileo | smcginnis: Do you want me to +1 the workflow or do you want to do it yourself? | 13:31 |
| *** IanGovett has joined #openstack-cinder | 13:31 | |
| *** merooney has quit IRC | 13:33 | |
| smcginnis | geguileo: If you're good with it then go ahead. | 13:34 |
| geguileo | smcginnis: Ok, done | 13:34 |
| smcginnis | geguileo: Awesome | 13:34 |
| *** bill_az has joined #openstack-cinder | 13:34 | |
| smcginnis | geguileo: Noticed your comment on there. The logger handles conversion if it's unicode. | 13:35 |
| *** lcurtis has joined #openstack-cinder | 13:35 | |
| *** JoeStack has quit IRC | 13:36 | |
| *** mriedem has joined #openstack-cinder | 13:37 | |
| *** salv-orlando has joined #openstack-cinder | 13:37 | |
| geguileo | smcginnis: Yep, I was thinking of % | 13:37 |
| smcginnis | ;) | 13:37 |
| *** setmason has joined #openstack-cinder | 13:37 | |
| geguileo | smcginnis: I'm a little overwhelmed with all the stuff I have to do %-D | 13:38 |
| *** setmason has quit IRC | 13:38 | |
| *** thangp has joined #openstack-cinder | 13:38 | |
| smcginnis | geguileo: Hah, yeah. Suddenly after L-3 it will slow down. :) | 13:39 |
| geguileo | smcginnis: I hope so!! | 13:39 |
| *** merooney has joined #openstack-cinder | 13:39 | |
| smcginnis | geguileo: And then the summit and it all starts back up again. | 13:40 |
| geguileo | smcginnis: XD XD | 13:40 |
| e0ne | guys, what are your thoughts about https://review.openstack.org/#/c/217857/? | 13:40 |
| e0ne | personaly, i'm going to -1 on it | 13:40 |
| openstackgerrit | Tina Tang proposed openstack/cinder: Clone cg support in VNX driver https://review.openstack.org/215530 | 13:41 |
| *** setmason has joined #openstack-cinder | 13:41 | |
| smcginnis | e0ne: Looking... | 13:41 |
| smcginnis | e0ne: Not sure if that is the right fix for what the bug is describing. | 13:42 |
| *** dansmith is now known as superdan | 13:42 | |
| * smcginnis 's 20 second analysis | 13:42 | |
| e0ne | agree. posting my comments there | 13:42 |
| smcginnis | And updating pot file. | 13:43 |
| e0ne | smcginnis: great skilll "20s analysis for everything" | 13:43 |
| e0ne | :) | 13:43 |
| smcginnis | Don't touch the pot. Unless you're in one of those states. | 13:43 |
| smcginnis | Sorry, that was bad. :D | 13:43 |
| e0ne | "I'm not a fun of the such solution at all. IMO, if we need to disable AZ, we need to do it in a scheduler or even disable AZ filter at all." - copy&paste from gerrit | 13:44 |
| smcginnis | +1 | 13:44 |
| smcginnis | To you, not the patch. :) | 13:44 |
| e0ne | :) | 13:45 |
| *** garthb has joined #openstack-cinder | 13:45 | |
| *** bnemec has joined #openstack-cinder | 13:46 | |
| *** edmondsw has joined #openstack-cinder | 13:48 | |
| *** dimsum__ has quit IRC | 13:49 | |
| *** dimsum__ has joined #openstack-cinder | 13:50 | |
| *** rushiagr is now known as rushiagr_away | 13:51 | |
| dulek | e0ne: Disabling AvailabilityZoneFilter will not work. | 13:52 |
| dulek | e0ne: AZ is validated on an c-api level. | 13:52 |
| *** eharney has joined #openstack-cinder | 13:53 | |
| *** guest1 has quit IRC | 13:53 | |
| e0ne | dulek: it means we've got other issue | 13:55 |
| dulek | e0ne: Why do you think that? | 13:55 |
| e0ne | dulek: if we have AvailabilityZoneFilter in scheduler, we must validate AZ in scheduler | 13:56 |
| dulek | e0ne: AZ is validated in the API in the same way source_volid is validated. | 13:56 |
| smcginnis | WOw, the gate queue is going to take a long time to drain I think. | 13:57 |
| dulek | e0ne: Besides you can skip scheduler if for example you're creating volume from source. | 13:57 |
| *** merooney has quit IRC | 13:58 | |
| *** smoriya has quit IRC | 13:59 | |
| dulek | e0ne: Yup, then you have CONF.cloned_volume_same_az option - this is checked in API, because such call will bypass the scheduler. | 13:59 |
| *** ankit_ag has quit IRC | 13:59 | |
| dulek | e0ne: That's why I think the fallback fix is valid *for this release*. | 13:59 |
| e0ne | dulek: oh.. it's crazy | 14:00 |
| dulek | e0ne: Otherwise we may create backward incompatibilities, which are probably unacceptable at this time in the cycle. | 14:00 |
| *** setmason has quit IRC | 14:00 | |
| dulek | e0ne: I don't like it either. ;) | 14:00 |
| openstackgerrit | wanghao proposed openstack/cinder: Add pagination to backups https://review.openstack.org/204493 | 14:01 |
| *** haomaiwang has quit IRC | 14:01 | |
| e0ne | dulek: i mean our current AZ validation | 14:01 |
| dulek | e0ne: I'm working on a spec for Mitaka to sort this stuff out. | 14:01 |
| *** jwcroppe has quit IRC | 14:01 | |
| dulek | smcginnis: Any chance of 20 second analysis of my previous messages? ;) | 14:01 |
| *** haomaiwang has joined #openstack-cinder | 14:02 | |
| smcginnis | dulek: Sorry, gotta wrap up something else first. Then I'll take a look. 20s with a few minutes latency. :) | 14:02 |
| *** kaisers has quit IRC | 14:02 | |
| dulek | smcginnis: Got it, superpowers have its cooldown time. ;) | 14:02 |
| e0ne | dulek: adding new hack, that introduce new conf parameter which breaks AZ concept only to make deployment easier it not an option for me | 14:03 |
| *** merooney has joined #openstack-cinder | 14:03 | |
| e0ne | dulek: it we need to get instance and volume in different AZs, we need to move validation from API to scheduler | 14:03 |
| e0ne | dulek: but i don't understand such use case | 14:04 |
| *** andreykurilin__ has quit IRC | 14:05 | |
| dulek | e0ne: Actually we don't want to do instance and volume in different AZs. | 14:08 |
| e0ne | dulek: as i understand, it's more a deployment issue: to put c-vol in some AZ | 14:08 |
| dulek | e0ne: We want Cinder to *ignore* AZs. | 14:08 |
| dulek | e0ne: Problem was created when Nova started passing AZ to Cinder when creating a volume. | 14:08 |
| dulek | e0ne: This was commit done back in Juno. Our ops run into problems with it when running internal setup here just a few weeks ago. | 14:09 |
| dulek | e0ne: And since then 2 ML threads about the issue started + bugreport and fix by Edmund. | 14:10 |
| e0ne | dulek: if we need igrore it, imo we should do it in a scheduler and remove validation from the api | 14:11 |
| dulek | e0ne: Ah, we're running Fuel here internally, so fuel 6.1 is affected by the issue. | 14:11 |
| smcginnis | dulek: So not having the full background, if we want Cinder to ignore AZs, should we just stop doing it? What was the reason it was added? | 14:11 |
| e0ne | dulek: i know about this bug in our distro:) | 14:11 |
| dulek | e0ne: Sorry. ;) | 14:12 |
| dulek | smcginnis: We all know purpose of AZs, current concept works well for LVM, where c-vol acts like nova-compute. | 14:12 |
| e0ne | dulek: but don't want to have a hack for it nor in our disto, neither in cinder master | 14:12 |
| smcginnis | dulek: So it's really only relevant in Cinder when using LVM? | 14:13 |
| dulek | smcginnis: Probably no, when we're running multiple c-vols with multiple backends we may have multiple AZs - one per c-vol. | 14:13 |
| dulek | smcginnis: Now there were no problems until this commit: https://git.openstack.org/cgit/openstack/nova/commit/?id=6060888b58db42eea826939852838f9e1c204d2c | 14:14 |
| dulek | smcginnis: Nova started to pass AZ to Cinder (hey, before that nobody cared!). | 14:14 |
| smcginnis | dulek: Just confused by this comment: < dulek> e0ne: We want Cinder to *ignore* AZs. | 14:14 |
| openstackgerrit | dane-fichter proposed openstack/cinder: Data transfer between encrypted volumes and images https://review.openstack.org/216567 | 14:15 |
| e0ne | dulek, smcginnis: why we can't move AZ filtering from API to scheduler like nova does? | 14:16 |
| dulek | smcginnis: Oh, sorry. So what I've meant is that you can have a deployment where you create AZs for VMs. But sometimes you don't want to care about AZs for volumes, because it is done on a different level. | 14:16 |
| smcginnis | dulek: Just seems overall broken in my opinion then. | 14:16 |
| smcginnis | We care except when we don't, but we don't _really_ know when we're supposed to care or not. | 14:17 |
| e0ne | smcginnis: +1 | 14:17 |
| dulek | smcginnis: Well, it's option that administrator sets. | 14:17 |
| dulek | smcginnis: A way of saying - my Ceph have crash map properly configured. | 14:17 |
| smcginnis | dulek: This sets the option to fallback to a different zone. But what if it's one of the cases where we really should enforce it? | 14:18 |
| dulek | smcginnis: It depends on deployment - that's why fallback it's a configuration option. | 14:18 |
| smcginnis | Still doesn't seem right to me. | 14:19 |
| dulek | Getting back to e0ne's question - we cannot skip validation in API and do it in scheduler. | 14:19 |
| dulek | https://github.com/openstack/cinder/blob/master/cinder/volume/flows/api/create_volume.py#L247-L306 | 14:19 |
| dulek | We skip scheduler in some cases. | 14:19 |
| dulek | And we have CONF.cloned_volume_same_az which checks if requested AZ is same as volume being cloned, or snapshot we're creating a volume from. | 14:20 |
| dulek | s/which checks/which enforces a check | 14:21 |
| *** crose has joined #openstack-cinder | 14:21 | |
| dulek | smcginnis: Well as a long term solution it probably isn't. We may stick to how it's done currently and accept that ops will hack Cinder manually. | 14:22 |
| *** zhenguo has joined #openstack-cinder | 14:22 | |
| dulek | smcginnis: That what ops at my company are doing and that's what bloomberg guys are doing also. | 14:22 |
| smcginnis | dulek: I'm a little concerned putting work arounds into the official code. It makes it easier to forget it's just a workaround. | 14:24 |
| smcginnis | dulek: But I see what you're saying. | 14:24 |
| dulek | smcginnis: I'm working on a spec to sort out AZs stuff in Cinder, but that's more work and impossible to be done in L> | 14:24 |
| *** aix has quit IRC | 14:25 | |
| smcginnis | dulek: Do you think it would be ready for M? | 14:25 |
| dulek | smcginnis: That's why I'm looking for a way to fix it now somehow. | 14:25 |
| e0ne | dulek: did you try to move AZ filtering from api to scheduler? how much such changes affect cinder code? | 14:25 |
| *** kvidvans has joined #openstack-cinder | 14:25 | |
| tbarron | eikke: I added a note as followup to yours in https://bugs.launchpad.net/cinder/+bug/1488690 | 14:25 |
| openstack | Launchpad bug 1488690 in Cinder "test_create_backing_from_stream_optimized_file fails with "AttributeError: '_SentinelObject' object has no attribute 'readlines'"" [Undecided,In progress] - Assigned to Jon Bernard (jbernard) | 14:25 |
| dulek | smcginnis, e0ne: Maybe there's better solution? I'm really open to ideas. | 14:25 |
| dulek | e0ne: Okay, let me estimate that... | 14:25 |
| smcginnis | dulek: Sorry, I'm just a critic. :) | 14:26 |
| dulek | smcginnis: Constructive one is always welcomed. :) | 14:26 |
| eikke | tbarron: any cclue why it succeeds when running tox on my local machine? | 14:26 |
| tbarron | eikke: no. How much concurrency do you have, or do you not think that's relevant? | 14:27 |
| eikke | at first I thought it'd be relevant, but I think I was wrong back then | 14:27 |
| smcginnis | dulek: I guess at a minimum I would like that changed from Closes-Bug to Partial-Bug so the actual issue isn't lost sight of. | 14:27 |
| eikke | basically: if 'concurrent test execution' would be in a single process/address space, any mocking would be impossible | 14:27 |
| *** crose has quit IRC | 14:27 | |
| guitarzan | dulek: it's an interesting problem I think | 14:28 |
| eikke | tbarron: hence (and thats also how at least nose does it), tests run in multiple processes, and as such a mock in one testcase doesnt affect others | 14:28 |
| guitarzan | dulek: setting up a cloud that cares about compute AZs but not volume AZs | 14:28 |
| dulek | guitarzan: I think Ceph users does it all the time. They have different mechanisms to achieve resiliency. | 14:28 |
| eikke | tbarron: honestly, I think the question is more 'Why did it work in the past', rather than 'Why is it broken now' | 14:29 |
| guitarzan | dulek: yes, but as DuncanT noted in the mailing list, you still have single points of failure, just not at the actual block device level | 14:29 |
| guitarzan | it is kind of breaking the basic idea of AZs, but I'm not saying I completely disagree with you :) | 14:30 |
| tbarron | eikke: and, as you asked, why does it work fine when run locally? | 14:30 |
| dulek | guitarzan: Have you seen my answer there? Can you comment if I'm reasonable. | 14:30 |
| dulek | guitarzan: I've meant - can you comment if answer is reasonable? ;) | 14:30 |
| guitarzan | dulek: it doesn't really speak to DuncanT's point | 14:30 |
| * smcginnis catches up on ML posts | 14:31 | |
| dulek | guitarzan: So I'm probably mistaken somewhere. Can you elaborate? | 14:31 |
| guitarzan | dulek: oh, I see what you're saying | 14:31 |
| *** stefan_amann has quit IRC | 14:32 | |
| guitarzan | dulek: I agree | 14:32 |
| guitarzan | I read the wrong response :) | 14:33 |
| *** timcl has quit IRC | 14:33 | |
| dulek | Glad to hear that. :) | 14:33 |
| dulek | https://github.com/openstack/cinder/blob/master/cinder/volume/flows/api/create_volume.py#L279-L282 | 14:34 |
| dulek | I'm thinking also if removal of these lines would hurt us much. | 14:34 |
| dulek | In case of new volume creation - validation will be done by scheduler | 14:34 |
| dulek | In case of validation if requested volume AZ matches the AZ of the snapshot the volume is created from - this is checked by lines 286-304. | 14:35 |
| tbarron | eikke: could it be a version-of-mock issue? if my local version fills in readlines automagically but in gerrit we sometimes get a mock that refuses to do that for the SentinelObject? | 14:36 |
| eikke | tbarron: that, I cant eexplain | 14:36 |
| DuncanT | dulek: Fair enough on the control plane if you don't aim for az resiliance at the control plane level, I guess | 14:36 |
| dulek | Ah, there's the difference - user won't get feedback, that he provided wrong AZ. | 14:36 |
| eikke | tbarron: ah, hmh, yeah | 14:36 |
| DuncanT | dulek: Thanks for taking a moment to explain | 14:36 |
| smcginnis | DuncanT: Aren't you supposed to not be working? :) | 14:36 |
| dulek | DuncanT: I do care, but don't think AZs are for that. :) | 14:36 |
| tbarron | eikke: I believe for a while we had a stricter version of mock that broke stuff for similar reasons | 14:37 |
| DuncanT | smcginnis: Well, I'm balancing out some of the time that I should have been working ;-) | 14:37 |
| smcginnis | DuncanT: Hah! | 14:37 |
| guitarzan | DuncanT: haha! | 14:37 |
| dulek | DuncanT: :D | 14:37 |
| eikke | tbarron: I wiped my .tox, so supposedly I should have the same mock version as gate, but still | 14:37 |
| smcginnis | OK, I guess if it's marked as a partial fix with a note in the bug that we need to evaluate the right approach, and you're working on an improved spec, it may be OK. | 14:38 |
| DuncanT | dulek: With a scheduler, db replica, api node and c-vol in every AZ, you can lose any one AZ (of 3, for example) and still be otherwise fully functional.... but not if all your c-vols are on one node | 14:38 |
| *** aix has joined #openstack-cinder | 14:38 | |
| smcginnis | Still don't like then that we would have config options we would need to deprecate and support for a few releases. | 14:38 |
| guitarzan | DuncanT: that's only true if HA cinder works | 14:38 |
| DuncanT | dulek: The cinder-has-no-AZs case still looks broken anyway | 14:38 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Efficient volume copy for generic volume migration https://review.openstack.org/183701 | 14:39 |
| DuncanT | guitarzan: Nah, have 3 c-vols all treating ceph as if it is 3 separate backends | 14:39 |
| guitarzan | there be dragons | 14:39 |
| guitarzan | that's actually interesting though... | 14:39 |
| dulek | DuncanT: You would need 3 different API endpoints for Cinder to achieve that, but yes, it's possible. | 14:40 |
| *** takedakn has quit IRC | 14:40 | |
| *** ronis has quit IRC | 14:40 | |
| dulek | guitarzan: There won't be dragons I think. c-vol's should work in parallel connected to different backends. | 14:40 |
| guitarzan | dulek: yeah, I caught up | 14:41 |
| guitarzan | except the idea of "different backends" is loaded here | 14:41 |
| dulek | DuncanT, guitarzan, smcginnis, e0ne: So what do you think of simple removal of these lines: https://github.com/openstack/cinder/blob/master/cinder/volume/flows/api/create_volume.py#L279-L282 | 14:41 |
| dulek | ? | 14:41 |
| guitarzan | the problem is still that api calls aren't directed to an AZ | 14:42 |
| guitarzan | dulek: would that actually help you? | 14:42 |
| *** julim has quit IRC | 14:43 | |
| DuncanT | guitarzan: An LB will just fail over to whatever is left up in terms of API servers... | 14:43 |
| dulek | guitarzan: Definitely. That's the place where it fails. Then if we're going through scheduler - it will validate the AZ. If not - validation will be done in API few lines lower (if cloned_volume_same_az is True). | 14:43 |
| dulek | The only difference will be that user won't get feedback that the AZ he provided is incorrect. | 14:44 |
| DuncanT | dulek: I'll need to have a think about the consequences of that | 14:44 |
| *** mtanino has joined #openstack-cinder | 14:44 | |
| dulek | DuncanT: One I've just mentioned. Second would be when you have AZFilter disabled - nothing will validate if AZ is correct. But if you disabled it, then you probably don't care. | 14:45 |
| *** kbyrne has quit IRC | 14:45 | |
| *** timcl has joined #openstack-cinder | 14:46 | |
| guitarzan | DuncanT: so you imagine the api sending messages to any rabbit in any AZ? | 14:46 |
| tbarron | eikke: yeah, my theory doesn't hold. I have mock 1.3.0 with requirements mock >= 1.2. And that's exactly what I see with the failure with eharney's fix attempt: http://logs.openstack.org/36/217936/1/check/gate-cinder-python27/2c02e8c/tox/py27-1.log | 14:46 |
| mtanino | smcginnis: Hi, Could you visit this again? https://review.openstack.org/#/c/183701/ I've fixed some nits. | 14:47 |
| tbarron | eikke: much as I don't like skipping tests, I guess this one should be pulled and re-written. | 14:47 |
| dulek | guitarzan: Ah, good point. RabbitMQ need to be common. ;) | 14:47 |
| *** setmason has joined #openstack-cinder | 14:47 | |
| eikke | tbarron: a patch to skip the test is waiting in gate | 14:47 |
| eikke | for good reason | 14:47 |
| eikke | (not the waiting part :P) | 14:48 |
| tbarron | eikke: yeah, but it would be better to fix the problem if we could. | 14:48 |
| *** aix has quit IRC | 14:48 | |
| tbarron | eikke: that said, it has my +1 | 14:48 |
| eharney | unfortunately i can't seem to even hit it locally | 14:48 |
| tbarron | eharney: me either. | 14:49 |
| eikke | maybe a fix could be to mock save_and_reraise_exception :-P | 14:49 |
| eikke | if the exception path is actually to be taken | 14:49 |
| smcginnis | mtanino: Thanks! Just waiting for Jenkins. | 14:49 |
| mtanino | smcginnis: I see. Thank you :) | 14:49 |
| eharney | tbarron: i think i just spotted another problem looking at it, though | 14:53 |
| tbarron | eharney: ? | 14:54 |
| eharney | tbarron: let me try a couple things, will hopefully submit another patch | 14:55 |
| *** annasort has joined #openstack-cinder | 14:55 | |
| *** jungleboyj has joined #openstack-cinder | 14:56 | |
| *** rushiagr_away is now known as rushiagr | 14:57 | |
| *** ebalduf has joined #openstack-cinder | 14:57 | |
| *** kbyrne has joined #openstack-cinder | 14:57 | |
| openstackgerrit | XinXiaohui proposed openstack/cinder: Calculate virtual free capacity and notify https://review.openstack.org/206923 | 14:58 |
| *** aix has joined #openstack-cinder | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| openstackgerrit | Takeaki Matsumoto proposed openstack/cinder: Fix url in API response to get original https://review.openstack.org/216591 | 15:01 |
| *** haomaiwang has joined #openstack-cinder | 15:02 | |
| DuncanT | guitarzan: Rabbit would be a cluster, and normally things would work pan-az, just means you can loose a whole AZ and not loose access to the other volumes | 15:02 |
| *** julim has joined #openstack-cinder | 15:02 | |
| *** skraynev has quit IRC | 15:03 | |
| *** mtanino has quit IRC | 15:03 | |
| guitarzan | DuncanT: yeah, I run clustered rabbit. I'm just trying to find the mental block in my brain :) | 15:04 |
| DuncanT | guitarzan: I tried running a clustered brain but the rest of it voted me out of the cluster | 15:05 |
| *** skraynev has joined #openstack-cinder | 15:05 | |
| guitarzan | I think my brain isn't liking the idea of a volume that isn't actually offline | 15:05 |
| *** hemnafk is now known as hemna | 15:06 | |
| guitarzan | but this idea seems pretty good I think | 15:06 |
| *** garthb has quit IRC | 15:06 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Fix MITM vulnerability for Brocade FC SAN lookup https://review.openstack.org/218341 | 15:13 |
| *** boris-42 has joined #openstack-cinder | 15:13 | |
| *** zhenguo has quit IRC | 15:14 | |
| Swanson | Got a really minor change that could impact some random CI failures. I'd like to have it running over the weekend. 1 +2 from hemna so it just needs another push. https://review.openstack.org/#/c/216859/ | 15:19 |
| *** edmondsw has quit IRC | 15:19 | |
| *** edmondsw has joined #openstack-cinder | 15:20 | |
| openstackgerrit | John Griffith proposed openstack/cinder: get_replication_updates call to driver is wrong https://review.openstack.org/218348 | 15:20 |
| *** jdurgin1 has joined #openstack-cinder | 15:21 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Fix mock/sentinel use in vmdk tests https://review.openstack.org/217936 | 15:23 |
| *** jaypipes has joined #openstack-cinder | 15:23 | |
| *** jaypipes is now known as leakypipes | 15:24 | |
| *** merooney has quit IRC | 15:24 | |
| *** edtubill has joined #openstack-cinder | 15:25 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Add new Connector APIs for path validation https://review.openstack.org/199764 | 15:30 |
| *** JoeStack has joined #openstack-cinder | 15:31 | |
| *** skraynev has quit IRC | 15:32 | |
| *** skylerberg has joined #openstack-cinder | 15:33 | |
| *** skraynev has joined #openstack-cinder | 15:33 | |
| openstackgerrit | dane-fichter proposed openstack/cinder: Data transfer between encrypted volumes and images https://review.openstack.org/216567 | 15:35 |
| *** timcl has quit IRC | 15:36 | |
| dulek | DuncanT: Just read joke on clustered-brain. Good one! ;) | 15:36 |
| sdague | is anyone addressing the cinder vmware unit tests being racey - http://logs.openstack.org/10/200610/16/gate/gate-cinder-python27/44155be/console.html#_2015-08-28_15_24_11_831 that's causing a bunch of gate resets? | 15:37 |
| smcginnis | sdague: Yep, patch to skip going through soon. | 15:39 |
| smcginnis | Hopefully. | 15:39 |
| sdague | smcginnis: is it posted already? | 15:39 |
| sdague | we can put that into the promote list | 15:39 |
| smcginnis | sdague: There was some discussion earlier this morning. Let me track it down. tbarron, do you have that handy? | 15:39 |
| *** chlong has quit IRC | 15:39 | |
| *** skraynev has quit IRC | 15:40 | |
| *** bluex-pl has quit IRC | 15:40 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 15:40 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Sync volume versionedobject to ORM https://review.openstack.org/218361 | 15:40 |
| Swanson | eharney: Thanks! | 15:41 |
| *** p0rtal has joined #openstack-cinder | 15:41 | |
| *** p0rtal has quit IRC | 15:41 | |
| smcginnis | sdague: I believe this is it: https://review.openstack.org/#/c/217882/ | 15:41 |
| *** p0rtal has joined #openstack-cinder | 15:41 | |
| smcginnis | Way down in the gate queue. | 15:42 |
| sdague | smcginnis: thanks | 15:42 |
| smcginnis | sdague: np | 15:43 |
| *** mtanino has joined #openstack-cinder | 15:46 | |
| openstackgerrit | Takeaki Matsumoto proposed openstack/cinder: Fix url in API response to get original https://review.openstack.org/216591 | 15:47 |
| *** ronis has joined #openstack-cinder | 15:47 | |
| *** markstur_ has quit IRC | 15:47 | |
| *** yuriy_n17 has quit IRC | 15:49 | |
| *** skraynev has joined #openstack-cinder | 15:50 | |
| *** rushiagr is now known as rushiagr_away | 15:51 | |
| *** p0rtal has quit IRC | 15:51 | |
| *** ircuser-1 has quit IRC | 15:52 | |
| *** markstur has joined #openstack-cinder | 15:54 | |
| *** kvidvans has quit IRC | 15:55 | |
| *** haomaiwang has quit IRC | 15:58 | |
| *** haomaiwang has joined #openstack-cinder | 15:58 | |
| *** p0rtal has joined #openstack-cinder | 15:59 | |
| *** p0rtal has quit IRC | 16:00 | |
| *** p0rtal has joined #openstack-cinder | 16:00 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** jordanP has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-cinder | 16:02 | |
| *** p0rtal has quit IRC | 16:03 | |
| hemna | angela-s, https://review.openstack.org/#/c/196640/ | 16:03 |
| *** sghanekar has joined #openstack-cinder | 16:03 | |
| *** p0rtal has joined #openstack-cinder | 16:03 | |
| *** jdurgin1 has quit IRC | 16:07 | |
| *** skraynev has quit IRC | 16:10 | |
| *** leakypipes has quit IRC | 16:11 | |
| *** vilobhmm1 has joined #openstack-cinder | 16:14 | |
| *** rhedlind has joined #openstack-cinder | 16:15 | |
| *** dannywilson has joined #openstack-cinder | 16:17 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Fix MITM vulnerability for Brocade FC SAN lookup https://review.openstack.org/218341 | 16:17 |
| *** _cjones_ has joined #openstack-cinder | 16:19 | |
| *** 18VAAKDGR is now known as lpetrut1 | 16:20 | |
| *** leeantho has joined #openstack-cinder | 16:20 | |
| hemna | looks like the Mellanox CI is totally broken | 16:20 |
| *** agarciam has quit IRC | 16:20 | |
| *** merooney has joined #openstack-cinder | 16:21 | |
| *** merooney has quit IRC | 16:21 | |
| *** chlong has joined #openstack-cinder | 16:22 | |
| *** merooney has joined #openstack-cinder | 16:22 | |
| *** skraynev has joined #openstack-cinder | 16:23 | |
| *** garthb has joined #openstack-cinder | 16:23 | |
| *** agarciam has joined #openstack-cinder | 16:24 | |
| guitarzan | hemna: quick question, how do I know what interface nova and/or brick are going to use for iscsi? | 16:25 |
| guitarzan | hemna: hmm, actually this might be a matter of what my connection hash looks like... I'll check :) | 16:26 |
| *** timcl has joined #openstack-cinder | 16:26 | |
| hemna | the outbound interface ? | 16:26 |
| hemna | I presume that's a routing table issue no ? | 16:26 |
| *** e0ne has quit IRC | 16:28 | |
| *** panbalag has joined #openstack-cinder | 16:28 | |
| guitarzan | hemna: ya, i think so, I just need to figure out which of devstack's interfaces to use on the iscsi target side | 16:29 |
| *** annegentle has quit IRC | 16:31 | |
| Swanson | If your job fails in gate you should have to mail donuts to everyone behind you in gate at the time. | 16:32 |
| angela-s | hemna: i retriggered the CI job yesterday, still waiting in the queue, quite a backlog. the last CI run for this was before we fixed the format of the comment message, so that's why it doesn't show up in the right hand side | 16:33 |
| *** vilobhmm1 has quit IRC | 16:33 | |
| *** dflorea has joined #openstack-cinder | 16:34 | |
| panbalag | Hi.. can someone let me know the command to do a cinder volume retype? | 16:35 |
| panbalag | I don't see a 'cinder retype' in the command list. | 16:35 |
| *** jistr has quit IRC | 16:35 | |
| *** salv-orlando has quit IRC | 16:36 | |
| jgriffith | panbalag: https://gist.github.com/5bfb6d1e094883fd668b | 16:36 |
| *** esker has joined #openstack-cinder | 16:37 | |
| jgriffith | panbalag: maybe check "cinder --version" ? perhaps your on Havanna ? | 16:37 |
| panbalag | jgriffith: I'm on 1.2.1 .. | 16:38 |
| *** mriedem is now known as mriedem_lunch | 16:38 | |
| jgriffith | panbalag: then it should be there, unless it's disabled in policy file | 16:38 |
| *** esker has quit IRC | 16:38 | |
| Swanson | Did they recently disable it? Cause it is showing up as not a valid subcommand when I do a cinder help retype | 16:39 |
| jgriffith | panbalag: or if we borked something... I'm on 1.1.1 | 16:39 |
| Swanson | 1.3.1 | 16:39 |
| panbalag | jgriffith: when I try "cinder help retype'" I get the error " not a valid subcommand" ... | 16:39 |
| *** esker has joined #openstack-cinder | 16:39 | |
| jgriffith | geesh. | 16:39 |
| panbalag | jgriffith: how do I enable it ? | 16:39 |
| *** annegentle has joined #openstack-cinder | 16:40 | |
| jgriffith | panbalag: not sure, it is in fact broken, looking now | 16:40 |
| openstackgerrit | Jay Bryant proposed openstack/python-cinderclient: Volume status management for volume migration https://review.openstack.org/189547 | 16:40 |
| *** earlephilhower has joined #openstack-cinder | 16:40 | |
| earlephilhower | Is there a way to get the list of Jenkins merges w/timestamps for the entire OpenStack github? Something changed 28hrs ago and it's causing 100% failure rate on my CI now. :( | 16:43 |
| *** setmason_ has joined #openstack-cinder | 16:43 | |
| jgriffith | panbalag: Swanson cinder --os-volume-api-version 2 retype | 16:44 |
| *** jwcroppe has joined #openstack-cinder | 16:44 | |
| * jgriffith honestly thought that the whole v1-v2 train wreck had been fixed :( | 16:44 | |
| panbalag | jgriffith: That worked..Thanks | 16:44 |
| *** chlong has quit IRC | 16:45 | |
| *** setmason has quit IRC | 16:45 | |
| *** setmason_ is now known as setmason | 16:45 | |
| jgriffith | Why the F are we setting V1 as default again ? | 16:45 |
| Swanson | jgriffith: Hmmmm... I thought v2 was the default | 16:45 |
| jgriffith | Swanson: me too :) | 16:45 |
| jgriffith | Swanson: it appears we're messed up in devstack deploy again though | 16:46 |
| *** agarciam has quit IRC | 16:46 | |
| jgriffith | Swanson: honestly this whole thing has been a nightmare, between the keystone changes and the fact that we haven't been pushing cinderclient and testing it well | 16:46 |
| *** agarciam has joined #openstack-cinder | 16:46 | |
| jgriffith | Swanson: I have something to look at during bug fix mode :) | 16:46 |
| Swanson | jgriffith: :) And here I was just worrying about seeing an openstack-dev/grenade change in gate ahead of me. This should give me something else to be concerned about. | 16:47 |
| jgriffith | LOL | 16:47 |
| tbarron | smcginnis: sdague: sorry, was at lunch. As smcginnis said: https://review.openstack.org/#/c/217882/ | 16:48 |
| smcginnis | tbarron: Hah, no worries. Just knew you were part of that discussion so I thought I would ping you while I looked in case you still had it handy. | 16:48 |
| sdague | tbarron: yeh, smcginnis hooked me up. We promoted that in the gate queue to hopefully reduce overall failure rate | 16:50 |
| *** p0rtal has quit IRC | 16:51 | |
| tbarron | smcginnis: sdague: excellent, and thanks. | 16:51 |
| *** Apoorva has joined #openstack-cinder | 16:51 | |
| *** gouthamr has quit IRC | 16:52 | |
| *** Apoorva has quit IRC | 16:52 | |
| eikke | jay, gate succss for the py27 run of the vmdk skip patch | 16:52 |
| *** agarciam has quit IRC | 16:53 | |
| *** Apoorva has joined #openstack-cinder | 16:53 | |
| *** agarciam has joined #openstack-cinder | 16:53 | |
| jgriffith | eikke: ? | 16:53 |
| eikke | jgriffith: just that gate-cinder-python27 passed for 2178824, hopefully other jobs soon as well | 16:54 |
| jgriffith | eikke: ahh... got ya :) | 16:54 |
| jgriffith | Isn't this like the second or third time the vmware unit tests have hosed us? | 16:54 |
| jgriffith | eharney: ^^ | 16:54 |
| *** subhadeep_de has quit IRC | 16:57 | |
| *** mudassirlatif has joined #openstack-cinder | 16:57 | |
| *** timcl has quit IRC | 16:59 | |
| *** JoeStack has quit IRC | 17:00 | |
| *** mtanino has quit IRC | 17:00 | |
| *** haomaiwang has quit IRC | 17:01 | |
| smcginnis | jgriffith: I'll beat them up next week when I'm out there. ;) | 17:01 |
| raildo | hey guys, quickly question... I saw this documentation about cinder client explaning how to use the quota operations http://docs.openstack.org/user-guide-admin/cli_cinder_quotas.html | 17:01 |
| jgriffith | smcginnis: haha | 17:02 |
| raildo | anyone know how can I do this in the openstack client? | 17:02 |
| *** haomaiwang has joined #openstack-cinder | 17:02 | |
| raildo | is this already implemented in openstack client? I only find the quota commands for nova, and not for cinder =/ | 17:03 |
| hemna | angela-s, ok I'll wait then and watch. | 17:05 |
| *** julim has quit IRC | 17:06 | |
| smcginnis | raildo: You're talking python-openstackclient, not python-cinderclient, right? | 17:06 |
| raildo | smcginnis: python-openstackclient | 17:06 |
| smcginnis | raildo: I don't have much experience with that one. | 17:06 |
| raildo | smcginnis: btw I found this code https://github.com/openstack/python-openstackclient/blob/d87403a7bf868d96c6c3185a59ac9bac4ba264fa/openstackclient/common/quota.py#L166 | 17:06 |
| smcginnis | raildo: It's possible it hasn;t been added there yet. | 17:07 |
| jgriffith | raildo: it looks like it might be in master... looking now | 17:07 |
| *** setmason has quit IRC | 17:09 | |
| *** julim has joined #openstack-cinder | 17:09 | |
| *** setmason has joined #openstack-cinder | 17:10 | |
| *** erhudy has joined #openstack-cinder | 17:10 | |
| angela-s | hemna: thanks, or i'll ping you once it runs. | 17:11 |
| hemna | angela-s, ok thanks for letting me know. | 17:11 |
| raildo | smcginnis: jgriffith hum.. I think that the 'opoenstack quota <command>' works for cinder too: https://github.com/openstack/python-openstackclient/blob/d87403a7bf868d96c6c3185a59ac9bac4ba264fa/openstackclient/common/quota.py | 17:11 |
| ericksonsantos | raildo, I think that is here that openstackclient gets the quota for volume: https://github.com/openstack/python-openstackclient/blob/d87403a7bf868d96c6c3185a59ac9bac4ba264fa/openstackclient/common/quota.py#L202-L206 | 17:11 |
| *** salv-orlando has joined #openstack-cinder | 17:12 | |
| raildo | ericksonsantos: yes! let's do test this :) | 17:12 |
| ericksonsantos | raildo, sure | 17:12 |
| *** vilobhmm1 has joined #openstack-cinder | 17:13 | |
| vilobhmm1 | jungleboyj, smcginnis : thanks ! :) | 17:14 |
| smcginnis | vilobhmm1: np | 17:15 |
| smcginnis | raildo, ericksonsantos: Looks right! | 17:15 |
| *** timcl has joined #openstack-cinder | 17:15 | |
| *** jlk has left #openstack-cinder | 17:16 | |
| *** e0ne has joined #openstack-cinder | 17:18 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 17:18 |
| ericksonsantos | smcginnis, nice! | 17:18 |
| *** lpetrut1 has quit IRC | 17:19 | |
| *** subhadeep_de has joined #openstack-cinder | 17:19 | |
| openstackgerrit | Anna Sortland proposed openstack/cinder: Earlier authority check for create volume API https://review.openstack.org/217899 | 17:20 |
| *** dflorea has quit IRC | 17:20 | |
| *** aix has quit IRC | 17:20 | |
| *** dflorea has joined #openstack-cinder | 17:25 | |
| *** dflorea has quit IRC | 17:30 | |
| *** dflorea has joined #openstack-cinder | 17:31 | |
| *** dflorea has quit IRC | 17:32 | |
| *** Yogi1 has quit IRC | 17:33 | |
| *** changbl has quit IRC | 17:33 | |
| *** e0ne has quit IRC | 17:33 | |
| *** dflorea has joined #openstack-cinder | 17:35 | |
| jungleboyj | vilobhmm1: Welcome! | 17:35 |
| smcginnis | Cool, later. | 17:36 |
| smcginnis | Oops. :-D | 17:36 |
| *** ronis has quit IRC | 17:36 | |
| raildo | lol | 17:36 |
| smcginnis | Split screen is going to kill me. | 17:36 |
| raildo | smcginnis: I have this same problem, some times I cam here and put "vim" | 17:37 |
| smcginnis | raildo: Yeah, that too. :) | 17:37 |
| *** mriedem_lunch is now known as mriedem | 17:40 | |
| *** jungleboyj has quit IRC | 17:41 | |
| *** anshul has quit IRC | 17:42 | |
| *** vgridnev has joined #openstack-cinder | 17:43 | |
| *** merooney has quit IRC | 17:44 | |
| smcginnis | Anyone else getting this on fresh devstack deployments: functions-common:466 Cloning not allowed in this configuration | 17:45 |
| *** jaypipes has joined #openstack-cinder | 17:46 | |
| *** merooney has joined #openstack-cinder | 17:47 | |
| *** p0rtal has joined #openstack-cinder | 17:48 | |
| *** ebalduf has quit IRC | 17:49 | |
| *** ebalduf has joined #openstack-cinder | 17:50 | |
| *** Yogi1 has joined #openstack-cinder | 17:50 | |
| *** jwcroppe has quit IRC | 17:52 | |
| sdague | smcginnis: so the cinder unit test fix died on a different cinder unit test race - https://jenkins04.openstack.org/job/gate-cinder-python27/1436/console | 17:52 |
| sdague | I'm about to call it a week, but that should get investigated | 17:52 |
| *** mudassirlatif_ has joined #openstack-cinder | 17:54 | |
| *** e0ne has joined #openstack-cinder | 17:55 | |
| smcginnis | sdague: Thanks for the heads up. We'll look in to it. | 17:56 |
| smcginnis | sdague: Have a good one. | 17:56 |
| *** mudassirlatif has quit IRC | 17:57 | |
| *** mudassirlatif_ is now known as mudassirlatif | 17:57 | |
| *** ronis has joined #openstack-cinder | 17:57 | |
| smcginnis | Hmm, haven't seen that one before" | 17:59 |
| smcginnis | oslo_db.exception.DBConnectionError: (psycopg2.OperationalError) unknown error | 17:59 |
| *** haomaiwang has quit IRC | 18:01 | |
| *** esker has quit IRC | 18:01 | |
| *** 20WABZC7L has joined #openstack-cinder | 18:02 | |
| tbarron | smcginnis: sdague: so can we do a recheck on 217882 right now and get it pushed to the head of queue again? | 18:02 |
| smcginnis | tbarron: That's what I was just looking in to. :) | 18:03 |
| tbarron | smcginnis: excellent | 18:03 |
| smcginnis | tbarron: Doesn't look like it. :[ | 18:04 |
| tbarron | smcginnis: how did you find that answer? ask on infra? | 18:04 |
| smcginnis | tbarron: I'm a guy, I don't ask directions. | 18:04 |
| smcginnis | ;) | 18:04 |
| smcginnis | tbarron: Just reading this: http://docs.openstack.org/infra/manual/core.html | 18:05 |
| smcginnis | tbarron: And trying things. | 18:05 |
| tbarron | smcginnis: but what did sdague do last time? my concern is that this one is sitting behind a bunch or rechecked jobs that are just going to fail again. | 18:06 |
| smcginnis | tbarron: OK, it's still in the gate queue, but I was able to also kick it off in the check queue. We'll see if that will do it. | 18:06 |
| smcginnis | tbarron: Yeah, once this makes it back to the gate queue we should see if someone in infra can do whatever he did to bump it up. | 18:07 |
| *** shyama has joined #openstack-cinder | 18:10 | |
| Swanson | Gate appears to be running about 24 hours today. | 18:10 |
| * tbarron watche #openstack-infra with admiration | 18:10 | |
| *** timcl has quit IRC | 18:13 | |
| *** kvidvans has joined #openstack-cinder | 18:13 | |
| tbarron | smcginnis: ty! | 18:13 |
| smcginnis | tbarron: Thank you for the suggestion! | 18:14 |
| smcginnis | tbarron: And chiming in over there. | 18:14 |
| *** aix has joined #openstack-cinder | 18:14 | |
| smcginnis | Now we know for the future if something like this happens again. | 18:15 |
| smcginnis | Infra to the rescue. | 18:15 |
| *** lprice1 has quit IRC | 18:16 | |
| hemna | jobs launched just took a nosedive | 18:16 |
| smcginnis | Oh joy. | 18:16 |
| *** dflorea has quit IRC | 18:16 | |
| tbarron | yeah, we should only bug like that infrequently, but this one got workflow yesterday at 6:07 pm ET, tests are failing because of it quite frequently, and people are issuing rechecks on those failures which just fail all over again. | 18:17 |
| *** dflorea has joined #openstack-cinder | 18:17 | |
| smcginnis | Yeah, definitely only when really needed. | 18:18 |
| eikke | gate 2.7 failed on 217882 :( | 18:19 |
| *** shyama has quit IRC | 18:20 | |
| *** mtanino has joined #openstack-cinder | 18:23 | |
| hemna | boo! | 18:25 |
| smcginnis | eikke: Getting requed | 18:25 |
| hemna | can we spike the football with another +A ? | 18:25 |
| hemna | :P | 18:25 |
| smcginnis | hemna: Already talked to infra and they are going to get it going again once the current run fails. | 18:26 |
| smcginnis | Unforatunately that looks like it will be at least 30 minutes out. | 18:26 |
| hemna | arg | 18:26 |
| smcginnis | kwarg | 18:27 |
| tbarron | lprice: all we can do right now is check if you are failing for some reason other than https://bugs.launchpad.net/cinder/+bug/1488690 | 18:31 |
| openstack | Launchpad bug 1488690 in Cinder "test_create_backing_from_stream_optimized_file fails with "AttributeError: '_SentinelObject' object has no attribute 'readlines'"" [Undecided,In progress] - Assigned to Jon Bernard (jbernard) | 18:31 |
| tbarron | sorry, wrong window | 18:32 |
| eikke | I'm still puzzled what causes that to be triggered so consistently now | 18:32 |
| *** p0rtal_ has joined #openstack-cinder | 18:35 | |
| *** dflorea has quit IRC | 18:37 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Remove executable bits on files https://review.openstack.org/218407 | 18:38 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Detect addition of executable files https://review.openstack.org/218408 | 18:38 |
| *** p0rtal has quit IRC | 18:38 | |
| *** jaypipes has quit IRC | 18:40 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder-specs: Fix Get Volume Driver Capabilities Spec https://review.openstack.org/216394 | 18:41 |
| *** skraynev has quit IRC | 18:41 | |
| *** e0ne has quit IRC | 18:42 | |
| *** skraynev has joined #openstack-cinder | 18:42 | |
| smcginnis | tbarron, hemna, eikke: Been requeued in the gate queue at the top of the heap! | 18:43 |
| tbarron | smcginnis: :-) | 18:44 |
| Swanson | That poor nova change. It is going to go over 24 hours in the queue. | 18:44 |
| hemna | top of the chain baby | 18:45 |
| *** mudassirlatif has quit IRC | 18:47 | |
| *** xyang1 has joined #openstack-cinder | 18:52 | |
| *** mudassirlatif has joined #openstack-cinder | 18:55 | |
| *** hodos has joined #openstack-cinder | 18:55 | |
| smcginnis | py27 run has passed this time. Don't want to jinx it, but things are looking good. | 18:56 |
| *** timcl has joined #openstack-cinder | 18:56 | |
| hemna | 27 and 34 passed | 18:57 |
| eikke | yeah! | 18:58 |
| patrickeast | yay! | 18:59 |
| *** rady has joined #openstack-cinder | 19:00 | |
| * tbarron is looking the other way, fingers crossed | 19:00 | |
| *** 20WABZC7L has quit IRC | 19:01 | |
| *** haomaiwang has joined #openstack-cinder | 19:02 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Fix a merge problem in VMAX driver https://review.openstack.org/218421 | 19:06 |
| *** dannywilson has quit IRC | 19:09 | |
| smcginnis | xyang1: ping | 19:11 |
| xyang1 | smcginnis: Hi | 19:11 |
| smcginnis | xyang1: Hey! I saw you signed up (or were signed up) for this patch: https://review.openstack.org/#/c/186312/ | 19:11 |
| smcginnis | xyang1: Jay and I have +2'd it, but I didn't want to +A until I talked to you. | 19:12 |
| *** tpsilva has quit IRC | 19:12 | |
| smcginnis | xyang1: Just wanted to see if you had any input on it first or if you wanted to review it yet. | 19:12 |
| xyang1 | smcginnis: I have not looked at the latest. I can take a look | 19:13 |
| smcginnis | xyang1: Cool, thanks! | 19:14 |
| xyang1 | smcginnis: Np | 19:14 |
| *** dannywilson has joined #openstack-cinder | 19:17 | |
| vgridnev | hey cinder folks, could you please review the change: https://review.openstack.org/#/c/214194/ . It's required for the future BDD CI | 19:18 |
| *** Lee1092 has quit IRC | 19:23 | |
| *** Apoorva_ has joined #openstack-cinder | 19:25 | |
| *** Apoorva has quit IRC | 19:25 | |
| *** hodos|2 has joined #openstack-cinder | 19:28 | |
| *** hodos has quit IRC | 19:31 | |
| *** changbl has joined #openstack-cinder | 19:37 | |
| *** devlaps has joined #openstack-cinder | 19:37 | |
| *** IanGovett has quit IRC | 19:39 | |
| *** Apoorva_ has quit IRC | 19:41 | |
| smcginnis | tbarron, hemna, eikke: It made it! | 19:41 |
| *** rushil has joined #openstack-cinder | 19:42 | |
| *** eharney has quit IRC | 19:43 | |
| *** thangp has quit IRC | 19:46 | |
| *** Apoorva has joined #openstack-cinder | 19:47 | |
| *** bill_az has quit IRC | 19:51 | |
| *** Apoorva has quit IRC | 19:51 | |
| panbalag | jgriffith: do you know anyone who works on cinder retype? I'm receiving an exception when trying to retype.. the retype is successful, but the source volume is not deleted..I see an exception in volume.log..so wondering if anyone here who could help | 19:51 |
| jgriffith | panbalag: so you're doing a retype with migration? Migration doesn't work | 19:52 |
| Swanson | It doesn't? | 19:52 |
| jgriffith | panbalag: or depending on who you ask "it's just misunderstood" | 19:52 |
| panbalag | jgriffith: I used "cinder --os-volume-api-version 2 retype --migration-policy on-demand vol2 nfs" | 19:53 |
| jgriffith | panbalag: Swanson here's a few patches trying to fix things: https://review.openstack.org/#/q/owner:%22Vincent+Hou%22+status:open,n,z | 19:54 |
| *** rushil has quit IRC | 19:54 | |
| *** rushil has joined #openstack-cinder | 19:54 | |
| jgriffith | panbalag: You might want to check with vincent; I've sort of washed my hands of that at this point and tell most people "just don't do it" | 19:55 |
| jgriffith | panbalag: we'll likely get it fixed up for this release, but currently it's sort of wonkie | 19:55 |
| Swanson | jgriffith: poor vincent. I swear to god it worked at one point. | 19:55 |
| jgriffith | Swanson: well... it "does" sometimes | 19:55 |
| jgriffith | Swanson: panbalag but it has all sorts of issues with cleanup and error handling | 19:56 |
| *** lpetrut has joined #openstack-cinder | 19:56 | |
| jgriffith | Swanson: panbalag and the bigger problem is, it "says" it worked... but look on your backend and make sure | 19:56 |
| panbalag | jgriffith: the exception I see happens when the source volume (attached to an instance) is being deleted.. http://paste.openstack.org/show/431284/ | 19:56 |
| panbalag | jgriffith: I only verified the 'cinder list' output when I said it was successful | 19:57 |
| jgriffith | panbalag: interesting... | 19:57 |
| Swanson | jgriffith: Thanks. I know we worked our way through some failures at one point. Looks like it kept being fixed after that. | 19:57 |
| jgriffith | Endpoint Not Found... that's a new one | 19:57 |
| * jgriffith hasn't seen that one before | 19:57 | |
| panbalag | jgriffith: is there any configuration that needs to be enabled specifically for retype to work? wondering if I'm missing any configs | 19:58 |
| panbalag | jgriffith: ok. I checked the backend - it didn't get through.. | 20:00 |
| *** haomaiwang has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-cinder | 20:02 | |
| jgriffith | panbalag: :( Yeah, that's been my biggest complaint since day one | 20:03 |
| jgriffith | panbalag: it'll silently fail... swap all the info in the db and say it's "good" | 20:03 |
| jgriffith | panbalag: and you don't know until you actually try and use the volume that things didn't work | 20:03 |
| tbarron | smcginnis: hurray! | 20:04 |
| *** dflorea has joined #openstack-cinder | 20:07 | |
| *** merooney has quit IRC | 20:08 | |
| *** dflorea has quit IRC | 20:09 | |
| *** dflorea has joined #openstack-cinder | 20:09 | |
| patrickeast | jgriffith: hey, did you ever get the image cache working with your backend? i just realized i never really followed up on your review (aside from fixing the permissions stuff) | 20:10 |
| jgriffith | patrickeast: I did not :( | 20:10 |
| jgriffith | patrickeast: continued to run into the glance permissions problem. | 20:11 |
| jgriffith | Somebody else tried it and saw the same thing IIRC | 20:11 |
| patrickeast | jgriffith: was that after i removed all the checks for permissions? | 20:11 |
| jgriffith | patrickeast: errr... probably not :) | 20:12 |
| jgriffith | patrickeast: but, it was the glance call that was failing | 20:12 |
| jgriffith | patrickeast: not an op in the cinder code | 20:12 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Adding a tool that allows querying of LUNs https://review.openstack.org/204191 | 20:12 |
| jgriffith | patrickeast: the actual glance client call was rejected | 20:12 |
| patrickeast | jgriffith: oh really? | 20:13 |
| *** rushil has quit IRC | 20:13 | |
| jgriffith | patrickeast: yeah | 20:13 |
| patrickeast | jgriffith: the one i see on the review comment is "INFO cinder.image.cache [req-ee548891-ecd4-4c3c-b6c2-0bcfe2f2ef90 None] Unable to use ImageVolumeCache because image 5784e30f-64ee-4481-ab82-02eea526716a is not accessible by internal tenant." | 20:13 |
| patrickeast | which is from cinder | 20:13 |
| patrickeast | thats a check i added into the cache code | 20:13 |
| jgriffith | patrickeast: I'll load it up and check it out again | 20:13 |
| *** Yogi1 has quit IRC | 20:13 | |
| patrickeast | jgriffith: that would be awesome... definitely want to fix that if there is still an issue | 20:13 |
| jgriffith | patrickeast: but pretty sure I hacked some stuff out and got an error from the glance call itself | 20:14 |
| *** Apoorva has joined #openstack-cinder | 20:14 | |
| jgriffith | patrickeast: sure, I'll have a look at it | 20:14 |
| openstackgerrit | Jon Bernard proposed openstack/cinder: Add support for file I/O volume migration https://review.openstack.org/187270 | 20:14 |
| jgriffith | patrickeast: BTW... getting configs into the driver on init is proving to be a PITA | 20:14 |
| patrickeast | jgriffith: for replication? | 20:14 |
| jgriffith | patrickeast: because it turns out we parse the config file for the driver on driver-init | 20:14 |
| jgriffith | patrickeast: yeah | 20:14 |
| patrickeast | jgriffith: haha yea | 20:15 |
| patrickeast | jgriffith: we've pretty much decided we will only support unmanaged devices | 20:15 |
| jgriffith | patrickeast: which means the only way I can really do this is wait til *all* volume managers are up and initialized, and then issue a call | 20:15 |
| jgriffith | patrickeast: hehe... yeah, I'm kind of on the opposite side of that one | 20:15 |
| jgriffith | patrickeast: too much change for all the associated calls fro this stage of the release cycle for my liking | 20:16 |
| patrickeast | jgriffith: oh no, sorry, didn't mean to change the apis | 20:16 |
| jgriffith | patrickeast: adding an endpoint is easy enough but making sure I catch every single call that might have it switched sucks | 20:16 |
| patrickeast | jgriffith: just that our driver would ignore replication settings if they werent specified as unmanaged devices | 20:16 |
| jgriffith | patrickeast: ? | 20:16 |
| patrickeast | jgriffith: because parsing the managed ones is proving troublesome | 20:16 |
| jgriffith | patrickeast: oh.. yeah, I knew what you meant | 20:16 |
| jgriffith | yeah... totally cool | 20:16 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: WIP: Set executor to Putils https://review.openstack.org/206549 | 20:17 |
| jgriffith | patrickeast: I think I might add an admin API like "update_replication_settings" and pass it in that way... assuming nobody disagrees? | 20:17 |
| * jgriffith is looking at patrickeast and dannywilson | 20:17 | |
| jgriffith | :) | 20:17 |
| jgriffith | patrickeast: or init_managed_replication | 20:18 |
| jgriffith | thoughts? | 20:18 |
| patrickeast | jgriffith: so what i don't really understand is why we can't just get to the config file contents directly in the driver code... i guess i don't really understand enough about how oslo_config works, but it seems like a wrapper around the python config file parsing | 20:18 |
| jgriffith | patrickeast: yeah... so in theory you could do "CONF.backend-name.option" | 20:18 |
| patrickeast | jgriffith: ehh my only hesitation is that its kind of late in the game for that isn't it? | 20:18 |
| jgriffith | BUUUUT | 20:18 |
| jgriffith | we don't actually parse the config file and fill that in until the startup | 20:19 |
| jgriffith | patrickeast: LOL | 20:19 |
| jgriffith | patrickeast: Well it was my argument two days ago that it was too late in the game to implement it and you disagreed | 20:19 |
| jgriffith | :) | 20:19 |
| *** merooney has joined #openstack-cinder | 20:19 | |
| *** akerr has quit IRC | 20:19 | |
| jgriffith | patrickeast: besides... it's not like it will impact anybody :) | 20:20 |
| *** bill_az has joined #openstack-cinder | 20:20 | |
| *** merooney has quit IRC | 20:20 | |
| patrickeast | jgriffith: not exactly true... anyway, my point is that even if you put the review up today for it do you really think it would get merged by the freeze date? | 20:20 |
| patrickeast | jgriffith: like driver implementations aside | 20:20 |
| jgriffith | patrickeast: what's not true? | 20:20 |
| *** panbalag has quit IRC | 20:21 | |
| * jgriffith isn't following | 20:21 | |
| patrickeast | jgriffith: a few things i guess, it will impact folks... like you and me writing the implementations, right? the core reviewers who are presumably swamped with the like 30 things still on the list for L3, etc | 20:22 |
| patrickeast | jgriffith: what i meant by too late in the game though is that the feature freeze is like monday, right? | 20:22 |
| patrickeast | jgriffith: adding a new api would have to merge in record time | 20:22 |
| patrickeast | jgriffith: thats all | 20:22 |
| jgriffith | patrickeast: oh... yeah | 20:22 |
| jgriffith | patrickeast: meh... I suppose, it's a single call but yeah... maybe not even worth considering | 20:23 |
| patrickeast | jgriffith: don't get me wrong, i like the idea of having that api | 20:23 |
| patrickeast | jgriffith: definitely not advocating that we shouldn't have that kind of ability | 20:23 |
| jgriffith | patrickeast: my point about impacting anybody was you apparantly have a driver that doesn't need it, so you wouldn't use it anyway | 20:23 |
| patrickeast | jgriffith: well... we would use it if we could | 20:23 |
| jgriffith | patrickeast: yeah... who knows. We'll see how things shake out in M | 20:23 |
| jgriffith | patrickeast: haha... Ok, fair enough | 20:24 |
| *** p0rtal_ is now known as p0rtal | 20:24 | |
| smcginnis | jgriffith, patrickeast, winston-d, thingee, xyang1: Looks like our summit session is the last block on Tuesday. 5:30pm | 20:24 |
| patrickeast | jgriffith: so back to the config thing... why can't we parse it earlier? or can we and we just dont? | 20:24 |
| jgriffith | patrickeast: we can't without a lot of change | 20:25 |
| xyang1 | smcginnis: Right | 20:25 |
| patrickeast | jgriffith: what if we parse it twice? :o | 20:25 |
| jgriffith | patrickeast: because it goes through and sets the global CONF entries as it starts the services | 20:25 |
| patrickeast | jgriffith: ooh | 20:25 |
| patrickeast | jgriffith: i see | 20:25 |
| jgriffith | patrickeast: yeah... that's kinda what I was wondering | 20:25 |
| kmartin | smcginnis, almost prime time! | 20:25 |
| jgriffith | patrickeast: service.py startup | 20:25 |
| *** julim has quit IRC | 20:25 | |
| jgriffith | that's where the magic happens | 20:25 |
| smcginnis | kmartin: That will probably be the night where there's a good early vendor party. :) | 20:26 |
| jgriffith | patrickeast: anyway.... no big deal, we'll look at it in M | 20:26 |
| kmartin | smcginnis, less questions that way :) | 20:26 |
| jgriffith | smcginnis: LOL... addind is a word!!! | 20:26 |
| jgriffith | :) | 20:27 |
| smcginnis | jgriffith: Hah! :) | 20:27 |
| jgriffith | I don't mind changing it if you promise to put the +2/A back on it after I do so :) | 20:27 |
| smcginnis | jgriffith: Don't bother. | 20:27 |
| openstackgerrit | John Griffith proposed openstack/cinder: get_replication_updates call to driver is wrong https://review.openstack.org/218348 | 20:27 |
| smcginnis | jgriffith: But I would. | 20:27 |
| jgriffith | LOL | 20:27 |
| jgriffith | too late | 20:27 |
| smcginnis | jgriffith: I mean I would put the +2/+A back, not that I would update the typo. :) | 20:28 |
| jgriffith | I can't remember if gerrit will auto add the votes back in for commit mesg updates or not | 20:28 |
| smcginnis | jgriffith: Doesn't look like it. I got it. | 20:28 |
| smcginnis | Bam | 20:29 |
| kmartin | smcginnis, my 12 year old has an English paper due, can I put in gerrit for you to review? :) | 20:29 |
| patrickeast | lol | 20:30 |
| smcginnis | Haha! | 20:30 |
| smcginnis | I just pointed it out to be funny. I really don't care! | 20:30 |
| smcginnis | I don't -1 for those. | 20:30 |
| *** dflorea has quit IRC | 20:30 | |
| smcginnis | kmartin: That would be pretty funny to use gerritt to do paper reviews. I guess we already do with docs, but would be awesome to see a school spin up an instance for that. | 20:31 |
| smcginnis | Oh well, it's beer:30 here. Have a good weekend all. Might be scarce most of next week. | 20:32 |
| kmartin | smcginnis, yeah jgriffith and I would not make core in that system | 20:32 |
| jgriffith | LOL | 20:32 |
| kmartin | enjoy your weekend | 20:32 |
| jgriffith | smcginnis: enjoy! | 20:32 |
| hemna | smcginnis, why not use a hacking check with libpspell ? | 20:32 |
| smcginnis | hemna: Awesome. I'm going to propose it. :P | 20:33 |
| hemna | it'll auto -1 as smcginnis | 20:33 |
| smcginnis | ;) | 20:33 |
| *** smcginnis is now known as smcginnis_afk | 20:34 | |
| asselin_ | All, sorry about all the 'unstable' checks on our ci. Our log server filled up. It's fixed now (but some more in the queue will report unstable) | 20:35 |
| patrickeast | anyone else seeing issues with their CI system and tempest.api.volume.admin.test_volumes_backup.VolumesBackupsV2Test.test_volume_backup_create_get_detailed_list_restore_delete? | 20:35 |
| patrickeast | http://ec2-54-67-51-189.us-west-1.compute.amazonaws.com/76/216376/1/check/PureFCDriver-tempest-dsvm-volume-multipath/952a4cb/logs/screen-c-bak.txt.gz?level=TRACE | 20:35 |
| patrickeast | only my FC driver seems to be affected by it | 20:35 |
| *** lpetrut has quit IRC | 20:36 | |
| *** dflorea has joined #openstack-cinder | 20:36 | |
| Swanson | patrickeast: I've not seen that one. | 20:37 |
| patrickeast | asselin_: do you guys have a monitoring system like nagios or something? problems like that are almost a thing of the past for me once i set one up | 20:38 |
| patrickeast | asselin_: although it kills my inbox :( | 20:38 |
| asselin_ | patrickeast, not yet, we have someone who'll start working on that | 20:39 |
| tbarron | patrickeast: the restore fails doing volume_file.write() with IOError: Operation not permitted | 20:40 |
| patrickeast | tbarron: yea, seems like a valid error if it can't write to the disk | 20:40 |
| patrickeast | tbarron: the interesting thing is that its intermittent | 20:40 |
| tbarron | patrickeast: is it trying to write to the same volume as the adjacent 'Fibre volume not yet found' warning? | 20:40 |
| patrickeast | tbarron: good question... not sure | 20:41 |
| tbarron | patrickeast: that's all I see there ... | 20:42 |
| patrickeast | tbarron: looks like it found the volume http://ec2-54-67-51-189.us-west-1.compute.amazonaws.com/76/216376/1/check/PureFCDriver-tempest-dsvm-volume-multipath/952a4cb/logs/screen-c-bak.txt.gz?#_2015-08-24_19_52_39_592 | 20:42 |
| patrickeast | oops http://ec2-54-67-51-189.us-west-1.compute.amazonaws.com/76/216376/1/check/PureFCDriver-tempest-dsvm-volume-multipath/952a4cb/logs/screen-c-bak.txt.gz?#_2015-08-24_19_52_41_591 | 20:42 |
| *** dustins_ has joined #openstack-cinder | 20:44 | |
| *** dflorea has quit IRC | 20:44 | |
| *** dustins has quit IRC | 20:45 | |
| *** dustins_ is now known as dustins | 20:45 | |
| *** krtaylor has quit IRC | 20:51 | |
| *** edtubill has left #openstack-cinder | 20:51 | |
| tbarron | patrickeast: LOL the NetApp proxy-screener held up that earlier link for the longest time when I retrieved it | 20:54 |
| patrickeast | lol | 20:54 |
| tbarron | patrickeast: mebbe it saw PureFCDriver :-) | 20:54 |
| patrickeast | tbarron: clearly something sketchy | 20:55 |
| *** jaypipes has joined #openstack-cinder | 20:55 | |
| tbarron | patrickeast: anyways, looks like volume 405e58ed-0bba-4614-9141-47a3bd1feb35 is having its export removed right before an attempt to write a chunk to it. | 20:55 |
| patrickeast | tbarron: hmm yea seems like that shouldn't have happened | 20:56 |
| tbarron | maybe the restore is taking too long and there's a timeout and test cleanup has started .... | 20:57 |
| tbarron | on that note, I'm outta here. It's east coast weekend now :-) | 20:57 |
| patrickeast | tbarron: ohh yea that would make sense | 20:58 |
| *** esker has joined #openstack-cinder | 20:58 | |
| patrickeast | tbarron: thanks for taking a look! have a good weekend | 20:58 |
| *** esker has quit IRC | 20:58 | |
| tbarron | cy'all later. I'll be doing some reviews and checking mine over coffee in the morning. | 20:58 |
| *** tbarron has left #openstack-cinder | 20:58 | |
| *** timcl has quit IRC | 20:58 | |
| *** esker has joined #openstack-cinder | 20:59 | |
| *** haomaiwang has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-cinder | 21:01 | |
| *** raildo is now known as raildo-afk | 21:02 | |
| *** krtaylor has joined #openstack-cinder | 21:04 | |
| *** IlyaG has joined #openstack-cinder | 21:04 | |
| *** _cjones_ has quit IRC | 21:04 | |
| *** dflorea has joined #openstack-cinder | 21:05 | |
| *** IlyaG has quit IRC | 21:07 | |
| *** dustins has quit IRC | 21:10 | |
| *** dustins has joined #openstack-cinder | 21:10 | |
| *** dustins has quit IRC | 21:11 | |
| *** hemna has quit IRC | 21:11 | |
| *** IlyaG has joined #openstack-cinder | 21:18 | |
| *** dflorea has quit IRC | 21:25 | |
| *** jungleboyj has joined #openstack-cinder | 21:26 | |
| jungleboyj | kmartin: Ping | 21:27 |
| kmartin | jungleboyj, yo | 21:27 |
| jungleboyj | kmartin: Need more help seeing up a Storwize. ;-) | 21:27 |
| *** eharney has joined #openstack-cinder | 21:28 | |
| kmartin | lol | 21:28 |
| jungleboyj | kmartin: Just kidding. Is there someone from HP that is working on code for the community to improve the ability to bring existing resources into Cinder's management? | 21:28 |
| *** bkopilov has quit IRC | 21:28 | |
| kmartin | jungleboyj, what type of resources? | 21:29 |
| jungleboyj | I am assuming existing volumes and/or backends with existing volumes. | 21:29 |
| kmartin | jungleboyj, not that I'm aware of | 21:30 |
| *** dflorea has joined #openstack-cinder | 21:30 | |
| *** dflorea has quit IRC | 21:31 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Fix a merge problem in VMAX driver https://review.openstack.org/218421 | 21:32 |
| jungleboyj | kmartin: Ok, interesting. One of our developers thought that maybe such work was being done. We have code to help. :-) | 21:32 |
| kmartin | jungleboyj, I can ask around | 21:33 |
| jungleboyj | Please do. Let me know if you hear anything. | 21:34 |
| *** bkopilov has joined #openstack-cinder | 21:34 | |
| *** IlyaG has quit IRC | 21:35 | |
| *** martyturner has quit IRC | 21:36 | |
| *** dflorea has joined #openstack-cinder | 21:36 | |
| *** mriedem has quit IRC | 21:37 | |
| *** dimsum__ has quit IRC | 21:38 | |
| *** cbader has quit IRC | 21:40 | |
| Swanson | Did pure have a replication v2 patch in the pipeline? | 21:42 |
| patrickeast | Swanson: yea | 21:42 |
| *** bkopilov has quit IRC | 21:42 | |
| patrickeast | Swanson: https://review.openstack.org/#/c/214825/ | 21:42 |
| Swanson | patrickeast: Thanks! Wanted to see what an implementation looked like. | 21:43 |
| *** martyturner has joined #openstack-cinder | 21:46 | |
| *** annegentle has quit IRC | 21:47 | |
| *** jaypipes has quit IRC | 21:47 | |
| *** ebalduf has quit IRC | 21:47 | |
| *** annegentle has joined #openstack-cinder | 21:47 | |
| *** martyturner has left #openstack-cinder | 21:49 | |
| *** jaypipes has joined #openstack-cinder | 21:49 | |
| *** hemna has joined #openstack-cinder | 21:51 | |
| *** jaypipes has quit IRC | 21:53 | |
| *** jaypipes has joined #openstack-cinder | 21:53 | |
| *** setmason has left #openstack-cinder | 21:54 | |
| eikke | what's the reason for gate jobs to be 'aborted by anonymous'? | 21:55 |
| Swanson | eikke: Dunno. Try #openstack-infra | 21:56 |
| *** dimsum__ has joined #openstack-cinder | 21:56 | |
| *** Apoorva has quit IRC | 21:58 | |
| *** setmason has joined #openstack-cinder | 21:58 | |
| *** Apoorva has joined #openstack-cinder | 21:59 | |
| *** dims_ has joined #openstack-cinder | 22:00 | |
| *** xyang1 has quit IRC | 22:00 | |
| *** haomaiwang has quit IRC | 22:01 | |
| *** bkopilov has joined #openstack-cinder | 22:01 | |
| *** haomaiwang has joined #openstack-cinder | 22:02 | |
| *** dimsum__ has quit IRC | 22:02 | |
| *** setmason has quit IRC | 22:03 | |
| jgriffith | Swanson: patrickeast just FYI, I'm about to give that a -2 :( | 22:03 |
| jgriffith | Swanson: patrickeast we kinda need to learn from past mistakes | 22:03 |
| patrickeast | jgriffith: so... no one gets replication then? | 22:04 |
| jgriffith | Swanson: patrickeast there's a number of things int the replication code that still need worked out | 22:04 |
| jgriffith | patrickeast: that's what I'm thinking... if we can't make any additions/changes to what's there | 22:04 |
| jgriffith | patrickeast: and since we said nobody can implement it after last Sunday (and the base code didn't even get submitted as final until then) | 22:05 |
| Swanson | jgriffith: Is there a way in v2 replication to move a volume from being on a secondary array for one backend to being on the primary array of another? | 22:05 |
| Swanson | Or is that just normal migration? | 22:05 |
| jgriffith | patrickeast: You're statement earlier about "can't add an api call or any new methods" kind of got me thinking about it | 22:05 |
| jgriffith | Swanson: not sure I follow | 22:06 |
| jgriffith | patrickeast: you made a good point about things impacting other peoples implementation | 22:06 |
| jgriffith | patrickeast: and I suspect that as more people try and implement this they're likely going to have tweaks that they propose. | 22:07 |
| patrickeast | jgriffith: i kind of get where you are coming from... but i would like to make a few things clear (for the record as it were...) we met the freeze requirements thingee set out, and i didn't mean we couldn't add new stuff, just that i was skeptical there would be time to do it | 22:07 |
| Swanson | jgriffith: 2 arrays. Each on a backend. (dell1 and dell2). dell1 uses dell2 as its replication array and dell2 uses dell1 as it's array. Dell1 burns. So the backend has failed over its volumes to the replication volumes on dell2. Dell2 should probably just handle those volumes. | 22:07 |
| patrickeast | maybe its just a process thing though... in my mind having it in the tree just makes things easier to iterate on and improve | 22:07 |
| patrickeast | having a bunch of out of tree secret impls | 22:08 |
| jgriffith | patrickeast: haha! | 22:08 |
| patrickeast | and proposing changes/fixes to the core feature seems less good | 22:08 |
| patrickeast | that being said | 22:08 |
| patrickeast | having 20+ | 22:08 |
| patrickeast | in the tree | 22:08 |
| jgriffith | patrickeast: https://github.com/j-griffith | 22:08 |
| patrickeast | thats hard to iterate on | 22:08 |
| jgriffith | nothing secret on my side if that's what you meant | 22:08 |
| *** jaypipes has quit IRC | 22:08 | |
| jgriffith | note the nova repo with the essex-replication branch, and the replication v2 branch in my cinder repo as well | 22:09 |
| patrickeast | jgriffith: haha, maybe wrong words to use | 22:09 |
| jgriffith | patrickeast: I was hoping that was all that was | 22:09 |
| patrickeast | jgriffith: not secret, just less easy to see in one place | 22:09 |
| Swanson | Gate is averaging 25 hours to get a job through. | 22:09 |
| patrickeast | jgriffith: anyway, if you don't agree with that i can understand | 22:09 |
| jgriffith | Swanson: :( | 22:09 |
| jgriffith | patrickeast: nah... it's fine | 22:10 |
| jgriffith | patrickeast: I don't think I"m getting my point across at all so that's ok | 22:10 |
| jgriffith | patrickeast: black and white interpretation of the various deadlines and process rules, you're absolutely rigth | 22:11 |
| jgriffith | right | 22:11 |
| jgriffith | patrickeast: although... I will say it's impossible that your driver "worked" because the method was broken and if it were called it would blow up until today | 22:12 |
| jgriffith | patrickeast: thus the patch I pushed to fix it this afternoon | 22:12 |
| patrickeast | jgriffith: yea, and i mean, i'm not trying to be a rule nazi about that stuff, i understand they are more like guidelines we set for ourselves | 22:12 |
| patrickeast | jgriffith: no argument there, i'm taking care of that | 22:12 |
| patrickeast | jgriffith: and i think i understand what you are saying, which is that its easier to modify the core feature without any implementations | 22:12 |
| patrickeast | jgriffith: and it prevents half-baked features from releasing | 22:13 |
| jgriffith | patrickeast: yeah, that's all i was getting at | 22:13 |
| jgriffith | patrickeast: and avoids any API changes ... or the whole situation we had in the first place that led to V2 to begin with (one impl at the end of the release) | 22:13 |
| jgriffith | but.. that's cool | 22:13 |
| jgriffith | I'm surely not sweating it | 22:14 |
| patrickeast | jgriffith: i'm not sure if comparing it to the v1 replication is the same... but i see your point | 22:14 |
| jgriffith | patrickeast: well... I'll go ahead and throw my version up on gerrit later | 22:15 |
| jgriffith | patrickeast: won't merge for certain but at least something else as a reference will be available | 22:15 |
| patrickeast | jgriffith: i guess the only thing i want to argue is that having implementations in tree is useful, even ones that will have to change a bit as they settle down and the core feature is tweaked | 22:15 |
| Swanson | It would be nice if one could add features to drivers of a stable branch. Up until milestone 1 of the next release or something. | 22:16 |
| patrickeast | jgriffith: i wouldn't push on it if we had a lvm replication impl ;) | 22:16 |
| jgriffith | patrickeast: haha! It's useful for Pure to say "hey we have replication in Liberty" LOL | 22:16 |
| jgriffith | Swanson: yeah... I think we really need to look at how we do releases all the way around | 22:16 |
| patrickeast | jgriffith: honestly i was going to ensure it was labelled as experimental and shouldn't be used | 22:16 |
| jgriffith | Swanson: I was talking to bswartz abou tthis the other day | 22:16 |
| jgriffith | patrickeast: I know.. that was a smart ass comment | 22:17 |
| jgriffith | patrickeast: anyway... we'll carry on | 22:17 |
| jgriffith | no problem | 22:17 |
| *** edmondsw has quit IRC | 22:17 | |
| Swanson | jgriffith: The issue I see is that companies trail on releases. So it would really be nice to be able to support everything in a release eventually. | 22:17 |
| jgriffith | Swanson: no doubt! | 22:18 |
| jgriffith | Swanson: the thing that's annoying is most distros/customers are at least n-1 | 22:18 |
| *** ronis has quit IRC | 22:18 | |
| jgriffith | Swanson: often n-2 | 22:18 |
| jgriffith | Swanson: so you work on this pretty new feature, it takes two cycles and never sees the light of day for 18-24 months | 22:18 |
| jgriffith | Swanson: until you do the backport for them :) | 22:19 |
| Swanson | jgriffith: Yep. We showed up with Kilo. Most people are juno or icehouse. | 22:19 |
| jgriffith | Swanson: cuz they "gotta have it" | 22:19 |
| *** vgridnev has quit IRC | 22:19 | |
| jgriffith | but forget about upgrading their cloud, because we refuse as a dev community to make deployment and upgrades better | 22:19 |
| Swanson | upgrades are even possible? | 22:20 |
| jgriffith | Swanson: LOL... YESS!!!!! | 22:20 |
| *** Apoorva has quit IRC | 22:24 | |
| *** Apoorva has joined #openstack-cinder | 22:24 | |
| *** ganso_ has quit IRC | 22:25 | |
| *** bnemec has quit IRC | 22:27 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Fix QoS keys not being available to scheduler https://review.openstack.org/214826 | 22:28 |
| openstackgerrit | Michael Price proposed openstack/cinder: Add support for volume groups and netapp_raid_type https://review.openstack.org/203300 | 22:31 |
| openstackgerrit | Michael Price proposed openstack/cinder: Add additional SSC extra specs to E-Series driver https://review.openstack.org/215808 | 22:31 |
| openstackgerrit | Michael Price proposed openstack/cinder: Implement AutoSupport for NetApp E-Series driver https://review.openstack.org/215807 | 22:31 |
| openstackgerrit | Michael Price proposed openstack/cinder: Implement thin provisioning support for E-Series https://review.openstack.org/215833 | 22:31 |
| openstackgerrit | Anish Bhatt proposed openstack/os-brick: Add support for --interface option in iscsiadm https://review.openstack.org/193451 | 22:36 |
| hemna | ok, off to do my community service at Knee Deep Brewery. Beer isn't going to drink itself. | 22:39 |
| *** hemna is now known as hemnafk | 22:39 | |
| *** dflorea has quit IRC | 22:39 | |
| *** dflorea has joined #openstack-cinder | 22:42 | |
| *** ndipanov has quit IRC | 22:42 | |
| *** dflorea has quit IRC | 22:44 | |
| *** eharney has quit IRC | 22:47 | |
| *** dflorea has joined #openstack-cinder | 22:53 | |
| *** lcurtis has quit IRC | 22:54 | |
| *** dflorea has quit IRC | 22:55 | |
| *** p0rtal has quit IRC | 22:58 | |
| *** p0rtal has joined #openstack-cinder | 22:59 | |
| *** jamielennox is now known as jamielennox|away | 23:00 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** vincent_hou has joined #openstack-cinder | 23:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 23:02 | |
| *** p0rtal has quit IRC | 23:03 | |
| *** annegentle has quit IRC | 23:04 | |
| *** dannywilson has quit IRC | 23:05 | |
| *** dflorea has joined #openstack-cinder | 23:05 | |
| *** vincent_hou has quit IRC | 23:07 | |
| *** p0rtal has joined #openstack-cinder | 23:07 | |
| *** hodos|2 has quit IRC | 23:08 | |
| *** Apoorva has quit IRC | 23:13 | |
| *** dflorea has quit IRC | 23:15 | |
| *** Apoorva has joined #openstack-cinder | 23:17 | |
| *** dflorea has joined #openstack-cinder | 23:20 | |
| *** mtanino has quit IRC | 23:21 | |
| *** david-lyle has quit IRC | 23:24 | |
| *** Apoorva has quit IRC | 23:30 | |
| *** garthb has quit IRC | 23:32 | |
| *** Apoorva has joined #openstack-cinder | 23:32 | |
| *** ganso_ has joined #openstack-cinder | 23:35 | |
| *** dflorea has quit IRC | 23:39 | |
| *** Apoorva has quit IRC | 23:41 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/215440 | 23:42 |
| *** salv-orlando has quit IRC | 23:45 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/210918 | 23:47 |
| *** p0rtal has quit IRC | 23:47 | |
| *** dimsum__ has joined #openstack-cinder | 23:52 | |
| *** dims_ has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!