ncsa / puppet-profile_update_os

NCSA Common Puppet Profiles - configure functionality for upgrading OS packages
0 stars 0 forks source link

Add timestamp to reboot warnings #24

Closed billglick closed 4 months ago

billglick commented 1 year ago

Currently when reboot notices appear in the console we are suppressing the wall banner with -n so no timestamp of when the wall message appears for the user. So for example if a console session is left idle overnight, see the following warning doesn't tell the user much about when the upcoming reboot actually is:

[root@asd-pup01 ~]#                                                                                
This server (asd-pup01.internal.ncsa.edu) will be updated and rebooted in 24 hours.

[root@asd-pup01 ~]# 

I suggest we append date +%r or similar to wall messages about pending reboots. Or simply remove -n option from wall commands.

This needs to be addressed in the following files (and perhaps elsewhere):

billglick commented 1 year ago

I think we should do this option:

Remove -n option from wall commands

That is much more informative for the user and makes the message stand out a bit.