08:01:20 <dkushwaha_> #startmeeting tacker
08:01:21 <openstack> Meeting started Tue Dec 18 08:01:20 2018 UTC and is due to finish in 60 minutes.  The chair is dkushwaha_. Information about MeetBot at http://wiki.debian.org/MeetBot.
08:01:22 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
08:01:24 <openstack> The meeting name has been set to 'tacker'
08:01:31 <dkushwaha_> #topic Roll Call
08:01:47 <dkushwaha_> who is here for Tacker weekly meeting?
08:04:26 <bhagyashris> O/
08:04:36 <dkushwaha_> hi bhagyashris
08:04:51 <bhagyashris> Hi
08:05:25 <dkushwaha_> seems no more members available today
08:06:12 <dkushwaha_> #topic Open Discussion
08:06:39 <dkushwaha_> bhagyashris, do you have something to discuss ?
08:07:03 <bhagyashris> Yeah,  just give me one min
08:07:17 <dkushwaha_> sure
08:07:50 <bhagyashris> Regarding : Add VNF to monitor after restarting tacker service
08:08:19 <bhagyashris> So I have fix the review comments and uploaded patch for community review https://review.openstack.org/#/c/614964/
08:08:36 <bhagyashris> Anf this change is made on current master
08:09:56 <bhagyashris> and yesterday phouc has cherry pick on stable/rocky and uploaded for CG: https://review.openstack.org/#/c/625461/
08:10:55 <bhagyashris> So just wanted to know which one will merge
08:12:15 <bhagyashris> Or will be consider
08:13:38 <dkushwaha_> bhagyashris, i am ok to merge  https://review.openstack.org/#/c/614964 on master, and as  https://review.openstack.org/#/c/625461 is from same on stable/Rocky, so this one will be merged in stable/Rocky.
08:14:22 <dkushwaha_> although as Phouch suggested, let me try first https://review.openstack.org/#/c/486924 for master
08:15:20 <dkushwaha_> bhagyashris, i will test  https://review.openstack.org/#/c/625461 on stable and will merge it first.
08:15:36 <bhagyashris> dkushwaha_: yeah me to have same understanding about cherry pick first the change should go on master and then on stable/branch
08:16:29 <bhagyashris> dkushwaha_: ok. Thank you :)
08:17:46 <bhagyashris> dkushwaha_: Want to discuss regarding : [WIP] Implement VNF monitoring using Mistral https://review.openstack.org/#/c/486924/48
08:17:49 <dkushwaha_> the suitable wy is  https://review.openstack.org/#/c/486924, so I request to do some deep review of this patch(as it is big enough)
08:18:10 <dkushwaha_> sure
08:18:48 <bhagyashris> dkushwaha_: yeah :). Actually I have tested this patch on my environment and facing one issue at mistral side
08:19:28 <bhagyashris> # link: http://paste.openstack.org/show/737527/
08:22:21 <bhagyashris> dkushwaha_: yeah this https://review.openstack.org/#/c/486924 will solve the issue but on stable branches the issue will remain as is, so this https://review.openstack.org/#/c/614964 minimal change will help to resolve this
08:22:39 <dkushwaha_> bhagyashris, thanks for testing patch. I am not sure about the issue, but will check it again
08:23:47 <bhagyashris> dkushwaha_: Welcome I will check deep in free time :)
08:24:47 <dkushwaha_> i suspect , I made some mistake while resolving merge conflicts, will update the patch after resolving issue and other hardcoded values
08:25:16 <bhagyashris> dkushwaha_: Thank you for reviewing vdu_autoheal patch now fixing review comments and upload ASAP :)
08:25:57 <dkushwaha_> bhagyashris, cool
08:26:01 <bhagyashris> dkushwaha_:  ok I will check the new patch set and also test in my environment and will let you know
08:26:13 <bhagyashris> The result
08:26:26 <dkushwaha_> great
08:26:36 <dkushwaha_> bhagyashris, anything else to discuss?
08:27:00 <bhagyashris> dkushwaha_:  No thats all from my side
08:27:02 <bhagyashris> :)
08:27:10 <dkushwaha_> Ok, Thanks for joining meeting. :)
08:27:22 <bhagyashris> :)
08:27:33 <dkushwaha_> Closing this meeting for now
08:27:41 <dkushwaha_> #endmeeting