wrong feature dependencies in newly added features. 56/70856/5
authorFaseela K <faseela.k@ericsson.com>
Thu, 12 Apr 2018 15:23:30 +0000 (20:53 +0530)
committerFaseela K <faseela.k@ericsson.com>
Thu, 12 Apr 2018 16:55:19 +0000 (22:25 +0530)
Change-Id: Ie592de3ee8f6ba01c86e7d411dad8d5c411cf59c
Signed-off-by: Faseela K <faseela.k@ericsson.com>
features/odl-genius-tools/pom.xml
features/odl-genius/pom.xml
srm/shell/pom.xml
srm/shell/src/main/java/org/opendaylight/genius/srm/shell/SrmDebugCommand.java
tools/api/pom.xml
tools/testutils/pom.xml

index 35918cfda7acc3ab0a071abf1452aa156a72c7b2..0d5211a25037da2669811634acee51d27d508c9b 100644 (file)
@@ -25,13 +25,22 @@ and is available at http://www.eclipse.org/legal/epl-v10.html INTERNAL
 
   <dependencies>
     <dependency>
-      <groupId>org.opendaylight.genius</groupId>
-      <artifactId>tools-api</artifactId>
-      <version>${project.version}</version>
+      <groupId>org.opendaylight.controller</groupId>
+      <artifactId>odl-mdsal-broker</artifactId>
+      <version>1.8.0-SNAPSHOT</version>
+      <type>xml</type>
+      <classifier>features</classifier>
+    </dependency>
+    <dependency>
+      <groupId>org.opendaylight.infrautils</groupId>
+      <artifactId>odl-infrautils-metrics</artifactId>
+      <version>1.4.0-SNAPSHOT</version>
+      <type>xml</type>
+      <classifier>features</classifier>
     </dependency>
     <dependency>
       <groupId>org.opendaylight.genius</groupId>
-      <artifactId>tools-testutils</artifactId>
+      <artifactId>tools-api</artifactId>
       <version>${project.version}</version>
     </dependency>
   </dependencies>
index caf6dae8fa2b32938562ed58713b998e9fd192cb..6804dfe540e30e34668257c4ac68abfdb0b7c232 100644 (file)
       <type>xml</type>
       <classifier>features</classifier>
     </dependency>
+    <dependency>
+      <groupId>org.opendaylight.genius</groupId>
+      <artifactId>odl-genius-srm</artifactId>
+      <type>xml</type>
+      <classifier>features</classifier>
+    </dependency>
     <dependency>
       <groupId>org.opendaylight.infrautils</groupId>
       <artifactId>odl-infrautils-diagstatus</artifactId>
       <type>xml</type>
       <classifier>config</classifier>
     </dependency>
-    <dependency>
-      <groupId>org.opendaylight.genius</groupId>
-      <artifactId>srm-impl</artifactId>
-      <version>${project.version}</version>
-    </dependency>
-    <dependency>
-      <groupId>org.opendaylight.genius</groupId>
-      <artifactId>srm-shell</artifactId>
-      <version>${project.version}</version>
-    </dependency>
   </dependencies>
 </project>
index 32cfd78ca878c93b513715919e4a4ad03f859bed..6327ba30772006fdc61f7c7b09bee39b4d6f5f98 100644 (file)
@@ -25,11 +25,6 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <name>ODL :: genius :: ${project.artifactId}</name>
 
   <dependencies>
-    <dependency>
-      <groupId>org.opendaylight.genius</groupId>
-      <artifactId>mdsalutil-api</artifactId>
-      <version>${genius.version}</version>
-    </dependency>
     <dependency>
       <groupId>org.opendaylight.genius</groupId>
       <artifactId>srm-api</artifactId>
