08:00:58 <dkushwaha> #startmeeting tacker
08:00:59 <openstack> Meeting started Tue Dec  3 08:00:58 2019 UTC and is due to finish in 60 minutes.  The chair is dkushwaha. Information about MeetBot at http://wiki.debian.org/MeetBot.
08:01:00 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
08:01:04 <openstack> The meeting name has been set to 'tacker'
08:01:11 <dkushwaha> #topic Roll Call
08:01:15 <keiko-k> Hello
08:01:25 <tpatil> Hi
08:02:27 <nitinuikey> HI
08:02:37 <hyunsikyang> HI Longtime no See!
08:03:01 <dkushwaha> hello all
08:03:59 <dkushwaha> Lets start..
08:04:04 <dkushwaha> #topic BP
08:04:55 <dkushwaha> tpatil, do you have something to discuss from your side?
08:05:33 <tpatil> VNF LCM API based on ETSI NFV-SOL specification
08:06:03 <tpatil> we are working on adding the final db schema changes and also will reply to your queries/comments.
08:06:57 <dkushwaha> ok
08:07:17 <tpatil> also in another specs, we want to enhance vnf packages APIs
08:07:32 <dkushwaha> link please
08:07:52 <tpatil> #link : https://review.opendev.org/#/c/696819
08:09:40 <tpatil> We have started working on adding three APIs
08:09:51 <tpatil> Request you all to please review this specs as well
08:10:52 <dkushwaha> tpatil, sure.
08:11:15 <tpatil> dkushwaha: Thanks
08:11:29 <dkushwaha> just a quick query, is there any db changes required due to this spec?
08:12:15 <tpatil> Yes, only one small change
08:12:32 <tpatil> we are planning to add 'size' db column in 'vnf_packages'.
08:13:10 <tpatil> to fetch vnf package csar zip file, we need the size of the file for validation of Range header
08:14:15 <dkushwaha> tpatil, not sure if size is needed to add in db, but ok first i review
08:14:54 <tpatil> this is needed for validation purpose only, becoz the size of the csar file is only known to the tacker-conductor
08:15:19 <tpatil> to get the size of the zip file, we will need to call tacker-conductor all the time
08:15:32 <tpatil> if it's there in the db, it would solve the problem
08:16:57 <dkushwaha> ok
08:17:34 <dkushwaha> tpatil, isn't csar have some check-sum to validate?
08:17:55 <tpatil> that's not yet implemented
08:18:45 <tpatil> from check-sum if we can get the size, then it's worth adding that support. in that case, we can drop adding size column
08:19:50 <tpatil> but I don't think it can get us the size of the file
08:20:31 <dkushwaha> tpatil, yes, checksum didnot provide size.
08:20:50 <dkushwaha> ok, i will review the spec
08:21:02 <tpatil> dkushwaha: Thanks
08:22:09 <dkushwaha> moving next..
08:22:51 <dkushwaha> from my side, i am working on BP Attach existing volume with VNF
08:22:59 <dkushwaha> #link https://blueprints.launchpad.net/tacker/+spec/attach-existing-volume
08:24:21 <dkushwaha> In current implementation, tosca parser take OS::Cinder::Volume for volume, and there is no support for volume ids
08:25:07 <dkushwaha> I checked with heat team, and working on identifying gaps.
08:25:27 <dkushwaha> Hopefuly, will push initial code within this week
08:26:17 <dkushwaha> that is from my side
08:27:38 <dkushwaha> #topic OpenDiscussion
08:28:03 <hyunsikyang> from my side,I am working on two implementation. last time I didn't merge it due to the test code. We will finish ASAP.
08:28:23 <hyunsikyang> but code is finished.
08:29:57 <tpatil> Delete VNF should fail with 409 error
08:29:59 <tpatil> Need review: https://review.opendev.org/#/c/618086
08:30:45 <dkushwaha> hyunsikyang, are you working on Prometheus BP?
08:31:11 <dkushwaha> tpatil, ack
08:31:34 <hyunsikyang> not now, Finish two things and do prometheus one.
08:32:04 <dkushwaha> hyunsikyang, nice.
08:33:45 <dkushwaha> Anything else from other folks ?
08:36:50 <dkushwaha> Ok, so lets closing this meeting. Will be available on tacker channel
08:37:07 <dkushwaha> Thanks all for joining.
08:37:12 <dkushwaha> #endmeeting