Rename benchmark workflow and fix OpenLDAP version-capture SIGPIPE

Valera V Harseko

2 days ago 75c17bfaedc1288975c725926de2aa61649a7e83

Rename benchmark workflow and fix OpenLDAP version-capture SIGPIPE

Renames the workflow and its assets, and fixes the job failing before the
benchmark ran.

- Rename .github/workflows/performance.yml -> benchmark.yml (workflow name
  "LDAP benchmark: OpenDJ vs OpenLDAP") and .github/performance/ ->
  .github/benchmark/; update all internal paths and the concurrency group.
- Fix the "Capture OpenLDAP version" step failing with exit 141: `slapd -VV`
  piped to `head -1` got SIGPIPE when head closed the pipe early, and under
  `pipefail` that failed the whole step before any JMeter run. Capture the
  output fully and take the first line in bash, with a fallback to the image
  reference.
- Harden the OpenDJ version step the same way: guard ldapsearch with `|| true`
  so a transient bind failure cannot trip pipefail.

 .github/benchmark/benchmark.jmx

 .github/benchmark/people.ldif

 .github/benchmark/summary.sh

 .github/workflows/benchmark.yml

File was renamed from .github/workflows/performance.yml
@@ -11,7 +11,7 @@
# information: "Portions copyright [year] [name of copyright owner]".
#
# Copyright 2026 3A Systems, LLC.
name: Performance
name: "LDAP benchmark: OpenDJ vs OpenLDAP"

# LDAP benchmark: OpenDJ vs OpenLDAP.
# Runs manually (workflow_dispatch) and automatically after the "Release" workflow completes.
@@ -49,7 +49,7 @@
  contents: read

concurrency:
  group: performance-${{ github.ref }}
  group: benchmark-${{ github.ref }}
  cancel-in-progress: true

defaults:
@@ -113,11 +113,15 @@
            sleep 2
          done
          ldapadd -x -H ldap://localhost:2389 -D "cn=admin,$BASEDN" -w password \
            -f .github/performance/people.ldif || true
            -f .github/benchmark/people.ldif || true

      - name: Capture OpenLDAP version
        run: |
          ver="$(docker exec openldap sh -c '/usr/sbin/slapd -VV 2>&1 || slapd -VV 2>&1' | head -1)"
          # No `| head -1`: head closes the pipe early, slapd -VV then gets SIGPIPE and under
          # `pipefail` fails the step with exit 141. Capture fully, take the first line in bash.
          ver="$(docker exec openldap sh -c '/usr/sbin/slapd -VV 2>&1 || slapd -VV 2>&1' || true)"
          ver="${ver%%$'\n'*}"
          [ -n "$ver" ] || ver="$OPENLDAP_IMAGE"
          echo "OpenLDAP: $ver"
          {
            echo "OPENLDAP_VER<<EOF"
@@ -128,7 +132,7 @@
      - name: Benchmark OpenLDAP
        run: |
          rm -rf openldap openldap.jtl
          HEAP="$HEAP" "$JMETER_BIN" -n -t .github/performance/benchmark.jmx \
          HEAP="$HEAP" "$JMETER_BIN" -n -t .github/benchmark/benchmark.jmx \
            -Jhost=localhost -Jport=2389 \
            -Jbasedn="$BASEDN" \
            -Jadminbinddn="cn=admin,$BASEDN" -Jadminbindpw=password \
@@ -155,12 +159,14 @@
            sleep 2
          done
          ldapadd -x -H ldap://localhost:1389 -D "cn=Directory Manager" -w password \
            -f .github/performance/people.ldif || true
            -f .github/benchmark/people.ldif || true

      - name: Capture OpenDJ version
        run: |
          ver="$(ldapsearch -x -LLL -H ldap://localhost:1389 -D 'cn=Directory Manager' -w password \
                  -b '' -s base fullVendorVersion 2>/dev/null | sed -n 's/^fullVendorVersion: //p')"
          # `|| true` guards the bind so a transient ldapsearch failure can't trip pipefail;
          # sed reads all input (no early pipe close), so no SIGPIPE here.
          ver="$( { ldapsearch -x -LLL -H ldap://localhost:1389 -D 'cn=Directory Manager' -w password \
                    -b '' -s base fullVendorVersion 2>/dev/null || true; } | sed -n 's/^fullVendorVersion: //p')"
          [ -n "$ver" ] || ver="$OPENDJ_IMAGE"
          echo "OpenDJ: $ver"
          {
@@ -172,7 +178,7 @@
      - name: Benchmark OpenDJ
        run: |
          rm -rf opendj opendj.jtl
          HEAP="$HEAP" "$JMETER_BIN" -n -t .github/performance/benchmark.jmx \
          HEAP="$HEAP" "$JMETER_BIN" -n -t .github/benchmark/benchmark.jmx \
            -Jhost=localhost -Jport=1389 \
            -Jbasedn="$BASEDN" \
            -Jadminbinddn="cn=Directory Manager" -Jadminbindpw=password \
@@ -184,7 +190,7 @@
      # ---------------------------------------------------------------- Report
      - name: Build job summary
        run: |
          bash .github/performance/summary.sh \
          bash .github/benchmark/summary.sh \
            openldap/statistics.json opendj/statistics.json \
            "$OPENLDAP_VER" "$OPENDJ_VER" \
            "$OPENLDAP_IMAGE" "$OPENDJ_IMAGE" >> "$GITHUB_STEP_SUMMARY"