Skip to content

[7.x] Add execution context service (#102039)#104686

Merged
mshustov merged 1 commit intoelastic:7.xfrom
mshustov:backport/7.x/pr-102039
Jul 7, 2021
Merged

[7.x] Add execution context service (#102039)#104686
mshustov merged 1 commit intoelastic:7.xfrom
mshustov:backport/7.x/pr-102039

Conversation

@mshustov
Copy link
Contributor

@mshustov mshustov commented Jul 7, 2021

Backports the following commits to 7.x:

* add execution context service on the server-side

* integrate execution context service into http service

* add integration tests for execution context + http server

* update core code

* update integration tests

* update settings docs

* add execution context test plugin

* add a client-side test

* remove requestId from execution context

* add execution context service for the client side

* expose execution context service to plugins

* add execution context service for the server-side

* update http service

* update elasticsearch service

* move integration tests from http to execution_context service

* integrate in es client

* expose to plugins

* refactor functional tests

* remove x-opaque-id from create_cluster tests

* update test plugin package.json

* fix type errors in the test mocks

* fix elasticsearch service tests

* add escaping to support non-ascii symbols in description field

* improve test coverage

* update docs

* remove unnecessary import

* update docs

* Apply suggestions from code review

Co-authored-by: Josh Dover <1813008+joshdover@users.noreply.github.com>

* address comments

* remove execution context cleanup

* add option to disable execution_context service on the server side

* put x-opaque-id test back

* put tests back

* add header size limitation to the server side as well

* fix integration tests

* address comments

Co-authored-by: Josh Dover <1813008+joshdover@users.noreply.github.com>
@mshustov mshustov added the backport This PR is a backport of another PR label Jul 7, 2021
@mshustov mshustov enabled auto-merge (squash) July 7, 2021 15:50
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
core 365 368 +3

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
core 1056 1071 +15

Public APIs missing exports

Total count of every type that is part of your API that should be exported but is not. This will cause broken links in the API documentation system. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats exports for more detailed information.

id before after diff
core 30 31 +1

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
core 420.5KB 421.9KB +1.4KB
Unknown metric groups

API count

id before after diff
core 2296 2327 +31

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

@mshustov mshustov merged commit 73adcb4 into elastic:7.x Jul 7, 2021
@mshustov mshustov deleted the backport/7.x/pr-102039 branch July 8, 2021 07:39
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