| *** alexpilotti has quit IRC | 00:02 | |
| *** mtanino has joined #openstack-cinder | 00:08 | |
| *** yangyapeng has quit IRC | 00:08 | |
| *** jwcroppe has joined #openstack-cinder | 00:09 | |
| *** salv-orl_ has quit IRC | 00:13 | |
| *** neophy has joined #openstack-cinder | 00:16 | |
| *** diogogmt has joined #openstack-cinder | 00:19 | |
| *** dank_ has joined #openstack-cinder | 00:19 | |
| *** patrickeast has joined #openstack-cinder | 00:20 | |
| *** ildikov has joined #openstack-cinder | 00:21 | |
| *** laughterwym has joined #openstack-cinder | 00:26 | |
| *** laughter_ has joined #openstack-cinder | 00:26 | |
| *** laughte__ has joined #openstack-cinder | 00:27 | |
| *** laught___ has joined #openstack-cinder | 00:28 | |
| *** arch-nemesis has quit IRC | 00:29 | |
| *** laugh____ has joined #openstack-cinder | 00:29 | |
| *** laughterwym has quit IRC | 00:30 | |
| *** laughter_ has quit IRC | 00:31 | |
| *** laughte__ has quit IRC | 00:31 | |
| *** laught___ has quit IRC | 00:33 | |
| *** Julien-zte has joined #openstack-cinder | 00:38 | |
| *** ducttape_ has joined #openstack-cinder | 00:38 | |
| *** ducttape_ has quit IRC | 00:38 | |
| *** sheel has joined #openstack-cinder | 00:43 | |
| *** rhefner has quit IRC | 00:53 | |
| *** diogogmt has quit IRC | 00:53 | |
| *** rhefner has joined #openstack-cinder | 00:55 | |
| *** suyash has quit IRC | 00:55 | |
| *** diogogmt has joined #openstack-cinder | 00:55 | |
| *** cheneydc has joined #openstack-cinder | 00:59 | |
| *** akerr has joined #openstack-cinder | 01:01 | |
| *** ducttape_ has joined #openstack-cinder | 01:04 | |
| *** akerr_ has joined #openstack-cinder | 01:04 | |
| *** mtanino has quit IRC | 01:05 | |
| *** akerr has quit IRC | 01:07 | |
| *** ducttape_ has quit IRC | 01:08 | |
| *** karthikp_ has quit IRC | 01:14 | |
| *** yangyapeng has joined #openstack-cinder | 01:15 | |
| *** yangyapeng has quit IRC | 01:15 | |
| *** yangyapeng has joined #openstack-cinder | 01:16 | |
| *** alkhodos has quit IRC | 01:16 | |
| *** karthikp_ has joined #openstack-cinder | 01:16 | |
| openstackgerrit | yuyafei proposed openstack/cinder: Volume manage should get ceiling of volume size https://review.openstack.org/319764 | 01:17 |
|---|---|---|
| *** anteaya has quit IRC | 01:18 | |
| *** diogogmt has quit IRC | 01:18 | |
| *** daneyon_ has joined #openstack-cinder | 01:23 | |
| *** EinstCrazy has joined #openstack-cinder | 01:24 | |
| *** daneyon has quit IRC | 01:28 | |
| *** Julien-zte has quit IRC | 01:28 | |
| *** mtanino has joined #openstack-cinder | 01:30 | |
| *** asselin has joined #openstack-cinder | 01:34 | |
| *** Apoorva has quit IRC | 01:34 | |
| *** mtanino has quit IRC | 01:35 | |
| *** jwcroppe has quit IRC | 01:36 | |
| *** cdelatte has quit IRC | 01:37 | |
| *** asselin has quit IRC | 01:38 | |
| *** Lee1092 has joined #openstack-cinder | 01:38 | |
| *** yangxi has joined #openstack-cinder | 01:40 | |
| *** ducttape_ has joined #openstack-cinder | 01:41 | |
| *** karthikp_ has quit IRC | 01:43 | |
| *** ducttape_ has quit IRC | 01:51 | |
| *** karthikp_ has joined #openstack-cinder | 01:55 | |
| openstackgerrit | wanghao proposed openstack/cinder-specs: transfer snapshots with volumes https://review.openstack.org/319831 | 01:57 |
| *** jwcroppe has joined #openstack-cinder | 01:57 | |
| *** bardia has quit IRC | 01:58 | |
| *** yangxi has quit IRC | 01:59 | |
| *** karthikp_ has quit IRC | 01:59 | |
| *** EinstCrazy has quit IRC | 02:00 | |
| *** EinstCrazy has joined #openstack-cinder | 02:00 | |
| *** IlyaG has quit IRC | 02:00 | |
| *** yangxi has joined #openstack-cinder | 02:02 | |
| *** neophy has quit IRC | 02:07 | |
| *** neophy has joined #openstack-cinder | 02:13 | |
| *** hoonetorg has quit IRC | 02:20 | |
| *** hoonetorg has joined #openstack-cinder | 02:20 | |
| *** markvoelker has quit IRC | 02:21 | |
| *** ducttape_ has joined #openstack-cinder | 02:24 | |
| *** salv-orlando has joined #openstack-cinder | 02:38 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Fix image-volume cache to use volume object when evicting https://review.openstack.org/320187 | 02:38 |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Fix useless api_version of Manager class https://review.openstack.org/309283 | 02:39 |
| openstackgerrit | Patrick East proposed openstack/cinder: Fix image-volume cache to use volume object when evicting https://review.openstack.org/320187 | 02:39 |
| *** salv-orlando has quit IRC | 02:43 | |
| *** ducttape_ has quit IRC | 02:43 | |
| *** enriquetaso has quit IRC | 02:48 | |
| *** akerr_ has quit IRC | 02:48 | |
| *** diogogmt has joined #openstack-cinder | 02:49 | |
| *** karthikp_ has joined #openstack-cinder | 02:51 | |
| *** karthikp_ has quit IRC | 02:56 | |
| *** enriquetaso has joined #openstack-cinder | 02:59 | |
| *** karthikp_ has joined #openstack-cinder | 03:00 | |
| *** karthikp_ has quit IRC | 03:02 | |
| *** raunak has joined #openstack-cinder | 03:06 | |
| *** erlon_ has joined #openstack-cinder | 03:08 | |
| openstackgerrit | xiexs proposed openstack/python-cinderclient: Make __repr__ print encryption_id for VolumeEncryptionType class https://review.openstack.org/319931 | 03:10 |
| *** Julien-zte has joined #openstack-cinder | 03:16 | |
| *** gouthamr_ has quit IRC | 03:17 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Fix retype from non-repl to repl type in Pure drivers https://review.openstack.org/320201 | 03:19 |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Allow API user to update multiattach attr of volume https://review.openstack.org/309674 | 03:21 |
| *** markvoelker has joined #openstack-cinder | 03:22 | |
| *** markvoelker has quit IRC | 03:27 | |
| *** savihou has joined #openstack-cinder | 03:27 | |
| *** sdake_ has quit IRC | 03:29 | |
| *** yangxi has quit IRC | 03:37 | |
| *** dave-mcc_ has quit IRC | 03:38 | |
| *** _alastor__ has quit IRC | 03:38 | |
| *** karthikp_ has joined #openstack-cinder | 03:44 | |
| *** karthikp_ has quit IRC | 03:49 | |
| *** _alastor__ has joined #openstack-cinder | 03:55 | |
| *** david-lyle_ has joined #openstack-cinder | 03:55 | |
| *** david-lyle has quit IRC | 03:57 | |
| *** Julien-zte has quit IRC | 04:01 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Fix image-volume cache to use volume object when evicting https://review.openstack.org/320187 | 04:06 |
| *** links has joined #openstack-cinder | 04:07 | |
| *** salv-orlando has joined #openstack-cinder | 04:08 | |
| *** salv-orlando has quit IRC | 04:17 | |
| *** sdake has joined #openstack-cinder | 04:17 | |
| *** sdake has quit IRC | 04:18 | |
| *** markvoelker has joined #openstack-cinder | 04:23 | |
| *** savihou has quit IRC | 04:26 | |
| *** sgotliv has joined #openstack-cinder | 04:27 | |
| *** markvoelker has quit IRC | 04:28 | |
| *** Poornima has joined #openstack-cinder | 04:36 | |
| *** asselin has joined #openstack-cinder | 04:46 | |
| *** shausy has joined #openstack-cinder | 04:48 | |
| openstackgerrit | Merged openstack/cinder: Fix AttributeError when obtaining 'name' attribute from 'snap' https://review.openstack.org/319827 | 04:48 |
| *** pgadiya has joined #openstack-cinder | 04:53 | |
| *** GB21 has joined #openstack-cinder | 04:58 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Fix wrong usage of exception's attribute message https://review.openstack.org/285712 | 05:06 |
| *** enriquetaso has quit IRC | 05:08 | |
| *** ChubYann has quit IRC | 05:11 | |
| *** karthikp_ has joined #openstack-cinder | 05:12 | |
| *** karthikp_ has quit IRC | 05:13 | |
| *** kambiz has quit IRC | 05:13 | |
| *** links has quit IRC | 05:14 | |
| *** ntpttr has quit IRC | 05:15 | |
| *** dansmith has quit IRC | 05:16 | |
| *** kambiz has joined #openstack-cinder | 05:16 | |
| *** ntpttr has joined #openstack-cinder | 05:16 | |
| *** shausy has quit IRC | 05:17 | |
| *** links has joined #openstack-cinder | 05:18 | |
| *** shausy has joined #openstack-cinder | 05:18 | |
| *** dansmith has joined #openstack-cinder | 05:18 | |
| *** dansmith is now known as Guest71132 | 05:18 | |
| *** avishay has joined #openstack-cinder | 05:18 | |
| *** links has quit IRC | 05:24 | |
| *** markvoelker has joined #openstack-cinder | 05:24 | |
| *** yangxi has joined #openstack-cinder | 05:24 | |
| *** savihou has joined #openstack-cinder | 05:28 | |
| *** markvoelker has quit IRC | 05:29 | |
| *** iceyao has joined #openstack-cinder | 05:34 | |
| *** chenying has quit IRC | 05:37 | |
| openstackgerrit | xiexs proposed openstack/python-cinderclient: Fix Capabilities.__repr__ to remove the undefined attribute https://review.openstack.org/320223 | 05:37 |
| *** chenying has joined #openstack-cinder | 05:38 | |
| *** iceyao has quit IRC | 05:39 | |
| *** iceyao has joined #openstack-cinder | 05:39 | |
| *** GB21 has quit IRC | 05:44 | |
| *** lpetrut has joined #openstack-cinder | 05:53 | |
| *** erlon_ has quit IRC | 05:56 | |
| *** GB21 has joined #openstack-cinder | 05:56 | |
| *** wN has quit IRC | 05:56 | |
| *** Fdaisuke has quit IRC | 06:00 | |
| *** shausy has quit IRC | 06:00 | |
| *** Fdaisuke has joined #openstack-cinder | 06:03 | |
| *** links has joined #openstack-cinder | 06:04 | |
| openstackgerrit | xiexs proposed openstack/python-cinderclient: Fix Service.__repr__ to remove the undefined attribute https://review.openstack.org/320230 | 06:07 |
| *** mjura has joined #openstack-cinder | 06:07 | |
| *** rcernin has joined #openstack-cinder | 06:08 | |
| openstackgerrit | haobing proposed openstack/cinder: Allow remove description and name for consisgroup https://review.openstack.org/312063 | 06:09 |
| *** sdake has joined #openstack-cinder | 06:11 | |
| *** mjura has quit IRC | 06:13 | |
| *** mjura has joined #openstack-cinder | 06:13 | |
| *** scottda has quit IRC | 06:14 | |
| *** serverascode has quit IRC | 06:15 | |
| *** DuncanT has quit IRC | 06:15 | |
| *** scottda has joined #openstack-cinder | 06:16 | |
| *** zhenguo_ has quit IRC | 06:16 | |
| *** zhenguo_ has joined #openstack-cinder | 06:18 | |
| *** wN has joined #openstack-cinder | 06:18 | |
| *** wN has quit IRC | 06:18 | |
| *** wN has joined #openstack-cinder | 06:18 | |
| *** DuncanT has joined #openstack-cinder | 06:20 | |
| openstackgerrit | Merged openstack/cinder: Sheepdog:make full use of all sheepdog nodes https://review.openstack.org/296220 | 06:21 |
| *** cppforlife_ has quit IRC | 06:21 | |
| *** fmccrthy has quit IRC | 06:21 | |
| *** med_ has quit IRC | 06:21 | |
| *** fmccrthy has joined #openstack-cinder | 06:22 | |
| *** cppforlife_ has joined #openstack-cinder | 06:23 | |
| *** markvoelker has joined #openstack-cinder | 06:25 | |
| *** med_ has joined #openstack-cinder | 06:25 | |
| *** med_ has quit IRC | 06:25 | |
| *** med_ has joined #openstack-cinder | 06:25 | |
| *** sdake has quit IRC | 06:25 | |
| *** harlowja has quit IRC | 06:28 | |
| *** markvoelker has quit IRC | 06:29 | |
| openstackgerrit | Bhagyashri Shewale proposed openstack/cinder: Return BadRequest for invalid 4byte unicode character https://review.openstack.org/295097 | 06:31 |
| *** serverascode has joined #openstack-cinder | 06:32 | |
| *** cebruns has quit IRC | 06:33 | |
| *** cebruns has joined #openstack-cinder | 06:35 | |
| *** sdake has joined #openstack-cinder | 06:37 | |
| *** salv-orlando has joined #openstack-cinder | 06:37 | |
| *** neophy has quit IRC | 06:40 | |
| *** liverpooler has joined #openstack-cinder | 06:41 | |
| *** earlephilhower has quit IRC | 06:46 | |
| *** chip_ has quit IRC | 06:49 | |
| *** chip_ has joined #openstack-cinder | 06:50 | |
| *** yangxi_ has joined #openstack-cinder | 06:51 | |
| *** sdake has quit IRC | 06:53 | |
| *** yangxi has quit IRC | 06:54 | |
| flip214 | openstackclient 2.4.0 doesn't have "volume attach" anymore. how can I do that now? | 06:57 |
| flip214 | hmmm, perhaps the cinder client works for that use case | 06:57 |
| flip214 | or rather nova | 06:58 |
| *** lpetrut has quit IRC | 06:58 | |
| flip214 | no, it doesn't. ERROR (AttributeError): 'Namespace' object has no attribute 'os_user_id' | 06:58 |
| *** tesseract has joined #openstack-cinder | 07:04 | |
| *** chip_ has quit IRC | 07:07 | |
| *** chip_ has joined #openstack-cinder | 07:08 | |
| *** tesseract has quit IRC | 07:08 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Manage/unmanage snapshot in ScaleIO driver https://review.openstack.org/316742 | 07:09 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Fix wrong usage of exception's attribute message https://review.openstack.org/285712 | 07:09 |
| tobasco | dulek: you around? | 07:18 |
| tobasco | dulek: do you know if this one was backported to stable/liberty https://review.openstack.org/#/c/238866/ | 07:19 |
| tobasco | seeing that the code does not exist in the ubuntu repo I'm applying this manually | 07:19 |
| *** rwsu has joined #openstack-cinder | 07:19 | |
| *** rwsu has left #openstack-cinder | 07:20 | |
| *** lennyb_ has joined #openstack-cinder | 07:23 | |
| *** markvoelker has joined #openstack-cinder | 07:26 | |
| *** lennyb_ has quit IRC | 07:26 | |
| *** lennyb_ has joined #openstack-cinder | 07:27 | |
| *** nkrinner has joined #openstack-cinder | 07:28 | |
| *** lny has joined #openstack-cinder | 07:30 | |
| *** markvoelker has quit IRC | 07:32 | |
| *** lny has quit IRC | 07:32 | |
| *** lennyb_ has quit IRC | 07:33 | |
| *** sgotliv_ has joined #openstack-cinder | 07:35 | |
| *** sgotliv has quit IRC | 07:37 | |
| *** yangxi_ has quit IRC | 07:38 | |
| *** sgotliv__ has joined #openstack-cinder | 07:40 | |
| *** yangxi has joined #openstack-cinder | 07:42 | |
| *** sgotliv_ has quit IRC | 07:42 | |
| *** yangxi_ has joined #openstack-cinder | 07:43 | |
| *** sgotliv_ has joined #openstack-cinder | 07:43 | |
| *** sgotliv__ has quit IRC | 07:45 | |
| *** yangxi has quit IRC | 07:47 | |
| *** savihou has quit IRC | 07:48 | |
| *** savihou has joined #openstack-cinder | 07:48 | |
| *** sgotliv_ has quit IRC | 07:48 | |
| *** lpetrut has joined #openstack-cinder | 07:48 | |
| *** sgotliv has joined #openstack-cinder | 07:50 | |
| *** jordanP has joined #openstack-cinder | 07:51 | |
| *** sgotliv_ has joined #openstack-cinder | 07:53 | |
| *** adisky has joined #openstack-cinder | 07:55 | |
| *** sgotliv has quit IRC | 07:55 | |
| *** arecknag has joined #openstack-cinder | 07:56 | |
| *** Fdaisuke has quit IRC | 07:57 | |
| *** sgotliv_ has quit IRC | 07:58 | |
| *** sgotliv_ has joined #openstack-cinder | 07:58 | |
| *** Fdaisuke has joined #openstack-cinder | 07:59 | |
| *** GB21 has quit IRC | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| *** jseiler has quit IRC | 08:01 | |
| *** jseiler has joined #openstack-cinder | 08:02 | |
| *** raunak has quit IRC | 08:03 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| dulek | tobasco: It looks like it isn't there. Sorry. | 08:12 |
| dulek | FYI: I've broke a bone in my wirst, so my | 08:13 |
| dulek | my hand is in a cast. I write extremely slowly now. ;) | 08:13 |
| lixiaoy1 | dulek: sorry to hear that. Take care~~ | 08:14 |
| *** belmoreira has joined #openstack-cinder | 08:15 | |
| *** lpetrut has quit IRC | 08:17 | |
| *** GB21 has joined #openstack-cinder | 08:17 | |
| *** e0ne has joined #openstack-cinder | 08:20 | |
| *** salv-orlando has quit IRC | 08:25 | |
| *** afazekas_ is now known as afazekas | 08:27 | |
| *** markvoelker has joined #openstack-cinder | 08:28 | |
| tobasco | dulek: Ok, ty I'm gonna try pushing a backport for it, not gonna stress your hand about it ;) get well, I'll probably hit you up if I get stuck | 08:28 |
| openstackgerrit | WeiJi proposed openstack/cinder: Normalize the options use singele quotes https://review.openstack.org/320292 | 08:28 |
| *** e0ne has quit IRC | 08:31 | |
| *** e0ne has joined #openstack-cinder | 08:32 | |
| dulek | tobasco: http://releases.openstack.org/ | 08:32 |
| *** markvoelker has quit IRC | 08:32 | |
| dulek | tobasco: Liberty is security-supported now, so you probably won't be able to do so. | 08:33 |
| *** caspinol has joined #openstack-cinder | 08:33 | |
| tobasco | dulek: aw, even when the stuff isn't working without that 2 line fix? | 08:33 |
| dulek | tobasco: Dura lex sed lex. ;) Sorry, that are global OpenStack release guidelines. | 08:35 |
| tobasco | dulek: well, you can't have everything here in live, I'll have to live with it until upgrade then | 08:36 |
| tobasco | dulek: ty for feedback | 08:36 |
| *** EinstCrazy has quit IRC | 08:37 | |
| *** EinstCrazy has joined #openstack-cinder | 08:40 | |
| *** ig0r_ has joined #openstack-cinder | 08:40 | |
| *** laugh____ has quit IRC | 08:42 | |
| *** laughterwym has joined #openstack-cinder | 08:43 | |
| *** The_Ball has quit IRC | 08:47 | |
| *** laughterwym has quit IRC | 08:48 | |
| *** EinstCrazy has quit IRC | 08:51 | |
| *** EinstCrazy has joined #openstack-cinder | 08:57 | |
| *** lpetrut has joined #openstack-cinder | 09:07 | |
| *** lpetrut1 has joined #openstack-cinder | 09:12 | |
| *** caspinol has quit IRC | 09:13 | |
| *** lpetrut has quit IRC | 09:13 | |
| *** lpetrut1 is now known as lpetrut | 09:13 | |
| *** jsheeren has joined #openstack-cinder | 09:15 | |
| jsheeren | hi all, i have a quick question about netapp and copy offloading. do i understand correctly that the glance volume must be mounted where the cinder-volume service lives? | 09:16 |
| *** ociuhandu has joined #openstack-cinder | 09:17 | |
| *** sdake has joined #openstack-cinder | 09:20 | |
| *** ociuhandu has quit IRC | 09:20 | |
| *** GB21 has quit IRC | 09:21 | |
| *** yangxi_ has quit IRC | 09:21 | |
| *** yangxi has joined #openstack-cinder | 09:22 | |
| *** yangxi has quit IRC | 09:24 | |
| *** yangxi has joined #openstack-cinder | 09:27 | |
| *** zigo_ is now known as zigo | 09:28 | |
| *** jordanP has quit IRC | 09:32 | |
| *** The_Ball has joined #openstack-cinder | 09:36 | |
| *** yangxi has quit IRC | 09:37 | |
| *** Poornima has quit IRC | 09:37 | |
| *** GB21 has joined #openstack-cinder | 09:37 | |
| *** Poornima has joined #openstack-cinder | 09:38 | |
| *** gcb has quit IRC | 09:40 | |
| *** zhongjun_ has quit IRC | 09:41 | |
| *** gcb has joined #openstack-cinder | 09:41 | |
| *** zhongjun_ has joined #openstack-cinder | 09:41 | |
| openstackgerrit | Philipp Marek proposed openstack/cinder: DRBD: ensure_export() might not have 'provider_location' https://review.openstack.org/320328 | 09:41 |
| *** jordanP has joined #openstack-cinder | 09:45 | |
| *** sgotliv has joined #openstack-cinder | 09:46 | |
| *** sgotliv_ has quit IRC | 09:47 | |
| *** yangxi has joined #openstack-cinder | 09:48 | |
| *** zhongjun_ has quit IRC | 09:50 | |
| *** zhongjun_ has joined #openstack-cinder | 09:51 | |
| *** sgotliv_ has joined #openstack-cinder | 09:51 | |
| *** sgotliv has quit IRC | 09:52 | |
| *** Poornima has quit IRC | 09:52 | |
| *** mvk has quit IRC | 09:53 | |
| *** caspinol has joined #openstack-cinder | 09:56 | |
| *** iceyao_ has joined #openstack-cinder | 09:59 | |
| *** iceyao_ has quit IRC | 10:00 | |
| *** iceyao__ has joined #openstack-cinder | 10:00 | |
| *** iceyao__ has quit IRC | 10:00 | |
| *** iceyao_ has joined #openstack-cinder | 10:01 | |
| *** cheneydc has quit IRC | 10:01 | |
| *** sdake has quit IRC | 10:01 | |
| *** iceyao__ has joined #openstack-cinder | 10:02 | |
| *** iceyao_ has quit IRC | 10:02 | |
| *** iceyao has quit IRC | 10:02 | |
| *** yangxi has quit IRC | 10:03 | |
| *** Poornima has joined #openstack-cinder | 10:04 | |
| *** lprice has joined #openstack-cinder | 10:06 | |
| *** lprice1 has quit IRC | 10:06 | |
| *** avishay has quit IRC | 10:09 | |
| *** e0ne has quit IRC | 10:13 | |
| *** e0ne has joined #openstack-cinder | 10:14 | |
| openstackgerrit | xiexs proposed openstack/python-cinderclient: Add max-width feature to limit the width of the printed table https://review.openstack.org/319044 | 10:14 |
| openstackgerrit | xiexs proposed openstack/python-cinderclient: Add max-width feature to limit the width of the printed table https://review.openstack.org/319044 | 10:17 |
| *** zhongjun_ has quit IRC | 10:18 | |
| *** zhongjun_ has joined #openstack-cinder | 10:19 | |
| *** mvk has joined #openstack-cinder | 10:22 | |
| *** avishay has joined #openstack-cinder | 10:24 | |
| *** markvoelker has joined #openstack-cinder | 10:29 | |
| *** asselin has quit IRC | 10:31 | |
| *** EinstCrazy has quit IRC | 10:32 | |
| *** EinstCrazy has joined #openstack-cinder | 10:32 | |
| *** yangyapeng has quit IRC | 10:34 | |
| *** laughterwym has joined #openstack-cinder | 10:34 | |
| *** markvoelker has quit IRC | 10:34 | |
| *** EinstCrazy has quit IRC | 10:37 | |
| *** jsheeren has quit IRC | 10:40 | |
| *** laughterwym has quit IRC | 10:40 | |
| *** e0ne_ has joined #openstack-cinder | 10:41 | |
| *** laughterwym has joined #openstack-cinder | 10:41 | |
| *** e0ne has quit IRC | 10:42 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Add logging when filtering returns nothing https://review.openstack.org/306285 | 10:42 |
| *** jsheeren has joined #openstack-cinder | 10:45 | |
| *** laughterwym has quit IRC | 10:47 | |
| *** adisky has quit IRC | 10:50 | |
| *** caspinol has quit IRC | 10:53 | |
| johnthetubaguy | ildikov: been looking through the spec, sorry I have some questions, but I think its only small tweaks | 10:59 |
| johnthetubaguy | ildikov: I am going to get myself some food, but I added the comments on the spec | 11:00 |
| ildikov | johnthetubaguy: cool, tnx, I'll check | 11:02 |
| *** laughterwym has joined #openstack-cinder | 11:02 | |
| *** cdelatte has joined #openstack-cinder | 11:03 | |
| *** yangxi has joined #openstack-cinder | 11:03 | |
| *** caspinol has joined #openstack-cinder | 11:04 | |
| *** yangxi has quit IRC | 11:08 | |
| *** avishay has quit IRC | 11:10 | |
| *** salv-orlando has joined #openstack-cinder | 11:15 | |
| *** sdake has joined #openstack-cinder | 11:17 | |
| *** salv-orlando has quit IRC | 11:19 | |
| *** iceyao__ has quit IRC | 11:23 | |
| *** avishay has joined #openstack-cinder | 11:23 | |
| *** sdague has joined #openstack-cinder | 11:25 | |
| *** ociuhandu has joined #openstack-cinder | 11:28 | |
| *** ankit_ag has joined #openstack-cinder | 11:28 | |
| *** markvoelker has joined #openstack-cinder | 11:30 | |
| *** caspinol has quit IRC | 11:31 | |
| *** ig0r_ has quit IRC | 11:34 | |
| *** markvoelker has quit IRC | 11:35 | |
| openstackgerrit | Bertrand Lallau proposed openstack/cinder: Convert to strutils.bool_from_string https://review.openstack.org/320373 | 11:35 |
| *** julim has joined #openstack-cinder | 11:45 | |
| *** salv-orlando has joined #openstack-cinder | 11:46 | |
| *** yhayashi has quit IRC | 11:48 | |
| *** sdake has quit IRC | 11:49 | |
| *** savihou has quit IRC | 11:51 | |
| *** savihou has joined #openstack-cinder | 11:51 | |
| *** ig0r_ has joined #openstack-cinder | 11:57 | |
| *** ducttape_ has joined #openstack-cinder | 11:59 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Add logging when filtering returns nothing https://review.openstack.org/306285 | 12:00 |
| dulek | geguileo: Hi! How dependent is https://review.openstack.org/#/c/315541 on refactoring of create, save, destroy in ovo? Looks to me like just a little, isn't it? | 12:00 |
| geguileo | dulek: That's the patch you are referring, so I guess you mean the refactor sqlalchemy patch, right? | 12:01 |
| dulek | geguileo: Yes, sorry. | 12:01 |
| geguileo | dulek: I think it's not really dependent | 12:02 |
| geguileo | dulek: But all that depend on that one do depend on the OVO refactor | 12:02 |
| *** markvoelker has joined #openstack-cinder | 12:02 | |
| geguileo | dulek: So both refactors could go on their own, but I'll need both for the job distribution patches I'm working on | 12:03 |
| dulek | geguileo: Oh… | 12:03 |
| dulek | geguileo: I'm asking because I see 315541 as a rather dangerous one. And it breaks Python's explicit is better than implicit. | 12:03 |
| geguileo | dulek: I don't think it breaks it more than the get_id mechanism | 12:03 |
| geguileo | dulek: And we have a lot of code duplication | 12:04 |
| dulek | geguileo: It seems like too much magic to me. It even adds more lines than it removes. | 12:04 |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Add logging when filtering returns nothing https://review.openstack.org/306285 | 12:04 |
| geguileo | dulek: And for every new OVO we introduce we introduce even more | 12:04 |
| geguileo | dulek: Well, those are mostly comments | 12:04 |
| geguileo | dulek: Because we almost had none of those | 12:04 |
| geguileo | dulek: And when I add the Cluster OVO it removes a lot of code | 12:05 |
| geguileo | dulek: Because the Cluster create, save, delete, don't need to be added | 12:06 |
| dulek | You're probably right on the comments… Okay, I'll take look on whole chain to judge that. | 12:06 |
| geguileo | dulek: The chain is broken right now... | 12:07 |
| dulek | Sure, but it still should give me a better picture. | 12:07 |
| geguileo | dulek: Because I had a longer chain, but with the job distribution changes I just have a couple so that we could check the general idea of the new stuff | 12:07 |
| geguileo | dulek: Ok | 12:07 |
| * dulek feels like stutterer when writing using one hand only. | 12:09 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add unit tests for Huawei drivers https://review.openstack.org/283029 | 12:10 |
| *** ducttape_ has quit IRC | 12:12 | |
| *** Lee1092 has quit IRC | 12:13 | |
| *** avishay has quit IRC | 12:14 | |
| *** GB21 has quit IRC | 12:20 | |
| *** adrianofr has joined #openstack-cinder | 12:27 | |
| *** salv-orlando has quit IRC | 12:35 | |
| *** timcl has joined #openstack-cinder | 12:36 | |
| *** charlesr has joined #openstack-cinder | 12:37 | |
| *** charlesr has quit IRC | 12:37 | |
| *** crose has quit IRC | 12:38 | |
| *** crose has joined #openstack-cinder | 12:38 | |
| smcginnis | dulek: What happened with the hand?! | 12:41 |
| dulek | smcginnis: It's in maintenance mode for next few weeks. :D | 12:44 |
| openstackgerrit | wanghao proposed openstack/cinder-specs: transfer snapshots with volumes https://review.openstack.org/319831 | 12:44 |
| dulek | I've tripped over and broke small bone in the wrist. It's quite common injury it turns out. | 12:45 |
| smcginnis | dulek: Ouch! I hope it heals quickly. | 12:48 |
| smcginnis | dulek: Probably one of the worse injuries for a software guy. :] | 12:48 |
| *** catintheroof has joined #openstack-cinder | 12:49 | |
| dulek | Doctors say it should be in a cast for ~6 weeks. | 12:49 |
| dulek | Well, at least I've discovered that speech recognition in android works quite well for both Polish and English. | 12:50 |
| *** asselin has joined #openstack-cinder | 12:50 | |
| *** diogogmt has quit IRC | 12:50 | |
| dulek | Unfortunately new Gerrit UI doesn't work well on mobile devices. :P | 12:51 |
| smcginnis | dulek: Hah, that's good. | 12:51 |
| smcginnis | dulek: Yeah, gerrit on any kind of mobile device is completely useless. | 12:51 |
| *** gouthamr has joined #openstack-cinder | 12:51 | |
| *** timcl has quit IRC | 12:52 | |
| *** diogogmt has joined #openstack-cinder | 12:52 | |
| *** timcl has joined #openstack-cinder | 12:53 | |
| *** akshai has joined #openstack-cinder | 12:53 | |
| *** akshai_ has joined #openstack-cinder | 12:54 | |
| *** sheel has quit IRC | 12:55 | |
| *** jordanP has quit IRC | 12:55 | |
| *** jordanP has joined #openstack-cinder | 12:57 | |
| *** david-lyle has joined #openstack-cinder | 12:57 | |
| *** akshai has quit IRC | 12:58 | |
| *** jistr is now known as jistr|mtg | 13:01 | |
| *** david-lyle_ has quit IRC | 13:01 | |
| *** edmondsw has joined #openstack-cinder | 13:05 | |
| *** rlrossit has joined #openstack-cinder | 13:07 | |
| *** alyson_ has joined #openstack-cinder | 13:07 | |
| *** diogogmt has quit IRC | 13:07 | |
| openstackgerrit | Bertrand Lallau proposed openstack/cinder: Convert to strutils.bool_from_string https://review.openstack.org/320373 | 13:07 |
| *** sdake has joined #openstack-cinder | 13:09 | |
| *** xyang1 has joined #openstack-cinder | 13:09 | |
| *** salv-orlando has joined #openstack-cinder | 13:11 | |
| *** sdake has quit IRC | 13:13 | |
| *** gfm has joined #openstack-cinder | 13:16 | |
| *** gfm is now known as gfm-r | 13:16 | |
| *** eharney has quit IRC | 13:17 | |
| *** gfm-r has quit IRC | 13:17 | |
| *** porrua has joined #openstack-cinder | 13:17 | |
| *** pgadiya has quit IRC | 13:21 | |
| *** pgadiya has joined #openstack-cinder | 13:22 | |
| *** dave-mccowan has joined #openstack-cinder | 13:24 | |
| *** dustins has joined #openstack-cinder | 13:24 | |
| *** lprice has quit IRC | 13:25 | |
| *** dustins has quit IRC | 13:29 | |
| *** dustins has joined #openstack-cinder | 13:29 | |
| *** abhishekk has joined #openstack-cinder | 13:29 | |
| abhishekk | mikal: hi you around | 13:30 |
| *** eharney has joined #openstack-cinder | 13:30 | |
| *** ildikov has quit IRC | 13:34 | |
| *** ildikov has joined #openstack-cinder | 13:34 | |
| *** Poornima has quit IRC | 13:36 | |
| *** pgadiya has quit IRC | 13:36 | |
| *** BigWillie has joined #openstack-cinder | 13:38 | |
| johnthetubaguy | ildikov: thanks for your reply on the spec, I think we should include the detail about requiring a cinder microversion, if thats possible | 13:38 |
| *** gman-tx has joined #openstack-cinder | 13:39 | |
| ildikov | johnthetubaguy: I wanted to clarify here whether we see any other option, personally I don't | 13:43 |
| johnthetubaguy | sounds good | 13:43 |
| ildikov | johnthetubaguy: if that's the case I will add it of course | 13:43 |
| *** jungleboyj has quit IRC | 13:43 | |
| ildikov | johnthetubaguy: do you think I can remove the volume_snapshot_create case? | 13:43 |
| *** iceyao has joined #openstack-cinder | 13:45 | |
| *** nikeshm has joined #openstack-cinder | 13:45 | |
| *** baumann has joined #openstack-cinder | 13:46 | |
| ildikov | I mean to block it for multiattach volumes | 13:46 |
| *** ametts has joined #openstack-cinder | 13:46 | |
| *** Lee1092 has joined #openstack-cinder | 13:48 | |
| dulek | smcginnis: On your recent email. ZadaraStorage CI is the only one third party CI in Cinder that have this "(non-voting)" added to the result in Gerrit. | 13:48 |
| smcginnis | dulek: I believe Brocade does as well. | 13:49 |
| smcginnis | dulek: No harm either way if it is there or not. | 13:49 |
| smcginnis | dulek: Not necessary IMO. | 13:49 |
| *** merooney has joined #openstack-cinder | 13:49 | |
| dulek | smcginnis: Okay, just wanted to clarify why I think that email was sent. | 13:50 |
| johnthetubaguy | ildikov: not sure, I think we probably need to keep it | 13:51 |
| smcginnis | dulek: Ah, OK. | 13:51 |
| johnthetubaguy | ildikov: I just think we need a little more background on what its doing, I think thats technically a cinder only API | 13:51 |
| ildikov | johnthetubaguy: if possible I would like to minimise the round trips between Nova and Cinder | 13:52 |
| *** jbernard_ is now known as jbernard | 13:52 | |
| ildikov | johnthetubaguy: so I'm definitely on the side of getting more background and do what's necessary | 13:53 |
| johnthetubaguy | ildikov: I suspect we really should be doing more storing of the volume info in the BDM, to stop all the extra cinder calls, but thats a different thing, but agreed! | 13:54 |
| ildikov | johnthetubaguy: from the weekly meetings it seems to me that we're on the way to understand how Cinder and Nova interacts as it got a bit out of control | 13:54 |
| johnthetubaguy | yeah, +1 | 13:54 |
| ildikov | johnthetubaguy: and also we should keep things on Cinder side that belongs there | 13:55 |
| ildikov | johnthetubaguy: whenever and wherever w can simplify Nova and clean up the responsibilities we should do that | 13:55 |
| ildikov | *we | 13:55 |
| johnthetubaguy | ildikov: +1 | 13:56 |
| smcginnis | ildikov: Definitely +1 | 13:56 |
| smcginnis | I think the lines of responsibility really got blurred. | 13:57 |
| smcginnis | Or started out that way with cinder coming out of nova. | 13:57 |
| jgriffith | smcginnis: nah.. back then it was easy :) | 13:57 |
| smcginnis | jgriffith: Heh, no doubt. | 13:57 |
| johnthetubaguy | we didn't do stability or error handling then, life was easy going | 13:57 |
| ildikov | smcginnis: yeah, I think this didn't get fixed after Cinder came out of Nova | 13:57 |
| smcginnis | The good old days. It was a simpler time... | 13:57 |
| *** Julien-zte has joined #openstack-cinder | 13:58 | |
| ildikov | then let's aim for a simpler future after the cleanups :) | 13:58 |
| * smcginnis cues the Andy Griffin Show music | 13:58 | |
| johnthetubaguy | looks like we never fully adopted the reserve_volume call | 13:58 |
| johnthetubaguy | but yeah, heading the right way now, thats the main thing | 13:58 |
| ildikov | jgriffith: BTW, any progress with your patches that you can share? :) | 13:59 |
| smcginnis | Oh, Griffith, not Griffin. What a coincidence. | 13:59 |
| smcginnis | Or is it? :) | 14:00 |
| *** jgregor has joined #openstack-cinder | 14:00 | |
| ildikov | johnthetubaguy: do you know why is there a separate attach flow for BFV? I mean the case when we are simply attaching a Cinder volume at boot time | 14:00 |
| jgriffith | ildikov: yeah, I'm back at work today... I ended up having to take the last several days off :( | 14:00 |
| *** cknight has joined #openstack-cinder | 14:00 | |
| jgriffith | smcginnis: ? | 14:00 |
| ildikov | johnthetubaguy: is it only to handle BFV as one big item or there's another reason behind? | 14:00 |
| jgriffith | LOL | 14:00 |
| jgriffith | got it | 14:00 |
| smcginnis | Ignore me, I'm rambling on nonsense this morning. :) | 14:00 |
| *** crose has quit IRC | 14:00 | |
| ildikov | jgriffith: I hope everything is fine | 14:00 |
| jgriffith | smcginnis: my nick name used to be "opey" | 14:00 |
| smcginnis | Hah! | 14:01 |
| jgriffith | ildikov: things are ok, thank you | 14:01 |
| ildikov | jgriffith: and also I think we're fine with very WIP state patches as well, so if you can upload and point people where they can help out we would be happy to | 14:01 |
| jgriffith | Or I guess it's "Opie" | 14:01 |
| ildikov | jgriffith: or at least I would be happy to chime in :) | 14:01 |
| jgriffith | ildikov: I keep reworking things | 14:01 |
| jgriffith | ildikov: so what I did last night was try and simplify things under the assumption of no multi-attach and using nova as is today | 14:02 |
| *** ducttape_ has joined #openstack-cinder | 14:02 | |
| jgriffith | ildikov: I think have a patch for nova, cindreclient, and another for cinder to implement a new way of attach/detach | 14:02 |
| ildikov | jgriffith: sure, but for instance I can play with unit tests in one patch while you're working on refactoring some items, etc | 14:02 |
| jgriffith | ildikov: keeping the cinder code so that an older nova will stil work | 14:02 |
| jgriffith | ildikov: oh... well if someobdy wants to volunteer to help on the unit test madness (and it is madness) I'll push some WIP code up when I get to my desk | 14:03 |
| ildikov | jgriffith: in the meantime we're trying to do some cleanup and rationalisation in Nova, so I hope these will meet nicely | 14:03 |
| *** darosale has joined #openstack-cinder | 14:03 | |
| jgriffith | ildikov: ahh... got ya | 14:03 |
| *** diablo_rojo has joined #openstack-cinder | 14:04 | |
| *** ankit_ag has quit IRC | 14:04 | |
| *** jistr|mtg is now known as jistr | 14:04 | |
| jgriffith | awww.. who put a check in devstack for xenial :( | 14:04 |
| ildikov | jgriffith: I'm sure I will not like you and the Cinder team when looking at the non-unit unit tests, but that's an acceptable price for progress I think :) | 14:04 |
| jgriffith | ildikov: haha... yeah, they're pretty awful | 14:05 |
| smcginnis | jgriffith: Do they block it now? | 14:05 |
| jgriffith | smcginnis: yeah, because theyr'e not unit tests at all | 14:05 |
| smcginnis | Oh, I meant the xenial comment. | 14:05 |
| jgriffith | smcginnis: oh sorry :) | 14:05 |
| smcginnis | jgriffith: But yeah, I get you on the "unit" tests. | 14:06 |
| jgriffith | smcginnis: so they added a check now and it stops devstack from running, but you can set "FORCE" and it'll work | 14:06 |
| savihou | smcginnis: WRT dulek's note regarding the non-voting tag (ZadaraStorage CI); is there any concern with it? I've sent out earlier todat a voting rights nomination request for it .. | 14:06 |
| smcginnis | Devstack was working fine for me on Xenial. | 14:06 |
| *** baumann has quit IRC | 14:06 | |
| ildikov | jgriffith: yeah, I can imagine, I was just assuming that you have better things to do within those patches than the unit tests, so I can do that in parallel and maybe have better progress this way if I succeed to fix things | 14:07 |
| *** cknight1 has joined #openstack-cinder | 14:07 | |
| jgriffith | smcginnis: me too... been using it since it came out | 14:07 |
| jgriffith | smcginnis: LVM on restarts is a train wreck, but other than that | 14:07 |
| jgriffith | ildikov: nahh, it's part of submitting code | 14:07 |
| *** ducttape_ has quit IRC | 14:08 | |
| *** cknight has quit IRC | 14:08 | |
| *** dustins has quit IRC | 14:09 | |
| ildikov | jgriffith: anyway, just let me know if I can make myself useful and help out with your patches :) | 14:09 |
| *** dustins has joined #openstack-cinder | 14:09 | |
| ildikov | jgriffith: also due to the holiday next Monday in the US and also UK we will have the next week's meeting on Thursday, 1700UTC | 14:09 |
| *** ducttape_ has joined #openstack-cinder | 14:10 | |
| *** Guest71132 is now known as dansmith | 14:11 | |
| ildikov | smcginnis: jgriffith: also we had a small discussion with johnthetubaguy on the multiattach spec review here: https://review.openstack.org/#/c/304681/3/specs/newton/approved/multi-attach-volume.rst | 14:11 |
| *** dansmith is now known as Guest21621 | 14:12 | |
| ildikov | smcginnis: jgriffith: two items are in question, which is the volume_snapshot_create API in Nova and the other one is a Cinder API microversion which we think will be required for multi-attach | 14:12 |
| *** jwcroppe_ has joined #openstack-cinder | 14:13 | |
| ildikov | smcginnis: jgriffith: from the two items myself I'm certainly not educated on the volume_snapshot one | 14:13 |
| smcginnis | ildikov: Yeah, we'll need a microversion bump for the API changes. | 14:13 |
| ildikov | smcginnis: jgriffith: do you happen to have any background on that one? | 14:13 |
| * smcginnis looking now... | 14:13 | |
| *** Guest21621 is now known as dansmith | 14:14 | |
| *** diogogmt has joined #openstack-cinder | 14:14 | |
| ildikov | smcginnis: ok, I thought so too, I will add it to the spec as well, I mean the microversion | 14:14 |
| *** jwcroppe has quit IRC | 14:14 | |
| *** julim has quit IRC | 14:15 | |
| smcginnis | ildikov: So the snapshot question is around the Nova side with OS assisted snapshot? Am I understanding that right? | 14:16 |
| *** takedakn has joined #openstack-cinder | 14:16 | |
| ildikov | smcginnis: kind of yes | 14:16 |
| *** merooney has quit IRC | 14:16 | |
| jgriffith | ildikov: I think the current proposals for multi-attach are all wrong | 14:17 |
| ildikov | smcginnis: johnthetubaguy pointed out that this snapshot API is mainly for Cinder to use | 14:17 |
| smcginnis | ildikov: So that part I think would be tricky (OS assisted) but on the cinder side we can snap no problem. | 14:17 |
| jgriffith | ildikov: I don't think it needs be that hard | 14:17 |
| jgriffith | just a second... have to pay attention to this part of the meeting :) | 14:17 |
| *** sheel has joined #openstack-cinder | 14:18 | |
| ildikov | jgriffith: well, there aren't too many restrictions there by now I think, but we can discuss | 14:18 |
| ildikov | smcginnis: in what sense do you mean it's tricky? | 14:18 |
| smcginnis | ildikov: If it's not OS assisted then we can just snap the volume, so no big deal there. | 14:20 |
| smcginnis | ildikov: But if it is OS assisted... | 14:20 |
| smcginnis | ildikov: And we have multiple attachements... | 14:21 |
| smcginnis | ildikov: Then it seems we would need some kind of coordination across the multiple attached OS's for that. | 14:21 |
| ildikov | smcginnis: in what cases OS assisted is used? | 14:21 |
| smcginnis | ildikov: Just not a 1:1 relationship anymore. | 14:21 |
| smcginnis | ildikov: I don't have a lot of experience there, so maybe someone else can chime in. | 14:21 |
| smcginnis | ildikov: But from my understanding, the OS will do some work to flush IO so we can make sure the snapshot is good consistent data. | 14:22 |
| *** lprice has joined #openstack-cinder | 14:22 | |
| smcginnis | ildikov: Without OS involvement there is no guarantee that all IO has been written to disk at the time it is snapped. | 14:23 |
| ildikov | smcginnis: is that used for "local" volumes as pointed out in the comments? | 14:23 |
| *** diogogmt has quit IRC | 14:23 | |
| ildikov | smcginnis: or it can be applicable for all types? | 14:24 |
| openstackgerrit | Merged openstack/cinder: Add hacking check to prevent assert_called_once https://review.openstack.org/319965 | 14:26 |
| *** merooney has joined #openstack-cinder | 14:26 | |
| *** lprice1 has joined #openstack-cinder | 14:28 | |
| *** rhagarty has joined #openstack-cinder | 14:28 | |
| thrawn01 | hello all, I'm attempting to implement this user story for cinder https://review.openstack.org/#/c/291298 should I first create a cinder blueprint before attempting the work? | 14:28 |
| smcginnis | ildikov: Sorry, looking... | 14:28 |
| *** baumann has joined #openstack-cinder | 14:28 | |
| *** logan- has quit IRC | 14:29 | |
| *** lprice has quit IRC | 14:29 | |
| smcginnis | thrawn01: What does that require in Cinder? | 14:29 |
| *** savihou has quit IRC | 14:29 | |
| *** zhongjun_ has quit IRC | 14:29 | |
| smcginnis | thrawn01: And how is this different than defcore? | 14:30 |
| *** logan- has joined #openstack-cinder | 14:30 | |
| *** jungleboyj has joined #openstack-cinder | 14:30 | |
| thrawn01 | smcginnis: I was planning on doing somthing similar to this https://review.openstack.org/#/c/264719/ | 14:31 |
| smcginnis | thrawn01: Hmm, I suppose just file a blueprint for now. | 14:33 |
| smcginnis | thrawn01: I'll have to look into that more to really understand what's going on there. | 14:33 |
| *** takedakn has quit IRC | 14:34 | |
| smcginnis | ildikov: This is all I can find: https://wiki.openstack.org/wiki/Nova/InstanceLevelSnapshots | 14:34 |
| *** Julien-z_ has joined #openstack-cinder | 14:34 | |
| *** e0ne has joined #openstack-cinder | 14:34 | |
| *** e0ne_ has quit IRC | 14:35 | |
| thrawn01 | smcginnis: it's an attempt to make simplify configuration by classifiying the config options by stability | 14:35 |
| ildikov | smcginnis: hmm, thanks | 14:36 |
| ildikov | smcginnis: I think I will keep that block in the spec about blocking that for multi-attach | 14:36 |
| *** martyturner has joined #openstack-cinder | 14:36 | |
| *** Julien-zte has quit IRC | 14:37 | |
| ildikov | smcginnis: not adding or removing that restriction later seems safer for now as that doc did not bring me that much closer to get what's going on :) | 14:37 |
| ildikov | smcginnis: do you have an idea who I could ask for more details on this one? | 14:37 |
| smcginnis | ildikov: It's a nova side thing, so maybe over there? Maybe. | 14:39 |
| *** abhishekk has left #openstack-cinder | 14:39 | |
| ildikov | smcginnis: ok, will try to find someone, tnx :) | 14:40 |
| *** zzzeek has quit IRC | 14:40 | |
| *** david-lyle has quit IRC | 14:42 | |
| *** mvk has quit IRC | 14:42 | |
| *** cebruns has quit IRC | 14:42 | |
| *** Swanson has quit IRC | 14:42 | |
| *** dank_ has quit IRC | 14:42 | |
| *** rmk has quit IRC | 14:42 | |
| *** akshai_ has quit IRC | 14:42 | |
| *** rcernin has quit IRC | 14:47 | |
| *** sdake has joined #openstack-cinder | 14:49 | |
| *** akshai has joined #openstack-cinder | 14:50 | |
| *** yangyapeng has joined #openstack-cinder | 14:52 | |
| *** savihou has joined #openstack-cinder | 14:54 | |
| openstackgerrit | Merged openstack/cinder-specs: Add support for FICON attached ECKD volumes for Linux on System z https://review.openstack.org/302221 | 14:55 |
| *** jsheeren has quit IRC | 14:56 | |
| *** links has quit IRC | 14:57 | |
| *** GB21 has joined #openstack-cinder | 14:57 | |
| *** jsheeren has joined #openstack-cinder | 14:58 | |
| *** adisky has joined #openstack-cinder | 14:58 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: Fix 3PAR cg display name issues https://review.openstack.org/309443 | 14:59 |
| *** rcernin has joined #openstack-cinder | 15:00 | |
| *** julim has joined #openstack-cinder | 15:00 | |
| *** jdurgin1 has joined #openstack-cinder | 15:01 | |
| *** diablo_rojo has quit IRC | 15:03 | |
| *** zzzeek has joined #openstack-cinder | 15:03 | |
| openstackgerrit | Merged openstack/cinder: Fix up RPC cleanup process https://review.openstack.org/316895 | 15:03 |
| *** savihou has quit IRC | 15:03 | |
| *** jistr is now known as jistr|mtg | 15:04 | |
| *** e0ne has quit IRC | 15:06 | |
| *** merooney has quit IRC | 15:06 | |
| *** dustins has quit IRC | 15:06 | |
| *** jungleboyj has quit IRC | 15:07 | |
| *** merooney has joined #openstack-cinder | 15:09 | |
| *** diogogmt has joined #openstack-cinder | 15:10 | |
| *** jsheeren has quit IRC | 15:11 | |
| *** sgotliv_ has quit IRC | 15:12 | |
| *** jungleboyj has joined #openstack-cinder | 15:12 | |
| *** merooney has quit IRC | 15:13 | |
| *** savihou has joined #openstack-cinder | 15:14 | |
| *** Junhongl_ has joined #openstack-cinder | 15:14 | |
| *** e0ne has joined #openstack-cinder | 15:18 | |
| *** Junhongl_ has left #openstack-cinder | 15:20 | |
| openstackgerrit | Aleksey Ruban proposed openstack/cinder: NexentaStor 5 NFS backend driver. https://review.openstack.org/319349 | 15:21 |
| *** wilson has joined #openstack-cinder | 15:26 | |
| *** wilson is now known as Guest79880 | 15:26 | |
| *** mjura has quit IRC | 15:26 | |
| *** Guest79880 has quit IRC | 15:27 | |
| *** wilson_liu has joined #openstack-cinder | 15:27 | |
| *** timcl1 has joined #openstack-cinder | 15:30 | |
| *** timcl has quit IRC | 15:33 | |
| *** salv-orlando has quit IRC | 15:34 | |
| *** cdelatte has quit IRC | 15:35 | |
| *** links has joined #openstack-cinder | 15:36 | |
| *** daneyon has joined #openstack-cinder | 15:36 | |
| *** julim has quit IRC | 15:37 | |
| *** enriquetaso has joined #openstack-cinder | 15:37 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Add VolumeAttachStatus Enum https://review.openstack.org/290885 | 15:37 |
| *** arecknag has quit IRC | 15:39 | |
| *** daneyon_ has quit IRC | 15:39 | |
| *** julim has joined #openstack-cinder | 15:39 | |
| *** jistr|mtg is now known as jistr | 15:40 | |
| openstackgerrit | Stefan Amann proposed openstack/os-brick: WIP: Support FICON-attached ECKD volumes on System z https://review.openstack.org/320536 | 15:40 |
| *** leeantho has joined #openstack-cinder | 15:40 | |
| *** diablo_rojo has joined #openstack-cinder | 15:40 | |
| *** dank_ has joined #openstack-cinder | 15:40 | |
| *** rmk has joined #openstack-cinder | 15:40 | |
| *** diablo_rojo has quit IRC | 15:40 | |
| *** diablo_rojo has joined #openstack-cinder | 15:40 | |
| *** Junhongl_ has joined #openstack-cinder | 15:40 | |
| *** salv-orlando has joined #openstack-cinder | 15:41 | |
| openstackgerrit | Aleksey Ruban proposed openstack/cinder: Nexenta: Add NDB driver for NexentaEdge. https://review.openstack.org/320060 | 15:42 |
| *** Junhongl_ has quit IRC | 15:44 | |
| *** Junhongl_ has joined #openstack-cinder | 15:45 | |
| *** delattec has joined #openstack-cinder | 15:45 | |
| *** david-lyle has joined #openstack-cinder | 15:45 | |
| *** mvk has joined #openstack-cinder | 15:45 | |
| *** cebruns has joined #openstack-cinder | 15:45 | |
| *** Swanson has joined #openstack-cinder | 15:45 | |
| *** wilson_liu has quit IRC | 15:46 | |
| *** Junhongl_ has quit IRC | 15:47 | |
| *** Junhongl has joined #openstack-cinder | 15:48 | |
| *** arch-nemesis has joined #openstack-cinder | 15:50 | |
| *** belmoreira has quit IRC | 15:51 | |
| *** Junhongl has quit IRC | 15:51 | |
| *** jdurgin1 has quit IRC | 15:52 | |
| *** diogogmt has quit IRC | 15:54 | |
| *** lcurtis has joined #openstack-cinder | 15:56 | |
| *** delattec has quit IRC | 15:57 | |
| *** david-lyle has quit IRC | 15:57 | |
| *** mvk has quit IRC | 15:57 | |
| *** cebruns has quit IRC | 15:57 | |
| *** Swanson has quit IRC | 15:57 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Added support for failover_host failback https://review.openstack.org/310064 | 15:58 |
| *** diogogmt has joined #openstack-cinder | 15:59 | |
| hemna | ildikov, mornin | 15:59 |
| *** diablo_rojo has quit IRC | 15:59 | |
| ildikov | hemna: morning :) | 15:59 |
| *** diablo_rojo has joined #openstack-cinder | 16:00 | |
| hemna | so, yesterday I got to the point of getting some odd results in Nova | 16:01 |
| hemna | wrt BFV | 16:01 |
| hemna | ildikov, http://paste.openstack.org/show/498382/ | 16:01 |
| hemna | bdm.get(<KEY>) seems to always fail | 16:02 |
| hemna | bdm.get('volume_id') always returns None | 16:02 |
| hemna | but bdm.volume_id works | 16:02 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: RBD Thin Provisioning stats https://review.openstack.org/178262 | 16:02 |
| hemna | I dumped the BDM and volume_id is there | 16:02 |
| hemna | but bdm.get() doesn't work :( | 16:02 |
| *** jungleboyj has quit IRC | 16:03 | |
| ildikov | wow, I did look that deep into it | 16:03 |
| ildikov | I only started to wonder why we're not using the regular attach for BFV, at least to the case when we're attaching an already existing Cinder volume at boot time | 16:04 |
| *** jungleboyj has joined #openstack-cinder | 16:04 | |
| hemna | so, the other thing is | 16:04 |
| hemna | changing that code causes a failure, because check_attach is called later and the volume is already in attaching | 16:04 |
| hemna | so we'll have to change other places to account for it | 16:04 |
| hemna | but maybe that's not such a bad thing | 16:04 |
| hemna | yet another race in nova->cinder | 16:05 |
| *** delattec has joined #openstack-cinder | 16:06 | |
| *** david-lyle has joined #openstack-cinder | 16:06 | |
| *** mvk has joined #openstack-cinder | 16:06 | |
| *** cebruns has joined #openstack-cinder | 16:06 | |
| *** Swanson has joined #openstack-cinder | 16:06 | |
| ildikov | but the second check_attach should not be there at all | 16:06 |
| ildikov | I would assume at least | 16:06 |
| ildikov | I mean we have an option to use the regular attach flow I would guess, but maybe BFV is too messed up for that already | 16:07 |
| hemna | https://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L255 | 16:07 |
| hemna | that's the call that gets made | 16:07 |
| hemna | during BFV | 16:08 |
| ildikov | if not then we need to figure out when to call reserve and remove all the check_attach calls | 16:08 |
| hemna | have to figure out how to get that flag to be false | 16:08 |
| hemna | it defaults to True | 16:08 |
| hemna | it would be nice to eliminate it all together from that function | 16:08 |
| ildikov | but do we need that check_attach call? | 16:08 |
| hemna | but I could be starting down a rabbit hole | 16:08 |
| hemna | not with my changes | 16:08 |
| ildikov | I mean by using reserve everywhere we should be good with the True case | 16:09 |
| hemna | I just don't know all of the other cases to this attach() call where they expect check_attach to happen | 16:09 |
| hemna | :( | 16:09 |
| ildikov | and we don't need it when it's set to False anyway, which will be the moving cases I guess we talked about yesterday | 16:09 |
| ildikov | cannot we use the functest suit or tempest to identify when that thing is called? | 16:10 |
| *** dustins has joined #openstack-cinder | 16:10 | |
| hemna | I'll continue my hacking I guess | 16:10 |
| hemna | but the bdm.get() thing is....broken | 16:11 |
| ildikov | it's kind of luck that code would do only logging if I'm not mistaken | 16:11 |
| ildikov | but it would deserve a bug report and a fix | 16:12 |
| hemna | https://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L493 | 16:12 |
| hemna | that fails every time | 16:12 |
| hemna | None | 16:12 |
| *** kmartin has quit IRC | 16:12 | |
| hemna | yah it's only a test to do logging | 16:12 |
| hemna | which is...odd | 16:12 |
| hemna | I don't quite get why they made an inner _log_and_attach, instead of just a simple if then block. | 16:13 |
| *** raunak has joined #openstack-cinder | 16:13 | |
| ildikov | that bdm.get() is used elsewhere as well, so it's pretty odd | 16:13 |
| hemna | yah | 16:13 |
| hemna | I noticed that it's used in lots of places | 16:13 |
| *** e0ne has quit IRC | 16:14 | |
| *** Julien-z_ has quit IRC | 16:14 | |
| ildikov | hmm, I will try to look at it as well, but I don;t understand how that gets None there | 16:15 |
| hemna | I guess block_device_mapping could be a list ? | 16:15 |
| hemna | hence the map() | 16:15 |
| *** GB21 has quit IRC | 16:16 | |
| *** nkrinner is now known as nkrinner_afk | 16:16 | |
| *** timcl1 has quit IRC | 16:16 | |
| *** GB21 has joined #openstack-cinder | 16:18 | |
| *** xinli has joined #openstack-cinder | 16:19 | |
| ildikov | hemna: yeah, that might be the case, I still have hard time to follow the Nova code when it comes to BDMs... | 16:20 |
| hemna | ildikov, same here | 16:20 |
| *** savihou has quit IRC | 16:23 | |
| ildikov | can I have a series of dumb questions? :) | 16:24 |
| *** ig0r_ has quit IRC | 16:24 | |
| *** savihou has joined #openstack-cinder | 16:24 | |
| hemna | sure :) | 16:25 |
| *** timcl has joined #openstack-cinder | 16:25 | |
| hemna | I'm talking in the nova channel now about my bdm.get issue | 16:25 |
| ildikov | so in Nova we check volume status whether it's available or not | 16:26 |
| ildikov | but I guess that's not what reserve changes, right? | 16:27 |
| hemna | reserve changes the volume status to 'attaching' if it's 'available' | 16:28 |
| hemna | so yes | 16:28 |
| *** catintheroof has quit IRC | 16:28 | |
| *** savihou has quit IRC | 16:29 | |
| *** timcl1 has joined #openstack-cinder | 16:29 | |
| *** timcl has quit IRC | 16:30 | |
| eharney | smcginnis: i'm curious about https://review.openstack.org/#/c/319965/ -- posted a question there when you have a moment | 16:31 |
| ildikov | hemna: ok, so on your diagram for attach there is a second check_attach call after reserve: https://drive.google.com/file/d/0B1Kp6K43HLHyRkFMZHVHWDJ3amM/view?pref=2&pli=1 | 16:31 |
| hemna | yah it's done a few times :( | 16:32 |
| hemna | and that diagram is just for normal volume attach | 16:32 |
| hemna | not BFV | 16:32 |
| ildikov | hemna: check_attach checks whether the volume is 'available' or not | 16:32 |
| hemna | yup | 16:32 |
| ildikov | how it gets through the second check after reserve? | 16:32 |
| hemna | it shouldn't need to | 16:33 |
| ildikov | or that's another volume status that reserve sets? | 16:33 |
| hemna | nova shouldn't get that far if it didn't call reserve at the start | 16:33 |
| hemna | unless that code is called from some other workflow we don't know about | 16:33 |
| hemna | which is always what worries me in Nova. | 16:33 |
| hemna | reserve changes the status field | 16:34 |
| ildikov | I just would like to understand what Nova checks there as it checks something else and not what reserve modifies | 16:34 |
| ildikov | hmm, we use the status field as well | 16:34 |
| ildikov | I mean in check_attach | 16:34 |
| hemna | it looks at the volume['status'] field | 16:35 |
| hemna | which is what reserve changes to 'attaching' | 16:35 |
| *** sdake_ has joined #openstack-cinder | 16:35 | |
| ildikov | but check_attach explicitly checks for 'available' state if I saw correctly | 16:35 |
| hemna | yes | 16:36 |
| *** IlyaG has joined #openstack-cinder | 16:36 | |
| ildikov | so it's either not called for the second time or I don't understand how that gets through | 16:36 |
| ildikov | what do I miss? :) | 16:36 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Revert "Add hacking check to prevent assert_called_once" https://review.openstack.org/320565 | 16:36 |
| hemna | I don't think it's called | 16:36 |
| *** GB21 has quit IRC | 16:36 | |
| hemna | hence the do_check_attach=True flag | 16:36 |
| ildikov | it's on your diagram | 16:37 |
| hemna | must be passed in as False | 16:37 |
| *** GB21 has joined #openstack-cinder | 16:37 | |
| ildikov | ok, most probably that's what's missing, although I thought that flag is false for the cases, where we have the volume attached for some reason like for evacuate | 16:38 |
| *** sdake has quit IRC | 16:38 | |
| *** IlyaG has quit IRC | 16:38 | |
| ildikov | ok, right do_check_attach is set to fals for the normal case | 16:40 |
| ildikov | *false | 16:40 |
| ildikov | and it's set to true for BFV if I saw correctly, but if we do reserve or try to use the normal attach flow that's not needed either | 16:40 |
| *** delatte has joined #openstack-cinder | 16:43 | |
| *** timcl1 has quit IRC | 16:44 | |
| ildikov | hemna: it's messy... | 16:45 |
| *** mvk has quit IRC | 16:45 | |
| *** delattec has quit IRC | 16:46 | |
| *** wN has quit IRC | 16:48 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Added support for failover_host failback https://review.openstack.org/310064 | 16:48 |
| Swanson | If one does as Err does one becomes as Err is. | 16:48 |
| diablo_rojo | dulek: geguileo Where is the documentation for making a field into a versioned object? Got a link handy? | 16:49 |
| hemna | it's horribly messy | 16:49 |
| hemna | :( | 16:49 |
| *** wN has joined #openstack-cinder | 16:49 | |
| *** wN has quit IRC | 16:49 | |
| *** wN has joined #openstack-cinder | 16:49 | |
| geguileo | diablo_rojo: http://docs.openstack.org/developer/cinder/devref/rolling.upgrades.html | 16:49 |
| geguileo | diablo_rojo: I may be able to help you with it... | 16:50 |
| *** nikeshm has quit IRC | 16:50 | |
| geguileo | diablo_rojo: What are you trying to add? | 16:50 |
| *** harlowja has joined #openstack-cinder | 16:51 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Fix terminate_connection when failed over https://review.openstack.org/320571 | 16:51 |
| *** Apoorva has joined #openstack-cinder | 16:55 | |
| openstackgerrit | Aleksey Ruban proposed openstack/cinder: Replaced nas_ip with nas_host in NexentaStor5 NFS driver. https://review.openstack.org/319349 | 16:57 |
| *** jordanP has quit IRC | 16:58 | |
| ildikov | hemna: :(, I don't know what else to say... | 16:58 |
| *** e0ne has joined #openstack-cinder | 16:59 | |
| ildikov | hemna: although I still think we can go to the direction of adding reserve call to every flow where it's needed, remove check_attach and test and fix issues if we find | 16:59 |
| ildikov | hemna: but maybe I'm too optimistic... :) | 17:00 |
| lpetrut | hemna: Hi, could you please take a look over this os-brick patch? https://review.openstack.org/#/c/312999/ | 17:00 |
| *** kaisers has quit IRC | 17:01 | |
| *** kaisers has joined #openstack-cinder | 17:02 | |
| hemna | lpetrut, yah I will | 17:04 |
| hemna | I've just been busy with nova/cinder stuffs the last few days | 17:04 |
| hemna | I have it on my short list | 17:04 |
| *** darosale has quit IRC | 17:05 | |
| lpetrut | hemna: sure, thanks! | 17:06 |
| *** laughterwym has quit IRC | 17:08 | |
| *** laughterwym has joined #openstack-cinder | 17:09 | |
| *** ducttape_ has quit IRC | 17:09 | |
| *** e0ne has quit IRC | 17:12 | |
| *** avishay has joined #openstack-cinder | 17:13 | |
| *** laughterwym has quit IRC | 17:14 | |
| smcginnis | eharney: How did things pass with that invalid assert in there before? | 17:15 |
| smcginnis | eharney: If not needed, I'd be fine reverting that whole thing. | 17:16 |
| smcginnis | eharney: But the original patch was fixing a valid problem, so not sure how it ended up there in the first place. | 17:16 |
| *** lpetrut has quit IRC | 17:17 | |
| openstackgerrit | Aleksey Ruban proposed openstack/cinder: Nexenta: Add NDB driver for NexentaEdge. https://review.openstack.org/320060 | 17:19 |
| *** merooney has joined #openstack-cinder | 17:23 | |
| *** GB21 has quit IRC | 17:25 | |
| *** Suyi has joined #openstack-cinder | 17:25 | |
| *** ChubYann has joined #openstack-cinder | 17:26 | |
| *** darosale has joined #openstack-cinder | 17:28 | |
| *** ducttape_ has joined #openstack-cinder | 17:29 | |
| *** e0ne has joined #openstack-cinder | 17:30 | |
| *** jwcroppe has joined #openstack-cinder | 17:34 | |
| eharney | smcginnis: that's what i was trying to figure out -- my only guess is that it worked w/ an older version of mock | 17:35 |
| eharney | smcginnis: i'm hoping someone has a counterexample for me | 17:36 |
| smcginnis | eharney: Looks like gate is using 2.0.0: http://logs.openstack.org/04/319804/1/check/gate-cinder-python27-db/dc79161/console.html#_2016-05-23_09_50_01_590 | 17:36 |
| *** jwcroppe_ has quit IRC | 17:37 | |
| smcginnis | Just a little confused how it was there in the first place. | 17:37 |
| eharney | smcginnis: the gate used to not install the newest version from upper-constraints, right? | 17:37 |
| smcginnis | eharney: I think you're right. | 17:37 |
| eharney | smcginnis: requirements only demands 1.2 as a minimum | 17:37 |
| smcginnis | eharney: But if it's now using 2.0, why wasn't every Jenkins run failing with the invalid call in there? | 17:38 |
| eharney | smcginnis: do we know when the invalid call was introduced? | 17:38 |
| eharney | smcginnis: https://review.openstack.org/#/c/308626/ right? that was only merged on may 20... | 17:39 |
| eharney | and i don't see any other places that this hacking check would have caught | 17:40 |
| smcginnis | eharney: Yeah, that should be it. | 17:40 |
| smcginnis | eharney: But even that patch that added it was using 2.0 - http://logs.openstack.org/26/308626/7/check/gate-cinder-python27-db/6061ebc/console.html.gz#_2016-05-20_17_46_47_716 | 17:40 |
| smcginnis | eharney: So I'm just confused then how that didn't fail in the gate. | 17:41 |
| *** timcl has joined #openstack-cinder | 17:41 | |
| smcginnis | Unless 1.3 fails on the invalid call, but 2.0 has a regression where it doesn't... | 17:41 |
| eharney | hmm, good question | 17:42 |
| eharney | smcginnis: mock 2.0.0 added assert_called_once | 17:42 |
| smcginnis | eharney: LOL | 17:42 |
| eharney | so it's not a regression, it's just a totally incorrect hacking check | 17:43 |
| smcginnis | eharney: OK, yep. Let's pull that out of there. :) | 17:43 |
| eharney | https://bugs.python.org/issue26323 | 17:43 |
| smcginnis | eharney: But in the mean time, we probably should request the mock minimum be raised to 2.0 so it's not an issue. | 17:43 |
| eharney | smcginnis: given that this has only occurred once... i'm not sure how interesting the mean time is (i also have no idea how hard it is to request an openstack-wide bump to 2.0) | 17:44 |
| smcginnis | eharney: True. I'll probably request it and see how it goes. | 17:44 |
| smcginnis | eharney: But I don't mean reverting the hacking check should pend on that happening. | 17:44 |
| eharney | smcginnis: this is also kinda amusing: https://review.openstack.org/#/c/200562/ | 17:45 |
| smcginnis | eharney: Just that it probably should be done as well so if someone adds that call, folks with an older mock installed don't fail. | 17:45 |
| smcginnis | eharney: :D | 17:45 |
| eharney | maybe behavior flip-flopped a few times in mock... | 17:46 |
| openstackgerrit | Parashuram Hallur proposed openstack/cinder: Implementation for CoprHD Cinder Drivers. https://review.openstack.org/317643 | 17:46 |
| smcginnis | eharney: Sheesh | 17:46 |
| *** avishay has quit IRC | 17:48 | |
| *** avishay has joined #openstack-cinder | 17:49 | |
| avishay | Hey all. Any idea why I can't access the Cinder git repo? git fetch/review/etc time out on network... | 17:50 |
| smcginnis | avishay: I was just able to clone a fresh repo. | 17:51 |
| smcginnis | avishay: What are you getting? | 17:51 |
| Swanson | avishay, if your company is like ours I would talk to the firewall folk. | 17:51 |
| avishay | Swanson: I'm at home now actually | 17:51 |
| Swanson | avishay, I would ask your mom then. | 17:52 |
| avishay | home router issue...? | 17:52 |
| Swanson | :) | 17:52 |
| avishay | Swanson: :) | 17:52 |
| smcginnis | avishay: I'll smack him for you. | 17:52 |
| *** chris_morrell has joined #openstack-cinder | 17:52 | |
| smcginnis | avishay: What error are you getting? | 17:52 |
| avishay | smcginnis: it's just stuck for a long time, and then timeout | 17:53 |
| *** cknight1 has quit IRC | 17:53 | |
| avishay | probably something stupid on my end, will check... | 17:53 |
| smcginnis | avishay: Odd. Maybe a few seconds delay when I first hit enter on the clone, but then it went quick enough. | 17:53 |
| *** cknight has joined #openstack-cinder | 17:53 | |
| avishay | "Connection timed out" and "Network is unreachable" | 17:54 |
| smcginnis | avishay: Wait, are you pulling from github or the OpenStack git? | 17:54 |
| *** chris_morrell has quit IRC | 17:54 | |
| avishay | git://git.openstack.org/openstack/cinder.git | 17:54 |
| *** rlrossit has quit IRC | 17:54 | |
| Swanson | avishay, try http: | 17:54 |
| smcginnis | avishay: I thought we were told we shouldn't really use those. | 17:54 |
| smcginnis | Or maybe that is just for CIs. | 17:54 |
| avishay | hmm let me try... | 17:54 |
| smcginnis | avishay: Can you try github? https://github.com/openstack/cinder | 17:55 |
| Swanson | I use github. | 17:55 |
| avishay | trying now... | 17:55 |
| openstackgerrit | Gerald McBrearty proposed openstack/cinder: Storwize: terminate_connetion NPIV perf improvement https://review.openstack.org/316247 | 17:56 |
| openstackgerrit | Avishay Traeger proposed openstack/cinder: List manageable volumes and snapshots https://review.openstack.org/285296 | 17:58 |
| openstackgerrit | Avishay Traeger proposed openstack/cinder: Map volume/snapshot manage extensions to v3 https://review.openstack.org/307324 | 17:58 |
| openstackgerrit | Avishay Traeger proposed openstack/cinder: LVM driver: list manageable volumes and snapshots https://review.openstack.org/319660 | 17:58 |
| avishay | Swanson: smcginnis ^^^ bam :) | 17:59 |
| avishay | thanks a lot | 17:59 |
| smcginnis | avishay: Cool, glad you got it worked out. | 17:59 |
| avishay | no need to climb out of the basement and ask mom | 17:59 |
| Swanson | :) | 17:59 |
| *** karthikp_ has joined #openstack-cinder | 18:00 | |
| *** adisky has quit IRC | 18:00 | |
| *** rlrossit has joined #openstack-cinder | 18:01 | |
| *** bardia has joined #openstack-cinder | 18:03 | |
| *** merooney has quit IRC | 18:04 | |
| *** karthikp_ has quit IRC | 18:06 | |
| *** karthikp_ has joined #openstack-cinder | 18:07 | |
| *** karthik__ has joined #openstack-cinder | 18:09 | |
| *** karthikp_ has quit IRC | 18:09 | |
| *** merooney has joined #openstack-cinder | 18:09 | |
| *** sdake_ is now known as sdake | 18:11 | |
| *** david-lyle_ has joined #openstack-cinder | 18:12 | |
| avishay | bswartz: How do I know to use 7mode vs. clustered ONTAP driver? Is it according to the model or is it configurable on the backend? | 18:12 |
| *** Apoorva_ has joined #openstack-cinder | 18:13 | |
| *** timcl has quit IRC | 18:13 | |
| *** raunak_ has joined #openstack-cinder | 18:14 | |
| *** cknight1 has joined #openstack-cinder | 18:14 | |
| *** karthik__ has quit IRC | 18:14 | |
| *** cebruns has quit IRC | 18:15 | |
| *** cebruns has joined #openstack-cinder | 18:16 | |
| *** cknight has quit IRC | 18:16 | |
| *** Apoorva has quit IRC | 18:16 | |
| *** raunak has quit IRC | 18:16 | |
| *** david-lyle has quit IRC | 18:16 | |
| *** raunak_ is now known as raunak | 18:16 | |
| *** salv-orlando has quit IRC | 18:21 | |
| *** ig0r_ has joined #openstack-cinder | 18:23 | |
| *** earlephilhower has joined #openstack-cinder | 18:25 | |
| *** kmartin has joined #openstack-cinder | 18:26 | |
| *** rcernin has quit IRC | 18:29 | |
| bswartz | avishay: sorry I missed your second PM yesterday | 18:30 |
| avishay | bswartz: no worries, hard with the time zones | 18:31 |
| bswartz | avishay: it comes down to the software configuration -- the FAS8000 series can run both 7mode and clustered ONTAP | 18:31 |
| avishay | bswartz: OK cool, thank you! | 18:31 |
| *** karthikp_ has joined #openstack-cinder | 18:32 | |
| *** dustins has quit IRC | 18:33 | |
| *** dustins has joined #openstack-cinder | 18:33 | |
| *** karthikp_ has quit IRC | 18:35 | |
| *** karthikp_ has joined #openstack-cinder | 18:36 | |
| *** avishay has quit IRC | 18:37 | |
| *** karthikp_ has quit IRC | 18:40 | |
| *** karthikp_ has joined #openstack-cinder | 18:41 | |
| *** merooney has quit IRC | 18:44 | |
| *** iceyao has quit IRC | 18:46 | |
| *** karthikp_ has quit IRC | 18:47 | |
| *** links has quit IRC | 18:48 | |
| *** lpetrut has joined #openstack-cinder | 18:48 | |
| adrianofr | Hi guys. Can you take a look on https://review.openstack.org/#/c/285469 , pls? | 18:48 |
| adrianofr | it's been there for some time | 18:49 |
| adrianofr | and it's not getting much attention :/ | 18:49 |
| *** e0ne has quit IRC | 18:53 | |
| scottda | Whew! That review is going to require some afternoon stimulants. | 18:57 |
| *** porrua has quit IRC | 18:57 | |
| *** markvoelker_ has joined #openstack-cinder | 18:57 | |
| adrianofr | scottda: lol | 18:58 |
| adrianofr | scottda: there are lots of new unit tests | 18:58 |
| *** markvoelker has quit IRC | 18:59 | |
| *** merooney has joined #openstack-cinder | 18:59 | |
| *** porrua has joined #openstack-cinder | 18:59 | |
| *** markvoelker has joined #openstack-cinder | 19:00 | |
| *** markvoelker_ has quit IRC | 19:02 | |
| *** merooney has quit IRC | 19:02 | |
| *** martyturner has quit IRC | 19:02 | |
| *** merooney has joined #openstack-cinder | 19:04 | |
| *** martyturner has joined #openstack-cinder | 19:04 | |
| *** karthikp_ has joined #openstack-cinder | 19:05 | |
| *** karthik__ has joined #openstack-cinder | 19:09 | |
| *** karthikp_ has quit IRC | 19:10 | |
| *** BigWillie has quit IRC | 19:10 | |
| *** karthikp_ has joined #openstack-cinder | 19:11 | |
| *** karthik__ has quit IRC | 19:11 | |
| *** karthikp_ has quit IRC | 19:13 | |
| *** sheel has quit IRC | 19:15 | |
| *** mfisch has joined #openstack-cinder | 19:15 | |
| *** mfisch is now known as Guest92937 | 19:16 | |
| *** Guest92937 is now known as mfisch | 19:18 | |
| *** mfisch has quit IRC | 19:18 | |
| *** mfisch has joined #openstack-cinder | 19:18 | |
| *** sdake has quit IRC | 19:21 | |
| *** baumann has quit IRC | 19:21 | |
| *** sdake has joined #openstack-cinder | 19:21 | |
| *** timcl has joined #openstack-cinder | 19:22 | |
| *** baumann has joined #openstack-cinder | 19:22 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Added support for failover_host failback https://review.openstack.org/310064 | 19:24 |
| *** mvk has joined #openstack-cinder | 19:27 | |
| *** kmartin has quit IRC | 19:29 | |
| *** sdake has quit IRC | 19:30 | |
| *** sdake has joined #openstack-cinder | 19:31 | |
| *** diogogmt has quit IRC | 19:32 | |
| *** xyang1 has quit IRC | 19:35 | |
| *** e0ne has joined #openstack-cinder | 19:38 | |
| *** ig0r_ has quit IRC | 19:39 | |
| *** merooney has quit IRC | 19:41 | |
| *** salv-orlando has joined #openstack-cinder | 19:42 | |
| *** Lee1092 has quit IRC | 19:43 | |
| *** merooney has joined #openstack-cinder | 19:47 | |
| *** ociuhandu has quit IRC | 19:48 | |
| *** ducttape_ has quit IRC | 19:49 | |
| *** merooney has quit IRC | 19:52 | |
| *** porrua has quit IRC | 19:58 | |
| *** martyturner has quit IRC | 20:00 | |
| *** martyturner has joined #openstack-cinder | 20:00 | |
| *** leeantho has quit IRC | 20:01 | |
| *** sdake_ has joined #openstack-cinder | 20:02 | |
| *** sgotliv_ has joined #openstack-cinder | 20:04 | |
| openstackgerrit | Merged openstack/cinder: NetApp: Remove type checks from manage workflow https://review.openstack.org/318645 | 20:04 |
| *** Apoorva_ has quit IRC | 20:04 | |
| *** ociuhandu has joined #openstack-cinder | 20:05 | |
| *** sdake has quit IRC | 20:05 | |
| crinkle | could I ask for some more eyes on https://review.openstack.org/#/c/309652/ ? (appreciate the positive feedback so far) | 20:06 |
| *** ducttape_ has joined #openstack-cinder | 20:08 | |
| *** kfarr has joined #openstack-cinder | 20:12 | |
| bardia | Any cores got a few minutes? https://review.openstack.org/#/c/318382/2 :) | 20:14 |
| *** lpetrut has quit IRC | 20:14 | |
| bardia | smcginnis: if I want to get this fix (https://review.openstack.org/#/c/318382/2) into the mitaka branch do I just start a review from that stable/mitaka branch? | 20:15 |
| smcginnis | bardia: You will need to wait for it to merge in master first. | 20:16 |
| bardia | smcginnis: well of course :) | 20:16 |
| smcginnis | Then on that review you can click on the "cherry pick" button and have it cherry pick to stable/mitaka | 20:16 |
| bardia | ah great | 20:16 |
| smcginnis | Doesn't look like there should be any merge conflicts, so hopefully just that. | 20:16 |
| smcginnis | bardia: Otherwise there's a few more steps you need to go through to get it in shape. | 20:17 |
| smcginnis | bardia: But we can cross that bridge if/when we get there. | 20:17 |
| bardia | aye aye captain | 20:17 |
| smcginnis | ;) | 20:18 |
| *** kmartin has joined #openstack-cinder | 20:18 | |
| crinkle | thanks for the reviews smcginnis and hemna :) | 20:20 |
| smcginnis | crinkle: np | 20:20 |
| *** Apoorva has joined #openstack-cinder | 20:23 | |
| *** yangxi has joined #openstack-cinder | 20:34 | |
| *** julim has quit IRC | 20:35 | |
| *** chris_morrell has joined #openstack-cinder | 20:36 | |
| *** chris_morrell has quit IRC | 20:37 | |
| *** jungleboyj has quit IRC | 20:38 | |
| *** yangxi has quit IRC | 20:39 | |
| *** chris_morrell has joined #openstack-cinder | 20:43 | |
| openstackgerrit | Kendall Nelson proposed openstack/os-brick: WIP: Splitting Out Connectors from connector.py :WIP https://review.openstack.org/307974 | 20:44 |
| *** timcl has quit IRC | 20:48 | |
| *** e0ne has quit IRC | 20:49 | |
| *** e0ne has joined #openstack-cinder | 20:50 | |
| *** cknight1 has quit IRC | 20:51 | |
| *** chris_morrell has quit IRC | 20:51 | |
| *** eharney has quit IRC | 20:53 | |
| *** ducttape_ has quit IRC | 20:53 | |
| *** chris_morrell has joined #openstack-cinder | 20:57 | |
| *** dustins has quit IRC | 20:57 | |
| *** rcernin has joined #openstack-cinder | 21:03 | |
| *** dustins has joined #openstack-cinder | 21:03 | |
| *** chris_morrell has quit IRC | 21:05 | |
| *** karthikp_ has joined #openstack-cinder | 21:05 | |
| *** frakt has quit IRC | 21:06 | |
| *** jdurgin has quit IRC | 21:07 | |
| *** jdurgin has joined #openstack-cinder | 21:08 | |
| *** karthikp_ has quit IRC | 21:09 | |
| *** frakt has joined #openstack-cinder | 21:13 | |
| *** jgregor has quit IRC | 21:18 | |
| *** haplo37_ has joined #openstack-cinder | 21:19 | |
| *** jgregor has joined #openstack-cinder | 21:19 | |
| *** kfarr has quit IRC | 21:19 | |
| *** gouthamr has quit IRC | 21:19 | |
| openstackgerrit | Merged openstack/cinder: Mark the v1 API as deprecated https://review.openstack.org/309652 | 21:21 |
| *** ducttape_ has joined #openstack-cinder | 21:22 | |
| *** diablo_rojo has quit IRC | 21:23 | |
| *** rlrossit has quit IRC | 21:23 | |
| *** jgregor has quit IRC | 21:23 | |
| *** haplo37_ has quit IRC | 21:24 | |
| *** e0ne has quit IRC | 21:30 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Added support for failover_host failback https://review.openstack.org/310064 | 21:31 |
| *** ducttape_ has quit IRC | 21:33 | |
| openstackgerrit | Bardia Keyoumarsi proposed openstack/cinder: Coho Data: Reconnect to cluster on broken pipe https://review.openstack.org/318382 | 21:34 |
| *** sdake has joined #openstack-cinder | 21:34 | |
| *** sgotliv_ has quit IRC | 21:35 | |
| *** ametts has quit IRC | 21:35 | |
| *** diogogmt has joined #openstack-cinder | 21:36 | |
| *** baumann has left #openstack-cinder | 21:37 | |
| *** sdake_ has quit IRC | 21:38 | |
| *** rcernin has quit IRC | 21:43 | |
| *** karthikp_ has joined #openstack-cinder | 21:45 | |
| *** martyturner has quit IRC | 21:46 | |
| *** akshai has quit IRC | 21:48 | |
| *** karthikp_ has quit IRC | 21:50 | |
| Swanson | Infra looks somewhat backed up today. | 21:50 |
| *** ducttape_ has joined #openstack-cinder | 21:53 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: LeftHand Fix terminate_connection when failed over https://review.openstack.org/320686 | 21:59 |
| *** dustins has quit IRC | 22:00 | |
| *** sdake has quit IRC | 22:09 | |
| *** gouthamr has joined #openstack-cinder | 22:12 | |
| *** sdake has joined #openstack-cinder | 22:19 | |
| *** darosale has quit IRC | 22:26 | |
| *** gouthamr_ has joined #openstack-cinder | 22:34 | |
| *** jwcroppe_ has joined #openstack-cinder | 22:35 | |
| *** jwcroppe has quit IRC | 22:36 | |
| *** gouthamr has quit IRC | 22:37 | |
| *** ducttape_ has quit IRC | 22:46 | |
| *** xinli has quit IRC | 22:50 | |
| *** edmondsw has quit IRC | 22:50 | |
| *** ducttape_ has joined #openstack-cinder | 22:57 | |
| *** ducttape_ has quit IRC | 22:58 | |
| *** Julien-zte has joined #openstack-cinder | 23:00 | |
| *** openstackgerrit has quit IRC | 23:03 | |
| *** openstackgerrit has joined #openstack-cinder | 23:04 | |
| *** leeantho has joined #openstack-cinder | 23:05 | |
| *** leeantho has quit IRC | 23:05 | |
| *** leeantho has joined #openstack-cinder | 23:06 | |
| *** sdague has quit IRC | 23:09 | |
| *** markvoelker has quit IRC | 23:11 | |
| *** david-lyle_ is now known as david-lyle | 23:12 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Rework initialize_connection and attach https://review.openstack.org/320721 | 23:29 |
| *** leeantho has quit IRC | 23:32 | |
| smcginnis | jgriffith: Woot | 23:34 |
| *** Julien-zte has quit IRC | 23:38 | |
| *** gman-tx has quit IRC | 23:40 | |
| *** lcurtis has quit IRC | 23:41 | |
| *** arch-nemesis has quit IRC | 23:49 | |
| *** yhayashi has joined #openstack-cinder | 23:50 | |
| *** adrianofr has quit IRC | 23:52 | |
| *** sdake has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!