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

Remove ntp sync as ntpd started during rpm install #66

Merged
merged 1 commit into from
Mar 10, 2014
Merged

Remove ntp sync as ntpd started during rpm install #66

merged 1 commit into from
Mar 10, 2014

Conversation

DBezemer
Copy link
Contributor

@DBezemer DBezemer commented Mar 9, 2014

No description provided.

@zoharbabin
Copy link
Contributor

Wasn't this added few days ago?
Was #63 a mistake or something changed that removed the need of this step?

@jessp01
Copy link
Contributor

jessp01 commented Mar 10, 2014

Hi Zohar,

Not sure how #63 is related? that one is me adding info about the template of the ans file.
David, I think the force ntpdate was added by you at the time, no? in any event, I don't think we need it, do you?

@DBezemer
Copy link
Contributor Author

It was added by me yes, and I would say it's necessary, especially on virtual machines. However it conflicts with the start of NTPD during the postinst of kaltura-base. ntpdate needs the available ntp socket, which is occupied as soon as ntpd starts.

@jessp01
Copy link
Contributor

jessp01 commented Mar 10, 2014

ntpd would occupy the socket thus making ntpdate impossible to work.
But why did you find starting the NTP daemon was not enough on VMs?
Thanks,

May the source be with you,

Jess Portnoy

Join us for Kaltura Connect 2014, June 17-18, NYC. Register today!
http://bit.ly/kconnect14-emailsig

On Mon, 10 Mar 2014, David Bezemer wrote:

It was added by me yes, and I would say it's necessary, especially on virtual machines. However it conflicts with the start of NTPD during the postinst of
kaltura-base. ntpdate needs the available ntp socket, which is occupied as soon as ntpd starts.


Reply to this email directly or view it on
GitHub.[5270205__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcxMDA2OTA4MSwiZGF0YSI6eyJpZCI6MjcyOTMxNzl9fQ==--80e5e12c74aa0fb780ba9e15d7227d24a361436d.gif]

jessp01 pushed a commit that referenced this pull request Mar 10, 2014
Remove ntp sync as ntpd started during rpm install
@jessp01 jessp01 merged commit 269af4c into kaltura:master Mar 10, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants