[LU-9104] Unknown config param in llog fails mounting target Created: 10/Feb/17 Updated: 29/Nov/18 Resolved: 29/Nov/18 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.10.1, Lustre 2.11.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Rahul Deshmukh (Inactive) | Assignee: | James A Simmons |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||||||
| Severity: | 3 | ||||||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||||||
| Description |
|
If we have unknown config parameter in llog then it fails the target mounting. Steps to re-create it are as : $>sh llmount.sh $>export NOFORMAT=yes $>for OST in $(lctl get_param mgs.MGS.live.lustre | grep OST); do echo "max_rpcs_in_flight=50 on $OST" ; lctl conf_param $OST.ost.max_rpcs_in_flight=50 ; done <--- Note here ost.max.. is mention instead of osc.max.. $>sh llmountcleanup.sh $>sh llmount.sh <------- this hangs, as ost is not avilable |
| Comments |
| Comment by Gerrit Updater [ 10/Feb/17 ] |
|
Rahul Deshmukh (rahul.deshmukh@seagate.com) uploaded a new patch: https://review.whamcloud.com/25368 |
| Comment by Rahul Deshmukh (Inactive) [ 15/Feb/17 ] |
|
Pushed the patch and re-producer, please review. |
| Comment by Peter Jones [ 29/May/17 ] |
|
James Can you please organize reviews of this proposed test? Thanks Peter |
| Comment by Gerrit Updater [ 19/Jul/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/25368/ |
| Comment by Peter Jones [ 19/Jul/17 ] |
|
Landed for 2.11 |
| Comment by Gerrit Updater [ 26/Jul/17 ] |
|
Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/28232 |
| Comment by Gerrit Updater [ 14/Sep/17 ] |
|
John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/28232/ |
| Comment by Taizeng Wu [ 29/Nov/18 ] |
|
I am using lustre-2.10.5, when i mount mgs&mdt, it report unkown config parameter, how i remove wrong parameter.
``` [Thu Nov 29 15:00:14 2018] LDISKFS-fs (sdb): mounted filesystem with ordered data mode. Opts: errors=remount-ro [Thu Nov 29 15:00:19 2018] LustreError: 15c-8: MGC10.10.1.14@o2ib: The configuration from log 'public4-MDT0000' failed (-38). This may be the result of communication errors between this node and the MGS, a bad configuration, or other errors. See the syslog for more information. ``` |
| Comment by Taizeng Wu [ 29/Nov/18 ] |
|
I am trying to use `mount -t lustre -o nosvc xxx` which only start mgs, then execute ` lctl conf_param -d public4-MDT0000.mdt.job_cleanaup_interval` to remove wrong config. |
| Comment by Li Xi [ 29/Nov/18 ] |
|
The patch of https://review.whamcloud.com/25368/ might still has a bug in osd_process_config() When a llog config of type MDT can not be understood by class_process_proc_param(PARAM_MDT) or any other prefix (e.g. PARAM_HSM), it is passed to osd_process_config() and class_process_proc_param() is called for it for twice. The problem is why the -ENOSYS failure of class_process_proc_param(PARAM_OST) is not ignored in osd_process_config()? I am not sure about the llog processing codes, but I guess the process is: class_process_proc_param(PARAM_MDT) -> class_process_proc_param(PARAM_HSM) -> class_process_proc_param(PARAM_HSM) -> class_process_proc_param(PARAM_OSD) -> class_process_proc_param(PARAM_OST). So if osd_process_config() doesn't understand the llog record, it should ignore the -ENOSYS. Right? |
| Comment by Li Xi [ 29/Nov/18 ] |
|
I think we should fix this problem by ignoring ENOSYS in osd_process_config() so reopening the ticket. |
| Comment by Peter Jones [ 29/Nov/18 ] |
|
IMHO, given that this bug has existed in multiple shipped releases, it would be better to track this issue in a new ticket so that we can more easily track getting it into future releases. |
| Comment by James A Simmons [ 29/Nov/18 ] |
|
I can fix this under LU-8066 since we will be moving to sysfs which handles these error codes differently. |
| Comment by Peter Jones [ 29/Nov/18 ] |
|
great - thanks James |