[LU-8993] lctl debug_daemon should warn on relative pathnames Created: 09/Jan/17 Updated: 28/Aug/17 Resolved: 28/Aug/17 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.11.0 |
| Type: | Improvement | Priority: | Major |
| Reporter: | Oleg Drokin | Assignee: | Steve Guminski (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
lctl debug_daemon userspace part would happily accept relative pathname, but in fact the kernel part of the code wold reject anything that does not start with a / as EINVAL which produces a cryptic error message to the user like start debug_daemon on debug.log failed: Invalid argument We should either require absolute pathname in the tool or at least print a warning that the relative pathnames are unsupported and the command will likely fail (leading to the actual failure that would be a lot less cryptic). |
| Comments |
| Comment by Peter Jones [ 10/Jan/17 ] |
|
Steve Could you look into this? Thanks Peter |
| Comment by Steve Guminski (Inactive) [ 10/Jan/17 ] |
|
I'm already looking into lfs and lctl usability for other tickets, so this fits well with my current work. |
| Comment by Gerrit Updater [ 28/Aug/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/25485/ |
| Comment by Peter Jones [ 28/Aug/17 ] |
|
Landed for 2.11 |