From cf5a21a31e13aa72d3565259dd57b8e759161d7e Mon Sep 17 00:00:00 2001
From: Romain Buisson <rbuisson@users.noreply.github.com>
Date: Fri, 21 Jun 2024 12:12:59 +0200
Subject: [PATCH] OZ-429: Fine tune the Superset configs packaging (#353)

---
 base/configs/superset/assets/chart.json |  0
 base/configs/superset/pom.xml           |  1 +
 base/pom.xml                            | 58 ++++++++++++++++++++++++-
 3 files changed, 58 insertions(+), 1 deletion(-)
 create mode 100644 base/configs/superset/assets/chart.json

diff --git a/base/configs/superset/assets/chart.json b/base/configs/superset/assets/chart.json
new file mode 100644
index 00000000..e69de29b
diff --git a/base/configs/superset/pom.xml b/base/configs/superset/pom.xml
index f6a046c9..6f370bed 100644
--- a/base/configs/superset/pom.xml
+++ b/base/configs/superset/pom.xml
@@ -15,6 +15,7 @@
     <name>Ozone Cambodia Superset configs</name>
     <description>Superset configurations specific to Ozone Cambodia, to be used in Ozone Analytics</description>
     <packaging>pom</packaging>
+    <version>1.7.0-SNAPSHOT</version>
 
     <organization>
         <name>Ozone HIS</name>
diff --git a/base/pom.xml b/base/pom.xml
index 7cacf151..da1428c9 100644
--- a/base/pom.xml
+++ b/base/pom.xml
@@ -21,6 +21,8 @@
     <maven.compiler.source>8</maven.compiler.source>
     <maven.compiler.target>8</maven.compiler.target>
 
+    <supersetConfigsArtifactId>ozone-cambodia-superset-configs</supersetConfigsArtifactId>
+
     <!-- Classifier for the dependency report artifact -->
     <dependencyReportClassifier>dependencies</dependencyReportClassifier>
   </properties>
@@ -30,11 +32,40 @@
   </modules>
 
   <dependencies>
-    <!-- TODO: Add any implementation-specific dependencies -->
+
+    <!-- Ozone Cambodia Superset configs sub-module -->
+    <dependency>
+      <groupId>${project.groupId}</groupId>
+      <artifactId>${supersetConfigsArtifactId}</artifactId>
+      <version>${project.version}</version>
+      <type>zip</type>
+    </dependency>
+
   </dependencies>
 
   <build>
     <plugins>
+
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-dependency-plugin</artifactId>
+        <executions>
+          <execution>
+            <id>Unpack Cambodia Superset configs sub-module</id>
+            <phase>generate-resources</phase>
+            <goals>
+              <goal>unpack-dependencies</goal>
+            </goals>
+            <configuration>
+              <excludeTransitive>true</excludeTransitive>
+              <outputDirectory>
+                ${project.build.directory}/${project.artifactId}-${project.version}/configs/superset</outputDirectory>
+              <includeArtifactIds>${supersetConfigsArtifactId}</includeArtifactIds>
+            </configuration>
+          </execution>
+        </executions>
+      </plugin>
+
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-resources-plugin</artifactId>
@@ -72,6 +103,31 @@
               </resources>
             </configuration>
           </execution>
+
+          <execution>
+            <!-- Override the inherited Maven Parent plugin execution to exclude Superset configs -->
+            <!-- Override with local config files -->
+            <id>Copy local resources</id>
+            <phase>process-resources</phase>
+            <goals>
+              <goal>copy-resources</goal>
+            </goals>
+            <configuration>
+              <outputDirectory>
+                ${project.build.directory}/${project.artifactId}-${project.version}/distro/configs</outputDirectory>
+              <overwrite>true</overwrite>
+              <resources>
+                <resource>
+                  <directory>${project.basedir}/configs</directory>
+                  <excludes>
+                    <exclude>.gitkeep</exclude>
+                    <exclude>superset/</exclude>
+                  </excludes>
+                </resource>
+              </resources>
+            </configuration>
+          </execution>
+
         </executions>
       </plugin>