option for ui without nginx about ansible-consul

[vcastellm]

+1

[RockingRolli]

+1

[trumant]

#70 removes the nginx role dependency altogether but as noted, before making any changes to the install-ui functionality we should really get a TestKitchen test suite together for it.

[palmerabollo]

+1

#70 needs to change meta/main.yml to add a condition to the dependencies (when consul_enable_nginx_config == true)

dependencies:
  - { role: franklinkim.nginx, when: consul_is_ui == true and consul_enable_nginx_config and ansible_os_family == "Debian"}
  - { role: geerlingguy.nginx, when: consul_is_ui == true and consul_enable_nginx_config and ansible_os_family == "RedHat"}

[trumant]

FYI, there is now a TestKitchen test suite around the UI

[mattfinlayson]

👍

[spinx]

Did a PR for this: #90