ci: Adding the overwrite: true parameter to actions/upload-artifact@v4 command (#33593)

## Description
By default the value of overwrite is false. In previous versions of
upload-artifact, this setting was implicit and true. Making this change
to fix the error `Failed to CreateArtifact: Received non-retryable
error: Failed request: (409) Conflict: an artifact with this name
already exists on the workflow run` in Github actions.

## Automation

/ok-to-test tags=""

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!CAUTION]  
> If you modify the content in this section, you are likely to disrupt
the CI result for your PR.

<!-- end of auto-generated comment: Cypress test results  -->


## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [x] No
This commit is contained in:
Arpit Mohan 2024-05-20 17:22:22 +05:30 committed by GitHub
parent 15203d2a04
commit ddefab7306
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
10 changed files with 22 additions and 0 deletions

View File

@ -220,6 +220,7 @@ jobs:
with: with:
name: combined_failed_spec_ci name: combined_failed_spec_ci
path: ~/combined_failed_spec_ci path: ~/combined_failed_spec_ci
overwrite: true
- name: Get Latest flaky Tests - name: Get Latest flaky Tests
shell: bash shell: bash
@ -337,6 +338,7 @@ jobs:
with: with:
name: combined_failed_spec_ci name: combined_failed_spec_ci
path: ~/combined_failed_spec_ci path: ~/combined_failed_spec_ci
overwrite: true
- name: Get Latest flaky Tests - name: Get Latest flaky Tests
shell: bash shell: bash

View File

@ -368,6 +368,7 @@ jobs:
with: with:
name: cypress-console-logs name: cypress-console-logs
path: ${{ github.workspace }}/app/client/cypress/cypress-logs path: ${{ github.workspace }}/app/client/cypress/cypress-logs
overwrite: true
- name: Collect CI container logs - name: Collect CI container logs
if: failure() if: failure()
@ -383,6 +384,7 @@ jobs:
with: with:
name: dockerlogs name: dockerlogs
path: ~/dockerlogs path: ~/dockerlogs
overwrite: true
- name: Rename reports - name: Rename reports
if: failure() if: failure()
@ -396,6 +398,7 @@ jobs:
with: with:
name: results-${{github.run_attempt}} name: results-${{github.run_attempt}}
path: ~/results path: ~/results
overwrite: true
# Set status = failedtest # Set status = failedtest
- name: Set fail if there are test failures - name: Set fail if there are test failures
@ -420,6 +423,7 @@ jobs:
with: with:
name: server-logs-${{ matrix.job }} name: server-logs-${{ matrix.job }}
path: app/server/server-logs.log path: app/server/server-logs.log
overwrite: true
# Set status = success # Set status = success
- name: Save the status of the run - name: Save the status of the run

View File

@ -260,6 +260,7 @@ jobs:
with: with:
name: results-${{github.run_attempt}} name: results-${{github.run_attempt}}
path: ~/results path: ~/results
overwrite: true
- name: Upload cypress snapshots - name: Upload cypress snapshots
if: failure() if: failure()
@ -267,6 +268,7 @@ jobs:
with: with:
name: snapshots name: snapshots
path: ${{ github.workspace }}/app/client/cypress/snapshots path: ${{ github.workspace }}/app/client/cypress/snapshots
overwrite: true
# Set status = failedtest # Set status = failedtest
- name: Set fail if there are test failures - name: Set fail if there are test failures

View File

@ -371,6 +371,7 @@ jobs:
with: with:
name: cypress-console-logs name: cypress-console-logs
path: ${{ github.workspace }}/app/client/cypress/cypress-logs path: ${{ github.workspace }}/app/client/cypress/cypress-logs
overwrite: true
- name: Upload cypress snapshots - name: Upload cypress snapshots
if: always() if: always()
@ -378,6 +379,7 @@ jobs:
with: with:
name: snapshots name: snapshots
path: ${{ github.workspace }}/app/client/cypress/snapshots path: ${{ github.workspace }}/app/client/cypress/snapshots
overwrite: true
- name: Collect CI container logs - name: Collect CI container logs
if: failure() if: failure()
@ -393,6 +395,7 @@ jobs:
with: with:
name: dockerlogs name: dockerlogs
path: ~/dockerlogs path: ~/dockerlogs
overwrite: true
- name: Rename reports - name: Rename reports
if: failure() if: failure()
@ -406,6 +409,7 @@ jobs:
with: with:
name: results-${{github.run_attempt}} name: results-${{github.run_attempt}}
path: ~/results path: ~/results
overwrite: true
# Set status = failedtest # Set status = failedtest
- name: Set fail if there are test failures - name: Set fail if there are test failures
@ -431,6 +435,7 @@ jobs:
with: with:
name: server-logs-${{ matrix.job }} name: server-logs-${{ matrix.job }}
path: app/server/server-logs.log path: app/server/server-logs.log
overwrite: true
# Set status = success # Set status = success
- name: Save the status of the run - name: Save the status of the run

