pre-commit whitespace fixes

.pre-commit-config.yaml

@@ -19,10 +19,10 @@ repos:
           - --template={}
           - --pattern=[a-zA-Z0-9]{1,10}-[0-9]{1,6}
   - repo: https://github.com/ansible/ansible-lint.git
-    stages: [manual]
     rev: v24.2.2
     hooks:
       - id: ansible-lint
+        stages: [manual]
         additional_dependencies:
           - .
           - ansible-core==2.16.6

README.md

@@ -31,7 +31,7 @@ helper-script `bin/ansible-with-atl-env` and the corresponding
 ## Customizing your deployment
 
 To customise playbook behaviour, you can fork this repository and edit it as
-needed. However, for one-off tasks you can also override the default and 
+needed. However, for one-off tasks you can also override the default and
 calculated settings with special values. To do this, provide command-line overrides to
 [ansible-playbook](https://docs.ansible.com/ansible/latest/cli/ansible-playbook.html).
 
@@ -77,15 +77,15 @@ to "ansible-core-2.13" and/or manually set the stack's "pinned-ansible-sha" SSM
 
 ### Other customizable parameters
 
-For more deployment customization options, consult the following files for parameters you can 
+For more deployment customization options, consult the following files for parameters you can
 override:
 
 - [`/roles/product_install/defaults/main.yml`](roles/product_install/defaults/main.yml)
 - [`/group_vars/aws_node_local.yml`](group_vars/aws_node_local.yml)
 
-More customizable parameters are defined in specific roles -- specifically, in the 
+More customizable parameters are defined in specific roles -- specifically, in the
 role's `defaults/main.yml` file. Most of these parameters use the `atl_` prefix. You can
-use the following [Bitbucket code search query](https://confluence.atlassian.com/bitbucket/search-873876782.html) 
+use the following [Bitbucket code search query](https://confluence.atlassian.com/bitbucket/search-873876782.html)
 to find them:
 
     repo:dc-deployments-automation repo:dc-deployments-automation path:*/defaults/main.yml atl
@@ -108,7 +108,7 @@ environment and running tests.
 This suite is intended to consist of many small, composable roles that can
 be combined together into playbooks. Wherever possible, roles should be product-agnostic
 (e.g. downloads) and platform-agnostic. Functions that are product-specific or
-platform-specific are split off into separate roles. 
+platform-specific are split off into separate roles.
 
 Roles should be reasonably self-contained, with sensible defaults configured in
 `/roles/<role>/defaults/main.yml`. Like all playbook parameters, you can override
@@ -127,7 +127,7 @@ should be run first.
   more information.
 * Inventory files are under `inv/`. For AWS `cfn-init` the inventory
   `inv/aws_node_local` inventory is probably what you want.
-    * Note that this expects the environment to be setup with infrastructure information. 
+    * Note that this expects the environment to be setup with infrastructure information.
       Refer to the [Usage](#markdown-header-usage) section for more information.
 * Global group vars loaded automatically from `group_vars/<group>.yml`. In
   particular note `group_vars/aws_node_local.yml` which loads infrastructure

aws_confluence_synchrony_node.yml

@@ -12,7 +12,7 @@
     atl_use_system_jdk: true
     java_major_version: "11"
     atl_download_format: "tarball"
-    
+
     atl_startup_systemd_params:
       - "EnvironmentFile=/etc/atl"
       - "EnvironmentFile=/etc/atl.synchrony"

roles/bitbucket_dataset_restore/tasks/main.yml

@@ -30,7 +30,7 @@
     body_format: json
     return_content: yes
   register: output
-  until:  output.status == 200 
+  until:  output.status == 200
   retries: 6
   delay: 15
   failed_when: output is defined and output.json is defined and output.json.state != 'INITIALISING'
@@ -50,7 +50,7 @@
   delay: 10
 
 - name: create lock file
-  ansible.builtin.file: 
+  ansible.builtin.file:
     path: "{{ atl_product_home_shared }}/data/migration/import/lock.file"
     state: touch
   when: import_status.json.state == 'COMPLETED'

roles/confluence_config/templates/server.xml.j2

@@ -50,7 +50,7 @@
     <Connector port="{{ atl_apptunnel_port }}"
                connectionTimeout="20000"
                maxThreads="200"
-               minSpareThreads="10" 
+               minSpareThreads="10"
                enableLookups="false"
                acceptCount="10"
                URIEncoding="UTF-8"/>

roles/crowd_config/templates/server.xml.j2

@@ -1,39 +1,39 @@
 <?xml version="1.0" encoding="utf-8"?>
 
-<Server port="{{ atl_tomcat_mgmt_port }}" 
+<Server port="{{ atl_tomcat_mgmt_port }}"
         shutdown="SHUTDOWN">
 
     <Service name="Catalina">
 
-        <Connector acceptCount="{{ atl_tomcat_acceptcount }}" 
-                   connectionTimeout="{{ atl_tomcat_connectiontimeout }}" 
-                   disableUploadTimeout="true" 
-                   enableLookups="{{ atl_tomcat_enablelookups }}" 
-                   maxHttpHeaderSize="8192" 
-                   maxThreads="{{ atl_tomcat_maxthreads }}" 
-                   minSpareThreads="{{ atl_tomcat_minsparethreads }}" 
-                   port="{{ atl_tomcat_port }}" 
-                   redirectPort="{{ atl_tomcat_redirectport }}" 
-                   useBodyEncodingForURI="true" 
-                   URIEncoding="UTF-8" 
-                   compression="on" 
-                   sendReasonPhrase="true" 
-                   compressableMimeType="text/html,text/xml,application/xml,text/plain,text/css,application/json,application/javascript,application/x-javascript" 
-                   secure="{{ atl_tomcat_secure }}" 
-                   scheme="{{ atl_tomcat_scheme }}" 
-                   proxyName="{{ atl_proxy_name }}" 
-                   proxyPort="{{ atl_proxy_port }}" 
+        <Connector acceptCount="{{ atl_tomcat_acceptcount }}"
+                   connectionTimeout="{{ atl_tomcat_connectiontimeout }}"
+                   disableUploadTimeout="true"
+                   enableLookups="{{ atl_tomcat_enablelookups }}"
+                   maxHttpHeaderSize="8192"
+                   maxThreads="{{ atl_tomcat_maxthreads }}"
+                   minSpareThreads="{{ atl_tomcat_minsparethreads }}"
+                   port="{{ atl_tomcat_port }}"
+                   redirectPort="{{ atl_tomcat_redirectport }}"
+                   useBodyEncodingForURI="true"
+                   URIEncoding="UTF-8"
+                   compression="on"
+                   sendReasonPhrase="true"
+                   compressableMimeType="text/html,text/xml,application/xml,text/plain,text/css,application/json,application/javascript,application/x-javascript"
+                   secure="{{ atl_tomcat_secure }}"
+                   scheme="{{ atl_tomcat_scheme }}"
+                   proxyName="{{ atl_proxy_name }}"
+                   proxyPort="{{ atl_proxy_port }}"
                    protocol="{{ atl_tomcat_protocol }}">
         </Connector>
 
-        <Engine defaultHost="localhost" 
+        <Engine defaultHost="localhost"
                 name="Catalina">
-            <Host appBase="webapps" 
-                  autoDeploy="true" 
-                  name="localhost" 
+            <Host appBase="webapps"
+                  autoDeploy="true"
+                  name="localhost"
                   unpackWARs="true">
-              <Context path="{{ atl_tomcat_contextpath }}" 
-                       docBase="../../crowd-webapp" 
+              <Context path="{{ atl_tomcat_contextpath }}"
+                       docBase="../../crowd-webapp"
                        debug="0"
                        useHttpOnly="true">
                        <Manager pathname="">
@@ -45,20 +45,20 @@
         <!-- To connect to an external web server (typically Apache) -->
         <!-- Define an AJP 1.3 Connector on port 8009 -->
         <!--
-            <Connector port="8009" 
-                       enableLookups="{{ atl_tomcat_enablelookups }}" 
-                       redirectPort="{{ atl_tomcat_redirectport }}" 
+            <Connector port="8009"
+                       enableLookups="{{ atl_tomcat_enablelookups }}"
+                       redirectPort="{{ atl_tomcat_redirectport }}"
                        protocol=""AJP/1.3"" />
         -->
-        <Connector port="{{ atl_tomcat_redirectport }}" 
-                   protocol="{{ atl_tomcat_protocol }}" 
-                   relaxedPathChars="[]|" 
-                   relaxedQueryChars='[]|{}^\`"&lt;&gt;' 
-                   maxHttpHeaderSize="65536" 
-                   URIEncoding="UTF-8" 
-                   connectionTimeout="{{ atl_tomcat_connectiontimeout }}" 
-                   useBodyEncodingForURI="true" 
-                   compression="on" 
+        <Connector port="{{ atl_tomcat_redirectport }}"
+                   protocol="{{ atl_tomcat_protocol }}"
+                   relaxedPathChars="[]|"
+                   relaxedQueryChars='[]|{}^\`"&lt;&gt;'
+                   maxHttpHeaderSize="65536"
+                   URIEncoding="UTF-8"
+                   connectionTimeout="{{ atl_tomcat_connectiontimeout }}"
+                   useBodyEncodingForURI="true"
+                   compression="on"
                    compressableMimeType="text/html,text/xml,text/plain,text/css,application/json,application/javascript,application/x-javascript">
         </Connector>
     </Service>

roles/jira_config/templates/server.xml.j2

@@ -50,7 +50,7 @@
     <Connector port="{{ atl_apptunnel_port }}"
                connectionTimeout="20000"
                maxThreads="200"
-               minSpareThreads="10" 
+               minSpareThreads="10"
                enableLookups="false"
                acceptCount="10"
                URIEncoding="UTF-8"

roles/product_install/molecule/jira_all/converge.yml

@@ -7,7 +7,7 @@
     atl_product_user: "jira"
     # Version 8.14.0 is chosen deliberately as it is a version where jira and jira-software returns different buildNumber from MPAC
     # See DCD-1216 for context
-    atl_product_version: "8.14.0" 
+    atl_product_version: "8.14.0"
     atl_install_jsd_as_obr: true
     atl_systemd_service_name: "jira.service"
     atl_jdbc_encoding: 'UNICODE'

roles/product_install/tasks/main.yml

@@ -186,13 +186,13 @@
       failed_when: false
       ignore_errors: yes
       no_log: true
-      when: 
+      when:
         - ansible_ec2_placement_region is defined
         - atl_download_secret_name is defined
       tags:
         - runtime_pkg
 
-    # Fetch binary and copy to temp 
+    # Fetch binary and copy to temp
     # optionally use basic_auth creds from secrets_manager
     - name: Fetch binary
       ansible.builtin.get_url: