Skip to content

Commit

Permalink
Use common variable for TLS
Browse files Browse the repository at this point in the history
fixes #248

This is something between a workaround and a partial solution. While it
makes more sense to use a role related variable for both Logstash and
Elasticsearch output (other than before) it's still not the goal we
want.

So overhauling the whole process of determining whether we need TLS or
not is due. I'll start a discussion and we need to adjust all roles to
the solution we find. Now there are similarities but it's not exactly
the same in every role.
  • Loading branch information
widhalmt committed Sep 12, 2023
1 parent 96c04c0 commit 357865e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion roles/beats/templates/auditbeat.yml.j2
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ setup.template.settings:
setup.kibana:
{% if beats_auditbeat_output == "elasticsearch" %}
output.elasticsearch:
{% if elasticsearch_http_security | bool %}
{% if beats_security | bool %}
{% if elasticstack_full_stack | bool %}
hosts: [ {% for host in groups['elasticsearch'] %}"https://{{ host }}:{{ elasticstack_elasticsearch_http_port }}"{% if not loop.last %},{% endif %}{% endfor %}]
{% else %}
Expand Down
2 changes: 1 addition & 1 deletion roles/beats/templates/metricbeat.yml.j2
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ setup.template.settings:
setup.kibana:
{% if beats_metricbeat_output == "elasticsearch" %}
output.elasticsearch:
{% if elasticsearch_http_security | bool %}
{% if beats_security | bool %}
{% if elasticstack_full_stack | bool %}
hosts: [ {% for host in groups['elasticsearch'] %}"https://{{ host }}:{{ elasticstack_elasticsearch_http_port }}"{% if not loop.last %},{% endif %}{% endfor %}]
{% else %}
Expand Down

0 comments on commit 357865e

Please sign in to comment.