Thanks Darren. It has been checked in. Thanks, Suresh
From: Darren Shepherd <darren.sheph...@citrix.com<mailto:darren.sheph...@citrix.com>> Date: Monday, February 24, 2014 11:53 AM To: "dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>" <dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>>, Suresh Balineni <sbalin...@juniper.net<mailto:sbalin...@juniper.net>> Cc: Sunil Bakhru <sbak...@juniper.net<mailto:sbak...@juniper.net>>, Sheng Yang <sh...@yasker.org<mailto:sh...@yasker.org>>, Hugo <h...@trippaers.nl<mailto:h...@trippaers.nl>>, Animesh Chaturvedi <animesh.chaturv...@citrix.com<mailto:animesh.chaturv...@citrix.com>> Subject: Re: Event Interceptor fix - contrail plugin : Review Request I'm sorry, I thought I did this weeks ago. Do you not see the commit in master? I'll double check, maybe I forgot to push. Darren ________________________________ From: Suresh Balineni <sbalin...@juniper.net<mailto:sbalin...@juniper.net>> Date: 2014-02-24 19:19:57 +0000 To: Darren Shepherd <darren.sheph...@citrix.com<mailto:darren.sheph...@citrix.com>>,dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org> <dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>> Subject: Re: Event Interceptor fix - contrail plugin : Review Request Cc: Sheng Yang <sh...@yasker.org<mailto:sh...@yasker.org>>,Hugo <h...@trippaers.nl<mailto:h...@trippaers.nl>>,Animesh Chaturvedi <animesh.chaturv...@citrix.com<mailto:animesh.chaturv...@citrix.com>>,Sunil Bakhru <sbak...@juniper.net<mailto:sbak...@juniper.net>> Hi Darren, Can you please review and ship these changes (Master branch)? Without this, contrail implementation won't work. Thanks, Suresh From: Suresh Balineni <sbalin...@junipernetworks.onmicrosoft.com<mailto:sbalin...@junipernetworks.onmicrosoft.com>> Date: Monday, February 10, 2014 12:10 PM To: "dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>" <dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>>, Darren Shepherd <darren.sheph...@citrix.com<mailto:darren.sheph...@citrix.com>> Cc: Darren Shepherd <darren.sheph...@citrix.com<mailto:darren.sheph...@citrix.com>>, Hugo <h...@trippaers.nl<mailto:h...@trippaers.nl>>, Animesh Chaturvedi <animesh.chaturv...@citrix.com<mailto:animesh.chaturv...@citrix.com>>, Sheng Yang <sh...@yasker.org<mailto:sh...@yasker.org>>, Sunil Bakhru <sbak...@juniper.net<mailto:sbak...@juniper.net>> Subject: Re: Event Interceptor fix - contrail plugin : Review Request Hi Darren, Can you please review this ? Thanks, Suresh From: Suresh Balineni <sbalin...@juniper.net<mailto:sbalin...@juniper.net>> Date: Monday, February 3, 2014 3:24 PM Cc: Darren Shepherd <darren.sheph...@citrix.com<mailto:darren.sheph...@citrix.com>>, Hugo <h...@trippaers.nl<mailto:h...@trippaers.nl>>, Animesh Chaturvedi <animesh.chaturv...@citrix.com<mailto:animesh.chaturv...@citrix.com>>, Sheng Yang <sh...@yasker.org<mailto:sh...@yasker.org>>, Sunil Bakhru <sbak...@juniper.net<mailto:sbak...@juniper.net>> Subject: Re: Event Interceptor fix - contrail plugin : Review Request Hi Darren, Can you review this? Thanks, Suresh On Jan 30, 2014, at 4:23 PM, "Suresh Balineni" <sbalin...@juniper.net<mailto:sbalin...@juniper.net>> wrote: Hi Darren, Can you please review and ship the following event interceptor fix in CS master branch? I have implemented this as per your suggestion and it works fine. https://reviews.apache.org/r/17337/ This review is pending for while Thanks, Suresh