[LU-1345] Handle none integer values in sanity 215 test Created: 26/Apr/12 Updated: 25/Apr/14 Resolved: 14/May/12 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.3.0, Lustre 2.1.2 |
| Type: | Bug | Priority: | Minor |
| Reporter: | James A Simmons | Assignee: | Jian Yu |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | sanity, tests | ||
| Environment: |
Lustre 2.X test suite |
||
| Severity: | 3 |
| Rank (Obsolete): | 4626 |
| Description |
|
Sanity test 215 tests the values that come out of various /proc/sys/lnet/* files. The test validates the format that is expected. If your test setup has improper routes you can get a NA value instead of a integer which is not handled by the test. This is my case for /proc/sys/lnet/routers cat /proc/sys/lnet/routers I have a simple patch to fix this. |
| Comments |
| Comment by Ian Colle (Inactive) [ 26/Apr/12 ] |
|
James - can you please post a link to your patch here in the ticket? |
| Comment by James A Simmons [ 26/Apr/12 ] |
|
Just pushed the patches. http://review.whamcloud.com/#change,2603 for b2_1 http://review.whamcloud.com/#change,2602 for master Doesn't affect b1_8 |
| Comment by Peter Jones [ 27/Apr/12 ] |
|
Yu Jian Could you please review this suggested change Thanks Peter |
| Comment by James A Simmons [ 14/May/12 ] |
|
All patches have been landed. Peter you can close this ticket. |
| Comment by Peter Jones [ 14/May/12 ] |
|
Thanks Jsmes! Landed for 2.1.2 and 2.3 |