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

[build] automatically retry all downloads #119642

Merged
merged 2 commits into from
Nov 24, 2021

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Nov 24, 2021

We had a failure on CI recently because we weren't able to download the shasums for nodejs. This HTTP request was being done with axios directly because we're not trying to download to the filesystem and validate the shasums of the shasums, so we couldn't use the existing download() helper. This renames that helper to downloadToDisk() and adds a downloadToString() helper which wraps axios, adds retries and logging about the requests, and is now in use in all locations where Axios was used in the build scripts.

@spalger spalger added Team:Operations Team label for Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.16.0 v8.1.0 labels Nov 24, 2021
@spalger spalger force-pushed the implement/build-retry-all-downloads branch from 93333fb to 3a8a404 Compare November 24, 2021 18:24
@spalger spalger force-pushed the implement/build-retry-all-downloads branch from 3a8a404 to ae320d7 Compare November 24, 2021 18:28
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@spalger spalger marked this pull request as ready for review November 24, 2021 21:03
@spalger spalger requested a review from a team as a code owner November 24, 2021 21:03
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@spalger spalger merged commit 3358874 into elastic:main Nov 24, 2021
@spalger spalger deleted the implement/build-retry-all-downloads branch November 24, 2021 23:42
@spalger spalger added the auto-backport Deprecated - use backport:version if exact versions are needed label Nov 24, 2021
@kibanamachine
Copy link
Contributor

💔 Backport failed

Status Branch Result
8.0 Commit could not be cherrypicked due to conflicts
7.16 Commit could not be cherrypicked due to conflicts

To backport manually run:
node scripts/backport --pr 119642

spalger pushed a commit to spalger/kibana that referenced this pull request Nov 25, 2021
spalger pushed a commit to spalger/kibana that referenced this pull request Nov 25, 2021
(cherry picked from commit 3358874)
(cherry picked from commit 1513db7)
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Nov 25, 2021
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

2 similar comments
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

spalger pushed a commit that referenced this pull request Nov 30, 2021
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Nov 30, 2021
spalger pushed a commit that referenced this pull request Nov 30, 2021
TinLe pushed a commit to TinLe/kibana that referenced this pull request Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.16.0 v8.0.0 v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants