Skip to content

feat(ci): add bazel downloads retry#3492

Merged
chaokunyang merged 1 commit intoapache:mainfrom
BaldDemian:bazel-retry
Mar 18, 2026
Merged

feat(ci): add bazel downloads retry#3492
chaokunyang merged 1 commit intoapache:mainfrom
BaldDemian:bazel-retry

Conversation

@BaldDemian
Copy link
Contributor

@BaldDemian BaldDemian commented Mar 18, 2026

Why?

Bazel downloads should retry in case of network issues.

What does this PR do?

Actually, Bazel 8.2.1 (the version Fory currently uses) already supports downloads retrying by default. Check --experimental_repository_downloader_retries, --http_connector_attempts and --http_timeout_scaling in the doc.

I just manually doubled these values in .bazelrc

Related issues

Closes #3491.

AI Contribution Checklist

No.

Does this PR introduce any user-facing change?

No.

Benchmark

No.

Copy link
Collaborator

@chaokunyang chaokunyang left a comment

Choose a reason for hiding this comment

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

LGTM

@chaokunyang chaokunyang merged commit 056b94c into apache:main Mar 18, 2026
59 checks passed
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.

[CI] Add retry for bazel download

2 participants