| *** openstack has joined #openstack-cinder | 00:10 | |
| *** bill_az has quit IRC | 00:14 | |
| jgriffith | Ha! | 00:16 |
|---|---|---|
| jgriffith | I now have Freenode, HipChat, Slack succesfully going into my IRC client! | 00:16 |
| patrickeast | oo nice | 00:16 |
| jgriffith | good-bye all you crappy coms windows cluttering my desktop | 00:16 |
| patrickeast | i tried to get our slack to go to mine... didn't work :( | 00:17 |
| jgriffith | patrickeast: yeah, you have to enable the irc-gateway option | 00:17 |
| jgriffith | I was luck that folks were happy to set it | 00:17 |
| patrickeast | ahh gotcha | 00:17 |
| *** openstackgerrit has quit IRC | 00:17 | |
| *** openstackgerrit has joined #openstack-cinder | 00:18 | |
| *** sasukeh has joined #openstack-cinder | 00:19 | |
| *** dave-mccowan has joined #openstack-cinder | 00:20 | |
| *** jungleboyj has joined #openstack-cinder | 00:20 | |
| *** sasukeh has quit IRC | 00:23 | |
| *** jordant has joined #openstack-cinder | 00:24 | |
| sheel | smcginnis: thanks.. for https://review.openstack.org/#/c/285293 | 00:26 |
| sheel | it ll open gates for my patch:) | 00:26 |
| *** laughterwym has joined #openstack-cinder | 00:31 | |
| *** cknight has joined #openstack-cinder | 00:31 | |
| *** dims has quit IRC | 00:35 | |
| *** abhishekk_ has joined #openstack-cinder | 00:44 | |
| *** asselin_ has joined #openstack-cinder | 00:44 | |
| *** bardia_ has joined #openstack-cinder | 00:45 | |
| *** frontrunner2 has joined #openstack-cinder | 00:46 | |
| *** cbader_ has joined #openstack-cinder | 00:46 | |
| *** jungleboyj has quit IRC | 00:46 | |
| *** Nakato_ has joined #openstack-cinder | 00:48 | |
| *** mmotiani has quit IRC | 00:49 | |
| *** mkoderer__ has quit IRC | 00:49 | |
| *** zzzeek has quit IRC | 00:49 | |
| *** arch-nemesis has quit IRC | 00:49 | |
| *** frontrunner has quit IRC | 00:49 | |
| *** abhishekk has quit IRC | 00:49 | |
| *** cbader has quit IRC | 00:49 | |
| *** jbernard_ has quit IRC | 00:49 | |
| *** bardia has quit IRC | 00:49 | |
| *** asselin__ has quit IRC | 00:49 | |
| *** ccesario has quit IRC | 00:49 | |
| *** jdillaman has quit IRC | 00:49 | |
| *** Nakato has quit IRC | 00:49 | |
| *** mkoderer___ has joined #openstack-cinder | 00:49 | |
| *** ccesario has joined #openstack-cinder | 00:49 | |
| *** jbernard_ has joined #openstack-cinder | 00:50 | |
| *** arch-nemesis has joined #openstack-cinder | 00:50 | |
| *** zzzeek has joined #openstack-cinder | 00:50 | |
| *** jdillaman has joined #openstack-cinder | 00:52 | |
| *** jordant has quit IRC | 00:54 | |
| *** cheneydc has joined #openstack-cinder | 00:58 | |
| *** cknight has quit IRC | 01:00 | |
| *** haomaiwang has quit IRC | 01:01 | |
| *** haomaiwang has joined #openstack-cinder | 01:01 | |
| *** lcurtis has quit IRC | 01:01 | |
| *** garthb_ has quit IRC | 01:03 | |
| *** yangyapeng has joined #openstack-cinder | 01:09 | |
| openstackgerrit | Ice Yao proposed openstack/cinder: Add judgement of backing-up volume status https://review.openstack.org/288875 | 01:09 |
| openstackgerrit | Joshua Huber proposed openstack/cinder: Pass new volume size when cloning (blockbridge) https://review.openstack.org/290934 | 01:15 |
| *** yangds has quit IRC | 01:16 | |
| *** gouthamr has quit IRC | 01:16 | |
| *** dalgaaf has quit IRC | 01:18 | |
| *** gouthamr has joined #openstack-cinder | 01:18 | |
| *** mriedem has joined #openstack-cinder | 01:25 | |
| *** mmandel has quit IRC | 01:25 | |
| *** aorourke is now known as aorourke-afk | 01:28 | |
| *** dims has joined #openstack-cinder | 01:31 | |
| *** yangds has joined #openstack-cinder | 01:33 | |
| *** arch-nemesis has quit IRC | 01:37 | |
| *** sasukeh has joined #openstack-cinder | 01:40 | |
| *** mtanino has quit IRC | 01:43 | |
| *** edmondsw has quit IRC | 01:44 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Mask chap password in log https://review.openstack.org/271595 | 01:50 |
| *** sasukeh has quit IRC | 01:50 | |
| *** Suyash has quit IRC | 01:52 | |
| *** sasukeh has joined #openstack-cinder | 01:54 | |
| *** haomaiwang has quit IRC | 01:58 | |
| *** baojg has joined #openstack-cinder | 02:02 | |
| *** bardia_ has quit IRC | 02:06 | |
| *** bardia has joined #openstack-cinder | 02:07 | |
| *** Lee1092 has joined #openstack-cinder | 02:14 | |
| *** huanan has quit IRC | 02:22 | |
| *** Roamer` has quit IRC | 02:22 | |
| *** huanan has joined #openstack-cinder | 02:25 | |
| *** IlyaG has joined #openstack-cinder | 02:27 | |
| *** ebalduf has quit IRC | 02:29 | |
| *** yangyapeng has quit IRC | 02:29 | |
| *** IlyaG has quit IRC | 02:33 | |
| *** Roamer` has joined #openstack-cinder | 02:36 | |
| *** ociuhandu has quit IRC | 02:38 | |
| *** ociuhandu has joined #openstack-cinder | 02:39 | |
| *** eharney has quit IRC | 02:40 | |
| *** bardia has quit IRC | 02:43 | |
| *** bardia has joined #openstack-cinder | 02:44 | |
| *** houming has joined #openstack-cinder | 02:47 | |
| *** bardia has quit IRC | 02:51 | |
| *** SergeyLukjanov has quit IRC | 02:56 | |
| *** netapp has quit IRC | 02:58 | |
| *** RustShen has joined #openstack-cinder | 02:59 | |
| *** laughterwym has quit IRC | 02:59 | |
| *** dansmith has quit IRC | 03:01 | |
| *** netapp has joined #openstack-cinder | 03:01 | |
| *** laughterwym has joined #openstack-cinder | 03:01 | |
| *** dansmith has joined #openstack-cinder | 03:01 | |
| *** yangyapeng has joined #openstack-cinder | 03:02 | |
| *** SergeyLukjanov has joined #openstack-cinder | 03:02 | |
| *** sasukeh has quit IRC | 03:10 | |
| *** guy has joined #openstack-cinder | 03:14 | |
| *** dims has quit IRC | 03:14 | |
| *** gouthamr has quit IRC | 03:14 | |
| guy | Hi everyone, in volume metadata, can I store anything in key/value? | 03:15 |
| *** mriedem has quit IRC | 03:17 | |
| *** apoorvad has quit IRC | 03:18 | |
| *** SergeyLukjanov has quit IRC | 03:19 | |
| *** bapalm has quit IRC | 03:24 | |
| *** SergeyLukjanov has joined #openstack-cinder | 03:24 | |
| *** tjcocozz has quit IRC | 03:25 | |
| *** yangyape_ has joined #openstack-cinder | 03:27 | |
| *** markvoelker has joined #openstack-cinder | 03:29 | |
| *** yangyap__ has joined #openstack-cinder | 03:29 | |
| *** yangyapeng has quit IRC | 03:31 | |
| *** yangyape_ has quit IRC | 03:32 | |
| *** sasukeh has joined #openstack-cinder | 03:34 | |
| *** houming has quit IRC | 03:39 | |
| *** bapalm has joined #openstack-cinder | 03:43 | |
| *** tjcocozz has joined #openstack-cinder | 03:43 | |
| *** markvoelker has quit IRC | 03:43 | |
| *** houming has joined #openstack-cinder | 03:45 | |
| *** dustins has joined #openstack-cinder | 03:46 | |
| *** sasukeh has quit IRC | 03:46 | |
| *** links has joined #openstack-cinder | 03:50 | |
| *** cbinx has quit IRC | 03:52 | |
| *** frontrunner2 has quit IRC | 03:58 | |
| *** dims has joined #openstack-cinder | 04:01 | |
| *** jwcroppe has joined #openstack-cinder | 04:02 | |
| *** sasukeh has joined #openstack-cinder | 04:06 | |
| *** dustins has quit IRC | 04:08 | |
| *** ebalduf has joined #openstack-cinder | 04:12 | |
| *** dims has quit IRC | 04:13 | |
| *** jgriffith_ has joined #openstack-cinder | 04:21 | |
| *** jgriffith_ has quit IRC | 04:22 | |
| *** mmotiani has joined #openstack-cinder | 04:24 | |
| *** rongze has joined #openstack-cinder | 04:28 | |
| *** yangyapeng has joined #openstack-cinder | 04:35 | |
| *** IlyaG has joined #openstack-cinder | 04:37 | |
| *** yangyap__ has quit IRC | 04:37 | |
| *** shausy has joined #openstack-cinder | 04:39 | |
| *** jamielennox is now known as jamielennox|away | 04:41 | |
| *** laughterwym has quit IRC | 04:44 | |
| *** RustShen has quit IRC | 04:47 | |
| *** RustShen has joined #openstack-cinder | 04:47 | |
| *** jdg__ has joined #openstack-cinder | 04:51 | |
| jgriffith | sigh | 04:52 |
| *** kragniz has quit IRC | 04:52 | |
| *** kragniz has joined #openstack-cinder | 04:54 | |
| *** IlyaG has quit IRC | 05:02 | |
| *** kragniz has quit IRC | 05:05 | |
| *** kragniz has joined #openstack-cinder | 05:05 | |
| *** rongze has left #openstack-cinder | 05:10 | |
| *** houming has quit IRC | 05:11 | |
| *** EinstCrazy has joined #openstack-cinder | 05:21 | |
| *** RustShen has quit IRC | 05:23 | |
| *** RustShen has joined #openstack-cinder | 05:24 | |
| *** jwcroppe has quit IRC | 05:25 | |
| *** jwcroppe has joined #openstack-cinder | 05:25 | |
| *** RustShen has quit IRC | 05:28 | |
| *** yangyapeng has quit IRC | 05:28 | |
| *** sgotliv has joined #openstack-cinder | 05:29 | |
| *** jwcroppe has quit IRC | 05:30 | |
| *** laughterwym has joined #openstack-cinder | 05:30 | |
| *** laughter_ has joined #openstack-cinder | 05:31 | |
| *** laughter_ has quit IRC | 05:31 | |
| *** laughterwym has quit IRC | 05:34 | |
| *** dansmith has quit IRC | 05:40 | |
| *** dansmith has joined #openstack-cinder | 05:46 | |
| *** 16WAADQPV has joined #openstack-cinder | 05:49 | |
| *** RustShen has joined #openstack-cinder | 05:50 | |
| *** laughterwym has joined #openstack-cinder | 05:51 | |
| *** laughterwym has joined #openstack-cinder | 05:51 | |
| *** laughterwym has quit IRC | 05:51 | |
| *** laughter_ has joined #openstack-cinder | 05:52 | |
| *** Poornima has joined #openstack-cinder | 05:53 | |
| *** melwitt has quit IRC | 05:54 | |
| *** med_ has quit IRC | 05:56 | |
| *** mfisch has quit IRC | 05:56 | |
| *** med_ has joined #openstack-cinder | 05:56 | |
| *** med_ is now known as Guest93664 | 05:56 | |
| openstackgerrit | AMIT KUMAR proposed openstack/cinder: Setting the value of limit argument from the request. https://review.openstack.org/290533 | 05:58 |
| *** melwitt has joined #openstack-cinder | 05:59 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Remove unused logging import and LOG global var https://review.openstack.org/261472 | 06:00 |
| *** laughter_ has quit IRC | 06:00 | |
| *** 16WAADQPV has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-cinder | 06:01 | |
| *** laughterwym has joined #openstack-cinder | 06:01 | |
| *** RustShen has quit IRC | 06:01 | |
| *** laughterwym has quit IRC | 06:01 | |
| *** RustShen has joined #openstack-cinder | 06:02 | |
| *** laughterwym has joined #openstack-cinder | 06:02 | |
| *** dansmith has quit IRC | 06:02 | |
| openstackgerrit | AMIT KUMAR proposed openstack/cinder: Setting the value of limit argument from the request https://review.openstack.org/290533 | 06:03 |
| *** EinstCrazy has quit IRC | 06:05 | |
| *** RustShen has quit IRC | 06:07 | |
| *** RustShen_ has joined #openstack-cinder | 06:07 | |
| *** lprice has joined #openstack-cinder | 06:07 | |
| *** dansmith has joined #openstack-cinder | 06:08 | |
| *** lprice1 has joined #openstack-cinder | 06:13 | |
| *** lprice has quit IRC | 06:16 | |
| *** fthiagogv has quit IRC | 06:16 | |
| *** guy has quit IRC | 06:16 | |
| *** tjcocozz has quit IRC | 06:23 | |
| *** bapalm has quit IRC | 06:23 | |
| *** mfisch has joined #openstack-cinder | 06:27 | |
| *** mfisch is now known as Guest38484 | 06:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/291017 | 06:28 |
| *** avishay has joined #openstack-cinder | 06:31 | |
| *** bapalm has joined #openstack-cinder | 06:42 | |
| *** tjcocozz has joined #openstack-cinder | 06:45 | |
| *** melwitt has quit IRC | 06:45 | |
| *** laughter_ has joined #openstack-cinder | 06:47 | |
| *** laughterwym has quit IRC | 06:49 | |
| *** melwitt has joined #openstack-cinder | 06:50 | |
| *** melwitt is now known as Guest98534 | 06:50 | |
| *** mjura has joined #openstack-cinder | 06:51 | |
| *** nkrinner has joined #openstack-cinder | 06:53 | |
| openstackgerrit | liuke proposed openstack/cinder: Huawei: Creating hypermetro failed in the remote pool https://review.openstack.org/282633 | 06:54 |
| *** dave-mccowan has quit IRC | 06:55 | |
| *** avishay has quit IRC | 06:55 | |
| *** Guest98534 is now known as melwitt | 06:55 | |
| *** melwitt has quit IRC | 06:58 | |
| *** melwitt has joined #openstack-cinder | 07:00 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:01 | |
| *** melwitt has quit IRC | 07:03 | |
| *** haomaiwang has quit IRC | 07:04 | |
| *** melwitt has joined #openstack-cinder | 07:04 | |
| *** haomaiwang has joined #openstack-cinder | 07:04 | |
| *** melwitt is now known as Guest37020 | 07:05 | |
| *** skraynev has quit IRC | 07:05 | |
| *** skraynev has joined #openstack-cinder | 07:06 | |
| openstackgerrit | Abhishek Kekane proposed openstack/cinder: volume upload-to-image fails for iso disk-format https://review.openstack.org/287606 | 07:18 |
| *** laughter_ has quit IRC | 07:18 | |
| *** xiaohui has quit IRC | 07:18 | |
| *** laughterwym has joined #openstack-cinder | 07:19 | |
| *** xiaohui has joined #openstack-cinder | 07:19 | |
| *** ildikov has quit IRC | 07:22 | |
| *** liverpooler has quit IRC | 07:23 | |
| *** lpetrut has joined #openstack-cinder | 07:25 | |
| *** ebalduf has quit IRC | 07:25 | |
| *** user_8767 has joined #openstack-cinder | 07:27 | |
| user_8767 | Allah is doing | 07:27 |
| user_8767 | sun is not doing Allah is doing | 07:27 |
| user_8767 | moon is not doing Allah is doing | 07:28 |
| user_8767 | stars are not doing Allah is doing | 07:28 |
| user_8767 | planets are not doing Allah is doing | 07:28 |
| user_8767 | galaxies are not doing Allah is doing | 07:28 |
| user_8767 | oceans are not doing Allah is doing | 07:28 |
| *** abhishekk_ has quit IRC | 07:29 | |
| user_8767 | mountains are not doing Allah is doing | 07:29 |
| user_8767 | trees are not doing Allah is doing | 07:29 |
| user_8767 | mom is not doing Allah is doing | 07:29 |
| user_8767 | dad is not doing Allah is doing | 07:29 |
| user_8767 | boss is not doing Allah is doing | 07:29 |
| user_8767 | job is not doing Allah is doing | 07:30 |
| user_8767 | dollar is not doing Allah is doing | 07:30 |
| user_8767 | degree is not doing Allah is doing | 07:30 |
| user_8767 | medicine is not doing Allah is doing | 07:30 |
| *** melwitt_ has joined #openstack-cinder | 07:30 | |
| *** abhishekk has joined #openstack-cinder | 07:30 | |
| user_8767 | cuwtomers are not doing Allah is doing | 07:30 |
| user_8767 | customers are not doing Allah is doing | 07:30 |
| user_8767 | you can not get a job without the permission of allah | 07:31 |
| user_8767 | you can not get married without the permission of allah | 07:31 |
| user_8767 | nobody can get angry at you without the permission of allah | 07:31 |
| user_8767 | light is not doing Allah is doing | 07:32 |
| user_8767 | fan is not doing Allah is doing | 07:32 |
| user_8767 | businessess are not doing Allah is doing | 07:32 |
| *** melwitt_ has quit IRC | 07:32 | |
| user_8767 | america is not doing Allah is doing | 07:32 |
| user_8767 | fire can not burn without the permission of allah | 07:32 |
| user_8767 | knife can not cut without the permission of allah | 07:33 |
| user_8767 | rulers are not doing Allah is doing | 07:33 |
| user_8767 | governments are not doing Allah is doing | 07:33 |
| user_8767 | sleep is not doing Allah is doing | 07:33 |
| user_8767 | hunger is not doing Allah is doing | 07:33 |
| *** rcernin has joined #openstack-cinder | 07:33 | |
| user_8767 | food does not take away the hunger Allah takes away the hunger | 07:34 |
| user_8767 | water does not take away the thirst Allah takes away the thirst | 07:34 |
| user_8767 | seeing is not doing Allah is doing | 07:34 |
| user_8767 | hearing is not doimg | 07:34 |
| *** ildikov has joined #openstack-cinder | 07:34 | |
| user_8767 | hearing is not doing Allah is doing | 07:35 |
| user_8767 | seasons are not doing Allah is doing | 07:35 |
| user_8767 | weather is not doing Allah is doing | 07:35 |
| user_8767 | humans are not doing Allah is doing | 07:35 |
| user_8767 | animals are not doing Allah is doing | 07:35 |
| user_8767 | the best amongst you are those who learn and teach quran | 07:36 |
| user_8767 | one letter read from book of Allah amounts to one good deed and Allah multiplies one good deed ten times | 07:36 |
| user_8767 | hearts get rusted as does iron with water to remove rust from heart recitation of Quran and rememberance of death | 07:37 |
| user_8767 | heart is likened to a mirror | 07:37 |
| user_8767 | when a person commits one sin a black dot sustains the heart | 07:37 |
| *** melwitt_ has joined #openstack-cinder | 07:39 | |
| *** nickv has joined #openstack-cinder | 07:39 | |
| *** melwitt_ is now known as melwitt | 07:40 | |
| *** anshul has joined #openstack-cinder | 07:40 | |
| *** melwitt has quit IRC | 07:40 | |
| *** belmoreira has joined #openstack-cinder | 07:41 | |
| *** user_8767 has quit IRC | 07:41 | |
| *** nickv has quit IRC | 07:41 | |
| *** melwitt has joined #openstack-cinder | 07:41 | |
| *** laughterwym has quit IRC | 07:43 | |
| *** laughterwym has joined #openstack-cinder | 07:43 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Remove unused logging import and LOG global var https://review.openstack.org/261472 | 07:44 |
| *** laughter_ has joined #openstack-cinder | 07:45 | |
| *** laughte__ has joined #openstack-cinder | 07:47 | |
| *** laughterwym has quit IRC | 07:48 | |
| *** laughter_ has quit IRC | 07:49 | |
| *** laughterwym has joined #openstack-cinder | 07:50 | |
| *** laughte__ has quit IRC | 07:51 | |
| *** laughter_ has joined #openstack-cinder | 07:51 | |
| *** avishay has joined #openstack-cinder | 07:52 | |
| *** e0ne has joined #openstack-cinder | 07:53 | |
| *** laughterwym has quit IRC | 07:54 | |
| openstackgerrit | Alon Marx proposed openstack/cinder: IBM XIV/DS8K: Implement Replication v2.1 https://review.openstack.org/288880 | 07:55 |
| *** RustShen_ has quit IRC | 07:55 | |
| *** laughter_ has quit IRC | 07:56 | |
| *** RustShen has joined #openstack-cinder | 07:56 | |
| *** abhishekk_ has joined #openstack-cinder | 07:57 | |
| *** abhishekk has quit IRC | 07:58 | |
| *** RustShen has quit IRC | 07:59 | |
| *** RustShen has joined #openstack-cinder | 08:00 | |
| *** RustShen has quit IRC | 08:00 | |
| *** RustShen has joined #openstack-cinder | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: VNX: Update replication for v2.1 https://review.openstack.org/285139 | 08:01 |
| *** haomaiwang has joined #openstack-cinder | 08:01 | |
| *** e0ne has quit IRC | 08:21 | |
| *** e0ne has joined #openstack-cinder | 08:25 | |
| *** lpetrut has quit IRC | 08:27 | |
| *** gcb has joined #openstack-cinder | 08:28 | |
| *** [1]Thelo has joined #openstack-cinder | 08:30 | |
| *** Thelo has quit IRC | 08:33 | |
| *** [1]Thelo is now known as Thelo | 08:33 | |
| *** liverpooler has joined #openstack-cinder | 08:34 | |
| *** liverpooler has quit IRC | 08:35 | |
| *** permalac has joined #openstack-cinder | 08:35 | |
| *** permalac has quit IRC | 08:35 | |
| *** liverpooler has joined #openstack-cinder | 08:35 | |
| *** rhefner has quit IRC | 08:36 | |
| *** cppforlife_ has quit IRC | 08:36 | |
| *** zhiyan has quit IRC | 08:36 | |
| *** zhenguo_ has quit IRC | 08:36 | |
| *** fmccrthy has quit IRC | 08:36 | |
| *** zzzeek has quit IRC | 08:36 | |
| *** ameade has quit IRC | 08:36 | |
| *** scottda has quit IRC | 08:36 | |
| *** zzzeek has joined #openstack-cinder | 08:38 | |
| *** arecknag has joined #openstack-cinder | 08:38 | |
| *** zhiyan has joined #openstack-cinder | 08:39 | |
| *** scottda has joined #openstack-cinder | 08:39 | |
| *** ameade has joined #openstack-cinder | 08:40 | |
| *** rhefner has joined #openstack-cinder | 08:40 | |
| *** permalac has joined #openstack-cinder | 08:41 | |
| *** permalac has quit IRC | 08:41 | |
| *** RustShen has quit IRC | 08:41 | |
| *** RustShen has joined #openstack-cinder | 08:41 | |
| *** fmccrthy has joined #openstack-cinder | 08:42 | |
| *** e0ne has quit IRC | 08:42 | |
| *** cppforlife_ has joined #openstack-cinder | 08:44 | |
| *** zhenguo_ has joined #openstack-cinder | 08:44 | |
| *** lpetrut has joined #openstack-cinder | 08:46 | |
| *** zhonghua has joined #openstack-cinder | 08:47 | |
| *** zhangjn has quit IRC | 08:49 | |
| *** zhonghua-lee has quit IRC | 08:50 | |
| *** laughterwym has joined #openstack-cinder | 08:51 | |
| *** cheneydc has quit IRC | 08:54 | |
| *** zhangjn has joined #openstack-cinder | 08:55 | |
| *** cheneydc has joined #openstack-cinder | 08:55 | |
| *** avishay has quit IRC | 08:55 | |
| *** salv-orlando has joined #openstack-cinder | 08:55 | |
| *** laughterwym has quit IRC | 08:56 | |
| *** jordanP has joined #openstack-cinder | 08:56 | |
| *** yangyapeng has joined #openstack-cinder | 08:58 | |
| *** ishant has joined #openstack-cinder | 08:58 | |
| *** permalac_ has joined #openstack-cinder | 09:00 | |
| *** permalac_ has quit IRC | 09:00 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 09:01 | |
| *** ndipanov has joined #openstack-cinder | 09:03 | |
| *** lpetrut has quit IRC | 09:03 | |
| openstackgerrit | Béla Vancsics proposed openstack/cinder: Use assert(Not)Equal/Less(Equal)/Greater(Equal) https://review.openstack.org/284597 | 09:03 |
| *** RustShen_ has joined #openstack-cinder | 09:05 | |
| *** RustShen has quit IRC | 09:05 | |
| *** RustShen_ has quit IRC | 09:05 | |
| *** RustShen has joined #openstack-cinder | 09:06 | |
| *** jistr has joined #openstack-cinder | 09:09 | |
| *** mvk has joined #openstack-cinder | 09:10 | |
| *** gcb has quit IRC | 09:11 | |
| *** sasukeh has quit IRC | 09:15 | |
| *** salv-orlando has quit IRC | 09:20 | |
| *** gcb_ has joined #openstack-cinder | 09:24 | |
| *** openstackgerrit has quit IRC | 09:30 | |
| *** openstackgerrit_ has joined #openstack-cinder | 09:31 | |
| *** openstackgerrit_ is now known as openstackgerrit | 09:31 | |
| *** openstackgerrit has quit IRC | 09:31 | |
| *** openstackgerrit_ has joined #openstack-cinder | 09:32 | |
| *** openstackgerrit_ is now known as openstackgerrit | 09:32 | |
| *** openstackgerrit has quit IRC | 09:32 | |
| *** openstackgerrit_ has joined #openstack-cinder | 09:33 | |
| *** openstackgerrit_ is now known as openstackgerrit | 09:33 | |
| *** openstackgerrit has quit IRC | 09:33 | |
| *** e0ne has joined #openstack-cinder | 09:33 | |
| *** openstackgerrit_ has joined #openstack-cinder | 09:34 | |
| *** openstackgerrit_ is now known as openstackgerrit | 09:34 | |
| *** sasukeh has joined #openstack-cinder | 09:40 | |
| *** laughterwym has joined #openstack-cinder | 09:57 | |
| *** tpsilva has joined #openstack-cinder | 10:00 | |
| *** haomaiwa_ has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-cinder | 10:01 | |
| *** cheneydc has quit IRC | 10:01 | |
| *** laughter_ has joined #openstack-cinder | 10:02 | |
| *** laughterwym has quit IRC | 10:02 | |
| *** laughterwym has joined #openstack-cinder | 10:04 | |
| *** liverpooler has quit IRC | 10:05 | |
| *** laughter_ has quit IRC | 10:06 | |
| *** sayalilunkad_lap has joined #openstack-cinder | 10:06 | |
| *** sasukeh has quit IRC | 10:08 | |
| *** laughter_ has joined #openstack-cinder | 10:08 | |
| *** laughterwym has quit IRC | 10:09 | |
| *** laughter_ has quit IRC | 10:09 | |
| *** laughterwym has joined #openstack-cinder | 10:09 | |
| *** liverpooler has joined #openstack-cinder | 10:10 | |
| *** yangyapeng has quit IRC | 10:10 | |
| *** bjornar has joined #openstack-cinder | 10:15 | |
| *** liverpooler has quit IRC | 10:17 | |
| openstackgerrit | Abhishek Kekane proposed openstack/cinder: DB purge command fails due to FK constraint https://review.openstack.org/291083 | 10:18 |
| *** jdg__ has quit IRC | 10:20 | |
| *** jdg has joined #openstack-cinder | 10:21 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: VNX: Update replication for v2.1 https://review.openstack.org/285139 | 10:23 |
| *** lpetrut has joined #openstack-cinder | 10:28 | |
| *** liverpooler has joined #openstack-cinder | 10:29 | |
| *** RustShen has quit IRC | 10:33 | |
| *** sasukeh has joined #openstack-cinder | 10:38 | |
| *** mvk has quit IRC | 10:42 | |
| *** dims has joined #openstack-cinder | 10:47 | |
| *** bapalm has quit IRC | 10:51 | |
| *** dims has quit IRC | 10:52 | |
| *** eantyshev has joined #openstack-cinder | 10:53 | |
| *** baojg has quit IRC | 10:56 | |
| *** dims has joined #openstack-cinder | 10:56 | |
| *** ociuhandu has quit IRC | 10:56 | |
| *** baojg has joined #openstack-cinder | 10:56 | |
| *** salv-orlando has joined #openstack-cinder | 10:57 | |
| *** sasukeh has quit IRC | 10:59 | |
| *** sdague has joined #openstack-cinder | 11:00 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-cinder | 11:01 | |
| *** laughterwym has quit IRC | 11:01 | |
| *** baojg has quit IRC | 11:01 | |
| *** laughterwym has joined #openstack-cinder | 11:02 | |
| *** laughterwym has quit IRC | 11:02 | |
| *** laughterwym has joined #openstack-cinder | 11:02 | |
| *** cristicalin has joined #openstack-cinder | 11:06 | |
| *** bapalm has joined #openstack-cinder | 11:08 | |
| *** mvk has joined #openstack-cinder | 11:10 | |
| *** sasukeh has joined #openstack-cinder | 11:18 | |
| *** dims has quit IRC | 11:19 | |
| *** dims has joined #openstack-cinder | 11:21 | |
| *** avishay has joined #openstack-cinder | 11:22 | |
| *** bapalm has quit IRC | 11:24 | |
| *** ishant has quit IRC | 11:25 | |
| *** tjcocozz has quit IRC | 11:26 | |
| *** kmartin has quit IRC | 11:29 | |
| *** salv-orlando has quit IRC | 11:30 | |
| *** cristicalin has quit IRC | 11:35 | |
| *** merooney has joined #openstack-cinder | 11:38 | |
| *** EinstCrazy has joined #openstack-cinder | 11:40 | |
| *** laughter_ has joined #openstack-cinder | 11:42 | |
| *** tjcocozz has joined #openstack-cinder | 11:43 | |
| *** bapalm has joined #openstack-cinder | 11:44 | |
| *** laughterwym has quit IRC | 11:44 | |
| *** sayalilunkad_lap has quit IRC | 11:45 | |
| *** arecknag has quit IRC | 11:50 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Set expected_attrs in volume obj to what we join https://review.openstack.org/290569 | 11:51 |
| robcresswell | rhagarty: Left some more comments inline. I'd like a couple more improvements to be made if possible, but otherwise its looking really good. | 11:56 |
| *** dims has quit IRC | 11:56 | |
| robcresswell | rhagarty: Sorry, on this patch I meant https://review.openstack.org/#/c/257630/ | 11:56 |
| *** laughter_ has quit IRC | 11:57 | |
| *** laughterwym has joined #openstack-cinder | 11:57 | |
| *** laughterwym has quit IRC | 11:58 | |
| *** laughterwym has joined #openstack-cinder | 11:58 | |
| *** laughterwym has joined #openstack-cinder | 11:58 | |
| *** laughterwym has quit IRC | 11:59 | |
| *** laughterwym has joined #openstack-cinder | 11:59 | |
| *** haomaiwang has quit IRC | 12:01 | |
| *** laughterwym has quit IRC | 12:01 | |
| *** haomaiwang has joined #openstack-cinder | 12:01 | |
| *** laughterwym has joined #openstack-cinder | 12:02 | |
| *** laughterwym has quit IRC | 12:02 | |
| *** fthiagogv has joined #openstack-cinder | 12:03 | |
| *** laughterwym has joined #openstack-cinder | 12:04 | |
| *** dims has joined #openstack-cinder | 12:04 | |
| *** laughterwym has quit IRC | 12:05 | |
| *** erlon has joined #openstack-cinder | 12:05 | |
| *** laughterwym has joined #openstack-cinder | 12:05 | |
| *** laughterwym has quit IRC | 12:05 | |
| *** laughterwym has joined #openstack-cinder | 12:09 | |
| *** laughterwym has quit IRC | 12:09 | |
| *** laughterwym has joined #openstack-cinder | 12:12 | |
| *** laughterwym has quit IRC | 12:13 | |
| *** laughterwym has joined #openstack-cinder | 12:16 | |
| *** laughterwym has quit IRC | 12:20 | |
| *** laughterwym has joined #openstack-cinder | 12:20 | |
| *** laughterwym has joined #openstack-cinder | 12:20 | |
| *** yangyapeng has joined #openstack-cinder | 12:23 | |
| *** laughterwym has quit IRC | 12:26 | |
| *** laughterwym has joined #openstack-cinder | 12:26 | |
| *** dongshengyang has joined #openstack-cinder | 12:30 | |
| *** yangds has quit IRC | 12:30 | |
| *** laughterwym has quit IRC | 12:31 | |
| *** jwcroppe has joined #openstack-cinder | 12:34 | |
| dulek | geguileo: Hi, please take a look at my replies on 290569. Am I missing something with the joinedload of volume_type.extra_specs? | 12:35 |
| *** laughterwym has joined #openstack-cinder | 12:37 | |
| *** laughterwym has quit IRC | 12:38 | |
| geguileo | dulek: My bad, I misread the code | 12:41 |
| openstackgerrit | Alon Marx proposed openstack/cinder: IBM XIV/DS8K: Implement Replication v2.1 https://review.openstack.org/288880 | 12:45 |
| *** ildikov has quit IRC | 12:48 | |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: DRBD driver: resize volume if cloned image is larger https://review.openstack.org/291150 | 12:48 |
| *** merooney has quit IRC | 12:49 | |
| *** timcl has joined #openstack-cinder | 12:50 | |
| *** sayalilunkad has quit IRC | 12:51 | |
| *** sayalilunkad has joined #openstack-cinder | 12:52 | |
| dulek | geguileo: Okay, but let me fix the other comments you've mentioned - I believe these are worth fixing. :) | 12:52 |
| *** cheneydc has joined #openstack-cinder | 12:52 | |
| geguileo | dulek: Ok, since they were already there I didn't want to downvote for that :-) | 12:53 |
| *** e0ne_ has joined #openstack-cinder | 12:53 | |
| *** e0ne has quit IRC | 12:54 | |
| *** timcl has quit IRC | 12:54 | |
| *** timcl has joined #openstack-cinder | 12:55 | |
| *** cdelatte has quit IRC | 12:57 | |
| *** gouthamr has joined #openstack-cinder | 12:57 | |
| *** porrua has joined #openstack-cinder | 13:00 | |
| *** haomaiwang has quit IRC | 13:01 | |
| *** haomaiwang has joined #openstack-cinder | 13:01 | |
| zigo | Guys, I have an issue with building Cinder backport for Jessie: https://mitaka-jessie.pkgs.mirantis.com/job/cinder/25/console | 13:01 |
| zigo | Does anyone know what's going on with this failed unit test? | 13:02 |
| zigo | It looks like the create operation worked, but not the delete. | 13:03 |
| dulek | zigo: Ah, it's about notifications. | 13:03 |
| zigo | dulek: Could you expand on this? :) | 13:04 |
| dulek | zigo: I'm not sure, but isn't it possible this is kind of race condition related to mocking config options? | 13:04 |
| zigo | dulek: Oh, like if I restart a build, it may work? | 13:04 |
| * zigo relaunches a build | 13:05 | |
| dulek | zigo: I think so. Or you may try also to set test runner to single thread. | 13:05 |
| zigo | dulek: That's the default unless I do --parallel in the "testr run", no? | 13:06 |
| zigo | (which is my case) | 13:06 |
| zigo | Let's wait for this 2nd build, if it fails again, I'll disable this test. | 13:06 |
| dulek | zigo: In master it defaults to number of cores on the machine I think. | 13:06 |
| *** cheneydc has quit IRC | 13:07 | |
| zigo | dulek: tox.ini seem to use ostestr, which I'm not (I'm simply using testr init / testr run when building the Debian package). | 13:08 |
| dulek | zigo: Okay, let's see the second build results. | 13:08 |
| zigo | So I believe I'm using a single thread. | 13:08 |
| zigo | Yup. | 13:08 |
| zigo | It will take a while to get in my build queue, as I've scheduled other stuff as well. | 13:09 |
| *** frontrunner has joined #openstack-cinder | 13:10 | |
| *** haomaiwang has quit IRC | 13:11 | |
| *** cheneydc has joined #openstack-cinder | 13:13 | |
| *** dave-mccowan has joined #openstack-cinder | 13:13 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Host selection in backup service https://review.openstack.org/290248 | 13:14 |
| openstackgerrit | LisaLi proposed openstack/cinder: Host selection in backup service https://review.openstack.org/290248 | 13:16 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Add volume_type to volume object expected_attrs https://review.openstack.org/290569 | 13:19 |
| *** cdelatte has joined #openstack-cinder | 13:20 | |
| *** edmondsw has joined #openstack-cinder | 13:21 | |
| *** merooney has joined #openstack-cinder | 13:25 | |
| *** mtanino has joined #openstack-cinder | 13:27 | |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: vzstorage: add ploop volume format support https://review.openstack.org/262246 | 13:28 |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: Use versionedobjects in remotefs.py https://review.openstack.org/261558 | 13:28 |
| *** lixiaoy1 has joined #openstack-cinder | 13:28 | |
| *** laughterwym has joined #openstack-cinder | 13:29 | |
| *** sayalilunkad has quit IRC | 13:32 | |
| *** yangyapeng has quit IRC | 13:35 | |
| *** sayalilunkad has joined #openstack-cinder | 13:35 | |
| *** yangyapeng has joined #openstack-cinder | 13:36 | |
| *** yangyapeng has quit IRC | 13:37 | |
| openstackgerrit | Alon Marx proposed openstack/cinder: IBM XIV/DS8K: Implement Replication v2.1 https://review.openstack.org/288880 | 13:38 |
| *** yangyapeng has joined #openstack-cinder | 13:38 | |
| *** dims has quit IRC | 13:38 | |
| *** dims has joined #openstack-cinder | 13:39 | |
| *** porunov has joined #openstack-cinder | 13:40 | |
| *** Poornima has quit IRC | 13:41 | |
| *** yangyape_ has joined #openstack-cinder | 13:41 | |
| *** yangyapeng has quit IRC | 13:43 | |
| *** Poornima has joined #openstack-cinder | 13:44 | |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: SMBFS: fix parsing volume type extra specs and metadata https://review.openstack.org/262249 | 13:46 |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: SMBFS: fix parsing volume type extra specs and metadata https://review.openstack.org/262249 | 13:48 |
| *** links has quit IRC | 13:48 | |
| *** delattec has joined #openstack-cinder | 13:49 | |
| *** markvoelker has joined #openstack-cinder | 13:50 | |
| *** cdelatte has quit IRC | 13:52 | |
| *** akerr has joined #openstack-cinder | 13:53 | |
| *** rlrossit has joined #openstack-cinder | 13:55 | |
| *** mriedem has joined #openstack-cinder | 13:59 | |
| *** salv-orlando has joined #openstack-cinder | 13:59 | |
| *** iceyao has joined #openstack-cinder | 14:00 | |
| *** cknight has joined #openstack-cinder | 14:02 | |
| *** openstackgerrit has quit IRC | 14:03 | |
| *** akshai has joined #openstack-cinder | 14:04 | |
| *** merooney has quit IRC | 14:04 | |
| *** jwcroppe has quit IRC | 14:04 | |
| *** Julien-zte has joined #openstack-cinder | 14:04 | |
| *** haomaiwa_ has joined #openstack-cinder | 14:04 | |
| *** openstackgerrit_ has joined #openstack-cinder | 14:04 | |
| *** openstackgerrit_ is now known as openstackgerrit | 14:04 | |
| *** openstackgerrit has quit IRC | 14:04 | |
| *** jwcroppe has joined #openstack-cinder | 14:04 | |
| *** avishay has quit IRC | 14:05 | |
| *** openstackgerrit_ has joined #openstack-cinder | 14:05 | |
| *** openstackgerrit_ is now known as openstackgerrit | 14:05 | |
| *** openstackgerrit has quit IRC | 14:05 | |
| *** openstackgerrit_ has joined #openstack-cinder | 14:06 | |
| *** openstackgerrit_ is now known as openstackgerrit | 14:06 | |
| *** openstackgerrit has quit IRC | 14:07 | |
| *** openstackgerrit_ has joined #openstack-cinder | 14:07 | |
| *** openstackgerrit_ is now known as openstackgerrit | 14:08 | |
| *** openstackgerrit has quit IRC | 14:08 | |
| *** openstackgerrit_ has joined #openstack-cinder | 14:09 | |
| *** openstackgerrit_ is now known as openstackgerrit | 14:09 | |
| *** jwcroppe has quit IRC | 14:09 | |
| *** openstackgerrit has quit IRC | 14:09 | |
| *** laughterwym has quit IRC | 14:09 | |
| *** openstackgerrit_ has joined #openstack-cinder | 14:10 | |
| *** openstackgerrit_ is now known as openstackgerrit | 14:10 | |
| *** laughterwym has joined #openstack-cinder | 14:11 | |
| *** Guest41345 has joined #openstack-cinder | 14:11 | |
| *** laughterwym has quit IRC | 14:13 | |
| *** laughterwym has joined #openstack-cinder | 14:14 | |
| flip214 | would someone like to be the second +2 for a DRBD patch? https://review.openstack.org/#/c/287807 | 14:16 |
| *** jgregor has joined #openstack-cinder | 14:17 | |
| *** salv-orlando has quit IRC | 14:19 | |
| *** laughter_ has joined #openstack-cinder | 14:20 | |
| *** dustins has joined #openstack-cinder | 14:20 | |
| *** mtanino has quit IRC | 14:22 | |
| *** laughterwym has quit IRC | 14:24 | |
| *** ildikov has joined #openstack-cinder | 14:25 | |
| *** salv-orlando has joined #openstack-cinder | 14:26 | |
| openstackgerrit | Bharath kumar proposed openstack/cinder: Remove the image-volume db entry if cloned_volume() fails If the cloned_volume() functions fails the db entry will happen ,even if already exists in the db.So removing db cache entry from the db if it exists , whenever a call to create the image-volume ca https://review.openstack.org/291204 | 14:27 |
| *** dalgaaf has joined #openstack-cinder | 14:28 | |
| *** dims has quit IRC | 14:29 | |
| *** salv-orlando has quit IRC | 14:29 | |
| *** laughter_ has quit IRC | 14:29 | |
| *** laughterwym has joined #openstack-cinder | 14:29 | |
| *** ociuhandu has joined #openstack-cinder | 14:30 | |
| *** timcl has quit IRC | 14:30 | |
| *** laughter_ has joined #openstack-cinder | 14:31 | |
| *** bharath has joined #openstack-cinder | 14:31 | |
| *** Julien-zte has quit IRC | 14:32 | |
| *** timcl has joined #openstack-cinder | 14:33 | |
| *** baumann has joined #openstack-cinder | 14:33 | |
| *** xyang1 has joined #openstack-cinder | 14:34 | |
| *** laughterwym has quit IRC | 14:34 | |
| *** edtubill has joined #openstack-cinder | 14:35 | |
| *** dims has joined #openstack-cinder | 14:35 | |
| *** edtubill has quit IRC | 14:40 | |
| *** edtubill has joined #openstack-cinder | 14:43 | |
| *** laughter_ has quit IRC | 14:44 | |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: SMBFS: fix parsing volume type extra specs and metadata https://review.openstack.org/262249 | 14:44 |
| openstackgerrit | Bharath kumar proposed openstack/cinder: Remove the image-volume db entry if cloned_volume() fails https://review.openstack.org/291204 | 14:45 |
| zigo | dulek: Indeed, the 2nd build of Cinder passed without failure. I'm therefore filing a bug, reporting this unit test as failing non-deterministically. | 14:46 |
| zigo | https://bugs.launchpad.net/cinder/+bug/1555668 | 14:49 |
| openstack | Launchpad bug 1555668 in Cinder "test_create_delete_volume fails non-deterministically" [Undecided,New] | 14:49 |
| *** salv-orlando has joined #openstack-cinder | 14:50 | |
| *** edtubill has quit IRC | 14:50 | |
| *** eharney has joined #openstack-cinder | 14:52 | |
| *** Zhongjun_ has joined #openstack-cinder | 14:52 | |
| *** markstur has joined #openstack-cinder | 14:54 | |
| *** markstur has left #openstack-cinder | 14:55 | |
| *** edtubill has joined #openstack-cinder | 14:58 | |
| *** bharath has quit IRC | 14:58 | |
| *** belmoreira has quit IRC | 14:59 | |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: Support backup import on another Storage database https://review.openstack.org/153969 | 15:00 |
| *** haomaiwa_ has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-cinder | 15:01 | |
| *** mtanino has joined #openstack-cinder | 15:01 | |
| *** Poornima has quit IRC | 15:02 | |
| *** edtubill has quit IRC | 15:02 | |
| *** iceyao has quit IRC | 15:02 | |
| *** iceyao has joined #openstack-cinder | 15:03 | |
| *** ebalduf has joined #openstack-cinder | 15:03 | |
| *** mmandel has joined #openstack-cinder | 15:04 | |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: Support backup import on another Storage database https://review.openstack.org/153969 | 15:05 |
| *** iceyao has quit IRC | 15:06 | |
| krotscheck | smcginnis: Hey, has RC1 been cut yet? Do I need to port https://review.openstack.org/#/c/286795/ to a different branch? | 15:11 |
| smcginnis | krotscheck: Not yet. That will be next week. | 15:11 |
| smcginnis | krotscheck: We need that in M, right? | 15:12 |
| *** laughterwym has joined #openstack-cinder | 15:12 | |
| krotscheck | smcginnis: Yep. | 15:13 |
| krotscheck | smcginnis: Thanks - I'll see if I can solicit some more cores. | 15:13 |
| smcginnis | krotscheck: K, thought so. Thanks! | 15:13 |
| smcginnis | xyang1, DuncanT, and other cores: Take a peek if you have time: https://review.openstack.org/#/c/286795/ | 15:14 |
| xyang1 | smcginnis: sure | 15:14 |
| krotscheck | xyang1: Thanks :) | 15:14 |
| smcginnis | xyang1: Thank you. :) | 15:14 |
| *** laughterwym has quit IRC | 15:15 | |
| *** laughterwym has joined #openstack-cinder | 15:15 | |
| *** salv-orlando has quit IRC | 15:16 | |
| *** adrianofr_ has quit IRC | 15:17 | |
| *** arch-nemesis has joined #openstack-cinder | 15:17 | |
| *** Yogi1 has joined #openstack-cinder | 15:17 | |
| openstackgerrit | Merged openstack/cinder: Imported Translations from Zanata https://review.openstack.org/291017 | 15:18 |
| *** adrianofr_ has joined #openstack-cinder | 15:18 | |
| *** knikolla has joined #openstack-cinder | 15:19 | |
| *** laughterwym has quit IRC | 15:20 | |
| mtanino | dulek: Hi, are you there? | 15:23 |
| mtanino | dulek: I've updated this based on your comments. https://review.openstack.org/#/c/274204/ Thanks :) | 15:23 |
| dulek | mtanino: I've seen that, need to take a closer look. | 15:24 |
| mtanino | dulek: Thank you :) anytime if you have a chance. | 15:25 |
| *** crose has joined #openstack-cinder | 15:25 | |
| *** cheneydc has quit IRC | 15:29 | |
| *** sasukeh has quit IRC | 15:31 | |
| scottda | Crap! I didn't get approved for travel to Austin. | 15:32 |
| smcginnis | scottda: Seriously?!? | 15:32 |
| scottda | yup | 15:32 |
| *** eharney has quit IRC | 15:32 | |
| smcginnis | scottda: Hmm, anyone I can ping to change minds? | 15:32 |
| scottda | I might go anyway, on my own dime. | 15:32 |
| smcginnis | Man... | 15:33 |
| scottda | smcginnis: Thanks for the offer, but I don't think that would help. Just let me know if anyone is hiring :) | 15:33 |
| smcginnis | :) | 15:33 |
| *** alyson_ has joined #openstack-cinder | 15:33 | |
| smcginnis | I hear there's a storage company down the road that's pretty involved in OpenStack. | 15:33 |
| *** aorourke-afk is now known as aorourke | 15:34 | |
| *** laughterwym has joined #openstack-cinder | 15:34 | |
| scottda | Yeah, but I've heard their tech lead is a real tyrant. | 15:34 |
| smcginnis | Your coworkers might be a pain to work with, but they don't seem too bad. :D | 15:34 |
| *** eharney has joined #openstack-cinder | 15:35 | |
| *** cknight has left #openstack-cinder | 15:37 | |
| flip214 | would some core please add another review to smcginnis' one at https://review.openstack.org/#/c/287807? Thanks a lot. | 15:38 |
| *** cknight has joined #openstack-cinder | 15:38 | |
| *** IlyaG has joined #openstack-cinder | 15:38 | |
| *** laughterwym has quit IRC | 15:39 | |
| *** jwcroppe has joined #openstack-cinder | 15:39 | |
| *** laughterwym has joined #openstack-cinder | 15:43 | |
| *** laughterwym has quit IRC | 15:45 | |
| *** laughterwym has joined #openstack-cinder | 15:46 | |
| *** laughter_ has joined #openstack-cinder | 15:46 | |
| *** Yogi1 has quit IRC | 15:46 | |
| *** EinstCrazy has quit IRC | 15:49 | |
| *** laughterwym has quit IRC | 15:50 | |
| *** ildikov has quit IRC | 15:50 | |
| *** dustins has quit IRC | 15:52 | |
| *** shyh has joined #openstack-cinder | 15:56 | |
| shyh | Can some one please help me with a review https://review.openstack.org/#/c/286116 | 15:56 |
| *** kfarr has joined #openstack-cinder | 15:59 | |
| flip214 | Thanks a lot! | 15:59 |
| *** jistr has quit IRC | 15:59 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-cinder | 16:01 | |
| *** huanan has quit IRC | 16:04 | |
| *** huanan has joined #openstack-cinder | 16:05 | |
| *** avishay has joined #openstack-cinder | 16:06 | |
| *** haomaiwang has quit IRC | 16:07 | |
| *** nkrinner has quit IRC | 16:08 | |
| mtanino | Hi, Would any core please review the policy bug fix? https://review.openstack.org/#/c/274204/ Want to push this to RC-1. Thanks! | 16:09 |
| *** mjura has quit IRC | 16:09 | |
| *** dustins has joined #openstack-cinder | 16:10 | |
| openstackgerrit | Thomas Bechtold proposed openstack/cinder: Add apiclient to requirements.txt https://review.openstack.org/291271 | 16:10 |
| geguileo | smcginnis: ping - missing release note? | 16:10 |
| *** darosale has joined #openstack-cinder | 16:15 | |
| *** kmartin has joined #openstack-cinder | 16:16 | |
| *** sayalilunkad has quit IRC | 16:16 | |
| smcginnis | geguileo: Where's that? | 16:17 |
| *** sayalilunkad has joined #openstack-cinder | 16:19 | |
| *** edtubill has joined #openstack-cinder | 16:20 | |
| *** jgregor1 has joined #openstack-cinder | 16:23 | |
| *** jgregor has quit IRC | 16:23 | |
| *** bharath has joined #openstack-cinder | 16:28 | |
| *** ildikov has joined #openstack-cinder | 16:29 | |
| *** leeantho has joined #openstack-cinder | 16:29 | |
| *** bjornar has quit IRC | 16:29 | |
| *** jdurgin1 has joined #openstack-cinder | 16:31 | |
| *** bardia has joined #openstack-cinder | 16:31 | |
| *** bharathk has joined #openstack-cinder | 16:32 | |
| *** bharath has quit IRC | 16:32 | |
| *** kfarr has quit IRC | 16:33 | |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: DRBD driver: resize volume if cloned image is larger https://review.openstack.org/291150 | 16:37 |
| *** shausy has quit IRC | 16:37 | |
| geguileo | smcginnis: Sorry, was in a meeting | 16:41 |
| geguileo | smcginnis: I don't see a release note for the feature where you can force delete a volume with the snapshots | 16:41 |
| geguileo | smcginnis: And I don't know if it's missing or I'm looking in the wrong place | 16:42 |
| smcginnis | geguileo: That one merged, right? There's not one in current master for it. | 16:43 |
| eharney | oops, i didn't reno that one i think | 16:43 |
| geguileo | smcginnis: I cannot find it | 16:43 |
| geguileo | eharney: Baaaaaaaaaaaaaaaad! | 16:43 |
| smcginnis | eharney: I thought that was your, but having trouble tracking it down. | 16:43 |
| smcginnis | Oh, derp | 16:44 |
| smcginnis | https://review.openstack.org/#/c/264982/ | 16:44 |
| smcginnis | Easy enough to add one now. | 16:44 |
| smcginnis | You want to grab that Eric? Or one of us can. | 16:44 |
| eharney | smcginnis: sure i'll do it | 16:45 |
| smcginnis | eharney: Thanks! | 16:45 |
| smcginnis | geguileo: Good catch. I don't doubt we have others missing, but hopefully we've captured most of the big ones. | 16:46 |
| geguileo | I was reviewing them to see if we had missed something else | 16:46 |
| openstackgerrit | Merged openstack/cinder: LeftHand: Create cloned volume didn't honor size https://review.openstack.org/290890 | 16:46 |
| *** e0ne_ has quit IRC | 16:46 | |
| geguileo | and I don't remember any other one getting fixed, except the API races that we have removed | 16:47 |
| shyh | Can some one please help me with a review https://review.openstack.org/#/c/286116 | 16:47 |
| openstackgerrit | Merged openstack/cinder: Storwize: Update replication to v2.1 https://review.openstack.org/287483 | 16:48 |
| openstackgerrit | Merged openstack/cinder: Cleanup Mitaka release notes https://review.openstack.org/290100 | 16:48 |
| smcginnis | shyh: Can you push up an update with the trivial fixes I pointed out? That might help. | 16:49 |
| eharney | smcginnis: are we doing another client release for mitaka? the cascade delete option missed the last one (trying to figure out what to write about that) | 16:53 |
| smcginnis | eharney: No, I think we're too late for M. But we can get one soon after feature freeze is over. | 16:53 |
| eharney | smcginnis: no problem | 16:53 |
| *** avishay has quit IRC | 16:56 | |
| *** jwcroppe has quit IRC | 16:58 | |
| *** jwcroppe has joined #openstack-cinder | 16:59 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Permit volume type operations for policy authorized users https://review.openstack.org/274204 | 16:59 |
| openstackgerrit | Merged openstack/cinder: Moved CORS middleware configuration into oslo-config-generator https://review.openstack.org/286795 | 16:59 |
| *** mrrowden has joined #openstack-cinder | 16:59 | |
| shyh | Thanks sean, will do | 17:00 |
| smcginnis | shyh: Thanks! | 17:00 |
| openstackgerrit | Merged openstack/cinder: Add volume_type to volume object expected_attrs https://review.openstack.org/290569 | 17:01 |
| openstackgerrit | Merged openstack/cinder: DRBD: Policy-based waiting for completion https://review.openstack.org/287807 | 17:02 |
| openstackgerrit | Merged openstack/cinder: Switch failover-host from rpc call to cast https://review.openstack.org/290857 | 17:02 |
| openstackgerrit | Merged openstack/cinder: Show qos_specs_id based on policy https://review.openstack.org/241659 | 17:02 |
| *** EinstCrazy has joined #openstack-cinder | 17:03 | |
| *** jwcroppe has quit IRC | 17:03 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Add release note for delete volume with snaps https://review.openstack.org/291302 | 17:03 |
| eharney | smcginnis: geguileo: release note ^ | 17:03 |
| smcginnis | eharney: Thank you | 17:04 |
| *** leeantho has quit IRC | 17:04 | |
| geguileo | eharney: Thanks | 17:04 |
| *** dims has quit IRC | 17:04 | |
| *** ndipanov has quit IRC | 17:04 | |
| *** dims has joined #openstack-cinder | 17:06 | |
| *** laughter_ has quit IRC | 17:07 | |
| *** timcl has quit IRC | 17:07 | |
| *** bradjones has joined #openstack-cinder | 17:08 | |
| openstackgerrit | Merged openstack/cinder: NetApp: volume resize using clone fails with QoS https://review.openstack.org/289910 | 17:08 |
| *** EinstCrazy has quit IRC | 17:08 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Add release note for delete volume with snaps https://review.openstack.org/291302 | 17:10 |
| *** laughterwym has joined #openstack-cinder | 17:11 | |
| *** anshul has quit IRC | 17:16 | |
| openstackgerrit | Michael Price proposed openstack/cinder: Implement CHAP Authentication for E-Series Driver https://review.openstack.org/246990 | 17:16 |
| *** cbinx has joined #openstack-cinder | 17:18 | |
| jgriffith | smcginnis: ping | 17:20 |
| smcginnis | jgriffith: Hey! | 17:21 |
| jgriffith | smcginnis: yo! | 17:21 |
| *** fthiagogv has quit IRC | 17:21 | |
| jgriffith | smcginnis: when you were looking at drivers for the extend on clone thing.... | 17:21 |
| jgriffith | smcginnis: did you happen to get a feel for how many did NOT implement extend? | 17:21 |
| jgriffith | smcginnis: I'm asking because I'm wondering if it's better/easier to put the logic up in the manager, and just call the driver methods from there | 17:21 |
| smcginnis | jgriffith: I would say the majority did not. Probably ~75%. | 17:22 |
| jgriffith | smcginnis: as opposed to have everybody patch their driver | 17:22 |
| jgriffith | Ohhh! | 17:22 |
| jgriffith | never mind then :) | 17:22 |
| smcginnis | ALl of the remotefs ones, many others. | 17:22 |
| jgriffith | although that would raise the exception | 17:22 |
| smcginnis | jgriffith: I do think it would be easier to put the logic in the manager. | 17:22 |
| smcginnis | jgriffith: I think I had suggested that. | 17:22 |
| jgriffith | smcginnis: I think you did yes | 17:22 |
| smcginnis | jgriffith: The one drawback would be optimization. | 17:23 |
| jgriffith | smcginnis: but I was on a tangent about using capability reporting at the API layer | 17:23 |
| jgriffith | smcginnis: hmmm... good point | 17:23 |
| smcginnis | In that if for some storage it is easier to just create the new volume at the new size. | 17:23 |
| jgriffith | smcginnis: that actually would be an issue for someone like me | 17:23 |
| smcginnis | Rather than clone, then extend. | 17:23 |
| jgriffith | ok.. never mind :) | 17:23 |
| smcginnis | But I think we could support that. | 17:23 |
| smcginnis | We would probably have to change the interface slightly though. | 17:23 |
| jgriffith | yeah | 17:23 |
| *** haomaiwang has joined #openstack-cinder | 17:23 | |
| smcginnis | Pass in the new desired size, check a result to see if the new volume is that size, if not call extend. | 17:24 |
| jgriffith | I guess either way it's not something we should be mucking with at this stage of the release | 17:24 |
| smcginnis | jgriffith: Yeah, probably not. | 17:24 |
| jgriffith | smcginnis: Oh... I like that idea | 17:24 |
| jgriffith | smcginnis: best of both | 17:24 |
| smcginnis | jgriffith: On the plus side, many of the filed bugs have already been fixed or patches proposed. | 17:24 |
| jgriffith | Well in that case!!! Excellent! | 17:24 |
| smcginnis | Since many have obviously been "broken" from day 1, it's at least an improvement. ;) | 17:25 |
| *** haomaiwang has quit IRC | 17:28 | |
| Swanson | I fixed it so keep it the way it is. | 17:28 |
| jgriffith | Swanson: :) | 17:31 |
| *** shyh has quit IRC | 17:32 | |
| *** esker has joined #openstack-cinder | 17:32 | |
| Swanson | jgriffith, Looks like the reviews for init time shenanigans hit on your alternative suggestion. | 17:32 |
| jgriffith | uh oh | 17:33 |
| *** apoorvad has joined #openstack-cinder | 17:34 | |
| jgriffith | sigh.... ok | 17:34 |
| jgriffith | I'll change it again | 17:34 |
| Swanson | jgriffith, just going to move it to the init rpc bit? | 17:35 |
| jgriffith | yeah | 17:35 |
| *** esker has quit IRC | 17:35 | |
| *** esker has joined #openstack-cinder | 17:35 | |
| *** laughterwym has quit IRC | 17:36 | |
| Swanson | jgriffith, that kibosh the model_update idea in the future or does that just require more selling to get it in than we have before rc1? | 17:37 |
| *** liverpooler has quit IRC | 17:37 | |
| jgriffith | Swanson: probably kaboshes it | 17:37 |
| jgriffith | Swanson: but frankly I think in N it would be good to clean up that spaghetti | 17:38 |
| jgriffith | independent of this whole rep thing | 17:38 |
| *** Zhongjun_ has quit IRC | 17:38 | |
| Swanson | jgriffith, seems reasonable. Or make a 4th init function and just go for it. | 17:38 |
| jgriffith | Swanson: grrrrrrrrr | 17:39 |
| jgriffith | Swanson: why not, everybody else is doing it :) | 17:39 |
| Swanson | everybody gets a trophy init function! We're all winners! | 17:40 |
| *** mgarza has joined #openstack-cinder | 17:40 | |
| jgriffith | LOL | 17:40 |
| hemna | I'm working on a patch to the 3PAR drivers to get extend to work | 17:42 |
| hemna | it's a PITA | 17:42 |
| *** bardia has quit IRC | 17:45 | |
| *** edtubill has quit IRC | 17:47 | |
| openstackgerrit | Bharath kumar proposed openstack/cinder: Remove the image-volume db entry if cloned_volume() fails https://review.openstack.org/291204 | 17:49 |
| *** timcl has joined #openstack-cinder | 17:50 | |
| *** jdurgin1 has quit IRC | 17:51 | |
| *** e0ne has joined #openstack-cinder | 17:53 | |
| *** dims has quit IRC | 17:54 | |
| *** bjornar has joined #openstack-cinder | 17:58 | |
| openstackgerrit | Doug Hellmann proposed openstack/os-brick: Update reno for stable/mitaka https://review.openstack.org/291330 | 17:58 |
| openstackgerrit | Doug Hellmann proposed openstack/python-cinderclient: Update reno for stable/mitaka https://review.openstack.org/291332 | 18:00 |
| *** timcl has quit IRC | 18:01 | |
| *** mriedem has quit IRC | 18:01 | |
| e0ne | eharney: hi. I've got a question about your patch https://review.openstack.org/178262 - RBD Thin Provisioning stats | 18:03 |
| *** dims has joined #openstack-cinder | 18:03 | |
| e0ne | eharney: TBH, I didn't test it yet but | 18:04 |
| *** jungleboyj has joined #openstack-cinder | 18:04 | |
| e0ne | eharney: will 'rbd du' command from ceph infernalis (http://docs.ceph.com/docs/master/release-notes/#v9.2.0-infernalis) will help us to fix this bug? | 18:04 |
| *** wilson_liu has quit IRC | 18:06 | |
| *** wilson_liu has joined #openstack-cinder | 18:06 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Move replication_status update to service.py https://review.openstack.org/290917 | 18:11 |
| jgriffith | Swanson: dulek patrickeast ^^ | 18:11 |
| *** jordanP has quit IRC | 18:12 | |
| *** laughterwym has joined #openstack-cinder | 18:12 | |
| *** mriedem has joined #openstack-cinder | 18:13 | |
| dulek | jgriffith: Looking now. :) | 18:13 |
| jgriffith | geguileo: I'm curious, what are your objections to replication status being in the service table? | 18:14 |
| jgriffith | geguileo: given that replication is designed to be a "service based" feature... why would that seem wrong? | 18:15 |
| jgriffith | geguileo: do you have an alternative suggestion? | 18:15 |
| geguileo | jgriffith: Maybe I'm misunderstanding something | 18:15 |
| patrickeast | haha, guess the other init method won out after all | 18:15 |
| jgriffith | patrickeast: :) | 18:15 |
| geguileo | jgriffith: We don't have replication on scheduler or backups, right? | 18:16 |
| jgriffith | patrickeast: you must have psychic powers | 18:16 |
| patrickeast | lol | 18:16 |
| jgriffith | geguileo: no, but those services aren't replicatable (at least not right now) | 18:16 |
| *** lpetrut has quit IRC | 18:16 | |
| jgriffith | geguileo: they certainly could be made to be though | 18:16 |
| geguileo | So why should that code be in the generic service? | 18:16 |
| jgriffith | geguileo: it would follow the same design pattern... redundant services | 18:16 |
| jgriffith | geguileo: because it's an attribute of the service | 18:17 |
| jgriffith | geguileo: it's just an extension of disabled really | 18:18 |
| geguileo | jgriffith: Yeah, but I think those fields should never had made it that table in the first place | 18:18 |
| patrickeast | im wondering if maybe once tiramisu comes around and we try and re-introduce more fine grained replication things if we wont just end up with a replications table or something and a column that might just say 'volume' 'group' 'c-vol' etc | 18:18 |
| geguileo | jgriffith: Because they only apply to c-vol | 18:18 |
| patrickeast | that has the status and all that junk in there | 18:18 |
| *** esker has quit IRC | 18:18 | |
| jgriffith | geguileo: ok, and again... I'll ask; how would you address the problem then? | 18:18 |
| jgriffith | geguileo: would you make it a foreign key? | 18:18 |
| *** esker has joined #openstack-cinder | 18:18 | |
| geguileo | jgriffith: Another table that contains replication status for each service and only c-vol nodes adds data in there? | 18:19 |
| dulek | jgriffith: -1, sorry, but if I'm not missing something there's one problematic codepath. | 18:19 |
| dulek | geguileo's point seems valid to me. But it's not catastrophic IMO that we have it in services. | 18:20 |
| *** laughterwym has quit IRC | 18:20 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Lazy load a project quotas's default values https://review.openstack.org/279304 | 18:21 |
| *** dims has quit IRC | 18:21 | |
| geguileo | dulek: Well, catastrophic to me is that we have those fields in the DB, this would pale in comparison, I agree | 18:22 |
| geguileo | dulek: But I don't think that's good enough reason if there's an alternative | 18:22 |
| *** dims has joined #openstack-cinder | 18:22 | |
| geguileo | dulek: If the alternative is a lot more complicated then I would say it's ok that it goes in Service | 18:22 |
| *** esker has quit IRC | 18:23 | |
| *** neophy has joined #openstack-cinder | 18:25 | |
| jgriffith | geguileo: I"m sorry but I just don't see your side of this at all | 18:25 |
| jgriffith | geguileo: it's fine to criticize something, but if you can't offer an alternative solution that's a bit unfair | 18:25 |
| geguileo | jgriffith: Ok, no big deal | 18:25 |
| jgriffith | geguileo: well, actually it is kind of a big deal | 18:26 |
| jgriffith | geguileo: for me in particular | 18:26 |
| *** bharath has joined #openstack-cinder | 18:26 | |
| geguileo | jgriffith: Wouldn't dulek's suggestion work? | 18:26 |
| *** bharathk has quit IRC | 18:26 | |
| jgriffith | geguileo: I didn't catch that, lemme scroll back | 18:27 |
| geguileo | jgriffith: No, the one in the comments of the patch | 18:27 |
| geguileo | Not here | 18:27 |
| jgriffith | geguileo: Regarding service.py? | 18:27 |
| jgriffith | geguileo: I completely agree with him on that and changed it | 18:27 |
| geguileo | jgriffith: So now we are disagreeing only if those fields should go in the DB or not? | 18:28 |
| dulek | jgriffith: I guess geguileo means the most recent one. | 18:28 |
| jgriffith | dulek: I'm confused, we'd never get to that save if the service was not found | 18:28 |
| jgriffith | geguileo: yes | 18:28 |
| geguileo | jgriffith: I thought we were talking about the code going in service.py | 18:29 |
| dulek | jgriffith: Whoops, did I've not noticed something. | 18:29 |
| jgriffith | dulek: I'll add a reraise on that | 18:29 |
| jgriffith | dulek: nahh... we're good | 18:29 |
| jgriffith | geguileo: no.. sorry | 18:29 |
| e0ne | smcginnis: just a formality, please, one more +1 from you for https://review.openstack.org/287115 - not it passes CI | 18:29 |
| jgriffith | geguileo: I was talking about your statment that it shouldn't be in the DB model (Service table) | 18:29 |
| jgriffith | geguileo: at least I thought that's what you were saying | 18:29 |
| *** martyturner has joined #openstack-cinder | 18:29 | |
| *** timcl has joined #openstack-cinder | 18:29 | |
| geguileo | jgriffith: Well, if 2 out of 3 services don't require it... | 18:29 |
| jgriffith | geguileo: Ok, so that is what you're saying :) | 18:30 |
| geguileo | jgriffith: I was saying both things | 18:30 |
| jgriffith | sigh | 18:30 |
| geguileo | The service.py part and the DB part | 18:30 |
| jgriffith | geguileo: so why do we have AZ in there then? | 18:30 |
| geguileo | But the DB part was mostly a rant at this point... | 18:30 |
| *** openstackgerrit_ has joined #openstack-cinder | 18:30 | |
| jgriffith | geguileo: yes, and from a fellow ranter I'd like to understand better :) | 18:30 |
| jgriffith | geguileo: or at least try and persuade you :) | 18:30 |
| *** [1]Thelo has joined #openstack-cinder | 18:31 | |
| geguileo | lol | 18:31 |
| jgriffith | geguileo: rpc version is another one that isn't necessarily applicable to all of them | 18:31 |
| geguileo | jgriffith: AZ was before my time | 18:31 |
| geguileo | jgriffith: rpc version is applicable to them all | 18:31 |
| jgriffith | geguileo: good... because it doesn't really *do* anything anyway :) | 18:31 |
| *** martyturner has quit IRC | 18:31 | |
| jgriffith | geguileo: alright... I'll make you a deal | 18:32 |
| geguileo | jgriffith: And I'm moving those 2 version fields out of that table in my job-distribution patches | 18:32 |
| jgriffith | geguileo: I'll create a new replication table when N opens up, and introduce backrefs to the service entry. | 18:32 |
| jgriffith | geguileo: or... have a separate c-vol table | 18:32 |
| *** Thelo has quit IRC | 18:33 | |
| *** [1]Thelo is now known as Thelo | 18:33 | |
| jgriffith | geguileo: oh... well then, my work here i sdone | 18:33 |
| jgriffith | is | 18:33 |
| jgriffith | :) | 18:33 |
| *** martyturner has joined #openstack-cinder | 18:33 | |
| *** kfarr has joined #openstack-cinder | 18:34 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Move replication_status update to service.py https://review.openstack.org/290917 | 18:35 |
| eharney | e0ne: no, we don't need to call shell commands to gather RBD stats. the patch/reviews already outlines how to do it | 18:35 |
| e0ne | eharney: I believe, it will be librdb call using python rbd module | 18:36 |
| eharney | e0ne: the patch already does the required librbd calls... i just need to plug the result into one more place. so i'm not sure what you mean | 18:36 |
| e0ne | eharney: probably, I need to take a closer look on your patch | 18:37 |
| e0ne | eharney: I ping you once, I'll have more time to test it | 18:37 |
| jgriffith | geguileo: looking at your patches I don't see where you moved this? I'm curious what you came up with here? | 18:38 |
| geguileo | jgriffith: I'm talking about the rpc and obj versions | 18:39 |
| geguileo | jgriffith: I moved them to the nodes table | 18:39 |
| geguileo | jgriffith: But since we have rolling upgrades I cannot delete them in the same release as I move them | 18:39 |
| jgriffith | geguileo: hmmm... nodes table.... | 18:39 |
| jgriffith | geguileo: well when that lands it would certainly be a logical place | 18:40 |
| jgriffith | geguileo: so this one: https://review.openstack.org/#/c/286599/ | 18:41 |
| bharath | https://review.openstack.org/#/c/291204/ | 18:41 |
| geguileo | jgriffith: Yes | 18:42 |
| jgriffith | geguileo: but then you'd make the same argument, that data there is only used in 1 out of 3 of the nodes we have right? | 18:42 |
| geguileo | jgriffith: You mean the rpc version info | 18:42 |
| geguileo | jgriffith: That's used in all 3 services | 18:42 |
| jgriffith | geguileo: I'm also unclear on why you need a separate table for that? | 18:42 |
| geguileo | jgriffith: For the version or for the nodes? | 18:43 |
| jgriffith | geguileo: sure, but why not just keep it in the service table? | 18:43 |
| geguileo | jgriffith: Because you will be upgrading your nodes one by one | 18:43 |
| jgriffith | geguileo: I mean... sure; multiple services on same node maybe... but so? | 18:43 |
| geguileo | jgriffith: And they will have different versiosn | 18:43 |
| jgriffith | geguileo: and if they're on different nodes, that just works by service no? | 18:44 |
| jgriffith | geguileo: service/host | 18:44 |
| geguileo | jgriffith: The pinning is done by the whole cloud | 18:44 |
| jgriffith | geguileo: I don't think I'm communicating well; perhaps I should just let this go for now. Maybe we can walk through it in Austin? | 18:45 |
| geguileo | jgriffith: Yeah, probably I'm a little bit dense right now | 18:45 |
| geguileo | as it's the end of the day for me | 18:45 |
| geguileo | It's almost 8pm here... | 18:45 |
| *** bardia has joined #openstack-cinder | 18:45 | |
| jgriffith | geguileo: nah... history has shown these sorts of talk-past events are on me :) | 18:46 |
| geguileo | lol | 18:46 |
| jgriffith | geguileo: to be clear, I'm not being argumentative or anything, truly trying to understand your viewpoint and the code you have going here | 18:46 |
| jgriffith | thats' all | 18:46 |
| jgriffith | geguileo: I'm actually interested :) | 18:46 |
| geguileo | jgriffith: That's how I read it | 18:46 |
| *** angela-s has joined #openstack-cinder | 18:47 | |
| jgriffith | geguileo: good enough then | 18:47 |
| geguileo | jgriffith: And it's always good to talk it with someone else, because sometimes I see things I missed :-) | 18:47 |
| jgriffith | alright, well I'll quit harassing you for now | 18:47 |
| geguileo | jaja | 18:47 |
| geguileo | ttyl | 18:48 |
| jgriffith | geguileo: good night | 18:48 |
| geguileo | jgriffith: Thanks, and have a nice day | 18:48 |
| *** eharney has quit IRC | 18:48 | |
| jgriffith | dulek: LOL... yes, I just noticed I didn't change the commit header | 18:49 |
| openstackgerrit | John Griffith proposed openstack/cinder: Move replication_status update to init_with_rpc https://review.openstack.org/290917 | 18:50 |
| *** martyturner has quit IRC | 18:57 | |
| *** Suyash has joined #openstack-cinder | 18:57 | |
| openstackgerrit | Kaitlin Farr proposed openstack/cinder: Specify key algorithm and size for create_key https://review.openstack.org/251503 | 18:58 |
| *** martyturner has joined #openstack-cinder | 19:02 | |
| *** eharney has joined #openstack-cinder | 19:03 | |
| *** leeantho has joined #openstack-cinder | 19:03 | |
| *** paul_ has joined #openstack-cinder | 19:11 | |
| *** lpetrut has joined #openstack-cinder | 19:13 | |
| dulek | diablo_rojo: I've answered your yesterday's question on review. I think at least. | 19:15 |
| *** jistr has joined #openstack-cinder | 19:15 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 19:16 |
| *** edtubill has joined #openstack-cinder | 19:18 | |
| *** kfarr has quit IRC | 19:20 | |
| *** laughterwym has joined #openstack-cinder | 19:20 | |
| *** zul has quit IRC | 19:25 | |
| *** zul has joined #openstack-cinder | 19:26 | |
| *** laughterwym has quit IRC | 19:27 | |
| *** akerr has quit IRC | 19:27 | |
| *** dims_ has joined #openstack-cinder | 19:28 | |
| *** mvk has quit IRC | 19:28 | |
| *** dims has quit IRC | 19:29 | |
| *** bharath has quit IRC | 19:31 | |
| *** akerr has joined #openstack-cinder | 19:31 | |
| *** rlrossit has quit IRC | 19:34 | |
| hemna | patrickeast, ping | 19:36 |
| patrickeast | hemna: hola | 19:36 |
| hemna | patrickeast, question for you related to the create_cloned_volume fiasco | 19:36 |
| patrickeast | haha | 19:36 |
| patrickeast | hemna: sure, whats up? | 19:36 |
| hemna | regarding the image cache code. When create_cloned_volume is called, is the cloned volume size the same as the image (cached volume) ? | 19:37 |
| hemna | before calling extend_volume ? | 19:37 |
| *** rlrossi__ has joined #openstack-cinder | 19:37 | |
| *** rlrossi__ has quit IRC | 19:37 | |
| smcginnis | hemna: You mean the fs on the clone? | 19:38 |
| *** rlrossi__ has joined #openstack-cinder | 19:38 | |
| patrickeast | so | 19:38 |
| hemna | smcginnis, no | 19:38 |
| hemna | I mean the volume object being passed into create_cloned_volume | 19:38 |
| patrickeast | lemmie pull up the code to make sure i'm thinking of the spot you are | 19:38 |
| patrickeast | but if you mean the cache hit path | 19:38 |
| patrickeast | when we clone from src | 19:39 |
| patrickeast | the source volume is image sized, and the new volume is requested size | 19:39 |
| hemna | yah the cache hit path. | 19:39 |
| patrickeast | and iirc we don't explicitly do a extend | 19:39 |
| hemna | ok crap | 19:39 |
| patrickeast | we only do extend on the cache miss path | 19:39 |
| hemna | *sigh* | 19:39 |
| patrickeast | for the original volume | 19:39 |
| patrickeast | let me double check | 19:39 |
| patrickeast | but i doubt i would have done it any other way | 19:40 |
| hemna | I guess that makes sense | 19:40 |
| patrickeast | ... since my driver does the extend on create from source :P | 19:40 |
| hemna | and why ours works. if we call extend on our cloned volume, it would fail today. | 19:40 |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: DRBD driver: resize volume if cloned image is larger https://review.openstack.org/291150 | 19:40 |
| hemna | because our volume is still busy being created. | 19:40 |
| *** rcernin has quit IRC | 19:40 | |
| hemna | I've had to completely rework our create_cloned_volume code entirely. | 19:40 |
| hemna | :( | 19:40 |
| patrickeast | :( | 19:41 |
| hemna | ok I'll enable the cache and test it out. | 19:42 |
| hemna | I just was curious before I started testing it. | 19:42 |
| *** e0ne has quit IRC | 19:43 | |
| patrickeast | hemna: yea so this call https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L631 | 19:43 |
| patrickeast | hemna: those volumes are definitely not the same size | 19:43 |
| patrickeast | image_volume is minimal image size | 19:44 |
| hemna | ok, so volume (which is the destination) is the user requested size ? | 19:44 |
| jgriffith | patrickeast: wouldn't be very useful otherwise | 19:44 |
| patrickeast | yep | 19:44 |
| hemna | ok | 19:44 |
| patrickeast | jgriffith: +1 | 19:44 |
| hemna | well, I'm just super confused about it all | 19:44 |
| jgriffith | patrickeast: it would mean your cache could ONLY be a volume of N size | 19:44 |
| hemna | I thought we were calling extend_volume() in the workflow somewhere. | 19:45 |
| patrickeast | jgriffith: haha yea exactly | 19:45 |
| hemna | I guess not. | 19:45 |
| patrickeast | hemna: we do | 19:45 |
| patrickeast | hemna: on cache miss, when we are creating the image volume | 19:45 |
| * hemna is confused. | 19:45 | |
| patrickeast | hemna: thats how we get the minimal sized one | 19:45 |
| hemna | so only on a miss ? | 19:45 |
| patrickeast | hemna: we create a volume thats the smallest possible for the image, do the dd of image data onto it, then extend it | 19:45 |
| hemna | create, extend, mount, copy data, etc. | 19:45 |
| patrickeast | hemna: but before the extend | 19:45 |
| hemna | ok | 19:45 |
| jgriffith | hemna: create, mount, copy then extend | 19:46 |
| patrickeast | hemna: we clone | 19:46 |
| patrickeast | hemna: and the small one that isn't extended becomes the cache entry | 19:46 |
| jgriffith | save the wasted dd time that way | 19:46 |
| patrickeast | exactly | 19:46 |
| hemna | ok | 19:46 |
| hemna | I'll test this out and see how much I screwed things up. | 19:46 |
| jgriffith | zeroing those blocks on a 250GiB volume for a 200MiB image is a bit expensive | 19:46 |
| *** timcl has quit IRC | 19:47 | |
| patrickeast | so, what changed that you are concerned about breaking? | 19:47 |
| patrickeast | just that you guys extend in your clone now? | 19:47 |
| patrickeast | or that you don't extend in your clone? | 19:47 |
| hemna | patrickeast, we/I never looked at volume['size'] in create_cloned_volume. | 19:47 |
| patrickeast | ahh ok | 19:47 |
| hemna | as did many drivers | 19:47 |
| *** IlyaG has quit IRC | 19:47 | |
| patrickeast | so yea you are probably actually getting minimally sized volumes instead of the requested size | 19:47 |
| hemna | so we are going to lose a lot of our efficiencies in the clone. | 19:48 |
| jgriffith | hemna: the caching stuff is optional | 19:48 |
| *** martyturner has quit IRC | 19:48 | |
| jgriffith | hemna: you certainly don't have to implement it at all | 19:48 |
| hemna | as our array can't modify the destination volume, while the copy is ongoing. | 19:48 |
| patrickeast | yea same problem you had originally for the cache, right? | 19:48 |
| hemna | so I'm having to put wait loops in create_cloned_Volume now to make sure it's finished. | 19:48 |
| hemna | yah it just meant our cached clones were incorrect size | 19:49 |
| hemna | well anything created with create_cloned_volume was potentially the incorrect size. | 19:49 |
| patrickeast | yea | 19:49 |
| hemna | unless the sizes were the same... | 19:49 |
| hemna | ASS | 19:49 |
| patrickeast | any command with --source-volid (or whatever its called) | 19:49 |
| hemna | yup | 19:50 |
| hemna | wah wah wah... | 19:50 |
| *** martyturner has joined #openstack-cinder | 19:50 | |
| *** timcl has joined #openstack-cinder | 19:51 | |
| hemna | we were broked for a while | 19:51 |
| hemna | oopsie | 19:51 |
| patrickeast | the funny part is that with like what 20 drivers doing the wrong thing no one noticed? | 19:52 |
| patrickeast | clearly its a often used code path | 19:52 |
| smcginnis | >20 | 19:52 |
| Swanson | customer work arounds are going to break | 19:53 |
| hemna | yah :( | 19:54 |
| *** esker has joined #openstack-cinder | 19:55 | |
| *** ildikov has quit IRC | 19:58 | |
| *** cknight has quit IRC | 19:59 | |
| *** ildikov has joined #openstack-cinder | 20:00 | |
| *** Yogi1 has joined #openstack-cinder | 20:04 | |
| *** sbezverk has joined #openstack-cinder | 20:06 | |
| *** rlrossit has joined #openstack-cinder | 20:07 | |
| *** paul_ has quit IRC | 20:07 | |
| sbezverk | Hello, I am looking for some docs fo iscsi/lvm communication api. Anybody has any pointers? | 20:07 |
| openstackgerrit | Gerald McBrearty proposed openstack/cinder: Return volume_type extra specs based on policy https://review.openstack.org/291405 | 20:10 |
| *** e0ne has joined #openstack-cinder | 20:12 | |
| openstackgerrit | Gerald McBrearty proposed openstack/cinder: Return volume_type extra specs based on policy https://review.openstack.org/291405 | 20:12 |
| *** diogogmt has joined #openstack-cinder | 20:13 | |
| *** cknight has joined #openstack-cinder | 20:15 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - SnapVX and other snapshot improvements https://review.openstack.org/279881 | 20:20 |
| *** laughterwym has joined #openstack-cinder | 20:20 | |
| *** timcl has quit IRC | 20:20 | |
| *** Lee1092 has quit IRC | 20:22 | |
| *** e0ne has quit IRC | 20:23 | |
| *** ociuhandu has quit IRC | 20:24 | |
| *** laughterwym has quit IRC | 20:25 | |
| *** esker has quit IRC | 20:28 | |
| *** esker has joined #openstack-cinder | 20:30 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Disallow quota deletes if default under usage https://review.openstack.org/291417 | 20:35 |
| *** esker has quit IRC | 20:35 | |
| *** esker has joined #openstack-cinder | 20:35 | |
| *** esker has quit IRC | 20:36 | |
| *** erlon has quit IRC | 20:36 | |
| *** sheel has quit IRC | 20:37 | |
| *** IlyaG has joined #openstack-cinder | 20:37 | |
| jgriffith | patrickeast: Swanson one thing to keep in mind is most distros are a year behind | 20:42 |
| jgriffith | patrickeast: Swanson a lot of the stuff we see and fix in a cycle isn't even going to hit anybodys environment for another few months | 20:43 |
| jgriffith | sbezverk: can you be a little more specific? | 20:43 |
| Swanson | jgriffith, Good point. | 20:43 |
| jgriffith | Swanson: it's a gripe of mine, we close stable for backports before it really even sees production | 20:43 |
| jgriffith | Swanson: distros are getting faster now though so this should start to be a bit more reasonable in the coming year | 20:44 |
| Swanson | jgriffith, I think we saw someone last week or the week before looking for juno help. | 20:44 |
| jgriffith | Swanson: yeah :) | 20:44 |
| jgriffith | I still get that fairly often, and the occasional Havana or even Essex just last month | 20:45 |
| sbezverk | jgriffith ideally I would like to have docs for api calls used by cinder to communicate with iscsi driver.. | 20:45 |
| *** IlyaG has quit IRC | 20:45 | |
| jgriffith | sbezverk: hmm... yeah, that's something we need to work on :) | 20:45 |
| jgriffith | sbezverk: best I can offer right now is the code itself in cinder/volume/targets/driver.py | 20:45 |
| jgriffith | sbezverk: that will give you the base class anyway... you can diverge from there into the various tgt drivers | 20:46 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/python-cinderclient: Don't reset volume status when resetting migration status https://review.openstack.org/288732 | 20:47 |
| sbezverk | jgriffith thank you, it is a start :-) | 20:50 |
| jgriffith | sbezverk: better than a poke in the eye with a sharp stick :) | 20:50 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/python-cinderclient: Don't reset volume status when resetting migration status https://review.openstack.org/288732 | 20:52 |
| sbezverk | jgriffith one more question, iscsid and tgtd seems like providing the same service, what was the reason cinder driver uses iscsi? | 20:52 |
| jgriffith | sbezverk: they're different actually | 20:52 |
| jgriffith | sbezverk: there's initiator side and target implementation | 20:52 |
| jgriffith | sbezverk: so for backends OTHER than the reference LVM the target stuff is irrelevant | 20:53 |
| jgriffith | sbezverk: but iscsid is still needed to do things like iSCSI connect so we can attach on the Cinder node | 20:53 |
| jgriffith | sbezverk: for what it's worth almost ALL of the distros forget to add this to their Cinder nodes | 20:53 |
| jgriffith | :( | 20:54 |
| jgriffith | sbezverk: and then things fail when you try and do 'create volume from image' and things like that | 20:54 |
| sbezverk | jgriffith I was refering to the target side.. the reason I brought it up, is iscsid cannot be easily containerized, but tgtd works fine | 20:55 |
| jgriffith | sbezverk: right, we're talking about the same thing :) | 20:55 |
| jgriffith | sbezverk: iscsid == opne-iscsi daemon | 20:55 |
| jgriffith | open | 20:55 |
| *** JoseMello has joined #openstack-cinder | 20:55 | |
| jgriffith | sbezverk: that's the one hang up I have with containerizing c-vol service | 20:55 |
| *** cknight has quit IRC | 20:56 | |
| bswartz | jgriffith: why? | 20:56 |
| *** esker has joined #openstack-cinder | 20:56 | |
| sbezverk | jgriffith I was trying to understand how difficult to substitute open-iscsi by tgtd | 20:56 |
| *** haomaiwang has joined #openstack-cinder | 20:56 | |
| bswartz | +1 for containerizing c-vol | 20:57 |
| jgriffith | bswartz: I can't make initiators work in a container... because of the ip faking/bridging | 20:57 |
| bswartz | jgriffith: IMO c-vol should not touch data | 20:57 |
| guitarzan | time for a fully userland initiator? | 20:57 |
| jgriffith | guitarzan: +1 | 20:57 |
| smcginnis | jgriffith: Have you tried lately? I thought Docker 1.9 had some improvements there. | 20:57 |
| bswartz | jgriffith: data-touching operations belong in a different service, for reasons I mentioned at midcycle | 20:57 |
| smcginnis | Though I haven't looked in to it enough. | 20:57 |
| jgriffith | bswartz: so then how do you do image-->volume, volume-->image and backup? | 20:57 |
| bswartz | jgriffith: cinder-data service or somesuch | 20:58 |
| jgriffith | bswartz: oh... sure, but then you just kick the can so to speak | 20:58 |
| jgriffith | bswartz: or what I think I remember you saying was a worker-instance maybe? | 20:58 |
| bswartz | at least that can can be managed separately | 20:58 |
| jgriffith | bswartz: yeah, I hear ya | 20:58 |
| jgriffith | smcginnis: so no, and yes I think 1.10 actually may help me solve it | 20:58 |
| jgriffith | smcginnis: but no, I have had zero time to mess with it since last fall | 20:59 |
| smcginnis | Too much to do. Not enough coffee to do it. | 20:59 |
| jgriffith | smcginnis: besides, there are now whole projects in OpenStack that are far better at that whole thing than I :) | 20:59 |
| smcginnis | ;) | 20:59 |
| sbezverk | jgriffith I build docker based on 1.10 with tgtd running and it also interacts with lvm backend | 20:59 |
| sbezverk | jgriffith it can share iscsi targets, how would you suggest it could be tested if this would work with cinder? | 21:00 |
| *** haomaiwang has quit IRC | 21:01 | |
| openstackgerrit | Jon Bernard proposed openstack/cinder: RBD: remove duplicate clone test https://review.openstack.org/291425 | 21:02 |
| *** martyturner has quit IRC | 21:02 | |
| *** cknight has joined #openstack-cinder | 21:06 | |
| *** cknight has quit IRC | 21:10 | |
| *** akerr has quit IRC | 21:10 | |
| aorourke | jgriffith, ping | 21:10 |
| aorourke | jgriffith, replication :) .... patrickeast Swanson | 21:11 |
| smcginnis | Never heard of it. | 21:11 |
| aorourke | If you attempt to fail a host back but the backend is not ready and it returns an error, there is currently no way to keep the backend in a failed-over state. | 21:12 |
| aorourke | Currently we have an UnableToFailOver exception we can raise where we can keep it in enabled state. I think we need something like UnableToFailBack where we can keep it in failed-over state. Thoughts? | 21:12 |
| aorourke | since we support failing back already... | 21:12 |
| aorourke | i will make the patch, just wanted to get some thoughts first | 21:13 |
| patrickeast | wait, what happens right now? if you raise an exception it should go to an error state, right? | 21:13 |
| *** krtaylor has quit IRC | 21:13 | |
| patrickeast | or it goes back to the un-failed-over and original backend_id? | 21:13 |
| aorourke | patrickeast, it gets stuck in failing-over state if you raise an exception | 21:13 |
| *** cknight has joined #openstack-cinder | 21:13 | |
| aorourke | patrickeast, and there is no way to get out of that | 21:13 |
| aorourke | patrickeast, even in error state, I don't think there is a way to get out of that either | 21:14 |
| patrickeast | right | 21:14 |
| patrickeast | i had a spec for that around somewhere | 21:14 |
| * patrickeast takes a note to go find that | 21:14 | |
| aorourke | patrickeast, i think it would be nice to just keep it in failed-over state. then we can try again later after the issue is fixed | 21:14 |
| aorourke | patrickeast, mdenny filed a bug on this https://bugs.launchpad.net/cinder/+bug/1555814 | 21:15 |
| openstack | Launchpad bug 1555814 in Cinder "cinder.volume.manager ValueError: incomplete format exception in cheesecake on Failing back to default backend" [Undecided,New] | 21:15 |
| aorourke | there it is ^ | 21:15 |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - SSl connection is not picking up values https://review.openstack.org/280933 | 21:15 |
| patrickeast | aorourke: so, it seems like maybe the bug is that the unable to failover should leave things in the failed-over state, or put it to an error state, if we are trying to fail back | 21:15 |
| aorourke | patrickeast, exactly | 21:15 |
| Swanson | I don't let you try to fail back. First failover "test" will be a customer's last. | 21:15 |
| jgriffith | sbezverk: so the only question is getting open-iscsi (intiator) stuff to work | 21:15 |
| aorourke | patrickeast, leaving it in a failed-over state would be ideal | 21:15 |
| jgriffith | sbezverk: you'd need to create a target and make sure you can attach it to the container... NOT the container host but the container itself | 21:16 |
| aorourke | patrickeast, well we leave it up to the driver | 21:16 |
| Swanson | So keeping it in that state would seem to be peachy. | 21:16 |
| jgriffith | sbezverk: this is where things fell apart for me | 21:16 |
| patrickeast | aorourke: yea i'm sold, lets get a patch up for it | 21:16 |
| aorourke | patrickeast, sounds good | 21:16 |
| smcginnis | hemna: ping | 21:17 |
| jgriffith | aorourke: you might be raising the wrong exception | 21:17 |
| jgriffith | aorourke: if you raise a "VolumeDriverException" it sets the status to error | 21:17 |
| Swanson | am I wrong in thinking nothing is actually merging out of gate? 15 items at 0 minutes to go... | 21:18 |
| aorourke | jgriffith, right. but i dont see a way to keep it in failed-over state | 21:18 |
| hemna | smcginnis, yough | 21:18 |
| aorourke | jgriffith, because it should not go back to enabled | 21:18 |
| smcginnis | hemna: Hey, is this implemented? | 21:18 |
| smcginnis | hemna: I know there was one that went in for iSCSI. | 21:18 |
| smcginnis | hemna: But I didn't see one for FC. | 21:18 |
| jgriffith | aorourke: Huh? | 21:18 |
| jgriffith | aorourke: whatever, sounds like you and patrickeast have it figured out | 21:18 |
| aorourke | jgriffith, the issue I am having is it the backend is not ready, there is no way to keep the replicaiton-status in a failed-over state | 21:19 |
| aorourke | ok | 21:19 |
| jgriffith | aorourke: my contention is that that SHOULD put it in an error state | 21:19 |
| hemna | is what? | 21:19 |
| hemna | sorry, I haven't been following the channel much today | 21:19 |
| jgriffith | aorourke: we issue a command, you can't comply... that's an error | 21:19 |
| smcginnis | hemna: Doh, copy paste fail. ;) | 21:20 |
| smcginnis | hemna: Hey, is this implemented? https://blueprints.launchpad.net/cinder/+spec/brick-scsi-wwn | 21:20 |
| hemna | hehe ok | 21:20 |
| jgriffith | aorourke: not a "ok, I'll just stay like I am and pretend you never asked me to do anything" | 21:20 |
| hemna | I thought I had missed something | 21:20 |
| smcginnis | hemna: No, I did. ;) | 21:20 |
| aorourke | jgriffith, well we already have exception.InvalidReplicationTarget where we do not throw it in an error state | 21:20 |
| hemna | :) | 21:20 |
| jgriffith | aorourke: which is based of of an InvalidRequest which is different than "I don't feel like doing what you asked right now" | 21:20 |
| jgriffith | or "can't do what you asked" | 21:21 |
| jgriffith | aorourke: but like I said, up to you guys | 21:21 |
| hemna | smcginnis, yah I think that's in there. | 21:21 |
| jgriffith | aorourke: it makes for a kinda crappy API IMHO though | 21:21 |
| sbezverk | jgriffith: not sure I undersatnd the difference, could clarify? I thought container will not own the target, the host will own it, container is just a pieces of software, interface between a user and the resource offered by host. | 21:21 |
| smcginnis | hemna: OK, thanks. | 21:21 |
| bswartz | I would love to see a webserver return that | 21:21 |
| bswartz | HTTP 500 I don't feel like doing what you asked right now | 21:22 |
| smcginnis | bswartz: Kind of like the github 404 page. ;) | 21:22 |
| jgriffith | sbezverk: yes, and that's the problem when you try and attach it "in" the container so to speak | 21:22 |
| jgriffith | sbezverk: know what I mean? | 21:22 |
| jgriffith | sbezverk: I think there are some things added in 1.10 as smcginnis mentioned that might make this doable, but as of last fall I couldn't see a way to make it work | 21:22 |
| jgriffith | sbezverk: you need a virtual networking stack IN the container | 21:23 |
| bswartz | tumblr has a good 404 page | 21:23 |
| sbezverk | jgriffith: yes, but I do not get it why it is required to attach in in the container? | 21:23 |
| bswartz | ah crap they got rid of it | 21:23 |
| smcginnis | bswartz: Awesome | 21:23 |
| jgriffith | bswartz: they rotate them :) | 21:23 |
| smcginnis | bswartz: Oh, something better before? | 21:23 |
| bswartz | oh no there it is | 21:24 |
| jgriffith | http://log.maniacalrage.net/post/3942771810/tumblrs-new-error-page-view-full-size-look-i | 21:24 |
| smcginnis | Hah! | 21:24 |
| jgriffith | smcginnis: bswartz if you google "tumblr 404 images" you'll get a whole slew of things to chuckle at | 21:24 |
| smcginnis | "Tumblr isn't a CMS. It's a vehicle for getting us into celebrities' parties." | 21:25 |
| jgriffith | smcginnis: https://www.google.com/?espv=2#tbm=isch&tbs=rimg%3ACQh_19iQQW9eHIjj9CaUPbPkYwkjS5M6YbNiUoTC-rg89aK5_19x3FLLNNH-rDGXdC5yI5O5Ar5vpz-X8FeNRa2v_1ifCoSCf0JpQ9s-RjCEZfpEtNK46xqKhIJSNLkzphs2JQRnWqWR7ltmZIqEgmhML6uDz1orhEfwjIj0wM9pSoSCX_13HcUss00fEXOSaPo_1XpgmKhIJ6sMZd0LnIjkRVjd0O9IriCoqEgk7kCvm-nP5fxEgTw1FivP89ioSCQV41Fra_1-J8EUEw5zQY058i&q=tumblr%20404 | 21:25 |
| jgriffith | smcginnis: LOL | 21:25 |
| jgriffith | "404 error blow me" LOL | 21:26 |
| smcginnis | 404, Deal with it. | 21:27 |
| *** cristicalin has joined #openstack-cinder | 21:28 | |
| *** cristicalin has quit IRC | 21:28 | |
| jgriffith | eharney: smcginnis https://review.openstack.org/#/c/291302/2 | 21:29 |
| *** boris-42 has joined #openstack-cinder | 21:29 | |
| jgriffith | eharney: smcginnis from an outsider reading release notes it's my opinion there may be a few unanswered questions there | 21:30 |
| sbezverk | jgriffith: would you be interested to check the setup with the docker container and tgtd, to help identify if something is missing or if it still has issues you hit in the past? | 21:30 |
| jgriffith | sbezverk: sure | 21:30 |
| jgriffith | sbezverk: you have a docker-compose or dockerfile? | 21:30 |
| smcginnis | jgriffith: 404 PTL not here | 21:30 |
| jgriffith | smcginnis: HA!!!! | 21:30 |
| jgriffith | Nice!!! | 21:30 |
| *** liverpooler has joined #openstack-cinder | 21:32 | |
| *** bjornar has quit IRC | 21:32 | |
| sbezverk | jgriffith: well, it is a future part of kolla, so it is a mixture of Docker file and ansible yml.. | 21:33 |
| jgriffith | sbezverk: ahh... fair enough | 21:33 |
| jgriffith | sbezverk: shoot me a link or whatever and some insructions and I'll give it a go | 21:33 |
| *** haplo37 has joined #openstack-cinder | 21:34 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Use 'is None' rather than 'is_null()' in matches https://review.openstack.org/291448 | 21:35 |
| sbezverk | jgriffith: sure, as soon as it is packaged, right now it seats in my local workspace, I will ping you. Also could do a live demo if it is ok with you.. | 21:37 |
| jgriffith | sbezverk: sounds good | 21:37 |
| jgriffith | sbezverk: if you want to set up a live demo maybe we could schedule something for tomorrow mid-morning perhaps? | 21:37 |
| jgriffith | sbezverk: I'd love to check it out and chat a bit | 21:37 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: Allow recovering from an invalid failback command https://review.openstack.org/291449 | 21:37 |
| jgriffith | sbezverk: I've had some issues with Kolla that maybe you could help me with :) | 21:38 |
| haplo37 | Hi, i use on my openrc OS_ENDPOINT_TYPE='internalURL' but when i use the command cinder service-list for example, the endpoint use by this command is public and not internal. Do you know how to force internal in my openrc ? | 21:38 |
| sbezverk | jgriffith: sounds like a plan, what is your time zone? | 21:39 |
| jgriffith | sbezverk: Mountain... | 21:39 |
| jgriffith | so it's 14:39 here currently | 21:39 |
| *** jistr has quit IRC | 21:40 | |
| *** openstackstatus has quit IRC | 21:42 | |
| *** openstackstatus has joined #openstack-cinder | 21:43 | |
| *** ChanServ sets mode: +v openstackstatus | 21:43 | |
| haplo37 | I find it : export CINDER_ENDPOINT_TYPE='internalURL' :-) | 21:44 |
| jgriffith | mtanino: do we want to allow non-admins to manage things? https://review.openstack.org/#/c/274204/ | 21:44 |
| jgriffith | mtanino: what I mean is... is that patch really something that we want to do? | 21:44 |
| jgriffith | mtanino: I guess since it's still tied to policy it's cool | 21:45 |
| jgriffith | mtanino: just wanted to make sure I was thinking this through completely | 21:45 |
| *** tpsilva has quit IRC | 21:47 | |
| jgriffith | mtanino: never mind, I convinced myself :) | 21:47 |
| mtanino | jgriffith: This patch allowes volume type operation such as special user ex. "storage_type_admin", don't enable all of non admin users | 21:47 |
| mtanino | oh! | 21:47 |
| jgriffith | mtanino: so all you have to do with me is ignore me for a few minutes... I'll come around :) | 21:47 |
| *** cknight has quit IRC | 21:48 | |
| *** mrrowden has quit IRC | 21:48 | |
| guitarzan | next patch gets rid of admin and goes all policy | 21:48 |
| *** edtubill has quit IRC | 21:49 | |
| *** sgotliv has quit IRC | 21:51 | |
| mtanino | guitarzan: Does the patch go to something wrong direction?? | 21:51 |
| *** dustins has quit IRC | 21:51 | |
| *** sdague has quit IRC | 21:52 | |
| *** dims has joined #openstack-cinder | 21:52 | |
| *** jamielennox|away is now known as jamielennox | 21:52 | |
| *** dims_ has quit IRC | 21:52 | |
| *** Yogi1 has quit IRC | 21:52 | |
| *** porrua has quit IRC | 21:52 | |
| *** gouthamr has quit IRC | 21:53 | |
| *** neophy has quit IRC | 21:54 | |
| *** ociuhandu has joined #openstack-cinder | 21:55 | |
| ntpttr | quick question about api microversions - when I create a unit test to try and test out a change for microversion 3.1, it creates an APIVersionRequest object, and sets its minor and major versions here https://github.com/openstack/cinder/blob/master/cinder/api/openstack/api_version_request.py#L94-L99 | 21:55 |
| ntpttr | but match.group(1) and match.group(2) are always integers, so 3.1 will never be used | 21:55 |
| ntpttr | this is making my test not go down the path it should, which is the v3.1 microversion of the API call | 21:56 |
| ntpttr | and on the doc page here, it outlines at the bottom that you should be putting in the microversion you want when initializing the api_version, http://docs.openstack.org/developer/cinder/devref/api_microversion_dev.html#when-do-i-need-a-new-microversion | 21:56 |
| *** ociuhandu has quit IRC | 21:56 | |
| ntpttr | in the example it's 3.6, but that seems pointless since the '.6' part of that will never be used - is this a bug or am I missing something here? | 21:57 |
| *** ociuhandu has joined #openstack-cinder | 21:57 | |
| patrickeast | aorourke: jgriffith: i was just lookin at https://review.openstack.org/#/c/291449/1 when it occurs to me that you could just throw a InvalidReplicationTarget for 'default' if you aren't ready to failback yet | 21:58 |
| patrickeast | since it would, at that time, be an invalid target | 21:59 |
| patrickeast | and we could change the handler for that exception to just leave the status where it started, instead of reset to enabled | 21:59 |
| patrickeast | so if it was failed-over it stays there | 21:59 |
| aorourke | patrickeast, i looked at that as well, but then you go back to 'enabled' state, which really is not accurate | 21:59 |
| *** abhishekk_ has quit IRC | 21:59 | |
| aorourke | patrickeast, ah good point | 21:59 |
| *** abhishekk_ has joined #openstack-cinder | 22:00 | |
| *** leeantho_ has joined #openstack-cinder | 22:00 | |
| *** melwitt has quit IRC | 22:00 | |
| patrickeast | aorourke: i tend to agree with jgriffith's point about that unabletofailback exception going to an error state | 22:00 |
| patrickeast | ideally we could then do something and recover from the error states... but thats a challenge for another day | 22:01 |
| *** leeantho_ has quit IRC | 22:01 | |
| *** leeantho_ has joined #openstack-cinder | 22:01 | |
| *** delattec has quit IRC | 22:02 | |
| aorourke | patrickeast, would you be open to allowing InvalidReplicationTarget to just preserve the original state? | 22:03 |
| aorourke | patrickeast, i think it is changed to 'failing-over' state at the API level though... | 22:03 |
| *** leeantho has quit IRC | 22:04 | |
| patrickeast | aorourke: hmm, well we could just peek at the secondary_backend_id and do the 'correct' state based on that | 22:04 |
| patrickeast | aorourke: i'm not really too opposed to just putting the status into an error if it wasn't ready | 22:06 |
| patrickeast | the idea being that an admin would have made sure things were ok beforehand | 22:06 |
| scottda | ntpttr: This bug is a duplicate: https://bugs.launchpad.net/cinder/+bug/1555817 | 22:06 |
| openstack | Launchpad bug 1555817 in Cinder "'Matches' function returns error when used how it's documented" [Undecided,In progress] - Assigned to Nate Potter (ntpttr) | 22:06 |
| aorourke | patrickeast, ideally yes they would have made sure things were back up and running | 22:06 |
| scottda | And your fix will break the API | 22:06 |
| ntpttr | scottda: okay, so what should be passed in to the matches function when it reccomends on this page to use strings? http://docs.openstack.org/developer/cinder/devref/api_microversion_dev.html#when-do-i-need-a-new-microversion | 22:07 |
| ntpttr | passing in strings creates an error | 22:07 |
| scottda | Yes. I've filed a bug and posted a patch for the fix... | 22:09 |
| scottda | https://bugs.launchpad.net/cinder/+bug/1550337 | 22:09 |
| openstack | Launchpad bug 1550337 in Manila "api_version_request.matches does not accept a string or None" [Medium,In progress] - Assigned to Scott DAngelo (scott-dangelo) | 22:09 |
| ntpttr | scottda: okay, sorry I didn't see the duplicate - I'll abandon the patch | 22:09 |
| scottda | https://review.openstack.org/#/c/285293/ | 22:09 |
| ntpttr | scottda: thanks for the info | 22:09 |
| scottda | ntpttr: No problem. Let me know if you need help, I'd be glad to walk you through how it works. | 22:10 |
| scottda | ntpttr: BTW, I'm on vacation next week :) | 22:10 |
| ntpttr | scottda: sounds good - so I should probably make my current patch dependant on yours so that my passing in strings will work | 22:11 |
| *** rcernin has joined #openstack-cinder | 22:11 | |
| scottda | ntpttr: Yup. That should work | 22:11 |
| scottda | Have you posted your patch yet? Which is it? | 22:11 |
| *** esker has quit IRC | 22:13 | |
| *** knikolla has quit IRC | 22:13 | |
| ntpttr | scottda: it's https://review.openstack.org/#/c/230617/ - I'm working on adding microversions to it since it updates the parameters you can pass in to upload_volume_to_image | 22:13 |
| *** porunov has quit IRC | 22:14 | |
| ntpttr | scottda: adding the microversion was simple enough following the documentation, just working on updating the tests now | 22:14 |
| scottda | OK, cool. I'll keep an eye on your patch | 22:14 |
| scottda | ntpttr: Look at the unit tests for my fix to the matches() string issue for ideas to use in your unit tests. Let me know if you need help. | 22:15 |
| ntpttr | scottda: is this the first patch to implement a microversion in Cinder then? I was confused at first because it didn't look like there were any! I guess it is a relatively recent thing | 22:15 |
| ntpttr | scottda: okay thanks | 22:16 |
| *** melwitt has joined #openstack-cinder | 22:16 | |
| *** melwitt has quit IRC | 22:16 | |
| aorourke | patrickeast, jgriffith, this isn't the right approach. i abandoned the patch. in a future release we can talk about recovering from an error state | 22:16 |
| scottda | ntpttr: This is up for review and competing with you for first use of a microversion: https://review.openstack.org/#/c/278605/ | 22:16 |
| jgriffith | aorourke: ? | 22:16 |
| aorourke | jgriffith, it should be treated as an error | 22:17 |
| jgriffith | aorourke: you mean your patch to just leave it in failed-over | 22:17 |
| patrickeast | aorourke: i just put a comment on there | 22:17 |
| aorourke | yes | 22:17 |
| jgriffith | aorourke: cool, but I do agree with you that there's a problem there | 22:17 |
| patrickeast | i think making invalidreplicationtarget know how to do the right thing would fix this nicely | 22:17 |
| scottda | ntpttr: See that ^^ for info on your patch as well (i.e. need to update MAX_API_VERSION, etc.) | 22:17 |
| jgriffith | patrickeast: hmm.. that's an idea | 22:17 |
| aorourke | patrickeast, that's an easy fix | 22:18 |
| ntpttr | scottda: actually I do have a question - did you see my question a couple of minutes ago in this channel? | 22:18 |
| ntpttr | scottda: I got all of the MAX_API_VERSION bumps and things like that from the doc, it was nice and clear | 22:18 |
| aorourke | jgriffith, if secondary_backend_id is default we can put it in failed-over, otherwise enabled. all in the invalidreplicationtarget exception | 22:18 |
| jgriffith | patrickeast: aorourke so something else to consider... | 22:18 |
| scottda | I did, but I did not understand. Maybe PM me to keep the noise here down. | 22:18 |
| jgriffith | patrickeast: aorourke the failover being a toggle seems to create some challenges/confusion | 22:18 |
| jgriffith | patrickeast: aorourke should we consider adding a failback in N? | 22:19 |
| jgriffith | patrickeast: aorourke isolate the two and hammer them out that way? | 22:19 |
| aorourke | jgriffith, i think we should add a failback API/command in N | 22:19 |
| jgriffith | aorourke: I'm down with that | 22:19 |
| aorourke | jgriffith, it would be more straight forward | 22:19 |
| jgriffith | aorourke: I fully intended this to 'grow' in subsequent releases | 22:20 |
| patrickeast | jgriffith: yea we could add some of that raspberry sauce on there, but i'm not sure if these kind of problems would really go away | 22:20 |
| patrickeast | or if we just have two methods with them | 22:20 |
| jgriffith | patrickeast: well, they won't go away... but they'll be easier to isolate and at least try to standardize on | 22:20 |
| patrickeast | jgriffith: true | 22:20 |
| *** laughterwym has joined #openstack-cinder | 22:20 | |
| *** angela-s has quit IRC | 22:21 | |
| aorourke | patrickeast, jgriffith in the mean time, do we want to go with invalidreplicationtarget knowing how to fix this? | 22:22 |
| aorourke | either way, UnableToFailOver or InvalidReplicationTarget needs to be changed in order to save the replication_status of the host. | 22:22 |
| aorourke | I just noticed they are not being updated and get stuck in a 'failing-over' state when you raise them | 22:22 |
| *** rcernin has quit IRC | 22:22 | |
| jgriffith | aorourke: thinking.... | 22:23 |
| * jgriffith sees and smells smoke | 22:23 | |
| jgriffith | aorourke: I guess I'm not seeing how that's really different from what we just discussed? | 22:24 |
| jgriffith | aorourke: maybe I'm confused | 22:24 |
| jgriffith | aorourke: OH | 22:24 |
| jgriffith | aorourke: I'm missing the service.save | 22:24 |
| jgriffith | aorourke: well shit, that's no bueano | 22:25 |
| aorourke | jgriffith, yes, that needs to be fixed either way | 22:25 |
| jgriffith | aorourke: that certainly needs fixed | 22:25 |
| jgriffith | aorourke: I do prefer the states are left as they are currently | 22:25 |
| jgriffith | aorourke: but if you and patrickeast have a strong opinion on changing those that's ok with me | 22:25 |
| aorourke | jgriffith, meaning if invalidreplicationtarget is raised preserve the state? | 22:25 |
| *** arch-nemesis has quit IRC | 22:26 | |
| jgriffith | aorourke: yes... sorry, that wasn't clear | 22:26 |
| *** darosale has quit IRC | 22:26 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Add release note for delete volume with snaps https://review.openstack.org/291302 | 22:27 |
| jgriffith | aorourke: it's just unfortunate because now with patrickeast 's change that we don't wait for the response we have no good way to notify a caller that this "didn't work" | 22:27 |
| aorourke | jgriffith, patrickeast ok so in except InvalidReplicationTarget just keep the state as is? if when that was raised the state was 'failed-over' keep it like that, 'enabled' keep it enabled. that is not how it currently works though | 22:27 |
| jgriffith | aorourke: which is why I leaned towards any time the driver can't fulfill a request it should be an error | 22:27 |
| patrickeast | ehh we didn't really let them know much as-is | 22:27 |
| patrickeast | you pretty much have to check the logs to see what happened | 22:28 |
| *** laughterwym has quit IRC | 22:28 | |
| jgriffith | patrickeast: the exception would bubble back up I think.. | 22:28 |
| jgriffith | err..not | 22:28 |
| jgriffith | well, it should have :) | 22:28 |
| patrickeast | haha | 22:28 |
| patrickeast | moot point now ;) | 22:28 |
| jgriffith | patrickeast: true-statement | 22:28 |
| jgriffith | patrickeast: aorourke regardless, I was just trying to point out the issue I have with the proposals here | 22:29 |
| jgriffith | patrickeast: aorourke my ultimate preference for *now* would be if the driver can't do it for whatever reason we go to an error state for rep | 22:29 |
| jgriffith | patrickeast: aorourke otherwise there's no real way for a caller to know what happened or why | 22:29 |
| aorourke | jgriffith, even if the target is invalid? | 22:30 |
| jgriffith | patrickeast: aorourke yeah, kinda... but I'm not that steadfast on this | 22:30 |
| jgriffith | aorourke: I'm just trying to point out what I perceive as an issue. We can just live with it, but it is an issue IMO | 22:31 |
| openstackgerrit | Merged openstack/cinder: Dell SC: Active_backend_id wrong type https://review.openstack.org/290806 | 22:31 |
| aorourke | jgriffith, it is an issue, yes, because currently if you specify an invalid target to failover-host, you have to check the logs to see what happened | 22:31 |
| aorourke | jgriffith, the command returns instantly and gives no errors | 22:31 |
| patrickeast | yea... i guess we have to weigh which is the less evil, silent failures that require looking through logs (and for drivers to log reasonable explainations) or getting stuck in a terminal state | 22:31 |
| *** diogogmt has quit IRC | 22:32 | |
| jgriffith | patrickeast: aorourke right to both of you | 22:32 |
| jgriffith | patrickeast: aorourke but I don't have a reset-state in there right now so less evil may be to ignore it :( | 22:33 |
| *** esker has joined #openstack-cinder | 22:33 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 22:34 |
| *** mriedem has quit IRC | 22:34 | |
| *** esker has quit IRC | 22:38 | |
| *** huanan has quit IRC | 22:38 | |
| *** chlong has quit IRC | 22:39 | |
| *** huanan has joined #openstack-cinder | 22:39 | |
| aorourke | patrickeast, jgriffith well i will put up a patch that adds service.save() to UnableToFailOver and InvalidReplicationTarget. I can add the fix that patrickeast proposed to preserve the state replication_status when InvalidReplicationTarget is raised if we are ok with that | 22:39 |
| jgriffith | aorourke: patrickeast I'm willing to go with what the two of you decide. | 22:40 |
| jgriffith | aorourke: patrickeast as long it's not what we had before :) | 22:41 |
| aorourke | jgriffith, it would be an addition to except exception.InvalidReplicationTarget where we would check secondary_backend_id to put replication_status back to whatever it was before the call (either failed-over or enabled) instead of leaving it in 'failing-over' state | 22:43 |
| *** cdelatte has joined #openstack-cinder | 22:43 | |
| jgriffith | aorourke: ok, all I really care about right now is fixing the missing "service.save()" | 22:43 |
| aorourke | jgriffith, yes that absolutely needs to get done | 22:44 |
| jgriffith | aorourke: honestly that's the most important part in my mind | 22:44 |
| aorourke | jgriffith, definitely | 22:44 |
| jgriffith | aorourke: what happens after that I don't think is going to be ideal no matter what for M... but that's ok | 22:44 |
| jgriffith | aorourke: we need to stop trying to release something perfect at the expense of building a house of cards :) | 22:44 |
| *** sgotliv has joined #openstack-cinder | 22:45 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: Fix up failover_host exceptions to preserve states https://review.openstack.org/291475 | 22:48 |
| aorourke | jgriffith, tell me how that ^ looks. I can remove the other stuff aside from service.save() if we want | 22:48 |
| *** baumann has left #openstack-cinder | 22:49 | |
| jgriffith | aorourke: that seems like a good solution for where we're at | 22:49 |
| jgriffith | aorourke: thanks!! | 22:49 |
| *** crose has quit IRC | 22:51 | |
| patrickeast | sry got side tracked with foosball | 22:51 |
| patrickeast | aorourke: yea i'm on board with that as a fix for M | 22:51 |
| jgriffith | patrickeast: that reminds me... the keg should have been refreshed :) | 22:52 |
| * jgriffith goes to see what beer we have today :) | 22:52 | |
| patrickeast | oo | 22:52 |
| *** mgarza has quit IRC | 22:54 | |
| *** rlrossit has quit IRC | 22:58 | |
| *** jgregor1 has quit IRC | 23:01 | |
| *** jgregor has joined #openstack-cinder | 23:02 | |
| *** diogogmt has joined #openstack-cinder | 23:04 | |
| *** krtaylor has joined #openstack-cinder | 23:05 | |
| jungleboyj | hemna: Ping? | 23:08 |
| hemna | yough | 23:08 |
| jungleboyj | hemna: Silly question oh master of brick. | 23:08 |
| hemna | shoot | 23:08 |
| jungleboyj | If I resize a VM with volumes attached do the volumes need to be reattached afterward the resize completes? | 23:08 |
| hemna | so.... | 23:09 |
| hemna | brick does have extend capability | 23:09 |
| hemna | for attached volumes | 23:09 |
| jungleboyj | hemna: Wait. | 23:09 |
| hemna | but Cinder's API won't allow it FWIW | 23:09 |
| jungleboyj | hemna: Not talking about resizing the volume. | 23:09 |
| hemna | ok | 23:09 |
| jungleboyj | Talking about resizing the VM. | 23:09 |
| hemna | hrmm | 23:09 |
| hemna | what is resizing a vm mean ? | 23:10 |
| jungleboyj | Means changing the CPU or RAM allocated to the VM. | 23:11 |
| hemna | ok | 23:11 |
| jungleboyj | It appears to shut down the VM and migrate it. | 23:11 |
| *** haplo37 has quit IRC | 23:11 | |
| jungleboyj | I am seeing an issue in Icehouse *gagging* where they can't detach the volume after they do a resize. | 23:11 |
| hemna | as long as you don't nuke the volumes from the host and/or remove the volume entries in the domain, you should be ok | 23:11 |
| hemna | icehouse | 23:11 |
| hemna | heh | 23:11 |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 23:11 |
| hemna | I don't think brick existed in icehouse | 23:11 |
| *** krtaylor has quit IRC | 23:11 | |
| *** mmandel has quit IRC | 23:12 | |
| hemna | fwiw | 23:12 |
| jungleboyj | hemna: No, I don't think it did. | 23:12 |
| openstackgerrit | Merged openstack/cinder: Correcting thin provisioning behavior https://review.openstack.org/275408 | 23:15 |
| hemna | jungleboyj, I'm sure that's an ancient libvirt as well | 23:17 |
| smcginnis | jungleboyj: Sounds like something on the compute side of things. | 23:19 |
| hemna | that's what I'm thinking | 23:20 |
| *** laughterwym has joined #openstack-cinder | 23:20 | |
| jungleboyj | smcginnis: Yeah, that is what I am thinking. | 23:22 |
| *** jwcroppe has joined #openstack-cinder | 23:24 | |
| jungleboyj | smcginnis: hemna Now all their volumes have disappeared. | 23:24 |
| * jungleboyj wants to scream | 23:24 | |
| *** laughterwym has quit IRC | 23:25 | |
| smcginnis | jungleboyj: Well, problem solved then. :P | 23:25 |
| *** krtaylor has joined #openstack-cinder | 23:25 | |
| hemna | jungleboyj, does the host still have the volumes in /dev/disk/by-path ? | 23:25 |
| hemna | also, does the domain xml still have the volumes ? | 23:26 |
| hemna | maybe the editing of the domain xml for the 'resizing' nuked em? | 23:26 |
| *** xyang1 has quit IRC | 23:26 | |
| jungleboyj | Ooops, using the wrong tenant. | 23:27 |
| *** haomaiwang has joined #openstack-cinder | 23:27 | |
| *** esker has joined #openstack-cinder | 23:27 | |
| *** esker has quit IRC | 23:28 | |
| *** esker has joined #openstack-cinder | 23:28 | |
| hemna | :) | 23:28 |
| *** melwitt has joined #openstack-cinder | 23:29 | |
| *** haomaiwang has quit IRC | 23:32 | |
| *** gouthamr has joined #openstack-cinder | 23:40 | |
| *** sgotliv has quit IRC | 23:40 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 23:48 |
| *** yhayashi has joined #openstack-cinder | 23:52 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 23:52 |
| *** jwcroppe has quit IRC | 23:57 | |
| mc_nair | anyone know if there's Tempest tests out there for volume migration? | 23:57 |
| *** JoseMello has quit IRC | 23:58 | |
| *** sheel has joined #openstack-cinder | 23:58 | |
| jungleboyj | mc_nair: I think that might be one of the 'ToDos' | 23:59 |
| patrickeast | i thought someone was working on that | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!