chore(workflows): simplify YAML syntax and improve readability

- Removed unnecessary quotes around strings in YAML files.
- Consolidated list items to single-line format where applicable.
- Ensured consistent naming conventions for job steps.
- Improved readability by removing extra blank lines.

Author: CoMfUcIoS

.github/workflows/test-add-compiler-matrix.yml

@@ -1,70 +1,58 @@
 ---
-name: "Add compiler matrix"
-
+name: Add compiler matrix
 on:
   pull_request:
     paths:
-      - ".github/workflows/**/*"
-      - "spec/**/*"
-      - "lib/**/*"
-      - "tasks/**/*"
-      - "functions/**/*"
-      - "types/**/*"
-      - "plans/**/*"
-      - "hiera/**/*"
-      - "manifests/**/*"
-      - "templates/**/*"
-      - "files/**/*"
-      - "metadata.json"
-      - "Rakefile"
-      - "Gemfile"
-      - "provision.yaml"
-      - ".rspec"
-      - ".rubocop.yml"
-      - ".puppet-lint.rc"
-      - ".fixtures.yml"
+      - .github/workflows/**/*
+      - spec/**/*
+      - lib/**/*
+      - tasks/**/*
+      - functions/**/*
+      - types/**/*
+      - plans/**/*
+      - hiera/**/*
+      - manifests/**/*
+      - templates/**/*
+      - files/**/*
+      - metadata.json
+      - Rakefile
+      - Gemfile
+      - provision.yaml
+      - .rspec
+      - .rubocop.yml
+      - .puppet-lint.rc
+      - .fixtures.yml
     branches: [main]
   workflow_dispatch: {}
-
 jobs:
   test-add-compiler:
-    name: "PE ${{ matrix.version }} ${{ matrix.architecture }} on ${{ matrix.image }}"
+    name: PE ${{ matrix.version }} ${{ matrix.architecture }} on ${{ matrix.image }}
     runs-on: ubuntu-20.04
     env:
       BOLT_GEM: true
       BOLT_DISABLE_ANALYTICS: true
-      LANG: "en_US.UTF-8"
+      LANG: en_US.UTF-8
     strategy:
       fail-fast: false
       matrix:
-        architecture:
-          - standard
-          - large
-          - extra-large
-        version:
-          - 2021.7.7
-          - 2023.6.0
-        image:
-          - centos-7
-          - almalinux-cloud/almalinux-8
+        architecture: [standard, large, extra-large]
+        version: [2021.7.7, 2023.6.0]
+        image: [centos-7, almalinux-cloud/almalinux-8]
     steps:
-      - name: "Checkout Source"
+      - name: Checkout Source
         uses: actions/checkout@v2
-
-      - name: "Activate Ruby 2.7"
+      - name: Activate Ruby 2.7
         uses: ruby/setup-ruby@v1
         with:
-          ruby-version: "2.7"
+          ruby-version: '2.7'
           bundler-cache: true
-
-      - name: "Print bundle environment"
+      - name: Print bundle environment
         if: ${{ github.repository_owner == 'puppetlabs' }}
         run: |
           echo ::group::info:bundler
             bundle env
           echo ::endgroup::
-
-      - name: "Provision test cluster"
+      - name: Provision test cluster
         timeout-minutes: 15
         run: |
           echo ::group::prepare
@@ -74,49 +62,42 @@ jobs:
             echo '    ServerAliveCountMax 2'   >> $HOME/.ssh/config
             bundle exec rake spec_prep
           echo ::endgroup::
-
           echo ::group::provision
             bundle exec bolt plan run peadm_spec::provision_test_cluster \
               --modulepath spec/fixtures/modules \
               provider=provision_service \
               image=${{ matrix.image }} \
               architecture=${{ matrix.architecture }}-with-extra-compiler
           echo ::endgroup::
-
           echo ::group::info:request
             cat request.json || true; echo
           echo ::endgroup::
-
           echo ::group::info:inventory
             sed -e 's/password: .*/password: "[redacted]"/' < spec/fixtures/litmus_inventory.yaml || true
           echo ::endgroup::
-
-      - name: "Install PE on test cluster"
+      - name: Install PE on test cluster
         timeout-minutes: 120
         run: |
           bundle exec bolt plan run peadm_spec::install_test_cluster \
             --inventoryfile spec/fixtures/litmus_inventory.yaml \
             --modulepath spec/fixtures/modules \
             architecture=${{ matrix.architecture }} \
             version=${{ matrix.version }}
-
-      - name: 'Run add_compiler plan'
+      - name: Run add_compiler plan
         timeout-minutes: 50
         run: |
           bundle exec bolt plan run peadm_spec::add_compiler -v \
             --inventoryfile spec/fixtures/litmus_inventory.yaml \
             --modulepath spec/fixtures/modules \
             architecture=${{ matrix.architecture }}
-
-      - name: 'Tear down test cluster'
+      - name: Tear down test cluster
         if: ${{ always() }}
         continue-on-error: true
-        run: |
+        run: |-
           if [ -f spec/fixtures/litmus_inventory.yaml ]; then
             echo ::group::tear_down
               bundle exec rake 'litmus:tear_down'
             echo ::endgroup::
-
             echo ::group::info:request
               cat request.json || true; echo
             echo ::endgroup::

.github/workflows/test-add-compiler.yaml

@@ -1,69 +1,59 @@
 ---
