-
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
[Inventory] Fix discover link #198690
[Inventory] Fix discover link #198690
Conversation
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
const discoverLink = discoverLocator?.getRedirectUrl({ | ||
indexPatternId: dataView?.id ?? '', | ||
columns: ACTIVE_COLUMNS, | ||
query: { query: kuery ?? '', language: 'kuery' }, | ||
query: { query: kueryWithEntityDefinitionFilters, language: 'kuery' }, |
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.
Note
I pass it as a kuery cause the filter doesn't support pattern linke *
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.
LGTM
💛 Build succeeded, but was flaky
Failed CI StepsMetrics [docs]Async chunks
History
|
Starting backport for target branches: 8.x |
## Summary closes elastic#198533 - filter `builtin*` - add simple e2e (cherry picked from commit bc80825)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
# Backport This will backport the following commits from `main` to `8.x`: - [[Inventory] Fix discover link (#198690)](#198690) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Katerina","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-01T20:21:11Z","message":"[Inventory] Fix discover link (#198690)\n\n## Summary\r\n\r\ncloses #198533 \r\n\r\n- filter `builtin*`\r\n- add simple e2e","sha":"bc80825cd89029282bfbdd17a916392d99663fa0","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","ci:project-deploy-observability","v8.17.0"],"title":"[Inventory] Fix discover link","number":198690,"url":"https://github.com/elastic/kibana/pull/198690","mergeCommit":{"message":"[Inventory] Fix discover link (#198690)\n\n## Summary\r\n\r\ncloses #198533 \r\n\r\n- filter `builtin*`\r\n- add simple e2e","sha":"bc80825cd89029282bfbdd17a916392d99663fa0"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/198690","number":198690,"mergeCommit":{"message":"[Inventory] Fix discover link (#198690)\n\n## Summary\r\n\r\ncloses #198533 \r\n\r\n- filter `builtin*`\r\n- add simple e2e","sha":"bc80825cd89029282bfbdd17a916392d99663fa0"}},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Katerina <[email protected]>
Summary
closes #198533
builtin*