Bug 6396: Integrate vpp-renderer with clustering singleton service 74/43774/3
authorVladimir Lavor <vlavor@cisco.com>
Fri, 12 Aug 2016 11:30:34 +0000 (13:30 +0200)
committerVladimir Lavor <vlavor@cisco.com>
Wed, 31 Aug 2016 10:32:06 +0000 (12:32 +0200)
Change-Id: I40ddc664f7520425472fe1abf89cd954d2edb8ea
Signed-off-by: Vladimir Lavor <vlavor@cisco.com>
renderers/vpp/src/main/java/org/opendaylight/controller/config/yang/config/vpp_provider/impl/GbpVppProviderInstance.java
renderers/vpp/src/main/java/org/opendaylight/controller/config/yang/config/vpp_provider/impl/GbpVppProviderModule.java
renderers/vpp/src/main/java/org/opendaylight/controller/config/yang/config/vpp_provider/impl/GbpVppProviderModuleFactory.java
renderers/vpp/src/main/resources/org/opendaylight/blueprint/vpp-renderer.xml

index b33484f0aa43c2f8f6ea97a325246da57a32e843..f483a03bc31770f614b9c507d5b9099b1e327825 100644 (file)
@@ -8,19 +8,76 @@
 
 package org.opendaylight.controller.config.yang.config.vpp_provider.impl;
 
+import com.google.common.base.Preconditions;
+import com.google.common.util.concurrent.Futures;
+import com.google.common.util.concurrent.ListenableFuture;
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.sal.binding.api.BindingAwareBroker;
+import org.opendaylight.mdsal.singleton.common.api.ClusterSingletonService;
+import org.opendaylight.mdsal.singleton.common.api.ClusterSingletonServiceProvider;
+import org.opendaylight.mdsal.singleton.common.api.ClusterSingletonServiceRegistration;
+import org.opendaylight.mdsal.singleton.common.api.ServiceGroupIdentifier;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
-public class GbpVppProviderInstance implements AutoCloseable {
+public class GbpVppProviderInstance implements ClusterSingletonService, AutoCloseable {
 
+    private static final Logger LOG = LoggerFactory.getLogger(GbpVppProviderInstance.class);
+
+    private static final ServiceGroupIdentifier IDENTIFIER =
+            ServiceGroupIdentifier.create(GbpVppProviderInstance.class.getName());
+    private final DataBroker dataBroker;
+    private final BindingAwareBroker bindingAwareBroker;
+    private final ClusterSingletonServiceProvider clusterSingletonService;
+    private ClusterSingletonServiceRegistration singletonServiceRegistration;
     private VppRenderer renderer;
 
-    public GbpVppProviderInstance (DataBroker dataBroker, BindingAwareBroker broker) {
-        renderer = new VppRenderer(dataBroker, broker);
+    public GbpVppProviderInstance(final DataBroker dataBroker,
+                                  final BindingAwareBroker bindingAwareBroker,
+                                  final ClusterSingletonServiceProvider clusterSingletonService) {
+        this.dataBroker = Preconditions.checkNotNull(dataBroker);
+        this.bindingAwareBroker = Preconditions.checkNotNull(bindingAwareBroker);
+        this.clusterSingletonService = Preconditions.checkNotNull(clusterSingletonService);
+    }
+
+    public void initialize() {
+        LOG.info("Clustering session initiated for {}", this.getClass().getSimpleName());
+        singletonServiceRegistration = clusterSingletonService.registerClusterSingletonService(this);
     }
+
+    @Override
+    public void instantiateServiceInstance() {
+        LOG.info("Instantiating {}", this.getClass().getSimpleName());
+        renderer = new VppRenderer(dataBroker, bindingAwareBroker);
+    }
+
+    @Override
+    public ListenableFuture<Void> closeServiceInstance() {
+        LOG.info("Instance {} closed", this.getClass().getSimpleName());
+        try {
+            renderer.close();
+        } catch (Exception e) {
+            LOG.warn("Exception while closing ... {}", e.getMessage());
+        }
+        return Futures.immediateFuture(null);
+    }
+
     @Override
     public void close() throws Exception {
-        renderer.close();
+
+        LOG.info("Clustering provider closed for {}", this.getClass().getSimpleName());
+        if (singletonServiceRegistration != null) {
+            try {
+                singletonServiceRegistration.close();
+            } catch (Exception e) {
+                LOG.warn("{} closed unexpectedly", this.getClass().getSimpleName(), e.getMessage());
+            }
+            singletonServiceRegistration = null;
+        }
     }
 
+    @Override
+    public ServiceGroupIdentifier getIdentifier() {
+        return IDENTIFIER;
+    }
 }
index feb106533ce63214ad964e0ead246f6baea9deb5..556524509e5fd78ba379f24e3e75c19a446b2326 100644 (file)
@@ -11,7 +11,7 @@ package org.opendaylight.controller.config.yang.config.vpp_provider.impl;
 import org.opendaylight.controller.sal.common.util.NoopAutoCloseable;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-
+@Deprecated
 public class GbpVppProviderModule extends org.opendaylight.controller.config.yang.config.vpp_provider.impl.AbstractGbpVppProviderModule {
 
     private static final Logger LOG = LoggerFactory.getLogger(GbpVppProviderModule.class);
index 438da95cbaed562bcf4106c2bea5ea3f966ad866..1bd2093f478f693d7f50e7c069747ede3ab946da 100644 (file)
@@ -15,6 +15,7 @@
 * Do not modify this file unless it is present under src/main directory
 */
 package org.opendaylight.controller.config.yang.config.vpp_provider.impl;
+@Deprecated
 public class GbpVppProviderModuleFactory extends org.opendaylight.controller.config.yang.config.vpp_provider.impl.AbstractGbpVppProviderModuleFactory {
 
 }
index daf5e222a1fcd8c3e6a6f1d9f0fa050f2103f026..ba47a38e5c62605f8953bb398ad52951a893c7eb 100644 (file)
@@ -5,10 +5,12 @@
 
     <reference id="dataBroker" interface="org.opendaylight.controller.md.sal.binding.api.DataBroker"/>
     <reference id="broker" interface="org.opendaylight.controller.sal.binding.api.BindingAwareBroker"/>
+    <reference id="clusterSingletonService" interface="org.opendaylight.mdsal.singleton.common.api.ClusterSingletonServiceProvider"/>
 
     <bean id="vppRenderer" class="org.opendaylight.controller.config.yang.config.vpp_provider.impl.GbpVppProviderInstance"
-        destroy-method="close">
+        init-method="initialize" destroy-method="close">
         <argument ref="dataBroker"/>
         <argument ref="broker"/>
+        <argument ref="clusterSingletonService" />
     </bean>
 </blueprint>
\ No newline at end of file