Skip to content

[7.17] Test config settings that are exposed to the browser (#129438) - #129857

Merged
jportner merged 4 commits into
elastic:7.17from
jportner:backport/7.17/pr-129438
Apr 11, 2022
Merged

jportner merged 4 commits into
elastic:7.17from
jportner:backport/7.17/pr-129438

Conversation

@jportner

@jportner jportner commented Apr 9, 2022

Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 7.17:

Questions ?

Please refer to the Backport tool documentation

(cherry picked from commit 27ff7d3)

# Conflicts:
#	.github/CODEOWNERS
#	packages/kbn-config-schema/src/index.ts
@jportner jportner added the backport This PR is a backport of another PR label Apr 9, 2022
@jportner
jportner enabled auto-merge (squash) April 9, 2022 15:00
Joe Portner added 3 commits April 11, 2022 10:22
This is a partial backport of #111681, so the Kibana security
plugin is enabled but Elasticsearch security is still disabled.
The exposed config keys are slightly different in the 7.17 branch.
The enterpriseSearch plugin does not have a required dependency on
the security plugin in the 7.17 branch, so our bacported
assertions for these tests needed to change accordingly.
@jportner
jportner merged commit 6ba02ec into elastic:7.17 Apr 11, 2022
@kibana-ci

Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

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
@kbn/config-schema 107 111 +4
Unknown metric groups

API count

id before after diff
@kbn/config-schema 109 113 +4

History

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

@jportner
jportner deleted the backport/7.17/pr-129438 branch April 11, 2022 17:47
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