sensu / sensu-ansible

An Ansible role to deploy a fully dynamic Sensu stack!
https://ansible-sensu.readthedocs.io
MIT License
126 stars 96 forks source link

fix(group) rabbitmq_servers in default was not updated to new namespace. #207

Closed michaelpporter closed 5 years ago

jaredledvina commented 5 years ago

Noting here, that it's interesting that the integration tests didn't fail without this. I probably need to dig some more and add additional tests to catch this type of failure in the future.

michaelpporter commented 5 years ago
sensu_rabbitmq_server: true

Overrides

sensu_rabbitmq_host: "{{ groups['sensu_rabbitmq_servers'][0] }}"

Groups are not used in the testing it seems.

jaredledvina commented 5 years ago

Ahhhhhhh, makes sense, since we do the full integration test with just one container per Operating System.

michaelpporter commented 5 years ago

molecule.yaml

 inventory:
    group_vars:
      all:
        sensu_master: true
        sensu_include_dashboard: true
        sensu_rabbitmq_server: true
        sensu_redis_server: true
        sensu_rabbitmq_host: "{{ ansible_hostname }}"
        sensu_redis_host: "{{ ansible_hostname }}"
        sensu_api_host: "{{ ansible_hostname }}"
        ansible_default_ipv4:
          address: 127.0.0.1