08:02:14 <dkushwaha> #startmeeting tacker
08:02:15 <openstack> Meeting started Tue Aug 20 08:02:14 2019 UTC and is due to finish in 60 minutes.  The chair is dkushwaha. Information about MeetBot at http://wiki.debian.org/MeetBot.
08:02:16 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
08:02:18 <openstack> The meeting name has been set to 'tacker'
08:02:25 <dkushwaha> #topic Roll Call
08:02:41 <dkushwaha> who is here for Tacker weekly meeting?
08:03:03 <joxyuki> hello
08:03:06 <takahashi-tsc> Hi.
08:03:10 <k-keiko> Hi
08:03:16 <nnsingh> Hi
08:04:08 <dkushwaha> hi all
08:04:56 <hyunsikyang> hi
08:06:26 <dkushwaha> hi hyunsikyang
08:06:30 <dkushwaha> #chair joxyuki
08:06:31 <openstack> Current chairs: dkushwaha joxyuki
08:07:39 <dkushwaha> joxyuki, do you have something to discuss?
08:08:19 <joxyuki> VNF package spec was merged and niraj pushed couple of patches.
08:08:35 <joxyuki> kindly request you to review them.
08:09:12 <dkushwaha> joxyuki, nice.
08:09:46 <dkushwaha> I was on leave last whole week, so didn't check much, but have seen couple of WIP patches.
08:10:56 <dkushwaha> #topic Open Discussion
08:12:13 <joxyuki> dkushwaha, posted a comment on https://review.opendev.org/#/c/666496/
08:12:14 <patchbot> patch 666496 - tacker - Adds support force delete for NS - 7 patch sets
08:13:18 <joxyuki> what do you think about it?
08:16:22 <dkushwaha> joxyuki, not understand your comment
08:17:07 <dkushwaha> there ns is {'ns': {'attributes': {'force': True}}}
08:17:32 <dkushwaha> so  kwargs={'ns': {'ns': {'attributes': {'force': True}}}
08:17:50 <dkushwaha> why to delete manually
08:19:12 <joxyuki> dkushwaha, oh, sorry. I mean, "don't we need to set force_delete flag to mistral workflow created here?", otherwise, vnf deletion works without force flag.
08:20:02 <joxyuki> So, there is possibility that NS is forcely deleted but some VNFs are stucking.
08:20:49 <joxyuki> in this case, users can manually delete stucking VNFs with force flag.
08:21:06 <joxyuki> But are you expecting this opration?
08:23:17 <dkushwaha> joxyuki, i think, I am passing it in expected way in line #923, but let me check it again
08:23:35 <joxyuki> currently force_delete flag is ignored in workflow of NS deletion. https://github.com/openstack/tacker/blob/master/tacker/nfvo/drivers/workflow/workflow_generator.py#L255
08:26:24 <tpatil> Just wanted to update about the vnf packages implementation progress
08:27:36 <tpatil> We are writing the unit/functional tests now. Restful apis are implemented
08:28:09 <tpatil> we will update the existing patches by Thursday
08:29:03 <tpatil> If you face any problems in setting up the env, please let us know. once the unit and functional tests are implemented, we will add the required documentation.
08:29:18 <tpatil> that's all I wanted to update here. Thank you.
08:30:00 <dkushwaha> joxyuki, I got your point, and yes it seems issue.
08:30:16 <dkushwaha> joxyuki, will update the patch.
08:30:31 <dkushwaha> thanks for your quick comment
08:31:17 <dkushwaha> tpatil, thanks for update. will review the patches
08:31:50 <tpatil> dkushwaha: Thanks
08:33:00 <dkushwaha> joxyuki, Please review https://review.opendev.org/#/c/673404
08:33:01 <patchbot> patch 673404 - tacker - Make non-admin's VIM accessible for admin - 1 patch set
08:33:56 <joxyuki> got it. I can do it tomorrow.
08:34:00 <dkushwaha> hyunsikyang, do you have some update from your side?
08:34:10 <dkushwaha> joxyuki, thanks
08:34:37 <hyunsikyang> I uploaded code as a WIP.
08:35:20 <hyunsikyang> and I finished test. But I think,, I should  upload some test code
08:35:22 <hyunsikyang> right?
08:36:16 <hyunsikyang> And I will uploade user guide for this spec soon seperately
08:36:50 <dkushwaha> hyunsikyang, yes, adding test cases with feature is the best practice. you can also put them in separate patch set
08:37:19 <hyunsikyang> ok.
08:38:54 <dkushwaha> hyunsikyang, could you please share the link of your WIP patch(i think i missed it)
08:39:11 <hyunsikyang> https://review.opendev.org/#/c/674761/
08:39:12 <patchbot> patch 674761 - tacker - [WIP] Adding Multi-Interface for Containerized VNF... - 6 patch sets
08:39:15 <hyunsikyang> here
08:39:23 <dkushwaha> hyunsikyang, ok
08:39:29 <dkushwaha> thanks
08:40:59 <hyunsikyang> you're welcome
08:41:39 <dkushwaha> Do we have something to discuss? otherwise we can close this meeting
08:41:49 <takahashi-tsc> Our activities update. Now we evaluate scaling action and faced issues that mgmt_driver does not work.
08:42:02 <takahashi-tsc> https://review.opendev.org/#/c/576670/
08:42:02 <patchbot> patch 576670 - tacker - when scale vnf , mgmt driver doesn't work - 4 patch sets
08:42:38 <takahashi-tsc> Above patch progress is now stopping... We will take over or make new patch for fix it,
08:45:35 <dkushwaha> takahashi-tsc, thanks for heading up for this fix
08:46:21 <dkushwaha> this is the long pending work item from vnf-scaling BP https://blueprints.launchpad.net/tacker/+spec/vnf-scaling
08:48:45 <dkushwaha> Thanks Folks for joining this meeting.
08:48:51 <dkushwaha> Closing this meeting now
08:49:11 <joxyuki> thanks folks
08:51:41 <dkushwaha_> it seems i lost my connection
08:51:52 <dkushwaha_> takahashi-tsc, thanks for heading up for this fix
08:52:03 <dkushwaha_> this is the long pending work item from vnf-scaling BP https://blueprints.launchpad.net/tacker/+spec/vnf-scaling
08:52:49 <takahashi-tsc> This fix is needed for our actual use case. We start to develop and I think we make the patch by this weekend or next week.
08:53:33 <dkushwaha_> takahashi-tsc, nice
08:54:26 <dkushwaha_> Thanks Folks for joining this meeting.
08:54:39 <dkushwaha_> Closing this meeting now
08:54:42 <joxyuki> #endmeeting