Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix github upload artifact error #229

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

dzane17
Copy link
Collaborator

@dzane17 dzane17 commented Jan 30, 2025

Description

Fix github upload artifact error on 2.x, 2.19 branches. Verified fix on 2.x branch in #228.

Error:

Run actions/upload-artifact@v4
  with:
    name: query-insights-plugin-ubuntu-latest
    path: query-insights-artifacts
    if-no-files-found: warn
    compression-level: 6
    overwrite: false
    include-hidden-files: false
  env:
    ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION: true
    JAVA_HOME_17.0.14_x64: /__t/jdk/17.0.14/x64
    JAVA_HOME: /__t/jdk/17.0.14/x64
    JAVA_HOME_17_0_14_X64: /__t/jdk/17.0.14/x64
/usr/bin/docker exec  c0d55acae0991e6b[2](https://github.com/opensearch-project/query-insights/actions/runs/12941332812/job/36097209054#step:10:2)c1058b7b22b7e16661fdfab119b0ed99f5ad7ca118eee4a sh -c "cat /etc/*release | grep ^ID"
With the provided path, there will be 1 file uploaded
Artifact name is valid!
Root directory input is valid!
Error: Failed to CreateArtifact: Received non-retryable error: Failed request: ([4](https://github.com/opensearch-project/query-insights/actions/runs/12941332812/job/36097209054#step:10:4)09) Conflict: an artifact with this name already exists on the workflow run

Ref: https://github.com/opensearch-project/security-analytics/blob/dca74ce58b638de442f49b06cce78a113c55bc2b/.github/workflows/ci.yml#L67-L72

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dzane17 dzane17 marked this pull request as ready for review January 30, 2025 21:13
@dzane17 dzane17 merged commit 8f6ea9b into opensearch-project:main Jan 30, 2025
14 checks passed
@dzane17 dzane17 deleted the main-upload-artifact branch January 30, 2025 21:26
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jan 30, 2025
Signed-off-by: David Zane <[email protected]>
(cherry picked from commit 8f6ea9b)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jan 30, 2025
Signed-off-by: David Zane <[email protected]>
(cherry picked from commit 8f6ea9b)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
dzane17 pushed a commit that referenced this pull request Jan 30, 2025
(cherry picked from commit 8f6ea9b)

Signed-off-by: David Zane <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
dzane17 pushed a commit that referenced this pull request Jan 30, 2025
(cherry picked from commit 8f6ea9b)

Signed-off-by: David Zane <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants