> On May 5, 2014, 11:32 p.m., Rajesh Battala wrote: > > plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/api/command/DeleteServiceInstanceCmd.java, > > line 40 > > <https://reviews.apache.org/r/19270/diff/1/?file=521583#file521583line40> > > > > As the service vm is specific to juniper-contrail, instead of using > > vm_destroy create an specific event type. refer > > EVENT_INTERNAL_LB_VM_STOP/START
Added separete EVENTs for service instance VM creation and destroy. - Sachchidanand ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19270/#review42253 ----------------------------------------------------------- On May 7, 2014, 3:44 p.m., Sachchidanand Vaidya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19270/ > ----------------------------------------------------------- > > (Updated May 7, 2014, 3:44 p.m.) > > > Review request for cloudstack and Rajesh Battala. > > > Repository: cloudstack-git > > > Description > ------- > > This patch has following changes: > - Added service-instance delete command to Juniper Contrail Plugin. > - Fixes to plugin code to handle instance-delete operation properly. > - Fix related to service-instance persistence and re-creation during Mgmt > server reboot. > > > Diffs > ----- > > api/src/com/cloud/event/EventTypes.java 075b1c8 > client/tomcatconf/commands.properties.in d3a24f6 > > plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/api/command/CreateServiceInstanceCmd.java > 59c4e61 > > plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/api/command/DeleteServiceInstanceCmd.java > PRE-CREATION > > plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailElementImpl.java > 05723b0 > > plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailGuru.java > f81c0d0 > > plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ServerDBSyncImpl.java > b9a3ed3 > > plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ServiceManager.java > e4ef26d > > plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ServiceManagerImpl.java > f34eacc > > plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/ServiceInstanceModel.java > e79053c > > Diff: https://reviews.apache.org/r/19270/diff/ > > > Testing > ------- > > Unit tests passed and additional testing done locally. > > > Thanks, > > Sachchidanand Vaidya > >
