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

[7.x] [Fleet] Finer-grained error information from install/upgrade API (#95649) #97400

Merged
merged 1 commit into from
Apr 18, 2021

Conversation

skh
Copy link
Contributor

@skh skh commented Apr 18, 2021

Backports the following commits to 7.x:

…stic#95649)

* Intercept installation errors and add meta info.

* Adjust mock.

* Catch errors in all steps of install/upgrade.

* Adjust handler for direct package upload.

* Don't throw not-found errors on assets during rollback.

* Correctly catch errors from _installPackage()

* Propagate error from installResult in bulk install case.

* Add tests for rollback.

* Remove unused code.

* Skipping test that doesn't test what it says.

* Fix and reenable test.
@skh skh added the backport label Apr 18, 2021
@skh skh enabled auto-merge (squash) April 18, 2021 14:49
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

API count

id before after diff
fleet 1069 1071 +2

API count missing comments

id before after diff
fleet 979 981 +2

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

@skh skh merged commit 1f9c76b into elastic:7.x Apr 18, 2021
@skh skh deleted the backport/7.x/pr-95649 branch April 18, 2021 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants