Remove the need for root access (#48)

#43

Author: edeandrea

scripts/infra.sh

@@ -21,6 +21,8 @@ exit_abnormal() {
 start_postgres() {
   echo "Starting PostgreSQL database '${DB_CONTAINER_NAME}'"
   local pid=$(${engine} run \
+    --cpus 2 \
+    --memory 128m \
     -d \
     --rm \
     --name ${DB_CONTAINER_NAME} \

scripts/perf-lab/helpers/cgroups.yml

@@ -34,12 +34,20 @@ scripts:
             - script: sudo
               with:
                 command: echo "+memory" >>  /sys/fs/cgroup/${{CGROUP_NAME:perf-cgroup}}/cgroup.subtree_control
-            - script: sudo
-              with:
-                command: echo ${{CGROUP_CPUSET}} > /sys/fs/cgroup/${{CGROUP_NAME:perf-cgroup}}/tasks/cpuset.cpus
-            - script: sudo
-              with:
-                command: echo ${{CGROUP_MAX_MEM}} >  /sys/fs/cgroup/${{CGROUP_NAME:perf-cgroup}}/tasks/memory.max
+            - read-state: ${{CGROUP_CPUSET}}
+              then:
+                - regex: .+
+                  then:
+                    - script: sudo
+                      with:
+                        command: echo ${{CGROUP_CPUSET}} > /sys/fs/cgroup/${{CGROUP_NAME:perf-cgroup}}/tasks/cpuset.cpus
+            - read-state: ${{CGROUP_MAX_MEM}}
+              then:
+                - regex: .+
+                  then:
+                    - script: sudo
+                      with:
+                        command: echo ${{CGROUP_MAX_MEM}} >  /sys/fs/cgroup/${{CGROUP_NAME:perf-cgroup}}/tasks/memory.max
           else:
             - log: "Non-Linux OS doesn't support cgroups"
 

scripts/perf-lab/run-benchmarks.sh

@@ -20,18 +20,17 @@ help() {
   echo "  -f <OUTPUT_DIR>                     The directory containing the run output"
   echo "                                          Default: /tmp"
   echo "  -g <GRAALVM_VERSION>                The GraalVM version to use if running any native tests (from SDKMAN)"
-  echo "                                          Default: 25-graalce"
+  echo "                                          Default: 25.0.1-graalce"
   echo "  -h <HOST>                           The HOST to run the benchmarks on"
   echo "                                          LOCAL is a keyword that can be used to run everything on the local machine"
   echo "                                          Default: LOCAL"
   echo "  -i <ITERATIONS>                     The number of iterations to run each test"
   echo "                                          Default: 3"
   echo "  -j <JAVA_VERSION>                   The Java version to use (from SDKMAN)"
-  echo "                                          Default: 25-tem"
+  echo "                                          Default: 25.0.1-tem"
   echo "  -l <SCM_REPO_URL>                   The SCM repo url"
   echo "                                          Default: 'https://github.com/quarkusio/spring-quarkus-perf-comparison.git'"
   echo "  -m <CGROUPS_MAX_MEMORY>             Constrain available memory via cgroups"
-  echo "                                          Default: 14G"
   echo "  -n <NATIVE_QUARKUS_BUILD_OPTIONS>   Native build options to be passed to Quarkus native build process"
   echo "  -o <NATIVE_SPRING_BUILD_OPTIONS>    Native build options to be passed to Spring native build process"
   echo "  -p <PROFILER>                       Enable profiling with async profiler"
@@ -208,11 +207,11 @@ ${JBANG_CMD} qDup@hyperfoil \
 CGROUPS_CPUS=""
 SCM_REPO_URL="https://github.com/quarkusio/spring-quarkus-perf-comparison.git"
 SCM_REPO_BRANCH="main"
-GRAALVM_VERSION="25-graalce"
+GRAALVM_VERSION="25.0.1-graalce"
 HOST="LOCAL"
 ITERATIONS="3"
-JAVA_VERSION="25-tem"
-CGROUPS_MAX_MEMORY="14G"
+JAVA_VERSION="25.0.1-tem"
+CGROUPS_MAX_MEMORY=""
 NATIVE_QUARKUS_BUILD_OPTIONS=""
 NATIVE_SPRING_BUILD_OPTIONS=""
 PROFILER="none"