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

Retry if fail with ARTIFACT_EXISTS error code #3422

Merged
merged 1 commit into from
Jan 28, 2024

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Jan 27, 2024

With the new parallel product assembly it seem to happen that a download is performed when another thread has already added it.

In such case, instead of fail the makeLocallyAvailable method with an error that the artifact is already there just sleep a little bit and then retry the attempt.

With the new parallel product assembly it seem to happen that a download
is performed when another thread has already added it.

In such case, instead of fail the makeLocallyAvailable method with an
error that the artifact is already there just sleep a little bit and
then retry the attempt.
@laeubi laeubi added the backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change label Jan 27, 2024
Copy link

Test Results

  579 files  ±0    579 suites  ±0   3h 42m 22s ⏱️ + 6m 13s
  392 tests ±0    384 ✅ ±0   7 💤 ±0  1 ❌ ±0 
1 176 runs  ±0  1 153 ✅ ±0  22 💤 ±0  1 ❌ ±0 

For more details on these failures, see this check.

Results for commit 142b420. ± Comparison against base commit 502a622.

@laeubi laeubi merged commit dc94fbf into eclipse-tycho:main Jan 28, 2024
9 of 11 checks passed
Copy link

💚 All backports created successfully

Status Branch Result
tycho-4.0.x

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant