[LU-10255] sanity 129 wrong comparison operator used

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.3
• Affects Version/s: Lustre 2.10.1, Lustre 2.11.0
• Severity: 3
• Assignee: James Nunez (Inactive)
• Reporter: Nikolay Borisov (Inactive)

Description

When sanity 129 is run the following error from bash is printed (this doesn't cause direct test failure):

./sanity.sh: line 9244: [: true: integer expression expected

This is due to has_warning being set to either true/false but those values aren't being evaluated as integers but as strings. So the correct comparison is actually:

 [ $has_warnings = "false" ] 

Comments

[Gerrit Updater]

John L. Hammond ([e-mail]) merged in patch https://review.whamcloud.com/30151/
Subject: LU-10255 tests: Fix wrong comparison operator in sanity 129
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: 3dc028ea4916b1f2ca1f829c7338a5ff355f44d5

[Peter Jones]

Landed for 2.11

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/30149/
Subject: LU-10255 tests: Fix wrong comparison operator in sanity 129
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 35531ce8bd1bfd79808723ee843f5ca8e10e105d

[Gerrit Updater]

Nikolay Borisov ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30151
Subject: LU-10255 tests: Fix wrong comparison operator in sanity 129
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: 010373cb5caf6bd454d666afc8ea9bd3e0d01e70

[Gerrit Updater]

Nikolay Borisov ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30149
Subject: LU-10255 tests: Fix wrong comparison operator in sanity 129
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 2ce68c96844d8d91878322b99db6364f16ef51a2