Wednesday, 2016-06-22

« Tuesday, 2016-06-21 Index Thursday, 2016-06-23 »
*** manjong has quit IRC00:03
*** mbound has quit IRC00:07
*** manjong has joined #tacker00:14
*** manjong has quit IRC00:19
*** manjong has joined #tacker00:27
*** manjong has quit IRC00:32
*** manjong has joined #tacker00:43
*** manjong has quit IRC00:47
*** prashantD has quit IRC00:50
*** manjong has joined #tacker00:59
*** manjong has quit IRC01:03
*** mbound has joined #tacker01:08
*** mbound has quit IRC01:13
*** aasmith has joined #tacker01:21
*** prashantD has joined #tacker01:36
*** s3wong has quit IRC01:38
*** bobh has joined #tacker01:39
*** tbh has joined #tacker01:46
openstackgerritKawaguchi Kentaro proposed openstack/python-tackerclient: Change the initial letter to capitals in Tacker CLI help texts  https://review.openstack.org/33259801:57
*** prashantD has quit IRC02:34
*** tbh has quit IRC02:46
*** dkushwaha_ is now known as dkushwaha02:49
*** sripriya has joined #tacker02:49
*** sripriya has quit IRC02:53
*** sripriya has joined #tacker02:53
*** bobh has quit IRC03:06
*** aasmith has quit IRC03:10
sridhar_ramsripriya: ping03:13
sripriyasridhar_ram: pong03:14
sridhar_ramsripriya: regarding the dsvm failures due to mandatory name... do you where things stand ?03:14
sripriyasridhar_ram: tbh pushed a patch earlier today with the fix https://review.openstack.org/#/c/332413/03:17
sridhar_ramsripriya: but that patchset is failing dsvm as well03:17
sridhar_ramsripriya: also, i was wondering how do we got into this situation in first place03:18
sripriyasridhar_ram: yes, and is related to autoflavorcreation test case itself03:18
sripriyasridhar_ram: tbh’s patch last dsvm run was in june 1st week and the mandatory name fix went in afterwards03:18
sripriyasridhar_ram: the jenkins merge will not run func tests again since we have not enabled it and silently merged without complaining03:19
sridhar_ramsripriya: u mean, because we marked dsvm non-voting, it wouldn't get run in the merge queue ?03:20
sripriyasridhar_ram: for your reference https://review.openstack.org/#/c/300822/03:20
sripriyasridhar_ram: jenkins gate does not have dsvm job enabled03:21
sripriyasridhar_ram: we run dsvm only on jenkins check03:21
sridhar_ramlooking up project-config to get this straight..03:22
sripriyasridhar_ram: is it that only voting jobs can be enabled on gate?03:23
*** manjong has joined #tacker03:25
sridhar_rami'm caught up now.. :)03:27
sridhar_ramyes, only voting jobs can in the "gate" queue03:27
sridhar_rami remember infra folks asked me to remove tacker-dsvm as it was marked non-voting03:28
*** vishnoianil has quit IRC03:29
sridhar_ramstill, when manikanta's name patchset shd've failed.. ?03:29
sripriyasridhar_ram: why would it fail?03:30
sripriyasridhar_ram: auto flavor test case just got merged 2 days back03:31
sridhar_ramhere is the sequence as i see..03:33
sridhar_ramtbh's func test merged first .. https://review.openstack.org/#/c/300822/   at Jun 20 6:53 PM03:34
sridhar_rammanikanta's name fix seems to be have squeeze in https://review.openstack.org/#/c/318980/03:35
sridhar_ramwithout re-running dsvm in the check queue... because it was already in the merge queue03:36
sridhar_ramideally manikanta's fix shouldn't have merged as tbh's fix didn't have the name mandatory...03:36
sripriyasridhar_ram: yes, on that particular day there was an intermittent jenkins issue which  did not merge earlier03:37
sridhar_rami've a feeling i'm saying the same thing you said few mins back ? :)03:37
sripriyasridhar_ram: else i think it would have gone earlier itself (manikanta’s)03:38
sridhar_ramyeah, that was due to "POST_SUCCESS" failure.. looks unrelated03:38
sripriyasridhar_ram: right03:38
sridhar_rami want to prevent such incidents.. at source..03:38
sridhar_ram..in the future03:38
sripriyasridhar_ram: make dsvm voting ? :-)03:39
sridhar_ramyep, i was just writing that... !03:39
sridhar_ramthis one looks like a timing quirk..03:39
sripriyasridhar_ram: yes …03:39
sridhar_ramif a recheck happened before +A on mani's fix, it would have re-run check q jobs and it would've failed03:40
sripriyasridhar_ram: thats correct03:40
sridhar_rami guess, it is time to switch dsvm voting and see how it goes...03:40
sridhar_ramanshukch: ping03:41
sripriyasridhar_ram: yes, of late i did not see the intermittent issue on vnf status failing in test cases03:41
sripriyasridhar_ram: i think it is time to zoom in on that issue in gate whenever it occurs03:41
sridhar_ramsripriya: agree03:42
*** tbh has joined #tacker03:46
*** manjong has quit IRC03:47
openstackgerritbharaththiruveedula proposed openstack/tacker: Pass VNF/VNFD name in auto resource creation functional tests  https://review.openstack.org/33241303:49
*** manjong has joined #tacker03:49
*** manjong has quit IRC03:55
openstackgerritKawaguchi Kentaro proposed openstack/python-tackerclient: Change the initial letter to capitals in Tacker CLI help texts  https://review.openstack.org/33259803:55
openstackgerritMerged openstack/tacker-horizon: Updated from global requirements  https://review.openstack.org/33239703:58
openstackgerritMerged openstack/python-tackerclient: Updated from global requirements  https://review.openstack.org/33238203:59
*** anshukch has quit IRC03:59
*** anshukch has joined #tacker03:59
*** sripriya has quit IRC04:04
*** manjong has joined #tacker04:05
*** KanagarajM has joined #tacker04:08
*** manjong has quit IRC04:09
openstackgerritxu-haiwei proposed openstack/python-tackerclient: Fix ext-show command error  https://review.openstack.org/33262004:30
anshukchFor the bug https://bugs.launchpad.net/tacker/+bug/1594495, masking password in db/nfvo/nfvo_db.py:_make_vim_dict before returning response of CRUD operations is the fix. Rather than doing in python-client. Is it correct?05:01
openstackLaunchpad bug 1594495 in tacker "Mask hashed password in VIM response for VIM CRUD APIs" [Undecided,New] - Assigned to Anshu Kumar (anshu-choubey)05:01
*** manjong has joined #tacker05:03
openstackgerritbharaththiruveedula proposed openstack/tacker: Pass VNF/VNFD name in auto resource creation functional tests  https://review.openstack.org/33241305:08
*** neel has joined #tacker05:09
*** manjong has quit IRC05:12
*** manjong has joined #tacker05:14
*** janki has joined #tacker05:26
*** manjong has quit IRC05:34
*** haiwei has joined #tacker05:58
*** amotoki has joined #tacker06:01
openstackgerritKanagaraj Manickam proposed openstack/tacker: Scaling policy API  https://review.openstack.org/33066706:16
*** anshukch has quit IRC06:17
openstackgerritMerged openstack/tacker: Updated from global requirements  https://review.openstack.org/33239606:19
*** sripriya has joined #tacker06:20
openstackgerritgongysh proposed openstack/tacker: Use importutils and service from oslo  https://review.openstack.org/32745806:23
tbhsripriya, Hi06:45
sripriyatbh: hello06:45
tbhsripriya, according to your comments, I couldn't understand why it is not nfv_user credentials06:46
sripriyatbh: it is nfv_user credentials. what is happening in code is once you set the username, password and register the options, the conf object is getting reloaded with values from tacker.conf so it overwrites previously registered06:47
sripriya*values06:47
*** pester has joined #tacker06:47
sripriyatbh: we do not want to initialize tacker client using ‘tacker’ username and ‘service’ tenant credentials06:48
sripriyatbh: we should be using the vim parameters to invoke the client06:49
sripriyatbh: did this answer your question?06:49
tbhsripriya,  yeah got it,  I was thinking how it worked locally  :)06:50
sripriyatbh: yeah, time to fix the base!06:51
tbhsripriya, okay I will read vim parameters from local-vim.yaml06:51
sripriyatbh: thats correct, also use ‘v3’ on auth_url06:51
tbhsure06:51
*** neeldhwaj has joined #tacker06:51
tbhsripriya,  in the same patch?06:52
sripriyatbh: yes06:52
*** anshukch has joined #tacker06:56
*** sripriya has quit IRC06:57
*** manjong has joined #tacker06:58
*** manjong has quit IRC06:59
openstackgerritKanagaraj Manickam proposed openstack/tacker: TOSCA definitions and sample template for scaling support  https://review.openstack.org/32952807:00
openstackgerritKanagaraj Manickam proposed openstack/tacker: Heat driver update for scaling support  https://review.openstack.org/33066607:00
openstackgerritKanagaraj Manickam proposed openstack/tacker: Scaling policy API  https://review.openstack.org/33066707:00
*** neel has quit IRC07:04
*** fxpester has quit IRC07:04
*** Ravikiran_K has joined #tacker07:06
openstackgerritbharaththiruveedula proposed openstack/tacker: Pass VNF/VNFD name in auto resource creation functional tests  https://review.openstack.org/33241307:09
*** vishnoianil has joined #tacker07:14
*** trozet has joined #tacker07:15
*** manjong has joined #tacker07:15
*** manjong has quit IRC07:20
openstackgerritxu-haiwei proposed openstack/python-tackerclient: Fix ext-show command error  https://review.openstack.org/33262007:41
*** trozet has quit IRC07:47
openstackgerritxu-haiwei proposed openstack/python-tackerclient: Fix ext-show command error  https://review.openstack.org/33262007:47
*** trozet has joined #tacker07:50
openstackgerritKanagaraj Manickam proposed openstack/tacker: TOSCA definitions and sample template for scaling support  https://review.openstack.org/32952807:57
openstackgerritKanagaraj Manickam proposed openstack/tacker: Refactor heat driver create() handler  https://review.openstack.org/33066407:57
openstackgerritKanagaraj Manickam proposed openstack/tacker: Correct the reference and actual position in assertEqual  https://review.openstack.org/33066507:57
openstackgerritKanagaraj Manickam proposed openstack/tacker: Heat driver update for scaling support  https://review.openstack.org/33066607:57
openstackgerritKanagaraj Manickam proposed openstack/tacker: Scaling policy API  https://review.openstack.org/33066707:57
openstackgerritKanagaraj Manickam proposed openstack/tacker: Heat driver config updates  https://review.openstack.org/32952707:58
*** trozet has quit IRC07:59
openstackgerritxu-haiwei proposed openstack/python-tackerclient: Fix ext-show command error  https://review.openstack.org/33262008:01
*** zeih has joined #tacker08:18
*** mbound has joined #tacker08:23
*** dmk0202 has joined #tacker08:23
*** aasmith has joined #tacker08:26
*** amotoki has quit IRC08:28
*** zeih has quit IRC08:29
*** aasmith has quit IRC08:32
*** neeldhwaj has quit IRC08:38
*** links has joined #tacker08:39
*** anshukch has quit IRC08:42
*** aasmith has joined #tacker08:46
*** amotoki has joined #tacker08:53
*** trozet has joined #tacker08:54
*** zeih has joined #tacker08:57
*** zeih has quit IRC09:09
*** zeih has joined #tacker09:09
*** homeway has joined #tacker09:14
*** trozet has quit IRC09:15
openstackgerritbharaththiruveedula proposed openstack/tacker: Pass VNF/VNFD name in auto resource creation functional tests  https://review.openstack.org/33241309:21
*** homeway has quit IRC09:23
*** neeldhwaj has joined #tacker09:25
*** anshukch has joined #tacker09:30
*** zeih has quit IRC09:47
openstackgerritbharaththiruveedula proposed openstack/tacker: Pass VNF/VNFD name in auto resource creation functional tests  https://review.openstack.org/33241309:53
*** manjong has joined #tacker09:57
*** manjong has quit IRC10:02
*** uck has joined #tacker10:16
*** zeih has joined #tacker10:17
*** zeih has quit IRC10:28
*** uck has quit IRC10:31
*** DaveJ__ has joined #tacker10:41
*** Ravikiran_K has quit IRC11:09
*** zeih has joined #tacker11:13
*** zeih has quit IRC12:04
*** neeldhwaj has quit IRC12:08
*** zeih has joined #tacker12:13
*** janki has quit IRC12:31
*** zeih has quit IRC12:41
*** zeih has joined #tacker12:49
*** DaveJ__ has quit IRC12:50
*** uck has joined #tacker13:01
*** lamt has joined #tacker13:01
*** links has quit IRC13:09
KanagarajMsridhar_ram, ping13:13
*** zeih has quit IRC13:18
*** bobh__ has joined #tacker13:27
*** zeih has joined #tacker13:27
*** trozet has joined #tacker13:28
*** uck has quit IRC13:31
*** bobh__ has quit IRC13:33
*** KanagarajM has quit IRC13:37
*** bobh_ has joined #tacker13:52
*** uck has joined #tacker13:59
*** zeih has quit IRC14:01
*** zeih has joined #tacker14:23
*** manjong has joined #tacker14:28
*** manjong has quit IRC14:32
*** amotoki has quit IRC14:36
*** gongysh has joined #tacker14:36
gongyshsridhar_ram, hi14:40
*** zeih has quit IRC14:47
*** uck has quit IRC14:48
*** KanagarajM has joined #tacker15:07
*** anshukch has quit IRC15:10
*** KanagarajM has quit IRC15:15
*** tbh has quit IRC15:16
*** gongysh has quit IRC15:22
*** mbound has quit IRC15:27
*** KanagarajM has joined #tacker15:39
*** prashantD has joined #tacker15:39
*** trozet has quit IRC15:43
*** prashantD has quit IRC15:51
*** KanagarajM_ has joined #tacker15:59
*** sripriya has joined #tacker15:59
*** lamt has quit IRC16:01
*** KanagarajM has quit IRC16:02
*** dmk0202 has quit IRC16:02
KanagarajM_sripriya, bobh_ would like to discuss on the CLI part of the scaling. do you have few mins now :)16:07
*** aasmith has quit IRC16:08
sripriyaKanagarajM_: sure i have few mins, heading to office in a little time16:09
*** KanagarajM has joined #tacker16:09
*** KanagarajM_ has quit IRC16:12
sripriyaKanagarajM: ^16:13
*** sripriya has quit IRC16:15
*** links has joined #tacker16:16
*** KanagarajM has quit IRC16:25
*** prashantD has joined #tacker16:27
*** mbound has joined #tacker16:27
*** mbound has quit IRC16:32
*** links has quit IRC16:40
*** tbh has joined #tacker16:42
*** anshukch has joined #tacker17:08
*** gongysh has joined #tacker17:13
*** sripriya has joined #tacker17:35
tbhsripriya, ping17:39
sripriyatbh: pong, i looked in to your latest dsvm results, the issue still exists17:40
sripriyatbh: need to debug a bit further to understand the actual cause of that error17:41
sripriyatbh: did you have any insight?17:41
tbhsripriya, yes and I tried to create heat stack which has only one resource nova flavor17:41
sripriyatbh: hmm i tried that as well and i do see in logs now thw request context is to nfv_user and nfv which has admin role17:42
tbhsripriya, I got the same error "ERROR: You are not authorized to use OS::Nova::Flavor" even using admin user and tenant17:42
sripriyatbh: really? what was the template you used?17:43
sripriyatbh: did you try running the test locally and you see this error?17:43
tbhsripriya, http://paste.openstack.org/show/521249/17:44
tbhsripriya, even locally it is not working now, but with nova client it works17:44
tbhwith same credentials17:44
sripriyatbh: i see, i did try the heat template with both server and flavor resource and it worked fine, i did not try just the flavor create. let me spin this template17:45
tbhsripriya, sure17:45
sripriyatbh: works fine locally on nfv tenant using nfv_usee17:47
sripriya*user17:47
tbhsripriya, hmm, probably I messed up my env :)17:48
sripriyatbh: but we see this issue on jenkins as well :-(17:49
sripriyatbh: let me pull your gerrit and look into the issue closely17:49
tbhsure17:50
*** s3wong has joined #tacker18:04
*** gongysh has quit IRC18:08
tbhsripriya,18:14
tbhsripriya, this change gave introduced the problem https://github.com/openstack/heat/commit/ac86702172ddf01f5bdc3f3cd99d2e32ad9b7024#diff-3f766f2e06014f8bc590cd3a2df8bd4fR218:14
tbhsripriya, rollbacking this change works for me18:14
sripriyatbh: trying to understand this change18:16
sripriyatbh: this change is applicable if current user is not the same as requesting user18:16
tbhsripriya,  I didn't get this, which change you are referring to18:22
tbhsripriya, https://github.com/openstack/heat/commit/ac86702172ddf01f5bdc3f3cd99d2e32ad9b7024#diff-48096e565bde0d938851c82f5a6fc62fR31  ?18:22
sripriyatbh: thats correct18:22
tbhsripriya, but we get req.context.is_admin as false if we have that change18:35
*** mbound has joined #tacker18:39
sripriyatbh: interesting thing is if i invoke the vnfd template through cli, i get the same error even without the recent changes in heat18:43
tbhsripriya, with the user who has admin role?18:44
sripriyatbh: yes18:44
tbhsripriya, that's really interesting :)18:45
sripriyatbh: i do not have these policy changes which they did in heat, but it fails if i do via cli through tacker, works fine if i directly create a stack in heat18:45
*** bobh_ has quit IRC18:52
tbhsripriya, I tried with tacker cli it worked with out those changes18:52
tbhsome code mismatches18:53
sripriyatbh: pulled in the latest changes, able to reproduce the same error18:57
sripriyatbh: https://github.com/openstack/heat/blob/master/etc/heat/policy.json#L218:58
tbhsripriya, yes I thought the project should be "admin", even that is not working18:59
sripriyatbh: it is working fine at my end19:02
sripriyatbh: admin context19:02
sripriyatbh: and admin tenant stack request19:02
tbhsripriya, but not with any user with admin role, right?19:03
sripriyatbh: nope19:04
sripriyatbh: previously this worked for any user with ‘admin’ role19:04
sripriyatbh: now they enforce context_is_admin as ‘admin’ role on ‘admin project only19:04
tbhsripriya, yes, according to that we should get error with user with admin role but not in admin project19:05
sripriyatbh: yes19:05
sripriyatbh: looks like they did have a long discussion on this specific issue :-)19:10
sripriyahttps://review.openstack.org/#/c/316627/19:10
tbhsripriya, https://review.openstack.org/#/c/242232/19:11
sripriyatbh: and they knew this was breaking OS::Nova::Flavor create for non admin project19:11
tbhsripriya, yes19:17
tbhsripriya, I was checking how keystone determines a project is admin project or not19:18
tbhusing "admin_project_name" param19:18
tbhgoing through comments19:18
sripriyatbh: i see19:18
sripriyatbh: from comments, it is suggested that we modify the heat default policy.json file if we need to add custom rules such as handling nova flavor requests, etc19:19
sripriyatbh: so we need to assign ‘admin’ role only on flavor create instead of context_is_admin19:20
tbhsripriya, yeah probably need to change "resource_types:OS::Nova::Flavor": "role:admin"19:20
sripriyatbh: yup19:21
sripriyatbh: our devstack script needs to handle this of modifying flavor rule with just admin19:23
*** bobh_ has joined #tacker19:24
tbhsripriya, can't we make "admin_project_name" in keystone.conf to accept a list of admin projects19:24
tbhjust random thought :)19:24
sripriyatbh: there can be only one admin project :-)19:24
tbhsripriya, yes19:25
sripriyatbh:  i think you should call it a day, it is super late for you :)19:26
tbhsripriya, yeah  I will go now19:27
sripriyatbh: i can try to push a change for this and we can see how it goes19:28
*** tbh has quit IRC19:31
*** bharatht_ has joined #tacker19:31
bharatht_sripriya, systems hanged at the same time :)19:31
*** bharatht_ is now known as tbh19:31
*** tbh has quit IRC19:37
*** tbh has joined #tacker19:38
sripriyatbh: https://bugs.launchpad.net/tacker/+bug/159531119:50
openstackLaunchpad bug 1595311 in tacker "Auto Flavor creation using vnf-create fails with heat's policy rule for is_admin_context" [Critical,Confirmed] - Assigned to Sripriya (sseetha)19:50
tbhsripriya, this patch https://review.openstack.org/#/c/316627/ is supposed to break so many existing  features I guess19:53
sripriyatbh: yup19:53
sripriyatbh: will break all projects which use resources with ‘admin’ roles19:54
sripriyatbh: in heat19:54
tbhsripriya, exactly19:55
sripriyatbh: this is only specific to heat and not creating that resource directly in the particular service19:55
sripriyatbh: affects all user comign via heat19:55
tbhtrue19:55
sripriyatbh: will add you as  a co-author for the bug, thanks for looking into the issue with me.19:56
tbhsripriya, np :)19:56
*** uck has joined #tacker20:02
*** tbh has quit IRC20:15
*** uck has quit IRC20:18
openstackgerritSripriya Seetharam proposed openstack/tacker: Allow flavor creation for non admin projects  https://review.openstack.org/33302120:56
*** anshukch_ has joined #tacker21:22
*** anshukch has quit IRC21:22
openstackgerritSripriya Seetharam proposed openstack/tacker: Allow flavor creation for non admin projects  https://review.openstack.org/33302121:29
openstackgerritSripriya Seetharam proposed openstack/tacker: Allow flavor creation for non admin projects  https://review.openstack.org/33302122:30
openstackgerritSripriya Seetharam proposed openstack/tacker: Allow flavor creation for non admin projects  https://review.openstack.org/33302122:34
*** bobh_ has quit IRC22:45
*** sripriya has quit IRC22:55
*** sripriya has joined #tacker22:59
openstackgerritManikantha Srinivas Tadi proposed openstack/tacker-horizon: Set the max_length of the VNFD/VNF/VIM Name to 255.  https://review.openstack.org/33306123:22
openstackgerritManikantha Srinivas Tadi proposed openstack/tacker-horizon: Register VIM description field modified to be rendered as multi lines.  https://review.openstack.org/33306323:32
openstackgerritxu-haiwei proposed openstack/python-tackerclient: Fix ext-show command error  https://review.openstack.org/33262023:48
*** gongysh has joined #tacker23:50
« Tuesday, 2016-06-21 Index Thursday, 2016-06-23 »

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