[LU-8649] Print console message in recovery when waiting for first client Created: 29/Sep/16 Updated: 19/Mar/18 Resolved: 09/Jan/18 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.11.0, Lustre 2.10.4 |
| Type: | Improvement | Priority: | Major |
| Reporter: | Andreas Dilger | Assignee: | Emoly Liu |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||
| Rank (Obsolete): | 9223372036854775807 | ||||
| Description |
|
When the server is in recovery but waiting for the first client to connect, it would be useful to print a message on the console every 10 minutes or so, in case the admins are waiting for recovery to complete. This has been reported a few times in the past, and it isn't obvious that recovery will not start until the first client connects. It would also be useful to print something similar in the recovery_status file in /proc. |
| Comments |
| Comment by Andreas Dilger [ 17/Feb/17 ] |
|
It would also be useful to improve the recovery_status file by adding WAITING_FOR_CLIENTS when recovery hasn't started yet would make it more clear to the administrator that recovery won't start until the clients connect. The difficulty is that I don't think we can replace RECOVERING completely, since userspace scripts may be checking this. |
| Comment by Andreas Dilger [ 23/Feb/17 ] |
|
jgarlough wrote:
Adding something like "NOT_NEEDED", or "UNNECESSARY", or "CLEAN_STARTUP" would be make this more clear. |
| Comment by Andreas Dilger [ 14/May/17 ] |
|
Another user hit this problem (from lustre-discuss):
|
| Comment by Peter Jones [ 14/Dec/17 ] |
|
Emoly Could you please look into this one? Thanks Peter |
| Comment by Gerrit Updater [ 25/Dec/17 ] |
|
Emoly Liu (emoly.liu@intel.com) uploaded a new patch: https://review.whamcloud.com/30656 |
| Comment by Gerrit Updater [ 09/Jan/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30656/ |
| Comment by Minh Diep [ 09/Jan/18 ] |
|
Landed for 2.11 |
| Comment by Gerrit Updater [ 09/Jan/18 ] |
|
Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/30811 |
| Comment by Gerrit Updater [ 19/Mar/18 ] |
|
John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/30811/ |