Tuesday, 2016-05-24

*** alexpilotti has quit IRC00:02
*** mtanino has joined #openstack-cinder00:08
*** yangyapeng has quit IRC00:08
*** jwcroppe has joined #openstack-cinder00:09
*** salv-orl_ has quit IRC00:13
*** neophy has joined #openstack-cinder00:16
*** diogogmt has joined #openstack-cinder00:19
*** dank_ has joined #openstack-cinder00:19
*** patrickeast has joined #openstack-cinder00:20
*** ildikov has joined #openstack-cinder00:21
*** laughterwym has joined #openstack-cinder00:26
*** laughter_ has joined #openstack-cinder00:26
*** laughte__ has joined #openstack-cinder00:27
*** laught___ has joined #openstack-cinder00:28
*** arch-nemesis has quit IRC00:29
*** laugh____ has joined #openstack-cinder00:29
*** laughterwym has quit IRC00:30
*** laughter_ has quit IRC00:31
*** laughte__ has quit IRC00:31
*** laught___ has quit IRC00:33
*** Julien-zte has joined #openstack-cinder00:38
*** ducttape_ has joined #openstack-cinder00:38
*** ducttape_ has quit IRC00:38
*** sheel has joined #openstack-cinder00:43
*** rhefner has quit IRC00:53
*** diogogmt has quit IRC00:53
*** rhefner has joined #openstack-cinder00:55
*** suyash has quit IRC00:55
*** diogogmt has joined #openstack-cinder00:55
*** cheneydc has joined #openstack-cinder00:59
*** akerr has joined #openstack-cinder01:01
*** ducttape_ has joined #openstack-cinder01:04
*** akerr_ has joined #openstack-cinder01:04
*** mtanino has quit IRC01:05
*** akerr has quit IRC01:07
*** ducttape_ has quit IRC01:08
*** karthikp_ has quit IRC01:14
*** yangyapeng has joined #openstack-cinder01:15
*** yangyapeng has quit IRC01:15
*** yangyapeng has joined #openstack-cinder01:16
*** alkhodos has quit IRC01:16
*** karthikp_ has joined #openstack-cinder01:16
openstackgerrityuyafei proposed openstack/cinder: Volume manage should get ceiling of volume size  https://review.openstack.org/31976401:17
*** anteaya has quit IRC01:18
*** diogogmt has quit IRC01:18
*** daneyon_ has joined #openstack-cinder01:23
*** EinstCrazy has joined #openstack-cinder01:24
*** daneyon has quit IRC01:28
*** Julien-zte has quit IRC01:28
*** mtanino has joined #openstack-cinder01:30
*** asselin has joined #openstack-cinder01:34
*** Apoorva has quit IRC01:34
*** mtanino has quit IRC01:35
*** jwcroppe has quit IRC01:36
*** cdelatte has quit IRC01:37
*** asselin has quit IRC01:38
*** Lee1092 has joined #openstack-cinder01:38
*** yangxi has joined #openstack-cinder01:40
*** ducttape_ has joined #openstack-cinder01:41
*** karthikp_ has quit IRC01:43
*** ducttape_ has quit IRC01:51
*** karthikp_ has joined #openstack-cinder01:55
openstackgerritwanghao proposed openstack/cinder-specs: transfer snapshots with volumes  https://review.openstack.org/31983101:57
*** jwcroppe has joined #openstack-cinder01:57
*** bardia has quit IRC01:58
*** yangxi has quit IRC01:59
*** karthikp_ has quit IRC01:59
*** EinstCrazy has quit IRC02:00
*** EinstCrazy has joined #openstack-cinder02:00
*** IlyaG has quit IRC02:00
*** yangxi has joined #openstack-cinder02:02
*** neophy has quit IRC02:07
*** neophy has joined #openstack-cinder02:13
*** hoonetorg has quit IRC02:20
*** hoonetorg has joined #openstack-cinder02:20
*** markvoelker has quit IRC02:21
*** ducttape_ has joined #openstack-cinder02:24
*** salv-orlando has joined #openstack-cinder02:38
openstackgerritPatrick East proposed openstack/cinder: Fix image-volume cache to use volume object when evicting  https://review.openstack.org/32018702:38
openstackgerritCao ShuFeng proposed openstack/python-cinderclient: Fix useless api_version of Manager class  https://review.openstack.org/30928302:39
openstackgerritPatrick East proposed openstack/cinder: Fix image-volume cache to use volume object when evicting  https://review.openstack.org/32018702:39
*** salv-orlando has quit IRC02:43
*** ducttape_ has quit IRC02:43
*** enriquetaso has quit IRC02:48
*** akerr_ has quit IRC02:48
*** diogogmt has joined #openstack-cinder02:49
*** karthikp_ has joined #openstack-cinder02:51
*** karthikp_ has quit IRC02:56
*** enriquetaso has joined #openstack-cinder02:59
*** karthikp_ has joined #openstack-cinder03:00
*** karthikp_ has quit IRC03:02
*** raunak has joined #openstack-cinder03:06
*** erlon_ has joined #openstack-cinder03:08
openstackgerritxiexs proposed openstack/python-cinderclient: Make __repr__ print encryption_id for VolumeEncryptionType class  https://review.openstack.org/31993103:10
*** Julien-zte has joined #openstack-cinder03:16
*** gouthamr_ has quit IRC03:17
openstackgerritPatrick East proposed openstack/cinder: Fix retype from non-repl to repl type in Pure drivers  https://review.openstack.org/32020103:19
openstackgerritCao ShuFeng proposed openstack/cinder: Allow API user to update multiattach attr of volume  https://review.openstack.org/30967403:21
*** markvoelker has joined #openstack-cinder03:22
*** markvoelker has quit IRC03:27
*** savihou has joined #openstack-cinder03:27
*** sdake_ has quit IRC03:29
*** yangxi has quit IRC03:37
*** dave-mcc_ has quit IRC03:38
*** _alastor__ has quit IRC03:38
*** karthikp_ has joined #openstack-cinder03:44
*** karthikp_ has quit IRC03:49
*** _alastor__ has joined #openstack-cinder03:55
*** david-lyle_ has joined #openstack-cinder03:55
*** david-lyle has quit IRC03:57
*** Julien-zte has quit IRC04:01
openstackgerritPatrick East proposed openstack/cinder: Fix image-volume cache to use volume object when evicting  https://review.openstack.org/32018704:06
*** links has joined #openstack-cinder04:07
*** salv-orlando has joined #openstack-cinder04:08
*** salv-orlando has quit IRC04:17
*** sdake has joined #openstack-cinder04:17
*** sdake has quit IRC04:18
*** markvoelker has joined #openstack-cinder04:23
*** savihou has quit IRC04:26
*** sgotliv has joined #openstack-cinder04:27
*** markvoelker has quit IRC04:28
*** Poornima has joined #openstack-cinder04:36
*** asselin has joined #openstack-cinder04:46
*** shausy has joined #openstack-cinder04:48
openstackgerritMerged openstack/cinder: Fix AttributeError when obtaining 'name' attribute from 'snap'  https://review.openstack.org/31982704:48
*** pgadiya has joined #openstack-cinder04:53
*** GB21 has joined #openstack-cinder04:58
openstackgerritChangBo Guo(gcb) proposed openstack/cinder: Fix wrong usage of exception's attribute message  https://review.openstack.org/28571205:06
*** enriquetaso has quit IRC05:08
*** ChubYann has quit IRC05:11
*** karthikp_ has joined #openstack-cinder05:12
*** karthikp_ has quit IRC05:13
*** kambiz has quit IRC05:13
*** links has quit IRC05:14
*** ntpttr has quit IRC05:15
*** dansmith has quit IRC05:16
*** kambiz has joined #openstack-cinder05:16
*** ntpttr has joined #openstack-cinder05:16
*** shausy has quit IRC05:17
*** links has joined #openstack-cinder05:18
*** shausy has joined #openstack-cinder05:18
*** dansmith has joined #openstack-cinder05:18
*** dansmith is now known as Guest7113205:18
*** avishay has joined #openstack-cinder05:18
*** links has quit IRC05:24
*** markvoelker has joined #openstack-cinder05:24
*** yangxi has joined #openstack-cinder05:24
*** savihou has joined #openstack-cinder05:28
*** markvoelker has quit IRC05:29
*** iceyao has joined #openstack-cinder05:34
*** chenying has quit IRC05:37
openstackgerritxiexs proposed openstack/python-cinderclient: Fix Capabilities.__repr__ to remove the undefined attribute  https://review.openstack.org/32022305:37
*** chenying has joined #openstack-cinder05:38
*** iceyao has quit IRC05:39
*** iceyao has joined #openstack-cinder05:39
*** GB21 has quit IRC05:44
*** lpetrut has joined #openstack-cinder05:53
*** erlon_ has quit IRC05:56
*** GB21 has joined #openstack-cinder05:56
*** wN has quit IRC05:56
*** Fdaisuke has quit IRC06:00
*** shausy has quit IRC06:00
*** Fdaisuke has joined #openstack-cinder06:03
*** links has joined #openstack-cinder06:04
openstackgerritxiexs proposed openstack/python-cinderclient: Fix Service.__repr__ to remove the undefined attribute  https://review.openstack.org/32023006:07
*** mjura has joined #openstack-cinder06:07
*** rcernin has joined #openstack-cinder06:08
openstackgerrithaobing proposed openstack/cinder: Allow remove description and name for consisgroup  https://review.openstack.org/31206306:09
*** sdake has joined #openstack-cinder06:11
*** mjura has quit IRC06:13
*** mjura has joined #openstack-cinder06:13
*** scottda has quit IRC06:14
*** serverascode has quit IRC06:15
*** DuncanT has quit IRC06:15
*** scottda has joined #openstack-cinder06:16
*** zhenguo_ has quit IRC06:16
*** zhenguo_ has joined #openstack-cinder06:18
*** wN has joined #openstack-cinder06:18
*** wN has quit IRC06:18
*** wN has joined #openstack-cinder06:18
*** DuncanT has joined #openstack-cinder06:20
openstackgerritMerged openstack/cinder: Sheepdog:make full use of all sheepdog nodes  https://review.openstack.org/29622006:21
*** cppforlife_ has quit IRC06:21
*** fmccrthy has quit IRC06:21
*** med_ has quit IRC06:21
*** fmccrthy has joined #openstack-cinder06:22
*** cppforlife_ has joined #openstack-cinder06:23
*** markvoelker has joined #openstack-cinder06:25
*** med_ has joined #openstack-cinder06:25
*** med_ has quit IRC06:25
*** med_ has joined #openstack-cinder06:25
*** sdake has quit IRC06:25
*** harlowja has quit IRC06:28
*** markvoelker has quit IRC06:29
openstackgerritBhagyashri Shewale proposed openstack/cinder: Return BadRequest for invalid 4byte unicode character  https://review.openstack.org/29509706:31
*** serverascode has joined #openstack-cinder06:32
*** cebruns has quit IRC06:33
*** cebruns has joined #openstack-cinder06:35
*** sdake has joined #openstack-cinder06:37
*** salv-orlando has joined #openstack-cinder06:37
*** neophy has quit IRC06:40
*** liverpooler has joined #openstack-cinder06:41
*** earlephilhower has quit IRC06:46
*** chip_ has quit IRC06:49
*** chip_ has joined #openstack-cinder06:50
*** yangxi_ has joined #openstack-cinder06:51
*** sdake has quit IRC06:53
*** yangxi has quit IRC06:54
flip214openstackclient 2.4.0 doesn't have "volume attach" anymore. how can I do that now?06:57
flip214hmmm, perhaps the cinder client works for that use case06:57
flip214or rather nova06:58
*** lpetrut has quit IRC06:58
flip214no, it doesn't.   ERROR (AttributeError): 'Namespace' object has no attribute 'os_user_id'06:58
*** tesseract has joined #openstack-cinder07:04
*** chip_ has quit IRC07:07
*** chip_ has joined #openstack-cinder07:08
*** tesseract has quit IRC07:08
openstackgerritMatan Sabag proposed openstack/cinder: Manage/unmanage snapshot in ScaleIO driver  https://review.openstack.org/31674207:09
openstackgerritChangBo Guo(gcb) proposed openstack/cinder: Fix wrong usage of exception's attribute message  https://review.openstack.org/28571207:09
tobascodulek: you around?07:18
tobascodulek: do you know if this one was backported to stable/liberty https://review.openstack.org/#/c/238866/07:19
tobascoseeing that the code does not exist in the ubuntu repo I'm applying this manually07:19
*** rwsu has joined #openstack-cinder07:19
*** rwsu has left #openstack-cinder07:20
*** lennyb_ has joined #openstack-cinder07:23
*** markvoelker has joined #openstack-cinder07:26
*** lennyb_ has quit IRC07:26
*** lennyb_ has joined #openstack-cinder07:27
*** nkrinner has joined #openstack-cinder07:28
*** lny has joined #openstack-cinder07:30
*** markvoelker has quit IRC07:32
*** lny has quit IRC07:32
*** lennyb_ has quit IRC07:33
*** sgotliv_ has joined #openstack-cinder07:35
*** sgotliv has quit IRC07:37
*** yangxi_ has quit IRC07:38
*** sgotliv__ has joined #openstack-cinder07:40
*** yangxi has joined #openstack-cinder07:42
*** sgotliv_ has quit IRC07:42
*** yangxi_ has joined #openstack-cinder07:43
*** sgotliv_ has joined #openstack-cinder07:43
*** sgotliv__ has quit IRC07:45
*** yangxi has quit IRC07:47
*** savihou has quit IRC07:48
*** savihou has joined #openstack-cinder07:48
*** sgotliv_ has quit IRC07:48
*** lpetrut has joined #openstack-cinder07:48
*** sgotliv has joined #openstack-cinder07:50
*** jordanP has joined #openstack-cinder07:51
*** sgotliv_ has joined #openstack-cinder07:53
*** adisky has joined #openstack-cinder07:55
*** sgotliv has quit IRC07:55
*** arecknag has joined #openstack-cinder07:56
*** Fdaisuke has quit IRC07:57
*** sgotliv_ has quit IRC07:58
*** sgotliv_ has joined #openstack-cinder07:58
*** Fdaisuke has joined #openstack-cinder07:59
*** GB21 has quit IRC07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:00
*** jseiler has quit IRC08:01
*** jseiler has joined #openstack-cinder08:02
*** raunak has quit IRC08:03
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
dulektobasco: It looks like it isn't there. Sorry.08:12
dulekFYI: I've broke a bone in my wirst, so my08:13
dulekmy hand is in a cast. I write extremely slowly now. ;)08:13
lixiaoy1dulek: sorry to hear that. Take care~~08:14
*** belmoreira has joined #openstack-cinder08:15
*** lpetrut has quit IRC08:17
*** GB21 has joined #openstack-cinder08:17
*** e0ne has joined #openstack-cinder08:20
*** salv-orlando has quit IRC08:25
*** afazekas_ is now known as afazekas08:27
*** markvoelker has joined #openstack-cinder08:28
tobascodulek: 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 stuck08:28
openstackgerritWeiJi proposed openstack/cinder: Normalize the options use singele quotes  https://review.openstack.org/32029208:28
*** e0ne has quit IRC08:31
*** e0ne has joined #openstack-cinder08:32
dulektobasco: http://releases.openstack.org/08:32
*** markvoelker has quit IRC08:32
dulektobasco: Liberty is security-supported now, so you probably won't be able to do so.08:33
*** caspinol has joined #openstack-cinder08:33
tobascodulek: aw, even when the stuff isn't working without that 2 line fix?08:33
dulektobasco: Dura lex sed lex. ;) Sorry, that are global OpenStack release guidelines.08:35
tobascodulek: well, you can't have everything here in live, I'll have to live with it until upgrade then08:36
tobascodulek: ty for feedback08:36
*** EinstCrazy has quit IRC08:37
*** EinstCrazy has joined #openstack-cinder08:40
*** ig0r_ has joined #openstack-cinder08:40
*** laugh____ has quit IRC08:42
*** laughterwym has joined #openstack-cinder08:43
*** The_Ball has quit IRC08:47
*** laughterwym has quit IRC08:48
*** EinstCrazy has quit IRC08:51
*** EinstCrazy has joined #openstack-cinder08:57
*** lpetrut has joined #openstack-cinder09:07
*** lpetrut1 has joined #openstack-cinder09:12
*** caspinol has quit IRC09:13
*** lpetrut has quit IRC09:13
*** lpetrut1 is now known as lpetrut09:13
*** jsheeren has joined #openstack-cinder09:15
jsheerenhi 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-cinder09:17
*** sdake has joined #openstack-cinder09:20
*** ociuhandu has quit IRC09:20
*** GB21 has quit IRC09:21
*** yangxi_ has quit IRC09:21
*** yangxi has joined #openstack-cinder09:22
*** yangxi has quit IRC09:24
*** yangxi has joined #openstack-cinder09:27
*** zigo_ is now known as zigo09:28
*** jordanP has quit IRC09:32
*** The_Ball has joined #openstack-cinder09:36
*** yangxi has quit IRC09:37
*** Poornima has quit IRC09:37
*** GB21 has joined #openstack-cinder09:37
*** Poornima has joined #openstack-cinder09:38
*** gcb has quit IRC09:40
*** zhongjun_ has quit IRC09:41
*** gcb has joined #openstack-cinder09:41
*** zhongjun_ has joined #openstack-cinder09:41
openstackgerritPhilipp Marek proposed openstack/cinder: DRBD: ensure_export() might not have 'provider_location'  https://review.openstack.org/32032809:41
*** jordanP has joined #openstack-cinder09:45
*** sgotliv has joined #openstack-cinder09:46
*** sgotliv_ has quit IRC09:47
*** yangxi has joined #openstack-cinder09:48
*** zhongjun_ has quit IRC09:50
*** zhongjun_ has joined #openstack-cinder09:51
*** sgotliv_ has joined #openstack-cinder09:51
*** sgotliv has quit IRC09:52
*** Poornima has quit IRC09:52
*** mvk has quit IRC09:53
*** caspinol has joined #openstack-cinder09:56
*** iceyao_ has joined #openstack-cinder09:59
*** iceyao_ has quit IRC10:00
*** iceyao__ has joined #openstack-cinder10:00
*** iceyao__ has quit IRC10:00
*** iceyao_ has joined #openstack-cinder10:01
*** cheneydc has quit IRC10:01
*** sdake has quit IRC10:01
*** iceyao__ has joined #openstack-cinder10:02
*** iceyao_ has quit IRC10:02
*** iceyao has quit IRC10:02
*** yangxi has quit IRC10:03
*** Poornima has joined #openstack-cinder10:04
*** lprice has joined #openstack-cinder10:06
*** lprice1 has quit IRC10:06
*** avishay has quit IRC10:09
*** e0ne has quit IRC10:13
*** e0ne has joined #openstack-cinder10:14
openstackgerritxiexs proposed openstack/python-cinderclient: Add max-width feature to limit the width of the printed table  https://review.openstack.org/31904410:14
openstackgerritxiexs proposed openstack/python-cinderclient: Add max-width feature to limit the width of the printed table  https://review.openstack.org/31904410:17
*** zhongjun_ has quit IRC10:18
*** zhongjun_ has joined #openstack-cinder10:19
*** mvk has joined #openstack-cinder10:22
*** avishay has joined #openstack-cinder10:24
*** markvoelker has joined #openstack-cinder10:29
*** asselin has quit IRC10:31
*** EinstCrazy has quit IRC10:32
*** EinstCrazy has joined #openstack-cinder10:32
*** yangyapeng has quit IRC10:34
*** laughterwym has joined #openstack-cinder10:34
*** markvoelker has quit IRC10:34
*** EinstCrazy has quit IRC10:37
*** jsheeren has quit IRC10:40
*** laughterwym has quit IRC10:40
*** e0ne_ has joined #openstack-cinder10:41
*** laughterwym has joined #openstack-cinder10:41
*** e0ne has quit IRC10:42
openstackgerritPranali Deore proposed openstack/cinder: Add logging when filtering returns nothing  https://review.openstack.org/30628510:42
*** jsheeren has joined #openstack-cinder10:45
*** laughterwym has quit IRC10:47
*** adisky has quit IRC10:50
*** caspinol has quit IRC10:53
johnthetubaguyildikov: been looking through the spec, sorry I have some questions, but I think its only small tweaks10:59
johnthetubaguyildikov: I am going to get myself some food, but I added the comments on the spec11:00
ildikovjohnthetubaguy: cool, tnx, I'll check11:02
*** laughterwym has joined #openstack-cinder11:02
*** cdelatte has joined #openstack-cinder11:03
*** yangxi has joined #openstack-cinder11:03
*** caspinol has joined #openstack-cinder11:04
*** yangxi has quit IRC11:08
*** avishay has quit IRC11:10
*** salv-orlando has joined #openstack-cinder11:15
*** sdake has joined #openstack-cinder11:17
*** salv-orlando has quit IRC11:19
*** iceyao__ has quit IRC11:23
*** avishay has joined #openstack-cinder11:23
*** sdague has joined #openstack-cinder11:25
*** ociuhandu has joined #openstack-cinder11:28
*** ankit_ag has joined #openstack-cinder11:28
*** markvoelker has joined #openstack-cinder11:30
*** caspinol has quit IRC11:31
*** ig0r_ has quit IRC11:34
*** markvoelker has quit IRC11:35
openstackgerritBertrand Lallau proposed openstack/cinder: Convert to strutils.bool_from_string  https://review.openstack.org/32037311:35
*** julim has joined #openstack-cinder11:45
*** salv-orlando has joined #openstack-cinder11:46
*** yhayashi has quit IRC11:48
*** sdake has quit IRC11:49
*** savihou has quit IRC11:51
*** savihou has joined #openstack-cinder11:51
*** ig0r_ has joined #openstack-cinder11:57
*** ducttape_ has joined #openstack-cinder11:59
openstackgerritPranali Deore proposed openstack/cinder: Add logging when filtering returns nothing  https://review.openstack.org/30628512:00
dulekgeguileo: 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
geguileodulek: That's the patch you are referring, so I guess you mean the refactor sqlalchemy patch, right?12:01
dulekgeguileo: Yes, sorry.12:01
geguileodulek: I think it's not really dependent12:02
geguileodulek: But all that depend on that one do depend on the OVO refactor12:02
*** markvoelker has joined #openstack-cinder12:02
geguileodulek: So both refactors could go on their own, but I'll need both for the job distribution patches I'm working on12:03
dulekgeguileo: Oh…12:03
dulekgeguileo: I'm asking because I see 315541 as a rather dangerous one. And it breaks Python's explicit is better than implicit.12:03
geguileodulek: I don't think it breaks it more than the get_id mechanism12:03
geguileodulek: And we have a lot of code duplication12:04
dulekgeguileo: It seems like too much magic to me. It even adds more lines than it removes.12:04
openstackgerritPranali Deore proposed openstack/cinder: Add logging when filtering returns nothing  https://review.openstack.org/30628512:04
geguileodulek: And for every new OVO we introduce we introduce even more12:04
geguileodulek: Well, those are mostly comments12:04
geguileodulek: Because we almost had none of those12:04
geguileodulek: And when I add the Cluster OVO it removes a lot of code12:05
geguileodulek: Because the Cluster create, save, delete, don't need to be added12:06
dulekYou're probably right on the comments… Okay, I'll take look on whole chain to judge that.12:06
geguileodulek: The chain is broken right now...12:07
dulekSure, but it still should give me a better picture.12:07
geguileodulek: 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 stuff12:07
geguileodulek: Ok12:07
* dulek feels like stutterer when writing using one hand only.12:09
openstackgerritWilson Liu proposed openstack/cinder: Add unit tests for Huawei drivers  https://review.openstack.org/28302912:10
*** ducttape_ has quit IRC12:12
*** Lee1092 has quit IRC12:13
*** avishay has quit IRC12:14
*** GB21 has quit IRC12:20
*** adrianofr has joined #openstack-cinder12:27
*** salv-orlando has quit IRC12:35
*** timcl has joined #openstack-cinder12:36
*** charlesr has joined #openstack-cinder12:37
*** charlesr has quit IRC12:37
*** crose has quit IRC12:38
*** crose has joined #openstack-cinder12:38
smcginnisdulek: What happened with the hand?!12:41
duleksmcginnis: It's in maintenance mode for next few weeks. :D12:44
openstackgerritwanghao proposed openstack/cinder-specs: transfer snapshots with volumes  https://review.openstack.org/31983112:44
dulekI've tripped over and broke small bone in the wrist. It's quite common injury it turns out.12:45
smcginnisdulek: Ouch! I hope it heals quickly.12:48
smcginnisdulek: Probably one of the worse injuries for a software guy. :]12:48
*** catintheroof has joined #openstack-cinder12:49
dulekDoctors say it should be in a cast for ~6 weeks.12:49
dulekWell, at least I've discovered that speech recognition in android works quite well for both Polish and English.12:50
*** asselin has joined #openstack-cinder12:50
*** diogogmt has quit IRC12:50
dulekUnfortunately new Gerrit UI doesn't work well on mobile devices. :P12:51
smcginnisdulek: Hah, that's good.12:51
smcginnisdulek: Yeah, gerrit on any kind of mobile device is completely useless.12:51
*** gouthamr has joined #openstack-cinder12:51
*** timcl has quit IRC12:52
*** diogogmt has joined #openstack-cinder12:52
*** timcl has joined #openstack-cinder12:53
*** akshai has joined #openstack-cinder12:53
*** akshai_ has joined #openstack-cinder12:54
*** sheel has quit IRC12:55
*** jordanP has quit IRC12:55
*** jordanP has joined #openstack-cinder12:57
*** david-lyle has joined #openstack-cinder12:57
*** akshai has quit IRC12:58
*** jistr is now known as jistr|mtg13:01
*** david-lyle_ has quit IRC13:01
*** edmondsw has joined #openstack-cinder13:05
*** rlrossit has joined #openstack-cinder13:07
*** alyson_ has joined #openstack-cinder13:07
*** diogogmt has quit IRC13:07
openstackgerritBertrand Lallau proposed openstack/cinder: Convert to strutils.bool_from_string  https://review.openstack.org/32037313:07
*** sdake has joined #openstack-cinder13:09
*** xyang1 has joined #openstack-cinder13:09
*** salv-orlando has joined #openstack-cinder13:11
*** sdake has quit IRC13:13
*** gfm has joined #openstack-cinder13:16
*** gfm is now known as gfm-r13:16
*** eharney has quit IRC13:17
*** gfm-r has quit IRC13:17
*** porrua has joined #openstack-cinder13:17
*** pgadiya has quit IRC13:21
*** pgadiya has joined #openstack-cinder13:22
*** dave-mccowan has joined #openstack-cinder13:24
*** dustins has joined #openstack-cinder13:24
*** lprice has quit IRC13:25
*** dustins has quit IRC13:29
*** dustins has joined #openstack-cinder13:29
*** abhishekk has joined #openstack-cinder13:29
abhishekkmikal: hi you around13:30
*** eharney has joined #openstack-cinder13:30
*** ildikov has quit IRC13:34
*** ildikov has joined #openstack-cinder13:34
*** Poornima has quit IRC13:36
*** pgadiya has quit IRC13:36
*** BigWillie has joined #openstack-cinder13:38
johnthetubaguyildikov: thanks for your reply on the spec, I think we should include the detail about requiring a cinder microversion, if thats possible13:38
*** gman-tx has joined #openstack-cinder13:39
ildikovjohnthetubaguy: I wanted to clarify here whether we see any other option, personally I don't13:43
johnthetubaguysounds good13:43
ildikovjohnthetubaguy: if that's the case I will add it of course13:43
*** jungleboyj has quit IRC13:43
ildikovjohnthetubaguy: do you think I can remove the volume_snapshot_create case?13:43
*** iceyao has joined #openstack-cinder13:45
*** nikeshm has joined #openstack-cinder13:45
*** baumann has joined #openstack-cinder13:46
ildikovI mean to block it for multiattach volumes13:46
*** ametts has joined #openstack-cinder13:46
*** Lee1092 has joined #openstack-cinder13:48
duleksmcginnis: 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
smcginnisdulek: I believe Brocade does as well.13:49
smcginnisdulek: No harm either way if it is there or not.13:49
smcginnisdulek: Not necessary IMO.13:49
*** merooney has joined #openstack-cinder13:49
duleksmcginnis: Okay, just wanted to clarify why I think that email was sent.13:50
johnthetubaguyildikov: not sure, I think we probably need to keep it13:51
smcginnisdulek: Ah, OK.13:51
johnthetubaguyildikov: I just think we need a little more background on what its doing, I think thats technically a cinder only API13:51
ildikovjohnthetubaguy: if possible I would like to minimise the round trips between Nova and Cinder13:52
*** jbernard_ is now known as jbernard13:52
ildikovjohnthetubaguy: so I'm definitely on the side of getting more background and do what's necessary13:53
johnthetubaguyildikov: 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
ildikovjohnthetubaguy: 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 control13:54
johnthetubaguyyeah, +113:54
ildikovjohnthetubaguy: and also we should keep things on Cinder side that belongs there13:55
ildikovjohnthetubaguy: whenever and wherever w can simplify Nova and clean up the responsibilities we should do that13:55
ildikov*we13:55
johnthetubaguyildikov: +113:56
smcginnisildikov: Definitely +113:56
smcginnisI think the lines of responsibility really got blurred.13:57
smcginnisOr started out that way with cinder coming out of nova.13:57
jgriffithsmcginnis: nah.. back then it was easy :)13:57
smcginnisjgriffith: Heh, no doubt.13:57
johnthetubaguywe didn't do stability or error handling then, life was easy going13:57
ildikovsmcginnis: yeah, I think this didn't get fixed after Cinder came out of Nova13:57
smcginnisThe good old days. It was a simpler time...13:57
*** Julien-zte has joined #openstack-cinder13:58
ildikovthen let's aim for a simpler future after the cleanups :)13:58
* smcginnis cues the Andy Griffin Show music13:58
johnthetubaguylooks like we never fully adopted the reserve_volume call13:58
johnthetubaguybut yeah, heading the right way now, thats the main thing13:58
ildikovjgriffith: BTW, any progress with your patches that you can share? :)13:59
smcginnisOh, Griffith, not Griffin. What a coincidence.13:59
smcginnisOr is it? :)14:00
*** jgregor has joined #openstack-cinder14:00
ildikovjohnthetubaguy: 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 time14:00
jgriffithildikov: yeah, I'm back at work today... I ended up having to take the last several days off :(14:00
*** cknight has joined #openstack-cinder14:00
jgriffithsmcginnis: ?14:00
ildikovjohnthetubaguy: is it only to handle BFV as one big item or there's another reason behind?14:00
jgriffithLOL14:00
jgriffithgot it14:00
smcginnisIgnore me, I'm rambling on nonsense this morning. :)14:00
*** crose has quit IRC14:00
ildikovjgriffith: I hope everything is fine14:00
jgriffithsmcginnis: my nick name used to be "opey"14:00
smcginnisHah!14:01
jgriffithildikov: things are ok, thank you14:01
ildikovjgriffith: 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 to14:01
jgriffithOr I guess it's "Opie"14:01
ildikovjgriffith: or at least I would be happy to chime in :)14:01
jgriffithildikov: I keep reworking things14:01
jgriffithildikov: so what I did last night was try and simplify things under the assumption of no multi-attach and using nova as is today14:02
*** ducttape_ has joined #openstack-cinder14:02
jgriffithildikov: I think have a patch for nova, cindreclient, and another for cinder to implement a new way of attach/detach14:02
ildikovjgriffith: sure, but for instance I can play with unit tests in one patch while you're working on refactoring some items, etc14:02
jgriffithildikov: keeping the cinder code so that an older nova will stil work14:02
jgriffithildikov: 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 desk14:03
ildikovjgriffith: in the meantime we're trying to do some cleanup and rationalisation in Nova, so I hope these will meet nicely14:03
*** darosale has joined #openstack-cinder14:03
jgriffithildikov: ahh... got ya14:03
*** diablo_rojo has joined #openstack-cinder14:04
*** ankit_ag has quit IRC14:04
*** jistr|mtg is now known as jistr14:04
jgriffithawww.. who put a check in devstack for xenial :(14:04
ildikovjgriffith: 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
jgriffithildikov: haha... yeah, they're pretty awful14:05
smcginnisjgriffith: Do they block it now?14:05
jgriffithsmcginnis: yeah, because theyr'e not unit tests at all14:05
smcginnisOh, I meant the xenial comment.14:05
jgriffithsmcginnis: oh sorry :)14:05
smcginnisjgriffith: But yeah, I get you on the "unit" tests.14:06
jgriffithsmcginnis: so they added a check now and it stops devstack from running, but you can set "FORCE" and it'll work14:06
savihousmcginnis: 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
smcginnisDevstack was working fine for me on Xenial.14:06
*** baumann has quit IRC14:06
ildikovjgriffith: 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 things14:07
*** cknight1 has joined #openstack-cinder14:07
jgriffithsmcginnis: me too... been using it since it came out14:07
jgriffithsmcginnis: LVM on restarts is a train wreck, but other than that14:07
jgriffithildikov: nahh, it's part of submitting code14:07
*** ducttape_ has quit IRC14:08
*** cknight has quit IRC14:08
*** dustins has quit IRC14:09
ildikovjgriffith: anyway, just let me know if I can make myself useful and help out with your patches :)14:09
*** dustins has joined #openstack-cinder14:09
ildikovjgriffith: also due to the holiday next Monday in the US and also UK we will have the next week's meeting on Thursday, 1700UTC14:09
*** ducttape_ has joined #openstack-cinder14:10
*** Guest71132 is now known as dansmith14:11
ildikovsmcginnis: 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.rst14:11
*** dansmith is now known as Guest2162114:12
ildikovsmcginnis: 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-attach14:12
*** jwcroppe_ has joined #openstack-cinder14:13
ildikovsmcginnis: jgriffith: from the two items myself I'm certainly not educated on the volume_snapshot one14:13
smcginnisildikov: Yeah, we'll need a microversion bump for the API changes.14:13
ildikovsmcginnis: jgriffith: do you happen to have any background on that one?14:13
* smcginnis looking now...14:13
*** Guest21621 is now known as dansmith14:14
*** diogogmt has joined #openstack-cinder14:14
ildikovsmcginnis: ok, I thought so too, I will add it to the spec as well, I mean the microversion14:14
*** jwcroppe has quit IRC14:14
*** julim has quit IRC14:15
smcginnisildikov: So the snapshot question is around the Nova side with OS assisted snapshot? Am I understanding that right?14:16
*** takedakn has joined #openstack-cinder14:16
ildikovsmcginnis: kind of yes14:16
*** merooney has quit IRC14:16
jgriffithildikov: I think the current proposals for multi-attach are all wrong14:17
ildikovsmcginnis: johnthetubaguy pointed out that this snapshot API is mainly for Cinder to use14:17
smcginnisildikov: So that part I think would be tricky (OS assisted) but on the cinder side we can snap no problem.14:17
jgriffithildikov: I don't think it needs be that hard14:17
jgriffithjust a second... have to pay attention to this part of the meeting :)14:17
*** sheel has joined #openstack-cinder14:18
ildikovjgriffith: well, there aren't too many restrictions there by now I think, but we can discuss14:18
ildikovsmcginnis: in what sense do you mean it's tricky?14:18
smcginnisildikov: If it's not OS assisted then we can just snap the volume, so no big deal there.14:20
smcginnisildikov: But if it is OS assisted...14:20
smcginnisildikov: And we have multiple attachements...14:21
smcginnisildikov: Then it seems we would need some kind of coordination across the multiple attached OS's for that.14:21
ildikovsmcginnis: in what cases OS assisted is used?14:21
smcginnisildikov: Just not a 1:1 relationship anymore.14:21
smcginnisildikov: I don't have a lot of experience there, so maybe someone else can chime in.14:21
smcginnisildikov: 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-cinder14:22
smcginnisildikov: Without OS involvement there is no guarantee that all IO has been written to disk at the time it is snapped.14:23
ildikovsmcginnis: is that used for "local" volumes as pointed out in the comments?14:23
*** diogogmt has quit IRC14:23
ildikovsmcginnis: or it can be applicable for all types?14:24
openstackgerritMerged openstack/cinder: Add hacking check to prevent assert_called_once  https://review.openstack.org/31996514:26
*** merooney has joined #openstack-cinder14:26
*** lprice1 has joined #openstack-cinder14:28
*** rhagarty has joined #openstack-cinder14:28
thrawn01hello 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
smcginnisildikov: Sorry, looking...14:28
*** baumann has joined #openstack-cinder14:28
*** logan- has quit IRC14:29
*** lprice has quit IRC14:29
smcginnisthrawn01: What does that require in Cinder?14:29
*** savihou has quit IRC14:29
*** zhongjun_ has quit IRC14:29
smcginnisthrawn01: And how is this different than defcore?14:30
*** logan- has joined #openstack-cinder14:30
*** jungleboyj has joined #openstack-cinder14:30
thrawn01smcginnis: I was planning on doing somthing similar to this https://review.openstack.org/#/c/264719/14:31
smcginnisthrawn01: Hmm, I suppose just file a blueprint for now.14:33
smcginnisthrawn01: I'll have to look into that more to really understand what's going on there.14:33
*** takedakn has quit IRC14:34
smcginnisildikov: This is all I can find: https://wiki.openstack.org/wiki/Nova/InstanceLevelSnapshots14:34
*** Julien-z_ has joined #openstack-cinder14:34
*** e0ne has joined #openstack-cinder14:34
*** e0ne_ has quit IRC14:35
thrawn01smcginnis: it's an attempt to make simplify configuration by classifiying the config options by stability14:35
ildikovsmcginnis: hmm, thanks14:36
ildikovsmcginnis: I think I will keep that block in the spec about blocking that for multi-attach14:36
*** martyturner has joined #openstack-cinder14:36
*** Julien-zte has quit IRC14:37
ildikovsmcginnis: 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
ildikovsmcginnis: do you have an idea who I could ask for more details on this one?14:37
smcginnisildikov: It's a nova side thing, so maybe over there? Maybe.14:39
*** abhishekk has left #openstack-cinder14:39
ildikovsmcginnis: ok, will try to find someone, tnx :)14:40
*** zzzeek has quit IRC14:40
*** david-lyle has quit IRC14:42
*** mvk has quit IRC14:42
*** cebruns has quit IRC14:42
*** Swanson has quit IRC14:42
*** dank_ has quit IRC14:42
*** rmk has quit IRC14:42
*** akshai_ has quit IRC14:42
*** rcernin has quit IRC14:47
*** sdake has joined #openstack-cinder14:49
*** akshai has joined #openstack-cinder14:50
*** yangyapeng has joined #openstack-cinder14:52
*** savihou has joined #openstack-cinder14:54
openstackgerritMerged openstack/cinder-specs: Add support for FICON attached ECKD volumes for Linux on System z  https://review.openstack.org/30222114:55
*** jsheeren has quit IRC14:56
*** links has quit IRC14:57
*** GB21 has joined #openstack-cinder14:57
*** jsheeren has joined #openstack-cinder14:58
*** adisky has joined #openstack-cinder14:58
openstackgerritAlex O'Rourke proposed openstack/cinder: Fix 3PAR cg display name issues  https://review.openstack.org/30944314:59
*** rcernin has joined #openstack-cinder15:00
*** julim has joined #openstack-cinder15:00
*** jdurgin1 has joined #openstack-cinder15:01
*** diablo_rojo has quit IRC15:03
*** zzzeek has joined #openstack-cinder15:03
openstackgerritMerged openstack/cinder: Fix up RPC cleanup process  https://review.openstack.org/31689515:03
*** savihou has quit IRC15:03
*** jistr is now known as jistr|mtg15:04
*** e0ne has quit IRC15:06
*** merooney has quit IRC15:06
*** dustins has quit IRC15:06
*** jungleboyj has quit IRC15:07
*** merooney has joined #openstack-cinder15:09
*** diogogmt has joined #openstack-cinder15:10
*** jsheeren has quit IRC15:11
*** sgotliv_ has quit IRC15:12
*** jungleboyj has joined #openstack-cinder15:12
*** merooney has quit IRC15:13
*** savihou has joined #openstack-cinder15:14
*** Junhongl_ has joined #openstack-cinder15:14
*** e0ne has joined #openstack-cinder15:18
*** Junhongl_ has left #openstack-cinder15:20
openstackgerritAleksey Ruban proposed openstack/cinder: NexentaStor 5 NFS backend driver.  https://review.openstack.org/31934915:21
*** wilson has joined #openstack-cinder15:26
*** wilson is now known as Guest7988015:26
*** mjura has quit IRC15:26
*** Guest79880 has quit IRC15:27
*** wilson_liu has joined #openstack-cinder15:27
*** timcl1 has joined #openstack-cinder15:30
*** timcl has quit IRC15:33
*** salv-orlando has quit IRC15:34
*** cdelatte has quit IRC15:35
*** links has joined #openstack-cinder15:36
*** daneyon has joined #openstack-cinder15:36
*** julim has quit IRC15:37
*** enriquetaso has joined #openstack-cinder15:37
openstackgerritKendall Nelson proposed openstack/cinder: Add VolumeAttachStatus Enum  https://review.openstack.org/29088515:37
*** arecknag has quit IRC15:39
*** daneyon_ has quit IRC15:39
*** julim has joined #openstack-cinder15:39
*** jistr|mtg is now known as jistr15:40
openstackgerritStefan Amann proposed openstack/os-brick: WIP: Support FICON-attached ECKD volumes on System z  https://review.openstack.org/32053615:40
*** leeantho has joined #openstack-cinder15:40
*** diablo_rojo has joined #openstack-cinder15:40
*** dank_ has joined #openstack-cinder15:40
*** rmk has joined #openstack-cinder15:40
*** diablo_rojo has quit IRC15:40
*** diablo_rojo has joined #openstack-cinder15:40
*** Junhongl_ has joined #openstack-cinder15:40
*** salv-orlando has joined #openstack-cinder15:41
openstackgerritAleksey Ruban proposed openstack/cinder: Nexenta: Add NDB driver for NexentaEdge.  https://review.openstack.org/32006015:42
*** Junhongl_ has quit IRC15:44
*** Junhongl_ has joined #openstack-cinder15:45
*** delattec has joined #openstack-cinder15:45
*** david-lyle has joined #openstack-cinder15:45
*** mvk has joined #openstack-cinder15:45
*** cebruns has joined #openstack-cinder15:45
*** Swanson has joined #openstack-cinder15:45
*** wilson_liu has quit IRC15:46
*** Junhongl_ has quit IRC15:47
*** Junhongl has joined #openstack-cinder15:48
*** arch-nemesis has joined #openstack-cinder15:50
*** belmoreira has quit IRC15:51
*** Junhongl has quit IRC15:51
*** jdurgin1 has quit IRC15:52
*** diogogmt has quit IRC15:54
*** lcurtis has joined #openstack-cinder15:56
*** delattec has quit IRC15:57
*** david-lyle has quit IRC15:57
*** mvk has quit IRC15:57
*** cebruns has quit IRC15:57
*** Swanson has quit IRC15:57
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Added support for failover_host failback  https://review.openstack.org/31006415:58
*** diogogmt has joined #openstack-cinder15:59
hemnaildikov, mornin15:59
*** diablo_rojo has quit IRC15:59
ildikovhemna: morning :)15:59
*** diablo_rojo has joined #openstack-cinder16:00
hemnaso,  yesterday I got to the point of getting some odd results in Nova16:01
hemnawrt BFV16:01
hemnaildikov, http://paste.openstack.org/show/498382/16:01
hemnabdm.get(<KEY>) seems to always fail16:02
hemnabdm.get('volume_id') always returns None16:02
hemnabut bdm.volume_id works16:02
openstackgerritIvan Kolodyazhny proposed openstack/cinder: RBD Thin Provisioning stats  https://review.openstack.org/17826216:02
hemnaI dumped the BDM and volume_id is there16:02
hemnabut bdm.get() doesn't work :(16:02
*** jungleboyj has quit IRC16:03
ildikovwow, I did look that deep into it16:03
ildikovI 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 time16:04
*** jungleboyj has joined #openstack-cinder16:04
hemnaso, the other thing is16:04
hemnachanging that code causes a failure, because check_attach is called later and the volume is already in attaching16:04
hemnaso we'll have to change other places to account for it16:04
hemnabut maybe that's not such a bad thing16:04
hemnayet another race in nova->cinder16:05
*** delattec has joined #openstack-cinder16:06
*** david-lyle has joined #openstack-cinder16:06
*** mvk has joined #openstack-cinder16:06
*** cebruns has joined #openstack-cinder16:06
*** Swanson has joined #openstack-cinder16:06
ildikovbut the second check_attach should not be there at all16:06
ildikovI would assume at least16:06
ildikovI mean we have an option to use the regular attach flow I would guess, but maybe BFV is too messed up for that already16:07
hemnahttps://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L25516:07
hemnathat's the call that gets made16:07
hemnaduring BFV16:08
ildikovif not then we need to figure out when to call reserve and remove all the check_attach calls16:08
hemnahave to figure out how to get that flag to be false16:08
hemnait defaults to True16:08
hemnait would be nice to eliminate it all together from that function16:08
ildikovbut do we need that check_attach call?16:08
hemnabut I could be starting down a rabbit hole16:08
hemnanot with my changes16:08
ildikovI mean by using reserve everywhere we should be good with the True case16:09
hemnaI just don't know all of the other cases to this attach() call where they expect check_attach to happen16:09
hemna:(16:09
ildikovand we don't need it when it's set to False anyway, which will be the moving cases I guess we talked about yesterday16:09
ildikovcannot we use the functest suit or tempest to identify when that thing is called?16:10
*** dustins has joined #openstack-cinder16:10
hemnaI'll continue my hacking I guess16:10
hemnabut the bdm.get() thing is....broken16:11
ildikovit's kind of luck that code would do only logging if I'm not mistaken16:11
ildikovbut it would deserve a bug report and a fix16:12
hemnahttps://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L49316:12
hemnathat fails every time16:12
hemnaNone16:12
*** kmartin has quit IRC16:12
hemnayah it's only a test to do logging16:12
hemnawhich is...odd16:12
hemnaI 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-cinder16:13
ildikovthat bdm.get() is used elsewhere as well, so it's pretty odd16:13
hemnayah16:13
hemnaI noticed that it's  used in lots of places16:13
*** e0ne has quit IRC16:14
*** Julien-z_ has quit IRC16:14
ildikovhmm, I will try to look at it as well, but I don;t understand how that gets None there16:15
hemnaI guess block_device_mapping could be a list ?16:15
hemnahence the map()16:15
*** GB21 has quit IRC16:16
*** nkrinner is now known as nkrinner_afk16:16
*** timcl1 has quit IRC16:16
*** GB21 has joined #openstack-cinder16:18
*** xinli has joined #openstack-cinder16:19
ildikovhemna: yeah, that might be the case, I still have hard time to follow the Nova code when it comes to BDMs...16:20
hemnaildikov, same here16:20
*** savihou has quit IRC16:23
ildikovcan I have a series of dumb questions? :)16:24
*** ig0r_ has quit IRC16:24
*** savihou has joined #openstack-cinder16:24
hemnasure :)16:25
*** timcl has joined #openstack-cinder16:25
hemnaI'm talking in the nova channel now about my bdm.get issue16:25
ildikovso in Nova we check volume status whether it's available or not16:26
ildikovbut I guess that's not what reserve changes, right?16:27
hemnareserve changes the volume status to 'attaching' if it's 'available'16:28
hemnaso yes16:28
*** catintheroof has quit IRC16:28
*** savihou has quit IRC16:29
*** timcl1 has joined #openstack-cinder16:29
*** timcl has quit IRC16:30
eharneysmcginnis: i'm curious about https://review.openstack.org/#/c/319965/  -- posted a question there when you have a moment16:31
ildikovhemna: 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=116:31
hemnayah it's done a few times :(16:32
hemnaand that diagram is just for normal volume attach16:32
hemnanot BFV16:32
ildikovhemna: check_attach checks whether the volume is 'available' or not16:32
hemnayup16:32
ildikovhow it gets through the second check after reserve?16:32
hemnait shouldn't need to16:33
ildikovor that's another volume status that reserve sets?16:33
hemnanova shouldn't get that far if it didn't call reserve at the start16:33
hemnaunless that code is called from some other workflow we don't know about16:33
hemnawhich is always what worries me in Nova.16:33
hemnareserve changes the status field16:34
ildikovI just would like to understand what Nova checks there as it checks something else and not what reserve modifies16:34
ildikovhmm, we use the status field as well16:34
ildikovI mean in check_attach16:34
hemnait looks at the volume['status'] field16:35
hemnawhich is what reserve changes to 'attaching'16:35
*** sdake_ has joined #openstack-cinder16:35
ildikovbut check_attach explicitly checks for 'available' state if I saw correctly16:35
hemnayes16:36
*** IlyaG has joined #openstack-cinder16:36
ildikovso it's either not called for the second time or I don't understand how that gets through16:36
ildikovwhat do I miss? :)16:36
openstackgerritEric Harney proposed openstack/cinder: Revert "Add hacking check to prevent assert_called_once"  https://review.openstack.org/32056516:36
hemnaI don't think it's called16:36
*** GB21 has quit IRC16:36
hemnahence the do_check_attach=True flag16:36
ildikovit's on your diagram16:37
hemnamust be passed in as False16:37
*** GB21 has joined #openstack-cinder16:37
ildikovok, 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 evacuate16:38
*** sdake has quit IRC16:38
*** IlyaG has quit IRC16:38
ildikovok, right do_check_attach is set to fals for the normal case16:40
ildikov*false16:40
ildikovand 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 either16:40
*** delatte has joined #openstack-cinder16:43
*** timcl1 has quit IRC16:44
ildikovhemna: it's messy...16:45
*** mvk has quit IRC16:45
*** delattec has quit IRC16:46
*** wN has quit IRC16:48
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Added support for failover_host failback  https://review.openstack.org/31006416:48
SwansonIf one does as Err does one becomes as Err is.16:48
diablo_rojodulek: geguileo Where is the documentation for making a field into a versioned object? Got a link handy?16:49
hemnait's horribly messy16:49
hemna:(16:49
*** wN has joined #openstack-cinder16:49
*** wN has quit IRC16:49
*** wN has joined #openstack-cinder16:49
geguileodiablo_rojo: http://docs.openstack.org/developer/cinder/devref/rolling.upgrades.html16:49
geguileodiablo_rojo: I may be able to help you with it...16:50
*** nikeshm has quit IRC16:50
geguileodiablo_rojo: What are you trying to add?16:50
*** harlowja has joined #openstack-cinder16:51
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Fix terminate_connection when failed over  https://review.openstack.org/32057116:51
*** Apoorva has joined #openstack-cinder16:55
openstackgerritAleksey Ruban proposed openstack/cinder: Replaced nas_ip with nas_host in NexentaStor5 NFS driver.  https://review.openstack.org/31934916:57
*** jordanP has quit IRC16:58
ildikovhemna: :(, I don't know what else to say...16:58
*** e0ne has joined #openstack-cinder16:59
ildikovhemna: 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 find16:59
ildikovhemna: but maybe I'm too optimistic... :)17:00
lpetruthemna: Hi, could you please take a look over this os-brick patch? https://review.openstack.org/#/c/312999/17:00
*** kaisers has quit IRC17:01
*** kaisers has joined #openstack-cinder17:02
hemnalpetrut, yah I will17:04
hemnaI've just been busy with nova/cinder stuffs the last few days17:04
hemnaI have it on my short list17:04
*** darosale has quit IRC17:05
lpetruthemna: sure, thanks!17:06
*** laughterwym has quit IRC17:08
*** laughterwym has joined #openstack-cinder17:09
*** ducttape_ has quit IRC17:09
*** e0ne has quit IRC17:12
*** avishay has joined #openstack-cinder17:13
*** laughterwym has quit IRC17:14
smcginniseharney: How did things pass with that invalid assert in there before?17:15
smcginniseharney: If not needed, I'd be fine reverting that whole thing.17:16
smcginniseharney: 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 IRC17:17
openstackgerritAleksey Ruban proposed openstack/cinder: Nexenta: Add NDB driver for NexentaEdge.  https://review.openstack.org/32006017:19
*** merooney has joined #openstack-cinder17:23
*** GB21 has quit IRC17:25
*** Suyi has joined #openstack-cinder17:25
*** ChubYann has joined #openstack-cinder17:26
*** darosale has joined #openstack-cinder17:28
*** ducttape_ has joined #openstack-cinder17:29
*** e0ne has joined #openstack-cinder17:30
*** jwcroppe has joined #openstack-cinder17:34
eharneysmcginnis: that's what i was trying to figure out -- my only guess is that it worked w/ an older version of mock17:35
eharneysmcginnis: i'm hoping someone has a counterexample for me17:36
smcginniseharney: 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_59017:36
*** jwcroppe_ has quit IRC17:37
smcginnisJust a little confused how it was there in the first place.17:37
eharneysmcginnis: the gate used to not install the newest version from upper-constraints, right?17:37
smcginniseharney: I think you're right.17:37
eharneysmcginnis: requirements only demands 1.2 as a minimum17:37
smcginniseharney: But if it's now using 2.0, why wasn't every Jenkins run failing with the invalid call in there?17:38
eharneysmcginnis: do we know when the invalid call was introduced?17:38
eharneysmcginnis: https://review.openstack.org/#/c/308626/  right?  that was only merged on may 20...17:39
eharneyand i don't see any other places that this hacking check would have caught17:40
smcginniseharney: Yeah, that should be it.17:40
smcginniseharney: 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_71617:40
smcginniseharney: So I'm just confused then how that didn't fail in the gate.17:41
*** timcl has joined #openstack-cinder17:41
smcginnisUnless 1.3 fails on the invalid call, but 2.0 has a regression where it doesn't...17:41
eharneyhmm, good question17:42
eharneysmcginnis: mock 2.0.0 added assert_called_once17:42
smcginniseharney: LOL17:42
eharneyso it's not a regression, it's just a totally incorrect hacking check17:43
smcginniseharney: OK, yep. Let's pull that out of there. :)17:43
eharneyhttps://bugs.python.org/issue2632317:43
smcginniseharney: 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
eharneysmcginnis: 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
smcginniseharney: True. I'll probably request it and see how it goes.17:44
smcginniseharney: But I don't mean reverting the hacking check should pend on that happening.17:44
eharneysmcginnis: this is also kinda amusing: https://review.openstack.org/#/c/200562/17:45
smcginniseharney: 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
smcginniseharney: :D17:45
eharneymaybe behavior flip-flopped a few times in mock...17:46
openstackgerritParashuram Hallur proposed openstack/cinder: Implementation for CoprHD Cinder Drivers.  https://review.openstack.org/31764317:46
smcginniseharney: Sheesh17:46
*** avishay has quit IRC17:48
*** avishay has joined #openstack-cinder17:49
avishayHey all.  Any idea why I can't access the Cinder git repo?  git fetch/review/etc time out on network...17:50
smcginnisavishay: I was just able to clone a fresh repo.17:51
smcginnisavishay: What are you getting?17:51
Swansonavishay, if your company is like ours I would talk to the firewall folk.17:51
avishaySwanson: I'm at home now actually17:51
Swansonavishay, I would ask your mom then.17:52
avishayhome router issue...?17:52
Swanson:)17:52
avishaySwanson: :)17:52
smcginnisavishay: I'll smack him for you.17:52
*** chris_morrell has joined #openstack-cinder17:52
smcginnisavishay: What error are you getting?17:52
avishaysmcginnis: it's just stuck for a long time, and then timeout17:53
*** cknight1 has quit IRC17:53
avishayprobably something stupid on my end, will check...17:53
smcginnisavishay: 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-cinder17:53
avishay"Connection timed out" and "Network is unreachable"17:54
smcginnisavishay: Wait, are you pulling from github or the OpenStack git?17:54
*** chris_morrell has quit IRC17:54
avishaygit://git.openstack.org/openstack/cinder.git17:54
*** rlrossit has quit IRC17:54
Swansonavishay, try http:17:54
smcginnisavishay: I thought we were told we shouldn't really use those.17:54
smcginnisOr maybe that is just for CIs.17:54
avishayhmm let me try...17:54
smcginnisavishay: Can you try github? https://github.com/openstack/cinder17:55
SwansonI use github.17:55
avishaytrying now...17:55
openstackgerritGerald McBrearty proposed openstack/cinder: Storwize: terminate_connetion NPIV perf improvement  https://review.openstack.org/31624717:56
openstackgerritAvishay Traeger proposed openstack/cinder: List manageable volumes and snapshots  https://review.openstack.org/28529617:58
openstackgerritAvishay Traeger proposed openstack/cinder: Map volume/snapshot manage extensions to v3  https://review.openstack.org/30732417:58
openstackgerritAvishay Traeger proposed openstack/cinder: LVM driver: list manageable volumes and snapshots  https://review.openstack.org/31966017:58
avishaySwanson: smcginnis ^^^ bam :)17:59
avishaythanks a lot17:59
smcginnisavishay: Cool, glad you got it worked out.17:59
avishayno need to climb out of the basement and ask mom17:59
Swanson:)17:59
*** karthikp_ has joined #openstack-cinder18:00
*** adisky has quit IRC18:00
*** rlrossit has joined #openstack-cinder18:01
*** bardia has joined #openstack-cinder18:03
*** merooney has quit IRC18:04
*** karthikp_ has quit IRC18:06
*** karthikp_ has joined #openstack-cinder18:07
*** karthik__ has joined #openstack-cinder18:09
*** karthikp_ has quit IRC18:09
*** merooney has joined #openstack-cinder18:09
*** sdake_ is now known as sdake18:11
*** david-lyle_ has joined #openstack-cinder18:12
avishaybswartz: 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-cinder18:13
*** timcl has quit IRC18:13
*** raunak_ has joined #openstack-cinder18:14
*** cknight1 has joined #openstack-cinder18:14
*** karthik__ has quit IRC18:14
*** cebruns has quit IRC18:15
*** cebruns has joined #openstack-cinder18:16
*** cknight has quit IRC18:16
*** Apoorva has quit IRC18:16
*** raunak has quit IRC18:16
*** david-lyle has quit IRC18:16
*** raunak_ is now known as raunak18:16
*** salv-orlando has quit IRC18:21
*** ig0r_ has joined #openstack-cinder18:23
*** earlephilhower has joined #openstack-cinder18:25
*** kmartin has joined #openstack-cinder18:26
*** rcernin has quit IRC18:29
bswartzavishay: sorry I missed your second PM yesterday18:30
avishaybswartz: no worries, hard with the time zones18:31
bswartzavishay: it comes down to the software configuration -- the FAS8000 series can run both 7mode and clustered ONTAP18:31
avishaybswartz: OK cool, thank you!18:31
*** karthikp_ has joined #openstack-cinder18:32
*** dustins has quit IRC18:33
*** dustins has joined #openstack-cinder18:33
*** karthikp_ has quit IRC18:35
*** karthikp_ has joined #openstack-cinder18:36
*** avishay has quit IRC18:37
*** karthikp_ has quit IRC18:40
*** karthikp_ has joined #openstack-cinder18:41
*** merooney has quit IRC18:44
*** iceyao has quit IRC18:46
*** karthikp_ has quit IRC18:47
*** links has quit IRC18:48
*** lpetrut has joined #openstack-cinder18:48
adrianofrHi guys. Can you take a look on https://review.openstack.org/#/c/285469 , pls?18:48
adrianofrit's been there for some time18:49
adrianofrand it's not getting much attention :/18:49
*** e0ne has quit IRC18:53
scottdaWhew! That review is going to require some afternoon stimulants.18:57
*** porrua has quit IRC18:57
*** markvoelker_ has joined #openstack-cinder18:57
adrianofrscottda: lol18:58
adrianofrscottda: there are lots of new unit tests18:58
*** markvoelker has quit IRC18:59
*** merooney has joined #openstack-cinder18:59
*** porrua has joined #openstack-cinder18:59
*** markvoelker has joined #openstack-cinder19:00
*** markvoelker_ has quit IRC19:02
*** merooney has quit IRC19:02
*** martyturner has quit IRC19:02
*** merooney has joined #openstack-cinder19:04
*** martyturner has joined #openstack-cinder19:04
*** karthikp_ has joined #openstack-cinder19:05
*** karthik__ has joined #openstack-cinder19:09
*** karthikp_ has quit IRC19:10
*** BigWillie has quit IRC19:10
*** karthikp_ has joined #openstack-cinder19:11
*** karthik__ has quit IRC19:11
*** karthikp_ has quit IRC19:13
*** sheel has quit IRC19:15
*** mfisch has joined #openstack-cinder19:15
*** mfisch is now known as Guest9293719:16
*** Guest92937 is now known as mfisch19:18
*** mfisch has quit IRC19:18
*** mfisch has joined #openstack-cinder19:18
*** sdake has quit IRC19:21
*** baumann has quit IRC19:21
*** sdake has joined #openstack-cinder19:21
*** timcl has joined #openstack-cinder19:22
*** baumann has joined #openstack-cinder19:22
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Added support for failover_host failback  https://review.openstack.org/31006419:24
*** mvk has joined #openstack-cinder19:27
*** kmartin has quit IRC19:29
*** sdake has quit IRC19:30
*** sdake has joined #openstack-cinder19:31
*** diogogmt has quit IRC19:32
*** xyang1 has quit IRC19:35
*** e0ne has joined #openstack-cinder19:38
*** ig0r_ has quit IRC19:39
*** merooney has quit IRC19:41
*** salv-orlando has joined #openstack-cinder19:42
*** Lee1092 has quit IRC19:43
*** merooney has joined #openstack-cinder19:47
*** ociuhandu has quit IRC19:48
*** ducttape_ has quit IRC19:49
*** merooney has quit IRC19:52
*** porrua has quit IRC19:58
*** martyturner has quit IRC20:00
*** martyturner has joined #openstack-cinder20:00
*** leeantho has quit IRC20:01
*** sdake_ has joined #openstack-cinder20:02
*** sgotliv_ has joined #openstack-cinder20:04
openstackgerritMerged openstack/cinder: NetApp: Remove type checks from manage workflow  https://review.openstack.org/31864520:04
*** Apoorva_ has quit IRC20:04
*** ociuhandu has joined #openstack-cinder20:05
*** sdake has quit IRC20:05
crinklecould 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-cinder20:08
*** kfarr has joined #openstack-cinder20:12
bardiaAny cores got a few minutes? https://review.openstack.org/#/c/318382/2 :)20:14
*** lpetrut has quit IRC20:14
bardiasmcginnis: 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
smcginnisbardia: You will need to wait for it to merge in master first.20:16
bardiasmcginnis: well of course :)20:16
smcginnisThen on that review you can click on the "cherry pick" button and have it cherry pick to stable/mitaka20:16
bardiaah great20:16
smcginnisDoesn't look like there should be any merge conflicts, so hopefully just that.20:16
smcginnisbardia: Otherwise there's a few more steps you need to go through to get it in shape.20:17
smcginnisbardia: But we can cross that bridge if/when we get there.20:17
bardiaaye aye captain20:17
smcginnis;)20:18
*** kmartin has joined #openstack-cinder20:18
crinklethanks for the reviews smcginnis and hemna :)20:20
smcginniscrinkle: np20:20
*** Apoorva has joined #openstack-cinder20:23
*** yangxi has joined #openstack-cinder20:34
*** julim has quit IRC20:35
*** chris_morrell has joined #openstack-cinder20:36
*** chris_morrell has quit IRC20:37
*** jungleboyj has quit IRC20:38
*** yangxi has quit IRC20:39
*** chris_morrell has joined #openstack-cinder20:43
openstackgerritKendall Nelson proposed openstack/os-brick: WIP: Splitting Out Connectors from connector.py :WIP  https://review.openstack.org/30797420:44
*** timcl has quit IRC20:48
*** e0ne has quit IRC20:49
*** e0ne has joined #openstack-cinder20:50
*** cknight1 has quit IRC20:51
*** chris_morrell has quit IRC20:51
*** eharney has quit IRC20:53
*** ducttape_ has quit IRC20:53
*** chris_morrell has joined #openstack-cinder20:57
*** dustins has quit IRC20:57
*** rcernin has joined #openstack-cinder21:03
*** dustins has joined #openstack-cinder21:03
*** chris_morrell has quit IRC21:05
*** karthikp_ has joined #openstack-cinder21:05
*** frakt has quit IRC21:06
*** jdurgin has quit IRC21:07
*** jdurgin has joined #openstack-cinder21:08
*** karthikp_ has quit IRC21:09
*** frakt has joined #openstack-cinder21:13
*** jgregor has quit IRC21:18
*** haplo37_ has joined #openstack-cinder21:19
*** jgregor has joined #openstack-cinder21:19
*** kfarr has quit IRC21:19
*** gouthamr has quit IRC21:19
openstackgerritMerged openstack/cinder: Mark the v1 API as deprecated  https://review.openstack.org/30965221:21
*** ducttape_ has joined #openstack-cinder21:22
*** diablo_rojo has quit IRC21:23
*** rlrossit has quit IRC21:23
*** jgregor has quit IRC21:23
*** haplo37_ has quit IRC21:24
*** e0ne has quit IRC21:30
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Added support for failover_host failback  https://review.openstack.org/31006421:31
*** ducttape_ has quit IRC21:33
openstackgerritBardia Keyoumarsi proposed openstack/cinder: Coho Data: Reconnect to cluster on broken pipe  https://review.openstack.org/31838221:34
*** sdake has joined #openstack-cinder21:34
*** sgotliv_ has quit IRC21:35
*** ametts has quit IRC21:35
*** diogogmt has joined #openstack-cinder21:36
*** baumann has left #openstack-cinder21:37
*** sdake_ has quit IRC21:38
*** rcernin has quit IRC21:43
*** karthikp_ has joined #openstack-cinder21:45
*** martyturner has quit IRC21:46
*** akshai has quit IRC21:48
*** karthikp_ has quit IRC21:50
SwansonInfra looks somewhat backed up today.21:50
*** ducttape_ has joined #openstack-cinder21:53
openstackgerritAlex O'Rourke proposed openstack/cinder: LeftHand Fix terminate_connection when failed over  https://review.openstack.org/32068621:59
*** dustins has quit IRC22:00
*** sdake has quit IRC22:09
*** gouthamr has joined #openstack-cinder22:12
*** sdake has joined #openstack-cinder22:19
*** darosale has quit IRC22:26
*** gouthamr_ has joined #openstack-cinder22:34
*** jwcroppe_ has joined #openstack-cinder22:35
*** jwcroppe has quit IRC22:36
*** gouthamr has quit IRC22:37
*** ducttape_ has quit IRC22:46
*** xinli has quit IRC22:50
*** edmondsw has quit IRC22:50
*** ducttape_ has joined #openstack-cinder22:57
*** ducttape_ has quit IRC22:58
*** Julien-zte has joined #openstack-cinder23:00
*** openstackgerrit has quit IRC23:03
*** openstackgerrit has joined #openstack-cinder23:04
*** leeantho has joined #openstack-cinder23:05
*** leeantho has quit IRC23:05
*** leeantho has joined #openstack-cinder23:06
*** sdague has quit IRC23:09
*** markvoelker has quit IRC23:11
*** david-lyle_ is now known as david-lyle23:12
openstackgerritJohn Griffith proposed openstack/cinder: Rework initialize_connection and attach  https://review.openstack.org/32072123:29
*** leeantho has quit IRC23:32
smcginnisjgriffith: Woot23:34
*** Julien-zte has quit IRC23:38
*** gman-tx has quit IRC23:40
*** lcurtis has quit IRC23:41
*** arch-nemesis has quit IRC23:49
*** yhayashi has joined #openstack-cinder23:50
*** adrianofr has quit IRC23:52
*** sdake has quit IRC23:55

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!