Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #7598: Adding new features and timezones #824

Conversation

matya
Copy link
Contributor

@matya matya commented Dec 17, 2015

  • NTP Servers
    • can hold multiple server in one line, separated by commas
    • final list is stripped of duplicates
  • Timezones
    • Adding all standard timezones
    • Allowing textbox (useful for variable usage)
  • RTC Clock
    • Allow to be set to UTC or Localtime or not change
    • Is only set on systems actually having an RTC clock
  • Xen Time
    • New feature: Allow setting of Xen Hypervisor time via Dom0

- NTP Servers
    * can hold multiple server in one line, separated by commas
    * final list is stripped of duplicates

- Timezones
    * Adding all standard timezones
    * Allowing textbox (useful for variable usage)

- RTC Clock
    * Allow to be set to UTC or Localtime or not change
    * Is only set on systems actually having an RTC clock

- Xen Time
    * New feature: Allow setting of Xen Hypervisor time via Dom0
@ncharles
Copy link
Member

ha, commit is too big for me to comment on it
So, the bundle list in metadata is not correct ( https://github.com/matya/rudder-techniques/blob/matya-7598-add-features-to-timesettings/techniques/systemSettings/misc/clockConfiguration/4.0/metadata.xml#L33 ). You should have two entries NAME, one for each bundle

{

vars:
"ntpServers_all" slist => {&CLOCK_NTPSERVERS: { splitstring("&it&", " *, *", 100)};separator=", "&};
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i am amazed that it does work !

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just some cascaded splitting logic

@ncharles
Copy link
Member

this is one huge PR ! thank you very much for it, I've made some remark about the some minor points, that should be adressed
I have a question also: what happens if we set CLOCK_HWSYNC_RTC and a timezone on windows ? Doesn't it conflict (it would run tzutil even if not necessary, isn't it ?)

@matya
Copy link
Contributor Author

matya commented Jan 27, 2016

@ncharles Some short explanation what's going on...
RTC time and the Timezone of Windows are two different settings, however, they influence each other.

  1. RTC is the Real-Time Clock, that ticks on every motherboard, and sets the date/time on boot, before NTP can kick in. Useful if you have Databases starting before NTP, or for your log files not having a timejump from t=0 to t=...
  2. UTC is the "Coordinated Universal Time", which has no Timezone, no Leapsecond, and should actually be used to feed the RTC...
    ... however, by default Windows stores Local time in the RTC, and this depends on the timezone set. When you enable UTC for RTC and set a Timezone, windows should set the RTC always to UTC and the displayed time would depend on the Timezone.

I have currently no way of testing this additional code on a windows-client with NOVA, so the code currently is only "theoretical". After some google-work I found, that this registry-setting seem fail to work starting from Win8+: Hardware Clock in UTC

This was introduced because of the well-known PEBKAC-Problem, which hit the american people with their time difference even more in the early days, and now they are just keeping it for "historical reasons": Why does Windows keep your BIOS clock on local time?

I could throw it out, then the feature-set would stay as version 3.0, so no worsening on that, or this would need testing and would work with Win7 if I bet correctly... The call is yours which way to go ;-)

@matya
Copy link
Contributor Author

matya commented Jan 31, 2016

@ncharles After our discussion on IRC I'll close this to split up the portions in different tickets and PR's

@matya matya closed this Jan 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants