Skip to content

[7.x] [Ingest Manager] Remove success: true as top-level API response (#73223)#76542

Merged
jfsiii merged 1 commit intoelastic:7.xfrom
jfsiii:backport/7.x/pr-73223
Sep 2, 2020
Merged

[7.x] [Ingest Manager] Remove success: true as top-level API response (#73223)#76542
jfsiii merged 1 commit intoelastic:7.xfrom
jfsiii:backport/7.x/pr-73223

Conversation

@jfsiii
Copy link
Copy Markdown
Contributor

@jfsiii jfsiii commented Sep 2, 2020

Backports the following commits to 7.x:

* Big bang commit removing top-level success property in API response

Left in check-permissions and in array of objects returned by delete package configs

* Remove success property from  mocks

* Resolve conflict from upstream changes

* Remove success property (after upstream merge)

* Remove more 'success'es after merging in upstream

* Remove success from some tests

* Remove success from OpenAPI spec

* Revert prior try/catch. Use res.ok

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@jfsiii jfsiii added the backport This PR is a backport of another PR label Sep 2, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

async chunks size

id value diff baseline
ingestManager 1.1MB -272.0B 1.1MB

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

@jfsiii jfsiii requested review from jen-huang, nchaulet and ph September 2, 2020 19:08
@jfsiii jfsiii merged commit b04c99d into elastic:7.x Sep 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants