arvindshmicrosoft commented on a change in pull request #270: Support Accumulo
installs on Microsoft Azure
URL: https://github.com/apache/fluo-muchos/pull/270#discussion_r318765360
##########
File path: ansible/roles/common/tasks/hosts.yml
##########
@@ -18,4 +18,4 @@
- name: "ensure hostname is correct"
hostname: name={{ inventory_hostname }}
- name: "ensure /etc/hosts is correct"
- template: src=roles/common/templates/etc_hosts dest=/etc/hosts owner=root
group=root mode=0644
+ copy: src=/etc/etc_hosts dest=/etc/hosts owner=root group=root mode=0644
Review comment:
What we found was that the template-based generation of the /etc/hosts file
at each host was extremely time-consuming for larger clusters (100+ nodes).
Since the file is the same on each node, what we do is to generate it once on
the proxy (using template ansible/roles/proxy/templates/etc_hosts) and then
just copy that file from the proxy to each host. This way we cut down
significantly the time taken for this task (went from literally 10s of minutes
to seconds on these larger clusters).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
