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

Add support for decorating and handling 429 errors with the saved objects client #75664

Merged
merged 3 commits into from
Aug 24, 2020

Conversation

mikecote
Copy link
Contributor

In this PR, I'm enhancing the SavedObjectsErrorHelpers class and decorateEsError function to handle 429 errors.

The alerting team is planning to build a feature on top of these errors. This will also remove a dependency on Boom errors for our feature.

@mikecote mikecote added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc Feature:Saved Objects v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.10.0 labels Aug 21, 2020
@mikecote mikecote self-assigned this Aug 21, 2020
@mikecote mikecote marked this pull request as ready for review August 21, 2020 19:14
@mikecote mikecote requested a review from a team as a code owner August 21, 2020 19:14
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-platform (Team:Platform)

Copy link
Contributor

@pgayvallet pgayvallet left a comment

Choose a reason for hiding this comment

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

LGTM for platform changes

@mikecote
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

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

@mikecote mikecote merged commit 4dd0205 into elastic:master Aug 24, 2020
mikecote added a commit to mikecote/kibana that referenced this pull request Aug 24, 2020
…ects client (elastic#75664)

* Add support for decorating 429 errors in the saved objects client

* Update the docs

Co-authored-by: Elastic Machine <[email protected]>
mikecote added a commit that referenced this pull request Aug 24, 2020
…ects client (#75664) (#75762)

* Add support for decorating 429 errors in the saved objects client

* Update the docs

Co-authored-by: Elastic Machine <[email protected]>

Co-authored-by: Elastic Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Saved Objects release_note:skip Skip the PR/issue when compiling release notes Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v7.10.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants