Wednesday, 2017-01-11

*** dave-mccowan has quit IRC00:07
*** jamielennox is now known as jamielennox|away00:07
openstackgerritSzymon Wróblewski proposed openstack/cinder: Cleanup quota modules  https://review.openstack.org/41728200:08
*** harlowja has quit IRC00:09
*** yangyapeng has quit IRC00:09
*** LeoZhang has quit IRC00:09
*** LeoZhang has joined #openstack-cinder00:10
*** pck has quit IRC00:11
*** pck has joined #openstack-cinder00:11
*** jamielennox|away is now known as jamielennox00:11
*** markvoelker has quit IRC00:12
*** v12aml has quit IRC00:12
*** med_ has quit IRC00:12
*** afazekas has quit IRC00:12
*** markvoelker has joined #openstack-cinder00:12
*** pkoniszewski has quit IRC00:13
*** ildikov has quit IRC00:13
*** jroll has quit IRC00:14
*** pkoniszewski has joined #openstack-cinder00:16
*** med_ has joined #openstack-cinder00:16
*** med_ is now known as Guest4752000:17
*** v12aml has joined #openstack-cinder00:17
*** afazekas has joined #openstack-cinder00:17
*** Guest47520 is now known as medberry00:19
*** medberry has quit IRC00:19
*** medberry has joined #openstack-cinder00:19
*** lixiaoy1 has joined #openstack-cinder00:20
*** ildikov has joined #openstack-cinder00:21
*** jroll has joined #openstack-cinder00:21
*** porrua has joined #openstack-cinder00:23
*** harlowja has joined #openstack-cinder00:25
*** chenying has quit IRC00:29
*** chenying has joined #openstack-cinder00:29
*** porrua has quit IRC00:31
*** jwcroppe has quit IRC00:32
*** mriedem has joined #openstack-cinder00:35
*** mriedem1 has joined #openstack-cinder00:38
*** Julien-zte has joined #openstack-cinder00:40
*** mriedem has quit IRC00:42
*** wanghao has joined #openstack-cinder00:43
*** oomichi has quit IRC00:58
*** mriedem1 has quit IRC00:58
*** oomichi has joined #openstack-cinder01:00
*** viks has quit IRC01:00
*** LeoZhang has quit IRC01:11
*** dave-mccowan has joined #openstack-cinder01:11
*** diablo_rojo has quit IRC01:14
*** yangyapeng has joined #openstack-cinder01:18
*** huanxuan has joined #openstack-cinder01:22
*** raunak has quit IRC01:33
*** cppforlife_ has joined #openstack-cinder01:34
*** briancurtin has joined #openstack-cinder01:35
*** namnh has joined #openstack-cinder01:39
*** xyang has joined #openstack-cinder01:42
openstackgerritZhao Liqiang proposed openstack/cinder: RBD: the value of 'had_journaling' has not changed after 'update_features'.  https://review.openstack.org/41863801:42
*** diablo_rojo has joined #openstack-cinder01:42
*** markvoelker has quit IRC01:43
*** xyang has quit IRC01:44
openstackgerritZhao Liqiang proposed openstack/cinder: RBD: the value of 'had_journaling' has not changed after 'update_features'.  https://review.openstack.org/41863801:44
*** markvoelker has joined #openstack-cinder01:44
openstackgerritZhao Liqiang proposed openstack/cinder: RBD: the value of 'had_journaling' has not changed after 'update_features'.  https://review.openstack.org/41863801:46
*** xyang has joined #openstack-cinder01:46
*** markvoelker has quit IRC01:49
*** cdelatte has quit IRC01:54
*** bardia has quit IRC01:55
*** cknight1 has quit IRC02:03
*** diablo_rojo has quit IRC02:06
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] ignore me, kicking off a rally/py35 build  https://review.openstack.org/41864302:07
openstackgerritxianming.mao proposed openstack/cinder: Move useless FIXME  https://review.openstack.org/41439902:11
*** Jack_Iv has joined #openstack-cinder02:11
*** jose-phillips has quit IRC02:12
*** cknight has joined #openstack-cinder02:13
openstackgerritxianming.mao proposed openstack/cinder: Move useless FIXME  https://review.openstack.org/41439902:14
*** Jack_Iv has quit IRC02:16
*** chlong has quit IRC02:32
*** snarwade has joined #openstack-cinder02:43
*** raunak has joined #openstack-cinder02:49
*** dave-mccowan has quit IRC03:17
*** gouthamr has quit IRC03:21
*** diablo_rojo has joined #openstack-cinder03:27
openstackgerritwangxiyuan proposed openstack/cinder: FusionStorage: rename the CI_WIKI_NAME  https://review.openstack.org/41866003:27
*** lixiaoy1 has quit IRC03:29
*** nicolasbock has quit IRC03:31
*** salv-orl_ has quit IRC03:32
*** salv-orlando has joined #openstack-cinder03:32
*** Lee1092 has joined #openstack-cinder03:34
*** porrua has joined #openstack-cinder03:34
*** markvoelker has joined #openstack-cinder03:36
*** porrua has quit IRC03:40
*** links has joined #openstack-cinder03:47
openstackgerritTony Breeds proposed openstack/python-cinderclient: [DNM] Tetsing keystoneauth1 update  https://review.openstack.org/41866603:56
openstackgerritXu Ao proposed openstack/cinder: [RBD]Add a configuation to enable the volume to be multiattached  https://review.openstack.org/41866804:02
*** yangyapeng has quit IRC04:05
*** Jack_Iv has joined #openstack-cinder04:12
*** Jack_Iv has quit IRC04:16
*** salv-orl_ has joined #openstack-cinder04:23
*** salv-orlando has quit IRC04:26
*** jose-phillips has joined #openstack-cinder04:28
*** jose-phillips has quit IRC04:30
*** sudipto_ has joined #openstack-cinder04:32
*** sudipto has joined #openstack-cinder04:32
*** sheel has joined #openstack-cinder04:34
*** mdnadeem has joined #openstack-cinder04:35
*** psachin has joined #openstack-cinder04:40
*** yangyapeng has joined #openstack-cinder04:52
*** ccesario has quit IRC05:02
*** bardia has joined #openstack-cinder05:02
*** raunak_ has joined #openstack-cinder05:08
*** raunak has quit IRC05:09
*** raunak_ is now known as raunak05:09
*** lixiaoy1 has joined #openstack-cinder05:10
*** adisky_ has joined #openstack-cinder05:11
*** gcb has quit IRC05:13
*** ccesario has joined #openstack-cinder05:14
*** Cibo has quit IRC05:15
*** pbandark has joined #openstack-cinder05:15
*** sudipto_ has quit IRC05:17
*** sudipto has quit IRC05:17
*** Cibo has joined #openstack-cinder05:28
*** raunak has quit IRC05:35
*** lixiaoy1 has quit IRC05:40
*** sudipto_ has joined #openstack-cinder05:42
*** sudipto has joined #openstack-cinder05:42
*** Jack_Iv has joined #openstack-cinder05:47
*** ntpttr has quit IRC05:49
*** ntpttr_ has joined #openstack-cinder05:49
*** Jack_Iv has quit IRC05:49
*** diablo_rojo has quit IRC05:50
*** Jack_Iv has joined #openstack-cinder05:51
*** raunak has joined #openstack-cinder05:52
openstackgerritxianming.mao proposed openstack/cinder: Remove useless FIXME  https://review.openstack.org/41439906:02
*** nkrinner_afk is now known as nkrinner06:04
*** gcb has joined #openstack-cinder06:04
*** cknight has quit IRC06:05
*** raunak has quit IRC06:05
*** bardia has quit IRC06:05
*** tlian has quit IRC06:06
*** bkopilov has joined #openstack-cinder06:06
*** Jack_Iv has quit IRC06:07
*** Jack_Iv has joined #openstack-cinder06:07
*** raunak has joined #openstack-cinder06:10
*** lpetrut has joined #openstack-cinder06:10
*** dims has quit IRC06:13
*** Jack_Iv_ has joined #openstack-cinder06:15
*** raunak has quit IRC06:16
*** Jack_Iv has quit IRC06:19
*** Jack_Iv_ has quit IRC06:20
*** pcaruana has joined #openstack-cinder06:25
*** Poornima has joined #openstack-cinder06:35
*** sheel has quit IRC06:37
*** pgadiya has joined #openstack-cinder06:37
openstackgerritMerged openstack/cinder: Tests: Remove more InvalidUUID warnings  https://review.openstack.org/41848506:40
*** lpetrut has quit IRC06:43
*** mjura has joined #openstack-cinder06:44
openstackgerritXu Ao proposed openstack/cinder: [RBD]Add a configuation to enable the volume to be multiattached  https://review.openstack.org/41866806:44
openstackgerritZhao Liqiang proposed openstack/cinder: Create volume based on volume type with 'replication_enabled=Ture' seccessfully.  https://review.openstack.org/41873506:50
*** lpetrut has joined #openstack-cinder07:00
*** lkuchlan_ has joined #openstack-cinder07:05
*** lpetrut has quit IRC07:08
*** tesseract has joined #openstack-cinder07:08
*** ChubYann has quit IRC07:15
*** lpetrut has joined #openstack-cinder07:17
*** Jack_Iv has joined #openstack-cinder07:21
*** nherciu has joined #openstack-cinder07:21
*** Jack_Iv has quit IRC07:28
*** kaisers has quit IRC07:32
*** lpetrut has quit IRC07:33
*** kaisers1 has quit IRC07:33
openstackgerritZhao Liqiang proposed openstack/cinder: Create volume based on volume type with 'replication_enabled=Ture' seccessfully.  https://review.openstack.org/41873507:33
*** AlexeyAbashkin has joined #openstack-cinder07:42
openstackgerritMerged openstack/cinder: Revert "Swap volume type for migration started by retype"  https://review.openstack.org/40708907:43
*** kaisers has joined #openstack-cinder07:50
*** kaisers1 has joined #openstack-cinder07:50
openstackgerritZhao Liqiang proposed openstack/cinder: RBD: change the value of 'had_journaling'.  https://review.openstack.org/41863807:52
*** lixiaoy1 has joined #openstack-cinder07:52
*** bkopilov has quit IRC07:55
*** hogepodge has quit IRC07:56
*** Zer0Byte has joined #openstack-cinder08:02
*** Zer0Byte has quit IRC08:03
*** Zer0Byte has joined #openstack-cinder08:04
*** strigazi_AFK is now known as strigazi08:13
*** Cibo_ has joined #openstack-cinder08:16
*** gcb has quit IRC08:19
*** sudipto has quit IRC08:21
*** salv-orl_ has quit IRC08:21
*** sudipto_ has quit IRC08:23
*** sudipto has joined #openstack-cinder08:23
*** sudipto_ has joined #openstack-cinder08:23
*** lpetrut has joined #openstack-cinder08:24
*** lyarwood is now known as lyarwood_08:30
*** gcb has joined #openstack-cinder08:33
*** lixiaoy1 has quit IRC08:34
*** stakeda has quit IRC08:39
*** gcb has quit IRC08:40
*** pgadiya is now known as pgadiya|mtg08:50
*** pgadiya|mtg is now known as pgadiya08:50
*** bkopilov has joined #openstack-cinder08:51
*** Zer0Byte has quit IRC08:53
*** haplo37_ has quit IRC08:54
*** haplo37_ has joined #openstack-cinder08:57
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-cinder09:00
*** jordanP has joined #openstack-cinder09:02
openstackgerritZhao Liqiang proposed openstack/cinder: RBD: enable replication if needed.  https://review.openstack.org/41878609:04
*** lkuchlan_ has quit IRC09:08
*** namnh has quit IRC09:20
*** lkuchlan_ has joined #openstack-cinder09:21
*** wanghao has quit IRC09:28
*** mvk has quit IRC09:32
*** Julien-zte has quit IRC09:34
*** lkuchlan_ has quit IRC09:35
*** ntpttr_ has quit IRC09:41
*** ntpttr has joined #openstack-cinder09:42
*** Jack_Iv has joined #openstack-cinder09:44
openstackgerritArnon Yaari proposed openstack/cinder: Remove inheritance from deprecated classes  https://review.openstack.org/41845209:44
*** lkuchlan_ has joined #openstack-cinder09:48
*** Jack_Iv has quit IRC09:57
*** shubham0d has joined #openstack-cinder10:00
openstackgerritArnon Yaari proposed openstack/cinder: Use infinisdk for communication with InfiniBox storage array  https://review.openstack.org/41845110:01
*** lyarwood_ is now known as lyarwood10:02
*** e0ne has joined #openstack-cinder10:02
shubham0dgeguileo:everytime I try to do incremental backup I get this error.http://paste.openstack.org/show/594519/10:03
*** lyarwood is now known as lyarwood_10:03
geguileodid the first backup finish before you tried to do the incremental one?10:03
geguileoyou do a list of the backups and it must appear as available10:04
geguileothat's what the error message says10:04
shubham0dgeguileo:yes they are available.I have checked by cinder list10:05
geguileocinder list only list volumes, not backups10:05
geguileomessage talks about "parent backup", not source volume10:05
shubham0dgeguileo:I have run cinder backup-list and it giving me this.http://paste.openstack.org/show/594520/10:07
geguileothen the parent backup isn't available, it's in error10:08
geguileoyou have to look in the logs to see what the error is10:09
shubham0dgeguileo:ok10:09
geguileoor you can request another backup while looking at the c-back logs10:09
shubham0dgeguileo:ok10:10
*** mvk has joined #openstack-cinder10:10
*** yangyapeng has quit IRC10:17
*** Alexey_Abashkin has joined #openstack-cinder10:18
*** alonmarx has joined #openstack-cinder10:18
*** AlexeyAbashkin has quit IRC10:19
*** mdnadeem has quit IRC10:24
*** Alexey_Abashkin has quit IRC10:31
*** cristicalin has joined #openstack-cinder10:34
*** thiagolib has joined #openstack-cinder10:39
*** snarwade is now known as snarwade\afk10:39
*** mdnadeem has joined #openstack-cinder10:40
*** ripsomeday has joined #openstack-cinder10:43
ripsomedayHi there, is this a right channel to ask cinder driver development questions, or should I ask in openstack-dev instead?10:47
*** aarefiev has joined #openstack-cinder10:47
dulekripsomeday: That's the right channel, although I'm not sure if there are a lot of driver developers here in European time.10:49
ripsomedayok, thanks, will come back later then10:50
*** aarefiev_afk has quit IRC10:50
dulekripsomeday: Well, there's no harm in asking now, maybe it'll ring a bell for someone. :)10:51
ripsomedaydulek: ok, will try :)10:52
ripsomedayso the question is - assuming that cinder driver has multiple pools. is there any proper way to delete pool? I see something like this: 1) make sure no new volumes get into pool 2) migrate all the volumes to other pools 3) do whatever needs to be done in underlying storage to delete pool. what is the proper way to do 1st step?10:54
*** Jack_Iv has joined #openstack-cinder10:54
ripsomedayor driver is supposed to just not return disabled pool in get_volume_stats, but keep accepting operation on volumes that still on that pool?10:55
*** Jack_Iv has quit IRC10:59
*** gcb has joined #openstack-cinder11:00
*** AlexeyAbashkin has joined #openstack-cinder11:02
shubham0dcan anyone tell me the path of cinder logs? according to openstack documentation it should be in  /var/log/cinder/ but I have no such folder.11:04
*** Alexey_Abashkin has joined #openstack-cinder11:06
*** AlexeyAbashkin has quit IRC11:07
adrianofrshubham0d: If you are using devstack you can see then on the screen (screen -r)11:09
*** Alexey_Abashkin_ has joined #openstack-cinder11:09
shubham0dadrianofr:ok..thnx11:10
*** Alexey_Abashkin has quit IRC11:11
adrianofrshubham0d: np :)  ... they are also available in /opt/stack/logs11:12
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/41885511:13
shubham0dadrianofr:screen doesn't work but find on /opt/stack/logs11:14
*** Poornima has quit IRC11:14
*** sudipto has quit IRC11:15
*** sudipto_ has quit IRC11:15
*** sdague has joined #openstack-cinder11:15
openstackgerritxianming.mao proposed openstack/cinder: Add ERROR info  https://review.openstack.org/41613611:17
*** shubham0d has quit IRC11:18
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements  https://review.openstack.org/41886311:22
*** lseki has joined #openstack-cinder11:27
*** yangyapeng has joined #openstack-cinder11:33
*** nicolasbock has joined #openstack-cinder11:35
*** dims has joined #openstack-cinder11:39
*** haplo37_ has quit IRC11:40
*** JoseMello has joined #openstack-cinder11:40
dimse0ne : hey, i have a devstack w/ py35 + rally test, i see some failures - http://logs.openstack.org/43/418643/1/check/gate-rally-dsvm-py35-cinder-nv/4256dea/rally-plot/results.html.gz11:40
dimsAnyone else interested in poking at py35? ^^11:40
e0nedims: hi. I'll take a look on it a bit later11:41
*** cristicalin has quit IRC11:41
*** haplo37_ has joined #openstack-cinder11:43
dimsthanks e0ne11:44
*** ociuhandu has joined #openstack-cinder11:52
*** Julien-zte has joined #openstack-cinder11:53
*** Jack_Iv has joined #openstack-cinder11:55
*** snarwade\afk is now known as snarwade11:56
*** tommylikehu_ has joined #openstack-cinder11:57
*** AlexeyAbashkin has joined #openstack-cinder11:58
*** huanxuan has quit IRC11:58
*** Jack_Iv has quit IRC11:59
*** Alexey_Abashkin_ has quit IRC11:59
*** mvk has quit IRC12:03
*** alyson_ has joined #openstack-cinder12:05
*** mvk has joined #openstack-cinder12:15
openstackgerritMerged openstack/cinder: Make APIVersionRequest's null check more pythonic  https://review.openstack.org/35176812:17
openstackgerritMerged openstack/cinder: Remove unused PasteAppNotFound exception  https://review.openstack.org/41633312:17
openstackgerritMerged openstack/cinder: Don't translate exceptions w/ no message  https://review.openstack.org/41633412:17
*** AlexeyAbashkin has quit IRC12:20
openstackgerritMerged openstack/cinder: Fix create_group with group type name  https://review.openstack.org/41676912:23
openstackgerritMerged openstack/cinder: Tests: Split up NFS backup notification test  https://review.openstack.org/41375212:23
*** Julien-zte has quit IRC12:29
*** tommylikehu_ has quit IRC12:33
*** Julien-zte has joined #openstack-cinder12:40
openstackgerritFelix Ma proposed openstack/os-brick: Use dmsetup to remove ISCSI multipath device  https://review.openstack.org/41891012:42
*** gcb has quit IRC12:45
*** aviramb has joined #openstack-cinder12:47
*** pgadiya has quit IRC12:48
*** mdnadeem has quit IRC12:50
*** Jack_Iv has joined #openstack-cinder12:56
*** sudipto has joined #openstack-cinder12:56
*** sudipto_ has joined #openstack-cinder12:56
*** tommylikehu_ has joined #openstack-cinder12:59
*** tommylikehu_ has quit IRC12:59
*** tommylikehu_ has joined #openstack-cinder12:59
*** Jack_Iv has quit IRC13:00
*** timcl has joined #openstack-cinder13:02
*** LeoZhang has joined #openstack-cinder13:04
*** liverpooler has quit IRC13:05
*** chlong has joined #openstack-cinder13:08
*** LeoZhang has quit IRC13:10
*** LeoZhang has joined #openstack-cinder13:10
*** ducttape_ has quit IRC13:14
*** ducttape_ has joined #openstack-cinder13:17
jordanPsmcginnis, 2min about https://review.openstack.org/#/c/418922/ ?13:20
jordanPwould a feature flag, set to False by default would help here ?13:20
*** mvk has quit IRC13:21
jordanPmanage/unmanage snap is definitely a feature, though optional, of Cinder, so tests are good right ?13:21
smcginnisjordanP: Hey! Yeah, a feature flag would probably be a good answer to that.13:23
smcginnisjordanP: I love having a test to cover it. I just don't like a bunch of CIs failing because they don't support it. ;)13:23
jordanPok, I'll submit something like this right now13:23
smcginnisjordanP: I'll also comment on the patch.13:23
smcginnisjordanP: Awesome, thanks!13:23
jordanPsmcginnis, good13:23
*** mvk has joined #openstack-cinder13:33
jordanPit's here13:34
jordanPhttps://review.openstack.org/#/c/41892813:34
jordanPoomichi, mtreinish ^^13:34
*** AlexeyAbashkin has joined #openstack-cinder13:35
smcginnisjordanP: Thank you13:38
*** swamireddy has quit IRC13:39
*** snarwade has quit IRC13:42
*** ccesario has quit IRC13:50
*** Jack_Iv has joined #openstack-cinder13:54
*** david-lyle has quit IRC13:56
*** AlexeyAbashkin has quit IRC13:57
*** tongli has joined #openstack-cinder13:59
*** david-lyle has joined #openstack-cinder13:59
*** AlexeyAbashkin has joined #openstack-cinder13:59
*** lamt has joined #openstack-cinder14:01
*** links has quit IRC14:02
*** mriedem has joined #openstack-cinder14:03
*** kaisers_ has joined #openstack-cinder14:04
*** Jack_Iv has quit IRC14:04
*** Jack_Iv has joined #openstack-cinder14:06
*** Jack_Iv has quit IRC14:08
*** ntpttr has quit IRC14:09
*** ntpttr has joined #openstack-cinder14:13
*** tlian has joined #openstack-cinder14:13
jungleboyjMorning all.  :-)14:14
*** ducttape_ has quit IRC14:14
smcginnisjungleboyj: Did you finally get your TV set up?14:14
*** jamielennox is now known as jamielennox|away14:14
jungleboyjOh yeah!14:14
*** liverpooler has joined #openstack-cinder14:16
jungleboyjsmcginnis:  Ian and I had to MacGyver it into place.14:16
*** david-lyle has quit IRC14:16
smcginnisLoved that show. :D14:18
jungleboyjYeah.  :-)  It was too heavy for me to lift the whole thing alone and Ian couldn't lift it much.14:20
jungleboyjPut the base box on coolers to get it above the entertainment center, lifted it out of the packaging and then had Ian help lower it on to the stand which worked.  :-)14:21
smcginnisHah14:21
smcginnisNo hoists mounted into the ceiling.14:21
jungleboyj:-)  No, that could have helped.14:22
jungleboyjsmcginnis: https://review.openstack.org/#/c/418567/1  Question on this, are we not requiring for driverfixes that the fixes have merged into the intermediate releases?14:23
smcginnisjungleboyj: Oh, I see. It's in master, but not in Newton?14:26
smcginnisjungleboyj: Wait, no, that should be.14:27
smcginnisMerged June 6, 2016.14:27
smcginnisjungleboyj: They should be merged in intermediate releases before driverfixes/*, at least as much as possible I think.14:28
jungleboyjsmcginnis: It is in master but I don't see it in Newton.14:28
smcginnisjungleboyj: June 6 would have been just after the Newton-1 milestone. So it should be there.14:29
*** porrua has joined #openstack-cinder14:30
jungleboyjOh duh!  Ok, so master was Newton.  :-)14:30
jungleboyjsmcginnis: Thanks for the check there.14:31
*** dave-mccowan has joined #openstack-cinder14:31
smcginnis;)14:35
*** chlong has quit IRC14:37
*** dolphm has left #openstack-cinder14:38
*** lamt has quit IRC14:40
*** dustins has joined #openstack-cinder14:42
*** ducttape_ has joined #openstack-cinder14:45
*** jwcroppe has joined #openstack-cinder14:46
openstackgerritMerged openstack/cinder: Report multiattach capability for Hitachi VSP drivers  https://review.openstack.org/41342814:47
*** tongli has quit IRC14:49
*** cloudm2 has joined #openstack-cinder14:52
*** gouthamr has joined #openstack-cinder14:52
*** dustins has quit IRC14:54
*** chenying has quit IRC14:58
*** dustins has joined #openstack-cinder14:59
*** enriquetaso has quit IRC15:02
*** dustins_ has joined #openstack-cinder15:02
*** dustins has quit IRC15:04
*** enriquetaso has joined #openstack-cinder15:05
*** nkrinner is now known as nkrinner_afk15:09
*** psachin has quit IRC15:14
*** Julien-zte has quit IRC15:15
*** david-lyle has joined #openstack-cinder15:17
*** viks has joined #openstack-cinder15:19
*** david-lyle has quit IRC15:21
openstackgerritMichael Dovgal proposed openstack/cinder: Add volume type access operations notification  https://review.openstack.org/41836915:24
*** eantyshev has joined #openstack-cinder15:28
*** markvoelker has quit IRC15:28
*** markvoelker has joined #openstack-cinder15:29
eantysheve0ne: Hello! Could you please look at this change for Vzstorage driver: https://review.openstack.org/357679 ? It has +2 already, and waits for a rather long time15:30
*** tommylikehu_ has quit IRC15:33
*** markvoelker has quit IRC15:35
viksHi all, can someone please review https://review.openstack.org/#/c/417537/15:36
*** raunak has joined #openstack-cinder15:37
*** diablo_rojo_phon has joined #openstack-cinder15:38
*** raunak has quit IRC15:40
*** stvnoyes has joined #openstack-cinder15:41
*** mvk has quit IRC15:48
*** guyr-infinidat has joined #openstack-cinder15:49
*** knomura has joined #openstack-cinder15:49
*** wxy| has joined #openstack-cinder15:50
*** tommylikehu_ has joined #openstack-cinder15:52
*** diablo_rojo has joined #openstack-cinder15:53
*** shaner has quit IRC15:54
*** shaner has joined #openstack-cinder15:54
*** tommylikehu_ has quit IRC15:55
*** xyang has joined #openstack-cinder15:56
*** markvoelker has joined #openstack-cinder15:57
*** cknight has joined #openstack-cinder15:59
scottdahttps://www.irccloud.com/pastebin/woVg2TOL/16:00
scottdasmcginnis: ^^^16:00
*** pbandark has quit IRC16:02
*** lamt has joined #openstack-cinder16:03
*** lkuchlan_ has quit IRC16:03
smcginnisscottda: Ack16:06
*** Lee1092 has quit IRC16:06
*** salv-orlando has joined #openstack-cinder16:09
*** mjura has quit IRC16:10
*** bardia has joined #openstack-cinder16:10
*** chlong has joined #openstack-cinder16:11
*** jessegler has joined #openstack-cinder16:11
*** siddhi has joined #openstack-cinder16:12
*** ccesario has joined #openstack-cinder16:13
*** bkopilov has quit IRC16:16
openstackgerritwangxiyuan proposed openstack/cinder: Improvement to get group detail(Part 2)  https://review.openstack.org/40969416:21
*** salv-orl_ has joined #openstack-cinder16:22
*** salv-orlando has quit IRC16:25
*** bkopilov has joined #openstack-cinder16:28
dulekjgriffith: Not sure if you've seen that, but it catches why microversions were introduced back in Kilo: https://dague.net/2015/06/05/the-nova-api-in-kilo-and-beyond-2/16:29
dulekjgriffith: Not that I disagree that microversioned model has major disadvantages for developers.16:30
smcginnisI'm fine with microversions. It's more that I'm against this concept that we need to make sure anyone can always deploy from master and upgrade and everything will be supported just fine.16:31
*** AlexeyAbashkin has quit IRC16:32
jgriffithdulek yeah thanks!  I actually tend to point questions to that exact page16:32
jgriffithdulek I probably shouldn't have mentioned it at all16:32
jgriffithdulek but there's a number of things we've done that makes things more complex than they probably should or need to be which is dissappointing16:33
jgriffithdulek I do think there are better ways of doing it all the way around16:33
jgriffithbut it's hard to have any conversations around those things because 1. One project made a choice 2. Consistency across projects is good 3. Many (not all) people don't actually spend time trying to use this stuff16:34
*** enriquetaso has quit IRC16:34
dulekjgriffith: I guess cross project session at PTG called "Retrospective on Microversions" would help. :)16:35
jgriffithdulek :)16:35
jgriffithdulek the problem is honestly changing it at this point would probably be disastorous16:36
jgriffithdulek OpenStack has never been good at API's, and something like pivoting or making a significant change of any sort again would be super bad to do16:36
jgriffithIMO16:36
dulekjgriffith: Well, maybe there are ways of solving the burden that microversions place on developers? I don't think only Cinder people complain.16:37
jgriffithsmcginnis yeah, I'm kinda with you on that whole thing16:37
jgriffithdulek that's a great point16:37
jgriffithdulek and yes, I suspect there is a way to make that easier16:37
jgriffithdulek I think that's sort of where smcginnis is headed with his thoughts on things16:38
duleksmcginnis: I guess we'll need to ask ops in Milan - how many of them actually deploy from master? Maybe assumption that people do it isn't really true anymore?16:39
ildikovdulek: I have doubts about too many of them would deploy from master, but it's definitely a good data to have16:45
knomuraAnyone willing to a short review? ;) https://review.openstack.org/#/c/414076/16:45
*** nherciu has quit IRC16:51
*** siddhi has quit IRC16:53
*** cknight1 has joined #openstack-cinder16:55
openstackgerritPavel Gluschak proposed openstack/cinder: vzstorage: deprecate vzstorage_mount_options  https://review.openstack.org/41904216:57
*** cknight has quit IRC16:57
*** viks has left #openstack-cinder17:01
*** xyang has quit IRC17:02
*** breitz has quit IRC17:02
smcginnisjungleboyj: Could have sworn I've seen a patch for the tentative Pike schedule, but don't see it now in the list.17:03
*** cloudm2 has quit IRC17:03
*** breitz has joined #openstack-cinder17:03
jungleboyjOk, no biggie.17:03
smcginnisjungleboyj: Looks like it's typically been 8 weeks after the release.17:03
smcginnisjungleboyj: Week of April 17th maybe?17:04
jungleboyjCool.  That would make sense.17:04
*** enriquetaso has joined #openstack-cinder17:06
*** e0ne has quit IRC17:06
openstackgerritMerged openstack/cinder: Convert 'parallels' format name to 'ploop'  https://review.openstack.org/35767917:07
*** wxy| has quit IRC17:09
*** briancurtin has quit IRC17:11
*** raunak has joined #openstack-cinder17:12
*** pcaruana has quit IRC17:12
*** Zealoussnow_ has joined #openstack-cinder17:13
*** LeoZhang has quit IRC17:13
*** jose-phillips has joined #openstack-cinder17:13
*** wxy has quit IRC17:14
jgriffithdulek you still around?17:15
*** tesseract has quit IRC17:20
jgriffithmaybe somebody else might know why passing the volume object in as a function arg is giving me an error?17:21
jgriffithif you look here at https://review.openstack.org/#/c/414632/8/cinder/api/v3/attachments.py Line # 16817:21
jgriffithildikov captured the error that's hit when it tries to evaluate volume_ref above on line 16417:22
*** jessegler has quit IRC17:27
stevemarsmcginnis: y'all need to release cinderclient more often!17:29
*** strigazi is now known as strigazi_AFK17:29
smcginnisstevemar: Oh?17:29
stevemarlast release on sept 1, 2016 :O17:29
stevemarand before that jun1, looks like you folks just do one per dev cycle? crrraazy17:30
smcginnisstevemar: I'll take a look at what's gone in since then and see if it makes sense to do a release now or just wait closer to the client freeze.17:30
stevemar"wait closer to client freeze" << *shudders*17:30
smcginnisstevemar: :)17:31
stevemari guess no other projects depend on cinderclient i suppose (except osc)17:31
stevemarbut we don't use the new bits17:31
smcginnisstevemar: Well, nova, but things are pretty stable for most core functionality.17:31
stevemarthats what we all say17:31
stevemar:P17:32
smcginnis;)17:32
stevemarsmcginnis: dirk mueller will be bugging you for a release soon-ish i think17:32
stevemari stick to -- release early, release often17:32
smcginnisstevemar: Thanks for the warning. I'll look into getting something queued up.17:33
jgriffithstevemar is there a reason you *need* a new client?17:33
jgriffithstevemar I'm just curious if there's actually something "useful" that people want/need?  I guess there have been several adds that are good17:36
*** xinli has joined #openstack-cinder17:36
stevemarjgriffith: i was releasing a new keystoneauth version and the requirements team was running extra tests on the new version and noticed something funny with cinderclient17:36
jgriffithstevemar well there's lots of things "funny" with the client, new release won't fix that :)17:37
stevemarjgriffith: turned out to be unrelated, but it sounds like a new release would fix things17:37
stevemarjgriffith: convo here; http://eavesdrop.openstack.org/irclogs/%23openstack-requirements/%23openstack-requirements.2017-01-11.log.html#t2017-01-11T16:50:5717:37
stevemarjgriffith: sounds like https://review.openstack.org/#/c/400732/ was needed17:37
jgriffithstevemar ahh... got it, cool.  Thanks!17:38
jgriffithsmcginnis hey.. cruious; have you seen this:  http://paste.openstack.org/show/594602/17:38
jgriffithcurious even17:38
jgriffithlooks like a new tempest test was added recently17:38
*** lpetrut has quit IRC17:38
jgriffithand I apparantly don't implement manage_existing_snapshot17:39
jgriffithsurprisingly enough the ABC stuff didn't catch that either17:39
* jgriffith hides his sarcasm17:39
smcginnisjgriffith: Yeah, I have a revert up for that tempest patch.17:39
jgriffithsmcginnis link to get my vote?17:39
smcginnisjgriffith: And there's an alternative patch for adding a flag to enable them.17:39
smcginnisjgriffith: Revert patch: https://review.openstack.org/41892217:40
smcginnisjgriffith: Flag patch: https://review.openstack.org/41892817:40
smcginnisjgriffith: I'm kind of thinking it might be better to revert it. Different drivers do this in different wats.17:40
smcginnis*ways17:40
*** Jack_Iv has joined #openstack-cinder17:41
*** Jack_Iv has quit IRC17:41
jgriffithyeah17:41
smcginnisThough good to have at least some test coverage on that, so maybe being able to at least do it with LVM is worth it.17:41
jgriffithsmcginnis probably should yes17:43
jgriffithbut this is the *trap* of optional features17:44
jgriffithkind of annoying17:44
openstackgerritJustin A Wilson proposed openstack/python-cinderclient: Cinder Client Automatic Microversion Negotiation  https://review.openstack.org/41716917:44
smcginnisjgriffith: Yeah...17:44
jgriffithsmcginnis on the positive side, it's certainly a WIN for the CI efforts17:44
*** eharney has joined #openstack-cinder17:44
smcginnisjgriffith: True, that is a good point.17:44
jgriffithNow if I can just figure out WTF is up with this object thing :(17:45
smcginnisjgriffith: Looked at that briefly. Seems really odd to me.17:45
jgriffithdulek has a unique *gift* of knowing the EXACT right time to sign off for the night :)17:45
smcginnisskillz17:46
jgriffithsmcginnis so what's weird is that if I change it to named args in V3/attachments.py it gets rid of the problem...17:46
jgriffithbut then the wrap check in volume/api.py fails for another cryptic thing17:46
jgriffithI think maybe I'll just call it a day and go bowling17:47
smcginnisThat's really strange.17:47
smcginnisHah, sounds like a decent plan. ;)17:47
*** jordanP has quit IRC17:50
*** sudipto has quit IRC17:50
*** sudipto_ has quit IRC17:50
*** Jack_Iv has joined #openstack-cinder17:53
*** aarefiev is now known as aarefiev_afk17:54
*** AlexeyAbashkin has joined #openstack-cinder17:55
*** Jack_Iv has quit IRC17:55
*** Cibo_ has quit IRC17:58
*** AlexeyAbashkin has quit IRC17:58
*** xyang has joined #openstack-cinder17:58
*** AlexeyAbashkin has joined #openstack-cinder17:58
*** mvk has joined #openstack-cinder17:59
*** knomura has quit IRC17:59
*** ociuhandu has quit IRC18:01
*** liverpooler has quit IRC18:05
*** xyang has quit IRC18:07
*** xyang has joined #openstack-cinder18:07
*** xyang has quit IRC18:07
*** alonmarx has quit IRC18:07
*** dirk has joined #openstack-cinder18:07
dirksmcginnis: are there any plans regarding a python-cinderclient release for ocata?18:08
smcginnisdirk: Hah, stevemar warned me you might come asking. :)18:10
smcginnisdirk: I was planning on waiting until close to the client freeze, but if you need something in there I can try to get something out earlier.18:10
dirksmcginnis: ah, so you've been warned already, sorry ;)18:12
*** liverpooler has joined #openstack-cinder18:12
dirksmcginnis: no, nothing urgent, but I would expect a minimum version bump also to land, which basically means if it comes sooner then others can pick up that min bump earlier18:12
*** alonmarx has joined #openstack-cinder18:12
dirksmcginnis: any time within next week is perfectly good18:13
smcginnisdirk: OK. I'll take another look at what's in there queued up. There are one or two more things I am hoping land in Ocata, but I can always do another release to pick those up if needed.18:13
dirksmcginnis: there hasn't been a release for 3 months or so, that makes me a bit nervous about potential regressions  that would need to be sorted out before client freeze18:14
dirksmcginnis: yep, there can be more than one release before the client freeze ;)18:14
smcginnisdirk: Should be fairly safe at this point. I've usually only being doing the releases when new functionality has been needed.18:15
smcginnisdirk: But yeah, never hurts to get releases out regulary.18:15
*** david-lyle has joined #openstack-cinder18:15
*** david-lyle has quit IRC18:16
*** david-lyle has joined #openstack-cinder18:16
openstackgerritSean McGinnis proposed openstack/cinder: Mark the sheepdog driver as unsupported  https://review.openstack.org/41907918:17
*** enriquetaso has quit IRC18:19
*** Alexey_Abashkin has joined #openstack-cinder18:21
*** aviramb has quit IRC18:23
*** AlexeyAbashkin has quit IRC18:24
*** kfarr has joined #openstack-cinder18:25
*** lpetrut has joined #openstack-cinder18:31
jgriffithsmcginnis haha!  Figured it out18:32
jgriffithsmcginnis the wrap_check_policy REQUIRES that the 3'rd arg is a volume!18:32
jgriffithsmcginnis I was passing it in as the 4'th arg18:32
jgriffith:(18:32
jgriffithself, context, volume....18:32
jgriffithI had self, context, instance_uuid, volume18:32
*** guyr-infinidat has quit IRC18:34
hemnaeharney, ping18:34
eharneyhemna: hey18:34
smcginnisjgriffith: 90% of the time it ends up being something trivial like that. The other 10% of the time it's just something completely wrong. ;)18:34
hemnaeharney, do you happen to know what happened to the dmsetup source?  the redhat link seems to be incorrect now18:35
eharneyhemna: which link was that?18:35
hemnahttp://sources.redhat.com/dm sends me elsewhere18:35
hemnahttps://sourceware.org/dm/18:35
*** ntpttr has quit IRC18:36
hemnaand the link on that page for the web interface to CVS doesn't have dm anymore18:36
hemnacvs co timeout as well18:36
hemnawanted to see the source for dmsetup remove and what it actually does18:36
eharneyi think it's part of the lvm2 tree now: https://git.fedorahosted.org/cgit/lvm2.git/tree/tools/dmsetup.c18:37
hemnahuh lvm?!18:38
hemnastrange18:38
hemnathanks18:38
*** guyr-infinidat has joined #openstack-cinder18:38
*** adisky_ has quit IRC18:39
*** ntpttr has joined #openstack-cinder18:40
*** guyr-infinidat has quit IRC18:43
*** ociuhandu has joined #openstack-cinder18:45
*** chlong has quit IRC18:46
*** dustins_ has quit IRC18:52
*** kfarr has quit IRC18:54
*** dave-mccowan has quit IRC18:54
*** dave-mcc_ has joined #openstack-cinder18:54
*** lpetrut has quit IRC18:55
*** thiagolib has quit IRC18:58
*** chlong has joined #openstack-cinder18:59
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/41885518:59
*** viks has joined #openstack-cinder19:01
*** mtanino has joined #openstack-cinder19:02
*** briancurtin has joined #openstack-cinder19:09
*** ccesario has quit IRC19:12
*** wxy has joined #openstack-cinder19:12
*** dave-mcc_ has quit IRC19:15
*** ChubYann has joined #openstack-cinder19:16
hemnahttps://releases.openstack.org/schedule.ics19:24
hemnapretty cool!19:24
hemnajust imported that into my gmail calendar19:24
*** Jack_Iv has joined #openstack-cinder19:26
*** Jack_Iv has quit IRC19:27
patrickeasthemna: thats super helpful19:28
hemnanp.  I just saw the ML posts about it and thought I'd share19:29
*** dustins has joined #openstack-cinder19:30
jgriffithhemna I know you don't really care.. but to try and explain what I was saying earlier19:32
jgriffithhemna about api versions.. what I was getting at was at the beginning of the cycle you bump to a single number; so rolling release still works19:32
jgriffithhemna any incompat/new changes go under that "new" bumped version19:33
*** dustins_ has joined #openstack-cinder19:33
jgriffithhemna that version points to any "old" call that exists and hasn't been updated19:33
jgriffithso you just use inheritance to make it all come together, but the cool thing is you also still have the ability to specify a specific (or old) version of a call if you like19:34
hemnaoh, to v419:34
hemnanot v3.17 from v3.1619:34
smcginnisjgriffith: Only problem with that I see is the way we use keystone endpoints. :/19:34
hemnasmcginnis, +119:34
jgriffithyeah, whether it's something like "v4" or whatever is debateable19:34
hemnawhich is one of the reasons micro versions were created19:35
smcginnisAnd I've certainly gotten enough crap for even creating a v3.19:35
hemnato avoid the whole keystone debacle19:35
jgriffithyeah.. smcginnis that's where you're idea of things like using a global mv is kinda nifty19:35
*** dustins_ has quit IRC19:35
jgriffithhemna smcginnis honestly I think we should route things differently in terms of keystone19:35
smcginnisI think if we were able to do one microversion bump per release, that could simplify things a lot.19:35
jgriffithhemna smcginnis in other words it shouldn't know/care about versions19:35
hemnaI think we are too reliant on keystone as well19:35
smcginnisjgriffith: +2/+a19:35
jgriffithjust and endpoint and let said endpoint figure it out19:35
hemnawe should work w/o keystone for better standalone support19:36
*** dustins has quit IRC19:36
smcginnisShould just be one endpoint, then the client decides where to go from there.19:36
jgriffithit's sort of weird to have all that version disco and stuff in keystone IMHO19:36
smcginnisOr queries the API to find out.19:36
jgriffithsmcginnis +1000019:36
hemnayup19:36
jgriffithanyway, that's a high level view of what I've been trying to get at the last couple times this has come up19:36
jgriffithbetter or worse I don't know; but probably not worth taking much time on19:37
hemnaI think cinder has become way too damn complicated19:37
jgriffithhemna yeah :(19:37
jgriffithhemna complicated is "ok" when you get some real benefit from it19:37
hemnayah19:37
jgriffithbut I'm unclear how much we benefit from the complexity we have introduced19:37
jgriffithanyway... maybe I'll just start my new project before big tent goes away :)19:38
hemna:)19:38
jgriffithcinder-lite19:38
jgriffithor maybe small-block19:38
hemnathe HA stuff in theory is a nice idea, but it's just grossly overcomplicated now.19:38
_alastor_jgriffith: +119:38
smcginnisjgriffith: Hey, that was my name!19:38
smcginnis:P19:38
jgriffithsmcginnis which one?19:38
jgriffithsmall-block?19:39
smcginniscinder-lite19:39
jgriffithor cinder-lite19:39
jgriffithok... I'll credit you :)19:39
smcginnisAlthough now I kind of like paver better. ;)19:39
jgriffithhaha!  paver!19:39
hemna:)19:39
jgriffithsmcginnis you probably told me cinder-lite at some point and it's in my subconcious19:39
smcginnis;)19:39
jgriffithanother year or so everybody will be doing bare-metal and kubernetes anyway so it doesn't matter :)19:40
smcginnisOne way or another, I want to find a project to call paver. :)19:40
jgriffithsmcginnis well... I've got a few things in the hopper if you want to join forces to use that name :)19:40
smcginnisjgriffith: I'm game. Just so I can name it. ;)19:40
*** lkuchlan_ has joined #openstack-cinder19:40
openstackgerritJohn Griffith proposed openstack/cinder: Add new attachment APIS  https://review.openstack.org/41463219:41
jgriffithsmcginnis it's good to have the priorities straight in these matters!19:41
openstackgerritJohn Griffith proposed openstack/python-cinderclient: Attach/Detach V2  https://review.openstack.org/38771619:41
jgriffiththose patches should be reviewable now19:44
*** salv-orl_ has quit IRC19:45
*** Alexey_Abashkin has quit IRC19:48
*** AlexeyAbashkin has joined #openstack-cinder19:51
*** Alexey_Abashkin has joined #openstack-cinder19:56
*** AlexeyAbashkin has quit IRC19:57
patrickeastjgriffith: haha, the commentary on the version variable in the patch comment is great19:58
patrickeastjgriffith: i spent like an hour trying to figure out why i couldn't see the volume group options on the cinder cli since we hide them if you don't have the magic version numbers19:59
smcginnispatrickeast: Been there, done that.19:59
patrickeastfinally just updated my rc file to be like http://paste.openstack.org/show/594618/19:59
smcginnispatrickeast: Ooh, not a bad idea.20:00
smcginnisOr our help could just print out everything and tell you what version you need for those that matter. :)20:00
patrickeastsmcginnis: haha, thats the part that made me laugh20:00
patrickeastsmcginnis: because it *does* tell you what version is required, but only after you call the help with the right version20:00
smcginnisha20:01
scottdaThere are patches in-flight to do automatic version discovery for the CLI...20:02
scottdaThat would seem to solve most problems around "What does the server support?"20:02
patrickeastscottda: sweet, i figured it was just a temporary pain, still makes me feel better to complain :D20:03
scottdaBTW, patrickeast Happy New Year!20:03
scottdaWe've discussed whether help should show "just what is supported" or "everything, and the version required". We can always change to the latter, if that's what people want.20:04
smcginnisMy preference is the latter.20:04
scottdasmcginnis: Yes, I've logged your vote :)20:04
smcginnisGood way to discover "oh, I should upgrade my cloud so I can use this!"20:05
smcginnisscottda: Hehe. I'm persistent. ;)20:05
scottdaYeah, but deployers tend NOT to upgrade clouds...20:05
scottdaSeems a recent server had the median at Kilo20:05
scottdaThat's why it's weird to use the latest client. If your cloud is on Kilo...20:05
smcginnisExtra incentive for them to upgrade if their users are asking why they can't get access to certain versions.20:06
scottdaYou'd see all manner of features that aren't supported on the server in the Help Or CLI20:06
smcginnisBut yeah, I see your point.20:06
scottdaI really think the endpoint thing is the biggest SUX20:07
scottdaIF it weren't for that, we could do the "normal API thing", bump major versions for breaking changes.20:07
scottdahttp://<endpoint>/v4/blah...then http://<endpoint>/v5/blah20:07
scottdausers just hit the version they want. That's how most APIs work20:08
patrickeastscottda: thanks! yea i tend to like the "show everything" approach and just give an error when they try to use it20:08
smcginnisscottda: Totally agree on the endpoint thing.20:08
patrickeastthe endpoint stuff is a pain for sure20:08
scottdaBecause of keystone20:09
ildikovscottda: BTW you said you fixed the v3 endpoint locally on the meeting20:10
scottdaAnd microversions are just minor versions. They'd mean less if we bumped for major version / breaking changes.20:10
openstackgerritSean McGinnis proposed openstack/cinder: Mark the sheepdog driver as unsupported  https://review.openstack.org/41907920:10
ildikovscottda: what did you change besides the catalog_info?20:10
scottdahitting http://<url>/v4 would get you everything in v3, plus whatever20:10
scottdaildikov: I said that it works for me locally...20:11
*** Alexey_Abashkin_ has joined #openstack-cinder20:11
scottdaildikov: I just set the nova.conf setting for cinder and it works.20:11
*** knomura has joined #openstack-cinder20:11
ildikovscottda: wow, you're a magician :)20:11
scottdaAnyway, it seems that APIs are important, and I'm a fan of consistency within OpenStack....20:12
ildikovscottda: is that a latest Devstack install?20:12
scottdaildikov: Ha. I'll retest, there might be a regression.20:12
ildikovscottda: +1 for consistency20:12
scottdaildikov: No, this was a while back. I need to re-visit it.20:12
ildikovscottda: ok :)20:12
* scottda looks at today's task queue and resists the desire to go take a walk...20:13
*** Alexey_Abashkin has quit IRC20:13
*** e0ne has joined #openstack-cinder20:14
smcginnisscottda: Make it a run instead so you get back faster.20:15
scottdaha..if I run I might run away.20:15
*** knomura has quit IRC20:15
*** Alexey_Abashkin_ has quit IRC20:20
*** dustins has joined #openstack-cinder20:20
*** dustins has quit IRC20:20
scottdaalright API debaters, I've added the subject to the PTG agenda. smcginnis jgriffith patrickeast20:22
scottdamight just be a Big Giant Rathole20:22
scottdaBTW, smcginnis , How about a session on Ocata retrospective?20:22
scottdaI figure you should lead it. Plus I'm hogging all the sessions already.20:23
smcginnisscottda: At the PTG? As far as a developer retrospective?20:23
scottdasmcginnis: Yeah, kinda a post mortem.20:23
scottdaWe'd often do this at the Summit, but now....20:23
scottdaMight save us from making the same mistakes over and over...seems we always come back to the same things...20:24
scottdabut maybe it's just my mood. Hopefully you are used to ignoring me :)20:25
*** dustins has joined #openstack-cinder20:25
smcginnisscottda: Nah, sounds good. I've added it to the etherpad.20:26
*** lkuchlan_ has quit IRC20:28
*** Alexey_Abashkin_ has joined #openstack-cinder20:28
*** lseki has quit IRC20:30
*** dustins has quit IRC20:31
eharneysmcginnis: would you mind swinging by this patch again?   https://review.openstack.org/#/c/337814/20:40
*** xinli has quit IRC20:40
smcginniseharney: SUre20:40
smcginniseharney: Any idea what's up with the automodule patch of yours? Couldn't figure out why it has infinite recursion.20:40
smcginnisBut I'm no sphynx expert.20:40
eharneysmcginnis: it seems to be triggered by the patch itself... not really an expert on that but i'm guessing because it's specifying an automodule that matches a module that already exists... or the tools are just broken :)20:41
eharneyno idea really... i was considering just removing the automodule line20:41
smcginnisYeah, maybe better to just clear out that whole file.20:41
eharneybut i haven't gone and learned how it all works enough to know what to do with it20:41
*** e0ne has quit IRC20:43
*** asselin_ has joined #openstack-cinder20:44
*** asselin has quit IRC20:46
*** dustins has joined #openstack-cinder20:46
*** dustins_ has joined #openstack-cinder20:48
*** dustins has quit IRC20:49
jonesnsmcginnis: https://bugs.launchpad.net/ossn/+bug/1188189  You had closed this bug for cinder back in 2015, but it looks like it's crept back into several drivers. Any suggestions?20:51
openstackLaunchpad bug 1188189 in OpenStack Security Notes "Some server-side 'SSL' communication fails to check certificates (use of HTTPSConnection)" [High,Fix released] - Assigned to Robert Clark (robert-clark)20:51
*** arch-nemesis has joined #openstack-cinder20:54
*** xyang1 has joined #openstack-cinder20:55
*** alyson_ has quit IRC20:57
openstackgerritJohn Griffith proposed openstack/cinder: Add new attachment APIS  https://review.openstack.org/41463220:59
*** porrua has quit IRC21:00
*** Alexey_Abashkin_ has quit IRC21:00
*** jamielennox|away is now known as jamielennox21:01
patrickeastjonesn: you mean drivers that are not validating certificates when they call api's on their underlying storage devices?21:01
*** lyarwood_ is now known as lyarwood21:02
xyang1patrickeast: jonesn looks like smcginnis closed it because no more drivers uses httplib, instead requests is used21:10
*** jwcroppe has quit IRC21:14
*** Jack_Iv has joined #openstack-cinder21:15
*** diablo_rojo_phon has quit IRC21:20
*** crose has joined #openstack-cinder21:23
*** salv-orlando has joined #openstack-cinder21:25
*** jessegler has joined #openstack-cinder21:25
*** xinli has joined #openstack-cinder21:28
*** lpetrut has joined #openstack-cinder21:33
jungleboyjscottda: Log my vote for showing everything also please.  :-)21:34
jonesnxyang1: six.moves.http_client will still call httplib on 2.7, correct? I believe the qnap driver is using that, among a few others.21:35
xyang1jonesn: that's new, I only searched for httplib21:36
*** lpetrut has quit IRC21:38
xyang1jonesn: httpsconnection is not used, is it a problem using six.moves.http_client21:39
xyang1jonesn: actually HTTPSConnection is used21:41
*** Jack_Iv has quit IRC21:44
*** dave-mccowan has joined #openstack-cinder21:53
*** alonmarx has quit IRC22:00
smcginnisjonesn: Sorry, I had stepped away.22:01
smcginnisjonesn: If you see some other specific instances where you think it's wrong, please file new bugs for those.22:01
*** crose has quit IRC22:02
openstackgerritSamantha Blanco proposed openstack/cinder: Fixes hpelefthandclient AttributeError  https://review.openstack.org/39389422:05
xyang1smcginnis: Hi, I updated that bug with input from jonesn.  is it ok or do you want a new bug?22:13
smcginnisxyang1: I guess that one if fine.22:16
xyang1smcginnis: ok, thanks22:16
*** salv-orl_ has joined #openstack-cinder22:22
*** sdeath has joined #openstack-cinder22:24
sdeathAnyone here?22:25
*** viks has quit IRC22:25
*** salv-orlando has quit IRC22:26
patrickeastusually22:28
*** markvoelker has quit IRC22:29
sdeathQuestion about cinder and nova.  Openstack Newton on RHEL7, basic install, as-recommended by startup guide (one controller, two comp nodes, one storage[LVM/iSCSI]) - Hypervisor report on dashboard appears to be using local disk for free space calculations instead of the volume group on the storage node.  Google doesn't seem to be able to provide any very good definitive (hence IRC).  Any guidance?22:30
*** markvoelker has joined #openstack-cinder22:32
sdeathInstance starts are failing for reasons of "not enough disk space" - original config as detailed in Openstack Newton install guide.  Tried adding "images_type = lvm" and "images_volume_group = cinder-volumes" to Nova on compute nodes.22:33
sdeathNo soap there.22:33
patrickeastsdeath: are you booting the instances from cinder volumes? if not its going to be using local storage on the nova nodes22:34
*** markvoelker has quit IRC22:34
sdeathP:  I am indeed.22:34
patrickeastsdeath: is the volume big enough for the image?22:34
sdeathIf I increase the local disk space to 100GB, none of it is used.22:34
*** markvoelker has joined #openstack-cinder22:34
sdeath... and volumes are created on the storage node.22:34
sdeath(I can see the LVs being created)22:35
smcginnissdeath: There is (or was) an outstanding Nova bug that Cinder storage consumption was being used to calculate local nova-compute host free space.22:35
smcginnisBut the hypervisor report on the dashboard will always show local disk space. That's the hypervisor info.22:35
sdeathI'm assuming it's a configuration issue on my end, at least initially.22:35
sdeathI have "image_upload_use_cinder_backend = True", "cinder_endpoint_template = http://controller/8776/v2/%(tenant_id)s", and "cinder_os_region_name = RegionOne" set.22:37
sdeath... in controller:/etc/glance/glance-api.conf.22:37
*** markvoelker_ has joined #openstack-cinder22:39
*** markvoelker has quit IRC22:39
*** kberger has joined #openstack-cinder22:41
sdeathMy Googling has terminated, so far, in a conclusion that somehow disk_filter is using local_gb as a way to determine whether or not compute nodes meet the criteria to run instances.  Not enough space = that node goes byebye.22:41
sdeathNo nodes = no instances.22:41
sdeathIt doesn't seem to be understanding that the instance is backed by a volume on the storage node, and therefore it need not worry about it.  (It also doesn't appear to be checking current conditions - every time I start an instance, 30GB, let's say, it claws that 30GB out of the compute node's local storage...  without using any of it.  "df" shows no space used on the compute node.  That makes the existence of the storage node22:43
sdeath*useless.)22:43
sdeathAgain, I'm assuming there's a piece of this I'm missing, to tell it "Worry about the status of THE STORAGE NODE OVER HERE -------> [Cinderblock], and forget all about the damn local storage, because we don't care about that anyway".22:44
sdeathIt is also possible that I have lost my mind and/or my reading comprehension, and am missing something directly in front of my face.22:47
sdeath(likely, even)22:47
openstackgerritRakesh Jain proposed openstack/cinder: Autoselection of IBM SVC/Storwize IO group Currently in IBM storwize_SVC driver, user can specify only one IO group per backend definition. With this change, now user can specify a list of IO groups, and at the time of creating the volume, the driver will  https://review.openstack.org/41581622:47
*** catintheroof has quit IRC22:50
smcginnissdeath: No, that sounds exactly like the nova bug I mentioned.22:50
sdeathSMC:  so not a config issue, then.22:50
sdeathOr "quite possibly" not.22:50
*** catintheroof has joined #openstack-cinder22:51
sdeathThe way you said it, it sounded almost the other way around:  it's *not* using the Cinder storage to determine how much space it's got available, so it looks at the local storage, concludes it doesn't have enough, and blows chunks when requested to launch an instance.22:52
*** chlong has quit IRC22:55
*** catintheroof has quit IRC22:55
smcginnissdeath: Sorry, no, it ends up behaving the way you described.22:57
smcginnissdeath: So most likely not a configuration issue on your side.22:57
*** tommylikehu_ has joined #openstack-cinder22:58
sdeathHrm.23:02
sdeathWhat's the obstruction to a fix?23:03
*** raunak has quit IRC23:05
*** tommylikehu_ has quit IRC23:07
*** viks has joined #openstack-cinder23:09
*** dave-mccowan has quit IRC23:11
*** lamt has quit IRC23:11
*** knomura has joined #openstack-cinder23:12
*** knomura has quit IRC23:16
jgriffith@smcginnis +1 on your analysis, just noticed this again last week myself :(23:18
*** raunak has joined #openstack-cinder23:18
jgriffithsdeath smcginnis https://bugs.launchpad.net/nova/+bug/146917923:20
openstackLaunchpad bug 1469179 in OpenStack Compute (nova) "instance.root_gb should be 0 for volume-backed instances" [Medium,In progress] - Assigned to melanie witt (melwitt)23:20
SwansonI think I hit that just yesterday.23:21
jgriffithSwanson oh dear.. on master?23:21
*** arch-nemesis has quit IRC23:22
jgriffithadmittedly I was at least running against Mitaka IIRC23:22
jgriffithguess I should look and see if it was ever fixed, looks like every attempt was abandoned maybe :(23:22
Swansonjgriffith, Pretty sure that system was latest and greatest. Not impossible I hosed something else.23:29
*** markvoelker_ has quit IRC23:32
*** JoseMello has quit IRC23:35
*** markvoelker has joined #openstack-cinder23:40
*** xinli has quit IRC23:41
openstackgerritMerged openstack/cinder: Cascade + force volume delete parameters  https://review.openstack.org/33781423:42
*** sdeath has quit IRC23:47
*** dustins_ has quit IRC23:49
*** salv-orl_ has quit IRC23:52
*** xyang1 has quit IRC23:55

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