Skip to content
This repository has been archived by the owner on Oct 12, 2019. It is now read-only.

Commit

Permalink
Remove redundant shinken_enable_web_ui2 variable
Browse files Browse the repository at this point in the history
This variable was used once in the project to indicate the web ui should be
installed. Since we already need to add it to shinken_broker_modules it makes
more sense to re use that variable.
Closes: #2
  • Loading branch information
goetzk committed Mar 3, 2017
1 parent 7ba2e4c commit fb434e7
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 7 deletions.
3 changes: 0 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,6 @@ have not been implemented yet)
# Path of broker configuration
shinken_config_broker: /etc/shinken/brokers/

# Do we want the web UI?
shinken_enable_web_ui2: true

# List of modules to enable in the broker. 'webui2' enables web interface.
shinken_broker_modules: 'webui2'

Expand Down
3 changes: 0 additions & 3 deletions defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,6 @@ shinken_virtualenv: /srv/shinken/environment
# Path of broker configuration
shinken_config_broker: /etc/shinken/brokers/

# Do we want the web UI?
shinken_enable_web_ui2: true

# List of modules to enable in the broker. 'webui2' enables web interface.
shinken_broker_modules: 'webui2'

Expand Down
2 changes: 1 addition & 1 deletion tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,5 @@
- include: configure-shinken.yml

- include: configure-shinken-webui.yml
when: "{{ shinken_enable_web_ui2 }}"
when: '"webui2" in shinken_broker_modules'

0 comments on commit fb434e7

Please sign in to comment.