Update AIO workflow based on review comments

Author: Alex-Welsh

.github/workflows/stackhpc-all-in-one.yml

@@ -79,16 +79,16 @@ jobs:
       - name: Output image tag
         id: image_tag
         run: |
-          echo ::set-output name=image_tag::$(grep stackhpc_${{ inputs.os_distribution }}_${{ inputs.os_release }}_overcloud_host_image_version etc/kayobe/environments/ci-aio/stackhpc-ci.yml | awk '{print $2}')
+          echo image_tag=$(grep stackhpc_${{ inputs.os_distribution }}_${{ inputs.os_release }}_overcloud_host_image_version etc/kayobe/environments/ci-aio/stackhpc-ci.yml | awk '{print $2}') >> $GITHUB_OUTPUT
 
       # Use the image override if set, otherwise use overcloud-os_distribution-os_release-tag
       - name: Output image name
         id: image_name
         run: |
           if [ -z "${{ inputs.vm_image_override }}" ]; then
-            echo ::set-output name=image_name::overcloud-${{ inputs.os_distribution }}-${{ inputs.os_release }}-${{ steps.image_tag.outputs.image_tag }}
+            echo image_name=overcloud-${{ inputs.os_distribution }}-${{ inputs.os_release }}-${{ steps.image_tag.outputs.image_tag }} >> $GITHUB_OUTPUT
           else
-            echo ::set-output name=image_name::${{ inputs.vm_image_override }}
+            echo image_name=${{ inputs.vm_image_override }} >> $GITHUB_OUTPUT
           fi
 
       - name: Install terraform
@@ -268,13 +268,6 @@ jobs:
         run: |
           test $(wc -l < tempest-artifacts/failed-tests) -lt 1
 
-      - name: Upload logs artifact
-        uses: actions/upload-artifact@v3
-        with:
-          name: kolla-logs-${{ inputs.os_distribution }}-${{ inputs.os_release }}-${{ inputs.neutron_plugin }}
-          path: logs.zip
-        if: failure()
-
       - name: Destroy
         run: terraform destroy -auto-approve
         working-directory: ${{ github.workspace }}/terraform/aio

.github/workflows/stackhpc-pull-request.yml

@@ -75,8 +75,6 @@ jobs:
       os_distribution: rocky
       os_release: "8"
       neutron_plugin: ovs
-      # NOTE: The current SMS lab Rocky8 image has moved ahead of our release
-      # train snapshots, causing failures installing some packages.
       OS_CLOUD: sms-lab-release
     secrets: inherit
     if: github.repository == 'stackhpc/stackhpc-kayobe-config'
@@ -91,14 +89,12 @@ jobs:
       os_distribution: rocky
       os_release: "8"
       neutron_plugin: ovn
-      # NOTE: The current SMS lab Rocky8 image has moved ahead of our release
-      # train snapshots, causing failures installing some packages.
       OS_CLOUD: sms-lab-release
     secrets: inherit
     if: github.repository == 'stackhpc/stackhpc-kayobe-config'
 
-  all-in-one-ubuntu-focal-ovs:
-    name: aio (Ubuntu Focal OVS)
+  all-in-one-ubuntu-ovs:
+    name: aio (Ubuntu OVS)
     needs:
       - build-kayobe-image
     uses: ./.github/workflows/stackhpc-all-in-one.yml
@@ -112,8 +108,8 @@ jobs:
     secrets: inherit
     if: github.repository == 'stackhpc/stackhpc-kayobe-config'
 
-  all-in-one-ubuntu-focal-ovn:
-    name: aio (Ubuntu Focal OVN)
+  all-in-one-ubuntu-ovn:
+    name: aio (Ubuntu OVN)
     needs:
       - build-kayobe-image
     uses: ./.github/workflows/stackhpc-all-in-one.yml

etc/kayobe/environments/ci-aio/stackhpc-ci.yml

@@ -83,7 +83,7 @@ stackhpc_docker_registry_password: !vault |
 
 # Overcloud host image tags
 stackhpc_centos_8-stream_overcloud_host_image_version: "yoga-20230525T095243"
-stackhpc_rocky_8_overcloud_host_image_version: "yoga-20230524T084837"
+stackhpc_rocky_8_overcloud_host_image_version: "yoga-20230629T135322"
 stackhpc_rocky_9_overcloud_host_image_version: "yoga-20230515T145140"
 stackhpc_ubuntu_focal_overcloud_host_image_version: "yoga-20230609T120720"
 stackhpc_ubuntu_jammy_overcloud_host_image_version: "yoga-20230609T120720"

etc/kayobe/stackhpc-overcloud-host-images.yml

@@ -73,6 +73,8 @@ resource "null_resource" "kayobe-aio" {
       host = openstack_compute_instance_v2.kayobe-aio.access_ip_v4
       user = var.ssh_username
       private_key = file("id_rsa")
+      # Terraform will run the start script from /tmp by default. For the
+      # current images, /tmp is noexec, so the path must be changed
       script_path = "/home/${var.ssh_username}/start.sh"
     }