| *** Nakato has joined #openstack-cinder | 00:10 | |
| *** zhangjn has joined #openstack-cinder | 00:11 | |
| *** salv-orlando has quit IRC | 00:13 | |
| *** zhangjn has quit IRC | 00:16 | |
| *** changbl has joined #openstack-cinder | 00:17 | |
| *** EinstCrazy has quit IRC | 00:17 | |
| *** changbl has quit IRC | 00:17 | |
| *** changbl has joined #openstack-cinder | 00:17 | |
| *** changbl has quit IRC | 00:17 | |
| *** changbl has joined #openstack-cinder | 00:18 | |
| *** changbl_ has joined #openstack-cinder | 00:18 | |
| *** changbl_ has quit IRC | 00:18 | |
| *** akshai has joined #openstack-cinder | 00:19 | |
| *** changbl has quit IRC | 00:19 | |
| *** akshai_ has joined #openstack-cinder | 00:20 | |
| *** akshai has quit IRC | 00:23 | |
| *** changbl has joined #openstack-cinder | 00:29 | |
| *** Yogi1 has quit IRC | 00:32 | |
| *** Yogi1 has joined #openstack-cinder | 00:40 | |
| *** theanalyst has quit IRC | 00:53 | |
| *** EinstCrazy has joined #openstack-cinder | 00:56 | |
| *** zhangjn has joined #openstack-cinder | 00:56 | |
| *** theanalyst has joined #openstack-cinder | 00:57 | |
| *** Yogi1 has quit IRC | 01:09 | |
| *** zhangjn has quit IRC | 01:12 | |
| *** zhangjn has joined #openstack-cinder | 01:13 | |
| *** salv-orlando has joined #openstack-cinder | 01:14 | |
| *** Yogi1 has joined #openstack-cinder | 01:15 | |
| *** gcb has joined #openstack-cinder | 01:22 | |
| *** julim has quit IRC | 01:27 | |
| *** salv-orlando has quit IRC | 01:32 | |
| *** Lee1092 has joined #openstack-cinder | 01:39 | |
| *** haomaiwang has joined #openstack-cinder | 01:41 | |
| *** jerrygb has joined #openstack-cinder | 01:44 | |
| *** zhenguo has joined #openstack-cinder | 01:48 | |
| *** Yogi1 has quit IRC | 01:59 | |
| *** haomaiwang has quit IRC | 02:01 | |
| *** haomaiwang has joined #openstack-cinder | 02:01 | |
| *** [1]Thelo has joined #openstack-cinder | 02:02 | |
| *** rmk has quit IRC | 02:03 | |
| *** Thelo has quit IRC | 02:04 | |
| *** [1]Thelo is now known as Thelo | 02:04 | |
| *** jerrygb has quit IRC | 02:05 | |
| *** rushil has joined #openstack-cinder | 02:05 | |
| *** jerrygb has joined #openstack-cinder | 02:05 | |
| *** rmk has joined #openstack-cinder | 02:06 | |
| *** jerrygb has quit IRC | 02:10 | |
| *** salv-orlando has joined #openstack-cinder | 02:32 | |
| *** salv-orlando has quit IRC | 02:48 | |
| *** houming has joined #openstack-cinder | 02:50 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:01 | |
| *** rushil has quit IRC | 03:02 | |
| *** smcginnis has quit IRC | 03:03 | |
| *** Swanson has quit IRC | 03:04 | |
| *** Swanson has joined #openstack-cinder | 03:05 | |
| *** smcginnis has joined #openstack-cinder | 03:05 | |
| *** jerrygb has joined #openstack-cinder | 03:06 | |
| *** jerrygb has quit IRC | 03:11 | |
| *** coolsvap has quit IRC | 03:16 | |
| *** pots has joined #openstack-cinder | 03:18 | |
| *** coolsvap has joined #openstack-cinder | 03:19 | |
| *** rushil has joined #openstack-cinder | 03:26 | |
| *** changbl has quit IRC | 03:38 | |
| *** salv-orlando has joined #openstack-cinder | 03:47 | |
| *** pots has quit IRC | 03:47 | |
| *** houming has quit IRC | 03:52 | |
| *** zhangjn has quit IRC | 03:54 | |
| *** salv-orlando has quit IRC | 03:55 | |
| *** houming has joined #openstack-cinder | 03:57 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-cinder | 04:01 | |
| *** houming has quit IRC | 04:07 | |
| *** changbl has joined #openstack-cinder | 04:07 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (split IO) https://review.openstack.org/237387 | 04:09 |
|---|---|---|
| *** gouthamr has quit IRC | 04:11 | |
| *** akshai_ has quit IRC | 04:12 | |
| *** rushil has quit IRC | 04:15 | |
| *** rushil has joined #openstack-cinder | 04:16 | |
| *** rushil has quit IRC | 04:16 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (split IO) https://review.openstack.org/237387 | 04:18 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Update migrate_volume API to use versionedobjects https://review.openstack.org/228916 | 04:19 |
| *** zhangjn has joined #openstack-cinder | 04:20 | |
| *** asselin_ has joined #openstack-cinder | 04:36 | |
| *** asselin has quit IRC | 04:38 | |
| *** sgundur has joined #openstack-cinder | 04:43 | |
| *** sgundur has left #openstack-cinder | 04:43 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Update initialize_connection to use versionedobjects https://review.openstack.org/238894 | 04:48 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Register RPC and object versions https://review.openstack.org/209701 | 04:50 |
| *** zhangjn has quit IRC | 04:51 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** houming has joined #openstack-cinder | 05:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 05:01 | |
| *** zhangjn has joined #openstack-cinder | 05:14 | |
| *** ircuser-1 has quit IRC | 05:26 | |
| *** chhavi has joined #openstack-cinder | 05:27 | |
| *** asselin_ has left #openstack-cinder | 05:29 | |
| *** mdbooth has quit IRC | 05:33 | |
| *** rcernin has joined #openstack-cinder | 05:36 | |
| *** mdbooth has joined #openstack-cinder | 05:39 | |
| *** rcernin has quit IRC | 05:42 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Update retype limitation in volume/api https://review.openstack.org/245460 | 05:53 |
| *** haomaiwa_ has quit IRC | 06:01 | |
| *** 17WAA30H9 has joined #openstack-cinder | 06:01 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: FlashSystem reports error in _find_host_exhaustive() https://review.openstack.org/247546 | 06:09 |
| *** zhangjn has quit IRC | 06:10 | |
| *** chhavi has quit IRC | 06:15 | |
| *** jwcroppe has joined #openstack-cinder | 06:16 | |
| *** zhangjn has joined #openstack-cinder | 06:20 | |
| *** deepakcs has joined #openstack-cinder | 06:21 | |
| *** zhangjn has quit IRC | 06:22 | |
| *** chhavi has joined #openstack-cinder | 06:27 | |
| *** lpetrut has joined #openstack-cinder | 06:28 | |
| *** wanghao has joined #openstack-cinder | 06:39 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/248544 | 06:42 |
| *** zhangjn has joined #openstack-cinder | 06:49 | |
| *** ankit_ag has joined #openstack-cinder | 06:59 | |
| *** 17WAA30H9 has quit IRC | 07:01 | |
| *** chhavi has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:01 | |
| *** lpetrut has quit IRC | 07:01 | |
| *** shausy has joined #openstack-cinder | 07:01 | |
| *** shausy has quit IRC | 07:02 | |
| *** jwcroppe has quit IRC | 07:03 | |
| *** sgotliv has joined #openstack-cinder | 07:04 | |
| *** rcernin has joined #openstack-cinder | 07:07 | |
| *** shausy has joined #openstack-cinder | 07:08 | |
| *** sgotliv has quit IRC | 07:12 | |
| *** openstack has joined #openstack-cinder | 07:19 | |
| *** zhangjn has quit IRC | 07:19 | |
| *** zhangjn has joined #openstack-cinder | 07:32 | |
| *** alexschm has joined #openstack-cinder | 07:34 | |
| *** nkrinner has joined #openstack-cinder | 07:45 | |
| *** jerrygb has joined #openstack-cinder | 07:51 | |
| *** jerrygb has quit IRC | 07:57 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-cinder | 08:01 | |
| *** lpetrut has joined #openstack-cinder | 08:07 | |
| *** liverpooler has joined #openstack-cinder | 08:11 | |
| *** lpetrut has quit IRC | 08:26 | |
| *** liverpoo1er has joined #openstack-cinder | 08:31 | |
| *** haomaiwang has quit IRC | 08:32 | |
| *** haomaiwang has joined #openstack-cinder | 08:36 | |
| *** xek has joined #openstack-cinder | 08:46 | |
| *** jordanP has joined #openstack-cinder | 08:52 | |
| *** liverpoo1er has quit IRC | 08:56 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-cinder | 09:01 | |
| *** sgotliv has joined #openstack-cinder | 09:04 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (split IO) https://review.openstack.org/237387 | 09:04 |
| *** geguileo has joined #openstack-cinder | 09:15 | |
| *** salv-orlando has joined #openstack-cinder | 09:17 | |
| *** ociuhandu has quit IRC | 09:26 | |
| *** jistr has joined #openstack-cinder | 09:27 | |
| *** e0ne has joined #openstack-cinder | 09:28 | |
| *** vgridnev has joined #openstack-cinder | 09:29 | |
| *** deepakcs has quit IRC | 09:32 | |
| *** bluex has joined #openstack-cinder | 09:39 | |
| *** liverpooler has quit IRC | 09:39 | |
| *** bluex has quit IRC | 09:40 | |
| *** bluex has joined #openstack-cinder | 09:40 | |
| *** aix has joined #openstack-cinder | 09:43 | |
| openstackgerrit | LisaLi proposed openstack/cinder-specs: Retype encrypted volumes https://review.openstack.org/248593 | 09:43 |
| *** liverpooler has joined #openstack-cinder | 09:49 | |
| *** zhangjn has quit IRC | 09:49 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Move imageutils from oslo-incubator to cinder namespace https://review.openstack.org/247480 | 09:51 |
| *** aix has quit IRC | 09:52 | |
| *** zhenguo has quit IRC | 09:56 | |
| *** bluex has quit IRC | 09:59 | |
| *** ekarlso- has joined #openstack-cinder | 09:59 | |
| *** ekarlso has quit IRC | 10:00 | |
| *** gcb has quit IRC | 10:00 | |
| *** houming has quit IRC | 10:00 | |
| *** ekarlso- is now known as ekarlso | 10:00 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-cinder | 10:01 | |
| *** bluex has joined #openstack-cinder | 10:05 | |
| *** zhangjn has joined #openstack-cinder | 10:06 | |
| *** jerrygb has joined #openstack-cinder | 10:08 | |
| *** haypo has joined #openstack-cinder | 10:11 | |
| haypo | hi. my patch serie is almost approved (second patch already approved, first patch got a +2). anyone to approve the first patch? https://review.openstack.org/#/c/232249/ it's to port EMC VMAX to Python 3 | 10:11 |
| haypo | on the first patch, i got a -1 from Duncan Thomas, but i replied (that it's ok, it's not a bug) and he didn't change his vote or comment the change :-/ | 10:12 |
| *** jerrygb has quit IRC | 10:13 | |
| *** lpetrut has joined #openstack-cinder | 10:16 | |
| *** salv-orlando has quit IRC | 10:21 | |
| *** tpsilva has joined #openstack-cinder | 10:22 | |
| *** salv-orlando has joined #openstack-cinder | 10:22 | |
| *** aix has joined #openstack-cinder | 10:24 | |
| *** deepakcs has joined #openstack-cinder | 10:46 | |
| *** kaisers1 has joined #openstack-cinder | 10:49 | |
| *** deepakcs has quit IRC | 10:50 | |
| *** tsufiev_ is now known as tsufiev | 10:54 | |
| *** eduardo_ has joined #openstack-cinder | 10:54 | |
| *** bluex has quit IRC | 10:58 | |
| *** ociuhandu has joined #openstack-cinder | 11:00 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-cinder | 11:01 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder-specs: Support HA Active/Active configurations https://review.openstack.org/232599 | 11:07 |
| *** haomaiwang has quit IRC | 11:07 | |
| *** chenying has joined #openstack-cinder | 11:13 | |
| *** erlon has joined #openstack-cinder | 11:27 | |
| *** houming has joined #openstack-cinder | 11:37 | |
| *** bluex has joined #openstack-cinder | 11:39 | |
| *** aagrawal has joined #openstack-cinder | 11:53 | |
| *** aagrawal has quit IRC | 11:53 | |
| *** gcb has joined #openstack-cinder | 11:57 | |
| *** salv-orlando has quit IRC | 12:00 | |
| *** [1]Thelo has joined #openstack-cinder | 12:02 | |
| openstackgerrit | wanghao proposed openstack/cinder: Retyping volume got error under max vol limit https://review.openstack.org/233325 | 12:03 |
| *** Thelo has quit IRC | 12:04 | |
| *** [1]Thelo is now known as Thelo | 12:04 | |
| *** jaypipes has joined #openstack-cinder | 12:08 | |
| *** zhangjn has quit IRC | 12:11 | |
| *** EinstCrazy has quit IRC | 12:11 | |
| *** salv-orlando has joined #openstack-cinder | 12:14 | |
| *** sgotliv has quit IRC | 12:21 | |
| *** jwcroppe has joined #openstack-cinder | 12:33 | |
| *** EinstCrazy has joined #openstack-cinder | 12:34 | |
| *** marcusvrn_ has joined #openstack-cinder | 12:35 | |
| DuncanT | haypo: Give me five minutes and I'll look again | 12:38 |
| DuncanT | haypo: Sorry, I hadn't gotten around to coming back to that patch. Please feel free to poke me directly if anything similar happens in future | 12:41 |
| *** lpabon has joined #openstack-cinder | 12:42 | |
| dulek | e0ne: Can you check out my answer to your comment on https://review.openstack.org/#/c/245976/ ? I'm not sure how to move further with the patch. If you still think your comment applies, I'll happily update the spec. | 12:43 |
| *** cdelatte has joined #openstack-cinder | 12:45 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Update initialize_connection to use versionedobjects https://review.openstack.org/238894 | 12:47 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Update migrate_volume API to use versionedobjects https://review.openstack.org/228916 | 12:47 |
| *** aix has quit IRC | 12:47 | |
| *** aix has joined #openstack-cinder | 12:48 | |
| *** cristicalin has joined #openstack-cinder | 12:48 | |
| haypo | DuncanT: no problem. i didn't try IRC, i will next time ;) | 12:51 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Update replication to use versionedobjects https://review.openstack.org/238906 | 12:51 |
| haypo | DuncanT: i don't understand the exact consequence of changing command order. but at the same time, the behaviour before my change is not really reliable, so... | 12:52 |
| DuncanT | haypo: Yup. It's only the test, so I suggest we get Xing to take a look later to clean it up if necessary | 12:54 |
| haypo | DuncanT: with my patch, the behaviour is now well defined. commands are always sorted by the LUN identifier | 12:55 |
| *** pots has joined #openstack-cinder | 12:56 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 12:57 |
| *** jerrygb has joined #openstack-cinder | 12:59 | |
| *** jerrygb has quit IRC | 12:59 | |
| *** zhangjn has joined #openstack-cinder | 12:59 | |
| *** jerrygb has joined #openstack-cinder | 12:59 | |
| *** zhangjn has quit IRC | 12:59 | |
| *** haomaiwa_ has joined #openstack-cinder | 12:59 | |
| *** zhangjn has joined #openstack-cinder | 13:00 | |
| *** jerrygb_ has joined #openstack-cinder | 13:00 | |
| *** haomaiwa_ has quit IRC | 13:01 | |
| *** haomaiwang has joined #openstack-cinder | 13:01 | |
| *** dustins has joined #openstack-cinder | 13:01 | |
| *** jerrygb has quit IRC | 13:03 | |
| *** zhangjn has quit IRC | 13:06 | |
| *** sgotliv has joined #openstack-cinder | 13:06 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Add some missing fields to Volume object https://review.openstack.org/240847 | 13:07 |
| *** ankit_ag has quit IRC | 13:09 | |
| *** shausy has quit IRC | 13:10 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder-specs: Add incremental volume snapshot support https://review.openstack.org/248693 | 13:20 |
| *** haomaiwang has quit IRC | 13:22 | |
| *** isaacb has joined #openstack-cinder | 13:24 | |
| *** haomaiwa_ has joined #openstack-cinder | 13:24 | |
| *** akerr has joined #openstack-cinder | 13:26 | |
| *** gouthamr has joined #openstack-cinder | 13:28 | |
| *** jaypipes has quit IRC | 13:29 | |
| *** jaypipes has joined #openstack-cinder | 13:32 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Add guidelines for release notes to devref https://review.openstack.org/248701 | 13:34 |
| e0ne | dulek: hi. will take a look on it in 30min | 13:45 |
| *** jistr has quit IRC | 13:46 | |
| dulek | e0ne: I have all the time in the world. :) | 13:47 |
| e0ne | :) | 13:47 |
| *** ccesario has quit IRC | 13:49 | |
| *** amrith is now known as _amrith_ | 13:51 | |
| *** bswartz has quit IRC | 13:53 | |
| *** eharney has joined #openstack-cinder | 13:53 | |
| *** akshai has joined #openstack-cinder | 13:54 | |
| e0ne | dulek: I really don't want to get somthing that will work only if reviewer does everything right | 13:57 |
| e0ne | dulek: if we'll miss one migration in a review process - rolling upgrade will be broken. it doesn't look good | 13:59 |
| *** julim has joined #openstack-cinder | 13:59 | |
| dulek | e0ne: In the end we'll have a grenade job that will test that. | 14:00 |
| *** haomaiwa_ has quit IRC | 14:01 | |
| dulek | e0ne: Also Nova were able to go through Kilo even without a test that was blocking subtractive migrations. | 14:01 |
| dulek | e0ne: So it's possible. They've added a test in Liberty (I'm porting it to Cinder right now). | 14:01 |
| *** haomaiwang has joined #openstack-cinder | 14:01 | |
| dulek | We'll have test from the start which will be a great help, but still - careful reviews of migrations are needed. | 14:02 |
| e0ne | dulek: my point is: if we could miss something - we'll miss it sooner or later | 14:02 |
| openstackgerrit | Merged openstack/cinder: Port EMC VNX CLI to Python 3 https://review.openstack.org/232249 | 14:02 |
| dulek | e0ne: That's what superdan also emphasized and that's why I'm going to work on Grenade job once we have agreement on all the parts of rolling upgrades story. | 14:03 |
| *** cbader has joined #openstack-cinder | 14:03 | |
| dulek | e0ne: I don't see an obvious possibility to automate test more than that. | 14:04 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Don't build two tox envs for pep8(-constraints) https://review.openstack.org/247806 | 14:06 |
| openstackgerrit | Merged openstack/cinder: Port EMC VMAX to Python 3 https://review.openstack.org/243051 | 14:07 |
| openstackgerrit | Merged openstack/cinder: Eager load columns in volume_get_active_by_window https://review.openstack.org/248035 | 14:07 |
| *** martyturner has joined #openstack-cinder | 14:15 | |
| *** jgregor has joined #openstack-cinder | 14:18 | |
| e0ne | dulek: thanks for the clarifications! I've changed my vote | 14:19 |
| *** ccesario has joined #openstack-cinder | 14:20 | |
| *** raildo is now known as raildo-afk | 14:20 | |
| *** raildo-afk is now known as raildo | 14:20 | |
| *** bswartz has joined #openstack-cinder | 14:21 | |
| *** dustins has quit IRC | 14:21 | |
| *** dustins has joined #openstack-cinder | 14:22 | |
| *** merooney has joined #openstack-cinder | 14:22 | |
| *** salv-orlando has quit IRC | 14:23 | |
| *** eharney has quit IRC | 14:26 | |
| *** jasondotstar_afk is now known as jasondotstar | 14:33 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Move imageutils from oslo-incubator to cinder namespace https://review.openstack.org/247480 | 14:33 |
| openstackgerrit | Oleksii Butenko proposed openstack/python-cinderclient: Add new tests for cinderclient. https://review.openstack.org/246513 | 14:34 |
| *** superdan is now known as dansmith | 14:35 | |
| flip214 | hemna: JFI: doing "pip install ." within an os-brick git clone installs the rootwrap config to /usr/local/etc/os-brick/rootwrap.d/os-brick.filters, but it would be needed in /etc/nova/rootwrap.d/ | 14:36 |
| *** mriedem has joined #openstack-cinder | 14:38 | |
| *** circ-user-YpZtG has joined #openstack-cinder | 14:39 | |
| *** eharney has joined #openstack-cinder | 14:39 | |
| *** j_king_ is now known as j_king | 14:42 | |
| *** baumann has joined #openstack-cinder | 14:44 | |
| *** crose has joined #openstack-cinder | 14:50 | |
| *** mriedem has quit IRC | 14:52 | |
| *** bluex has quit IRC | 14:53 | |
| openstackgerrit | Oleksii Butenko proposed openstack/python-cinderclient: Add functional tests for python-cinderclient. https://review.openstack.org/246513 | 14:53 |
| *** bluex has joined #openstack-cinder | 14:53 | |
| *** aix has quit IRC | 14:54 | |
| *** houming has quit IRC | 14:54 | |
| *** mc_nair has joined #openstack-cinder | 14:54 | |
| *** lprice has joined #openstack-cinder | 14:55 | |
| openstackgerrit | zhangsong proposed openstack/cinder: Fix the bug of OSError when convert image https://review.openstack.org/243152 | 14:55 |
| *** mriedem has joined #openstack-cinder | 14:57 | |
| *** subscope has joined #openstack-cinder | 14:58 | |
| *** baumann has quit IRC | 14:58 | |
| *** lprice1 has quit IRC | 14:58 | |
| *** baumann has joined #openstack-cinder | 14:58 | |
| *** sgundur has joined #openstack-cinder | 14:58 | |
| *** julim_ has joined #openstack-cinder | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** Asselin has joined #openstack-cinder | 15:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 15:01 | |
| *** aix has joined #openstack-cinder | 15:01 | |
| *** Asselin has left #openstack-cinder | 15:01 | |
| *** openstackgerrit has quit IRC | 15:02 | |
| *** julim has quit IRC | 15:02 | |
| *** openstackgerrit has joined #openstack-cinder | 15:02 | |
| *** houming has joined #openstack-cinder | 15:03 | |
| *** _amrith_ is now known as amrith | 15:05 | |
| *** gouthamr has quit IRC | 15:07 | |
| *** breitz has joined #openstack-cinder | 15:08 | |
| *** akerr has quit IRC | 15:09 | |
| *** akerr has joined #openstack-cinder | 15:09 | |
| *** davechen has joined #openstack-cinder | 15:10 | |
| openstackgerrit | Oleksii Butenko proposed openstack/python-cinderclient: Add functional tests for python-cinderclient. https://review.openstack.org/246513 | 15:10 |
| *** liverpooler has quit IRC | 15:11 | |
| *** circ-user-YpZtG has quit IRC | 15:13 | |
| *** circ-user-YpZtG has joined #openstack-cinder | 15:16 | |
| *** sgundur has left #openstack-cinder | 15:17 | |
| *** EinstCrazy has quit IRC | 15:17 | |
| *** shhjwang has joined #openstack-cinder | 15:18 | |
| *** gouthamr has joined #openstack-cinder | 15:18 | |
| *** grumpycatt has quit IRC | 15:19 | |
| *** mdbooth has quit IRC | 15:21 | |
| *** shhjwang has quit IRC | 15:22 | |
| *** r-daneel has joined #openstack-cinder | 15:22 | |
| *** eduardo_ has quit IRC | 15:22 | |
| *** markus_z has joined #openstack-cinder | 15:23 | |
| *** houming has quit IRC | 15:24 | |
| *** salv-orlando has joined #openstack-cinder | 15:24 | |
| *** ntpttr has joined #openstack-cinder | 15:25 | |
| *** kfarr has joined #openstack-cinder | 15:28 | |
| *** amrith is now known as _amrith_ | 15:30 | |
| *** willsama has joined #openstack-cinder | 15:31 | |
| *** willsama has quit IRC | 15:31 | |
| *** thangp has joined #openstack-cinder | 15:32 | |
| *** mdbooth has joined #openstack-cinder | 15:32 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Block subtractive operations in DB migrations https://review.openstack.org/248768 | 15:33 |
| *** _amrith_ is now known as amrith | 15:33 | |
| *** jbernard has quit IRC | 15:34 | |
| *** davechen1 has joined #openstack-cinder | 15:36 | |
| *** davechen has quit IRC | 15:38 | |
| *** houming has joined #openstack-cinder | 15:39 | |
| *** haypo has left #openstack-cinder | 15:39 | |
| *** mdbooth has quit IRC | 15:40 | |
| *** mdbooth has joined #openstack-cinder | 15:40 | |
| *** davechen1 is now known as davechen | 15:41 | |
| *** salv-orlando has quit IRC | 15:43 | |
| *** bluex has quit IRC | 15:43 | |
| *** bluex has joined #openstack-cinder | 15:44 | |
| *** bluex has quit IRC | 15:45 | |
| *** bluex has joined #openstack-cinder | 15:45 | |
| openstackgerrit | Eric Harney proposed openstack/python-cinderclient: Use oslo_utils encodeutils and strutils https://review.openstack.org/248783 | 15:46 |
| *** isaacb has quit IRC | 15:51 | |
| *** xyang1 has joined #openstack-cinder | 15:52 | |
| *** eharney has quit IRC | 15:53 | |
| *** jgregor has quit IRC | 15:56 | |
| *** jgregor has joined #openstack-cinder | 15:57 | |
| *** haomaiwa_ has quit IRC | 16:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 16:01 | |
| openstackgerrit | Oleksii Butenko proposed openstack/python-cinderclient: Add functional tests for backup creation and deletion. https://review.openstack.org/246513 | 16:01 |
| mc_nair | dulek: was looking at moving QualityOfServiceSpecs to use VersionedObjects, but everything except 'QoSSpec_Name' and 'consumer' are just user defined key/value pairs. Do you think it still makes sense to convert this to a VersionedObject (guess we'd still be getting the benefits for name and consumer)? | 16:04 |
| *** anshul has quit IRC | 16:06 | |
| *** eharney has joined #openstack-cinder | 16:08 | |
| *** rcernin has quit IRC | 16:08 | |
| openstackgerrit | Oleksii Butenko proposed openstack/python-cinderclient: Add functional tests for backup creation and deletion https://review.openstack.org/246513 | 16:10 |
| *** lpetrut has quit IRC | 16:12 | |
| thangp | mc_nair: not everything in the db model needs to be a versionedobject. QualityOfServiceSpecs is part of VolumeTypes and acts like volume_metadata in volume. I think QualityOfServiceSpecs should just be a DictOfStringsField field in VolumeType. | 16:12 |
| *** salv-orlando has joined #openstack-cinder | 16:14 | |
| *** Pablo|off| has joined #openstack-cinder | 16:14 | |
| *** Pablo|off| is now known as pcaruana | 16:15 | |
| *** houming has quit IRC | 16:17 | |
| *** EinstCrazy has joined #openstack-cinder | 16:18 | |
| *** bluex has quit IRC | 16:19 | |
| *** rushil has joined #openstack-cinder | 16:20 | |
| *** alexschm has quit IRC | 16:20 | |
| *** rushil has quit IRC | 16:20 | |
| mc_nair | thangp: gotcha - thanks for the info | 16:21 |
| *** rushil has joined #openstack-cinder | 16:22 | |
| thangp | mc_nair: looking a little closer at QualityOfServiceSpecs, maybe it should be a versionedobject, since it looks to be associated and disassociated from any volumetype | 16:22 |
| *** markus_z has quit IRC | 16:23 | |
| *** EinstCrazy has quit IRC | 16:23 | |
| thangp | mc_nair: so it probably is a good idea to make it a vo. we would need to add a field to volumetype object for it too. | 16:23 |
| *** cristicalin has quit IRC | 16:24 | |
| *** daneyon_ has joined #openstack-cinder | 16:24 | |
| *** aix has quit IRC | 16:24 | |
| *** daneyon has quit IRC | 16:26 | |
| *** circ-user-YpZtG has quit IRC | 16:29 | |
| thangp | mc_nair: i think volume_type to qos_spec is a 1:n relationship | 16:29 |
| *** jdurgin1 has joined #openstack-cinder | 16:30 | |
| mc_nair | thangp: cool, think that's what dulek had in mind - I was looking for a place to get started helping w/ the VersionedObjects stuff. So just to be make sure I'm on the right path... since everything in QualityOfServiceSpecs other than id, name, and consumer are user-defined, the VersionedObjects will just have those fields defined and then one additional DictOfStringsField with the rest of the user-defined key/values? | 16:32 |
| *** jistr has joined #openstack-cinder | 16:33 | |
| thangp | mc_nair: yeah, you also need volume_types field too since it's a backref | 16:33 |
| mc_nair | thangp: ok, nice. Thanks again | 16:35 |
| thangp | mc_nair: there should be a ListOfObjectsField for volumetypes | 16:36 |
| thangp | mc_nair: np, sorry for the earlier confusion | 16:37 |
| openstackgerrit | Nicolas Trangez proposed openstack/cinder: Revert "Add Scality SRB driver" https://review.openstack.org/248810 | 16:41 |
| *** merooney has quit IRC | 16:41 | |
| *** jbernard has joined #openstack-cinder | 16:44 | |
| *** pots has quit IRC | 16:48 | |
| *** subscope has quit IRC | 16:50 | |
| *** rushil_ has joined #openstack-cinder | 16:51 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Block subtractive operations in DB migrations https://review.openstack.org/248768 | 16:53 |
| *** rushil has quit IRC | 16:55 | |
| *** haomaiwa_ has quit IRC | 17:01 | |
| *** 21WAAEGOI has joined #openstack-cinder | 17:01 | |
| *** vgridnev has quit IRC | 17:06 | |
| *** amrith is now known as _amrith_ | 17:09 | |
| *** sgundur has joined #openstack-cinder | 17:10 | |
| *** ir2ivps7 has joined #openstack-cinder | 17:13 | |
| openstackgerrit | Oleksii Butenko proposed openstack/python-cinderclient: Add functional tests: backup creation and deletion https://review.openstack.org/246513 | 17:13 |
| *** _amrith_ is now known as amrith | 17:15 | |
| dulek | mc_nair: I agree with thangp on that. I'm also not sure if we need IscsiTarget and Transfer? | 17:16 |
| *** sgotliv has quit IRC | 17:16 | |
| *** gcb has quit IRC | 17:18 | |
| *** leeantho has joined #openstack-cinder | 17:18 | |
| *** mdbooth has quit IRC | 17:18 | |
| *** 21WAAEGOI has quit IRC | 17:20 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder-specs: Remove Cinder API races https://review.openstack.org/207101 | 17:20 |
| *** rcernin has joined #openstack-cinder | 17:21 | |
| *** leeantho_ has joined #openstack-cinder | 17:21 | |
| *** leeantho_ has quit IRC | 17:21 | |
| *** leeantho_ has joined #openstack-cinder | 17:22 | |
| *** leeantho has quit IRC | 17:25 | |
| *** jerrygb_ has quit IRC | 17:26 | |
| *** jerrygb has joined #openstack-cinder | 17:33 | |
| *** rebase has joined #openstack-cinder | 17:35 | |
| *** jdurgin1 has quit IRC | 17:36 | |
| *** jerrygb has quit IRC | 17:36 | |
| *** ircuser-1 has joined #openstack-cinder | 17:36 | |
| *** nkrinner has quit IRC | 17:37 | |
| *** e0ne has quit IRC | 17:39 | |
| *** jerrygb has joined #openstack-cinder | 17:40 | |
| *** merooney has joined #openstack-cinder | 17:44 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder-specs: Remove Cinder API races https://review.openstack.org/207101 | 17:45 |
| *** aix has joined #openstack-cinder | 17:45 | |
| *** jerrygb has quit IRC | 17:45 | |
| *** merooney has quit IRC | 17:46 | |
| *** merooney has joined #openstack-cinder | 17:47 | |
| *** vgridnev has joined #openstack-cinder | 17:51 | |
| *** zul has quit IRC | 17:53 | |
| *** IlyaG has joined #openstack-cinder | 17:54 | |
| *** zul has joined #openstack-cinder | 17:58 | |
| *** apoorvad has joined #openstack-cinder | 18:02 | |
| *** daneyon has joined #openstack-cinder | 18:03 | |
| *** sghanekar__ has joined #openstack-cinder | 18:05 | |
| *** akerr has quit IRC | 18:06 | |
| *** daneyon_ has quit IRC | 18:07 | |
| *** EinstCrazy has joined #openstack-cinder | 18:10 | |
| dulek | geguileo: I've added your specs to https://etherpad.openstack.org/p/mitaka-cinder-spec-review-tracking | 18:14 |
| geguileo | dulek: Thanks!!! | 18:15 |
| dulek | geguileo: (I think smcginnis is looking there for specs reviews ;)) | 18:15 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Brick add param documentation to connectors https://review.openstack.org/246023 | 18:15 |
| dulek | geguileo: I've added some prioritetization there. Nothing very granular, but you may want to take a look if you agree. | 18:15 |
| geguileo | dulek: I think tooz locks should go before data corruption | 18:16 |
| geguileo | dulek: because we will need to know when we lose connection to the DLM | 18:16 |
| *** EinstCrazy has quit IRC | 18:17 | |
| geguileo | for the prevention | 18:17 |
| *** jordanP has quit IRC | 18:17 | |
| geguileo | But since they are both P1 I'll just move Tooz higher in the list :-) | 18:18 |
| *** aix has quit IRC | 18:18 | |
| dulek | geguileo: Oh, right, both are P1, but yeah. | 18:18 |
| geguileo | dulek: If you don't mind I'll move tooz locks to P0 | 18:18 |
| geguileo | Because I think it's at the level of API races | 18:18 |
| geguileo | It's like... basic stuff we need :-) | 18:19 |
| dulek | geguileo: Oh, certainly I won't mind that you think my spec's important. :P | 18:19 |
| geguileo | XD XD | 18:19 |
| *** IlyaG has quit IRC | 18:20 | |
| *** e0ne has joined #openstack-cinder | 18:22 | |
| dulek | geguileo: What's going on with the SQLAlchemy issue? I wonder in which version of SQLAlchemy we can expect a fix. | 18:23 |
| geguileo | dulek: I believe in next version | 18:24 |
| dulek | geguileo: I see it as resolved, but it seems your code is only on a feature branch? | 18:24 |
| geguileo | dulek: I thought it was the pre-release... | 18:24 |
| dulek | geguileo: Oh, may be, I've checked it some time ago. | 18:24 |
| geguileo | dulek: I'll ping zzzeek to see what can we expect | 18:25 |
| zzzeek | geguileo: oh no, that....was it a PG thing? | 18:25 |
| zzzeek | have totally fallen behind on all PRs | 18:25 |
| zzzeek | oh it was #200 | 18:25 |
| geguileo | zzzeek: The PR was resolved | 18:26 |
| geguileo | zzzeek: Just wondering about when that will be released | 18:26 |
| zzzeek | geguileo: what PR number. #200 is still open over here | 18:26 |
| *** jerrygb has joined #openstack-cinder | 18:27 | |
| geguileo | zzzeek: Oh, yeah it's still open | 18:27 |
| zzzeek | geguileo: yah i hvent had any PR time really. | 18:28 |
| zzzeek | geguileo: also this is a little biggish for the 1.0 series but you need it released ? | 18:28 |
| geguileo | zzzeek: Well, we need it to remove API races | 18:28 |
| dulek | geguileo: I believe your commit is on pr200 branch, which is pull request 200. ;) | 18:28 |
| geguileo | dulek: Yep, my mind is on Monday mode XD | 18:29 |
| *** e0ne has quit IRC | 18:29 | |
| geguileo | zzzeek: Because we do something like status='migrating', previous_status=status | 18:29 |
| *** e0ne has joined #openstack-cinder | 18:29 | |
| dulek | zzzeek: So we have right now that constraint on SQLAlchemy version in OpenStack: SQLAlchemy>=0.9.9,<1.1.0 | 18:29 |
| geguileo | zzzeek: And that fails on MariaDB (well, it fails on the reverse order) | 18:29 |
| zzzeek | geguileo: right. so, are you using literal text SQL for now? or is that an option maybe? | 18:30 |
| zzzeek | just one query in the wohle app right? | 18:30 |
| dulek | zzzeek: We're in process of implementing it right now. | 18:30 |
| geguileo | zzzeek: No, a couple of them | 18:30 |
| zzzeek | geguileo dulek "a couple" == 2 ? | 18:30 |
| geguileo | zzzeek: And I have the whole conditional_update method that builds the query automatically... | 18:31 |
| zzzeek | its just that i was tartgeting this at 1.1 and 1.1 is not anytime soon | 18:31 |
| zzzeek | geguileo: oh | 18:31 |
| geguileo | zzzeek: This is the base conditional update patch: https://review.openstack.org/#/c/205834/ | 18:31 |
| geguileo | zzzeek: And this is the one that adds ordering: https://review.openstack.org/#/c/231936/ | 18:32 |
| geguileo | zzzeek: And considering that for now I have 5 patches removing races and 2 of them already need it | 18:32 |
| geguileo | And I'm not even 30% done with API races | 18:33 |
| geguileo | It will probably increase quite a bit | 18:33 |
| zzzeek | geguileo: OK im mostly worried about the changes we made to the Update API and if that's going to break someone doing something, in a certain X.Y series we try to have zero chance of breakage for a new .Z | 18:35 |
| zzzeek | geguileo: I guess it is really just that one place | 18:35 |
| zzzeek | that something changes | 18:35 |
| geguileo | zzzeek: I believe we made the patch not affect anything not using lists/tuples | 18:35 |
| openstackgerrit | Merged openstack/cinder: Imported Translations from Zanata https://review.openstack.org/248544 | 18:35 |
| openstackgerrit | Merged openstack/cinder: Imported Translations from Zanata https://review.openstack.org/248544 | 18:35 |
| geguileo | zzzeek: And that the reason was for performance and to remove the possibility of breaking somebody elses code with this | 18:36 |
| zzzeek | geguileo: right. i know we were careful. its jsut for 1.0, I really hvae to check this a lot because this would be an exception to the general policy I've had in 1.0 of not adding any new features that change anything | 18:36 |
| zzzeek | geguileo: 1.0 had a shit-ton of regressions from 0.9, all of whcih came from changes that I made as carefully as we are making this one :) | 18:37 |
| geguileo | zzzeek: I understand :''-( | 18:37 |
| zzzeek | geguileo: its for The Project. I'll get it in, I've been travelling for 3 weeks so I finally have time for PRs | 18:37 |
| geguileo | zzzeek: It'd be great to see it get in | 18:38 |
| geguileo | because otherwise in Cinder we would have to generate manually a massive SQL query | 18:38 |
| geguileo | (at least I think it would be of considerable length) | 18:39 |
| zzzeek | geguileo: i see the gerrits so yes this isn't just one query i get that | 18:40 |
| geguileo | zzzeek: All I can ask is that you consider it | 18:40 |
| geguileo | zzzeek: But if it doesn't get in I'll look for a workaround in the meantime | 18:41 |
| zzzeek | geguileo: there's no other way to do what you need here | 18:41 |
| *** bardia has joined #openstack-cinder | 18:41 | |
| geguileo | zzzeek: Me generating the SQL myself :'-( | 18:41 |
| *** mriedem has left #openstack-cinder | 18:42 | |
| *** jerrygb has quit IRC | 18:42 | |
| *** mriedem has joined #openstack-cinder | 18:43 | |
| *** pots has joined #openstack-cinder | 18:44 | |
| *** jerrygb has joined #openstack-cinder | 18:46 | |
| *** boris-42 has quit IRC | 18:48 | |
| *** jerrygb has quit IRC | 18:51 | |
| dulek | e0ne: I was looking at https://review.openstack.org/247480/ and I don't understand any bit of what is going on in the python jobs… | 18:51 |
| e0ne | dulek: something went wrong with testscenarios and test generation | 18:52 |
| *** edtubill has joined #openstack-cinder | 18:53 | |
| e0ne | dulek: I'm going to fix it tomorrow morning | 18:53 |
| dulek | I'll look carefully, something similar may happen for oslo.scheduler. | 18:53 |
| e0ne | thanks :) | 18:54 |
| *** edtubill has quit IRC | 18:58 | |
| *** martyturner has quit IRC | 19:00 | |
| dulek | rhagarty_: Hi, I've seen you've been interested in https://review.openstack.org/#/c/245624/ | 19:01 |
| dulek | rhagarty_: Please take a look at my recent comment there. :) | 19:01 |
| *** jerrygb has joined #openstack-cinder | 19:03 | |
| *** ociuhandu has quit IRC | 19:12 | |
| *** diablo_rojo has joined #openstack-cinder | 19:12 | |
| *** merooney has quit IRC | 19:16 | |
| *** gouthamr has quit IRC | 19:16 | |
| *** edtubill has joined #openstack-cinder | 19:18 | |
| *** sgotliv has joined #openstack-cinder | 19:22 | |
| *** martyturner has joined #openstack-cinder | 19:24 | |
| *** IlyaG has joined #openstack-cinder | 19:30 | |
| *** salv-orlando has quit IRC | 19:34 | |
| *** gouthamr has joined #openstack-cinder | 19:35 | |
| *** amrith is now known as _amrith_ | 19:36 | |
| *** merooney has joined #openstack-cinder | 19:36 | |
| *** _amrith_ is now known as amrith | 19:36 | |
| *** angela-s has joined #openstack-cinder | 19:37 | |
| *** simondodsley_ has joined #openstack-cinder | 19:41 | |
| uberjay_ | ps mike, the second season of fargo has been pretty amazing so far | 19:42 |
| *** simondodsley_ is now known as simondodsley | 19:45 | |
| *** ntpttr has quit IRC | 19:45 | |
| *** ntpttr has joined #openstack-cinder | 19:46 | |
| *** ociuhandu has joined #openstack-cinder | 19:50 | |
| openstackgerrit | Jacob Gregor proposed openstack/cinder: Storwize: Split up __init__ into separate files https://review.openstack.org/240329 | 19:51 |
| uberjay_ | (ha ha, wrong channel, but still true!) | 19:52 |
| *** e0ne has quit IRC | 19:55 | |
| *** dims has joined #openstack-cinder | 19:56 | |
| *** asselin has joined #openstack-cinder | 20:02 | |
| *** asselin_ has joined #openstack-cinder | 20:03 | |
| *** jordanP has joined #openstack-cinder | 20:03 | |
| *** jordanP has quit IRC | 20:05 | |
| *** asselin_ has left #openstack-cinder | 20:09 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Changing PercentSynced to CopyState in isSynched https://review.openstack.org/246992 | 20:14 |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Fix for randomly selecting a portgroup https://review.openstack.org/243304 | 20:17 |
| *** asselin has quit IRC | 20:19 | |
| *** jistr has quit IRC | 20:23 | |
| *** IlyaG has quit IRC | 20:27 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Change naming convention for MV and SG for FAST https://review.openstack.org/247008 | 20:32 |
| *** e0ne has joined #openstack-cinder | 20:42 | |
| *** dustins has quit IRC | 20:47 | |
| *** gouthamr has quit IRC | 20:49 | |
| *** gouthamr has joined #openstack-cinder | 20:50 | |
| *** amrith is now known as _amrith_ | 20:56 | |
| *** salv-orlando has joined #openstack-cinder | 20:56 | |
| *** apoorvad has quit IRC | 21:00 | |
| *** gouthamr has quit IRC | 21:00 | |
| *** merooney has quit IRC | 21:00 | |
| *** crose has quit IRC | 21:03 | |
| *** apoorvad has joined #openstack-cinder | 21:05 | |
| *** e0ne has quit IRC | 21:08 | |
| *** Lee1092 has quit IRC | 21:09 | |
| *** krotscheck has quit IRC | 21:10 | |
| *** anteaya has quit IRC | 21:11 | |
| *** edtubill has quit IRC | 21:12 | |
| *** crose has joined #openstack-cinder | 21:12 | |
| *** jasondotstar has quit IRC | 21:12 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Brick add param documentation to connectors https://review.openstack.org/246023 | 21:15 |
| *** krotscheck has joined #openstack-cinder | 21:17 | |
| *** e0ne has joined #openstack-cinder | 21:17 | |
| *** jasondotstar has joined #openstack-cinder | 21:18 | |
| *** diegows has joined #openstack-cinder | 21:19 | |
| *** crose has quit IRC | 21:19 | |
| *** jerrygb has quit IRC | 21:22 | |
| *** jerrygb has joined #openstack-cinder | 21:26 | |
| *** anteaya has joined #openstack-cinder | 21:26 | |
| *** diablo_rojo has quit IRC | 21:27 | |
| *** vgridnev has quit IRC | 21:27 | |
| *** vgridnev has joined #openstack-cinder | 21:28 | |
| *** jerrygb has quit IRC | 21:31 | |
| *** david-lyle has quit IRC | 21:31 | |
| *** vgridnev has quit IRC | 21:32 | |
| *** krtaylor has quit IRC | 21:33 | |
| *** david-lyle has joined #openstack-cinder | 21:34 | |
| *** martyturner has quit IRC | 21:35 | |
| *** kfarr has quit IRC | 21:38 | |
| *** jordant has joined #openstack-cinder | 21:41 | |
| *** dims_ has joined #openstack-cinder | 21:43 | |
| *** e0ne has quit IRC | 21:43 | |
| *** thangp has quit IRC | 21:43 | |
| *** dims has quit IRC | 21:44 | |
| *** EinstCrazy has joined #openstack-cinder | 21:46 | |
| *** krtaylor has joined #openstack-cinder | 21:46 | |
| sgundur | jgriffith: Hi John, per ur bug https://bugs.launchpad.net/cinder/+bug/1508223 | 21:47 |
| openstack | Launchpad bug 1508223 in Cinder "driver initialized checks in manager should go away" [Low,Triaged] - Assigned to Shashi (shashirekha-j-gundur) | 21:47 |
| sgundur | I was wondering if u could clarify the below 2 points | 21:47 |
| sgundur | 1. u mentioned in the bug description that - is_initialized call in "most" cases is kind of pointless anyway; | 21:47 |
| sgundur | pls let me know which are these cases ? | 21:47 |
| sgundur | 2. I think that none of the utils.require_driver_initialized calls in manager.py are going through cinder/scheduler ; | 21:47 |
| sgundur | so how do we make sure all the calls to go through the scheduler ? | 21:47 |
| *** gouthamr has joined #openstack-cinder | 21:48 | |
| *** bswartz has quit IRC | 21:49 | |
| *** EinstCrazy has quit IRC | 21:51 | |
| *** david-lyle has quit IRC | 21:55 | |
| *** jordant1 has joined #openstack-cinder | 21:58 | |
| *** jordant has quit IRC | 21:58 | |
| *** harlowja_ has joined #openstack-cinder | 21:59 | |
| *** harlowja has quit IRC | 21:59 | |
| *** sgundur has quit IRC | 21:59 | |
| *** krtaylor has quit IRC | 22:03 | |
| *** david-lyle has joined #openstack-cinder | 22:03 | |
| *** [1]Thelo has joined #openstack-cinder | 22:03 | |
| *** david-lyle has quit IRC | 22:04 | |
| *** sgundur has joined #openstack-cinder | 22:05 | |
| *** Thelo has quit IRC | 22:05 | |
| *** [1]Thelo is now known as Thelo | 22:05 | |
| *** julim_ has quit IRC | 22:09 | |
| *** jerrygb has joined #openstack-cinder | 22:11 | |
| openstackgerrit | Bardia Keyoumarsi proposed openstack/cinder: Volume driver for Coho Data storage solutions https://review.openstack.org/246690 | 22:14 |
| *** salv-orl_ has joined #openstack-cinder | 22:15 | |
| *** krtaylor has joined #openstack-cinder | 22:15 | |
| *** JoseMello has joined #openstack-cinder | 22:15 | |
| *** _amrith_ is now known as amrith | 22:15 | |
| *** salv-orlando has quit IRC | 22:19 | |
| *** rcernin has quit IRC | 22:20 | |
| *** david-lyle has joined #openstack-cinder | 22:21 | |
| *** david-lyle has quit IRC | 22:25 | |
| openstackgerrit | Bardia Keyoumarsi proposed openstack/cinder: Volume driver for Coho Data storage solutions https://review.openstack.org/246690 | 22:26 |
| jgriffith | sgundur: couple of things (initialize_connection aside) | 22:27 |
| jgriffith | sgundur: it should seem a bit off that we have all of that code creeping into every command that we're doing something wrong | 22:27 |
| jgriffith | sgundur: my opinion is that in most cases (including init connection) that the driver should respond with an error if the backend isn't accessible | 22:28 |
| jgriffith | sgundur: and that error should be handled/dealt with in whatever manner | 22:28 |
| jgriffith | sgundur: drivers should be written "better" | 22:28 |
| jgriffith | sgundur: and handle things better, rather than just blowing up and crashing the volume service | 22:28 |
| jgriffith | sgundur: IMHO that's what should be fixed, instead of sprinkling driver init checks everywhere | 22:29 |
| jgriffith | sgundur: the comment about the scheduler.... | 22:29 |
| jgriffith | sgundur: if drivers can't be left to handle things properly and we have to put in more controls higher up (like what the checks do) | 22:30 |
| jgriffith | sgundur: then I'd argue that every call to a backend should then go through the scheduler and use it's view of the backend based on stats update | 22:30 |
| jgriffith | sgundur: I don't think that's necessary... I think that in most cases that calls are pointless and that's why I filed the bug | 22:31 |
| *** baumann has left #openstack-cinder | 22:31 | |
| *** jwcroppe has quit IRC | 22:31 | |
| *** lpabon has quit IRC | 22:33 | |
| *** daneyon_ has joined #openstack-cinder | 22:36 | |
| *** daneyon has quit IRC | 22:39 | |
| *** rushil_ has quit IRC | 22:39 | |
| *** salv-orlando has joined #openstack-cinder | 22:39 | |
| *** salv-orl_ has quit IRC | 22:42 | |
| *** mriedem is now known as mriedem_away | 22:46 | |
| *** dims_ has quit IRC | 22:55 | |
| *** ntpttr has quit IRC | 22:57 | |
| jgriffith | sgundur: I update my comment; it would appear the idea isn't very popular though :( | 23:01 |
| *** jgregor has quit IRC | 23:01 | |
| jgriffith | sgundur: perhaps I'm wrong... you might want to check with others before diving too deep in to it | 23:01 |
| *** eharney has quit IRC | 23:02 | |
| openstackgerrit | Jon Bernard proposed openstack/os-brick: Allow RBDClient to be used from a with-statement https://review.openstack.org/248936 | 23:02 |
| *** david-lyle has joined #openstack-cinder | 23:03 | |
| *** david-lyle has quit IRC | 23:05 | |
| *** ntpttr has joined #openstack-cinder | 23:13 | |
| *** david-lyle has joined #openstack-cinder | 23:23 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Change naming convention for MV and SG for FAST https://review.openstack.org/247008 | 23:24 |
| *** dims_ has joined #openstack-cinder | 23:25 | |
| *** david-lyle has quit IRC | 23:25 | |
| *** dims_ has quit IRC | 23:26 | |
| *** ntpttr has quit IRC | 23:32 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Extend Volume for VMAX3 https://review.openstack.org/248941 | 23:34 |
| *** EinstCrazy has joined #openstack-cinder | 23:34 | |
| *** diegows has quit IRC | 23:35 | |
| *** david-lyle has joined #openstack-cinder | 23:35 | |
| jgriffith | hemna: marked as opinion, not because I agree :) But because it's certainly not a big deal or worth taking up any more time | 23:36 |
| jgriffith | hemna: anyway, thanks for the input on it | 23:37 |
| *** jgriffith is now known as jgriffith_away | 23:38 | |
| *** simondodsley has quit IRC | 23:38 | |
| hemna | jgriffith_away, np | 23:39 |
| *** sgundur has quit IRC | 23:40 | |
| *** david-lyle has quit IRC | 23:41 | |
| *** david-ly_ has joined #openstack-cinder | 23:41 | |
| *** sgundur has joined #openstack-cinder | 23:41 | |
| *** EinstCrazy has quit IRC | 23:45 | |
| *** pots2 has joined #openstack-cinder | 23:48 | |
| *** pots has quit IRC | 23:51 | |
| *** sgundur has quit IRC | 23:52 | |
| openstackgerrit | Ryan McNair proposed openstack/python-cinderclient: Fix v2 qos-key command https://review.openstack.org/248944 | 23:52 |
| *** sgundur has joined #openstack-cinder | 23:53 | |
| *** davechen has quit IRC | 23:53 | |
| *** davechen has joined #openstack-cinder | 23:54 | |
| *** diegows has joined #openstack-cinder | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!