ansible: install ntp for centos.

[erikh]

Signed-off-by: Erik Hollensbe github@hollensbe.org

[mapuri]

lgtm.

Looks like we now have quite a lot of packages in the ansible playbooks. And if I get it correct, not all are needed for the build environment or for say netplugin service to work but some of them are useful debugging tools? I am wondering if it will be a good idea to re-arrange some of the ansible plays such that we can put infra service, build environment and debug tool installation plays into separate buckets respectively and have them run based on the environment. If it sounds ok, I can track and do this as a separate PR. It will also help me use these playbooks for configuring nodes as part of cluster management.

[erikh]

Go ahead, but I don’t think you’ll find much that should not exist
on deployed servers, maybe the vcs and go stuff. ntpd is going to need
to be on all hosts.

On 10 Sep 2015, at 8:35, Madhav Puri wrote:

lgtm.

Looks like we now have quite a lot of packages in the ansible
playbooks. And if I get it correct, not all are needed for the build
environment or for say netplugin service to work but some of them are
useful debugging tools? I am wondering if it will be a good idea to
re-arrange some of the ansible plays such that we can put infra
service, build environment and debug tool installation plays into
separate buckets respectively and have them run based on the
environment. If it sounds ok, I can track and do this as a separate
PR. It will also help me use these playbooks for configuring nodes as
part of cluster management.

---

Reply to this email directly or view it on GitHub:
#7 (comment)

[mapuri]

ok, that is fair. May be we can tackle this as we start using the playbooks elsewhere.

[erikh]

yeah, I think that’s a safer approach. Then we know what we need.
Let’s leave this issue open so we’re sure to remember that.

On 10 Sep 2015, at 9:26, Madhav Puri wrote:

ok, that is fair. May be we can tackle this as we start using the
playbooks elsewhere.

---

Reply to this email directly or view it on GitHub:
#7 (comment)

[mapuri]

sounds good Erik. I will file a separate issue, so that you can merge and close this PR.