-name: "Add Compiler test"
-
+name: Add Compiler test
 on:
   workflow_dispatch:
     inputs:
       image:
-        description: 'GCP image for test cluster'
+        description: GCP image for test cluster
         required: true
-        default: 'almalinux-cloud/almalinux-8'
+        default: almalinux-cloud/almalinux-8
       architecture:
-        description: 'PE architecture to test'
+        description: PE architecture to test
         required: true
-        default: 'large'
+        default: large
       version:
-        description: 'PE version to install'
+        description: PE version to install
         required: true
-        default: '2021.7.8'
+        default: 2021.7.8
       ssh-debugging:
-        description: 'Boolean; whether or not to pause for ssh debugging'
+        description: Boolean; whether or not to pause for ssh debugging
         required: true
         default: 'false'
-
 jobs:
   test-add-compiler:
-    name: "PE ${{ matrix.version }} ${{ matrix.architecture }} on ${{ matrix.image }}"
+    name: PE ${{ matrix.version }} ${{ matrix.architecture }} on ${{ matrix.image }}
     runs-on: ubuntu-20.04
     env:
       BOLT_GEM: true
       BOLT_DISABLE_ANALYTICS: true
-      LANG: 'en_US.UTF-8'
+      LANG: en_US.UTF-8
     strategy:
       fail-fast: false
       matrix:
-        architecture:
-          - "${{ github.event.inputs.architecture }}"
-        version:
-          - "${{ github.event.inputs.version }}"
-        image:
-          - "${{ github.event.inputs.image }}"
-
+        architecture: ['${{ github.event.inputs.architecture }}']
+        version: ['${{ github.event.inputs.version }}']
+        image: ['${{ github.event.inputs.image }}']
     steps:
-      - name: 'Start SSH session'
+      - name: Start SSH session
         if: ${{ github.event.inputs.ssh-debugging == 'true' }}
         uses: luchihoratiu/debug-via-ssh@main
         with:
           NGROK_AUTH_TOKEN: ${{ secrets.NGROK_AUTH_TOKEN }}
           SSH_PASS: ${{ secrets.SSH_PASS }}
-
-      - name: "Checkout Source"
-        uses: actions/checkout@v4
-
-      - name: "Activate Ruby 2.7"
+      - name: Checkout Source
+        uses: actions/checkout@v2
+      - name: Activate Ruby 2.7
         uses: ruby/setup-ruby@v1
         with:
-          ruby-version: "2.7"
+          ruby-version: '2.7'
           bundler-cache: true
-
-      - name: "Print bundle environment"
+      - name: Print bundle environment
         if: ${{ github.repository_owner == 'puppetlabs' }}
         run: |
           echo ::group::info:bundler
             bundle env
           echo ::endgroup::
-
-      - name: 'Provision test cluster (specified architecture with added compiler)'
+      - name: Provision test cluster (specified architecture with added compiler)
         timeout-minutes: 15
         run: |
           echo ::group::prepare
@@ -73,60 +63,51 @@ jobs:
             echo '    ServerAliveCountMax 2'   >> $HOME/.ssh/config
             bundle exec rake spec_prep
           echo ::endgroup::
-
           echo ::group::provision
             bundle exec bolt plan run peadm_spec::provision_test_cluster \
               --modulepath spec/fixtures/modules \
               provider=provision_service \
               image=${{ matrix.image }} \
               architecture=${{ matrix.architecture }}-with-extra-compiler
           echo ::endgroup::
-
           echo ::group::info:request
             cat request.json || true; echo
           echo ::endgroup::
-
           echo ::group::info:inventory
             sed -e 's/password: .*/password: "[redacted]"/' < spec/fixtures/litmus_inventory.yaml || true
           echo ::endgroup::
-
-      - name: 'Install PE on test cluster'
+      - name: Install PE on test cluster
         timeout-minutes: 120
         run: |
           bundle exec bolt plan run peadm_spec::install_test_cluster \
             --inventoryfile spec/fixtures/litmus_inventory.yaml \
             --modulepath spec/fixtures/modules \
             architecture=${{ matrix.architecture }} \
-            version=${{ matrix.version }} \
-            console_password=${{ secrets.CONSOLE_PASSWORD }}
-
-      - name: 'Run add_compiler plan'
+            version=${{ matrix.version }}
+      - name: Run add_compiler plan
         timeout-minutes: 50
         run: |
           bundle exec bolt plan run peadm_spec::add_compiler -v \
             --inventoryfile spec/fixtures/litmus_inventory.yaml \
             --modulepath spec/fixtures/modules \
             architecture=${{ matrix.architecture }} \
             --stream
-
-      - name: 'Wait as long as the file ${HOME}/pause file is present'
+      - name: Wait as long as the file ${HOME}/pause file is present
         if: ${{ always() && github.event.inputs.ssh-debugging == 'true' }}
         run: |
           while [ -f "${HOME}/pause" ] ; do
             echo "${HOME}/pause present, sleeping for 60 seconds..."
             sleep 60
           done 
           echo "${HOME}/pause absent, continuing workflow."
-
-      - name: 'Tear down test cluster'
+      - name: Tear down test cluster
         if: ${{ always() }}
         continue-on-error: true
-        run: |
+        run: |-
           if [ -f spec/fixtures/litmus_inventory.yaml ]; then
             echo ::group::tear_down
               bundle exec rake 'litmus:tear_down'
             echo ::endgroup::
-
             echo ::group::info:request
               cat request.json || true; echo
             echo ::endgroup::