From 76c69aee83a0ed7cc5f9d9f4b9bb7d73c4771585 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?David=20Mat=C4=9Bj=C4=8Dek?= <david.matejcek@omnifish.ee>
Date: Tue, 31 Oct 2023 15:39:47 +0100
Subject: [PATCH] Documenting the aggregator profiles + added qa profile +
 updated Jenkinsfile
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
---
 Jenkinsfile |  4 ++--
 pom.xml     | 29 ++++++++++++++++++++++++++++-
 2 files changed, 30 insertions(+), 3 deletions(-)

diff --git a/Jenkinsfile b/Jenkinsfile
index 68abb4da0f4..3b20dd0752e 100644
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -208,7 +208,7 @@ spec:
           dumpSysInfo()
           timeout(time: 1, unit: 'HOURS') {
             sh '''
-                mvn -B -e clean install -Pstaging -P'!docs'
+                mvn -B -e clean install -Pstaging,qa
             '''
           }
         }
@@ -243,7 +243,7 @@ spec:
           dumpSysInfo()
           timeout(time: 1, unit: 'HOURS') {
             sh '''
-                mvn -B -e clean install -Pstaging -Pdocs
+                mvn -B -e clean install -Pstaging -f docs -amd
             '''
           }
         }
diff --git a/pom.xml b/pom.xml
index aab11042a19..84be9c1a798 100644
--- a/pom.xml
+++ b/pom.xml
@@ -96,6 +96,10 @@
     </build>
 
     <profiles>
+        <!--
+        This profile is implicit when you run maven clean install and is automatically disabled
+        when you choose another from this set.
+        -->
         <profile>
             <id>default</id>
             <activation>
@@ -109,7 +113,10 @@
             </modules>
         </profile>
 
-        <!-- Ommits a variety of things in order to get the fastest build of a basic full glassfish server -->
+        <!--
+        Build glassfixh.zip as soon as possible.
+        Profile of the same id is also in parent, together they exclude unimportant modules and plugin executions.
+         -->
         <profile>
             <id>fastest</id>
             <modules>
@@ -119,6 +126,21 @@
             </modules>
         </profile>
 
+        <!--
+        Build all distributed binaries except documentation and run all maven tests.
+        -->
+        <profile>
+            <id>qa</id>
+            <modules>
+                <module>qa</module>
+                <module>nucleus</module>
+                <module>appserver</module>
+            </modules>
+        </profile>
+
+        <!--
+        Change version number in all artifacts in the repository.
+        -->
         <profile>
             <id>set-version-id</id>
             <modules>
@@ -132,6 +154,11 @@
             </modules>
         </profile>
 
+        <!--
+        Merges all exec files in the directory tree - useful when you want to se the code coverage
+        produced by integration tests executed by different modules, or exactly opposite - to see
+        not covered by tests at all.
+        -->
         <profile>
             <id>jacoco-merge</id>
             <build>