pmem / issues

Old issues repo for PMDK.
http://pmem.io
13 stars 7 forks source link

Tests: ex_librpmem_basic/TEST1, ex_librpmem_hello/TEST1, ex_librpmem_fibonacci/TEST[0-2] fail. #1062

Closed Greg091 closed 5 years ago

Greg091 commented 5 years ago

ISSUE:

Environment Information

Please provide a reproduction of the bug:

./RUNTESTS test_name -s test_number, example:

./RUNTESTS ex_librpmem_basic -s TEST1

How often bug is revealed: (always, often, rare): always

Actual behavior:

All the above tests fail in the same way. Below log of execution of ex_librpmem_basic TEST1.

./RUNTESTS ex_librpmem_basic -s TEST1
ex_librpmem_basic/TEST1: SETUP (short/non-pmem/debug/sockets/GPSPM)
../unittest/unittest.sh: line 2070: no match: pool_hdr.checksum_gen\(\)
RUNTESTS: stopping: ex_librpmem_basic/TEST1 failed, TEST=short FS=non-pmem BUILD=debug RPMEM_PROVIDER=sockets RPMEM_PM=GPSPM
ex_librpmem_basic/TEST1: SETUP (short/non-pmem/debug/sockets/APM)
../unittest/unittest.sh: line 2070: no match: pool_hdr.checksum_gen\(\)
RUNTESTS: stopping: ex_librpmem_basic/TEST1 failed, TEST=short FS=non-pmem BUILD=debug RPMEM_PROVIDER=sockets RPMEM_PM=APM
./RUNTESTS: line 142: /tmp/data-location.RlLrpOye: No such file or directory

Expected behavior:

Tests should pass.

Details

Additional information about Priority and Help Requested:

Are you willing to submit a pull request with a proposed change? (Yes, No)

Requested priority: (Showstopper, High, Medium, Low)

Greg091 commented 5 years ago

Verified on: 1.6.1-253-ga6ad15e55. Still fails.

janekmi commented 5 years ago

The issue is caused by special characters passed multiple times as a function argument, through SSH and to the shell on the remote node. I see two possible solutions:

janekmi commented 5 years ago

Please note the issue appears also in other tests:

$ ./RUNTESTS pmempool_sync_remote -s TEST20
$ ./RUNTESTS pmempool_sync_remote -s TEST21
Greg091 commented 5 years ago

Verified on: 1.6.1-338-g392789288