fluentd: Fix check for external elasticsearch

When enable_elasticsearch is False, and kolla_internal_fqdn is set
and elasticsearch_address is the default, fluentd is still enabling
output to elasticsearch.

elasticsearch_address defaults to kolla_internal_fqdn, but the
check to enable logging to an "external" elasticsearch host
compares with kolla_internal_vip_address (fqdn != vip address).

This patch fixes the comparison to use the kolla_internal_fqdn.

Closes-Bug: #1927880
Change-Id: If23a6b1273c2639d1296becc9d222546d52f63ac
(cherry picked from commit 283b0de)

Author: Scott Shambarger

ansible/roles/common/tasks/config.yml

@@ -111,7 +111,7 @@
   vars:
     log_direct_to_elasticsearch: >-
       {{ ( enable_elasticsearch | bool or
-           ( elasticsearch_address != kolla_internal_vip_address )) and
+           ( elasticsearch_address != kolla_internal_fqdn )) and
          not enable_monasca | bool }}
     fluentd_version: "{{ fluentd_labels.images.0.ContainerConfig.Labels.fluentd_version | default('0.12') }}"
     # Inputs

releasenotes/notes/bug-1927880-cc407f18f415bbd2.yaml

@@ -0,0 +1,6 @@
+---
+fixes:
+  - |
+    Fixes elasticsearch fluentd output being enabled when
+    elasticsearch is not enabled.
+    `LP#1927880 <https://launchpad.net/bugs/1927880>`__