[LU-7204] replay-single test_85b: syntax error : unary operator expected Created: 24/Sep/15  Updated: 01/Oct/16  Resolved: 22/Jun/16

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: Lustre 2.8.0
Fix Version/s: Lustre 2.9.0

Type: Bug Priority: Major
Reporter: Sundar Keshava (Inactive) Assignee: Jian Yu
Resolution: Fixed Votes: 0
Labels: None

Attachments: File replay-single_test_85b.tar    
Issue Links:
Duplicate
is duplicated by LU-7342 replay-single test_85a: @@@@@@ FAIL: ... Resolved
Epic/Theme: test
Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

This issue can be reproduce on single node setup.
With "-" in FSNAME=lustre-2 in cfg/local.sh
Server, Client: 2.7.58
[keshava@MRPbld-c2 lustre-2782]$ head ./lustre/ChangeLog
TBD Intel Corporation

  • version 2.8.0
[root@MRPbld-c2 lustre-2782]# ./lustre/tests/replay-single.sh 
Logging to shared log directory: /tmp/test_logs/1443075535
excepting tests: 61d 53a 53d
skipping tests SLOW=no: 1 2 3 4 6 12 16 44a 44b 65 66 67 68
MRPbld-c2: Checking config lustre mounted on /mnt/lustre-2
Checking servers environments
Checking clients MRPbld-c2 environments
Using TIMEOUT=20
disable quota as required
osd-ldiskfs.track_declares_assert=1
Cleaning up AT ...


== replay-single test 85b: check the cancellation of unused locks during recovery(EXTENT) == 11:48:56 (1443075536)
Pool lustre-2.test_85b created
OST lustre-2-OST0000_UUID added to pool lustre-2.test_85b
error: setstripe: pool name 'lustre-2.test_85b' is too long (max is 15 characters)
Create a new file with a specific striping pattern or
set the default striping pattern on an existing directory or
delete the default striping pattern from an existing directory
usage: setstripe -d <directory>   (to delete default striping)
 or
usage: setstripe [--stripe-count|-c <stripe_count>]
                 [--stripe-index|-i <start_ost_idx>]
                 [--stripe-size|-S <stripe_size>]
                 [--pool|-p <pool_name>]
                 [--ost-list|-o <ost_indices>]
                 <directory|filename>
	stripe_size:  Number of bytes on each OST (0 filesystem default)
	              Can be specified with k, m or g (in KB, MB and GB
	              respectively)
	start_ost_idx: OST index of first stripe (-1 default)
	stripe_count: Number of OSTs to stripe over (0 default, -1 all)
	pool_name:    Name of OST pool to use (default none)
	ost_indices:  List of OST indices, can be repeated multiple times
	              Indices be specified in a format of:
	                -o <ost_1>,<ost_i>-<ost_j>,<ost_n>
	              Or:
	                -o <ost_1> -o <ost_i>-<ost_j> -o <ost_n>
	              If --pool is set with --ost-list, then the OSTs
	              must be the members of the pool.
