Add name for cluster dump artifacts

kubeshop · Mar 8, 2024 · 89eb7b4 · 89eb7b4
1 parent b014446
commit 89eb7b4
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 1 deletion.
diff --git a/.github/actions/cloud-slack-e2e/action.yaml b/.github/actions/cloud-slack-e2e/action.yaml
@@ -110,6 +110,8 @@ runs:
     - name: Dump cluster state
       if: ${{ failure() }}
       uses: ./.github/actions/dump-cluster
+      with:
+        name: cloud-slack-e2e
 
     - name: Detect failed jobs
       if: ${{ failure() }}

diff --git a/.github/actions/dump-cluster/action.yaml b/.github/actions/dump-cluster/action.yaml
@@ -1,6 +1,11 @@
 name: Dump cluster state
 description: "Creates an artifacts with cluster dump"
 
+inputs:
+  name:
+    description: "Cluster name"
+    required: true
+
 runs:
   using: "composite"
   steps:
@@ -17,6 +22,6 @@ runs:
     - name: Upload artifacts
       uses: actions/upload-artifact@v2
       with:
-        name: cluster_dump_${{github.sha}}
+        name: cluster_dump_${{github.sha}}_${{ inputs.name }}
         path: "output"
         retention-days: 5 # Default 90 days
diff --git a/.github/workflows/branch-build.yml b/.github/workflows/branch-build.yml
@@ -273,6 +273,8 @@ jobs:
       - name: Dump cluster state
         if: ${{ failure() }}
         uses: ./.github/actions/dump-cluster
+        with:
+          name: cli-migration-e2e
 
       - name: Slack Notification
         uses: rtCamp/action-slack-notify@v2

diff --git a/test/cloud-slack-dev-e2e/cloud_slack_dev_e2e_test.go b/test/cloud-slack-dev-e2e/cloud_slack_dev_e2e_test.go
@@ -301,6 +301,7 @@ func TestCloudSlackE2E(t *testing.T) {
 			if !cfg.Slack.DisconnectWorkspaceAfterTests {
 				return
 			}
+			t.Log("Disconnecting Slack workspace...")
 			gqlCli.MustDeleteSlackWorkspace(t, cfg.BotkubeCloud.TeamOrganizationID, slackWorkspace.ID)
 		})