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

Added Chrony NTP support #764

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mpassalacqua
Copy link

Starting with Ubuntu 18.04 (Bionic), chrony becomes the NTP default service. ntpd is not installed by default.
Also the latest Juju ntp charm (for Bionic) deploy chrony instead of ntpd (https://jujucharms.com/ntp/) so Contrail 5 deployment via Juju fails.

ntpd is progressively replaced by chrony also on the other linux distributions (like RHEL) because of the Autokey protocol that has serious security issues.

This is a generic check (not dependent by Linux dist) to support trigger the NTP check depending by the running NTP service.

@pcarver
Copy link
Contributor

pcarver commented Dec 10, 2018

Contrail/Tungsten Fabric does not accept GitHub pull requests. All changes should be submitted via the Gerrit code review tool. If you're not familiar with Gerrit, take a look at section 4 of https://github.com/tungstenfabric/docs/blob/master/Contributor/GettingStarted/getting-started-with-opencontrail-development.md for more info.

The key points are that you need to login to https://review.opencontrail.org with an Ubuntu One account in order to establish your SSH public key. You need to clone the repos from Gerrit rather than (or in addition to) GitHub (or manually add an additional remote to an existing repo rather than cloning if you prefer). And you need to install and use the "git review" extension to Git in order to push changes to Gerrit.

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.

2 participants