)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"32921d63ef4a4afa66cb08357283ed273d3f28d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a10e99d0_9d466a65","updated":"2021-11-01 11:08:04.000000000","message":"Thanks for this. Please include a release note and if possible also document the new variable.\n\nIn addition to the comment in-line please also apply the change to Elasticsearch output config in 00-local.conf.j2\n","commit_id":"33367b80ca6d3fa069e4659aa4c5318cfb77de1c"}],"ansible/roles/common/templates/conf/output/01-es.conf.j2":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"32921d63ef4a4afa66cb08357283ed273d3f28d9","unresolved":true,"context_lines":[{"line_number":22,"context_line":"{% if fluentd_elasticsearch_logstash_format is defined %}"},{"line_number":23,"context_line":"       logstash_format {{ fluentd_elasticsearch_logstash_format }}"},{"line_number":24,"context_line":"{% else %}"},{"line_number":25,"context_line":"       logstash_format true"},{"line_number":26,"context_line":"{% endif %}"},{"line_number":27,"context_line":"{% if fluentd_elasticsearch_index_name is defined %}"},{"line_number":28,"context_line":"       index_name {{ fluentd_elasticsearch_index_name }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"57f32695_48872e92","line":25,"updated":"2021-11-01 11:08:04.000000000","message":"I expect you meant to remove line 25? Given that this is an optional setting [1], and defaults to false, could you just move the case when it is true to inside the else branch below (line 30)?\n\n[1] https://docs.fluentd.org/output/elasticsearch","commit_id":"33367b80ca6d3fa069e4659aa4c5318cfb77de1c"}]}