error: get_param: ldlm/namespaces/*OST0000*clilov/lock_unused_count: Found no match
before recovery: unused locks count = 
./lustre/tests/replay-single.sh: line 2716: [: !=: unary operator expected
 replay-single test_85b: @@@@@@ FAIL: unused locks () should be zero 
  Trace dump:
  = /home/keshava/intel/code/lustre-2782/lustre/tests/test-framework.sh:4738:error_noexit()
  = /home/keshava/intel/code/lustre-2782/lustre/tests/test-framework.sh:4769:error()
  = ./lustre/tests/replay-single.sh:2716:test_85b()
  = /home/keshava/intel/code/lustre-2782/lustre/tests/test-framework.sh:5016:run_one()
  = /home/keshava/intel/code/lustre-2782/lustre/tests/test-framework.sh:5053:run_one_logged()
  = /home/keshava/intel/code/lustre-2782/lustre/tests/test-framework.sh:4870:run_test()
  = ./lustre/tests/replay-single.sh:2733:main()
Dumping lctl log to /tmp/test_logs/1443075535/replay-single.test_85b.*.1443075556.log
Dumping logs only on local client.
FAIL 85b (20s)


 Comments   
Comment by Sundar Keshava (Inactive) [ 24/Sep/15 ]

Hi,

We are working on a patch, which is under "Awaiting Review" state, once it is finalized,
we will land the patch to upstream here.

Regards,
Keshava

Comment by Andreas Dilger [ 25/Sep/15 ]

As the error message states, this is because the pool name is too long, not because there is a dash '-' in the name.

Comment by Sundar Keshava (Inactive) [ 28/Sep/15 ]

Hi,

Even with '-' in FSNAME=lust-e(with 6 chars) also, fails with same error, where poolname is within limit.
There is one more error before "unary operator expected", i.e. "Found no match", because of '-' in name.

[root@MRPbld-c2 lustre-2782]# ./lustre/tests/replay-single.sh 
Logging to shared log directory: /tmp/test_logs/1443431081
excepting tests: 61d 53a 53d
skipping tests SLOW=no: 1 2 3 4 6 12 16 44a 44b 65 66 67 68
MRPbld-c2: Checking config lustre mounted on /mnt/lust-e
Checking servers environments
Checking clients MRPbld-c2 environments
Using TIMEOUT=20
disable quota as required
osd-ldiskfs.track_declares_assert=1
Cleaning up AT ...


== replay-single test 85b: check the cancellation of unused locks during recovery(EXTENT) == 14:34:42 (1443431082)
Pool lust-e.test_85b created
OST lust-e-OST0000_UUID added to pool lust-e.test_85b
error: get_param: ldlm/namespaces/*OST0000*clilov/lock_unused_count: Found no match
before recovery: unused locks count = 
./lustre/tests/replay-single.sh: line 2716: [: !=: unary operator expected
 replay-single test_85b: @@@@@@ FAIL: unused locks () should be zero 
  Trace dump:
  = /home/keshava/intel/code/lustre-2782/lustre/tests/test-framework.sh:4738:error_noexit()
  = /home/keshava/intel/code/lustre-2782/lustre/tests/test-framework.sh:4769:error()
  = ./lustre/tests/replay-single.sh:2716:test_85b()
  = /home/keshava/intel/code/lustre-2782/lustre/tests/test-framework.sh:5016:run_one()
  = /home/keshava/intel/code/lustre-2782/lustre/tests/test-framework.sh:5053:run_one_logged()
  = /home/keshava/intel/code/lustre-2782/lustre/tests/test-framework.sh:4870:run_test()
  = ./lustre/tests/replay-single.sh:2733:main()
Dumping lctl log to /tmp/test_logs/1443431081/replay-single.test_85b.*.1443431100.log
Dumping logs only on local client.
FAIL 85b (18s)
[root@MRPbld-c2 lustre-2782]#

Regards,
Keshava

Comment by Gerrit Updater [ 21/Oct/15 ]

HemaHarish (hema.yarramilli@seagate.com) uploaded a new patch: http://review.whamcloud.com/16902
Subject: LU-7204 test: handle FSNAME with dashes in replay-single 85b
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: cd42ea87d99fbfb17162d40620a77fc1ba85ea17

Comment by hemaharish [ 26/Oct/15 ]

Hi,

I didn't find test case failed related to 85b in reply-single.sh as mentioned in the Patch Set 1
Able to see test results upto test_70b only.

" Tests received by maloo, run on CentOS release 6.7/x86_64: (https://testing.hpdd.intel.com/test_sessions/1f9a1960-79c1-11e5-9fb3-5254006e85c2). Ran 15 tests. 1 tests failed: replay-single."

Comment by Saurabh Tandan (Inactive) [ 24/Feb/16 ]

Another instance found on b2_8 for failover testing , build# 6.
https://testing.hpdd.intel.com/test_sessions/54ec62da-d99d-11e5-9ebe-5254006e85c2
https://testing.hpdd.intel.com/test_sessions/c5a8e44c-d9c7-11e5-85dd-5254006e85c2

Comment by Gerrit Updater [ 22/Jun/16 ]

Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/16902/
Subject: LU-7204 test: handle FSNAME with dashes in replay-single 85b
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a186ede286518472a127caa46b048bcbbd203284

Comment by Jian Yu [ 22/Jun/16 ]

Patch landed to master branch for Lustre 2.9.0.

Generated at Sat Feb 10 02:06:53 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.