Monday, 2019-04-15

*** jsquare has quit IRC00:01
*** jsquare has joined #openstack-cinder00:02
*** brinzhang has joined #openstack-cinder00:38
*** Liang__ has joined #openstack-cinder00:50
*** Liang__ is now known as LiangFang00:54
*** Dinesh__Bhor has quit IRC01:19
*** Dinesh_Bhor has joined #openstack-cinder01:21
*** alkhodos has quit IRC01:33
*** ZhangYang_ has joined #openstack-cinder01:42
*** masayukig has joined #openstack-cinder01:56
*** masayukig has quit IRC02:10
*** masayukig has joined #openstack-cinder02:10
*** dave-mccowan has joined #openstack-cinder02:14
*** dave-mccowan has quit IRC02:17
*** psachin has joined #openstack-cinder03:29
*** bhagyashris has joined #openstack-cinder04:01
*** tejdeep has joined #openstack-cinder04:02
*** imacdonn has quit IRC04:05
*** anks2k has joined #openstack-cinder04:06
*** imacdonn has joined #openstack-cinder04:06
*** udesale has joined #openstack-cinder04:24
*** arshad777 has joined #openstack-cinder04:35
arshad777I am trying to install multi-node setup for stein release but facing below issues, please refer below link for the details:-04:36
arshad777http://paste.openstack.org/show/749180/04:36
arshad777can anyone suggest me the exact steps to configure devstack  multinode setup, along with controller and compute local.conf files04:37
*** whoami-rajat has joined #openstack-cinder04:37
*** tejdeep has quit IRC04:38
*** tejdeep has joined #openstack-cinder04:41
*** brinzh has joined #openstack-cinder04:57
*** raghavendrat has quit IRC04:58
*** bhagyashris has quit IRC04:58
*** brinzhang has quit IRC05:01
*** tejdeep has quit IRC05:01
*** rcernin has quit IRC05:08
*** rcernin has joined #openstack-cinder05:10
*** lemko has joined #openstack-cinder05:27
*** tkajinam has quit IRC05:28
*** raghavendrat has joined #openstack-cinder05:31
*** tejdeep has joined #openstack-cinder05:33
*** tejdeep has quit IRC05:36
*** anks2k has quit IRC05:45
*** tkajinam has joined #openstack-cinder05:54
*** Luzi has joined #openstack-cinder05:59
arshad777Hi, I tried to install multinode setup on bionic, but etcd service restart on compute node is failing06:11
arshad777even on xenial I am facing etcd service restart issue on compute node.06:11
arshad777After commenting etcd server restart command in stack.sh , I am to install devstack on compute node06:12
arshad777but I am not able to create an instance on controller node, it is throwing valid host not found06:14
*** sapd1_x has joined #openstack-cinder06:17
*** pcaruana has joined #openstack-cinder06:19
*** dpawlik has joined #openstack-cinder06:21
*** sahid has joined #openstack-cinder06:21
*** udesale has quit IRC06:26
*** dpawlik has quit IRC06:27
*** ircuser-1 has quit IRC06:29
*** sahid has quit IRC06:31
*** sahid has joined #openstack-cinder06:31
*** dpawlik has joined #openstack-cinder06:31
*** tejdeep has joined #openstack-cinder06:33
*** tkajinam_ has joined #openstack-cinder06:48
*** e0ne has joined #openstack-cinder06:48
*** e0ne has quit IRC06:49
*** tejdeep has quit IRC06:49
*** tkajinam has quit IRC06:51
*** e0ne has joined #openstack-cinder06:52
*** rcernin has quit IRC07:05
*** e0ne has quit IRC07:12
*** sdinescu has quit IRC07:16
*** udesale has joined #openstack-cinder07:17
*** udesale has quit IRC07:20
*** udesale has joined #openstack-cinder07:21
*** spotz has joined #openstack-cinder07:22
*** udesale has quit IRC07:22
*** udesale has joined #openstack-cinder07:22
*** e0ne has joined #openstack-cinder07:31
*** lemko has quit IRC07:36
*** udesale has quit IRC07:41
*** udesale has joined #openstack-cinder07:43
*** luksky has joined #openstack-cinder07:50
*** e0ne has quit IRC07:52
*** e0ne has joined #openstack-cinder07:57
*** gkadam has joined #openstack-cinder08:04
*** gkadam is now known as gkadam-afk08:06
*** helenafm has joined #openstack-cinder08:14
*** lemko has joined #openstack-cinder08:19
*** tkajinam_ has quit IRC08:20
*** boxiang has quit IRC08:32
*** boxiang has joined #openstack-cinder08:32
openstackgerritRajat Dhasmana proposed openstack/python-cinderclient master: Add 'is_public' support in '--filters' option  https://review.openstack.org/64169808:35
*** gkadam-afk is now known as gkadam08:36
openstackgerritRajat Dhasmana proposed openstack/python-cinderclient master: Add 'is_public' support in '--filters' option  https://review.openstack.org/64169808:36
*** udesale has quit IRC08:40
*** udesale has joined #openstack-cinder08:40
*** sapd1_x has quit IRC08:42
*** sapd1 has quit IRC08:59
*** sahid has quit IRC09:05
*** sahid has joined #openstack-cinder09:06
*** e0ne has quit IRC09:09
*** e0ne has joined #openstack-cinder09:18
*** LiangFang has quit IRC09:38
*** udesale has quit IRC09:54
*** udesale has joined #openstack-cinder09:55
*** e0ne has quit IRC09:55
*** e0ne has joined #openstack-cinder10:06
openstackgerritRajat Dhasmana proposed openstack/cinder-specs master: Untyped vol to default vol type  https://review.openstack.org/65148010:11
*** baojg has joined #openstack-cinder10:16
sahidgeguileo: o/ I'm working on an issue in cinder capacityfilter that you was involved, if you can have a look at it at some point that would be great10:17
sahidhttps://review.openstack.org/#/c/651172/10:17
*** udesale has quit IRC10:25
*** e0ne has quit IRC10:26
*** udesale has joined #openstack-cinder10:26
*** e0ne has joined #openstack-cinder10:35
*** carloss has joined #openstack-cinder10:58
*** e0ne has quit IRC11:10
*** dviroel has joined #openstack-cinder11:30
*** erlon has joined #openstack-cinder11:32
*** rosmaita has joined #openstack-cinder11:36
*** thgcorrea has joined #openstack-cinder11:37
*** luksky has quit IRC11:41
*** sapd1 has joined #openstack-cinder11:44
*** mvkr has joined #openstack-cinder11:48
geguileosahid: I'm on PTO until the 22nd11:58
*** brinzh has quit IRC12:13
*** luksky has joined #openstack-cinder12:14
*** e0ne has joined #openstack-cinder12:16
sahidgeguileo: ack no worries12:29
*** e0ne has quit IRC12:40
*** e0ne has joined #openstack-cinder12:41
*** davidsha has joined #openstack-cinder12:46
*** enriquetaso has joined #openstack-cinder12:46
*** boxiang has quit IRC12:57
*** boxiang has joined #openstack-cinder12:58
*** e0ne has quit IRC12:58
*** udesale has quit IRC12:59
*** udesale has joined #openstack-cinder13:02
*** udesale has quit IRC13:03
*** udesale has joined #openstack-cinder13:03
raghavendratis hemna: online ?13:12
*** e0ne has joined #openstack-cinder13:17
*** lseki has joined #openstack-cinder13:19
*** raghavendrat has quit IRC13:23
*** mriedem has joined #openstack-cinder13:24
*** jroll has quit IRC13:25
*** jroll has joined #openstack-cinder13:26
*** e0ne has quit IRC13:28
*** Liang__ has joined #openstack-cinder13:30
*** e0ne has joined #openstack-cinder13:34
*** eharney has joined #openstack-cinder13:37
*** raghavendrat has joined #openstack-cinder13:38
raghavendratis eharney: online?13:38
eharneyyes13:38
raghavendratwhenever you have some time, can you please have a look at https://review.openstack.org/#/c/634119/13:40
raghavendratif everything looks good, please provide +1 or +213:41
raghavendratappreciate your feedback13:41
eharneywill do13:42
raghavendratthanks13:43
*** udesale has quit IRC14:05
*** e0ne has quit IRC14:06
*** Luzi has quit IRC14:07
openstackgerritPablo Caruana proposed openstack/cinder stable/rocky: NetApp: Return all iSCSI targets-portals  https://review.openstack.org/65118314:08
*** e0ne has joined #openstack-cinder14:21
*** lpetrut has joined #openstack-cinder14:36
whoami-rajateharney: Hi, I've updated the patch to run is_public with --filters option. Can you PTAL ? https://review.openstack.org/#/c/641698/14:39
eharneywhoami-rajat: i think there's still a problem there, just added a comment14:40
whoami-rajateharney: hmm, I think we discussed to not send it in case of all_tenants as it wasnt properly handled. This arg is handled properly on API side and I think also required. But I will go through it again and look for a workaround. Thanks.14:47
eharneywhoami-rajat: oh, i assumed it wasn't required by the API, will look some more14:47
eharneywhoami-rajat: it doesn't seem to be required by the API, i think the client is just doing this wrong currently14:53
*** e0ne has quit IRC15:14
*** markvoelker has joined #openstack-cinder15:20
*** e0ne has joined #openstack-cinder15:23
*** luksky has quit IRC15:27
*** e0ne has quit IRC15:34
*** woojay has joined #openstack-cinder15:38
openstackgerritEric Harney proposed openstack/cinder master: Bump hacking version to 1.1.x  https://review.openstack.org/65270315:46
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax docs - additional information on new and existing functionality  https://review.openstack.org/65155916:03
*** luksky has joined #openstack-cinder16:05
*** lpetrut has quit IRC16:08
openstackgerritEric Harney proposed openstack/os-brick master: Ignore pep8 W503  https://review.openstack.org/65271216:09
*** sahid has quit IRC16:11
*** boxiang has quit IRC16:16
*** boxiang has joined #openstack-cinder16:17
hemna_mornin16:18
openstackgerritSean McGinnis proposed openstack/cinder master: Uncap jsonschema  https://review.openstack.org/65271716:25
*** lemko has quit IRC16:39
*** jmlowe has joined #openstack-cinder16:39
*** psachin has quit IRC16:42
*** jmlowe has quit IRC16:50
*** e0ne has joined #openstack-cinder16:51
*** davidsha has quit IRC16:54
*** boxiang has quit IRC17:10
*** zhubx has joined #openstack-cinder17:10
*** gkadam has quit IRC17:15
*** _erlon_ has joined #openstack-cinder17:25
*** jmlowe has joined #openstack-cinder17:30
whoami-rajateharney: i'm afraid but 'is_public=None' is indeed required. the code here[1] looks like it will execute filters['is_public'] = None but it is actually passing None value inside '_parse_is_public' func which returns True for None value [2]17:47
whoami-rajaton the other hand when we pass is_public=None it treats 'None' as a string  and returns None[3]17:47
whoami-rajat[1] https://github.com/openstack/cinder/blob/master/cinder/api/v2/types.py#L114-L11517:47
whoami-rajat[2] https://github.com/openstack/cinder/blob/master/cinder/api/v2/types.py#L77-L7917:47
whoami-rajat[3] https://github.com/openstack/cinder/blob/master/cinder/api/v2/types.py#L80-L8117:47
eharneywhoami-rajat: hrm.. i modified cinderclient to not send it and it seemed to work for me iirc17:47
hemna_is_public=None required ?!17:48
eharneyi don't think it is17:48
whoami-rajateharney:  returning only public vol types without is_public=None for me17:49
whoami-rajateharney: we are talking about vol types right not group types?17:52
eharneywhoami-rajat: and that's what normally happens if you just list from the API, right?17:52
eharneywhoami-rajat: erm now i'm not sure17:52
whoami-rajateharney:  hmm, the previous default behavior was to show both public and private types when nothing is passed.17:53
eharneyi think i was trying group types17:53
eharneywhoami-rajat: the previous default behavior for the shell client, not the API, right?17:53
whoami-rajateharney: previously is_public=None was the default behavior of the client (since we couldn't pass it automatically used None value)17:55
whoami-rajateharney: i've kept the same17:55
eharneywhoami-rajat: of the shell.17:55
whoami-rajateharney:  yep.17:55
whoami-rajatsee [1] is_public is always None and always appended with query_string17:56
whoami-rajat[1] https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/volume_types.py#L10217:56
eharneyso17:58
eharneywe changed this in https://review.openstack.org/#/c/286382/17:58
eharneyand then changed it again in https://review.openstack.org/#/c/577147/517:58
eharneybecause the first one "broke" osc17:59
*** e0ne has quit IRC18:00
whoami-rajatthe first link is for type-update18:00
eharneyoh, oops, https://review.openstack.org/#/c/552814/5 was the change18:01
whoami-rajatyes, correct18:03
whoami-rajatbut the fix didn't allow users to set is_public again.18:03
whoami-rajatwe hardcoded it somehow18:04
*** zul has joined #openstack-cinder18:05
eharneyi'm skeptical about the assumption that was made in here somewhere that listing types (via the API) should only return public types by default.  why is that?18:05
whoami-rajatin the second patch?18:06
eharneyin the API regardless of what the client does18:07
whoami-rajatfor non-admins it is only public, for admins by default yes it's public so we were sending is_public=None there.18:08
whoami-rajati'm also not sure are we filling gaps of API in client or vice versa.18:08
eharneyi don't follow the part about sending is_public=None, i'm just talking about the API18:10
whoami-rajatthe default of API is returning only public volume types.18:11
eharneyand the question was, why18:11
whoami-rajathmm not sure18:12
eharneyit seems like an odd thing to filter by default18:13
whoami-rajati'm not much familiar with the design implementation but maybe it was agreed upon.18:13
eharneyheh18:14
whoami-rajatbut i agree, by default for admins -> public+private for non-admins ->public makes sense18:14
whoami-rajatalso the thing confusing me is if we set API default to return only public types, why are we forcing the client to show us both?18:15
whoami-rajatboth client default and API default should match probably.18:15
eharneyi'm not sure it's correct that only admins see private types, i think it's supposed to be admins plus projects that have access to it18:16
eharney<insert standard grumbling about this is why we are supposed to have well thought-out specs>18:17
* whoami-rajat confusion intensifies18:17
eharneythe commit message for https://review.openstack.org/#/c/114395/ lead me to the detail about projects18:18
whoami-rajateharney: but here also they've clearly coded only public vol types for non-admins [1]18:21
whoami-rajat[1] https://review.openstack.org/#/c/114395/15/cinder/api/v2/types.py@10818:21
whoami-rajatno check for project here18:21
eharneyuhy18:21
*** ircuser-1 has joined #openstack-cinder18:23
whoami-rajatit quotes 'This feature18:25
whoami-rajatdoes not suggest the introduction of an owner for various reasons, one being18:25
whoami-rajatthat it is impossible to find the original owner of an existing volume type.'18:25
whoami-rajatin the original spec18:25
whoami-rajathttps://review.openstack.org/#/c/105546/4/specs/juno/private-volume-types.rst@3218:25
eharneyit all reads to me like, the current behavior is what it is because someone just happened to code it that way18:26
*** markvoelker has quit IRC18:26
eharneyso that leaves your current patch at either a) just keep doing that or b) figure out what it's supposed to do and fix things up18:27
whoami-rajatb) would require a long chain of discussions, i might add that topic too in the denver ptg etherpad18:31
whoami-rajata) would allow some flexibility for now and we can modify it anytime.18:31
*** tejdeep has joined #openstack-cinder18:31
whoami-rajateharney: https://etherpad.openstack.org/p/cinder-train-ptg-planning18:36
whoami-rajatadded the topic.18:36
eharneywhoami-rajat: great18:39
tejdeepZuul is failing at requirements-check http://logs.openstack.org/65/621465/38/check/requirements-check/90d544a/job-output.txt.gz for the patch https://review.openstack.org/#/c/621465/. This patch got rebased recently and i am able to successfully run tests locally. can some help me with this issue.18:50
eharneytejdeep: i think it's being handled here: https://review.openstack.org/#/c/652717/18:51
hemna_damn, notre dame burns18:53
hemna_remember going to see that at the Paris openstack conf way back when18:54
*** jmlowe has quit IRC18:55
tejdeepeharney Thanks for the update.18:57
*** e0ne has joined #openstack-cinder19:02
smcginnishemna_: It's really bad too.19:04
*** markvoelker has joined #openstack-cinder19:04
*** e0ne has quit IRC19:06
eharneyyeah, was just looking at all that :/19:11
*** eharney has quit IRC19:18
*** helenafm has quit IRC19:41
*** penick has quit IRC19:46
jungleboyjhemna_:  What?20:01
jungleboyjOMG.  That is awful.20:01
openstackgerritSofia Enriquez proposed openstack/cinder-tempest-plugin master: [WIP] Test the base_name  https://review.openstack.org/65277120:01
*** gnufied has joined #openstack-cinder20:02
openstackgerritSean McGinnis proposed openstack/cinder master: Drop use of git.openstack.org  https://review.openstack.org/65277420:06
*** jmlowe has joined #openstack-cinder20:10
*** baojg has quit IRC20:14
hemna_yah pretty terrible20:15
openstackgerritSean McGinnis proposed openstack/cinder master: Uncap jsonschema  https://review.openstack.org/65271720:23
*** pcaruana has quit IRC20:38
*** erlon has quit IRC21:06
openstackgerritSean McGinnis proposed openstack/cinder master: Filters fetching backups for incremental backup  https://review.openstack.org/48472921:20
*** eharney has joined #openstack-cinder21:35
*** mvkr has quit IRC21:38
smcginnisrosmaita: I'm cleaning up some old patches. Since you had some interest in the area, I was wondering if you wanted to pick up https://review.openstack.org/#/c/447975/21:49
smcginnisrosmaita: Just let me know and I will abandon if not.21:49
rosmaitasmcginnis: i'll pick it up21:50
openstackgerritSean McGinnis proposed openstack/cinder master: Fix handling of 'cinder_img_volume_type' image metadata  https://review.openstack.org/57161821:50
smcginnisrosmaita: Awesome, thank you!21:50
*** mvkr has joined #openstack-cinder21:51
*** whoami-rajat has quit IRC21:54
*** mriedem has quit IRC22:14
*** Liang__ has quit IRC22:18
*** Liang__ has joined #openstack-cinder22:19
*** dviroel has quit IRC22:21
*** carloss has quit IRC22:25
*** luksky has quit IRC22:25
*** erlon has joined #openstack-cinder22:49
*** tkajinam has joined #openstack-cinder22:53
*** markvoelker has quit IRC22:57
*** markvoelker has joined #openstack-cinder23:08
*** rcernin has joined #openstack-cinder23:24
openstackgerritSean McGinnis proposed openstack/os-brick master: Provide setting to ignore lvm descriptor leak warnings  https://review.openstack.org/65279723:32
openstackgerritSean McGinnis proposed openstack/cinder master: Remove local_dev from cinder  https://review.openstack.org/26075623:33
*** Liang__ has quit IRC23:36
openstackgerritSean McGinnis proposed openstack/cinder-specs master: Remove backend async migrate spec to Train  https://review.openstack.org/65279923:39
openstackgerritSean McGinnis proposed openstack/cinder-specs master: Remove backend async migrate spec  https://review.openstack.org/65279923:39
*** erlon has quit IRC23:48
*** erlon has joined #openstack-cinder23:50
*** erlon has quit IRC23:57

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