From eb381df7638e275c924b47d2e8b1d254854ad831 Mon Sep 17 00:00:00 2001 From: Jim Garlick Date: Sat, 4 Oct 2025 08:17:44 -0700 Subject: [PATCH] testsuite: add elcap node hwloc data Problem: LLNL's El Capitan system employs a node type that may be interesting to include in future tests of Flux's resource representation, particularly as it evolves beyond Rv1. Add a representative hwloc XML dump from - HPE/Cray EX255A login node on tuolomne - hwloc-2.12.1-4.t4 - rhwloc_local_topology_xml() from flux-core-0.77.0 Add a simple Rv1 encode test. --- t/Makefile.am | 1 + t/hwloc-data/elcap.xml | 1548 ++++++++++++++++++++++++++++++++++++++++ t/t0026-flux-R.t | 7 + 3 files changed, 1556 insertions(+) create mode 100644 t/hwloc-data/elcap.xml diff --git a/t/Makefile.am b/t/Makefile.am index b3ecedc7981e..6b8730b02520 100644 --- a/t/Makefile.am +++ b/t/Makefile.am @@ -542,6 +542,7 @@ check_LTLIBRARIES = \ dist_check_DATA = \ hwloc-data/sierra.xml \ hwloc-data/corona.xml \ + hwloc-data/elcap.xml \ valgrind/valgrind.supp test_ldadd = \ diff --git a/t/hwloc-data/elcap.xml b/t/hwloc-data/elcap.xml new file mode 100644 index 000000000000..fe7403ebc2c3 --- /dev/null +++ b/t/hwloc-data/elcap.xml @@ -0,0 +1,1548 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + 0 1 2 3 + 10 32 32 32 32 10 32 32 32 32 + 10 32 32 32 32 10 + + + 727 728 729 730 + 1000000 100000 100000 100000 100000 1000000 100000 100000 100000 100000 + 1000000 100000 100000 100000 100000 1000000 + + + 727 728 729 730 + 0 1 1 1 1 0 1 1 1 1 + 0 1 1 1 1 0 + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/t/t0026-flux-R.t b/t/t0026-flux-R.t index 02515baa1091..6686c93d1092 100755 --- a/t/t0026-flux-R.t +++ b/t/t0026-flux-R.t @@ -82,6 +82,13 @@ test_expect_success 'flux R encode --xml works with AMD RSMI gpus' ' test_debug "echo encode XML = $result" && test "$result" = "rank0/core[0-47],gpu[0-7]" ' +test_expect_success 'flux R encode --xml works with El Capitan node' ' + flux R encode --xml=$SHARNESS_TEST_SRCDIR/hwloc-data/elcap.xml \ + > R.elcap && + result=$(flux R decode --short < R.elcap) && + test_debug "echo encode XML = $result" && + test "$result" = "rank0/core[0-95],gpu[0-3]" +' test_expect_success 'flux R decode --include works' ' result=$(flux R encode -r 0-1023 | flux R decode --include 5-7 --short) && test_debug "echo $result" &&