View File

@ -294,6 +294,7 @@ jobs:
with: with:
name: client-build name: client-build
path: app/client/build.tar path: app/client/build.tar
overwrite: true
- name: Put release build in cache - name: Put release build in cache
if: success() && github.ref == 'refs/heads/release' && (github.event_name == 'push' || github.event_name == 'workflow_dispatch' || github.event_name == 'schedule') if: success() && github.ref == 'refs/heads/release' && (github.event_name == 'push' || github.event_name == 'workflow_dispatch' || github.event_name == 'schedule')

View File

@ -93,6 +93,7 @@ jobs:
with: with:
name: client-build name: client-build
path: app/client/build.tar path: app/client/build.tar
overwrite: true
server-build: server-build:
needs: needs:
@ -140,6 +141,7 @@ jobs:
with: with:
name: server-build name: server-build
path: app/server/dist/ path: app/server/dist/
overwrite: true
rts-build: rts-build:
needs: needs:
@ -193,6 +195,7 @@ jobs:
with: with:
name: rts-dist name: rts-dist
path: app/client/packages/rts/rts-dist.tar path: app/client/packages/rts/rts-dist.tar
overwrite: true
package: package:
needs: [prelude, client-build, server-build, rts-build] needs: [prelude, client-build, server-build, rts-build]

View File

@ -155,6 +155,7 @@ jobs:
with: with:
name: combined_failed_spec_ci name: combined_failed_spec_ci
path: ~/combined_failed_spec_ci path: ~/combined_failed_spec_ci
overwrite: true
- name: Get Latest flaky Tests - name: Get Latest flaky Tests
shell: bash shell: bash

View File

@ -153,6 +153,7 @@ jobs:
with: with:
name: rts-dist name: rts-dist
path: app/client/packages/rts/rts-dist.tar path: app/client/packages/rts/rts-dist.tar
overwrite: true
# Set status = success # Set status = success
- name: Save the status of the run - name: Save the status of the run

View File

@ -227,6 +227,7 @@ jobs:
name: failed-server-tests name: failed-server-tests
path: app/server/failed-server-tests.txt path: app/server/failed-server-tests.txt
if-no-files-found: ignore if-no-files-found: ignore
overwrite: true
- name: Fetch server build from cache - name: Fetch server build from cache
if: steps.changed-files-specific.outputs.any_changed == 'false' && success() && github.event_name != 'push' && github.event_name != 'workflow_dispatch' && github.event_name != 'schedule' if: steps.changed-files-specific.outputs.any_changed == 'false' && success() && github.event_name != 'push' && github.event_name != 'workflow_dispatch' && github.event_name != 'schedule'
@ -266,6 +267,7 @@ jobs:
with: with:
name: server-build name: server-build
path: app/server/dist/ path: app/server/dist/
overwrite: true
- name: Put release build in cache - name: Put release build in cache
if: success() && github.ref == 'refs/heads/release' && (github.event_name == 'push' || github.event_name == 'workflow_dispatch' || github.event_name == 'schedule') if: success() && github.ref == 'refs/heads/release' && (github.event_name == 'push' || github.event_name == 'workflow_dispatch' || github.event_name == 'schedule')

View File

@ -261,6 +261,7 @@ jobs:
with: with:
name: combined_failed_spec_ci name: combined_failed_spec_ci
path: ~/combined_failed_spec_ci path: ~/combined_failed_spec_ci
overwrite: true
- name: Return status for ui-matrix - name: Return status for ui-matrix
run: | run: |