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

[CI] Fix axios requests to allow retry #182386

Merged
merged 1 commit into from
May 3, 2024
Merged

Conversation

delanni
Copy link
Contributor

@delanni delanni commented May 2, 2024

Summary

Related to #181733 - I didn't check, and axios rejects requests' promises that don't result in 2xx.

This PR wraps the attempt in a try-catch to be able to retry.

@delanni delanni added release_note:skip Skip the PR/issue when compiling release notes v8.14.0 v8.15.0 labels May 2, 2024
@delanni delanni requested a review from a team as a code owner May 2, 2024 14:57
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

@delanni delanni merged commit 389025e into elastic:main May 3, 2024
24 checks passed
@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.14 Backport failed because of merge conflicts

You might need to backport the following PRs to 8.14:
- [DOCS] Add a bug fix for 8.13.3 release notes (#182373)
- [Security Solution] - Rename esql setting (#182432)
- [Search] Update pipeline descriptions (#182368)
- [Search] [Playground] Improve UX + fix vector field issue (#182342)
- [[ES

Manual backport

To create the backport manually run:

node scripts/backport --pr 182386

Questions ?

Please refer to the Backport tool documentation

yuliacech pushed a commit to yuliacech/kibana that referenced this pull request May 3, 2024
## Summary
Related to elastic#181733 - I didn't check, and axios rejects requests'
promises that don't result in `2xx`.

This PR wraps the attempt in a try-catch to be able to retry.
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add the label auto-backport or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 182386 locally

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Aug 3, 2024
@delanni delanni added backport:skip This commit does not require backporting and removed backport missing Added to PRs automatically when the are determined to be missing a backport. v8.14.0 labels Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants