[LU-8609] connect client health_check file to client import state

• Type: Improvement
• Resolution: Unresolved
• Priority: Minor
• Assignee: Emoly Liu
• Reporter: Andreas Dilger

Description

The /proc/fs/lustre/health_check file on the client does not currently report anything besides health=healthy, since this is only wired up to report request handling state on the server nodes, and if the ldiskfs filesystem has been mounted read-only.

It would be desirable to have health_check print out the OSC and MDC import states on the client (e.g. if imp->imp_state != LUSTRE_IMP_FULL as long as the import is not marked imp_deactive).

Issue Links

• is related to: LU-8066 Move lustre procfs handling to sysfs and debugfs.
• is related to: LU-10756 Send Uevents for interesting Lustre changes

Comments

[James A Simmons]

Link the work of LU-10756 since a patch exist that send uevents about the client imports state. Admins can setup udev rules to log or handle specific states. That is a more dynamic approach. We can supply more in depth information related to this ticket.

[Andreas Dilger]

Hmm, looking at this more closely, it seems that the health_check file never prints anything but "health=healthy" even on the server under normal circumstances, and I guess this shouldn't change. If there is a problem it will print "LBUG" or "NOT HEALTHY". What is more important is that the client report "health=NOT HEALTHY" if the import states are bad, not that individual connection states are printed.

[Andreas Dilger]

James, why not just move the whole file to debugfs? There are already "lfs osts}" and "{{lfs mdts" commands to print this state for users, the "health_check" file is mostly for HA monitoring scripts.

[James A Simmons]

This over laps with something Andreas suggested. In that we create two files for the health_checker. One to report the health and the second in debugfs that has more in depth details why it failed.

[Emoly Liu]

adilger,
What kind of OSC and MDC import states on the client should be printed by health_check? Is the following format OK?

health_check=healthy
target state:
  lustre-OST0000_UUID: FULL
  lustre-OST0001_UUID: FULL
  ...
  MGS: FULL
  ...