-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[EBT] fix browser-side elasticsearch context schema #172676
[EBT] fix browser-side elasticsearch context schema #172676
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, approving to unblock progress.
Any chance we can add or update a test in packages/core/analytics/core-analytics-browser-internal/src/analytics_service.test.ts
?
💚 Build Succeeded
Metrics [docs]Page load bundle
To update your PR or re-run it, just comment with: |
Pinging @elastic/kibana-core (Team:Core) |
## Summary The `cluster_build_flavor` was propagated to the browser-side and then used in the context, but without being explicitly defined in the schema. this PR fixes it by properly adding the field. (cherry picked from commit 754c298)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…172787) # Backport This will backport the following commits from `main` to `8.12`: - [[EBT] fix browser-side elasticsearch context schema (#172676)](#172676) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Pierre Gayvallet","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-12-07T09:08:45Z","message":"[EBT] fix browser-side elasticsearch context schema (#172676)\n\n## Summary\r\n\r\nThe `cluster_build_flavor` was propagated to the browser-side and then\r\nused in the context, but without being explicitly defined in the schema.\r\n\r\nthis PR fixes it by properly adding the field.","sha":"754c298d12f9c51334a35d0c8b0681da0126276e","branchLabelMapping":{"^v8.13.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Core","Feature:Telemetry","release_note:skip","v8.12.0","v8.13.0"],"number":172676,"url":"https://github.com/elastic/kibana/pull/172676","mergeCommit":{"message":"[EBT] fix browser-side elasticsearch context schema (#172676)\n\n## Summary\r\n\r\nThe `cluster_build_flavor` was propagated to the browser-side and then\r\nused in the context, but without being explicitly defined in the schema.\r\n\r\nthis PR fixes it by properly adding the field.","sha":"754c298d12f9c51334a35d0c8b0681da0126276e"}},"sourceBranch":"main","suggestedTargetBranches":["8.12"],"targetPullRequestStates":[{"branch":"8.12","label":"v8.12.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.13.0","labelRegex":"^v8.13.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/172676","number":172676,"mergeCommit":{"message":"[EBT] fix browser-side elasticsearch context schema (#172676)\n\n## Summary\r\n\r\nThe `cluster_build_flavor` was propagated to the browser-side and then\r\nused in the context, but without being explicitly defined in the schema.\r\n\r\nthis PR fixes it by properly adding the field.","sha":"754c298d12f9c51334a35d0c8b0681da0126276e"}}]}] BACKPORT--> Co-authored-by: Pierre Gayvallet <[email protected]>
Related to #171112, which fixes it on the server-side. |
Summary
The
cluster_build_flavor
was propagated to the browser-side and then used in the context, but without being explicitly defined in the schema.this PR fixes it by properly adding the field.