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: CI Artifact Download Timeout. Fixes #12452 #12454

Conversation

Garett-MacGowan
Copy link
Contributor

Fixes #12452

Motivation

The V4 actions seem to have a high tendency to timeout when downloading artifacts.
actions/download-artifact#249

Modifications

Downgrade actions/download-artifact and actions/upload-artifact from v4 to v3 to fix artifact download timeout issues. Add both download-artifact and upload-artifact to dependabot ignore dependency.

Verification

Submit a few empty commits to verify frequency of failures is reduced.

…v3 to fix artifact download timeout issues. Add both download-artifact and upload-artifact to dependabot ignore dependency. Fixes argoproj#12452.

Signed-off-by: Garett MacGowan <garettsoftware@gmail.com>
@Garett-MacGowan Garett-MacGowan changed the title fix: Downgrade actions/download-artifact and actions/upload-artifact from v4 to v3 fix: CI Artifact Download Timeout Jan 4, 2024
…2454

Signed-off-by: Garett MacGowan <garettsoftware@gmail.com>
@agilgur5 agilgur5 added type/dependencies PRs and issues specific to updating dependencies github_actions Pull requests that update Github_actions dependencies area/build Build or GithubAction/CI issues labels Jan 4, 2024
Copy link
Contributor

@agilgur5 agilgur5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a small comment to the dependabot.yml so that we indicate a reason. Otherwise LGTM

.github/dependabot.yml Outdated Show resolved Hide resolved
@agilgur5 agilgur5 self-assigned this Jan 4, 2024
Add comment to ignore actions/download-artifact and actions/upload-artifact until actions/download-artifact/issues/249 is resolved.

Co-authored-by: Anton Gilgur <4970083+agilgur5@users.noreply.github.com>
Signed-off-by: Garett MacGowan <garett.macgowan@gmail.com>
@Garett-MacGowan
Copy link
Contributor Author

Garett-MacGowan commented Jan 4, 2024

Seems to have reduced the error frequency, as expected. Good to go if tests succeed.

@Garett-MacGowan Garett-MacGowan marked this pull request as ready for review January 4, 2024 06:22
@Garett-MacGowan Garett-MacGowan changed the title fix: CI Artifact Download Timeout fix: CI Artifact Download Timeout. Fixes #12452 Jan 4, 2024
@terrytangyuan terrytangyuan merged commit 901fdab into argoproj:main Jan 4, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build Build or GithubAction/CI issues github_actions Pull requests that update Github_actions dependencies type/dependencies PRs and issues specific to updating dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Frequent test failures caused by image download timeout
3 participants