Tuesday, 2016-09-06

*** diga has joined #tacker00:00
openstackgerritdharmendra kushwaha proposed openstack/tacker: tacker resource and test_api_v2 refactoring  https://review.openstack.org/31774900:06
openstackgerritdharmendra kushwaha proposed openstack/tacker: tacker test_wsgi.py refactoring  https://review.openstack.org/31902800:07
*** dkushwaha has quit IRC00:23
*** bobh has joined #tacker00:29
openstackgerritdharmendra kushwaha proposed openstack/tacker: Alarm monitor: All in one  https://review.openstack.org/36543500:31
*** vishwanathj has joined #tacker00:35
vishwanathjgongysh hi...00:37
vishwanathjgongysh FYI: The functional test for logging VNF MOnitor and Scale operations will be uploaded after the https://review.openstack.org/#/c/363946/ and https://review.openstack.org/#/c/361680/ are merged.00:39
*** bobh has quit IRC00:42
gongyshvishwanathj, hi01:19
gongyshI think the functional test should accompany the patch01:19
*** diga has quit IRC01:21
gongyshChange should be gated by test cases in the first place. Many times when we write cases, we have to refactor or fix the codes themselves.01:21
vishwanathjgongysh KanagarajM and I followed similar approach for the common services patch and it worked well e.g;  https://review.openstack.org/#/c/358678/ and now https://review.openstack.org/#/c/363573/01:25
vishwanathjgongysh If you do not believe this to be the right approach, do you mind bringing it up in the Tuesday IRC meeting.... that would be great01:26
gongyshvishwanathj, sorry,  that is agreed specially by sridhar_ram.01:27
vishwanathjgongysh also in my case, my functional test for VNF scaling would be adding on top of https://review.openstack.org/#/c/363573/01:27
gongyshvishwanathj,  it is easily to add a functional test in your patch. you add a function, use a function test to verify it. that is a simple process.01:28
vishwanathjgongysh The blueprint for common services plugin captures all the tasks that need to be completed ...see work items section https://blueprints.launchpad.net/tacker/+spec/audit-support01:29
gongyshvishwanathj,  yes I don't -1 to the function we added.01:29
vishwanathjgongysh for the vnf functional test case, it makes sense to build on top of KanagarajM patchset https://review.openstack.org/#/c/363573/01:30
gongyshvishwanathj, I just think we add a function and should add a test to verify it in the same time. there is no need to do it after that patch is merged, which will add risk to revert the merged codes.01:31
vishwanathjgongysh lets please bring this up at the IRC meeting so that the confusion is cleared01:32
gongyshvishwanathj, sure. if you review other project's codes often, you will see this practise.01:33
gongyshvishwanathj, can you bring it up to the IRC meeting, since the meeting is midnight in china time?01:33
gongyshvishwanathj, it others agree to add test cases (unit and functional) after codes merged,  there is no problem with me.01:35
vishwanathjgongysh I dont see a problem with adding functional tests after the code with unit tests have been delivered...e.g; https://review.openstack.org/#/c/358678/ was merged after the code + unit tests were added....how about I put a separate patchset by tomorrow that has only functional tests for https://review.openstack.org/#/c/363946/ ?01:35
vishwanathjlet me add a separate patchset by Tuesday that captures functional test01:36
gongyshvishwanathj, you can see the codes changed for just functional tests.01:36
gongyshhttps://review.openstack.org/#/c/363573/01:36
gongyshin that patch, to test the function, the codes have to be changed. even with it, I cannot verify the feature manually.01:37
*** dkushwaha_away is now known as dkushwaha01:48
gongyshtung_doan, hi01:49
gongyshhow to set up and environment so that I can verify you patch?01:50
tung_doanHi gogysh, If you try with devstack, please enable ceilometer and aodh service in local.conf01:53
tung_doangogysh, it had a minor bug in my code and i need to fix it today, thanks01:54
gongyshcan you give me the exact enable_service statements for ceilometer and aodh?01:58
tung_doansure02:00
tung_doanCEILOMETER_BACKEND=mongodb02:01
tung_doanenable_plugin ceilometer https://git.openstack.org/openstack/ceilometer02:01
tung_doanenable_plugin ceilometer https://git.openstack.org/openstack/ceilometer02:02
tung_doan#please add these lines in local.conf.  thanks gonggysh02:02
tung_doanoh .sorry. the last line is: enable_plugin aodh https://git.openstack.org/openstack/aodh02:03
tung_doanwhen you try.. hope you got something like that: http://paste.openstack.org/show/566897/02:04
*** bobh has joined #tacker02:39
*** vishwanathj is now known as vishwanathj_zzz02:47
*** amotoki has joined #tacker02:48
*** amotoki has quit IRC02:50
openstackgerritTung Doan proposed openstack/tacker: Alarm monitor: All in one  https://review.openstack.org/36543502:51
*** bobh has quit IRC02:59
*** tung_doan has quit IRC03:02
*** amotoki has joined #tacker03:07
*** janki has joined #tacker03:09
*** tung_doan has joined #tacker03:11
*** amotoki has quit IRC03:18
*** diga has joined #tacker03:51
*** janki has quit IRC03:51
*** amotoki has joined #tacker03:54
*** hparekh has joined #tacker04:00
*** KanagarajM has joined #tacker04:29
openstackgerritMerged openstack/tacker: Delete unused pom.xml file from doc directory  https://review.openstack.org/36110304:36
*** links has joined #tacker04:39
digatrozet: Ping04:43
*** hparekh_ has joined #tacker04:46
*** hparekh has quit IRC04:46
*** hparekh has joined #tacker04:47
*** hparekh_ has quit IRC04:47
*** hparekh_ has joined #tacker04:49
*** hparekh has quit IRC04:49
openstackgerritdharmendra kushwaha proposed openstack/tacker: database_stubs file cleanup  https://review.openstack.org/36585804:57
gongyshsridhar_ram, hi04:59
sridhar_ramgongysh: hi04:59
gongyshsridhar_ram,  https://review.openstack.org/#/c/363398/ I got the root cause of ping test failed.05:00
openstackgerritdharmendra kushwaha proposed openstack/tacker: database_stubs file cleanup  https://review.openstack.org/36585805:00
sridhar_ramgongysh: that would be awesome, i'm looking up.. still catching up from coming back from vacation05:00
dkushwahasridhar_ram, hi. please review the patch  https://review.openstack.org/#/c/349806/ . I hope it is ready to merge05:03
*** Vijayendra has joined #tacker05:04
*** hparekh_ has quit IRC05:05
*** hparekh__ has joined #tacker05:05
sridhar_ramgongysh: i suspected user_data, this is interesting.. but why this has started failing recently ?  Has anything changed in the neutron side ?05:05
sridhar_ramalso, user_data worked in other testcases .. https://github.com/openstack/tacker/blob/master/tacker/tests/etc/samples/sample-tosca-vnfd-http-monitor.yaml05:07
*** hparekh__ has quit IRC05:07
sridhar_ramgongysh: it seems this issue is intermittent ?05:07
gongyshsridhar_ram, let me have another look at why the http monitor works05:11
sridhar_ramgongysh: sure05:12
sridhar_ramdkushwaha: looking up05:13
dkushwahasridhar_ram, Thanks05:14
sridhar_ramdkushwaha: can you respond to KanagarajM's comments in that patchset ?05:15
dkushwahasridhar_ram, ok,  i will update it quickly05:18
*** hparekh has joined #tacker05:18
openstackgerritTung Doan proposed openstack/tacker: Alarm monitor: All in one  https://review.openstack.org/36543505:31
*** janki has joined #tacker05:45
*** KanagarajM has quit IRC05:45
*** KanagarajM has joined #tacker05:46
*** KanagarajM has quit IRC06:00
*** KanagarajM has joined #tacker06:00
jankigongysh, ping06:12
gongyshjanki, hi06:13
jankigongysh, how are you?06:13
gongyshjanki, good.  thanks for asking06:14
jankigongysh, regarding the infra driver being derived from VIM, I couldnot find any place where vim stores this info.06:14
jankigongysh, should this be added while registering vim?06:15
*** KanagarajM has quit IRC06:15
*** saju_m has joined #tacker06:17
gongyshjanki,  we don't know VIM when we create VNFD, right?06:19
jankigongysh, yes06:20
gongyshjanki, so it seems we cannot do it at all.06:20
jankigongysh, so are you suggesting that we drop this idea?06:20
gongyshjanki, yes.06:21
jankigongysh, ok. so I will upload the deprecation patch addressing your comments06:21
gongyshjanki,  ok06:22
*** KanagarajM has joined #tacker06:22
*** KanagarajM has quit IRC06:26
*** KanagarajM has joined #tacker06:28
*** hparekh has quit IRC06:32
*** hparekh has joined #tacker06:32
openstackgerritJanki Chhatbar proposed openstack/tacker: Deprecate warning for infra_driver and mgmt_driver at server  https://review.openstack.org/36345506:37
*** Ravikiran_K has joined #tacker07:50
gongyshsridhar_ram, hi08:03
gongyshshould 'Onboard VNF ' on horizon be changed into 'Onboard VNFD'?08:04
gongyshjanki, hi08:10
digatrozet: let me know once you are online08:12
*** hparekh has quit IRC08:21
*** hparekh has joined #tacker08:21
*** gongysh has quit IRC08:25
*** gongysh has joined #tacker08:26
gongyshsridhar_ram, hi08:44
gongyshsridhar_ram, http monitor and ping monitor does not work the same.08:44
gongyshsridhar_ram, ping monitor respawn if it cannot ping. http monitor failed if it cannot http get.08:45
gongyshsridhar_ram, but after booting, we can default ping, the user data is used to fail the ping.08:46
gongyshsridhar_ram in the case of http ping, the VM cannot respond the http request after booting, .i.e when the vnf is active. it needs the user data to start the http and then stop the http server.08:49
gongyshsridhar_ram, without user data, the two cases in fact are failed. but the http monitor case failed to detect it.08:50
gongyshsridhar_ram, all in all, vnf 'active' does not mean 'http server is active'.08:50
openstackgerritdharmendra kushwaha proposed openstack/python-tackerclient: Add support for multi delete  https://review.openstack.org/34980608:51
jankigongysh, hey08:59
gongyshjanki, hi09:00
jankigongysh, hi09:00
openstackgerritdharmendra kushwaha proposed openstack/python-tackerclient: Add support for multi delete  https://review.openstack.org/34980609:03
jankigongysh, you had pinged me earlier...09:11
gongyshjanki, yes, but forgot what I want to say. sorry.09:12
jankigongysh, no issue :)09:12
jankigongysh, regarding your comment on not removing the infra_driver not found exception, IMO there wont be any case (currently) that infra driver will not be specified09:13
jankigongysh, since it is hard coded on client. so removed the exception09:13
gongyshjanki, if you are using curl -X POST, and not pass in infra_driver, the system will fail due to no infra_driver.09:14
gongyshjanki, so the infra_driver not specified exception still is useful.09:15
jankigongysh, ok, will add it.09:15
*** gongysh has quit IRC09:19
*** gongysh has joined #tacker09:21
*** mfedosin has joined #tacker09:25
*** gongysh has quit IRC09:26
openstackgerritJanki Chhatbar proposed openstack/tacker: Deprecate warning for infra_driver and mgmt_driver at server  https://review.openstack.org/36345509:26
*** saju_m has quit IRC09:57
dkushwahaKanagarajM, hi10:13
dkushwahaKanagarajM, please check the response in https://review.openstack.org/#/c/314716/ . Thanks10:15
*** amotoki_ has joined #tacker11:29
*** amotoki has quit IRC11:31
*** mfedosin has quit IRC11:37
*** dkushwaha has quit IRC11:40
*** bobh has joined #tacker11:46
*** mfedosin has joined #tacker11:51
*** achatterjee has joined #tacker11:59
*** _achatterjee_ has quit IRC12:02
*** bobh has quit IRC12:22
openstackgerritvenkatamahesh proposed openstack/tacker: [Devstack] Nova configuration is deprecated  https://review.openstack.org/36609412:38
*** vishwanathj_zzz is now known as vishwanathj12:57
*** diga has quit IRC12:59
*** trozet has quit IRC13:04
*** trozet has joined #tacker13:19
*** hparekh has quit IRC13:20
*** mike_m has quit IRC13:23
*** mike_m has joined #tacker13:23
openstackgerritvishwanath jayaraman proposed openstack/tacker: Logs events for VNF monitoring and actions  https://review.openstack.org/36168013:46
*** dkushwaha has joined #tacker13:51
*** gongysh has joined #tacker13:52
*** amotoki_ has quit IRC14:09
*** trozet has quit IRC14:12
*** trozet has joined #tacker14:13
*** amotoki has joined #tacker14:17
*** gongysh has quit IRC14:20
*** links has quit IRC14:21
*** janki has quit IRC14:26
*** bobh has joined #tacker14:29
*** sripriya has joined #tacker14:36
*** amotoki has quit IRC14:38
*** sripriya has quit IRC14:43
*** KanagarajM has quit IRC14:52
*** sripriya has joined #tacker14:59
*** amotoki has joined #tacker15:11
*** janki has joined #tacker15:13
jankitung_doan, ping15:18
tung_doanjanki: hi, pong15:30
jankitung_doan, I looked into alarm patch. it looks good. a minor comment - posted on gerrit15:31
tung_doanjanki: oh.. thank :) shot please15:31
jankitung_doan, I created a vnf from the sample alarm vnfd15:32
tung_doan*shot-->shoot15:32
tung_doanjanki: yes15:32
jankitung_doan, the vm though didnot seem to scale up when cpu usage went above 60%15:32
jankitung_doan, I suppose this should not be the expected behavior15:32
jankitung_doan, am I missing any steps?15:33
tung_doanjanki: totally right.. please try with repawning actions... I already dealt with Sridhar... scaling up/down can be a follow-on.. But I believe I can do it soon15:34
jankitung_doan, I believe this change needs to be reflected in the sample vnfd too15:35
tung_doanfor scaling up/down, it will need to have a bit modification in heat.py15:35
trozetsridhar_ram: ping?15:36
tung_doan"needs to be reflected in the sample vnfd".. could you elaborate it?15:37
jankihttps://review.openstack.org/#/c/365435/6/samples/tosca-templates/vnfd/tosca-vnfd-alarm.yaml L5715:37
jankitung_doan, ^^15:37
*** prashantD has joined #tacker15:38
tung_doanjanki..ah.. right.. thank :)15:38
jankitung_doan,                   resize_compute: vdu1_scaling_out15:38
jankitung_doan, I have posted 1 more comment regarding keystone version usage15:39
tung_doanjanki: if you want to change with repsawning action.. please change "vdu1_scaling_out" to "respawn"15:39
tung_doanjanki: yes,, regaeding to your comment, i will fix soon15:40
jankitung_doan, sure will do, thanks15:40
tung_doanjanki, thank for testing and looking in my spec :)15:40
jankitung_doan, my pleasure :)15:41
*** sripriya has quit IRC15:43
sridhar_ramtrozet: pong15:45
trozetsridhar_ram: hey heads up im going to be on PTO next week.  I've finished just about everything I had to do for OPNFV release, so next 3 days are going to be solely focused on finishing the tacker patch15:45
*** neel has joined #tacker15:46
sridhar_ramtrozet: ack.. TOSCA and Unit tests are the major outstanding things ?15:46
trozetsridhar_ram: yeah, and top down test15:46
sridhar_ramtrozet: okay.. i just came back from vacation myself.. will followup on the abstract driver15:47
trozetsridhar_ram: going to work on the patch today, post a new revision tonight, then setup client and server and see if it all works15:47
sridhar_ramtrozet: vnf-resource patchset looks close15:47
trozetsridhar_ram: yeah lets get those 2 in15:47
sridhar_ramtrozet: sounds good15:47
*** uck has joined #tacker15:48
sridhar_ramjanki: ping, are you planning to respin https://review.openstack.org/#/c/340838/ with unit test changes ?15:48
jankisridhar_ram, yes. I am modifying both unit and functional tests with more strict asserts15:49
jankisridhar_ram, I am facing problem in mocking heat. It doesnot seem to work for me.15:50
sridhar_ramjanki: sounds good.. this one has higher relative priority than the deprecate patchset...15:50
sridhar_ramjanki: we can discuss abt the mock issue after the meeting...15:51
jankisridhar_ram, I understand. Just need help with mocking heat. tbh has posted a comment for the same. didnot understand :(. waiting for him to get online15:51
jankisridhar_ram, sure, will discuss15:51
openstackgerritMerged openstack/tacker: Logs events for VNF monitoring and actions  https://review.openstack.org/36168015:51
sridhar_ramjanki: sure..15:52
openstackgerritvishwanath jayaraman proposed openstack/tacker: VNF monitoring event capture: Functional test  https://review.openstack.org/36622615:52
*** tbh has joined #tacker15:52
sridhar_ramjanki: catch him ^^^  :)15:54
*** uck has quit IRC15:55
jankisridhar_ram, thanks :)15:55
jankitbh, ping15:55
*** KanagarajM has joined #tacker15:55
tbhjanki, pong15:55
jankitbh, need your help in understanding your comment on unit test for vnf_details about mocking heat15:56
tbhjanki, sure15:56
*** uck has joined #tacker15:57
tbhjanki, you mean for this https://review.openstack.org/#/c/340838/29/tacker/tests/unit/vm/test_plugin.py  at L32?15:57
jankitbh, how should I add a condition?15:58
jankitbh, yes the same line15:58
tbhjanki, as we are looking to assert contents of "resources"15:59
*** manikanta_tadi has joined #tacker15:59
jankitbh, yes...16:00
*** sripriya has joined #tacker16:00
tbhjanki, we need to set return value at that line for get_resource_info16:00
*** mike_m has quit IRC16:01
*** mike_m has joined #tacker16:01
*** dandruta has joined #tacker16:01
openstackgerritvishwanath jayaraman proposed openstack/tacker: Logs events for VNF scale operations  https://review.openstack.org/36394616:11
*** manikanta_tadi has quit IRC16:34
*** diga has joined #tacker16:52
mike_msridhar_ram: with regard to up coming meeting for the EM discussion, which meeting is that, the normal tacker weekly? or a different one?16:59
*** sripriya has quit IRC16:59
sridhar_rammike_m: just our normal tacker weekly meeting...16:59
*** tbh has quit IRC16:59
digatrozet: Hi17:00
trozetdiga: hi17:00
mike_msridhar_ram: ah ok. I can reach out to caoyue then in the meantime.17:00
digatrozet:  do you have details of VNFFG ?17:00
sridhar_rammike_m: sounds good17:00
digatrozet: want to start testing on it17:00
trozetdiga: yeah so I need to push a new patch, but the overall flow is you create a VNFFGD, and then create a VNFFG from that17:01
trozetdiga: then it will invoke the openstack driver to create SFC by talking to neutron networking-sfc17:01
digatrozet: okay17:02
trozetdiga: there is an example VNFFGD in the unit tests17:02
digatrozet: okay17:02
trozetdiga: so in addition to my patch17:02
trozetdiga: we need17:02
trozethttps://review.openstack.org/#/c/347563/17:02
digatrozet: will follow that17:03
digatrozet: okay17:03
trozethttps://review.openstack.org/#/c/340838/17:03
trozetdiga: https://review.openstack.org/#/c/347568/17:03
trozetdiga: but that last patch 347568 needs to be updated by s3wong17:03
digatrozet: okay17:03
trozetdiga: I may update that patch, if s3wong doesn't have time17:04
*** KanagarajM has quit IRC17:04
trozetdiga: so for now I would recommend holding until I can get those patches updated.  Then will send you an email when it can be tested all together17:04
trozetdiga: should be within the next 1-2 days17:04
digatrozet: okay, I am starting end-to-end testing by early morning tomorrow17:04
trozetdiga: ok17:05
digatrozet: ok, can you mail me on - digambarpat@gmail.com17:05
digatrozet: once it is ready17:05
trozetdiga: yeah I will email you17:06
digatrozet: in meantime, I will go through the implementation17:06
trozetdiga: sure, reviews are also welcome17:06
digatrozet: Sure :)17:06
digatrozet: will ping you in the morning17:07
trozetdiga: ok thanks17:07
digaHave a nice day!17:07
*** janki has quit IRC17:08
*** vishnoianil has quit IRC17:11
*** diga has quit IRC17:11
*** uck_ has joined #tacker17:19
*** uck has quit IRC17:20
*** neel has quit IRC17:22
*** egonzalez90 has joined #tacker17:31
*** egonzalez90 has left #tacker17:33
*** LamT_ has joined #tacker17:37
*** amotoki has quit IRC17:39
*** Ravikiran_K has quit IRC17:40
*** sripriya has joined #tacker17:41
openstackgerritdharmendra kushwaha proposed openstack/python-tackerclient: Add support for multi delete  https://review.openstack.org/34980617:48
*** tbh has joined #tacker18:01
*** s3wong has joined #tacker18:02
*** tbh has quit IRC18:04
*** uck_ has quit IRC18:11
*** uck has joined #tacker18:12
*** tbh has joined #tacker18:18
*** tbh has quit IRC18:19
*** vishnoianil has joined #tacker18:22
*** dandruta has quit IRC18:30
*** mfedosin has quit IRC18:40
openstackgerritdharmendra kushwaha proposed openstack/tacker: Allow vdu (VM) names to be specified as a parameter  https://review.openstack.org/36354918:48
dkushwahasripriya, ping18:49
dkushwahasripriya, kindly review the patch https://review.openstack.org/#/c/363549/18:49
sripriyadkushwaha: ack, will review along with your other patches. thanks for reminding.18:50
dkushwahasripriya, thanks18:51
trozets3wong: ping?18:51
trozets3wong: was wondering if you are going to push a new patch set soon for networking-sfc driver, or fi I can go ahead and refactor it into the openstack driver?18:57
*** prashantD_ has joined #tacker19:17
*** prashantD has quit IRC19:20
*** mbound has joined #tacker19:32
*** sripriya has quit IRC19:46
*** mike_m has quit IRC19:51
*** LamT_ has quit IRC19:51
*** mike_m has joined #tacker19:52
*** yifei has quit IRC20:00
*** yifei has joined #tacker20:01
*** bobh has quit IRC20:01
*** sripriya has joined #tacker20:03
openstackgerritvishwanath jayaraman proposed openstack/tacker: VNF monitoring event capture: Functional test  https://review.openstack.org/36622620:06
*** sripriya_ has joined #tacker20:07
*** sripriya has quit IRC20:08
*** sripriya_ is now known as sripriya20:08
*** prashantD has joined #tacker20:21
*** prashantD_ has quit IRC20:23
*** uck has quit IRC20:33
*** uck has joined #tacker20:34
*** uck has quit IRC20:38
*** bobh has joined #tacker20:38
s3wongtrozet: pong20:38
s3wongtrozet: sorry, was at lunch20:38
s3wongtrozet: I am currently working on the patch that conform to your dictionary keys, and some bugs for the way current version is calling networking-sfc neutron python client20:40
s3wongtrozet: if I finish my day job stuff at work today, my goal is to push a new patch set tonight20:40
s3wongtrozet: that said, I have NOT refactor the driver to be a mixin of abstract driver + openstack VIM/infra driver20:41
*** uck has joined #tacker20:42
*** s3wong has quit IRC20:56
*** bobh has quit IRC21:03
*** s3wong has joined #tacker21:08
openstackgerritvishwanath jayaraman proposed openstack/tacker: VNF monitoring event capture: Functional test  https://review.openstack.org/36622621:34
*** mbound has quit IRC21:40
trozets3wong: oh21:42
trozets3wong: can you include that in your change?21:42
s3wongtrozet: I will do it one step at a time :-)21:43
trozets3wong: ok cool21:43
s3wongtrozet: I will first post what I have...21:43
trozets3wong: sure I just wanted to add your patch as a dependency and try to test it21:43
s3wongtrozet: I have NOT looked into refactoring into mixin of abstract driver + openstack infra driver yet...21:43
trozets3wong: it should be pretty easy21:44
s3wongtrozet: I want to make sure I can work with what you pass in, and that networking-sfc DB reflects the input21:44
trozets3wong: ok np. I'm going to keep working on my side and push a new patch probably tmrw morning21:46
s3wongtrozet: OK. Cool21:46
s3wongtrozet: we probably need to coordinate at some point to ensure it works end to end...21:46
trozets3wong: yeah.  I'm going ot install it in openstack, and just test the client with the server21:47
trozets3wong: then my plan is to depend on your patch, pull it in, then try the whole thing21:47
trozets3wong: diga is also going to help with that21:47
s3wongtrozet: sounds good21:47
s3wongtrozet: that's great21:47
trozets3wong: that's why im anxious for you to move it to the openstack driver21:47
trozets3wong: because my code is invoking expecting it to be there :)21:47
s3wongtrozet: given the time frame we have, it is nice to have multiple folks working on e2e verification21:48
trozets3wong: yeah i will be PTO next week21:48
trozets3wong: so trying to wrap this up in hte next few days at least my patch21:48
s3wongtrozet: oh, boy :-(21:48
trozets3wong: hehe I htink I can finish most of it, I'm only working on that the next few days21:49
s3wongtrozet: hopefully things work out then... time seems really tight21:49
trozets3wong: i can help with your patch as well while im trying to test21:50
s3wongtrozet: OK --- hopefully I won't be blocking you21:50
s3wongtrozet: after I post the patch, if you want, you can do the openstack infra driver mixin change --- I haven't looked into it, so you may be faster21:50
trozets3wong: sure I can help with that21:50
s3wongtrozet: Thanks!21:51
trozets3wong: np21:51
*** prashantD_ has joined #tacker22:01
*** prashantD has quit IRC22:04
sridhar_ramsripriya: can https://review.openstack.org/#/c/355529/ go in ? seems a minor one22:22
openstackgerritvishwanath jayaraman proposed openstack/tacker: VNF scaling: Functional test  https://review.openstack.org/36357322:24
sridhar_rambryan_att: just checking back, are you blocked w/ anything in getting tacker up and running in models tests?22:26
sripriyasridhar_ram: yes, just did.22:27
sridhar_ramsripriya: thanks22:27
openstackgerritMerged openstack/tacker-specs: Updates scaling REST API  https://review.openstack.org/35552922:31
*** sripriya has quit IRC22:37
bryan_attsridhar_ram: yes, still running into strange auth issues22:52
bryan_attsridhar_ram: from the heat log: "2016-09-06 20:09:13.298 1248684 ERROR heat.common.keystoneclient [req-418a4c67-ca32-4141-a77b-5b40f773fb45 - admin - default default] Domain admin client authentication failed"22:52
*** sripriya has joined #tacker22:53
sridhar_rambryan_att: are you pulling tacker directly off stable/mitaka ?22:54
bryan_attyes22:54
bryan_atthttps://www.irccloud.com/pastebin/txn234VY/22:55
bryan_attthese are the roles assigned to the tacker user, who is a member of the services project22:55
bryan_attsridhar_ram: here is the console output showing the tacker log https://www.irccloud.com/pastebin/HaQ7IFf6/Tacker_Create_VNF.log22:57
sripriyabryan_att: did you use ‘tacker’ user under ‘services’ project to create the default VIM?23:03
bryan_attno, I used the admin user23:04
*** prashantD_ has quit IRC23:05
sridhar_rambryan_att: can you share the heat logs ?23:05
bryan_attsridhar_ram: here is the console output showing the heat log from the VNF create operation https://www.irccloud.com/pastebin/JWW5gj9Z/23:06
bryan_attsridhar_ram: here is the console output showing the other heat logs https://www.irccloud.com/pastebin/xEe2SzDt/23:08
sripriyabryan_att: can i get the output of tacker vim-show <vim_name> as well?23:08
bryan_atthttps://www.irccloud.com/pastebin/E01uFyPs/23:09
bryan_attsridhar_ram: sripriya: the script I am using to install tacker is at https://git.opnfv.org/cgit/models/tree/tests/utils/tacker-setup.sh. I am trying to follow the guide at http://docs.openstack.org/developer/tacker/install/manual_installation.html but finding a number of things missing/unclear there.23:15
bryan_attI will provide them as patch requests once I get this working.23:16
sripriyabryan_att: regarding the heatclient error, i suspect the issue to be with the way the heat itself is configured for the admin user23:27
sripriyabryan_att: did you follow the link http://docs.openstack.org/mitaka/install-guide-ubuntu/heat-install.html to setup heat?23:27
bryan_attsripriya: it was installed by a JuJu Charm used in the JOID project. What config aspect would govern the authorization issue?\23:28
*** uck has quit IRC23:28
*** uck_ has joined #tacker23:28
bryan_attsripriya: note that there is no command "openstack domain ..." per http://docs.openstack.org/mitaka/install-guide-ubuntu/heat-install.html step 5 "Orchestration requires additional information in the Identity service to manage stacks. To add this information, complete these steps:"23:30
bryan_atthttps://www.irccloud.com/pastebin/9fOBSoHS/23:30
sripriyabryan_att: that is because the commands are available as part of keystone v323:31
bryan_attsripriya: we are not using keystone V3 in OPNFV AFAIK23:32
sripriyabryan_att: these commands are hidden if your environment auth_url is set to keystone v223:32
bryan_attit is23:32
bryan_attSo does that mean that Keystone V3 is absolutely required in order to use Heat? I did not think so.23:35
sripriyabryan_att: not required, but the guide assumes keystone v3 since v2.0 is deprecated23:36
sripriyabryan_att: i suspect some issue with the way the heat_admin_user with ‘admin’ role is setup for managing stacks23:37
bryan_attOK, so the guide needs to clarify how to do this with V2.0 as well, otherwise it's not very bloody useful23:38
sripriyabryan_att: the guide i was referring to was for the heat installation itself using v2.0 and not tacker installation guide23:40
*** diga has joined #tacker23:41
bryan_attyes, I know - but the heat guys need to clarify how to do the install. But for tacker, why isn't this working? I wold think that the Tacker project would need to document the setup gotchas given keystone V2 use.23:43
sripriyabryan_att: tacker should work for both v2.0 and v3 keystone versions of openstack, and we do specify the additional steps from tacker side that needs to be done if openstack is running v3, refer http://docs.openstack.org/developer/tacker/install/manual_installation.html#registering-default-vim23:46
bryan_attwell I've done everthing that the guide says to do for tacker, but it's not working. Not sure where to turn now.23:47
sripriyabryan_att: i would suggest you to try creating a normal stack inside heat just to isolate this problem23:48
bryan_attsripriya: I used the heat horizon form to create a stack using the translated template, and got the same error (authorization failed).23:56

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