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

[8.x] Consume elasticsearch.publicBaseUrl where possible (#192741) #193442

Merged
merged 2 commits into from
Sep 24, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

## Summary

This actually consumes the public base url in the cloud plugin and the
places depending on the `elasticsearchUrl` value populated there.

---------

Co-authored-by: Rodney Norris <rodney@tattdcodemonkey.com>
(cherry picked from commit b4a7b2e)
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #6 / When on the trusted applications page should search using expected exception item fields

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
enterpriseSearch 2.6MB 2.6MB +448.0B
fleet 1.8MB 1.8MB +122.0B
home 151.9KB 152.0KB +53.0B
indexManagement 679.7KB 679.7KB +45.0B
searchHomepage 54.8KB 54.8KB +53.0B
searchIndices 84.1KB 84.3KB +196.0B
searchPlayground 151.1KB 151.2KB +84.0B
serverlessSearch 326.9KB 327.2KB +368.0B
total +1.3KB

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
cloud 0 1 +1

Page load bundle

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

id before after diff
cloud 7.1KB 7.6KB +464.0B
cloudLinks 30.1KB 30.2KB +53.0B
total +517.0B
Unknown metric groups

API count

id before after diff
@kbn/core-elasticsearch-server 117 118 +1

History

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

cc @sphilipse

@kibanamachine kibanamachine merged commit d57df63 into elastic:8.x Sep 24, 2024
28 checks passed
sphilipse referenced this pull request Sep 24, 2024
…193838)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Consume elasticsearch.publicBaseUrl where possible
(#192741)](#192741)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Sander
Philipse","email":"94373878+sphilipse@users.noreply.github.com"},"sourceCommit":{"committedDate":"2024-09-19T14:57:47Z","message":"Consume
elasticsearch.publicBaseUrl where possible (#192741)\n\n##
Summary\r\n\r\nThis actually consumes the public base url in the cloud
plugin and the\r\nplaces depending on the `elasticsearchUrl` value
populated there.\r\n\r\n---------\r\n\r\nCo-authored-by: Rodney Norris
<rodney@tattdcodemonkey.com>","sha":"b4a7b2e2164974e5dd9b8c1708c35d58bd021105","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Search","backport:prev-major","v8.16.0"],"number":192741,"url":"https://github.com/elastic/kibana/pull/192741","mergeCommit":{"message":"Consume
elasticsearch.publicBaseUrl where possible (#192741)\n\n##
Summary\r\n\r\nThis actually consumes the public base url in the cloud
plugin and the\r\nplaces depending on the `elasticsearchUrl` value
populated there.\r\n\r\n---------\r\n\r\nCo-authored-by: Rodney Norris
<rodney@tattdcodemonkey.com>","sha":"b4a7b2e2164974e5dd9b8c1708c35d58bd021105"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/192741","number":192741,"mergeCommit":{"message":"Consume
elasticsearch.publicBaseUrl where possible (#192741)\n\n##
Summary\r\n\r\nThis actually consumes the public base url in the cloud
plugin and the\r\nplaces depending on the `elasticsearchUrl` value
populated there.\r\n\r\n---------\r\n\r\nCo-authored-by: Rodney Norris
<rodney@tattdcodemonkey.com>","sha":"b4a7b2e2164974e5dd9b8c1708c35d58bd021105"}},{"branch":"8.x","label":"v8.16.0","labelRegex":"^v8.16.0$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/193442","number":193442,"state":"OPEN"}]}]
BACKPORT-->
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.

3 participants