From 22bcbacacfc70e541dc9bf1dcb2c5b14b84c2e35 Mon Sep 17 00:00:00 2001 From: janekmi <jan.m.michalski@intel.com> Date: Mon, 5 Aug 2019 12:13:39 -0700 Subject: [PATCH] test: remove invalid expect_normal_exit expect_normal_exit is incompatible with copy_file_to_node and is causing trouble when used with valgrind. Ref: pmem/issues#1092 --- src/test/rpmem_basic/TEST15 | 4 ++-- src/test/rpmem_basic/TEST18 | 4 ++-- src/test/rpmem_basic/TEST5 | 4 ++-- src/test/rpmem_basic/TEST6 | 4 ++-- src/test/rpmem_basic/TEST7 | 4 ++-- 5 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/test/rpmem_basic/TEST15 b/src/test/rpmem_basic/TEST15 index 6ec938905..991a388ef 100755 --- a/src/test/rpmem_basic/TEST15 +++ b/src/test/rpmem_basic/TEST15 @@ -1,6 +1,6 @@ #!/usr/bin/env bash # -# Copyright 2016-2018, Intel Corporation +# Copyright 2016-2019, Intel Corporation # # Redistribution and use in source and binary forms, with or without # modification, are permitted provided that the following conditions @@ -62,7 +62,7 @@ create_poolset $DIR/pool2.set AUTO:$(get_node_devdax_path 0 0) O NOHDRS expect_normal_exit run_on_node 0 rm -rf $PART_DIR expect_normal_exit run_on_node 0 mkdir -p ${RPMEM_POOLSET_DIR[0]} expect_normal_exit run_on_node 0 mkdir -p $PART_DIR -expect_normal_exit copy_files_to_node 0 ${RPMEM_POOLSET_DIR[0]} $DIR/{pool0.set,pool1.set,pool2.set} +copy_files_to_node 0 ${RPMEM_POOLSET_DIR[0]} $DIR/{pool0.set,pool1.set,pool2.set} expect_normal_exit run_on_node 0 ../pmempool rm -sf ${RPMEM_POOLSET_DIR[0]}/{pool0.set,pool1.set,pool2.set} expect_normal_exit run_on_node 1 ../pmempool rm -sf $(get_node_devdax_path 0 0) diff --git a/src/test/rpmem_basic/TEST18 b/src/test/rpmem_basic/TEST18 index 1d6ca99e7..05322cd0e 100755 --- a/src/test/rpmem_basic/TEST18 +++ b/src/test/rpmem_basic/TEST18 @@ -1,6 +1,6 @@ #!/usr/bin/env bash # -# Copyright 2016-2018, Intel Corporation +# Copyright 2016-2019, Intel Corporation # # Redistribution and use in source and binary forms, with or without # modification, are permitted provided that the following conditions @@ -63,7 +63,7 @@ create_poolset $DIR/pool2.set AUTO:$(get_node_devdax_path 0 0) O NOHDRS expect_normal_exit run_on_node 0 rm -rf $PART_DIR expect_normal_exit run_on_node 0 mkdir -p ${RPMEM_POOLSET_DIR[0]} expect_normal_exit run_on_node 0 mkdir -p $PART_DIR -expect_normal_exit copy_files_to_node 0 ${RPMEM_POOLSET_DIR[0]} $DIR/{pool0.set,pool1.set,pool2.set} +copy_files_to_node 0 ${RPMEM_POOLSET_DIR[0]} $DIR/{pool0.set,pool1.set,pool2.set} expect_normal_exit run_on_node 0 ../pmempool rm -sf ${RPMEM_POOLSET_DIR[0]}/{pool0.set,pool1.set,pool2.set} expect_normal_exit run_on_node 1 ../pmempool rm -sf $(get_node_devdax_path 0 0) diff --git a/src/test/rpmem_basic/TEST5 b/src/test/rpmem_basic/TEST5 index d99a897fb..371b60d2f 100755 --- a/src/test/rpmem_basic/TEST5 +++ b/src/test/rpmem_basic/TEST5 @@ -1,6 +1,6 @@ #!/usr/bin/env bash # -# Copyright 2016-2018, Intel Corporation +# Copyright 2016-2019, Intel Corporation # # Redistribution and use in source and binary forms, with or without # modification, are permitted provided that the following conditions @@ -59,7 +59,7 @@ create_poolset $DIR/pool2.set 8M:$PART_DIR/pool2.part0 8M:$PART_DIR/pool2.part1 expect_normal_exit run_on_node 0 rm -rf $PART_DIR expect_normal_exit run_on_node 0 mkdir -p ${RPMEM_POOLSET_DIR[0]} expect_normal_exit run_on_node 0 mkdir -p $PART_DIR -expect_normal_exit copy_files_to_node 0 ${RPMEM_POOLSET_DIR[0]} $DIR/{pool0.set,pool1.set,pool2.set} +copy_files_to_node 0 ${RPMEM_POOLSET_DIR[0]} $DIR/{pool0.set,pool1.set,pool2.set} expect_abnormal_exit run_on_node 1 ssh ${NODE_ADDR[0]} "\$RPMEM_CMD --remove pool0.set" expect_abnormal_exit run_on_node 1 ssh ${NODE_ADDR[0]} "\$RPMEM_CMD --remove pool1.set" diff --git a/src/test/rpmem_basic/TEST6 b/src/test/rpmem_basic/TEST6 index a6be71a5d..f49f426b3 100755 --- a/src/test/rpmem_basic/TEST6 +++ b/src/test/rpmem_basic/TEST6 @@ -1,6 +1,6 @@ #!/usr/bin/env bash # -# Copyright 2016-2018, Intel Corporation +# Copyright 2016-2019, Intel Corporation # # Redistribution and use in source and binary forms, with or without # modification, are permitted provided that the following conditions @@ -59,7 +59,7 @@ create_poolset $DIR/pool2.set 8M:$PART_DIR/pool2.part0 8M:$PART_DIR/pool2.part1 expect_normal_exit run_on_node 0 rm -rf $PART_DIR expect_normal_exit run_on_node 0 mkdir -p ${RPMEM_POOLSET_DIR[0]} expect_normal_exit run_on_node 0 mkdir -p $PART_DIR -expect_normal_exit copy_files_to_node 0 ${RPMEM_POOLSET_DIR[0]} $DIR/{pool0.set,pool1.set,pool2.set} +copy_files_to_node 0 ${RPMEM_POOLSET_DIR[0]} $DIR/{pool0.set,pool1.set,pool2.set} expect_abnormal_exit run_on_node 1 ssh ${NODE_ADDR[0]} "\$RPMEM_CMD --remove pool0.set" expect_abnormal_exit run_on_node 1 ssh ${NODE_ADDR[0]} "\$RPMEM_CMD --remove pool1.set" diff --git a/src/test/rpmem_basic/TEST7 b/src/test/rpmem_basic/TEST7 index b2cf9e3e9..4b9c6117c 100755 --- a/src/test/rpmem_basic/TEST7 +++ b/src/test/rpmem_basic/TEST7 @@ -1,6 +1,6 @@ #!/usr/bin/env bash # -# Copyright 2016-2018, Intel Corporation +# Copyright 2016-2019, Intel Corporation # # Redistribution and use in source and binary forms, with or without # modification, are permitted provided that the following conditions @@ -63,7 +63,7 @@ create_poolset $DIR/pool2.set AUTO:$(get_node_devdax_path 0 0) O NOHDRS expect_normal_exit run_on_node 0 rm -rf $PART_DIR expect_normal_exit run_on_node 0 mkdir -p ${RPMEM_POOLSET_DIR[0]} expect_normal_exit run_on_node 0 mkdir -p $PART_DIR -expect_normal_exit copy_files_to_node 0 ${RPMEM_POOLSET_DIR[0]} $DIR/{pool0.set,pool1.set,pool2.set} +copy_files_to_node 0 ${RPMEM_POOLSET_DIR[0]} $DIR/{pool0.set,pool1.set,pool2.set} expect_normal_exit run_on_node 0 ../pmempool rm -sf ${RPMEM_POOLSET_DIR[0]}/{pool0.set,pool1.set,pool2.set} expect_normal_exit run_on_node 1 ../pmempool rm -sf $(get_node_devdax_path 0 0) -- GitLab