Merge "Make it possible to override automatic fluentd version detection"

This commit is contained in:
Zuul 2021-06-23 13:02:49 +00:00 committed by Gerrit Code Review
commit b22a7726aa
2 changed files with 15 additions and 3 deletions

View File

@ -33,7 +33,9 @@
action: "ensure_image" action: "ensure_image"
common_options: "{{ docker_common_options }}" common_options: "{{ docker_common_options }}"
image: "{{ service.image }}" image: "{{ service.image }}"
when: service | service_enabled_and_mapped_to_host when:
- fluentd_version is not defined or fluentd_binary is not defined
- service | service_enabled_and_mapped_to_host
- name: Fetch fluentd image labels - name: Fetch fluentd image labels
vars: vars:
@ -43,12 +45,16 @@
docker_image_info: docker_image_info:
name: "{{ service.image }}" name: "{{ service.image }}"
register: fluentd_labels register: fluentd_labels
when: service | service_enabled_and_mapped_to_host when:
- fluentd_version is not defined or fluentd_binary is not defined
- service | service_enabled_and_mapped_to_host
- name: Set fluentd facts - name: Set fluentd facts
set_fact: set_fact:
fluentd_binary: "{{ fluentd_labels.images.0.ContainerConfig.Labels.fluentd_binary }}" fluentd_binary: "{{ fluentd_labels.images.0.ContainerConfig.Labels.fluentd_binary }}"
when: common_services.fluentd | service_enabled_and_mapped_to_host when:
- fluentd_binary is not defined
- common_services.fluentd | service_enabled_and_mapped_to_host
- include_tasks: copy-certs.yml - include_tasks: copy-certs.yml
when: when:

View File

@ -0,0 +1,6 @@
---
features:
- |
Adds the ability to override the automatic detection of `fluentd_version`
and `fluentd_binary`. These can now be defined as extra variables. This
removes the dependency of having docker configured for config generation.