Friday, 2017-04-14

*** bobmel has quit IRC00:21
*** links has joined #tacker00:53
*** bobmel has joined #tacker00:54
*** yifei has joined #tacker01:03
*** s3wong has quit IRC02:06
*** vishnoianil has quit IRC02:25
*** KanagarajM has joined #tacker03:03
*** KanagarajM has quit IRC03:31
*** trinaths has joined #tacker03:53
*** KanagarajM has joined #tacker04:16
*** gongysh has joined #tacker04:23
*** gongysh has quit IRC04:31
trinathsKanagarajM: ping04:34
*** tbh has joined #tacker05:23
trinathsKanagarajM: Hi05:27
*** mohankumar has joined #tacker05:37
*** trinaths has quit IRC05:45
*** trinaths has joined #tacker05:59
*** trinaths has quit IRC05:59
*** gongysh has joined #tacker06:00
*** trinaths has joined #tacker06:01
*** KanagarajM has quit IRC06:17
trinathsgongysh: hi06:22
gongyshhi06:22
trinathsgongysh: w.r.t meeting discussion, http://eavesdrop.openstack.org/meetings/tacker/2017/tacker.2017-04-05-05.30.log.html#l-13006:24
trinathsgongysh:  there are no DB based functions for update_nsd06:24
trinathsgongysh:  also, for update vnfd and nsd there is not cli command too06:25
gongyshwe can add these stuff06:25
trinathsgongysh:  we need to implement update-vnfd and update-nsd from cli to server ?06:26
gongyshI think so06:27
trinathsgongysh:  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
gongyshfine06:30
trinathsgongysh:  okay.06:31
*** KanagarajM has joined #tacker06:31
trinathsgongysh:  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
openstackLaunchpad bug 1637360 in tacker "Make VIM name/description/is_default updatable" [Medium,In progress] - Assigned to Trinath Somanchi (trinath-somanchi-m)06:33
gongyshfine06:35
gongyshfor updating vim, we need to remove deep_copy statement, I think06:36
trinathsgongysh:  okay.06:37
trinathsgongysh: doing that change and merging KanagarajM commit  now.06:37
*** KanagarajM has quit IRC06:43
*** KanagarajM has joined #tacker06:48
*** amotoki has joined #tacker07:01
openstackgerrityanxubin proposed openstack/tacker master: Remove old oslo.messaging transport aliases  https://review.openstack.org/45442107:40
openstackgerritTrinath Somanchi proposed openstack/tacker master: Server side changes for update-vim and additional FTs.  https://review.openstack.org/44995607:40
*** trinaths has left #tacker07:50
*** mohankumar has quit IRC07:52
*** trinaths has joined #tacker08:23
trinathsgongysh:  why you want to remove deep_update ?08:23
gongyshtrinaths, it hides what is provided by user? make this logic complex.08:25
gongyshthe nfvo db should get what is provided by user.08:26
trinathsgongysh:  while I remove deep_update, all my update_vim operations fail.08:26
trinathsgongysh: all the upate args are ignored.08:26
trinathsgongysh:  I think we should not remove it. else update_vim doesn't work at all08:27
gongyshand 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
trinathsgongysh:  see the recent unit test fail, http://logs.openstack.org/56/449956/27/check/gate-tacker-python27-ubuntu-xenial/c275d12/testr_results.html.gz08:27
gongyshtrinaths, you should put more effort to know how the code works.08:27
gongyshand know we should do the update.08:28
trinathsgongysh:  agree.08:28
trinathsgongysh:  but then without deep_update the updated args are not returned.08:29
trinathsgongysh: then we need to check description and name while calling driver to register the vim08:33
trinathsgongysh:  thoughts ?08:33
gongyshtrinaths, we can introduce a method in driver: update_vim, it should tell if register the vim.08:34
trinathsgongysh:  what are pre-reqs for registering a VIM?08:37
gongyshI 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
trinathsgongysh:  agree08:38
trinathsgongysh:  will look into this08:39
gongyshso introduce a update_vim in driver interface, and let the driver to tell if register the vim when doing updating.08:39
trinathsgongysh:  true said. but i want to know/find when a VIM need to re-registered while UPDATE.08:44
*** Aqsa has joined #tacker08:49
trinathsgongysh:  i find no standard way when to register a vim. register vim is called while vim-create and vim-update.08:59
trinathsgongysh:  as I believe for update-vim, we can just update name, description and is_default.08:59
*** tbh has quit IRC09:03
trinathsgongysh:  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
gongyshdo 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 #tacker09:17
*** amotoki has quit IRC09:20
trinathswe cannot update vim name when other nfv resources are using it.09:23
*** yifei has quit IRC09:28
*** trinaths has left #tacker09:33
*** links has quit IRC09:33
*** Aqsa has quit IRC10:02
*** Aqsa has joined #tacker10:12
*** amotoki has joined #tacker10:31
*** amotoki_ has joined #tacker11:04
*** amotoki has quit IRC11:07
*** trinaths has joined #tacker11:09
*** gongysh has quit IRC11:21
*** KanagarajM has quit IRC11:39
*** trinaths has left #tacker11:45
*** amotoki_ has quit IRC11:46
*** amotoki has joined #tacker11:46
*** amotoki has quit IRC11:59
*** Aqsa has quit IRC12:02
*** ildikov is now known as coffeecat12:32
*** coffeecat is now known as coffee_cat12:32
*** tbh has quit IRC13:26
*** gongysh has joined #tacker13:48
*** uck has joined #tacker13:49
*** gongysh has quit IRC13:55
*** Yifei has joined #tacker14:08
*** jamesdenton has joined #tacker14:13
*** Yifei has quit IRC14:21
*** jamesdenton has quit IRC15:17
*** Qiming has quit IRC15:22
*** Qiming has joined #tacker15:30
*** uck has quit IRC15:42
*** uck has joined #tacker15:43
*** uck has quit IRC15:48
*** trinaths has joined #tacker15:56
*** amotoki has joined #tacker16:31
*** uck has joined #tacker16:37
*** sripriya has quit IRC16:38
*** sripriya has joined #tacker16:38
*** uck has quit IRC16:42
*** trinaths has quit IRC17:21
*** Aqsa has joined #tacker17:22
*** mohankumar has joined #tacker17:31
*** uck has joined #tacker17:37
*** uck has quit IRC17:42
*** mohankumar has quit IRC18:02
*** s3wong has joined #tacker18:05
*** vishnoianil has joined #tacker18:23
*** uck has joined #tacker18:38
*** uck has quit IRC18:42
*** Aqsa has quit IRC18:45
*** Aqsa has joined #tacker18:54
*** jamesdenton has joined #tacker19:20
*** Aqsa has quit IRC20:17
*** uck has joined #tacker20:30
*** vishwanathj has quit IRC20:38
*** vishwanathj has joined #tacker20:38
*** jamesdenton has quit IRC21:31
*** uck has quit IRC21:45
*** amotoki has quit IRC21:58

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