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

Could you update the server list? Updating it by yourself without accepting this is OK. #1

Merged
merged 1 commit into from Jul 29, 2013

Conversation

ftake
Copy link
Contributor

@ftake ftake commented Feb 5, 2013

  • remove NTP servers in Fukuoka University
    • add public NTP servers hosted by NICT

 * remove NTP servers in Fukuoka University
 * add public NTP servers hosted by NICT
@kobliha
Copy link
Member

kobliha commented Feb 8, 2013

Have you generated the new list using this script?
https://github.com/yast/yast-ntp-client/blob/master/data/get_servers.pl

It's not a static list. File ntp_servers.ycp will be overwritten with the script output next time anyone runs that script.

@mvidner should know more if needed.

@ftake
Copy link
Contributor Author

ftake commented Feb 12, 2013

@kobliha
This script is still used? The URL in the script returns 404 and the sever list is moved to
http://support.ntp.org/bin/view/Servers/StratumOneTimeServers

The new list includes the ntp server I wanted to add.
I guess ntp_servers.ycp has not been synchronized for several (5?) years.

If possible, let's update ntp_servers.ycp by the script but we need to confirm entries written by hand are not removed.
0bbc38c

@kobliha
Copy link
Member

kobliha commented Feb 12, 2013

@ftake The script ''should'' be used but it seems it's been updated four years ago for the last time.

IMO the current solution is not good enough, we need two files:
1.) Created by the (fixed) script
2.) Manually added servers

Then later in the code, these two files need to be read and merged into one list of servers.

@varkoly
Copy link
Contributor

varkoly commented Jun 26, 2013

I've removed https://github.com/yast/yast-ntp-client/blob/master/data/get_servers.pl becouse the used urls
http://www.eecis.udel.edu/~mills/ntp/clock{1,2}b.html do not work. I have to use an other way to generate the time server list.
Alternatives are: http://support.ntp.org/bin/view/Servers/WebSearch or
http://support.ntp.org/bin/view/Servers/NTPPoolServers
I do not know if we have to provide a time-server-list. On the website of NTPPoolServers you can read:
"The NTP Pool DNS system automatically picks time servers which are geographically close for you, but if you want to choose explicitly, there are sub-zones of pool.ntp.org."
There are sub-zones for the continents and for the countries too.

Alternatively we can actualize the timeserver list from
http://support.ntp.org/bin/view/Servers/StratumTwoTimeServers

@kobliha
Copy link
Member

kobliha commented Jun 26, 2013

@varkoly yast-devel mailing-list seems to be the best place for discussing the alternatives

@mvidner mvidner merged commit 25640b2 into yast:master Jul 29, 2013
@mvidner
Copy link
Member

mvidner commented Jul 29, 2013

Merged manually in 0ebe2ea. If you care about automating the list, please open an Issue or a bug.

@ftake
Copy link
Contributor Author

ftake commented Jul 29, 2013

Thank you.

Please mention the following report of bugzilla when you submit the package:
https://bugzilla.novell.com/show_bug.cgi?id=808223

NTP Pool is good idea. We may need also manual list because we have that "national" NTP server which is not in NTP Pool, for example.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants