#1598 Issue closed
: 430_prepare_timesync.sh needs to be improved¶
Labels: enhancement
, fixed / solved / done
gdha opened issue at 2017-11-27 11:52:¶
- rear version (/usr/sbin/rear -V): 2.2 and higher
- OS version (cat /etc/rear/os.conf or lsb_release -a): n/a
- Brief description of the issue:
- script
rescue/default/430_prepare_timesync.sh
needs an update with ntpd and/etc/inittab
- today it should become a systemd script - secondly, it might be useful to add NTPDATE as a third possibility to set-up time
- script
N3WWN commented at 2017-11-27 17:10:¶
Just a quick comment that adding chrony as another option to sync time may be a good idea since RHEL/CentOS 7.x use chrony by default.
I could help with this or take this on within the next few weeks, if you'd like.
gdha commented at 2017-11-27 17:21:¶
@N3WWN Hi Richard - indeed chronyd is used by RHEL and ubuntu and for my
rear-automated-testing I just added an ansible playbook for this fact
(https://github.com/gdha/rear-automated-testing/commit/ccfd1cf1a3a7690292738542d1cad22536c3f8a9)
However, that being said does not mean that our rescue image is chronyd
ready, therefore, your help would be very much appreciated.
gdha commented at 2017-11-29 16:43:¶
The NTPDATE feature should be implemented for rear-2.3 as then my
rear automated testing system can compare hours more easily. Chrony
is for 2.4 or later.
Therefore, I change the milestone to 2.3 for the ntpdate update.
N3WWN commented at 2017-11-29 16:59:¶
@gdha I can start work on the ntpdate feature today and get it to you ASAP. If I can't get it today, would next week be okay?
gdha commented at 2017-11-29 17:04:¶
@N3WWN Try to do it today, if not then I will check it tomorrow (only have a few hours before I leave for a trip). We will see - thanks for your kind offer!
N3WWN commented at 2017-11-29 17:15:¶
Will do and you're welcome! 😀
I will be away Thu-Mon which is why I say it may be next week, but will do what I can today.
jsmeix commented at 2017-11-30 10:52:¶
With
https://github.com/rear/rear/pull/1608
merged
the NTPDATE feature is implemented for ReaR 2.3
so that I close this issue as "fixed/solved/done".
For the further enhancement with Chrony for 2.4 or later
a separated new issue should be opened.
N3WWN commented at 2017-11-30 15:33:¶
Roger that @jsmeix ! I will open a new issue when I get back early next week.
[Export of Github issue for rear/rear.]