RANGER-5019: Fix spotbug checks for sub-modules (#441)

It fixes mvn builds at sub-module level and allows 'mvn install' to run at every submodule.

(cherry picked from commit 9930255)

Author: kumaab

agents-common/dev-support/spotbugsIncludeFile.xml

@@ -203,6 +203,7 @@
         <sonar.dynamicAnalysis>reuseReports</sonar.dynamicAnalysis>
         <sonar.java.coveragePlugin>jacoco</sonar.java.coveragePlugin>
         <sonar.language>java</sonar.language>
+        <spotbugs.failOnViolation>false</spotbugs.failOnViolation>
         <spotbugs.plugin.version>4.7.3.5</spotbugs.plugin.version>
         <spring-ldap-core.version>2.4.1</spring-ldap-core.version>
         <springframework.security.version>5.7.12</springframework.security.version>
@@ -659,14 +660,17 @@
                 <groupId>com.github.spotbugs</groupId>
                 <artifactId>spotbugs-maven-plugin</artifactId>
                 <version>${spotbugs.plugin.version}</version>
-                <configuration>
-                    <includeFilterFile>./dev-support/spotbugsIncludeFile.xml</includeFilterFile>
-                </configuration>
                 <executions>
                     <execution>
+                        <id>spotbugs-check</id>
                         <goals>
                             <goal>check</goal>
                         </goals>
+                        <phase>verify</phase>
+                        <configuration>
+                            <failOnError>${spotbugs.failOnViolation}</failOnError>
+                            <includeFilterFile>./dev-support/spotbugsIncludeFile.xml</includeFilterFile>
+                        </configuration>
                     </execution>
                 </executions>
             </plugin>

kms/dev-support/findbugsExcludeFile.xml

@@ -1208,7 +1208,7 @@
                             <rules>
                                 <evaluateBeanshell>
                                     <message>Looks like there are multiple SQL files with same version number prefix. Update prefix and build again.</message>
-                                    <condition>List sqlFilePaths = org.codehaus.plexus.util.FileUtils.getFileNames(new File("security-admin/db"), "**/*.sql", null, false);
+                                    <condition>List sqlFilePaths = org.codehaus.plexus.util.FileUtils.getFileNames(new File("${project.basedir}/db"), "**/*.sql", null, false);
                                         Map sqlFileMap    = new HashMap();
                                         Boolean noDupPrfx = true;
 
@@ -1255,7 +1255,7 @@
                             <rules>
                                 <evaluateBeanshell>
                                     <message>Looks like there are multiple JAVA files with same version number suffix. Update suffix and build again.</message>
-                                    <condition>List javaFilePaths  = org.codehaus.plexus.util.FileUtils.getFileNames(new File("security-admin/src/main/java/org/apache/ranger/patch"), "**/*.java", null, false);
+                                    <condition>List javaFilePaths  = org.codehaus.plexus.util.FileUtils.getFileNames(new File("${project.basedir}/src/main/java/org/apache/ranger/patch"), "**/*.java", null, false);
                                         Map javaFileMap     = new HashMap();
                                         Boolean noDupSuffix = true;