[LU-12795] lp_utils is duplicated and not used. Created: 22/Sep/19  Updated: 15/Nov/19  Resolved: 04/Oct/19

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

Type: Bug Priority: Minor
Reporter: Shaun Tancheff Assignee: Shaun Tancheff
Resolution: Fixed Votes: 0
Labels: None

Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

lustre/tests/lp_utils.c is duplicated but not used.
lustre/tests/lp_utils.h is duplicated but not needed.

Unify the users of lp_utils.h and remove the duplicate copy.

In addition cleanup the sprintf(msg, ..); FAIL(msg); pattern into a single FAILF(); macro.

 

 

 Comments   
Comment by Gerrit Updater [ 22/Sep/19 ]

Shaun Tancheff (stancheff@cray.com) uploaded a new patch: https://review.whamcloud.com/36255
Subject: LU-12795 tests: Introduce FAILF and start using it
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 468687a23334d81e34bdb33847b91529a0d9cb18

Comment by Gerrit Updater [ 23/Sep/19 ]

Shaun Tancheff (stancheff@cray.com) uploaded a new patch: https://review.whamcloud.com/36276
Subject: LU-12795 tests: Prefer FAILF in mpi tests
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: abd3aad88da36f5cf94e70f09e19b8c679a53d10

Comment by Gerrit Updater [ 04/Oct/19 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36255/
Subject: LU-12795 tests: Introduce FAILF and start using it
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8c8f46566c41c2cfd2e3950c0ecb42157df8984a

Comment by Gerrit Updater [ 04/Oct/19 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36276/
Subject: LU-12795 tests: Prefer FAILF in mpi tests
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: cc697263da464f16fd11cb8550a2838bdac42fd1

Comment by Peter Jones [ 04/Oct/19 ]

Landed for 2.13

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