fix: backup parameter of stream templates (#571)

Author: JSafieddine

CHANGELOG.md

@@ -17,6 +17,7 @@ BUG FIXES:
 - The default PID path has changed as of NGINX 1.27.5 and 1.28.0.
 - Properly wrap `http_version` number in quotes in both the template defaults and Molecule tests.
 - NGINX `set_real_ip_from` directive template parameter should be a list.
+- Fix backup parameter for stream templates.
 
 TESTS:
 

defaults/main/template.yml

@@ -794,6 +794,7 @@ nginx_config_stream_template_enable: false
 nginx_config_stream_template:
   - template_file: stream/default.conf.j2
     deployment_location: /etc/nginx/conf.d/streams/stream_default.conf
+    backup: true
     config:
       upstreams:  # Configure upstreams
         - name: stream_upstream  # Required

molecule/complete/converge.yml

@@ -760,6 +760,7 @@
         nginx_config_stream_template:
           - template_file: stream/default.conf.j2
             deployment_location: /etc/nginx/conf.d/streams/stream_default.conf
+            backup: true
             config:
               upstreams:
                 - name: stream_upstream

molecule/complete_plus/converge.yml

@@ -656,6 +656,7 @@
         nginx_config_stream_template:
           - template_file: stream/default.conf.j2
             deployment_location: /etc/nginx/conf.d/streams/stream_default.conf
+            backup: true
             config:
               upstreams:
                 - name: stream_upstream

tasks/config/template-config.yml

@@ -121,7 +121,7 @@
   ansible.builtin.template:
     src: "{{ item['template_file'] | default('stream/default.conf.j2') }}"
     dest: "{{ item['deployment_location'] | default('/etc/nginx/conf.d/stream/stream_default.conf') }}"
-    backup: true
+    backup: "{{ item['backup'] | default(true) }}"
     mode: "0644"
   loop: "{{ nginx_config_stream_template }}"
   loop_control: