[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/
Subject: LU-8993 utils: Use absolute pathname for debug_daemon log file
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 23f80c74c38dfd726ac1b50176989b3aa14e3a81

Comment by Peter Jones [ 28/Aug/17 ]

Landed for 2.11

Generated at Sat Feb 10 02:22:19 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.