index fbc57826e64a4672ea42ae1ad1718855942dcd8a..176a8d32031dea97b189630e31c3c86aa154f901 100644 (file)
@@ -12,8 +12,8 @@ import org.apache.karaf.shell.commands.Command;
 import org.apache.karaf.shell.commands.Option;
 import org.apache.karaf.shell.console.OsgiCommandSupport;
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
+import org.opendaylight.controller.md.sal.binding.api.WriteTransaction;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
-import org.opendaylight.genius.datastoreutils.SingleTransactionDataBroker;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.srm.ops.rev170711.ServiceOps;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 import org.slf4j.Logger;
@@ -24,10 +24,10 @@ public class SrmDebugCommand extends OsgiCommandSupport {
 
     private static final Logger LOG = LoggerFactory.getLogger(SrmDebugCommand.class);
 
-    private final SingleTransactionDataBroker txDataBroker;
+    private final DataBroker txDataBroker;
 
     public SrmDebugCommand(DataBroker dataBroker) {
-        this.txDataBroker = new SingleTransactionDataBroker(dataBroker);
+        this.txDataBroker = dataBroker;
     }
 
     @Option(name = "-c", aliases = {"--clear-ops"}, description = "Clear operations DS",
@@ -44,7 +44,9 @@ public class SrmDebugCommand extends OsgiCommandSupport {
 
     private void clearOpsDs() throws Exception {
         InstanceIdentifier<ServiceOps> path = getInstanceIdentifier();
-        txDataBroker.syncDelete(LogicalDatastoreType.OPERATIONAL, path);
+        WriteTransaction tx = txDataBroker.newWriteOnlyTransaction();
+        tx.delete(LogicalDatastoreType.OPERATIONAL, path);
+        tx.submit();
     }
 
     private static InstanceIdentifier<ServiceOps> getInstanceIdentifier() {
index ffa7ccf3c30db316e7367dbe31eca2ecbf6e69bb..2e2b8d668005d720d0e5183d5934eff42387fa51 100644 (file)
@@ -34,11 +34,6 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
       <groupId>org.opendaylight.controller</groupId>
       <artifactId>sal-binding-api</artifactId>
     </dependency>
-    <dependency>
-      <groupId>org.opendaylight.infrautils</groupId>
-      <artifactId>inject</artifactId>
-      <version>${genius.infrautils.version}</version>
-    </dependency>
     <dependency>
       <groupId>org.opendaylight.infrautils</groupId>
       <artifactId>metrics-api</artifactId>
@@ -49,6 +44,10 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
       <artifactId>infrautils-util</artifactId>
       <version>${genius.infrautils.version}</version>
     </dependency>
+    <dependency>
+      <groupId>javax.inject</groupId>
+      <artifactId>javax.inject</artifactId>
+    </dependency>
     <dependency>
       <groupId>org.immutables</groupId>
       <artifactId>value</artifactId>
@@ -62,48 +61,5 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
       <artifactId>pax-cdi-api</artifactId>
       <optional>true</optional>
     </dependency>
-
-    <!-- Dependencies with <scope>test -->
-    <dependency>
-      <groupId>org.opendaylight.yangtools</groupId>
-      <artifactId>testutils</artifactId>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>org.awaitility</groupId>
-      <artifactId>awaitility</artifactId>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>org.opendaylight.mdsal</groupId>
-      <artifactId>mdsal-binding-test-utils</artifactId>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>org.opendaylight.infrautils</groupId>
-      <artifactId>infrautils-testutils</artifactId>
-      <version>${genius.infrautils.version}</version>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>com.google.truth</groupId>
-      <artifactId>truth</artifactId>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>com.google.guava</groupId>
-      <artifactId>guava-testlib</artifactId>
-    </dependency>
-    <dependency>
-       <groupId>org.opendaylight.controller</groupId>
-       <artifactId>sal-binding-broker-impl</artifactId>
-       <scope>test</scope>
-     </dependency>
-     <dependency>
-       <groupId>org.opendaylight.controller</groupId>
-       <artifactId>sal-binding-broker-impl</artifactId>
-       <scope>test</scope>
-       <type>test-jar</type>
-     </dependency>
   </dependencies>
 </project>
index 04dcf583071440db91193771fb499a20f410b14c..e8664b1330d73fb92fdd32d1b5a31ef7c707c68b 100644 (file)
@@ -70,6 +70,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
       <groupId>org.opendaylight.infrautils</groupId>
       <artifactId>infrautils-testutils</artifactId>
       <version>1.4.0-SNAPSHOT</version>
+      <scope>compile</scope>
     </dependency>
   </dependencies>
 </project>