| *** uck has quit IRC | 00:17 | |
| *** bobh has joined #tacker | 00:19 | |
| openstackgerrit | vishwanath jayaraman proposed openstack/tacker: Ensures common services plugin is always loaded https://review.openstack.org/369142 | 00:19 |
|---|---|---|
| openstackgerrit | vishwanath jayaraman proposed openstack/tacker: VNF Scaling event capture: Functional Test https://review.openstack.org/366433 | 00:25 |
| *** sripriya has left #tacker | 00:30 | |
| *** prashantD has quit IRC | 00:31 | |
| *** vishwanathj has quit IRC | 00:43 | |
| *** gongysh has joined #tacker | 00:55 | |
| *** s3wong has quit IRC | 01:31 | |
| *** gongysh has quit IRC | 01:47 | |
| *** vishnoianil has quit IRC | 02:15 | |
| *** KanagarajM has joined #tacker | 02:21 | |
| *** gongysh has joined #tacker | 02:50 | |
| openstackgerrit | Naoya Harada proposed openstack/tacker: Reflect introduction of CP order in documents https://review.openstack.org/370520 | 02:58 |
| *** bobh has quit IRC | 03:00 | |
| *** KanagarajM has quit IRC | 03:18 | |
| *** gongysh has quit IRC | 03:21 | |
| *** gongysh has joined #tacker | 03:25 | |
| gongysh | sridhar_ram, is the vnffg ready for end to end testing? | 03:31 |
| sridhar_ram | gongysh: yes, i think it is close...I'm getting a tosca-parser error but the code looks almost there | 03:32 |
| gongysh | sridhar_ram, do we have a devref for how to get started? | 03:33 |
| sridhar_ram | diga is supposed to write one | 03:34 |
| sridhar_ram | I don't see him in the channel | 03:34 |
| gongysh | n-harada | 03:34 |
| gongysh | hi | 03:34 |
| gongysh | n-harada, regarding CP order, what will happen if the order is not provided? the order is a required? Do we have a validation if two order is the same? | 03:35 |
| *** sripriya has joined #tacker | 03:35 | |
| *** vishwanathj_ has joined #tacker | 03:35 | |
| gongysh | vishwanathj_, hi | 03:36 |
| *** vishwanathj_ has left #tacker | 03:36 | |
| openstackgerrit | Merged openstack/tacker: Add tacker gerrit dashbaord to developer info https://review.openstack.org/369848 | 03:48 |
| openstackgerrit | Sripriya Seetharam proposed openstack/tacker: Derive infra driver using VIM type https://review.openstack.org/367599 | 03:51 |
| *** hparekh has joined #tacker | 03:51 | |
| sripriya | gongysh: please review ^^ | 03:52 |
| gongysh | sripriya, hi | 03:54 |
| gongysh | heat driver and openstack driver seemingly share many codes, Can we let one inherit other? | 03:55 |
| *** KanagarajM has joined #tacker | 03:56 | |
| sripriya | gongysh: makes sense, but i would let them leave as two separate drivers that can be customized independently | 04:12 |
| gongysh | sripriya, I am afraid this will make us maintain them inconsistently, and double efforts. | 04:13 |
| gongysh | sripriya, if we have to add a new function on tosca template, we have to change two files. | 04:14 |
| sripriya | gongysh: we will not be touching the heat driver once we have deprecated it | 04:15 |
| gongysh | sripriya, you are saying 'two separate drivers that can be customized independently'? | 04:16 |
| sripriya | gongysh: well i meant openstack driver specifcally can be customized and need not affect heat workflow | 04:17 |
| gongysh | sripriya, in fact the openstack driver is heat driver, right? | 04:17 |
| sripriya | gongysh: yes and more (when we cherrypick fixes from master to stable) | 04:18 |
| gongysh | sripriya, I strongly consider renaming the current heat driver into openstack, and let the new heat driver to inherit it. | 04:19 |
| gongysh | sripriya, let sridhar_ram to decide. | 04:20 |
| sridhar_ram | sripriya: what is the effort in taking the inheritance route? | 04:20 |
| sridhar_ram | at this point, the window is close soon and this is a must-in for newton | 04:20 |
| sripriya | sridhar_ram: not much | 04:22 |
| sridhar_ram | sripriya: let's take that route then... lot cleaner | 04:22 |
| sripriya | sridhar_ram: that would mean any change affecting openstack will affect heat driver as well | 04:23 |
| sripriya | sridhar_ram: is that okay? | 04:24 |
| sridhar_ram | sripriya: yes, that would be the behavior only in stable/newton and that is okay... | 04:24 |
| sripriya | sridhar_ram: fine | 04:24 |
| sridhar_ram | sripriya: we will remove heat infra driver in ocata anyway | 04:24 |
| sridhar_ram | gongysh: again, we need to wrap up more patchset to close the newton shop.. | 04:27 |
| sridhar_ram | gongysh: can we let this one go .. https://review.openstack.org/#/c/334256/ ? | 04:27 |
| sridhar_ram | gongysh: it indeed as a functional test https://review.openstack.org/#/c/334256/12/tacker/tests/etc/samples/sample-tosca-vnfd.yaml ? | 04:29 |
| sridhar_ram | gongysh: beyond that we are just verifying heat | 04:29 |
| *** sripriya has quit IRC | 04:33 | |
| gongysh | sridhar_ram, done | 04:34 |
| *** sripriya has joined #tacker | 04:34 | |
| gongysh | sridhar_ram, I will have a test on monitor all in one patch, if it can pass, I will +2 on it | 04:35 |
| sridhar_ram | gongysh: great thanks.. | 04:35 |
| sridhar_ram | fyi, i'm going to mark both alarm monitor and vnffg as "Experimental" features so that we set appropriate expectation on our user community | 04:36 |
| gongysh | sridhar_ram, yes, make sense. | 04:39 |
| openstackgerrit | Merged openstack/tacker: Provide support for security groups for ConnectionPoints https://review.openstack.org/334256 | 04:42 |
| *** santoshk has joined #tacker | 04:46 | |
| sridhar_ram | gongysh: https://review.openstack.org/369848 - did u merge this by mistake ? it had only one +2 | 04:48 |
| sridhar_ram | fyi, just found this Tacker CLI reference guide here .. http://docs.openstack.org/cli-reference/tacker.html | 04:51 |
| sridhar_ram | looks it is auto-generated and looks cool !! | 04:52 |
| gongysh | sridhar_ram, sorry, but I think it is really simple. and have check it on the doc and I am using it to pick up reviews. | 04:52 |
| sridhar_ram | gongysh: yes, it is simple.. but please no exceptions to two +2 rule.. it just becomes subjective and it is a slippery slope. Avoid it at all cost. Only exception to this rule is proposal bot changes. | 04:53 |
| gongysh | sridhar_ram, the tacker client reference doc is gorgeous. looking beautiful. | 04:53 |
| *** Vijayendra has joined #tacker | 04:54 | |
| sridhar_ram | gongysh: yep !! | 04:54 |
| gongysh | sridhar_ram, ok, remember it. | 04:54 |
| gongysh | sridhar_ram, no next time. | 04:54 |
| sridhar_ram | gongysh: thanks! | 04:54 |
| sridhar_ram | dkushwaha: ping | 04:55 |
| dkushwaha | sridhar_ram, hello | 04:59 |
| *** santoshk has quit IRC | 05:00 | |
| sridhar_ram | dkushwaha: need ur help, alarm mon patchset https://review.openstack.org/365435 .. just went into conflict due to another merge. if you've a min, can you rebase and respin? | 05:01 |
| dkushwaha | sridhar_ram, yes sure. Let me check, will do quickly | 05:01 |
| sridhar_ram | dkushwaha: great, thanks! | 05:01 |
| gongysh | dkushwaha, don't forget my comment on config group, change the group name into [alarm_keystone] or something to indicate they are used by alarm receiver. | 05:06 |
| dkushwaha | gongysh, yes, Just checked, will fix it | 05:08 |
| *** tung_doan_away is now known as tung_doan | 05:08 | |
| gongysh | dkushwaha, do we integrate it with scale? | 05:08 |
| dkushwaha | gongysh, yes | 05:09 |
| tung_doan | hi gonggysh, | 05:10 |
| tung_doan | hi sridhar, | 05:14 |
| gongysh | dkushwaha, how to have a test, do we have tosca template for it? | 05:15 |
| openstackgerrit | Sripriya Seetharam proposed openstack/tacker: Derive infra driver using VIM type https://review.openstack.org/367599 | 05:16 |
| tung_doan | hi goonggysh, we already had tosca template | 05:16 |
| dkushwaha | gongysh, I have not tested to scaling yet. tung_doan can give better view on this | 05:16 |
| tung_doan | test scaling please you: tosca-vnfd-alarm.yaml | 05:17 |
| gongysh | tung_doan, ok | 05:17 |
| tung_doan | https://review.openstack.org/#/c/365435/27/samples/tosca-templates/vnfd/tosca-vnfd-alarm.yaml | 05:17 |
| *** KanagarajM_ has joined #tacker | 05:21 | |
| *** KanagarajM has quit IRC | 05:21 | |
| sridhar_ram | tung_doan: hi there...! | 05:22 |
| tung_doan | sridhar_ram: gonghysh: alarm monitor worked ok but seems like getexception with scale api in tacker-client: http://paste.openstack.org/show/577214/ | 05:25 |
| tung_doan | sridhar_ram: gonghysh:please se L372: https://review.openstack.org/#/c/365435/27/tacker/extensions/vnfm.py | 05:26 |
| *** vishnoianil has joined #tacker | 05:26 | |
| gongysh | tung_doan, action: | 05:26 |
| gongysh | resize_compute: | 05:26 |
| gongysh | action_name: SP1 | 05:26 |
| tung_doan | sridhar_ram: gonghysh:it means that i need to change restAPI | 05:26 |
| gongysh | why not just resize_compute: SP1 | 05:27 |
| gongysh | I think we also need resize out or in. | 05:27 |
| gongysh | sorry, the type should be passed in automatically. | 05:28 |
| tung_doan | gongysh: alarm monitor support: notification, scaling,... To be general: | 05:28 |
| tung_doan | gongysh: resize_compute: {action_name: <>, constraint: <>} | 05:29 |
| tung_doan | for example, with notificationL resize_compute: {action_name: notify, constraint: abc@gmail} | 05:30 |
| gongysh | for normal respawn: action: respawn | 05:30 |
| tung_doan | "resize out or in" if action is scaling.. base on monitoring policy, will can determine if it 's out or in | 05:31 |
| tung_doan | gongysh: do you have any idea to use only one tosca template format for actions: notify, repsawn, scaling? | 05:34 |
| tung_doan | gongysh: until now, your format is: resize_compute: {action_name: <>, constraint: <>} | 05:35 |
| tung_doan | *your --> the | 05:35 |
| gongysh | tung_doan, - vdu1_cpu_usage_monitoring_policy: | 05:36 |
| gongysh | type: tosca.policies.tacker.Alarming | 05:36 |
| gongysh | triggers: | 05:36 |
| gongysh | resize_compute: | 05:36 |
| gongysh | event_type: | 05:36 |
| gongysh | type: tosca.events.resource.utilization | 05:36 |
| gongysh | implementation: ceilometer | 05:36 |
| gongysh | targets: [VDU1] | 05:36 |
| gongysh | metrics: cpu_util | 05:36 |
| gongysh | condition: | 05:36 |
| gongysh | threshold: 50 | 05:36 |
| gongysh | constraint: utilization greater_than 50% | 05:36 |
| gongysh | period: 600 | 05:36 |
| gongysh | evaluations: 1 | 05:36 |
| gongysh | method: avg | 05:37 |
| gongysh | comparison_operator: gt | 05:37 |
| gongysh | action: | 05:37 |
| gongysh | resize_compute: | 05:37 |
| gongysh | action_name: SP1 | 05:37 |
| gongysh | resize_compute in resize_compute. Have you seen two resize computes? | 05:37 |
| tung_doan | gongysh: correct... need to be changed..do you have any idea? | 05:39 |
| *** neel has joined #tacker | 05:39 | |
| *** sripriya has quit IRC | 05:39 | |
| gongysh | tung_doan, how about this: | 05:43 |
| gongysh | - vdu1_cpu_usage_monitoring_policy: | 05:43 |
| gongysh | type: tosca.policies.tacker.Alarming | 05:43 |
| gongysh | triggers: | 05:43 |
| gongysh | event_type: | 05:43 |
| gongysh | type: tosca.events.resource.utilization | 05:43 |
| gongysh | implementation: ceilometer | 05:43 |
| gongysh | targets: [VDU1] | 05:43 |
| gongysh | metrics: cpu_util | 05:43 |
| gongysh | condition: | 05:43 |
| gongysh | threshold: 50 | 05:43 |
| gongysh | constraint: utilization greater_than 50% | 05:43 |
| gongysh | period: 600 | 05:43 |
| gongysh | evaluations: 1 | 05:43 |
| gongysh | method: avg | 05:43 |
| gongysh | comparison_operator: gt | 05:43 |
| gongysh | action: | 05:43 |
| gongysh | resize_compute: | 05:43 |
| gongysh | action_name: SP1 | 05:43 |
| gongysh | notify: {} | 05:43 |
| gongysh | respawn: | 05:43 |
| gongysh | sorry, have a new idea | 05:43 |
| gongysh | tung_doan: - vdu1_cpu_usage_monitoring_policy: | 05:45 |
| gongysh | type: tosca.policies.tacker.Alarming | 05:45 |
| gongysh | triggers: | 05:45 |
| gongysh | resize_compute | 05:45 |
| gongysh | event_type: | 05:45 |
| gongysh | type: tosca.events.resource.utilization | 05:45 |
| gongysh | implementation: ceilometer | 05:45 |
| gongysh | targets: [VDU1] | 05:45 |
| gongysh | metrics: cpu_util | 05:45 |
| gongysh | condition: | 05:45 |
| gongysh | threshold: 50 | 05:45 |
| gongysh | constraint: utilization greater_than 50% | 05:45 |
| gongysh | period: 600 | 05:45 |
| gongysh | evaluations: 1 | 05:45 |
| gongysh | method: avg | 05:45 |
| gongysh | comparison_operator: gt | 05:45 |
| gongysh | action: | 05:45 |
| gongysh | name: SP1 | 05:45 |
| gongysh | notify: | 05:45 |
| gongysh | event_type: | 05:45 |
| gongysh | type: tosca.events.resource.utilization | 05:45 |
| gongysh | implementation: ceilometer | 05:45 |
| gongysh | targets: [VDU1] | 05:45 |
| gongysh | metrics: cpu_util | 05:45 |
| gongysh | condition: | 05:45 |
| gongysh | threshold: 50 | 05:45 |
| gongysh | constraint: utilization greater_than 50% | 05:45 |
| gongysh | period: 600 | 05:45 |
| gongysh | evaluations: 1 | 05:45 |
| gongysh | method: avg | 05:45 |
| gongysh | comparison_operator: gt | 05:45 |
| gongysh | action: | 05:45 |
| gongysh | name: NF1 | 05:45 |
| gongysh | 05:45 | |
| gongysh | the action under resize_compute trigger should be used to scale targets | 05:45 |
| gongysh | the action under notification trigger should be notification. | 05:46 |
| *** tung_doan has quit IRC | 05:46 | |
| gongysh | :( | 05:46 |
| *** vishwanathj has joined #tacker | 05:47 | |
| vishwanathj | gongysh KanagarajM_ sridhar_ram Hi | 05:49 |
| gongysh | hi | 05:49 |
| sridhar_ram | vishwanathj: hi | 05:49 |
| vishwanathj | regarding https://review.openstack.org/#/c/369142/ what changes should I keep and what should I not keep? | 05:50 |
| *** tung_doan has joined #tacker | 05:51 | |
| vishwanathj | I see gongysh comment suggesting that I dont keep anything in setup.cfg and tacker/common/config.py ... however sridhar_ram suggest that I keep entry point in setup.cfg | 05:51 |
| *** s3wong has joined #tacker | 05:52 | |
| gongysh | that is KanagarajM_'s idea. | 05:53 |
| gongysh | I agree with him | 05:53 |
| vishwanathj | ok | 05:54 |
| gongysh | vishwanathj, how does we load the event plugin currently? | 05:54 |
| vishwanathj | as a service plugin | 05:54 |
| tung_doan | gonggysh.. regarding to tosca template.. looks good to me.. will change now. thanks | 05:54 |
| gongysh | I mean, we have many functional test cases to test the event plugin already, how does it start? | 05:55 |
| *** santoshk has joined #tacker | 05:55 | |
| vishwanathj | its loaded as part of _load_service_plugins() | 05:55 |
| gongysh | tung_doan, hi | 05:55 |
| vishwanathj | but we want to change it to make sure its always loaded | 05:55 |
| gongysh | tung_doan, can heat's trigger trigger more than one actoin? | 05:56 |
| tung_doan | gonggysh: it means that you need to have 2 or more urls for them | 05:57 |
| gongysh | tung_doan, I mean, I create one trigger, which can trigger more than actions, one is scaling, one is notification. | 05:57 |
| tung_doan | gonggysh: only 1 trigger.. yes we can but need ato change code in plugin to handle backend actions :) | 05:59 |
| tung_doan | gonggysh: not difficult | 06:00 |
| tung_doan | gonggysh: did you test scaling? | 06:01 |
| openstackgerrit | dharmendra kushwaha proposed openstack/tacker: Implement Alarm monitor https://review.openstack.org/365435 | 06:02 |
| gongysh | tung_doan, if that is case, we should redesign the tosca template to fit with it. one trigger, more than one actions. we can put off changing backend plugin code later. | 06:03 |
| gongysh | tung_doan, I have tested scale independently. it works, manually. | 06:04 |
| tung_doan | ok.. it like this idea. | 06:04 |
| tung_doan | gonggysh: thanks for testing.. appreciate it | 06:05 |
| tung_doan | *it like --> i like | 06:05 |
| *** vinod has quit IRC | 06:08 | |
| vishwanathj | sridhar_ram you there? | 06:10 |
| tung_doan | gonggysh: with your idea.. | 06:11 |
| *** sripriya has joined #tacker | 06:12 | |
| tung_doan | resize_compute: action_name: SP1 notify: {} respawn: | 06:12 |
| tung_doan | gonggysh: what is behind respawn? | 06:12 |
| *** janki has joined #tacker | 06:17 | |
| gongysh | tung_doan, the respawn should have no other parameter. | 06:26 |
| tung_doan | gonggysh: with notification action, we will need provide email address as well.. | 06:29 |
| tung_doan | gonggysh: with notification action, "action name: NF1" does make sense.. should be replaced by email address? | 06:31 |
| tung_doan | *does make sense --> does not make sense | 06:31 |
| * sridhar_ram wandered back .. | 06:33 | |
| sridhar_ram | vishwanathj: treat setup.cfg entry_point just as an "alias" | 06:33 |
| sridhar_ram | vishwanathj: just go w/ the initial suggestion from gongysh w/ setup.cfg entry | 06:34 |
| vishwanathj | ok ... no core plugin but service plugin right? | 06:34 |
| sridhar_ram | vishwanathj: yes, no core_plugin.. that was counter intuitive / bad idea from my part.. i take it back | 06:34 |
| vishwanathj | sounds good | 06:34 |
| *** sripriya has quit IRC | 06:36 | |
| gongysh | tung_doan, hi | 06:39 |
| gongysh | def handle_url(self, url): | 06:39 |
| gongysh | # alarm_url = 'http://host:port/v1.0/vnfs/vnf-uuid/mon-policy-name/action-name/8ef785' # noqa | 06:39 |
| gongysh | parts = urlparse.urlparse(url) | 06:39 |
| gongysh | p = parts.path.split('/') | 06:39 |
| gongysh | if len(p) != 7: | 06:39 |
| gongysh | return None | 06:39 |
| gongysh | if any((p[0] != '', p[2] != 'vnfs')): | 06:39 |
| gongysh | return None | 06:39 |
| *** neel has quit IRC | 06:39 | |
| gongysh | is it will interface with scale action? I mean the scale action may have the same url pattern | 06:40 |
| gongysh | please check it. | 06:40 |
| openstackgerrit | vishwanath jayaraman proposed openstack/tacker: Ensures common services plugin is always loaded https://review.openstack.org/369142 | 06:41 |
| tung_doan | gonggysh: yes totally samle | 06:41 |
| tung_doan | gonggysh: yes.. totally same | 06:41 |
| gongysh | tung_doan, so we should test the 'mon-policy-name' in the URL. | 06:41 |
| gongysh | maybe other stuff | 06:41 |
| tung_doan | gonggysh: plugin will test if monitoring-policy is not found... | 06:42 |
| gongysh | we should make sure the alarm_receiver.py will not interface with other URL. | 06:43 |
| gongysh | interfere with | 06:43 |
| tung_doan | gonggysh: sure... it's very important to get this | 06:44 |
| tung_doan | gonggysh: I looked back your idea about tosca template.. it seems one trigger with more than one action will be dificult.. I think it better to have 2 or more url for them | 06:49 |
| tung_doan | gonggysh: I mean url for different actions should be separated | 06:50 |
| openstackgerrit | Neeldhwaj Pathak proposed openstack/tacker: Validate the Tosca VNFD templates https://review.openstack.org/333852 | 06:51 |
| *** neel has joined #tacker | 06:51 | |
| n-harada | gongysh: hi, sorry i noticed your message just now. thanks for your new comment, i will think about it. | 06:52 |
| *** diga has joined #tacker | 07:06 | |
| openstackgerrit | Sridhar Ramaswamy proposed openstack/tacker: Add functional tests for ConnectionPoint features https://review.openstack.org/324101 | 07:07 |
| sridhar_ram | gongysh: just touched up the commit mesg of https://review.openstack.org/#/c/324101/ .. can u reapply ur +2 ? | 07:08 |
| gongysh | sure | 07:09 |
| gongysh | sridhar_ram, wait for CI's result | 07:10 |
| diga | s3wong: Hi | 07:11 |
| s3wong | diga: hello | 07:11 |
| gongysh | tung_doan, dkushwaha , will you care about the comment on https://review.openstack.org/#/c/365435/28/tacker/alarm_receiver.py? | 07:11 |
| diga | s3wong: I am starting the e2e testing now, will let you know if I get any problem | 07:13 |
| s3wong | diga: OK | 07:13 |
| *** Ravikiran_K has joined #tacker | 07:13 | |
| s3wong | diga: I may drop off in about 15-30 minutes --- if you run into problem after I drop off, please send me email | 07:13 |
| dkushwaha | gongysh, checking | 07:13 |
| diga | ok | 07:13 |
| s3wong | diga: you have my email address, I suppose? | 07:13 |
| diga | yes | 07:14 |
| diga | s3wong: I have your mail id | 07:14 |
| s3wong | diga: cool. Good luck! | 07:15 |
| diga | s3wong: can you tell me, vnffg create API params | 07:15 |
| s3wong | diga: I actually don't know without looking at the code | 07:16 |
| diga | s3wong: okay | 07:16 |
| diga | will figure out | 07:16 |
| diga | s3wong: Thanks | 07:16 |
| tung_doan | gonggysh: you mean scaling action (just a backend action of alarm monitoing) or scaling API? | 07:17 |
| sridhar_ram | s3wong: diga: what vnffgd template r u using? | 07:17 |
| gongysh | in alarm template, it should be scaling action. | 07:17 |
| s3wong | diga: didn't trozet use a TOSCA template to test? | 07:17 |
| diga | s3wong: he didn't tell me anything, I think there is one example available in test code he has suggested | 07:18 |
| s3wong | diga: probably want to start with his test template then | 07:20 |
| diga | sridhar_ram: we need to check, he hasn't given anything. | 07:20 |
| diga | s3wong: yes | 07:20 |
| diga | sridhar_ram: we need to check, he hasn't given anything. | 07:20 |
| diga | s3wong: yes | 07:20 |
| s3wong | diga: do you know which one? | 07:20 |
| diga | s3wong: No | 07:20 |
| sridhar_ram | diga: s3wong: check this out https://github.com/openstack/tacker/blob/master/tacker/tests/unit/vm/infra_drivers/heat/data/tosca_vnffgd_template.yaml | 07:20 |
| diga | okay | 07:20 |
| diga | sridhar_ram: thanks | 07:21 |
| *** santoshk has quit IRC | 07:21 | |
| tung_doan | gonggysh: if scaling action in template named "SP1", url should be like: http://ubuntu:9890/v1.0/vnfs/d4eb9383-b483-40df-820a-17310ab333bc/vdu1_cpu_usage_monitoring_policy/SP1/3ayy4e6o | 07:21 |
| diga | sridhar_ram: s3wong: today I can spend whole day on this. Hope I will do good progress on this | 07:24 |
| s3wong | diga: cool. Thanks for your effort on this | 07:24 |
| diga | s3wong: np | 07:24 |
| diga | brb | 07:25 |
| sridhar_ram | diga: thanks a lot | 07:26 |
| *** neel has quit IRC | 07:40 | |
| *** gongysh has quit IRC | 07:40 | |
| *** tbh has joined #tacker | 07:48 | |
| *** neel has joined #tacker | 07:53 | |
| *** tbh has quit IRC | 08:02 | |
| openstackgerrit | Merged openstack/tacker: Ensures common services plugin is always loaded https://review.openstack.org/369142 | 08:10 |
| openstackgerrit | Merged openstack/tacker: Use upper constraints for all jobs in tox.ini https://review.openstack.org/358948 | 08:12 |
| *** amotoki has joined #tacker | 08:14 | |
| s3wong | diga: dropping off --- time to sleep | 08:16 |
| s3wong | diga: please email me if you bump into any issue | 08:16 |
| s3wong | diga: good luck. And Thanks for your help | 08:16 |
| *** s3wong has quit IRC | 08:16 | |
| *** Ravikiran_K has quit IRC | 08:17 | |
| *** mbound has joined #tacker | 08:20 | |
| *** neel has quit IRC | 08:32 | |
| *** Vijayendra has quit IRC | 08:42 | |
| *** priya__ has quit IRC | 08:42 | |
| *** priya__ has joined #tacker | 08:42 | |
| *** Vijayendra has joined #tacker | 08:43 | |
| *** Vijayendra has quit IRC | 08:53 | |
| *** Vijayendra has joined #tacker | 08:53 | |
| *** hparekh has quit IRC | 09:01 | |
| *** mbound has quit IRC | 09:12 | |
| *** hparekh has joined #tacker | 09:32 | |
| *** hparekh has quit IRC | 09:32 | |
| *** neel has joined #tacker | 10:02 | |
| openstackgerrit | Naoya Harada proposed openstack/tacker: Reflect introduction of CP order in documents https://review.openstack.org/370520 | 10:09 |
| *** hparekh has joined #tacker | 10:09 | |
| *** gongysh has joined #tacker | 10:10 | |
| *** tbh has joined #tacker | 10:11 | |
| *** mbirru has quit IRC | 10:19 | |
| openstackgerrit | Janki Chhatbar proposed openstack/tacker-horizon: Adds VNFFG support in Tacker-horizon https://review.openstack.org/347779 | 10:55 |
| dkushwaha | gongysh, hi | 11:14 |
| dkushwaha | gongysh, I didn't get your point on https://review.openstack.org/#/c/365435/28/tacker/alarm_receiver.py . Kindly elaborate it | 11:16 |
| gongysh | dkushwaha, I am afraid that will not distinguish the alarm URL with other URLS. | 11:17 |
| dkushwaha | gongysh, got it, so in that case we need to update api. right? | 11:19 |
| dkushwaha | gongysh, i suspect thats why we are getting error in functional test http://logs.openstack.org/35/365435/28/check/gate-tacker-dsvm-functional/fa39fa4/console.html | 11:21 |
| gongysh | dkushwaha, I am investigating too | 11:22 |
| *** tbh has quit IRC | 11:22 | |
| dkushwaha | gongysh, that error was raised by https://github.com/openstack/tacker/blob/master/tacker/api/v1/base.py#L529 | 11:23 |
| dkushwaha | gongysh, You have valid point. Actually we have same url for both alarm and scale i.e. vnfs/vnf-id/actions | 11:30 |
| *** janki has quit IRC | 11:30 | |
| *** janki has joined #tacker | 11:32 | |
| *** janki has quit IRC | 11:39 | |
| tung_doan | gongysh.. no worries.. we already fixed it | 11:40 |
| gongysh | ok | 11:40 |
| *** neel has quit IRC | 11:42 | |
| *** dkushwaha is now known as dkushwaha_away | 11:42 | |
| *** tung_doan has quit IRC | 11:50 | |
| *** bobh has joined #tacker | 12:06 | |
| *** gongysh has quit IRC | 12:12 | |
| openstackgerrit | Tung Doan proposed openstack/tacker: Implement Alarm monitor https://review.openstack.org/365435 | 12:18 |
| *** tung_doan has joined #tacker | 12:22 | |
| *** hparekh has quit IRC | 12:31 | |
| *** bobh has quit IRC | 12:45 | |
| *** tamilhce has joined #tacker | 12:58 | |
| *** dkushwaha has joined #tacker | 13:02 | |
| *** DaveJ__ has joined #tacker | 13:11 | |
| *** DaveJ__ is now known as DaveJ | 13:15 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/tacker: Stop using config option sqlite_db https://review.openstack.org/360413 | 13:17 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/tacker: Fix formats for doc's information https://review.openstack.org/341281 | 13:18 |
| openstackgerrit | dharmendra kushwaha proposed openstack/tacker: Implement Alarm monitor https://review.openstack.org/365435 | 13:24 |
| *** bobh has joined #tacker | 13:28 | |
| *** vishnoianil has quit IRC | 13:29 | |
| *** bobh has quit IRC | 13:34 | |
| *** gongysh has joined #tacker | 13:51 | |
| *** gongysh has quit IRC | 13:55 | |
| *** prashantD has joined #tacker | 13:58 | |
| openstackgerrit | dharmendra kushwaha proposed openstack/tacker: Implement Alarm monitor https://review.openstack.org/365435 | 14:39 |
| *** mbound has joined #tacker | 14:50 | |
| *** tamilhce has quit IRC | 15:05 | |
| *** diga has quit IRC | 15:22 | |
| *** bobh has joined #tacker | 15:32 | |
| *** tbh has joined #tacker | 15:32 | |
| *** bobh has quit IRC | 15:36 | |
| *** uck has joined #tacker | 15:47 | |
| *** bobh has joined #tacker | 15:50 | |
| openstackgerrit | dharmendra kushwaha proposed openstack/tacker: Implement Alarm monitor https://review.openstack.org/365435 | 15:50 |
| *** tbh has quit IRC | 15:52 | |
| *** tbh has joined #tacker | 15:57 | |
| *** tbh_ has joined #tacker | 16:01 | |
| *** tbh_ has quit IRC | 16:02 | |
| *** tbh has quit IRC | 16:03 | |
| *** tbh has joined #tacker | 16:05 | |
| openstackgerrit | Merged openstack/tacker: Add functional tests for ConnectionPoint features https://review.openstack.org/324101 | 16:19 |
| *** tung_doan has quit IRC | 16:28 | |
| *** KanagarajM_ has quit IRC | 16:37 | |
| *** janki has joined #tacker | 16:43 | |
| *** tung_doan has joined #tacker | 16:54 | |
| *** uck_ has joined #tacker | 16:54 | |
| *** uck has quit IRC | 16:55 | |
| janki | sridhar_ram, I have addressed your comments on vnffg horizon support patch. Please review https://review.openstack.org/#/c/347779/ | 16:56 |
| *** vishnoianil has joined #tacker | 16:59 | |
| *** tung_doantung_do has joined #tacker | 17:09 | |
| *** s3wong has joined #tacker | 17:10 | |
| *** tung_doan has quit IRC | 17:11 | |
| *** tung_doantung_do is now known as tung_doan | 17:13 | |
| *** bobh has quit IRC | 17:16 | |
| sridhar_ram | janki: will try, can u also quickly try tosca_vnffgd_template.yaml in horizon ? | 17:18 |
| sridhar_ram | janki: vnffgd catalog create using horizon failed for me | 17:19 |
| janki | sridhar_ram, I had done that. Got invalid template error from client too | 17:19 |
| sridhar_ram | janki: try with this https://github.com/openstack/tacker/blob/master/tacker/tests/unit/vm/infra_drivers/heat/data/tosca_vnffgd_template.yaml | 17:20 |
| janki | sridhar_ram, for vnffgd to create, we need vnfs mentioned in template running? | 17:20 |
| sridhar_ram | janki: not during catalog creation time | 17:20 |
| janki | sridhar_ram, ya the same I had tried. got error from client and horizon. | 17:20 |
| janki | sridhar_ram, will debug further in this tomorrow. | 17:21 |
| sridhar_ram | janki: for me CLI works fine, but horizon failed | 17:21 |
| janki | could you share the error? Did you try with latest patch? The earlier one had no client calls implemented | 17:21 |
| janki | sridhar_ram, and hence it was giving "unable to retrieve error" | 17:22 |
| *** mbirru has joined #tacker | 17:24 | |
| *** Vijayendra has quit IRC | 17:26 | |
| *** tung_doan is now known as tung_doan_away | 17:28 | |
| *** tbh has quit IRC | 17:39 | |
| *** janki has quit IRC | 17:40 | |
| sridhar_ram | janki: I didn't try with latest one, will do soon | 17:47 |
| *** uck_ has quit IRC | 17:59 | |
| *** uck has joined #tacker | 18:11 | |
| *** uck has quit IRC | 18:13 | |
| *** uck has joined #tacker | 18:13 | |
| *** sripriya has joined #tacker | 18:15 | |
| *** bobh has joined #tacker | 18:16 | |
| *** tung_doan has joined #tacker | 18:20 | |
| *** s3wong has quit IRC | 18:21 | |
| *** bobh has quit IRC | 18:21 | |
| *** tung_doan has quit IRC | 18:24 | |
| *** uck has quit IRC | 18:49 | |
| *** uck has joined #tacker | 18:49 | |
| *** santoshk has joined #tacker | 18:52 | |
| *** uck has quit IRC | 19:20 | |
| *** sripriya has quit IRC | 19:25 | |
| *** bobh has joined #tacker | 19:35 | |
| *** tung_doan_away has quit IRC | 19:53 | |
| *** sripriya has joined #tacker | 19:53 | |
| openstackgerrit | vishwanath jayaraman proposed openstack/tacker: Events functional tests: verify resource state. https://review.openstack.org/371086 | 20:09 |
| *** uck has joined #tacker | 20:20 | |
| *** uck has quit IRC | 20:26 | |
| openstackgerrit | Sripriya Seetharam proposed openstack/tacker: Derive infra driver using VIM type https://review.openstack.org/367599 | 20:29 |
| openstackgerrit | Sripriya Seetharam proposed openstack/tacker: Derive infra driver using VIM type https://review.openstack.org/367599 | 20:34 |
| *** uck has joined #tacker | 20:56 | |
| *** s3wong has joined #tacker | 20:58 | |
| *** amit213 has quit IRC | 21:47 | |
| openstackgerrit | vishwanath jayaraman proposed openstack/tacker: Events functional tests: verify resource state. https://review.openstack.org/371086 | 21:49 |
| *** amit213 has joined #tacker | 21:51 | |
| openstackgerrit | Merged openstack/tacker: Reflect introduction of CP order in documents https://review.openstack.org/370520 | 21:54 |
| openstackgerrit | Sridhar Ramaswamy proposed openstack/tacker: VNFFG service chain Neutron networking-sfc driver https://review.openstack.org/347568 | 21:59 |
| *** uck_ has joined #tacker | 22:11 | |
| *** uck has quit IRC | 22:11 | |
| bobh | sridhar_ram: ping | 22:13 |
| sridhar_ram | bobh: hi | 22:14 |
| bobh | sridhar_ram: There is a heat-translator patch that updates the template from 2013-05-23 to 2014-10-16 to get additional parameter support in get_attr | 22:14 |
| bobh | sridhar_ram: I don't think it should cause any issues, but wanted to run it by you - I can also pull it down and try running tacker tests against it | 22:15 |
| sridhar_ram | bobh: so, all new generated heat template will have this newer version # /date ? | 22:16 |
| bobh | sridhar_ram: yes | 22:17 |
| bobh | sridhar_ram: probably about time really :-) | 22:17 |
| bobh | sridhar_ram: it's only been 2 years.... | 22:17 |
| sridhar_ram | bobh: do we have API control to force an older version? | 22:17 |
| bobh | sridhar_ram: no - that would be useful, but also adds a lot of complexity | 22:17 |
| sridhar_ram | bobh: concern is Tacker can place vnfs in Kilo, Liberty | 22:18 |
| sripriya | sridhar_ram: i had the same concern and was about to ping the same | 22:18 |
| sridhar_ram | bobh: do u know if the new version template will work with older heat engines? | 22:19 |
| bobh | sridhar_ram: Kilo supports up through 2015-04-30 | 22:19 |
| bobh | http://docs.openstack.org/developer/heat/template_guide/hot_spec.html | 22:20 |
| openstackgerrit | vishwanath jayaraman proposed openstack/tacker: VNF Scaling event capture: Functional Test https://review.openstack.org/366433 | 22:20 |
| sridhar_ram | bobh: can u pls do some point checks ? | 22:20 |
| sridhar_ram | bobh: particularly port_security_enabled flag | 22:21 |
| bobh | sridhar_ram: I'll run it through my testbed tonight - theoretically it should be fine | 22:21 |
| bobh | sridhar_ram: we should also consider deprecating Kilo support, since it is EOL | 22:21 |
| sridhar_ram | bobh: agree on deprecation, but I'd strongly request an API control to generate the target translation version | 22:23 |
| bobh | sridhar_ram: easier said than done but I agree - I'll bring it up to spzala and see what he thinks | 22:23 |
| sridhar_ram | bobh: thanks | 22:26 |
| bobh | sridhar_ram: Liberty goes EOL in 2 months - seems like only yesterday.... | 22:29 |
| sridhar_ram | bobh: true, but i'm worried some operators might be running beyond EOL due to the sad state of openstack upgrade story | 22:31 |
| sridhar_ram | bobh: but i agree, we have EOL policies to cover ourselves | 22:31 |
| bobh | sridhar_ram: I know - not to mention vendors.... | 22:31 |
| *** bobh has quit IRC | 22:38 | |
| openstackgerrit | Merged openstack/tacker: Updated from global requirements https://review.openstack.org/371129 | 22:40 |
| *** santoshk has quit IRC | 22:58 | |
| *** sripriya has quit IRC | 23:29 | |
| *** dkushwaha has left #tacker | 23:50 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!