Skip to content

Commit d077966

Browse files
author
Artem Ryabov
committed
Reverted test to Open MPI 4.0.x behavior.
The test has been updated for open-mpi/ompi#7202 in scope of mellanox-hpc#92. PR 7202 is not ported to Open MPI 4.0.x. Signed-off-by: Artem Ryabov <artemry@mellanox.com>
1 parent d0a03e3 commit d077966

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

jenkins/ompi/ompi_test.sh

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -537,12 +537,12 @@ function test_tune()
537537
echo "--mca mca_base_env_list \"XXX_A=1;XXX_B=2;XXX_C;XXX_D;XXX_E\"" > $WORKSPACE/test_tune.conf
538538
echo "mca_base_env_list=XXX_A=7;XXX_B=8" > $WORKSPACE/test_amca.conf
539539
val=$($OMPI_HOME/bin/mpirun $mca --np 2 --tune $WORKSPACE/test_tune.conf --am $WORKSPACE/test_amca.conf $abs_path/env_mpi | sed -n -e 's/^XXX_.*=//p' | sed -e ':a;N;$!ba;s/\n/+/g' | bc)
540-
# precedence goes left-to-right.
541-
# A is first set to 1 in "tune", and then reset to 7 in "amca".
542-
# B is first set to 2 in "tune", but then reset to 8 in "amca"
540+
# precedence goes right-to-left (changed in master to correct left-to-right).
541+
# A is first set to 7 in "amca", and then reset to 1 in "tune".
542+
# B is first set to 8 in "amca", but then reset to 2 in "tune"
543543
# C, D, E are taken from the environment as 3,4,5
544-
# return (7+8+3+4+5)*2=54
545-
if [ $val -ne 54 ]; then
544+
# return (1+2+3+4+5)*2=30
545+
if [ $val -ne 30 ]; then
546546
exit 1
547547
fi
548548

0 commit comments

Comments
 (0)