Override MaxRAMPercentage

Override MaxRAMPercentage for JVMs invoked by Maven to avoid
attempting to use too much of the pod RAM.

Signed-off-by: Jonathan Dowland <[email protected]>

Author: jmtd

modules/maven/default/artifacts/opt/jboss/container/maven/default/maven.sh

@@ -34,7 +34,7 @@ function maven_init_var_MAVEN_SETTINGS_XML() {
 }
 
 function maven_init_var_MAVEN_OPTS() {
-  export MAVEN_OPTS="${MAVEN_OPTS:-$(${JBOSS_CONTAINER_JAVA_JVM_MODULE}/java-default-options)}"
+  export MAVEN_OPTS="${MAVEN_OPTS:-$(${JBOSS_CONTAINER_JAVA_JVM_MODULE}/java-default-options) -XX:MaxRAMPercentage=25.0}"
 }
 
 function maven_init_var_MAVEN_ARGS() {

tests/features/java/memory.feature

@@ -1,17 +1,26 @@
-@ubi9
 Feature: OPENJDK-559 JVM Memory tests
 
+  @ubi9
   Scenario: Check default JVM max heap configuration
     Given container is started as uid 1000
     Then container log should contain -XX:MaxRAMPercentage=80.0
 
+  @ubi9
   Scenario: Check configured JVM max heap configuration
     Given container is started with env
     | variable           | value  |
     | JAVA_MAX_MEM_RATIO | 90.0   |
     Then container log should contain -XX:MaxRAMPercentage=90.0
 
+  @ubi9
   Scenario: Check default JVM initial heap configuration is unspecified
     Given container is started as uid 1000
     Then container log should not contain -XX:InitialRAMPercentage
     And  container log should not contain -Xms
+
+  # Not the runtime images
+  @ubi9/openjdk-11
+  @ubi9/openjdk-17
+  Scenario: Ensure Maven doesn't use MaxRAMPercentage=80
+    Given s2i build https://github.com/jboss-openshift/openshift-examples from spring-boot-sample-simple
+    Then s2i build log should match regex INFO Using MAVEN_OPTS.*-XX:MaxRAMPercentage=25.0$