Skip to content

[CI] Add resilience to artifacts fetch#622

Open
leo-automation wants to merge 1 commit into
devfrom
leo/retry-on-fail
Open

[CI] Add resilience to artifacts fetch#622
leo-automation wants to merge 1 commit into
devfrom
leo/retry-on-fail

Conversation

@leo-automation

Copy link
Copy Markdown
Collaborator

No description provided.

@ipanfilo ipanfilo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This change covers specific scenario - wheels download. But CI pipeline relies on network for many other steps: Docker image download, Wheels upload, Runtime components installation, Logs upload and so on.
Thus, the changes do not provide a solution for small network glitches.
If wheels downloading is by some reason more error prone than, say, docker image downloading, it should be checked why.
For replacing of all downloads with multi-attempt action, it might be better to use reusable action

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This script is not a part of CI but rather .github/scripts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants