*** bobmel has quit IRC | 00:21 | |
*** links has joined #tacker | 00:53 | |
*** bobmel has joined #tacker | 00:54 | |
*** yifei has joined #tacker | 01:03 | |
*** s3wong has quit IRC | 02:06 | |
*** vishnoianil has quit IRC | 02:25 | |
*** KanagarajM has joined #tacker | 03:03 | |
*** KanagarajM has quit IRC | 03:31 | |
*** trinaths has joined #tacker | 03:53 | |
*** KanagarajM has joined #tacker | 04:16 | |
*** gongysh has joined #tacker | 04:23 | |
*** gongysh has quit IRC | 04:31 | |
trinaths | KanagarajM: ping | 04:34 |
---|---|---|
*** tbh has joined #tacker | 05:23 | |
trinaths | KanagarajM: Hi | 05:27 |
*** mohankumar has joined #tacker | 05:37 | |
*** trinaths has quit IRC | 05:45 | |
*** trinaths has joined #tacker | 05:59 | |
*** trinaths has quit IRC | 05:59 | |
*** gongysh has joined #tacker | 06:00 | |
*** trinaths has joined #tacker | 06:01 | |
*** KanagarajM has quit IRC | 06:17 | |
trinaths | gongysh: hi | 06:22 |
gongysh | hi | 06:22 |
trinaths | gongysh: w.r.t meeting discussion, http://eavesdrop.openstack.org/meetings/tacker/2017/tacker.2017-04-05-05.30.log.html#l-130 | 06:24 |
trinaths | gongysh: there are no DB based functions for update_nsd | 06:24 |
trinaths | gongysh: also, for update vnfd and nsd there is not cli command too | 06:25 |
gongysh | we can add these stuff | 06:25 |
trinaths | gongysh: we need to implement update-vnfd and update-nsd from cli to server ? | 06:26 |
gongysh | I think so | 06:27 |
trinaths | gongysh: then they can be different commits. since update-vim is just a patch to add FT and check for existing update-vim functionality. | 06:29 |
gongysh | fine | 06:30 |
trinaths | gongysh: okay. | 06:31 |
*** KanagarajM has joined #tacker | 06:31 | |
trinaths | gongysh: w.r.t KanagarajM comments for merging commit (https://review.openstack.org/#/c/450528/) with update-vim (https://review.openstack.org/#/c/449956/) there was a discussion for update to nsd and vnfd. since update-vim commit is specific to bug https://bugs.launchpad.net/tacker/+bug/1637360 it doesnt make sense to add new functionality for updating vnfd and nsd. comments ? | 06:33 |
openstack | Launchpad bug 1637360 in tacker "Make VIM name/description/is_default updatable" [Medium,In progress] - Assigned to Trinath Somanchi (trinath-somanchi-m) | 06:33 |
gongysh | fine | 06:35 |
gongysh | for updating vim, we need to remove deep_copy statement, I think | 06:36 |
trinaths | gongysh: okay. | 06:37 |
trinaths | gongysh: doing that change and merging KanagarajM commit now. | 06:37 |
*** KanagarajM has quit IRC | 06:43 | |
*** KanagarajM has joined #tacker | 06:48 | |
*** amotoki has joined #tacker | 07:01 | |
openstackgerrit | yanxubin proposed openstack/tacker master: Remove old oslo.messaging transport aliases https://review.openstack.org/454421 | 07:40 |
openstackgerrit | Trinath Somanchi proposed openstack/tacker master: Server side changes for update-vim and additional FTs. https://review.openstack.org/449956 | 07:40 |
*** trinaths has left #tacker | 07:50 | |
*** mohankumar has quit IRC | 07:52 | |
*** trinaths has joined #tacker | 08:23 | |
trinaths | gongysh: why you want to remove deep_update ? | 08:23 |
gongysh | trinaths, it hides what is provided by user? make this logic complex. | 08:25 |
gongysh | the nfvo db should get what is provided by user. | 08:26 |
trinaths | gongysh: while I remove deep_update, all my update_vim operations fail. | 08:26 |
trinaths | gongysh: all the upate args are ignored. | 08:26 |
trinaths | gongysh: I think we should not remove it. else update_vim doesn't work at all | 08:27 |
gongysh | and we also should give backend driver to know what is changed. if just vim name changed, I don't think the driver should register it again. | 08:27 |
trinaths | gongysh: see the recent unit test fail, http://logs.openstack.org/56/449956/27/check/gate-tacker-python27-ubuntu-xenial/c275d12/testr_results.html.gz | 08:27 |
gongysh | trinaths, you should put more effort to know how the code works. | 08:27 |
gongysh | and know we should do the update. | 08:28 |
trinaths | gongysh: agree. | 08:28 |
trinaths | gongysh: but then without deep_update the updated args are not returned. | 08:29 |
trinaths | gongysh: then we need to check description and name while calling driver to register the vim | 08:33 |
trinaths | gongysh: thoughts ? | 08:33 |
gongysh | trinaths, we can introduce a method in driver: update_vim, it should tell if register the vim. | 08:34 |
trinaths | gongysh: what are pre-reqs for registering a VIM? | 08:37 |
gongysh | I don't know, you should to read the openstack dirver to know if it needs to register the vim when updating the vim. | 08:38 |
trinaths | gongysh: agree | 08:38 |
trinaths | gongysh: will look into this | 08:39 |
gongysh | so introduce a update_vim in driver interface, and let the driver to tell if register the vim when doing updating. | 08:39 |
trinaths | gongysh: true said. but i want to know/find when a VIM need to re-registered while UPDATE. | 08:44 |
*** Aqsa has joined #tacker | 08:49 | |
trinaths | gongysh: i find no standard way when to register a vim. register vim is called while vim-create and vim-update. | 08:59 |
trinaths | gongysh: as I believe for update-vim, we can just update name, description and is_default. | 08:59 |
*** tbh has quit IRC | 09:03 | |
trinaths | gongysh: here is_default is crucial. I understand that register_vim does placement discovery for the VIM and vim-auth-encode. since this register is for vnf placement, I feel update to 'name' and 'is_default' must re-register VIM. comments ?? | 09:03 |
gongysh | do you have idea what is the impact if we update the vim name when other nfv resources are using it? | 09:06 |
*** tbh has joined #tacker | 09:17 | |
*** amotoki has quit IRC | 09:20 | |
trinaths | we cannot update vim name when other nfv resources are using it. | 09:23 |
*** yifei has quit IRC | 09:28 | |
*** trinaths has left #tacker | 09:33 | |
*** links has quit IRC | 09:33 | |
*** Aqsa has quit IRC | 10:02 | |
*** Aqsa has joined #tacker | 10:12 | |
*** amotoki has joined #tacker | 10:31 | |
*** amotoki_ has joined #tacker | 11:04 | |
*** amotoki has quit IRC | 11:07 | |
*** trinaths has joined #tacker | 11:09 | |
*** gongysh has quit IRC | 11:21 | |
*** KanagarajM has quit IRC | 11:39 | |
*** trinaths has left #tacker | 11:45 | |
*** amotoki_ has quit IRC | 11:46 | |
*** amotoki has joined #tacker | 11:46 | |
*** amotoki has quit IRC | 11:59 | |
*** Aqsa has quit IRC | 12:02 | |
*** ildikov is now known as coffeecat | 12:32 | |
*** coffeecat is now known as coffee_cat | 12:32 | |
*** tbh has quit IRC | 13:26 | |
*** gongysh has joined #tacker | 13:48 | |
*** uck has joined #tacker | 13:49 | |
*** gongysh has quit IRC | 13:55 | |
*** Yifei has joined #tacker | 14:08 | |
*** jamesdenton has joined #tacker | 14:13 | |
*** Yifei has quit IRC | 14:21 | |
*** jamesdenton has quit IRC | 15:17 | |
*** Qiming has quit IRC | 15:22 | |
*** Qiming has joined #tacker | 15:30 | |
*** uck has quit IRC | 15:42 | |
*** uck has joined #tacker | 15:43 | |
*** uck has quit IRC | 15:48 | |
*** trinaths has joined #tacker | 15:56 | |
*** amotoki has joined #tacker | 16:31 | |
*** uck has joined #tacker | 16:37 | |
*** sripriya has quit IRC | 16:38 | |
*** sripriya has joined #tacker | 16:38 | |
*** uck has quit IRC | 16:42 | |
*** trinaths has quit IRC | 17:21 | |
*** Aqsa has joined #tacker | 17:22 | |
*** mohankumar has joined #tacker | 17:31 | |
*** uck has joined #tacker | 17:37 | |
*** uck has quit IRC | 17:42 | |
*** mohankumar has quit IRC | 18:02 | |
*** s3wong has joined #tacker | 18:05 | |
*** vishnoianil has joined #tacker | 18:23 | |
*** uck has joined #tacker | 18:38 | |
*** uck has quit IRC | 18:42 | |
*** Aqsa has quit IRC | 18:45 | |
*** Aqsa has joined #tacker | 18:54 | |
*** jamesdenton has joined #tacker | 19:20 | |
*** Aqsa has quit IRC | 20:17 | |
*** uck has joined #tacker | 20:30 | |
*** vishwanathj has quit IRC | 20:38 | |
*** vishwanathj has joined #tacker | 20:38 | |
*** jamesdenton has quit IRC | 21:31 | |
*** uck has quit IRC | 21:45 | |
*** amotoki has quit IRC | 21:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!