[LU-10255] sanity 129 wrong comparison operator used Created: 17/Nov/17 Updated: 19/Dec/17 Resolved: 29/Nov/17 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.10.1, Lustre 2.11.0 |
| Fix Version/s: | Lustre 2.11.0, Lustre 2.10.3 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Nikolay Borisov | Assignee: | James Nunez (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Severity: | 3 |
| Rank (Obsolete): | 9223372036854775807 |
| 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 |
| Comment by Gerrit Updater [ 17/Nov/17 ] |
|
Nikolay Borisov (nborisov@suse.com) uploaded a new patch: https://review.whamcloud.com/30149 |
| Comment by Gerrit Updater [ 17/Nov/17 ] |
|
Nikolay Borisov (nborisov@suse.com) uploaded a new patch: https://review.whamcloud.com/30151 |
| Comment by Gerrit Updater [ 29/Nov/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30149/ |
| Comment by Peter Jones [ 29/Nov/17 ] |
|
Landed for 2.11 |
| Comment by Gerrit Updater [ 19/Dec/17 ] |
|
John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/30151/ |