From 6df6ec9783c7fb49c02bda1dcc33002998a82027 Mon Sep 17 00:00:00 2001
From: Matej Perina <mperina@cisco.com>
Date: Thu, 18 Aug 2016 17:11:23 +0200
Subject: [PATCH] BUG 6396 ne-location-provider migration to blueprint

Change-Id: I5c63e24e1f715cdb224abfb6d2c8315d6e9a6227
Signed-off-by: Matej Perina <mperina@cisco.com>
---
 .../src/main/config/default-config.xml        |  5 ----
 .../cfg/NeLocationProviderInstance.java       | 26 +++++++++++++++++++
 .../cfg/NeLocationProviderModule.java         |  4 +--
 .../blueprint/ne-location-provider.xml        | 12 +++++++++
 .../main/yang/ne-location-provider-cfg.yang   | 10 -------
 5 files changed, 40 insertions(+), 17 deletions(-)
 create mode 100644 location-providers/ne-location-provider/src/main/java/org/opendaylight/controller/config/yang/config/ne/location/provider/cfg/NeLocationProviderInstance.java
 create mode 100644 location-providers/ne-location-provider/src/main/resources/org/opendaylight/blueprint/ne-location-provider.xml

diff --git a/location-providers/ne-location-provider/src/main/config/default-config.xml b/location-providers/ne-location-provider/src/main/config/default-config.xml
index 82b8b4b72..115cf5199 100644
--- a/location-providers/ne-location-provider/src/main/config/default-config.xml
+++ b/location-providers/ne-location-provider/src/main/config/default-config.xml
@@ -16,11 +16,6 @@
                         ne-location:ne-location-provider
                     </type>
                     <name>ne-location-provider</name>
-
-                    <data-broker>
-                      <type xmlns:binding="urn:opendaylight:params:xml:ns:yang:controller:md:sal:binding">binding:binding-async-data-broker</type>
-                      <name>binding-data-broker</name>
-                    </data-broker>
                 </module>
             </modules>
         </data>
diff --git a/location-providers/ne-location-provider/src/main/java/org/opendaylight/controller/config/yang/config/ne/location/provider/cfg/NeLocationProviderInstance.java b/location-providers/ne-location-provider/src/main/java/org/opendaylight/controller/config/yang/config/ne/location/provider/cfg/NeLocationProviderInstance.java
new file mode 100644
index 000000000..f894a91a9
--- /dev/null
+++ b/location-providers/ne-location-provider/src/main/java/org/opendaylight/controller/config/yang/config/ne/location/provider/cfg/NeLocationProviderInstance.java
@@ -0,0 +1,26 @@
+/*
+ * Copyright (c) 2016 Cisco Systems, Inc. and others.  All rights reserved.
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the Eclipse Public License v1.0 which accompanies this distribution,
+ * and is available at http://www.eclipse.org/legal/epl-v10.html
+ */
+
+package org.opendaylight.controller.config.yang.config.ne.location.provider.cfg;
+
+import org.opendaylight.controller.md.sal.binding.api.DataBroker;
+import org.opendaylight.groupbasedpolicy.ne.location.provider.NeLocationProvider;
+
+public class NeLocationProviderInstance implements AutoCloseable {
+
+    private NeLocationProvider provider;
+
+    public NeLocationProviderInstance (DataBroker dataBroker) {
+        provider = new NeLocationProvider(dataBroker);
+    }
+
+    @Override
+    public void close() throws Exception {
+        provider.close();
+    }
+}
diff --git a/location-providers/ne-location-provider/src/main/java/org/opendaylight/controller/config/yang/config/ne/location/provider/cfg/NeLocationProviderModule.java b/location-providers/ne-location-provider/src/main/java/org/opendaylight/controller/config/yang/config/ne/location/provider/cfg/NeLocationProviderModule.java
index 44b164302..b0c7a9a2b 100644
--- a/location-providers/ne-location-provider/src/main/java/org/opendaylight/controller/config/yang/config/ne/location/provider/cfg/NeLocationProviderModule.java
+++ b/location-providers/ne-location-provider/src/main/java/org/opendaylight/controller/config/yang/config/ne/location/provider/cfg/NeLocationProviderModule.java
@@ -8,7 +8,7 @@
 
 package org.opendaylight.controller.config.yang.config.ne.location.provider.cfg;
 
-import org.opendaylight.groupbasedpolicy.ne.location.provider.NeLocationProvider;
+import org.opendaylight.controller.sal.common.util.NoopAutoCloseable;
 
 public class NeLocationProviderModule extends org.opendaylight.controller.config.yang.config.ne.location.provider.cfg.AbstractNeLocationProviderModule {
     public NeLocationProviderModule(org.opendaylight.controller.config.api.ModuleIdentifier identifier, org.opendaylight.controller.config.api.DependencyResolver dependencyResolver) {
@@ -26,7 +26,7 @@ public class NeLocationProviderModule extends org.opendaylight.controller.config
 
     @Override
     public java.lang.AutoCloseable createInstance() {
-        return new NeLocationProvider(getDataBrokerDependency());
+        return NoopAutoCloseable.INSTANCE;
     }
 
 }
diff --git a/location-providers/ne-location-provider/src/main/resources/org/opendaylight/blueprint/ne-location-provider.xml b/location-providers/ne-location-provider/src/main/resources/org/opendaylight/blueprint/ne-location-provider.xml
new file mode 100644
index 000000000..d6598a30c
--- /dev/null
+++ b/location-providers/ne-location-provider/src/main/resources/org/opendaylight/blueprint/ne-location-provider.xml
@@ -0,0 +1,12 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<blueprint xmlns="http://www.osgi.org/xmlns/blueprint/v1.0.0"
+           xmlns:odl="http://opendaylight.org/xmlns/blueprint/v1.0.0"
+           odl:use-default-for-reference-types="true">
+
+    <reference id="dataBroker" interface="org.opendaylight.controller.md.sal.binding.api.DataBroker"/>
+
+    <bean id="nelocationProvider" class="org.opendaylight.controller.config.yang.config.ne.location.provider.cfg.NeLocationProviderInstance"
+        destroy-method="close">
+        <argument ref="dataBroker"/>
+    </bean>
+</blueprint>
diff --git a/location-providers/ne-location-provider/src/main/yang/ne-location-provider-cfg.yang b/location-providers/ne-location-provider/src/main/yang/ne-location-provider-cfg.yang
index 8a58edac6..16034dbd0 100644
--- a/location-providers/ne-location-provider/src/main/yang/ne-location-provider-cfg.yang
+++ b/location-providers/ne-location-provider/src/main/yang/ne-location-provider-cfg.yang
@@ -33,16 +33,6 @@ module ne-location-provider-cfg {
     augment "/config:modules/config:module/config:configuration" {
         case ne-location-provider {
             when "/config:modules/config:module/config:type = 'ne-location-provider'";
-
-            // Wires in the data-broker service
-            container data-broker {
-                uses config:service-ref {
-                    refine type {
-                        mandatory true;
-                        config:required-identity mdsal:binding-async-data-broker;
-                    }
-                }
-            }
         }
     }
 }
-- 
2.36.6