*** diga has joined #tacker | 00:00 | |
openstackgerrit | dharmendra kushwaha proposed openstack/tacker: tacker resource and test_api_v2 refactoring https://review.openstack.org/317749 | 00:06 |
---|---|---|
openstackgerrit | dharmendra kushwaha proposed openstack/tacker: tacker test_wsgi.py refactoring https://review.openstack.org/319028 | 00:07 |
*** dkushwaha has quit IRC | 00:23 | |
*** bobh has joined #tacker | 00:29 | |
openstackgerrit | dharmendra kushwaha proposed openstack/tacker: Alarm monitor: All in one https://review.openstack.org/365435 | 00:31 |
*** vishwanathj has joined #tacker | 00:35 | |
vishwanathj | gongysh hi... | 00:37 |
vishwanathj | gongysh 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 IRC | 00:42 | |
gongysh | vishwanathj, hi | 01:19 |
gongysh | I think the functional test should accompany the patch | 01:19 |
*** diga has quit IRC | 01:21 | |
gongysh | Change 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 |
vishwanathj | gongysh 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 |
vishwanathj | gongysh 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 great | 01:26 |
gongysh | vishwanathj, sorry, that is agreed specially by sridhar_ram. | 01:27 |
vishwanathj | gongysh also in my case, my functional test for VNF scaling would be adding on top of https://review.openstack.org/#/c/363573/ | 01:27 |
gongysh | vishwanathj, 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 |
vishwanathj | gongysh 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-support | 01:29 |
gongysh | vishwanathj, yes I don't -1 to the function we added. | 01:29 |
vishwanathj | gongysh for the vnf functional test case, it makes sense to build on top of KanagarajM patchset https://review.openstack.org/#/c/363573/ | 01:30 |
gongysh | vishwanathj, 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 |
vishwanathj | gongysh lets please bring this up at the IRC meeting so that the confusion is cleared | 01:32 |
gongysh | vishwanathj, sure. if you review other project's codes often, you will see this practise. | 01:33 |
gongysh | vishwanathj, can you bring it up to the IRC meeting, since the meeting is midnight in china time? | 01:33 |
gongysh | vishwanathj, it others agree to add test cases (unit and functional) after codes merged, there is no problem with me. | 01:35 |
vishwanathj | gongysh 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 |
vishwanathj | let me add a separate patchset by Tuesday that captures functional test | 01:36 |
gongysh | vishwanathj, you can see the codes changed for just functional tests. | 01:36 |
gongysh | https://review.openstack.org/#/c/363573/ | 01:36 |
gongysh | in 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 dkushwaha | 01:48 | |
gongysh | tung_doan, hi | 01:49 |
gongysh | how to set up and environment so that I can verify you patch? | 01:50 |
tung_doan | Hi gogysh, If you try with devstack, please enable ceilometer and aodh service in local.conf | 01:53 |
tung_doan | gogysh, it had a minor bug in my code and i need to fix it today, thanks | 01:54 |
gongysh | can you give me the exact enable_service statements for ceilometer and aodh? | 01:58 |
tung_doan | sure | 02:00 |
tung_doan | CEILOMETER_BACKEND=mongodb | 02:01 |
tung_doan | enable_plugin ceilometer https://git.openstack.org/openstack/ceilometer | 02:01 |
tung_doan | enable_plugin ceilometer https://git.openstack.org/openstack/ceilometer | 02:02 |
tung_doan | #please add these lines in local.conf. thanks gonggysh | 02:02 |
tung_doan | oh .sorry. the last line is: enable_plugin aodh https://git.openstack.org/openstack/aodh | 02:03 |
tung_doan | when you try.. hope you got something like that: http://paste.openstack.org/show/566897/ | 02:04 |
*** bobh has joined #tacker | 02:39 | |
*** vishwanathj is now known as vishwanathj_zzz | 02:47 | |
*** amotoki has joined #tacker | 02:48 | |
*** amotoki has quit IRC | 02:50 | |
openstackgerrit | Tung Doan proposed openstack/tacker: Alarm monitor: All in one https://review.openstack.org/365435 | 02:51 |
*** bobh has quit IRC | 02:59 | |
*** tung_doan has quit IRC | 03:02 | |
*** amotoki has joined #tacker | 03:07 | |
*** janki has joined #tacker | 03:09 | |
*** tung_doan has joined #tacker | 03:11 | |
*** amotoki has quit IRC | 03:18 | |
*** diga has joined #tacker | 03:51 | |
*** janki has quit IRC | 03:51 | |
*** amotoki has joined #tacker | 03:54 | |
*** hparekh has joined #tacker | 04:00 | |
*** KanagarajM has joined #tacker | 04:29 | |
openstackgerrit | Merged openstack/tacker: Delete unused pom.xml file from doc directory https://review.openstack.org/361103 | 04:36 |
*** links has joined #tacker | 04:39 | |
diga | trozet: Ping | 04:43 |
*** hparekh_ has joined #tacker | 04:46 | |
*** hparekh has quit IRC | 04:46 | |
*** hparekh has joined #tacker | 04:47 | |
*** hparekh_ has quit IRC | 04:47 | |
*** hparekh_ has joined #tacker | 04:49 | |
*** hparekh has quit IRC | 04:49 | |
openstackgerrit | dharmendra kushwaha proposed openstack/tacker: database_stubs file cleanup https://review.openstack.org/365858 | 04:57 |
gongysh | sridhar_ram, hi | 04:59 |
sridhar_ram | gongysh: hi | 04:59 |
gongysh | sridhar_ram, https://review.openstack.org/#/c/363398/ I got the root cause of ping test failed. | 05:00 |
openstackgerrit | dharmendra kushwaha proposed openstack/tacker: database_stubs file cleanup https://review.openstack.org/365858 | 05:00 |
sridhar_ram | gongysh: that would be awesome, i'm looking up.. still catching up from coming back from vacation | 05:00 |
dkushwaha | sridhar_ram, hi. please review the patch https://review.openstack.org/#/c/349806/ . I hope it is ready to merge | 05:03 |
*** Vijayendra has joined #tacker | 05:04 | |
*** hparekh_ has quit IRC | 05:05 | |
*** hparekh__ has joined #tacker | 05:05 | |
sridhar_ram | gongysh: i suspected user_data, this is interesting.. but why this has started failing recently ? Has anything changed in the neutron side ? | 05:05 |
sridhar_ram | also, user_data worked in other testcases .. https://github.com/openstack/tacker/blob/master/tacker/tests/etc/samples/sample-tosca-vnfd-http-monitor.yaml | 05:07 |
*** hparekh__ has quit IRC | 05:07 | |
sridhar_ram | gongysh: it seems this issue is intermittent ? | 05:07 |
gongysh | sridhar_ram, let me have another look at why the http monitor works | 05:11 |
sridhar_ram | gongysh: sure | 05:12 |
sridhar_ram | dkushwaha: looking up | 05:13 |
dkushwaha | sridhar_ram, Thanks | 05:14 |
sridhar_ram | dkushwaha: can you respond to KanagarajM's comments in that patchset ? | 05:15 |
dkushwaha | sridhar_ram, ok, i will update it quickly | 05:18 |
*** hparekh has joined #tacker | 05:18 | |
openstackgerrit | Tung Doan proposed openstack/tacker: Alarm monitor: All in one https://review.openstack.org/365435 | 05:31 |
*** janki has joined #tacker | 05:45 | |
*** KanagarajM has quit IRC | 05:45 | |
*** KanagarajM has joined #tacker | 05:46 | |
*** KanagarajM has quit IRC | 06:00 | |
*** KanagarajM has joined #tacker | 06:00 | |
janki | gongysh, ping | 06:12 |
gongysh | janki, hi | 06:13 |
janki | gongysh, how are you? | 06:13 |
gongysh | janki, good. thanks for asking | 06:14 |
janki | gongysh, regarding the infra driver being derived from VIM, I couldnot find any place where vim stores this info. | 06:14 |
janki | gongysh, should this be added while registering vim? | 06:15 |
*** KanagarajM has quit IRC | 06:15 | |
*** saju_m has joined #tacker | 06:17 | |
gongysh | janki, we don't know VIM when we create VNFD, right? | 06:19 |
janki | gongysh, yes | 06:20 |
gongysh | janki, so it seems we cannot do it at all. | 06:20 |
janki | gongysh, so are you suggesting that we drop this idea? | 06:20 |
gongysh | janki, yes. | 06:21 |
janki | gongysh, ok. so I will upload the deprecation patch addressing your comments | 06:21 |
gongysh | janki, ok | 06:22 |
*** KanagarajM has joined #tacker | 06:22 | |
*** KanagarajM has quit IRC | 06:26 | |
*** KanagarajM has joined #tacker | 06:28 | |
*** hparekh has quit IRC | 06:32 | |
*** hparekh has joined #tacker | 06:32 | |
openstackgerrit | Janki Chhatbar proposed openstack/tacker: Deprecate warning for infra_driver and mgmt_driver at server https://review.openstack.org/363455 | 06:37 |
*** Ravikiran_K has joined #tacker | 07:50 | |
gongysh | sridhar_ram, hi | 08:03 |
gongysh | should 'Onboard VNF ' on horizon be changed into 'Onboard VNFD'? | 08:04 |
gongysh | janki, hi | 08:10 |
diga | trozet: let me know once you are online | 08:12 |
*** hparekh has quit IRC | 08:21 | |
*** hparekh has joined #tacker | 08:21 | |
*** gongysh has quit IRC | 08:25 | |
*** gongysh has joined #tacker | 08:26 | |
gongysh | sridhar_ram, hi | 08:44 |
gongysh | sridhar_ram, http monitor and ping monitor does not work the same. | 08:44 |
gongysh | sridhar_ram, ping monitor respawn if it cannot ping. http monitor failed if it cannot http get. | 08:45 |
gongysh | sridhar_ram, but after booting, we can default ping, the user data is used to fail the ping. | 08:46 |
gongysh | sridhar_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 |
gongysh | sridhar_ram, without user data, the two cases in fact are failed. but the http monitor case failed to detect it. | 08:50 |
gongysh | sridhar_ram, all in all, vnf 'active' does not mean 'http server is active'. | 08:50 |
openstackgerrit | dharmendra kushwaha proposed openstack/python-tackerclient: Add support for multi delete https://review.openstack.org/349806 | 08:51 |
janki | gongysh, hey | 08:59 |
gongysh | janki, hi | 09:00 |
janki | gongysh, hi | 09:00 |
openstackgerrit | dharmendra kushwaha proposed openstack/python-tackerclient: Add support for multi delete https://review.openstack.org/349806 | 09:03 |
janki | gongysh, you had pinged me earlier... | 09:11 |
gongysh | janki, yes, but forgot what I want to say. sorry. | 09:12 |
janki | gongysh, no issue :) | 09:12 |
janki | gongysh, 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 specified | 09:13 |
janki | gongysh, since it is hard coded on client. so removed the exception | 09:13 |
gongysh | janki, if you are using curl -X POST, and not pass in infra_driver, the system will fail due to no infra_driver. | 09:14 |
gongysh | janki, so the infra_driver not specified exception still is useful. | 09:15 |
janki | gongysh, ok, will add it. | 09:15 |
*** gongysh has quit IRC | 09:19 | |
*** gongysh has joined #tacker | 09:21 | |
*** mfedosin has joined #tacker | 09:25 | |
*** gongysh has quit IRC | 09:26 | |
openstackgerrit | Janki Chhatbar proposed openstack/tacker: Deprecate warning for infra_driver and mgmt_driver at server https://review.openstack.org/363455 | 09:26 |
*** saju_m has quit IRC | 09:57 | |
dkushwaha | KanagarajM, hi | 10:13 |
dkushwaha | KanagarajM, please check the response in https://review.openstack.org/#/c/314716/ . Thanks | 10:15 |
*** amotoki_ has joined #tacker | 11:29 | |
*** amotoki has quit IRC | 11:31 | |
*** mfedosin has quit IRC | 11:37 | |
*** dkushwaha has quit IRC | 11:40 | |
*** bobh has joined #tacker | 11:46 | |
*** mfedosin has joined #tacker | 11:51 | |
*** achatterjee has joined #tacker | 11:59 | |
*** _achatterjee_ has quit IRC | 12:02 | |
*** bobh has quit IRC | 12:22 | |
openstackgerrit | venkatamahesh proposed openstack/tacker: [Devstack] Nova configuration is deprecated https://review.openstack.org/366094 | 12:38 |
*** vishwanathj_zzz is now known as vishwanathj | 12:57 | |
*** diga has quit IRC | 12:59 | |
*** trozet has quit IRC | 13:04 | |
*** trozet has joined #tacker | 13:19 | |
*** hparekh has quit IRC | 13:20 | |
*** mike_m has quit IRC | 13:23 | |
*** mike_m has joined #tacker | 13:23 | |
openstackgerrit | vishwanath jayaraman proposed openstack/tacker: Logs events for VNF monitoring and actions https://review.openstack.org/361680 | 13:46 |
*** dkushwaha has joined #tacker | 13:51 | |
*** gongysh has joined #tacker | 13:52 | |
*** amotoki_ has quit IRC | 14:09 | |
*** trozet has quit IRC | 14:12 | |
*** trozet has joined #tacker | 14:13 | |
*** amotoki has joined #tacker | 14:17 | |
*** gongysh has quit IRC | 14:20 | |
*** links has quit IRC | 14:21 | |
*** janki has quit IRC | 14:26 | |
*** bobh has joined #tacker | 14:29 | |
*** sripriya has joined #tacker | 14:36 | |
*** amotoki has quit IRC | 14:38 | |
*** sripriya has quit IRC | 14:43 | |
*** KanagarajM has quit IRC | 14:52 | |
*** sripriya has joined #tacker | 14:59 | |
*** amotoki has joined #tacker | 15:11 | |
*** janki has joined #tacker | 15:13 | |
janki | tung_doan, ping | 15:18 |
tung_doan | janki: hi, pong | 15:30 |
janki | tung_doan, I looked into alarm patch. it looks good. a minor comment - posted on gerrit | 15:31 |
tung_doan | janki: oh.. thank :) shot please | 15:31 |
janki | tung_doan, I created a vnf from the sample alarm vnfd | 15:32 |
tung_doan | *shot-->shoot | 15:32 |
tung_doan | janki: yes | 15:32 |
janki | tung_doan, the vm though didnot seem to scale up when cpu usage went above 60% | 15:32 |
janki | tung_doan, I suppose this should not be the expected behavior | 15:32 |
janki | tung_doan, am I missing any steps? | 15:33 |
tung_doan | janki: 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 soon | 15:34 |
janki | tung_doan, I believe this change needs to be reflected in the sample vnfd too | 15:35 |
tung_doan | for scaling up/down, it will need to have a bit modification in heat.py | 15:35 |
trozet | sridhar_ram: ping? | 15:36 |
tung_doan | "needs to be reflected in the sample vnfd".. could you elaborate it? | 15:37 |
janki | https://review.openstack.org/#/c/365435/6/samples/tosca-templates/vnfd/tosca-vnfd-alarm.yaml L57 | 15:37 |
janki | tung_doan, ^^ | 15:37 |
*** prashantD has joined #tacker | 15:38 | |
tung_doan | janki..ah.. right.. thank :) | 15:38 |
janki | tung_doan, resize_compute: vdu1_scaling_out | 15:38 |
janki | tung_doan, I have posted 1 more comment regarding keystone version usage | 15:39 |
tung_doan | janki: if you want to change with repsawning action.. please change "vdu1_scaling_out" to "respawn" | 15:39 |
tung_doan | janki: yes,, regaeding to your comment, i will fix soon | 15:40 |
janki | tung_doan, sure will do, thanks | 15:40 |
tung_doan | janki, thank for testing and looking in my spec :) | 15:40 |
janki | tung_doan, my pleasure :) | 15:41 |
*** sripriya has quit IRC | 15:43 | |
sridhar_ram | trozet: pong | 15:45 |
trozet | sridhar_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 patch | 15:45 |
*** neel has joined #tacker | 15:46 | |
sridhar_ram | trozet: ack.. TOSCA and Unit tests are the major outstanding things ? | 15:46 |
trozet | sridhar_ram: yeah, and top down test | 15:46 |
sridhar_ram | trozet: okay.. i just came back from vacation myself.. will followup on the abstract driver | 15:47 |
trozet | sridhar_ram: going to work on the patch today, post a new revision tonight, then setup client and server and see if it all works | 15:47 |
sridhar_ram | trozet: vnf-resource patchset looks close | 15:47 |
trozet | sridhar_ram: yeah lets get those 2 in | 15:47 |
sridhar_ram | trozet: sounds good | 15:47 |
*** uck has joined #tacker | 15:48 | |
sridhar_ram | janki: ping, are you planning to respin https://review.openstack.org/#/c/340838/ with unit test changes ? | 15:48 |
janki | sridhar_ram, yes. I am modifying both unit and functional tests with more strict asserts | 15:49 |
janki | sridhar_ram, I am facing problem in mocking heat. It doesnot seem to work for me. | 15:50 |
sridhar_ram | janki: sounds good.. this one has higher relative priority than the deprecate patchset... | 15:50 |
sridhar_ram | janki: we can discuss abt the mock issue after the meeting... | 15:51 |
janki | sridhar_ram, I understand. Just need help with mocking heat. tbh has posted a comment for the same. didnot understand :(. waiting for him to get online | 15:51 |
janki | sridhar_ram, sure, will discuss | 15:51 |
openstackgerrit | Merged openstack/tacker: Logs events for VNF monitoring and actions https://review.openstack.org/361680 | 15:51 |
sridhar_ram | janki: sure.. | 15:52 |
openstackgerrit | vishwanath jayaraman proposed openstack/tacker: VNF monitoring event capture: Functional test https://review.openstack.org/366226 | 15:52 |
*** tbh has joined #tacker | 15:52 | |
sridhar_ram | janki: catch him ^^^ :) | 15:54 |
*** uck has quit IRC | 15:55 | |
janki | sridhar_ram, thanks :) | 15:55 |
janki | tbh, ping | 15:55 |
*** KanagarajM has joined #tacker | 15:55 | |
tbh | janki, pong | 15:55 |
janki | tbh, need your help in understanding your comment on unit test for vnf_details about mocking heat | 15:56 |
tbh | janki, sure | 15:56 |
*** uck has joined #tacker | 15:57 | |
tbh | janki, you mean for this https://review.openstack.org/#/c/340838/29/tacker/tests/unit/vm/test_plugin.py at L32? | 15:57 |
janki | tbh, how should I add a condition? | 15:58 |
janki | tbh, yes the same line | 15:58 |
tbh | janki, as we are looking to assert contents of "resources" | 15:59 |
*** manikanta_tadi has joined #tacker | 15:59 | |
janki | tbh, yes... | 16:00 |
*** sripriya has joined #tacker | 16:00 | |
tbh | janki, we need to set return value at that line for get_resource_info | 16:00 |
*** mike_m has quit IRC | 16:01 | |
*** mike_m has joined #tacker | 16:01 | |
*** dandruta has joined #tacker | 16:01 | |
openstackgerrit | vishwanath jayaraman proposed openstack/tacker: Logs events for VNF scale operations https://review.openstack.org/363946 | 16:11 |
*** manikanta_tadi has quit IRC | 16:34 | |
*** diga has joined #tacker | 16:52 | |
mike_m | sridhar_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 IRC | 16:59 | |
sridhar_ram | mike_m: just our normal tacker weekly meeting... | 16:59 |
*** tbh has quit IRC | 16:59 | |
diga | trozet: Hi | 17:00 |
trozet | diga: hi | 17:00 |
mike_m | sridhar_ram: ah ok. I can reach out to caoyue then in the meantime. | 17:00 |
diga | trozet: do you have details of VNFFG ? | 17:00 |
sridhar_ram | mike_m: sounds good | 17:00 |
diga | trozet: want to start testing on it | 17:00 |
trozet | diga: yeah so I need to push a new patch, but the overall flow is you create a VNFFGD, and then create a VNFFG from that | 17:01 |
trozet | diga: then it will invoke the openstack driver to create SFC by talking to neutron networking-sfc | 17:01 |
diga | trozet: okay | 17:02 |
trozet | diga: there is an example VNFFGD in the unit tests | 17:02 |
diga | trozet: okay | 17:02 |
trozet | diga: so in addition to my patch | 17:02 |
trozet | diga: we need | 17:02 |
trozet | https://review.openstack.org/#/c/347563/ | 17:02 |
diga | trozet: will follow that | 17:03 |
diga | trozet: okay | 17:03 |
trozet | https://review.openstack.org/#/c/340838/ | 17:03 |
trozet | diga: https://review.openstack.org/#/c/347568/ | 17:03 |
trozet | diga: but that last patch 347568 needs to be updated by s3wong | 17:03 |
diga | trozet: okay | 17:03 |
trozet | diga: I may update that patch, if s3wong doesn't have time | 17:04 |
*** KanagarajM has quit IRC | 17:04 | |
trozet | diga: 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 together | 17:04 |
trozet | diga: should be within the next 1-2 days | 17:04 |
diga | trozet: okay, I am starting end-to-end testing by early morning tomorrow | 17:04 |
trozet | diga: ok | 17:05 |
diga | trozet: ok, can you mail me on - digambarpat@gmail.com | 17:05 |
diga | trozet: once it is ready | 17:05 |
trozet | diga: yeah I will email you | 17:06 |
diga | trozet: in meantime, I will go through the implementation | 17:06 |
trozet | diga: sure, reviews are also welcome | 17:06 |
diga | trozet: Sure :) | 17:06 |
diga | trozet: will ping you in the morning | 17:07 |
trozet | diga: ok thanks | 17:07 |
diga | Have a nice day! | 17:07 |
*** janki has quit IRC | 17:08 | |
*** vishnoianil has quit IRC | 17:11 | |
*** diga has quit IRC | 17:11 | |
*** uck_ has joined #tacker | 17:19 | |
*** uck has quit IRC | 17:20 | |
*** neel has quit IRC | 17:22 | |
*** egonzalez90 has joined #tacker | 17:31 | |
*** egonzalez90 has left #tacker | 17:33 | |
*** LamT_ has joined #tacker | 17:37 | |
*** amotoki has quit IRC | 17:39 | |
*** Ravikiran_K has quit IRC | 17:40 | |
*** sripriya has joined #tacker | 17:41 | |
openstackgerrit | dharmendra kushwaha proposed openstack/python-tackerclient: Add support for multi delete https://review.openstack.org/349806 | 17:48 |
*** tbh has joined #tacker | 18:01 | |
*** s3wong has joined #tacker | 18:02 | |
*** tbh has quit IRC | 18:04 | |
*** uck_ has quit IRC | 18:11 | |
*** uck has joined #tacker | 18:12 | |
*** tbh has joined #tacker | 18:18 | |
*** tbh has quit IRC | 18:19 | |
*** vishnoianil has joined #tacker | 18:22 | |
*** dandruta has quit IRC | 18:30 | |
*** mfedosin has quit IRC | 18:40 | |
openstackgerrit | dharmendra kushwaha proposed openstack/tacker: Allow vdu (VM) names to be specified as a parameter https://review.openstack.org/363549 | 18:48 |
dkushwaha | sripriya, ping | 18:49 |
dkushwaha | sripriya, kindly review the patch https://review.openstack.org/#/c/363549/ | 18:49 |
sripriya | dkushwaha: ack, will review along with your other patches. thanks for reminding. | 18:50 |
dkushwaha | sripriya, thanks | 18:51 |
trozet | s3wong: ping? | 18:51 |
trozet | s3wong: 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 #tacker | 19:17 | |
*** prashantD has quit IRC | 19:20 | |
*** mbound has joined #tacker | 19:32 | |
*** sripriya has quit IRC | 19:46 | |
*** mike_m has quit IRC | 19:51 | |
*** LamT_ has quit IRC | 19:51 | |
*** mike_m has joined #tacker | 19:52 | |
*** yifei has quit IRC | 20:00 | |
*** yifei has joined #tacker | 20:01 | |
*** bobh has quit IRC | 20:01 | |
*** sripriya has joined #tacker | 20:03 | |
openstackgerrit | vishwanath jayaraman proposed openstack/tacker: VNF monitoring event capture: Functional test https://review.openstack.org/366226 | 20:06 |
*** sripriya_ has joined #tacker | 20:07 | |
*** sripriya has quit IRC | 20:08 | |
*** sripriya_ is now known as sripriya | 20:08 | |
*** prashantD has joined #tacker | 20:21 | |
*** prashantD_ has quit IRC | 20:23 | |
*** uck has quit IRC | 20:33 | |
*** uck has joined #tacker | 20:34 | |
*** uck has quit IRC | 20:38 | |
*** bobh has joined #tacker | 20:38 | |
s3wong | trozet: pong | 20:38 |
s3wong | trozet: sorry, was at lunch | 20:38 |
s3wong | trozet: 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 client | 20:40 |
s3wong | trozet: if I finish my day job stuff at work today, my goal is to push a new patch set tonight | 20:40 |
s3wong | trozet: that said, I have NOT refactor the driver to be a mixin of abstract driver + openstack VIM/infra driver | 20:41 |
*** uck has joined #tacker | 20:42 | |
*** s3wong has quit IRC | 20:56 | |
*** bobh has quit IRC | 21:03 | |
*** s3wong has joined #tacker | 21:08 | |
openstackgerrit | vishwanath jayaraman proposed openstack/tacker: VNF monitoring event capture: Functional test https://review.openstack.org/366226 | 21:34 |
*** mbound has quit IRC | 21:40 | |
trozet | s3wong: oh | 21:42 |
trozet | s3wong: can you include that in your change? | 21:42 |
s3wong | trozet: I will do it one step at a time :-) | 21:43 |
trozet | s3wong: ok cool | 21:43 |
s3wong | trozet: I will first post what I have... | 21:43 |
trozet | s3wong: sure I just wanted to add your patch as a dependency and try to test it | 21:43 |
s3wong | trozet: I have NOT looked into refactoring into mixin of abstract driver + openstack infra driver yet... | 21:43 |
trozet | s3wong: it should be pretty easy | 21:44 |
s3wong | trozet: I want to make sure I can work with what you pass in, and that networking-sfc DB reflects the input | 21:44 |
trozet | s3wong: ok np. I'm going to keep working on my side and push a new patch probably tmrw morning | 21:46 |
s3wong | trozet: OK. Cool | 21:46 |
s3wong | trozet: we probably need to coordinate at some point to ensure it works end to end... | 21:46 |
trozet | s3wong: yeah. I'm going ot install it in openstack, and just test the client with the server | 21:47 |
trozet | s3wong: then my plan is to depend on your patch, pull it in, then try the whole thing | 21:47 |
trozet | s3wong: diga is also going to help with that | 21:47 |
s3wong | trozet: sounds good | 21:47 |
s3wong | trozet: that's great | 21:47 |
trozet | s3wong: that's why im anxious for you to move it to the openstack driver | 21:47 |
trozet | s3wong: because my code is invoking expecting it to be there :) | 21:47 |
s3wong | trozet: given the time frame we have, it is nice to have multiple folks working on e2e verification | 21:48 |
trozet | s3wong: yeah i will be PTO next week | 21:48 |
trozet | s3wong: so trying to wrap this up in hte next few days at least my patch | 21:48 |
s3wong | trozet: oh, boy :-( | 21:48 |
trozet | s3wong: hehe I htink I can finish most of it, I'm only working on that the next few days | 21:49 |
s3wong | trozet: hopefully things work out then... time seems really tight | 21:49 |
trozet | s3wong: i can help with your patch as well while im trying to test | 21:50 |
s3wong | trozet: OK --- hopefully I won't be blocking you | 21:50 |
s3wong | trozet: 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 faster | 21:50 |
trozet | s3wong: sure I can help with that | 21:50 |
s3wong | trozet: Thanks! | 21:51 |
trozet | s3wong: np | 21:51 |
*** prashantD_ has joined #tacker | 22:01 | |
*** prashantD has quit IRC | 22:04 | |
sridhar_ram | sripriya: can https://review.openstack.org/#/c/355529/ go in ? seems a minor one | 22:22 |
openstackgerrit | vishwanath jayaraman proposed openstack/tacker: VNF scaling: Functional test https://review.openstack.org/363573 | 22:24 |
sridhar_ram | bryan_att: just checking back, are you blocked w/ anything in getting tacker up and running in models tests? | 22:26 |
sripriya | sridhar_ram: yes, just did. | 22:27 |
sridhar_ram | sripriya: thanks | 22:27 |
openstackgerrit | Merged openstack/tacker-specs: Updates scaling REST API https://review.openstack.org/355529 | 22:31 |
*** sripriya has quit IRC | 22:37 | |
bryan_att | sridhar_ram: yes, still running into strange auth issues | 22:52 |
bryan_att | sridhar_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 #tacker | 22:53 | |
sridhar_ram | bryan_att: are you pulling tacker directly off stable/mitaka ? | 22:54 |
bryan_att | yes | 22:54 |
bryan_att | https://www.irccloud.com/pastebin/txn234VY/ | 22:55 |
bryan_att | these are the roles assigned to the tacker user, who is a member of the services project | 22:55 |
bryan_att | sridhar_ram: here is the console output showing the tacker log https://www.irccloud.com/pastebin/HaQ7IFf6/Tacker_Create_VNF.log | 22:57 |
sripriya | bryan_att: did you use ‘tacker’ user under ‘services’ project to create the default VIM? | 23:03 |
bryan_att | no, I used the admin user | 23:04 |
*** prashantD_ has quit IRC | 23:05 | |
sridhar_ram | bryan_att: can you share the heat logs ? | 23:05 |
bryan_att | sridhar_ram: here is the console output showing the heat log from the VNF create operation https://www.irccloud.com/pastebin/JWW5gj9Z/ | 23:06 |
bryan_att | sridhar_ram: here is the console output showing the other heat logs https://www.irccloud.com/pastebin/xEe2SzDt/ | 23:08 |
sripriya | bryan_att: can i get the output of tacker vim-show <vim_name> as well? | 23:08 |
bryan_att | https://www.irccloud.com/pastebin/E01uFyPs/ | 23:09 |
bryan_att | sridhar_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_att | I will provide them as patch requests once I get this working. | 23:16 |
sripriya | bryan_att: regarding the heatclient error, i suspect the issue to be with the way the heat itself is configured for the admin user | 23:27 |
sripriya | bryan_att: did you follow the link http://docs.openstack.org/mitaka/install-guide-ubuntu/heat-install.html to setup heat? | 23:27 |
bryan_att | sripriya: 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 IRC | 23:28 | |
*** uck_ has joined #tacker | 23:28 | |
bryan_att | sripriya: 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_att | https://www.irccloud.com/pastebin/9fOBSoHS/ | 23:30 |
sripriya | bryan_att: that is because the commands are available as part of keystone v3 | 23:31 |
bryan_att | sripriya: we are not using keystone V3 in OPNFV AFAIK | 23:32 |
sripriya | bryan_att: these commands are hidden if your environment auth_url is set to keystone v2 | 23:32 |
bryan_att | it is | 23:32 |
bryan_att | So does that mean that Keystone V3 is absolutely required in order to use Heat? I did not think so. | 23:35 |
sripriya | bryan_att: not required, but the guide assumes keystone v3 since v2.0 is deprecated | 23:36 |
sripriya | bryan_att: i suspect some issue with the way the heat_admin_user with ‘admin’ role is setup for managing stacks | 23:37 |
bryan_att | OK, so the guide needs to clarify how to do this with V2.0 as well, otherwise it's not very bloody useful | 23:38 |
sripriya | bryan_att: the guide i was referring to was for the heat installation itself using v2.0 and not tacker installation guide | 23:40 |
*** diga has joined #tacker | 23:41 | |
bryan_att | yes, 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 |
sripriya | bryan_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-vim | 23:46 |
bryan_att | well 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 |
sripriya | bryan_att: i would suggest you to try creating a normal stack inside heat just to isolate this problem | 23:48 |
bryan_att | sripriya: 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!