From d61fc746acd5de35706179db25953ff4e0fef2c2 Mon Sep 17 00:00:00 2001
From: Saroj <43822041+sarojsarab@users.noreply.github.com>
Date: Tue, 21 Mar 2023 08:27:11 +0530
Subject: [PATCH] ci: Removed the artifact deleting as we need the artifact in
 case of rerun on cancel (#21609)

## Description
- Removed the artifact deleting as we need the artifact in case of rerun
on cancel

## Type of change
- Workflow yml files

## How Has This Been Tested?
- Manual

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
---
 .github/workflows/integration-tests-command.yml | 14 --------------
 .github/workflows/test-build-docker-image.yml   | 14 --------------
 2 files changed, 28 deletions(-)

diff --git a/.github/workflows/integration-tests-command.yml b/.github/workflows/integration-tests-command.yml
index c258c3f048..71c813d172 100644
--- a/.github/workflows/integration-tests-command.yml
+++ b/.github/workflows/integration-tests-command.yml
@@ -108,13 +108,6 @@ jobs:
           name: failed-spec-ci
           path: ~/failed_spec_ci
 
-      # delete-artifact
-      - name: delete existing combined_failed_spec_ci artifact
-        uses: geekyeggo/delete-artifact@v2
-        with:
-          name: combined_failed_spec_ci
-          failOnError: false
-
       # In case for any ci job failure, create combined failed spec
       - name: "combine all specs for CI"
         if: needs.ci-test.result != 'success'
@@ -129,13 +122,6 @@ jobs:
         with:
           name: combined_failed_spec_ci
           path: ~/combined_failed_spec_ci
-      
-      # delete-artifact
-      - name: delete existing failed-spec-ci artifact
-        uses: geekyeggo/delete-artifact@v2
-        with:
-          name: failed-spec-ci
-          failOnError: false
 
       - name: Get Latest flaky Tests
         shell: bash
diff --git a/.github/workflows/test-build-docker-image.yml b/.github/workflows/test-build-docker-image.yml
index 974a4c7ce6..f1ee7d83aa 100644
--- a/.github/workflows/test-build-docker-image.yml
+++ b/.github/workflows/test-build-docker-image.yml
@@ -89,13 +89,6 @@ jobs:
           name: failed-spec-ci
           path: ~/failed_spec_ci
 
-      # delete-artifact
-      - name: delete existing combined_failed_spec_ci artifact
-        uses: geekyeggo/delete-artifact@v2
-        with:
-          name: combined_failed_spec_ci
-          failOnError: false
-
       # In case for any ci job failure, create combined failed spec
       - name: "combine all specs for CI"
         if: needs.ci-test.result != 'success'
@@ -112,13 +105,6 @@ jobs:
           name: combined_failed_spec_ci
           path: ~/combined_failed_spec_ci
 
-      # delete-artifact
-      - name: delete existing failed-spec-ci artifact
-        uses: geekyeggo/delete-artifact@v2
-        with:
-          name: failed-spec-ci
-          failOnError: false
-
       - name: Return status for ui-matrix
         run: |
           if [[ "${{ needs.ci-test.result }}" == "success" ]]; then