From bfd7f8f8ecd146ddea5adc0bb65b1557e7bfe898 Mon Sep 17 00:00:00 2001 From: Dan Bonachea Date: Mon, 11 May 2026 22:06:58 -0700 Subject: [PATCH 1/4] CI: Add a run-name for workflow_dispatch --- .github/workflows/build.yml | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 883029b57..dab477614 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -1,4 +1,5 @@ name: Smoke Test +run-name: ${{ github.event_name == 'workflow_dispatch' && 'Manual:' || '' }} ${{ inputs.run_name }} on: push: @@ -15,6 +16,10 @@ on: paths-ignore: *paths_ignore workflow_dispatch: inputs: + run_name: + description: 'Descriptive name for this run' + type: string + required: false configure_args: description: 'Additional configure arguments' type: string From 2f2a301fb14bef1917c09084f60696890fcf1e60 Mon Sep 17 00:00:00 2001 From: Dan Bonachea Date: Tue, 12 May 2026 12:30:40 -0700 Subject: [PATCH 2/4] CI: Improve GASNET_SUPERNODE_MAXSIZE handling * Don't overwrite a user-provided setting * Default to 1 instead of infinite --- .github/workflows/build.yml | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index dab477614..4b6f44bf7 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -270,8 +270,12 @@ jobs: - name: Run udp-conduit tests run: | - if [[ ${TEST_NODES} > 2 ]] ; then - export GASNET_SUPERNODE_MAXSIZE=2 + if [[ -z "$GASNET_SUPERNODE_MAXSIZE" ]] ; then + if [[ ${TEST_NODES} > 2 ]] ; then + export GASNET_SUPERNODE_MAXSIZE=2 + else + export GASNET_SUPERNODE_MAXSIZE=1 + fi fi make ${MAKE_FLAGS} -C udp-conduit run-tests TEST_NODES=${TEST_NODES} "${TEST_FILTER}" 2>&1 | tee output grep -q -e 'Test results: 0 failures' output From 11c1e331c0ea421d0f3e7d7ee46c9646c908265f Mon Sep 17 00:00:00 2001 From: Dan Bonachea Date: Tue, 12 May 2026 20:00:03 -0700 Subject: [PATCH 3/4] CI: Change the default envvars for dispatch GASNET_STATSFILE=- is pretty unreadable in the logs More importantly, providing a non-empty string default has the unintuitive behavior that clearing it out results in the default being applied! --- .github/workflows/build.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 4b6f44bf7..ab03f752e 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -35,9 +35,9 @@ on: default: 4 required: true env_vars: - description: 'Environment variable settings' + description: 'Environment vars: k1=v1 k2=v2' type: string - default: 'GASNET_STATSFILE=- GASNET_MALLOC_INIT=1' + default: '' required: false From cad1089308b599c7807b09b4de064f83caa60b4c Mon Sep 17 00:00:00 2001 From: Dan Bonachea Date: Tue, 12 May 2026 20:39:02 -0700 Subject: [PATCH 4/4] CI: Increase /dev/shm size for containers The docker default of 64MiB is too small and was causing SIGBUS crashes in testvis and testcore2. Add /dev/shm size to Version Info step --- .github/workflows/build.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index ab03f752e..514ec1075 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -130,6 +130,7 @@ jobs: container: image: ${{ matrix.container }} + options: --shm-size=2gb env: COMPILER_VERSION: ${{ matrix.version }} @@ -232,6 +233,7 @@ jobs: uname -a if test -r /etc/os-release ; then grep -e NAME -e VERSION /etc/os-release ; fi if test -x /usr/bin/sw_vers ; then /usr/bin/sw_vers ; fi + if test -d /dev/shm ; then df -h /dev/shm ; fi echo echo == Tool version info == echo PATH="$PATH"