Fix high priority findbugs reported issues

Fixed some code that was reporting findbugs errors

Implemented a suppression mechanism for findbugs
reported errors and a suppression file.

Change-Id: Ie8a2e84cc57ec6ddfa14d782ee89feb095b9dd59
diff --git a/pom.xml b/pom.xml
index 28b387e..99e2e08 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="UTF-8"?>
+ <?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0"
          xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
          xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
@@ -428,12 +428,6 @@
                 </plugin>
 
                 <plugin>
-                    <groupId>org.codehaus.mojo</groupId>
-                    <artifactId>findbugs-maven-plugin</artifactId>
-                    <version>3.0.0</version>
-                </plugin>
-
-                <plugin>
                     <groupId>org.apache.felix</groupId>
                     <artifactId>maven-scr-plugin</artifactId>
                     <version>1.20.0</version>
@@ -452,6 +446,22 @@
                         </supportedProjectTypes>
                     </configuration>
                 </plugin>
+                <plugin>
+                  <groupId>org.codehaus.mojo</groupId>
+                  <artifactId>findbugs-maven-plugin</artifactId>
+                  <version>3.0.0</version>
+                  <dependencies>
+                    <dependency>
+                      <groupId>org.onlab.tools</groupId>
+                      <artifactId>onos-build-conf</artifactId>
+                      <version>1.0</version>
+                    </dependency>
+                  </dependencies>
+                  <configuration>
+                    <effort>Max</effort>
+                    <excludeFilterFile>onos/findbugs-suppressions.xml</excludeFilterFile>
+                  </configuration>
+                </plugin>
 
                 <!-- TODO: add findbugs plugin for static code analysis; for explicit invocation only -->
                 <!--This plugin's configuration is used to store Eclipse m2e settings only. It has no influence on the Maven build itself.-->
@@ -586,7 +596,7 @@
                 <artifactId>maven-checkstyle-plugin</artifactId>
                 <version>2.12.1</version>
                 <configuration>
-                    <configLocation>onos/checkstyle.xml</configLocation>
+                    <!-- <configLocation>onos/checkstyle.xml</configLocation> -->
                 </configuration>
             </plugin>
 
@@ -598,10 +608,25 @@
                     <excludes>
                     </excludes>
                     <rulesets>
-                        <ruleset>onos/pmd.xml</ruleset>
+                        <!-- <ruleset>onos/pmd.xml</ruleset> -->
                     </rulesets>
                 </configuration>
             </plugin>
+            <plugin>
+              <groupId>org.codehaus.mojo</groupId>
+              <artifactId>findbugs-maven-plugin</artifactId>
+              <version>3.0.0</version>
+              <configuration>
+                <effort>$Max</effort>
+                <!-- <excludeFilterFile>${findbugs.excludeFilterFile}</excludeFilterFile> -->
+                <reportPlugins>
+                  <plugin>
+                    <groupId>org.codehaus.mojo</groupId>
+                    <artifactId>findbugs-maven-plugin</artifactId>
+                  </plugin>
+                </reportPlugins>
+              </configuration>
+            </plugin>
         </plugins>
     </reporting>
 </project>