[LU-8905] tests: sanity-hsm test_3[3-6] does not use ps correctly Created: 05/Dec/16 Updated: 29/Mar/22 Resolved: 29/Mar/22 |
|
| Status: | Closed |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Minor |
| Reporter: | CEA | Assignee: | John Hammond |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||||||
| Severity: | 3 | ||||||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||||||
| Description |
|
Tests 33 34 35 36 of sanity-hsm check that processes exit correctly (/ die when killed) using the following command: local there=$(ps -o pid,comm hp $pid >/dev/null) [[ -z $there ]] || error "Process dos not exit" Obviously $there is always empty and the test is a noop. The output should not be redirected to /dev/null. test_33 fails when fixed, this is related to LU-5216. |
| Comments |
| Comment by John Hammond [ 05/Dec/16 ] |
|
Here and in several other test scripts we should be using 'kill -0 $pid' instead of various contraptions based on 'ps'. |
| Comment by Peter Jones [ 05/Dec/16 ] |
|
John is investigating |
| Comment by John Hammond [ 29/Mar/22 ] |
|
Fixed by |