windmill/ansible-role-statsd
Code Issues Proposed changes

Add missing variable

Browse Source 
We don't properly define statsd_npm_name for npm based installs.

Also fix npm package installs, to add missing build dependencies.

Change-Id: I758ac9e469d4d2170ea0aa7be7194d43b1746f72
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This commit is contained in:
Paul Belanger 2019-01-29 21:27:43 -05:00
parent eeab595d3f
commit 02d4456166
2 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
defaults/main.yaml
View File

@ -37,6 +37,8 @@ statsd_git_version: master
statsd_install_method: git statsd_install_method: git
statsd_npm_name: statsd
# tasks/service.yaml # tasks/service.yaml
statsd_file_statsd_service_manage: true statsd_file_statsd_service_manage: true
statsd_file_statsd_service_group: root statsd_file_statsd_service_group: root

2
tasks/install.yaml
View File

@ -22,6 +22,6 @@
package: package:
name: "{{ statsd_build_depends }}" name: "{{ statsd_build_depends }}"
state: present state: present
when: statsd_install_method == 'git' when: statsd_install_method == 'git' or statsd_install_method == 'pip'
- include: "install/{{ statsd_install_method }}.yaml" - include: "install/{{ statsd_install_method }}.yaml"