Round up fractional cpu values for ActiveProcessorsCount (#944)

* Round up fractional cpu values for ActiveProcessorsCount

* Update upload-artifact action version

* Artifacts

---------

Co-authored-by: Yevhen Ivantsov <[email protected]>

Author: bianchi2

.github/workflows/e2e-tf-deployment.yaml

@@ -156,7 +156,7 @@ jobs:
 
       - name: Upload test log files
         if: always()
-        uses: actions/upload-artifact@v3.1.1
+        uses: actions/upload-artifact@v4
         with:
           name: e2e-test-artifacts
           path: tf/test/e2etest/artifacts/

.github/workflows/kind.yaml

@@ -97,7 +97,7 @@ jobs:
 
       - name: Upload test log files
         if: always()
-        uses: actions/upload-artifact@v3.1.1
+        uses: actions/upload-artifact@v4
         with:
-          name: kind-artifacts
+          name: kind-artifacts-${{inputs.dc_app}}
           path: logs/

.github/workflows/openshift.yaml

@@ -136,7 +136,7 @@ jobs:
 
       - name: Upload test log files
         if: always()
-        uses: actions/upload-artifact@v3.1.1
+        uses: actions/upload-artifact@v4
         with:
           name: kind-artifacts
           path: logs/

src/main/charts/bamboo/templates/config-jvm.yaml

@@ -9,7 +9,7 @@ data:
     {{- range .Values.bamboo.additionalJvmArgs }}
     {{ . }}
     {{- end }}
-    -XX:ActiveProcessorCount={{ include "flooredCPU" .Values.bamboo.resources.container.requests.cpu }}
+    -XX:ActiveProcessorCount={{ max 1 (include "flooredCPU" .Values.bamboo.resources.container.requests.cpu | float64  | int) }}
     {{- if or .Values.bamboo.additionalCertificates.secretName .Values.bamboo.additionalCertificates.secretList }}
     -Djavax.net.ssl.trustStore=/var/ssl/cacerts
     {{- end }}

src/main/charts/bitbucket/templates/config-jvm.yaml

@@ -9,7 +9,7 @@ data:
     {{- range .Values.bitbucket.additionalJvmArgs }}
     {{ . }}
     {{- end }}
-    -XX:ActiveProcessorCount={{ include "flooredCPU" .Values.bitbucket.resources.container.requests.cpu }}
+    -XX:ActiveProcessorCount={{ max 1 (include "flooredCPU" .Values.bitbucket.resources.container.requests.cpu | float64  | int) }}
     {{ include "common.jmx.javaagent" . | indent 4 | trim }}
     {{- if .Values.monitoring.exposeJmxMetrics }}
     -Dplugin.bitbucket-git.mesh.sidecar.jvmArgs=-javaagent:{{ .Values.monitoring.jmxExporterCustomJarLocation | default (printf "%s/jmx_prometheus_javaagent.jar"  .Values.volumes.sharedHome.mountPath) }}=9998:/opt/atlassian/jmx/jmx-config.yaml

src/main/charts/confluence/templates/config-jvm.yaml

@@ -15,7 +15,7 @@ data:
     {{- range .Values.confluence.additionalJvmArgs }}
     {{ . }}
     {{- end }}
-    -XX:ActiveProcessorCount={{ include "flooredCPU" .Values.confluence.resources.container.requests.cpu }}
+    -XX:ActiveProcessorCount={{ max 1 (include "flooredCPU" .Values.confluence.resources.container.requests.cpu | float64  | int) }}
     {{- if .Values.serviceAccount.eksIrsa.roleArn }}
     -Daws.webIdentityTokenFile=/var/run/secrets/eks.amazonaws.com/serviceaccount/token
     {{- end }}

src/main/charts/crowd/templates/config-jvm.yaml

@@ -11,7 +11,7 @@ data:
     {{- range .Values.crowd.additionalJvmArgs }}
     {{ . }}
     {{- end }}
-    -XX:ActiveProcessorCount={{ include "flooredCPU" .Values.crowd.resources.container.requests.cpu }}
+    -XX:ActiveProcessorCount={{ max 1 (include "flooredCPU" .Values.crowd.resources.container.requests.cpu | float64  | int) }}
     {{- if or .Values.crowd.additionalCertificates.secretName .Values.crowd.additionalCertificates.secretList }}
     -Djavax.net.ssl.trustStore=/var/ssl/cacerts
     {{- end }}

src/main/charts/jira/templates/config-jvm.yaml

@@ -10,7 +10,7 @@ data:
     {{- range .Values.jira.additionalJvmArgs }}
     {{ . }}
     {{- end }}
-    -XX:ActiveProcessorCount={{ include "flooredCPU" .Values.jira.resources.container.requests.cpu }}
+    -XX:ActiveProcessorCount={{ max 1 (include "flooredCPU" .Values.jira.resources.container.requests.cpu | float64  | int) }}
     {{- if .Values.serviceAccount.eksIrsa.roleArn }}
     -Daws.webIdentityTokenFile=/var/run/secrets/eks.amazonaws.com/serviceaccount/token
     {{- end }}

src/test/java/test/RequestsTest.java

@@ -74,4 +74,25 @@ void sts_resource_requests_and_limits(Product product) throws Exception {
             .hasSeparatedTextContaining("-XX:ActiveProcessorCount=1");
     }
 
+    @ParameterizedTest
+    @EnumSource(value = Product.class, names = {"bamboo_agent"}, mode = EnumSource.Mode.EXCLUDE)
+    void fractional_cpu_active_processors(Product product) throws Exception {
+        final var resources = helm.captureKubeResourcesFromHelmChart(product, Map.of(
+                product + ".resources.container.requests.cpu", "3.2"
+        ));
+
+        final var additionalJvmArgs = resources.get(Kind.ConfigMap, product.getHelmReleaseName() + "-jvm-config")
+                .getNode("data", "additional_jvm_args");
+        assertThat(additionalJvmArgs)
+                .hasSeparatedTextContaining("-XX:ActiveProcessorCount=3");
+
+        final var resourcesFractionalCPU = helm.captureKubeResourcesFromHelmChart(product, Map.of(
+                product + ".resources.container.requests.cpu", "0.1"
+        ));
+        final var additionalJvmArgsMilliCpu = resourcesFractionalCPU
+                .get(Kind.ConfigMap, product.getHelmReleaseName() + "-jvm-config").getNode("data", "additional_jvm_args");
+        assertThat(additionalJvmArgsMilliCpu)
+                .hasSeparatedTextContaining("-XX:ActiveProcessorCount=1");
+    }
+
 }