Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/50#discussion_r56727237
--- Diff: deployment/roles/metron_ui/tasks/main.yml ---
@@ -23,46 +23,33 @@
- { package: 'nodejs'}
- { package: 'npm'}
-- name: Download nvm installer
- get_url:
url=https://raw.githubusercontent.com/creationix/nvm/v0.30.1/install.sh
dest=/tmp/nvm-install.sh
-
-- name: Change nvm installer permissions
- file: path=/tmp/nvm-install.sh mode=0755
-
-- name: Install nvm
- shell: /tmp/nvm-install.sh creates=~/.nvm/nvm.sh
-
-- name: Install Metron UI
- shell: source ~/.bashrc && nvm install 0.10.35 && nvm use 0.10.35 && nvm
alias default 0.10.35 && npm install -g opensoc-ui
creates=/root/.nvm/v0.10.35/lib/node_modules/opensoc-ui/package.json
-
-- name: Add Metron UI config
+- name: Copy Metron UI source
--- End diff --
I am deploying Metron with your PR out to AWS/EC2. One thing I have
noticed so far is that this recursive copy takes FOREVER. Sorry, I don't have
a specific duration to quote, but it seems excessive.
One thing I remember reading is that the 'copy' module in Ansible does not
scale well to lots of files. We may need to tarball up the code and copy it
over that way. Or perhaps you can think of a better alternative.
http://docs.ansible.com/ansible/copy_module.html
> Note
> The âcopyâ module recursively copy facility does not scale to lots
(>hundreds) of files. For alternative, see synchronize module, which is a
wrapper around rsync.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
