*** yangyapeng has quit IRC | 00:01 | |
*** IlyaG has joined #openstack-cinder | 00:08 | |
*** karthikp_ has joined #openstack-cinder | 00:15 | |
*** kaisers1 has joined #openstack-cinder | 00:17 | |
*** kaisers has quit IRC | 00:19 | |
*** timcl has joined #openstack-cinder | 00:20 | |
*** ducttape_ has quit IRC | 00:22 | |
*** Suyi has quit IRC | 00:23 | |
*** timcl has quit IRC | 00:25 | |
*** mtanino has quit IRC | 00:27 | |
*** raunak has quit IRC | 00:28 | |
*** salv-orlando has quit IRC | 00:53 | |
*** salv-orlando has joined #openstack-cinder | 00:53 | |
*** raunak has joined #openstack-cinder | 00:54 | |
*** jwcroppe has joined #openstack-cinder | 00:57 | |
*** cheneydc has joined #openstack-cinder | 00:58 | |
*** markvoelker has joined #openstack-cinder | 01:13 | |
*** yangyapeng has joined #openstack-cinder | 01:14 | |
*** markvoelker has quit IRC | 01:18 | |
*** Apoorva has quit IRC | 01:23 | |
*** diogogmt has joined #openstack-cinder | 01:24 | |
openstackgerrit | yuyafei proposed openstack/cinder: SolidFire driver should return ceiling of volume size https://review.openstack.org/326776 | 01:24 |
---|---|---|
*** frontrunner has quit IRC | 01:31 | |
openstackgerrit | haobing proposed openstack/cinder: Allow remove description and name for consisgroup https://review.openstack.org/324465 | 01:32 |
*** EinstCrazy has joined #openstack-cinder | 01:35 | |
*** gouthamr has quit IRC | 01:40 | |
openstackgerrit | LisaLi proposed openstack/cinder: Handle quota exceed exception https://review.openstack.org/281666 | 01:41 |
*** EinstCra_ has joined #openstack-cinder | 01:41 | |
*** EinstCr__ has joined #openstack-cinder | 01:44 | |
*** EinstCrazy has quit IRC | 01:44 | |
*** EinstCra_ has quit IRC | 01:46 | |
*** Lee1092 has joined #openstack-cinder | 01:49 | |
*** IlyaG has quit IRC | 01:51 | |
*** mriedem has quit IRC | 01:52 | |
*** akshai has quit IRC | 02:01 | |
*** iceyao has joined #openstack-cinder | 02:04 | |
*** Julien-zte has joined #openstack-cinder | 02:08 | |
*** Julien-zte has quit IRC | 02:11 | |
*** akshai has joined #openstack-cinder | 02:12 | |
*** winston-d_ has joined #openstack-cinder | 02:18 | |
*** Suyi has joined #openstack-cinder | 02:22 | |
*** raunak has quit IRC | 02:22 | |
*** ducttape_ has joined #openstack-cinder | 02:23 | |
*** karthikp_ has quit IRC | 02:25 | |
*** karthikp_ has joined #openstack-cinder | 02:26 | |
*** iceyao has quit IRC | 02:28 | |
*** iceyao has joined #openstack-cinder | 02:29 | |
*** iceyao_ has joined #openstack-cinder | 02:31 | |
*** sdake has joined #openstack-cinder | 02:32 | |
*** bardia has quit IRC | 02:32 | |
*** ducttape_ has quit IRC | 02:33 | |
*** mtanino has joined #openstack-cinder | 02:34 | |
*** sdake has quit IRC | 02:34 | |
*** iceyao has quit IRC | 02:34 | |
*** xyang has joined #openstack-cinder | 02:37 | |
openstackgerrit | liuke proposed openstack/cinder: Huawei: Support reporting disk type of pool https://review.openstack.org/324163 | 02:42 |
*** openstackgerrit has quit IRC | 02:47 | |
*** openstackgerrit has joined #openstack-cinder | 02:48 | |
*** ducttape_ has joined #openstack-cinder | 02:48 | |
*** sheel has joined #openstack-cinder | 02:49 | |
*** dave-mccowan has quit IRC | 02:54 | |
*** karthikp_ has quit IRC | 02:58 | |
openstackgerrit | wuyuting proposed openstack/cinder: Fix infortrend driver bug: cloned with src vol size https://review.openstack.org/326792 | 03:00 |
*** raunak has joined #openstack-cinder | 03:07 | |
*** mtanino has quit IRC | 03:08 | |
*** EinstCr__ has quit IRC | 03:13 | |
*** ducttape_ has quit IRC | 03:24 | |
*** sage has quit IRC | 03:24 | |
*** sage has joined #openstack-cinder | 03:25 | |
*** karthikp_ has joined #openstack-cinder | 03:26 | |
*** sdake has joined #openstack-cinder | 03:28 | |
*** diogogmt has quit IRC | 03:32 | |
*** links has joined #openstack-cinder | 03:35 | |
openstackgerrit | Peter Wang proposed openstack/cinder: VNX: New Cinder driver in Newton https://review.openstack.org/312427 | 03:37 |
*** enriquetaso has quit IRC | 03:43 | |
*** EinstCrazy has joined #openstack-cinder | 03:44 | |
*** julim has quit IRC | 03:45 | |
*** EinstCrazy has quit IRC | 03:47 | |
*** lprice1 has joined #openstack-cinder | 03:47 | |
*** lprice has quit IRC | 03:47 | |
*** raunak has quit IRC | 03:49 | |
*** sdake has quit IRC | 03:53 | |
*** ebalduf_ has joined #openstack-cinder | 03:53 | |
*** jwcroppe has quit IRC | 03:54 | |
*** jwcroppe has joined #openstack-cinder | 03:55 | |
*** jwcroppe_ has joined #openstack-cinder | 03:56 | |
openstackgerrit | LisaLi proposed openstack/cinder: DO NOT MERGE https://review.openstack.org/282169 | 03:58 |
*** jwcroppe has quit IRC | 03:59 | |
openstackgerrit | Wilson Liu proposed openstack/cinder-specs: Migrate volume between backends in an async way https://review.openstack.org/312853 | 04:01 |
*** sgotliv has joined #openstack-cinder | 04:04 | |
*** karthikp_ has quit IRC | 04:09 | |
*** karthikp_ has joined #openstack-cinder | 04:09 | |
*** raunak has joined #openstack-cinder | 04:10 | |
*** EinstCrazy has joined #openstack-cinder | 04:22 | |
*** salv-orl_ has joined #openstack-cinder | 04:24 | |
*** ducttape_ has joined #openstack-cinder | 04:25 | |
*** salv-orlando has quit IRC | 04:27 | |
*** ducttape_ has quit IRC | 04:30 | |
kwathore | hi, can someone please review my patches:https://review.openstack.org/#/c/315376/ and https://review.openstack.org/#/c/319207/ | 04:40 |
*** garthb has joined #openstack-cinder | 04:42 | |
*** savihou has joined #openstack-cinder | 04:50 | |
*** garthb has quit IRC | 04:51 | |
*** adisky has joined #openstack-cinder | 04:54 | |
*** ChubYann has quit IRC | 04:59 | |
*** bardia has joined #openstack-cinder | 05:01 | |
*** sdake has joined #openstack-cinder | 05:02 | |
*** salv-orl_ has quit IRC | 05:03 | |
*** salv-orlando has joined #openstack-cinder | 05:03 | |
*** ebalduf_ has quit IRC | 05:05 | |
*** nikeshm has quit IRC | 05:08 | |
*** GB21 has joined #openstack-cinder | 05:09 | |
openstackgerrit | xiexs proposed openstack/python-cinderclient: Add max-width feature to limit the width of the printed table https://review.openstack.org/319044 | 05:09 |
*** Gao has joined #openstack-cinder | 05:12 | |
*** Lallana has quit IRC | 05:12 | |
openstackgerrit | Peter Wang proposed openstack/cinder: VNX: New Cinder driver in Newton https://review.openstack.org/312427 | 05:13 |
*** karthikp_ has quit IRC | 05:13 | |
*** savihou has quit IRC | 05:15 | |
*** savihou has joined #openstack-cinder | 05:16 | |
*** Poornima has joined #openstack-cinder | 05:17 | |
*** savihou has quit IRC | 05:17 | |
*** sgotliv_ has joined #openstack-cinder | 05:19 | |
*** sgotliv has quit IRC | 05:20 | |
*** savihou has joined #openstack-cinder | 05:21 | |
*** sdake has quit IRC | 05:32 | |
*** lpetrut has joined #openstack-cinder | 05:38 | |
*** sgotliv_ has quit IRC | 05:41 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Test config options as same as production enviroment(1/2) https://review.openstack.org/253328 | 05:42 |
*** GB21 has quit IRC | 05:44 | |
*** Suyi has quit IRC | 05:48 | |
*** pgadiya has joined #openstack-cinder | 05:48 | |
*** bardia has quit IRC | 05:49 | |
*** ManishD has joined #openstack-cinder | 05:49 | |
*** bardia has joined #openstack-cinder | 05:49 | |
*** avishay has joined #openstack-cinder | 05:55 | |
*** mtanino has joined #openstack-cinder | 05:56 | |
*** EinstCrazy has quit IRC | 05:59 | |
*** EinstCrazy has joined #openstack-cinder | 05:59 | |
*** EinstCrazy has quit IRC | 05:59 | |
*** yangyape_ has joined #openstack-cinder | 06:01 | |
*** yangyapeng has quit IRC | 06:02 | |
*** EinstCrazy has joined #openstack-cinder | 06:05 | |
*** GB21 has joined #openstack-cinder | 06:06 | |
*** lifeless has quit IRC | 06:13 | |
*** rcernin has joined #openstack-cinder | 06:13 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Test config options as same as production enviroment(2/2) https://review.openstack.org/326843 | 06:16 |
*** e0ne has joined #openstack-cinder | 06:21 | |
*** ducttape_ has joined #openstack-cinder | 06:26 | |
*** arecknag has joined #openstack-cinder | 06:27 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/os-brick: Make RBDImageMetadata and RBDVolumeIOWrapper re-usable https://review.openstack.org/326696 | 06:30 |
*** faizy has joined #openstack-cinder | 06:30 | |
*** bardia has quit IRC | 06:31 | |
*** ducttape_ has quit IRC | 06:31 | |
*** bardia has joined #openstack-cinder | 06:31 | |
*** avishay has quit IRC | 06:31 | |
*** lpetrut has quit IRC | 06:37 | |
openstackgerrit | xiexs proposed openstack/cinder: Make examples consistent with actul API https://review.openstack.org/326852 | 06:39 |
*** lpetrut has joined #openstack-cinder | 06:45 | |
*** bardia has quit IRC | 06:45 | |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: [WIP]enhanche error message [1/5] This change is intend to implement this approved guideline[1]. To return the error message to users, we define an error-code when an exception is raised in cinder-api layer. And this error-code is added to the raised exce https://review.openstack.org/296304 | 06:46 |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: [WIP]enhanche error message [1/5] https://review.openstack.org/296304 | 06:47 |
*** openstackgerrit has quit IRC | 06:48 | |
*** gcb has quit IRC | 06:48 | |
*** openstackgerrit has joined #openstack-cinder | 06:49 | |
*** raunak has quit IRC | 06:49 | |
*** belmoreira has joined #openstack-cinder | 06:49 | |
*** lpetrut has quit IRC | 06:49 | |
*** Julien-zte has joined #openstack-cinder | 06:53 | |
openstackgerrit | yuyafei proposed openstack/cinder: Virtual size should get ceiling of image https://review.openstack.org/326859 | 06:53 |
*** ManishD has quit IRC | 06:54 | |
*** sdake has joined #openstack-cinder | 06:54 | |
openstackgerrit | yuyafei proposed openstack/cinder: Virtual size should get ceiling of image https://review.openstack.org/326859 | 06:57 |
*** EinstCra_ has joined #openstack-cinder | 06:59 | |
*** tesseract has joined #openstack-cinder | 07:00 | |
*** liverpooler has joined #openstack-cinder | 07:01 | |
*** EinstCrazy has quit IRC | 07:02 | |
*** salv-orlando has quit IRC | 07:02 | |
*** salv-orlando has joined #openstack-cinder | 07:03 | |
*** EinstCra_ has quit IRC | 07:05 | |
*** EinstCrazy has joined #openstack-cinder | 07:05 | |
*** e0ne has quit IRC | 07:06 | |
*** ManishD has joined #openstack-cinder | 07:10 | |
*** sdake has quit IRC | 07:10 | |
*** gcb has joined #openstack-cinder | 07:13 | |
*** Gao has quit IRC | 07:16 | |
*** nkrinner has joined #openstack-cinder | 07:16 | |
*** Gao has joined #openstack-cinder | 07:17 | |
*** savihou has quit IRC | 07:18 | |
*** mtanino has quit IRC | 07:19 | |
openstackgerrit | Philipp Marek proposed openstack/cinder: DRBD: Allow to set options for new resources https://review.openstack.org/326377 | 07:24 |
openstackgerrit | huanan proposed openstack/cinder: Huawei: Support huawei consistency group https://review.openstack.org/260930 | 07:25 |
*** mjura has joined #openstack-cinder | 07:25 | |
*** salv-orlando has quit IRC | 07:32 | |
*** salv-orlando has joined #openstack-cinder | 07:32 | |
*** nkrinner has quit IRC | 07:37 | |
*** sgotliv_ has joined #openstack-cinder | 07:37 | |
*** nkrinner has joined #openstack-cinder | 07:38 | |
openstackgerrit | huyang proposed openstack/cinder: Huawei: Support iSCSI configuration in replication feature https://review.openstack.org/325682 | 07:41 |
openstackgerrit | LisaLi proposed openstack/os-brick: DO NOT MERGE https://review.openstack.org/326886 | 07:43 |
openstackgerrit | huyang proposed openstack/cinder: Huawei: Support iSCSI configuration in replication feature https://review.openstack.org/325682 | 07:45 |
openstackgerrit | huyang proposed openstack/cinder: Huawei: Support iSCSI configuration in replication feature https://review.openstack.org/325682 | 07:46 |
*** openstackgerrit has quit IRC | 07:48 | |
*** openstackgerrit has joined #openstack-cinder | 07:49 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-cinder | 08:02 | |
*** karthikp_ has joined #openstack-cinder | 08:09 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
*** salv-orlando has quit IRC | 08:10 | |
*** karthikp_ has quit IRC | 08:10 | |
*** salv-orlando has joined #openstack-cinder | 08:11 | |
*** houming has joined #openstack-cinder | 08:14 | |
*** yangyape_ has quit IRC | 08:15 | |
*** avishay has joined #openstack-cinder | 08:15 | |
*** EinstCrazy has quit IRC | 08:15 | |
*** EinstCrazy has joined #openstack-cinder | 08:18 | |
*** yangyapeng has joined #openstack-cinder | 08:19 | |
*** Julien-zte has quit IRC | 08:19 | |
*** houming has quit IRC | 08:19 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add error messages to conditional updates devref https://review.openstack.org/326324 | 08:23 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add docstrings to CG and CG Snapshot filters https://review.openstack.org/326323 | 08:23 |
*** lpetrut has joined #openstack-cinder | 08:25 | |
*** aarefiev_ is now known as aarefiev | 08:26 | |
openstackgerrit | Falk Reimann proposed openstack/cinder: Add Auth Version 3 support in Swift Backup Driver https://review.openstack.org/247977 | 08:27 |
*** ducttape_ has joined #openstack-cinder | 08:27 | |
*** ducttape_ has quit IRC | 08:33 | |
*** tyrola has joined #openstack-cinder | 08:34 | |
*** guardianJ has joined #openstack-cinder | 08:37 | |
*** guardianJ has left #openstack-cinder | 08:38 | |
*** EinstCrazy has quit IRC | 08:43 | |
*** EinstCrazy has joined #openstack-cinder | 08:43 | |
*** EinstCrazy has quit IRC | 08:45 | |
*** EinstCrazy has joined #openstack-cinder | 08:47 | |
*** openstackgerrit has quit IRC | 08:48 | |
*** openstackgerrit has joined #openstack-cinder | 08:49 | |
*** e0ne has joined #openstack-cinder | 08:50 | |
*** ManishD has quit IRC | 08:51 | |
*** GB21 has quit IRC | 08:53 | |
*** iceyao_ has quit IRC | 08:53 | |
*** bbhavsar has joined #openstack-cinder | 08:56 | |
lixiaoy1 | hemna: Hi Walter, please help to review the patch https://review.openstack.org/#/c/247372/ . And I added CI test which succeeds | 08:59 |
*** mvk has quit IRC | 09:03 | |
openstackgerrit | Lallana proposed openstack/cinder: huawei-iscsi-multipath-support https://review.openstack.org/324973 | 09:06 |
*** tyrola has quit IRC | 09:07 | |
*** GB21 has joined #openstack-cinder | 09:09 | |
*** yhayashi has quit IRC | 09:09 | |
flip214 | If I got a -1, but cinder core says my patch is good (via +2), is it enough for some other core to add a +2 to get it merged? Or do I need to get rid of the -1 before, in some way or other? | 09:11 |
flip214 | https://review.openstack.org/#/c/320328 | 09:12 |
*** sdake has joined #openstack-cinder | 09:16 | |
*** EinstCrazy has quit IRC | 09:18 | |
*** GB21 has quit IRC | 09:18 | |
*** belmoreira has quit IRC | 09:19 | |
*** EinstCrazy has joined #openstack-cinder | 09:21 | |
e0ne | flip214: hi. it's always depended on why it has -1 | 09:23 |
flip214 | because of an invalid (declared so by a cinder core) comment | 09:24 |
flip214 | e0ne: ^^ | 09:24 |
dulek | flip214: Give me few minutes, I'll try to make sure if provider_id is always there and review that. | 09:24 |
*** abhishek has joined #openstack-cinder | 09:27 | |
flip214 | dulek: thank you very much. eric and john left their comments in, and they conclude that it's always there. | 09:28 |
dulek | flip214: You're welcome. :) | 09:33 |
flip214 | dulek: thank you! | 09:33 |
flip214 | in case someone's idle, https://review.openstack.org/#/c/326377/1 would like a review too. just askin' ;) | 09:34 |
*** mvk has joined #openstack-cinder | 09:34 | |
*** GB21 has joined #openstack-cinder | 09:42 | |
openstackgerrit | Lallana proposed openstack/cinder: huawei-iscsi-multipath-support https://review.openstack.org/324973 | 09:42 |
*** belmoreira has joined #openstack-cinder | 09:48 | |
*** sdake has quit IRC | 09:49 | |
openstackgerrit | Bin Zhou proposed openstack/cinder: refactor QemuImgInfo https://review.openstack.org/326961 | 09:50 |
*** iceyao has joined #openstack-cinder | 09:53 | |
*** markvoelker has joined #openstack-cinder | 09:53 | |
openstackgerrit | huyang proposed openstack/cinder: Huawei: Support iSCSI configuration in replication feature https://review.openstack.org/325682 | 09:56 |
*** cheneydc has quit IRC | 09:56 | |
*** iceyao has quit IRC | 09:58 | |
*** nkrinner has quit IRC | 10:03 | |
openstackgerrit | haobing proposed openstack/cinder: Allow remove description and name for consisgroup https://review.openstack.org/324465 | 10:07 |
*** yangyapeng has quit IRC | 10:09 | |
*** mjura has quit IRC | 10:12 | |
*** mjura has joined #openstack-cinder | 10:15 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/os-brick: Make RBDImageMetadata and RBDVolumeIOWrapper re-usable https://review.openstack.org/326696 | 10:16 |
*** nkrinner has joined #openstack-cinder | 10:19 | |
*** salv-orl_ has joined #openstack-cinder | 10:25 | |
*** salv-orlando has quit IRC | 10:28 | |
*** ducttape_ has joined #openstack-cinder | 10:29 | |
*** winston-d_ has quit IRC | 10:29 | |
*** salv-orl_ has quit IRC | 10:34 | |
*** ducttape_ has quit IRC | 10:34 | |
*** salv-orlando has joined #openstack-cinder | 10:34 | |
*** lifeless has joined #openstack-cinder | 10:35 | |
*** winston-d_ has joined #openstack-cinder | 10:38 | |
*** sdague has joined #openstack-cinder | 10:42 | |
*** lifeless has quit IRC | 10:47 | |
*** lifeless_ has joined #openstack-cinder | 10:47 | |
*** tesseract has quit IRC | 10:50 | |
*** tesseract has joined #openstack-cinder | 10:51 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Re-use RBDImageMetadata and RBDVolumeIOWrapper from os-brick https://review.openstack.org/326999 | 11:02 |
*** afazekas is now known as __afazekas | 11:05 | |
*** e0ne_ has joined #openstack-cinder | 11:11 | |
*** e0ne has quit IRC | 11:13 | |
openstackgerrit | Merged openstack/cinder: DRBD: ensure_export() might not have 'provider_location' https://review.openstack.org/320328 | 11:14 |
*** diogogmt has joined #openstack-cinder | 11:27 | |
*** ducttape_ has joined #openstack-cinder | 11:38 | |
*** tpsilva has joined #openstack-cinder | 11:41 | |
openstackgerrit | Merged openstack/cinder: VMware: Remove unused methods https://review.openstack.org/325987 | 11:43 |
*** sdake has joined #openstack-cinder | 11:44 | |
*** gcb has quit IRC | 11:46 | |
*** alyson_ has joined #openstack-cinder | 11:48 | |
*** __afazekas is now known as afazekas | 11:49 | |
openstackgerrit | Alyson proposed openstack/cinder: HNAS: Add support for consistency groups https://review.openstack.org/327043 | 11:50 |
*** ducttape_ has quit IRC | 11:50 | |
*** adrianofr has joined #openstack-cinder | 11:51 | |
*** ducttape_ has joined #openstack-cinder | 11:51 | |
*** faizy has quit IRC | 11:53 | |
*** ducttape_ has quit IRC | 11:55 | |
*** markvoelker has quit IRC | 11:56 | |
*** salv-orlando has quit IRC | 11:59 | |
*** salv-orlando has joined #openstack-cinder | 12:00 | |
*** diogogmt has quit IRC | 12:05 | |
*** pwehrle has joined #openstack-cinder | 12:09 | |
*** markvoelker has joined #openstack-cinder | 12:12 | |
openstackgerrit | Michal Dulko proposed openstack/cinder: Update manage_existing to use volume object https://review.openstack.org/258132 | 12:13 |
*** sdake has quit IRC | 12:13 | |
*** EinstCrazy has quit IRC | 12:14 | |
*** EinstCrazy has joined #openstack-cinder | 12:14 | |
*** frontrunner has joined #openstack-cinder | 12:14 | |
*** salv-orl_ has joined #openstack-cinder | 12:15 | |
openstackgerrit | Michal Dulko proposed openstack/cinder: Update manage_existing to use volume object https://review.openstack.org/258132 | 12:16 |
*** EinstCrazy has quit IRC | 12:19 | |
*** salv-orlando has quit IRC | 12:19 | |
*** aurelien__ has joined #openstack-cinder | 12:20 | |
*** aurelien__ has quit IRC | 12:21 | |
*** GB21 has quit IRC | 12:23 | |
*** mriedem has joined #openstack-cinder | 12:24 | |
abhishek | hi Michal: could you please review https://review.openstack.org/287606 patch when you get time? | 12:30 |
*** itzdilip has joined #openstack-cinder | 12:30 | |
*** lprice1 has quit IRC | 12:33 | |
abhishek | hi dulek: could you please review https://review.openstack.org/287606 patch when you get time? | 12:33 |
*** avishay has quit IRC | 12:33 | |
dulek | abhishek: Tab opened. ;) | 12:34 |
*** lprice has joined #openstack-cinder | 12:34 | |
abhishek | dulek: thank you :) | 12:34 |
*** diablo_rojo has joined #openstack-cinder | 12:35 | |
*** pgadiya has quit IRC | 12:40 | |
*** eharney has joined #openstack-cinder | 12:42 | |
*** ducttape_ has joined #openstack-cinder | 12:44 | |
*** itzdilip has quit IRC | 12:44 | |
*** avishay has joined #openstack-cinder | 12:49 | |
*** julim has joined #openstack-cinder | 12:55 | |
*** timcl has joined #openstack-cinder | 12:57 | |
*** edmondsw has joined #openstack-cinder | 13:00 | |
*** gouthamr has joined #openstack-cinder | 13:02 | |
*** links has quit IRC | 13:03 | |
*** cknight has joined #openstack-cinder | 13:03 | |
*** ebalduf_ has joined #openstack-cinder | 13:03 | |
*** ducttape_ has quit IRC | 13:04 | |
*** rlrossit has joined #openstack-cinder | 13:08 | |
*** Poornima has quit IRC | 13:09 | |
*** dustins has joined #openstack-cinder | 13:16 | |
*** merooney has joined #openstack-cinder | 13:16 | |
*** links has joined #openstack-cinder | 13:18 | |
*** porrua has joined #openstack-cinder | 13:21 | |
*** sheel has quit IRC | 13:25 | |
*** akerr has joined #openstack-cinder | 13:31 | |
*** bbhavsar has quit IRC | 13:32 | |
*** abhishek has quit IRC | 13:41 | |
*** jungleboyj has joined #openstack-cinder | 13:43 | |
*** wanghao_ has joined #openstack-cinder | 13:45 | |
scottda | eharney: I believe I addressed your concerns for this patch. Would you have another look? https://review.openstack.org/#/c/303627/ | 13:46 |
eharney | scottda: will do | 13:46 |
*** asselin_ has joined #openstack-cinder | 13:48 | |
*** diablo_rojo has quit IRC | 13:49 | |
scottda | Looks like openstack client doesn't support token-get, and installing keysoneclient has failures. Does anyone have a good way to get a token in current devstack? | 13:52 |
smcginnis | scottda: I thought I've done that with osc? | 13:52 |
smcginnis | Maybe I'm just mixing things up though. Looking... | 13:53 |
scottda | smcginnis: Maybe I just don't know the proper syntax... | 13:53 |
*** avishay has quit IRC | 13:53 | |
*** diablo_rojo has joined #openstack-cinder | 13:53 | |
*** zhonghua-lee has joined #openstack-cinder | 13:53 | |
smcginnis | scottda: You could go hardcore and do it with curl: http://docs.openstack.org/developer/keystone/api_curl_examples.html | 13:53 |
scottda | ahhh...well "openstack token issue" works | 13:54 |
scottda | I guess that's what I was looking for. | 13:54 |
smcginnis | scottda: That's it! | 13:54 |
smcginnis | Yeah, just found this: http://docs.openstack.org/mitaka/install-guide-ubuntu/keystone-verify.html | 13:54 |
*** ametts has joined #openstack-cinder | 14:02 | |
*** Julien-zte has joined #openstack-cinder | 14:02 | |
*** timcl1 has joined #openstack-cinder | 14:03 | |
*** ebalduf_ has quit IRC | 14:04 | |
*** ducttape_ has joined #openstack-cinder | 14:05 | |
*** avishay has joined #openstack-cinder | 14:05 | |
*** timcl has quit IRC | 14:05 | |
*** rlrossit has quit IRC | 14:09 | |
*** ducttape_ has quit IRC | 14:09 | |
*** karthikp_ has joined #openstack-cinder | 14:10 | |
*** rlrossit has joined #openstack-cinder | 14:12 | |
*** tomoiaga has joined #openstack-cinder | 14:14 | |
tomoiaga | deleting an instance with a volume attached will keep the volum in the "in use" state. Is this by design ? | 14:15 |
*** erlon has joined #openstack-cinder | 14:21 | |
scottda | tomoiaga: No | 14:22 |
scottda | tomoiaga: Deleting an instance with a volume attached should result in the volume being detached and available. | 14:22 |
*** lixiaoy1 has quit IRC | 14:23 | |
*** lixiaoy1 has joined #openstack-cinder | 14:24 | |
tomoiaga | scottda: I thought so too, and I keep looking at the logs for some time now and tried different things but I see no issues, everything seems ok except the fact that the volume (attached, not a boot volume, I use local storage) remains in use with attachements to the deleted instance. | 14:24 |
tomoiaga | scottda: do you perhaps know how a volume should be changed (through notifications or direct api call from nova to cinder) ? I'd like to avoid browsing the source :) | 14:25 |
*** dave-mccowan has joined #openstack-cinder | 14:26 | |
scottda | tomoiaga: What version of openstack are you using? I just tested and it definitely works with current devstack. | 14:26 |
tomoiaga | scottda: Mitaka (ubuntu packages) | 14:26 |
scottda | tomoiaga: Nova calls cinder to detach the volume during the process of deleting the instance. | 14:26 |
*** mriedem1 has joined #openstack-cinder | 14:27 | |
*** mriedem has quit IRC | 14:27 | |
tomoiaga | scottda: I'll try to look through the source and insert some logs where the volume should be detached, to see what happens (default logs are clean, not sure if I should see any message related to volume detaching) | 14:28 |
*** savihou has joined #openstack-cinder | 14:28 | |
*** diablo_rojo has quit IRC | 14:29 | |
*** diablo_rojo has joined #openstack-cinder | 14:29 | |
*** links has quit IRC | 14:29 | |
scottda | tomoiaga: Cinder API logs should have something. It will end with: | 14:29 |
scottda | 2016-06-08 08:25:22.452 INFO cinder.volume.api [req-65877ba3-9d7d-44fa-a193-11aeb08c17ef 79e36e33a4a4457bad304455422611e4 a57c84a312a14f199662ca9bc852bed6] Detach volume completed successfully. | 14:29 |
scottda | But, then, you are probably missing that :) | 14:30 |
scottda | tomoiaga: Maybe look in c-api for: | 14:30 |
scottda | 2016-06-08 08:25:21.735 DEBUG cinder.api.openstack.wsgi [req-65877ba3-9d7d-44fa-a193-11aeb08c17ef 79e36e33a4a4457bad304455422611e4 a57c84a312a14f199662ca9bc852bed6] Action body: {"os-detach": {"attachment_id": null}} from (pid=19762) get_method /opt/stack/cinder/cinder/api/openstack/wsgi.py:988 | 14:30 |
tomoiaga | scottda: thanks, that really helps when searching the source :) | 14:31 |
*** jungleboyj has quit IRC | 14:33 | |
flip214 | where's the multi-node testing metting? in -meeting, or somewhere else? | 14:35 |
*** julim has quit IRC | 14:37 | |
scottda | flip214: testing meeting is here, in openstack-cinder in 20 minutes (1500 UTC) | 14:39 |
*** karthikp_ has quit IRC | 14:39 | |
scottda | tomoiaga: When searching nova source, keep in mind that nova calls a volume detach "delete" and not "detach" | 14:39 |
*** julim has joined #openstack-cinder | 14:40 | |
*** julim has quit IRC | 14:40 | |
tomoiaga | scottda: ah ok, I was just looking at the detach_volume function in the libvirt driver :) | 14:41 |
*** yangyapeng has joined #openstack-cinder | 14:43 | |
*** mriedem1 is now known as mriedem | 14:46 | |
*** jungleboyj has joined #openstack-cinder | 14:47 | |
*** mtanino has joined #openstack-cinder | 14:49 | |
*** itzdilip has joined #openstack-cinder | 14:51 | |
scottda | tomoiaga: That's correct for the driver, but in other places it's called "delete". Just FYI | 14:53 |
tomoiaga | scottda: ok, thank you. It seems the driver receives an empty block_device_mapping | 14:54 |
scottda | tomoiaga: OK, that would be a problem. I'm not sure why that would occur. Was the instance nova-migrated, or something else unusual? | 14:56 |
*** salv-orl_ has quit IRC | 14:56 | |
*** karthikp_ has joined #openstack-cinder | 14:57 | |
*** salv-orlando has joined #openstack-cinder | 14:57 | |
flip214 | scottda: thanks | 14:57 |
tomoiaga | scottda: no, I am just creating a new instance, I then create a volume and attach it to the instance. After I delete the instance, the volume remains in use and I can see it's attachment to the deleted instance | 14:57 |
*** links has joined #openstack-cinder | 14:57 | |
scottda | tomoiaga: OK. At some point it's probably worth filing a bug. Looks to me like a Nova issue. | 14:58 |
tomoiaga | scottda: indeed. I'll try to figure out exactly why this is happening first. I like doing that from time to time :) | 14:58 |
*** itzdilip has quit IRC | 14:58 | |
*** itzdilip has joined #openstack-cinder | 14:59 | |
tomoiaga | scottda: thank you for your help | 14:59 |
scottda | tomoiaga: No problem. Keep me informed if you file a bug, I'm interested. | 14:59 |
scottda | eharney, xyang1, gouthamr, akerr, smcginnis, cFouts, e0ne, geguileo, dulek, flip214 Let's talk about Cinder Testing | 15:00 |
dulek | Testing stuff? | 15:00 |
geguileo | scottda: Thanks! | 15:00 |
eharney | hey | 15:00 |
flip214 | right here | 15:00 |
scottda | #link https://etherpad.openstack.org/p/Cinder-testing | 15:00 |
tomoiaga | scottda: ok. I'll let you know | 15:00 |
*** tomoiaga has quit IRC | 15:00 | |
*** Yogi1 has joined #openstack-cinder | 15:01 | |
*** Julien-z_ has joined #openstack-cinder | 15:02 | |
*** cknight has quit IRC | 15:02 | |
*** cknight has joined #openstack-cinder | 15:02 | |
e0ne_ | hi | 15:02 |
gouthamr | hello o/ | 15:03 |
*** cknight has quit IRC | 15:03 | |
flip214 | uhhhmm? silence? | 15:03 |
scottda | So, the idea was to just get everyone together to talk about what we're working on, what needs doing, what needs reviewing, and keep us all informed. | 15:03 |
scottda | From the agenda: | 15:03 |
scottda | https://www.irccloud.com/pastebin/935GIRo7/ | 15:04 |
dulek | Anything we want to say on testing categories? | 15:04 |
scottda | We could stand to have some clarification around what is funtional, in-tree tempest, normal tempest, etc. | 15:04 |
scottda | And this could stand to be documented. | 15:04 |
*** cknight has joined #openstack-cinder | 15:04 | |
*** sage has quit IRC | 15:05 | |
*** zul_ has joined #openstack-cinder | 15:05 | |
scottda | Any volunteers for putting something up, probably in the devref? | 15:05 |
scottda | I think smcginnis had some good thoughts on the definitions. And he is not here. Should we volunteer him? | 15:05 |
*** zul has quit IRC | 15:05 | |
geguileo | Sounds like a plan ;-) | 15:06 |
*** Julien-zte has quit IRC | 15:06 | |
geguileo | I agree that we should have this documented | 15:06 |
scottda | OK, we'll wait for an actual volunteer. But it would help to know where the tests need to belong. | 15:06 |
*** zul_ has quit IRC | 15:06 | |
scottda | Next is "List current efforts and who is working on them" | 15:07 |
geguileo | scottda: Do you know smcginnis definitions? | 15:07 |
*** zul has joined #openstack-cinder | 15:07 | |
scottda | geguileo: I dont' recall exactly, but they sounded good at the time. | 15:07 |
*** sheel has joined #openstack-cinder | 15:07 | |
eharney | i know we discussed and agreed on what functional is for... i'm just not sure off-hand where we wrote it | 15:07 |
*** sage has joined #openstack-cinder | 15:07 | |
smcginnis | Sorry, delayed grabbing coffee. | 15:08 |
scottda | smcginnis: Do you recall your definitions of testing categories? | 15:08 |
smcginnis | I will write up a devref and put it up for review. | 15:08 |
e0ne_ | we discussed it at summit too | 15:08 |
scottda | smcginnis: Swell. Thanks. | 15:08 |
Swanson | You should never delay grabbing coffee. | 15:08 |
geguileo | smcginnis: Thanks! | 15:08 |
smcginnis | Basically I think unit tests should be isolated to one or a couple modules. | 15:08 |
e0ne_ | smcginnis: +2 | 15:09 |
smcginnis | Functional should be things that involve several components, but still within Cinder. | 15:09 |
e0ne_ | also, all DB-related tests should be moved to functional | 15:09 |
smcginnis | And tempest (in-tree or tempest proper) should be things that test across Cinder and other services. | 15:09 |
dulek | e0ne_: +1, unit tests shouldn't really need SQLite, but that's utopia… | 15:10 |
*** itzdilip has quit IRC | 15:10 | |
e0ne_ | I'm definatle agree with smcginnis | 15:10 |
geguileo | dulek: Depending on the tests | 15:10 |
geguileo | smcginnis: +1 to all definitions | 15:10 |
smcginnis | e0ne_: Cool, I'll try to expand on that and get something up for everyone to nitpick. ;) | 15:10 |
scottda | OK, sounds good and we can fine tune in the patch to devref, if that's good with everyone. | 15:10 |
dulek | geguileo: Oh yeah, conditional updates without SQLA would be rather hard to test… | 15:10 |
*** faizy has joined #openstack-cinder | 15:10 | |
geguileo | dulek: Yep ;-) | 15:11 |
scottda | So under current efforts and who is working on them... | 15:11 |
*** e0ne_ is now known as e0ne | 15:11 | |
scottda | Please sign up if that's something you are doing, or interested in. | 15:11 |
e0ne | I'm working on backup tests with obutenko | 15:11 |
e0ne | https://etherpad.openstack.org/p/cinder-tempest-tests | 15:11 |
scottda | Anyone actively working on multinode? | 15:12 |
scottda | patrickeast: Was interested in some of this stuff, I think multinode, but he's not on the list. So "ping" | 15:12 |
smcginnis | scottda: Yoou had looked in to some of that, right? | 15:13 |
dulek | I can team up with someone on tempest and grenade common parts. | 15:13 |
scottda | smcginnis: Yes, I want to, but haven't gotten to it. I will at some point, but probably at least 1 week out. | 15:13 |
scottda | I'm very interested in HA-AA testing, which I think has multi-node as a dependency. | 15:13 |
e0ne | scottda: +1 | 15:14 |
scottda | Let me know if anyone else it thinking about HA-AA testing. We can coordinate. | 15:14 |
e0ne | scottda, geguileo: I'll share my rally draft scenarios for HA and API races early next week | 15:14 |
geguileo | e0ne: Awesome !! | 15:15 |
scottda | e0ne: Great. Thanks. | 15:15 |
dulek | scottda: So there is at least two of us - upgrades have same dependency. | 15:15 |
geguileo | e0ne: I'm on PTO next week, but I'll connect to have a look at them :-) | 15:15 |
e0ne | geguileo: ok | 15:15 |
*** mriedem1 has joined #openstack-cinder | 15:15 | |
scottda | dulek: Cool. I guess we need some concrete multi-node test to get the plumbing in place, that'll make the next ones simpler. | 15:15 |
e0ne | to be clear: from my PoV: multinode and multinode grenade tests are not the same | 15:15 |
flip214 | ↵ | 15:15 |
smcginnis | Some of the same configuration required though, right? | 15:16 |
scottda | e0ne: Yeah. I think there's going to be some infra stuff for multi-node (not sure what). So that's what I'm hoping to get done with the simplest possible multi-node case. Just not sure what that is... | 15:16 |
*** mriedem has quit IRC | 15:17 | |
dulek | e0ne: Similar plumbing in devstack-gate will be needed - based on my colleague knowledge. | 15:17 |
e0ne | scottda: AFAIK, nova has some multinode job | 15:17 |
dulek | e0ne: Nova and Neutron currently. | 15:17 |
e0ne | dulek: cool! it means that infra is ready | 15:18 |
*** thingee has quit IRC | 15:18 | |
scottda | OK, well if we don't come up with something simpler, I might just look at HA tests with multi-node. | 15:18 |
smcginnis | Anyone know what happened to multinode testing from what was discussed in Tokyo? | 15:18 |
dulek | e0ne: Well, yes and no I think - some work in openstack-infra repos will probably be needed. But it should be possible. ;) | 15:18 |
smcginnis | At the time, they were saying it was set up and ready. | 15:19 |
smcginnis | But then I thought I heard that fell through. | 15:19 |
dulek | smcginnis: Isn't that the moment when Nova introduced some multinode jobs into their check? | 15:19 |
openstackgerrit | Merged openstack/os-brick: Copy encryptors from Nova to os-brick https://review.openstack.org/247372 | 15:19 |
smcginnis | dulek: Yeah, I do think nova was spearheading that. | 15:19 |
dulek | So they did the hardest part, but turning it on for another service isn't a noop. | 15:20 |
*** julim has joined #openstack-cinder | 15:20 | |
dulek | I guess the best we can do is to ask on #openstack-infra or on ML? | 15:20 |
*** julim has quit IRC | 15:20 | |
*** pwehrle has left #openstack-cinder | 15:20 | |
e0ne | do we have any docs from infra about multinode jobd? | 15:21 |
e0ne | s/jods/jobs | 15:21 |
*** julim has joined #openstack-cinder | 15:21 | |
*** akshai has quit IRC | 15:21 | |
scottda | I haven't seen any docs, but haven't looked to hard. | 15:21 |
smcginnis | dulek: I think it would be worth asking. We know we aren't the only ones that need it, so there's likely been some work done already or examples to follow. | 15:21 |
*** akshai has joined #openstack-cinder | 15:21 | |
*** julim has quit IRC | 15:22 | |
scottda | OK, I put my name by "multi-node Cinder" . Anyone else, sign up and we can work together. | 15:22 |
*** julim has joined #openstack-cinder | 15:22 | |
scottda | Same for Active-Active HA. I assume geguileo I interested in that as well :) | 15:22 |
*** sage has quit IRC | 15:23 | |
geguileo | scottda: I'm interested, but being realistic is going to be difficult to find the time to work on the tests until I finish the rest | 15:23 |
dulek | smcginnis: I've asked, let's see. | 15:24 |
scottda | geguileo: No worries, I'm interested in the work. You can just advise and review, if you've time. | 15:24 |
patrickeast | Oh dang I forgot this was this morning | 15:24 |
smcginnis | dulek: Thanks! | 15:24 |
geguileo | scottda: I'll certainly make time for that | 15:24 |
scottda | I'm looking for notes from the Summit, anyone have a link to the etherpad where we discussed testing in Austin? | 15:24 |
*** mriedem1 is now known as mriedem | 15:24 | |
*** faizy has quit IRC | 15:24 | |
*** sgotliv_ has quit IRC | 15:25 | |
patrickeast | I looked at the multi node stuff a bit, looks like right now it's done with node pool scripts to configure stuff and manage the subnodes | 15:25 |
*** itzdilip has joined #openstack-cinder | 15:26 | |
patrickeast | Still looking into doing it in a new job... easier with a 3rd party ci with less other stuff confusing the configs | 15:26 |
scottda | patrickeast: Cool. I might ping you on that to get started on this later this week. | 15:26 |
smcginnis | https://wiki.openstack.org/wiki/Design_Summit/Mitaka/Etherpads#Cinder | 15:26 |
*** arch-nemesis has joined #openstack-cinder | 15:26 | |
scottda | smcginnis: Thanks. | 15:26 |
ntpttr_ | scottda: if there's any way I can help with the microversion functional testing I'd be happy to | 15:26 |
smcginnis | Oh wait, wrong one. :) | 15:26 |
smcginnis | scottda: https://wiki.openstack.org/wiki/Design_Summit/Newton/Etherpads#Cinder | 15:26 |
scottda | ntpttr_: Cool. I'll be pinging you for that. | 15:27 |
smcginnis | Specifically: https://etherpad.openstack.org/p/cinder-newton-testingprocess | 15:27 |
scottda | smcginnis: Thanks. I'll organize that and update the Testing etherpad. Meanwhile, I put the links in. | 15:28 |
smcginnis | +1 | 15:28 |
scottda | Anything else on the subject of What's being done, by whom, etc? | 15:28 |
scottda | There's an item on there for a wish list. | 15:29 |
scottda | I reckon if we put up stuff that needs doing but has no owner, we might get someone to sign up. | 15:29 |
scottda | Testing is a good way for new contributors to get started, IMO. | 15:30 |
e0ne | scottda: +1 | 15:30 |
smcginnis | That would be good to start capturing areas where we would like to see some more coverage. | 15:30 |
geguileo | scottda: +1 | 15:30 |
e0ne | smcginnis: +1 | 15:30 |
*** jgregor has joined #openstack-cinder | 15:30 | |
smcginnis | That could really help when folks come asking for how they can get involved, or when we have training/mentoring sessions. | 15:30 |
scottda | OK, I added a category on that etherpad. Brainstorm away.... | 15:31 |
scottda | I had a question for 2 patches I have up: | 15:31 |
scottda | Tempest test for retype w/migration vs retype w/mig while volume attached (BFV): Do we need/want both? | 15:31 |
scottda | https://review.openstack.org/#/c/195443/ https://review.openstack.org/#/c/326681/ | 15:32 |
*** julim has quit IRC | 15:32 | |
smcginnis | scottda: So the difference is attached vs not attached? | 15:33 |
scottda | One factor is time to run the test. BootFromVolume is time-consuming, as is Cinder migrate of that volume. | 15:33 |
scottda | Not attached just doesn't have the time required for loading the image into the volume, and then Booting. | 15:33 |
scottda | non attached doesn't cover as much. It doesn't call Nova swap_volume, which doesn't ever get tested ATM. And we want this coverage for multi-attach. | 15:34 |
smcginnis | I think it's worth the time it takes to do BFV. | 15:34 |
*** timcl1 has quit IRC | 15:34 | |
eharney | i think we want both | 15:34 |
*** xinli has joined #openstack-cinder | 15:34 | |
smcginnis | If we have an issue there, that could cause a service outage. | 15:34 |
scottda | smcginnis: Definately need the BFV case. But is it worth it to test migrate/retype without a volume attached? | 15:34 |
scottda | eharney: Ok. I like both. Just thought I'd ask. | 15:34 |
smcginnis | Yeah, I think both too. | 15:35 |
scottda | And both of these patches are listed at the bottom of the etherpad under "Tests under Review" | 15:35 |
scottda | Which leads us to that.... | 15:35 |
*** arecknag has quit IRC | 15:36 | |
scottda | Just a place to list 'em. We don't get alerted to tests outside our repo, so sometimes they seem a bit negelected. | 15:36 |
scottda | neglected. | 15:36 |
Swanson | jgriffith, So just whack your exception and my exception and put a VolumeDriverRetryableException of some sort in it's stead? | 15:36 |
jgriffith | Swanson: exactly | 15:36 |
smcginnis | scottda: True | 15:36 |
jgriffith | Swanson: I've tried this before though.. I doubt it'll stick :( | 15:36 |
scottda | I think that covers what's on that etherpad and meeting Agenda. Anything else ATM? | 15:37 |
jgriffith | Swanson: I find the duplication of exceptions for each individual driver kind of pointless. At least at the higher layers | 15:37 |
smcginnis | scottda: On our wishlist, should we put something there to review tempest tests? | 15:37 |
smcginnis | scottda: I've noticed a couple with "Skipped until bug x is closed" and bug x has been closed for a few cycles. | 15:38 |
scottda | Sure, that seems good. | 15:38 |
smcginnis | I'm sure there's some work in there if someone has the time to start digging in to it. | 15:38 |
e0ne | scottda: thanks for the leadind this | 15:38 |
jgriffith | Swanson: I tried that whole "VolumeBackendException" to generalize a bunch of things... thinking that the driver interface should only return those *general* things. Then if they want to define some of their own in the dirver to use that's cool, but the rest of Cinder shouldn't know anything about them | 15:38 |
Swanson | jgriffith, I almost re-purposed an existing one under the theory that I'd catch it first. | 15:38 |
scottda | smcginnis: OK, that "Skipped" audit could go on the wishlist as something simpler for someone to do. | 15:38 |
*** laughterwym has joined #openstack-cinder | 15:39 | |
smcginnis | scottda: Awesome | 15:39 |
jgriffith | Swanson: :) yeah | 15:39 |
Swanson | jgriffith, was a little surprised that people defined their exceptions in the common file. | 15:39 |
scottda | e0ne: No problem. Thanks everyone for participating. I propose we keep this up every week at this time. Hopefully it'll be shorter in the future. | 15:39 |
jgriffith | Swanson: well.. when there were just 4 or 5 drivers it made more sense | 15:39 |
e0ne | scottda: +1 | 15:39 |
smcginnis | scottda: Thanks! | 15:39 |
jgriffith | Swanson: now it's just kinda nutty | 15:39 |
Swanson | jgriffith, fair point. | 15:39 |
jgriffith | Swanson: that file is a bit of a mess | 15:40 |
Swanson | jgriffith, I could also move mine out of there, I suppose. | 15:40 |
jgriffith | Swanson: it's easy to fix up though.... just kinda tedious | 15:40 |
scottda | OK, talk to ya'll next meeting...in 20 minutes. | 15:40 |
*** laughterwym has quit IRC | 15:40 | |
jgriffith | Swanson: yeah.. not a horrible idea. Lead by example :) | 15:40 |
geguileo | Thanks! | 15:41 |
jgriffith | Swanson: I would like to eventually clean that stuff up, provide enough general exceptions in there and have it be so that the drivers NEVER raise anything other than those to the outside world | 15:41 |
Swanson | jgriffith, I can just imagine that. "I checked how to do this. 190 drivers put theirs in exception.py. 1 put it in their own driver. I'll make a patch to move theirs into exception.py." | 15:41 |
*** laughterwym has joined #openstack-cinder | 15:42 | |
jgriffith | Swanson: that's kind of how some things have gone funny enough | 15:42 |
smcginnis | Swanson, jgriffith: We have downvoted new driver patches because they _didn't_ put their exception in the common location. | 15:42 |
Swanson | jgriffith, well consistency is good, I suppose. | 15:42 |
jgriffith | Swanson: just luck of the draw on which driver you use as a reference.. and it seems nobody uses the "reference" driver any more :( | 15:42 |
*** julim has joined #openstack-cinder | 15:42 | |
jgriffith | smcginnis: haha sounds about right | 15:42 |
Swanson | jgriffith, Make a reference.py driver that is just a link to lvm? | 15:43 |
smcginnis | Not saying it should be that way, just that's what we've kind of enforced so far. | 15:43 |
smcginnis | Hah | 15:43 |
*** julim has quit IRC | 15:43 | |
jgriffith | smcginnis: Swanson we should pick a standard and go with it | 15:43 |
jgriffith | smcginnis: sure, I get ya | 15:43 |
jgriffith | Swanson: that's not a terrible suggestion | 15:43 |
smcginnis | Swanson: Or we can clearly define the driver interface, then have a job that publishes the well written docstrings to our devref. :] | 15:43 |
Swanson | smcginnis, I'd like a unicorn. | 15:44 |
*** julim has joined #openstack-cinder | 15:44 | |
jgriffith | smcginnis: oh.. now there's a frikin brilliant idea!!! | 15:44 |
jgriffith | Swanson: it's not that hard to do honestly | 15:44 |
*** julim has quit IRC | 15:44 | |
smcginnis | Hmm, if only someone had started down that path.... :D | 15:44 |
jgriffith | Swanson: it's just getting by the bike-shedding of some folks that have some pie in the sky notion of how the interfaces work | 15:44 |
jgriffith | smcginnis: speaking of which.... how's your version of that coming along? | 15:45 |
jgriffith | smcginnis: LOL | 15:45 |
flip214 | one more core for https://review.openstack.org/#/c/326377/3, pretty please thanks a lot?! | 15:45 |
smcginnis | Swanson: Here's your unicorn: https://www.youtube.com/watch?v=YbYWhdLO43Q | 15:45 |
jgriffith | flip214: no | 15:45 |
jgriffith | LMAO!!! | 15:45 |
flip214 | jgriffith: ;) thanks for your past help in any case! | 15:45 |
smcginnis | jgriffith: I think the first step is good. Just want to make sure there's buy in on that before building on it too much. | 15:45 |
*** belmoreira has quit IRC | 15:46 | |
*** julim has joined #openstack-cinder | 15:46 | |
*** julim has quit IRC | 15:46 | |
Swanson | smcginnis, I've sent those out as gifts. | 15:46 |
flip214 | jgriffith: I've got a wonderful idea... why not write a bot that uses your credentials to just do +2 +1 on every cinder patch, unconditionally? if jenkins rejects it it won't merge anyway ;) | 15:46 |
smcginnis | flip214: Hah! | 15:46 |
jgriffith | flip214: I'd be a super star in review metrics | 15:46 |
jgriffith | flip214: just don't look closely at disagreement rate etc | 15:47 |
*** julim has joined #openstack-cinder | 15:47 | |
smcginnis | You could probably +1 every patch, and the metrics would probably still make your disagreement count low. :) | 15:47 |
dulek | I always wondered if a well-written bot that mimics reviews of others can achieve core status in any of the smaller projects. | 15:47 |
smcginnis | Hopefully I don't regret having said that. | 15:47 |
smcginnis | Oh man! | 15:48 |
flip214 | well, just have your bot remove the +1/+2 fast enough if others disagree. | 15:48 |
*** laughterwym has quit IRC | 15:48 | |
Swanson | Throwing this one out for cores... https://review.openstack.org/#/c/325506/3 Largely dell internal stuff that should help stability and smcginnis sanity. | 15:48 |
avishay | Hey all, speaking of reviews... list-manageable is getting lonely, could use some love | 15:48 |
patrickeast | jgriffith: Swanson: oh, driver retryable exceptions? i think i added one in for pure recently... | 15:48 |
*** laughterwym has joined #openstack-cinder | 15:48 | |
patrickeast | or maybe its still up for review | 15:48 |
smcginnis | Swanson: Did we want to go the exception route jgriffith was recommending? | 15:48 |
Swanson | smcginnis, I'd rather anything we do with that be a separate patch. | 15:49 |
flip214 | jgriffith: you got beaten in the review. | 15:49 |
Swanson | smcginnis, I'd rather not have a fight over something I think we need to land. | 15:49 |
flip214 | patrickeast: thank you very much. | 15:49 |
smcginnis | Swanson: True, that's something we can go back later and clean up. Especially if we want to touch the Pure and SolidFire cases. | 15:50 |
patrickeast | yea if you guys do it in another patch lets get all 3 | 15:50 |
smcginnis | +1 | 15:50 |
Swanson | patrickeast, agreed. | 15:50 |
patrickeast | https://review.openstack.org/#/c/324156/ was +A'ed but hasn't merged yet, waiting on https://review.openstack.org/#/c/324155/ | 15:50 |
patrickeast | so maybe not too late to fix that one before it lands | 15:50 |
smcginnis | Oh yeah, looked at that but was waiting for Jenkins. | 15:51 |
*** Apoorva has joined #openstack-cinder | 15:51 | |
*** laughterwym has quit IRC | 15:53 | |
xinli | dulek: After submitting PatchSet (19) , the test case issues get fixed, could you please review the code? https://review.openstack.org/#/c/251989 | 15:53 |
smcginnis | patrickeast: Oh wait, I was actually waiting on Pure CI. I don't see results there. | 15:53 |
patrickeast | smcginnis: oh, i'll make it re-run | 15:53 |
patrickeast | smcginnis: but it passed on the dependent patch, so was tested there in a round-about fashion | 15:53 |
dulek | xinli: I have that in todo. :) | 15:54 |
*** ivase has joined #openstack-cinder | 15:54 | |
smcginnis | patrickeast: Good enough for me. ;) | 15:54 |
dulek | https://review.openstack.org/#/c/307075 is holding up some other patches and already has two +2s, so if someone had a moment… :) | 15:55 |
*** rlrossit has quit IRC | 15:55 | |
*** enriquetaso has joined #openstack-cinder | 15:55 | |
*** merooney has quit IRC | 15:56 | |
*** cknight has quit IRC | 15:57 | |
*** julim has quit IRC | 15:57 | |
*** wanghao_ has quit IRC | 15:58 | |
*** nkrinner has quit IRC | 15:58 | |
*** chlong has quit IRC | 15:58 | |
flip214 | meeeeeeeeeeeeeeeting. | 15:59 |
*** timcl has joined #openstack-cinder | 15:59 | |
*** sage has joined #openstack-cinder | 16:01 | |
savihou | Only one core away :) https://review.openstack.org/#/c/263026/ , pretty please ? | 16:03 |
*** raunak has joined #openstack-cinder | 16:03 | |
avishay | DuncanT jgriffith hemna geguileo smcginnis and xing when she arrives... gentle reminder for list manageable reviews | 16:04 |
smcginnis | avishay: ack | 16:04 |
*** ducttape_ has joined #openstack-cinder | 16:06 | |
*** avishay has quit IRC | 16:07 | |
*** baumann has joined #openstack-cinder | 16:08 | |
*** ducttape_ has quit IRC | 16:10 | |
*** adisky has quit IRC | 16:10 | |
*** julim has joined #openstack-cinder | 16:11 | |
*** julim has quit IRC | 16:12 | |
*** julim has joined #openstack-cinder | 16:13 | |
*** anteaya has joined #openstack-cinder | 16:14 | |
*** links has quit IRC | 16:15 | |
*** salv-orlando has quit IRC | 16:18 | |
*** salv-orlando has joined #openstack-cinder | 16:18 | |
*** laughterwym has joined #openstack-cinder | 16:21 | |
*** thingee has joined #openstack-cinder | 16:22 | |
*** salv-orl_ has joined #openstack-cinder | 16:24 | |
*** timcl has quit IRC | 16:25 | |
*** suyash has joined #openstack-cinder | 16:26 | |
*** Julien-z_ has quit IRC | 16:26 | |
*** salv-orlando has quit IRC | 16:27 | |
*** e0ne has quit IRC | 16:30 | |
*** tesseract has quit IRC | 16:31 | |
*** mjura has quit IRC | 16:32 | |
*** laughterwym has quit IRC | 16:32 | |
*** laughterwym has joined #openstack-cinder | 16:33 | |
smcginnis | hemna: http://paste.openstack.org/show/508969/ | 16:35 |
smcginnis | hemna: Should probably get the encryptor and privsep changes out there. | 16:36 |
*** frontrunner has quit IRC | 16:36 | |
smcginnis | hemna: Could have sworn that we released the privsep change before. | 16:36 |
*** laughterwym has quit IRC | 16:38 | |
*** leeantho has joined #openstack-cinder | 16:39 | |
*** Guest47242 is now known as med_ | 16:41 | |
*** med_ has joined #openstack-cinder | 16:41 | |
smcginnis | hemna: Release patch queued up. Let me know if you think we need to wait for anything else. | 16:41 |
*** rcernin has quit IRC | 16:42 | |
hemna | yah we haven't released privsep out yet | 16:45 |
hemna | because we wanted some bake time | 16:45 |
hemna | lots of changes | 16:46 |
*** lpetrut has quit IRC | 16:46 | |
*** frontrunner has joined #openstack-cinder | 16:47 | |
openstackgerrit | Matt Smith proposed openstack/cinder: Adds ACL, IP Pool, Multipath to Datera Driver https://review.openstack.org/326685 | 16:49 |
*** mvk has quit IRC | 16:50 | |
*** merooney has joined #openstack-cinder | 16:51 | |
*** rlrossit has joined #openstack-cinder | 16:52 | |
*** ChubYann has joined #openstack-cinder | 16:52 | |
jay-mehta | Could anyone please review unit test patch https://review.openstack.org/#/c/319511/ . It is +1 | 16:54 |
*** cknight has joined #openstack-cinder | 16:55 | |
*** timcl has joined #openstack-cinder | 16:57 | |
*** laughterwym has joined #openstack-cinder | 17:00 | |
*** ducttape_ has joined #openstack-cinder | 17:01 | |
*** ducttape_ has quit IRC | 17:03 | |
*** ducttape_ has joined #openstack-cinder | 17:04 | |
patrickeast | hemna: oh hey, speaking of privsep, any idea how to make it not spew all the commands out to warning log level? http://openstack-logs.purestorage.com/32/258132/6/check/PureFCDriver-tempest-dsvm-volume-multipath/09f9ead/logs/screen-c-vol.txt.gz?level=WARNING | 17:04 |
hemna | patrickeast, I think that's a privsep library thingy | 17:04 |
hemna | I don't think it's a setting we can tweak. I'll look into it | 17:04 |
patrickeast | hemna: hmm yea, i poked around a bit and didn't see any specific warning level logging | 17:05 |
*** chris_morrell has joined #openstack-cinder | 17:06 | |
*** sgotliv_ has joined #openstack-cinder | 17:10 | |
*** sdake has joined #openstack-cinder | 17:10 | |
*** laughterwym has quit IRC | 17:14 | |
*** laughterwym has joined #openstack-cinder | 17:15 | |
*** kfarr has joined #openstack-cinder | 17:23 | |
*** IlyaG has joined #openstack-cinder | 17:31 | |
*** e0ne has joined #openstack-cinder | 17:32 | |
*** IlyaG has quit IRC | 17:32 | |
*** jungleboyj has quit IRC | 17:35 | |
*** e0ne has quit IRC | 17:43 | |
*** lpetrut has joined #openstack-cinder | 17:46 | |
*** e0ne has joined #openstack-cinder | 17:50 | |
*** laughterwym has quit IRC | 17:51 | |
*** sdake has quit IRC | 17:51 | |
*** salv-orl_ has quit IRC | 17:53 | |
*** salv-orlando has joined #openstack-cinder | 17:53 | |
*** e0ne has quit IRC | 17:53 | |
*** diablo_rojo has quit IRC | 17:54 | |
*** bardia has joined #openstack-cinder | 17:55 | |
*** rlrossit has quit IRC | 18:04 | |
*** rlrossit has joined #openstack-cinder | 18:08 | |
-openstackstatus- NOTICE: Zuul has been restarted to correct an error condition. Events since 17:30 may have been missed; please 'recheck' your changes if they were uploaded since then, or have "NOT_REGISTERED" errors. | 18:10 | |
*** asselin has quit IRC | 18:10 | |
*** frontrunner has quit IRC | 18:10 | |
*** Lee1092 has quit IRC | 18:13 | |
*** timcl has quit IRC | 18:14 | |
*** mriedem1 has joined #openstack-cinder | 18:17 | |
*** mriedem has quit IRC | 18:18 | |
dulek | scottda: http://eavesdrop.openstack.org/irclogs/%23openstack-dev/%23openstack-dev.2016-06-08.log.html#t2016-06-08T16:05:02 | 18:20 |
*** xinli has quit IRC | 18:20 | |
dulek | scottda: I hate reading bash, but that's what I'm doing now… | 18:20 |
*** diablo_rojo has joined #openstack-cinder | 18:21 | |
*** asselin has joined #openstack-cinder | 18:21 | |
*** mriedem1 is now known as mriedem | 18:26 | |
*** links has joined #openstack-cinder | 18:33 | |
*** lpetrut has quit IRC | 18:33 | |
*** frontrunner has joined #openstack-cinder | 18:36 | |
*** sdake has joined #openstack-cinder | 18:37 | |
*** alyson_ has quit IRC | 18:38 | |
*** lcurtis has joined #openstack-cinder | 18:38 | |
*** diogogmt has joined #openstack-cinder | 18:42 | |
openstackgerrit | Jay Mehta proposed openstack/cinder: Implement match-set type 3PAR FC VLUN creation https://review.openstack.org/309613 | 18:42 |
*** gouthamr has quit IRC | 18:45 | |
*** ducttape_ has quit IRC | 18:49 | |
*** arch-nemesis has quit IRC | 18:50 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Move OVO methods from CinderObject to Persistent https://review.openstack.org/312930 | 18:51 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove remotable from OVOs https://review.openstack.org/312929 | 18:51 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Refactor create, save, and destroy OVO methods https://review.openstack.org/315541 | 18:51 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add debug level logs on OVO backports https://review.openstack.org/321143 | 18:51 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Stop assuming persistent OVOs are also DictCompat https://review.openstack.org/321142 | 18:51 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Implement OVO class init mechanism https://review.openstack.org/321144 | 18:51 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Use manifest to backport OVOs during upgrades https://review.openstack.org/307075 | 18:51 |
*** ducttape_ has joined #openstack-cinder | 18:51 | |
*** krtaylor has quit IRC | 18:52 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder-specs: Update Job Distribution for A/A Specs https://review.openstack.org/327283 | 18:55 |
openstackgerrit | Alex Meade proposed openstack/cinder: NetApp: Log unexpected errors from backend https://review.openstack.org/322223 | 18:56 |
ameade | geguileo: ^^ added a sentence to the commit message | 18:56 |
ameade | just need another reviewer | 18:56 |
ameade | 6 line patch that's been there for a month :) | 18:56 |
openstackgerrit | nikesh proposed openstack/cinder: Provide Kaminario K2 all-flash array iSCSI driver https://review.openstack.org/327285 | 18:57 |
geguileo | ameade: Eric was faster than me | 18:58 |
*** chris_morrell has quit IRC | 18:58 | |
geguileo | eharney: You were fast!! :-) | 18:59 |
eharney | geguileo: aren't you supposed to be done with work by now? :) | 19:00 |
*** nikeshm has joined #openstack-cinder | 19:00 | |
ameade | wooo i'll take it | 19:00 |
geguileo | eharney: Yeah, I was about to finish for the day O:-) | 19:00 |
geguileo | eharney: Had just changed status to out, but I heard the ping... | 19:01 |
geguileo | so I came back | 19:01 |
geguileo | eharney: That's the problem with summer, it's 9pm and the sun's still up, so I don't realize how late it is and keep working | 19:01 |
geguileo | lol | 19:01 |
*** gouthamr has joined #openstack-cinder | 19:01 | |
*** timcl has joined #openstack-cinder | 19:02 | |
*** Apoorva has quit IRC | 19:05 | |
*** raunak has quit IRC | 19:05 | |
*** arch-nemesis has joined #openstack-cinder | 19:05 | |
scottda | geguileo: I thought that was normal in Spain, and you ate dinner at 10:00 PM and just stay up late? :) | 19:05 |
*** krtaylor has joined #openstack-cinder | 19:05 | |
scottda | oops. That was another ping... | 19:05 |
geguileo | scottda: Yep, I usually have dinner at 10 or 11 pm | 19:06 |
geguileo | scottda: But I try not to work until so late | 19:06 |
scottda | Sorry, didn't mean to drag you back | 19:06 |
scottda | dulek: Thanks for the pointer. I'll read some bash myself. | 19:07 |
*** ducttape_ has quit IRC | 19:07 | |
openstackgerrit | Merged openstack/cinder: DRBD: Allow to set options for new resources https://review.openstack.org/326377 | 19:08 |
openstackgerrit | Merged openstack/os-brick: Add support for processutils.execute https://review.openstack.org/309201 | 19:11 |
*** harlowja has quit IRC | 19:12 | |
*** e0ne has joined #openstack-cinder | 19:13 | |
*** salv-orlando has quit IRC | 19:14 | |
*** salv-orlando has joined #openstack-cinder | 19:14 | |
*** harlowja has joined #openstack-cinder | 19:15 | |
*** frontrunner has quit IRC | 19:21 | |
*** ducttape_ has joined #openstack-cinder | 19:22 | |
*** xinli has joined #openstack-cinder | 19:24 | |
*** rcernin has joined #openstack-cinder | 19:26 | |
*** diogogmt has quit IRC | 19:28 | |
openstackgerrit | nikesh proposed openstack/cinder: Provide Kaminario K2 all-flash array iSCSI driver https://review.openstack.org/327285 | 19:29 |
*** links has quit IRC | 19:31 | |
*** timcl has quit IRC | 19:34 | |
*** timcl has joined #openstack-cinder | 19:37 | |
*** salv-orlando has quit IRC | 19:39 | |
Swanson | patrickeast, Can I switch your exceptions from PureDriverException to the new VolumeDriverRetryableException where PureDriverException is included in the utils.retry? | 19:40 |
*** salv-orlando has joined #openstack-cinder | 19:40 | |
patrickeast | Swanson: yep | 19:40 |
openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX Driver - QoS support for the VMAX3 https://review.openstack.org/307502 | 19:42 |
*** lixiaoy1 has quit IRC | 19:44 | |
*** lixiaoy1 has joined #openstack-cinder | 19:44 | |
openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Oversubscription support https://review.openstack.org/307499 | 19:44 |
*** Apoorva has joined #openstack-cinder | 19:47 | |
*** sdake_ has joined #openstack-cinder | 19:47 | |
*** chris_morrell has joined #openstack-cinder | 19:47 | |
*** sdake has quit IRC | 19:48 | |
*** merooney has quit IRC | 19:48 | |
*** enriquetaso has quit IRC | 19:51 | |
openstackgerrit | Rajini Ram proposed openstack/cinder: Volume manage/unmanage support for Eqlx driver https://review.openstack.org/318748 | 19:56 |
*** e0ne has quit IRC | 20:04 | |
*** xinli has quit IRC | 20:04 | |
*** akerr has quit IRC | 20:17 | |
*** mvk has joined #openstack-cinder | 20:19 | |
*** raunak has joined #openstack-cinder | 20:22 | |
*** karthikp_ has quit IRC | 20:27 | |
*** jungleboyj has joined #openstack-cinder | 20:33 | |
*** sheel has quit IRC | 20:35 | |
*** gouthamr has quit IRC | 20:43 | |
*** Yogi1 has quit IRC | 20:44 | |
*** dave-mccowan has quit IRC | 20:44 | |
*** openstackgerrit has quit IRC | 20:48 | |
*** timcl has quit IRC | 20:48 | |
*** openstackgerrit has joined #openstack-cinder | 20:49 | |
*** timcl has joined #openstack-cinder | 20:50 | |
*** cknight has quit IRC | 20:53 | |
*** jbernard has quit IRC | 20:56 | |
*** erlon has quit IRC | 20:56 | |
*** iceyao has joined #openstack-cinder | 20:56 | |
*** porrua has quit IRC | 20:58 | |
*** dave-mccowan has joined #openstack-cinder | 21:00 | |
*** karthikp_ has joined #openstack-cinder | 21:00 | |
*** iceyao has quit IRC | 21:01 | |
*** sdake_ has quit IRC | 21:01 | |
*** jbernard has joined #openstack-cinder | 21:03 | |
*** julim has quit IRC | 21:04 | |
*** kfarr has quit IRC | 21:04 | |
*** chlong has joined #openstack-cinder | 21:06 | |
*** sgotliv_ has quit IRC | 21:08 | |
*** eharney has quit IRC | 21:09 | |
*** ivase has quit IRC | 21:13 | |
*** ametts has quit IRC | 21:14 | |
openstackgerrit | Merged openstack/cinder: Fix backup using temp snapshot code path https://review.openstack.org/321943 | 21:15 |
*** dustins has quit IRC | 21:17 | |
*** jungleboyj has quit IRC | 21:23 | |
*** diablo_rojo has left #openstack-cinder | 21:24 | |
*** mvk_ has joined #openstack-cinder | 21:25 | |
*** jwcroppe_ has quit IRC | 21:25 | |
*** rlrossit has quit IRC | 21:27 | |
*** mvk has quit IRC | 21:28 | |
*** lifeless_ is now known as lifeless | 21:30 | |
*** jgregor has quit IRC | 21:39 | |
*** cknight has joined #openstack-cinder | 21:39 | |
*** tpsilva has quit IRC | 21:47 | |
*** sdake has joined #openstack-cinder | 21:49 | |
*** jgregor has joined #openstack-cinder | 21:53 | |
*** rcernin is now known as rcernin|off | 21:54 | |
*** sdake has quit IRC | 21:54 | |
*** r-daneel has joined #openstack-cinder | 21:55 | |
*** sdake has joined #openstack-cinder | 21:55 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/327362 | 21:56 |
*** ducttape_ has quit IRC | 21:57 | |
openstackgerrit | Scott DAngelo proposed openstack/cinder: Update provider_location during snapshot backup https://review.openstack.org/327375 | 21:59 |
*** ducttape_ has joined #openstack-cinder | 22:04 | |
*** gouthamr has joined #openstack-cinder | 22:06 | |
*** karthikp_ has quit IRC | 22:15 | |
*** karthikp_ has joined #openstack-cinder | 22:15 | |
*** ducttape_ has quit IRC | 22:16 | |
*** ducttape_ has joined #openstack-cinder | 22:20 | |
*** rcernin|off has quit IRC | 22:21 | |
*** salv-orl_ has joined #openstack-cinder | 22:25 | |
*** salv-orlando has quit IRC | 22:28 | |
*** mriedem has quit IRC | 22:31 | |
*** edmondsw has quit IRC | 22:37 | |
*** cknight has quit IRC | 22:39 | |
*** jwcroppe has joined #openstack-cinder | 22:42 | |
*** salv-orl_ has quit IRC | 22:45 | |
*** leeantho has quit IRC | 22:46 | |
*** salv-orlando has joined #openstack-cinder | 22:46 | |
*** akerr has joined #openstack-cinder | 22:52 | |
*** akerr has quit IRC | 22:52 | |
*** lpetrut has joined #openstack-cinder | 22:56 | |
*** karthikp_ has quit IRC | 22:59 | |
*** sdague has quit IRC | 22:59 | |
*** arch-nemesis has quit IRC | 23:00 | |
openstackgerrit | Merged openstack/cinder: NetApp: Log unexpected errors from backend https://review.openstack.org/322223 | 23:01 |
openstackgerrit | John Griffith proposed openstack/cinder: Add new attach/detach API calls https://review.openstack.org/327408 | 23:02 |
openstackgerrit | John Griffith proposed openstack/python-cinderclient: Add new attach/detach API calls https://review.openstack.org/327409 | 23:03 |
*** ebalduf has quit IRC | 23:03 | |
*** ducttape_ has quit IRC | 23:03 | |
jgriffith | ildikov: hemna smcginnis patrickeast ^^ So that's what I'm thinking currently | 23:04 |
jgriffith | ildikov: hemna smcginnis patrickeast I'll get a nova patch set going that uses it | 23:04 |
*** ducttape_ has joined #openstack-cinder | 23:04 | |
jgriffith | tried to break it down to multiple commits by leveraging the existing manager calls/work-flow | 23:04 |
hemna | jgriffith, +1 | 23:04 |
hemna | looks nice | 23:04 |
*** hemna is now known as hemnafk | 23:07 | |
*** markvoelker has quit IRC | 23:13 | |
jgriffith | hemnafk: thanks | 23:14 |
jgriffith | hemnafk: let me know what you think after you've had a chance to look at it a bit more | 23:14 |
jgriffith | I'm out for now... cya | 23:14 |
*** jgriffith is now known as jgriffith_away | 23:14 | |
*** lpetrut has quit IRC | 23:15 | |
*** gouthamr_ has joined #openstack-cinder | 23:15 | |
*** gouthamr has quit IRC | 23:18 | |
*** lcurtis has quit IRC | 23:27 | |
*** markvoelker has joined #openstack-cinder | 23:44 | |
*** lprice1 has joined #openstack-cinder | 23:46 | |
*** lprice has quit IRC | 23:46 | |
*** markvoelker has quit IRC | 23:49 | |
*** ducttape_ has quit IRC | 23:57 | |
*** ducttape_ has joined #openstack-cinder | 23:58 | |
*** zhongjun_ has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!