-
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
[Logs Overview] Add a flyout to show category document examples #194867
Merged
Kerry350
merged 100 commits into
elastic:main
from
Kerry350:193450-show-category-docs-in-flyout
Oct 24, 2024
Merged
Changes from 90 commits
Commits
Show all changes
100 commits
Select commit
Hold shift + click to select a range
b3a1af8
Add rare logs scenario
weltenwort 0a43169
Add empty logs_overview package
weltenwort d373e59
Add gaussian distribution to the unstructured logs scenario
weltenwort 278365e
Add support for xstate5
weltenwort 1f4bd0f
Add xstate5 console inspector
weltenwort 76ea2a5
Add cateogorization service
weltenwort 0528e8b
Add categories grid
weltenwort 0b7fc2c
Enable translation linting for the package
weltenwort b428bd4
Improve typing of the sorting config
weltenwort 5be73af
Improve default column widths
weltenwort e0af1bb
Add loading screen with cancellation
weltenwort c70a37a
Merge branch 'main' into logs-enhanced-embeddable-iteration-1
weltenwort 169df23
Bump XState 5 version
weltenwort 901bee3
Start cleaning up names and introducing the overview component
weltenwort 1023d69
Provide a self-contained overview via logs_shared
weltenwort 0d35529
Fix unnecessary uiSettings dependency
weltenwort d551da4
Fix some refactoring mistakes
weltenwort 47153c8
Memoize time overview props
weltenwort dc8ea70
Update zod import
weltenwort 3793877
Add service filters and error reporting
weltenwort c69a83b
Fix fetching remaining categories
weltenwort 870d347
Add logs overview feature flag
weltenwort d08f104
Add logs overview in infra hosts
weltenwort 6ecd056
Fix sampling probability of second pass
weltenwort b0b1fe7
Signal categories limit being reached
weltenwort 8c09788
Respect the abort signal when categorizing
weltenwort b028eab
Add empty state screen
weltenwort 6a1ed55
Merge branch 'main' of github.com:elastic/kibana into logs-enhanced-e…
weltenwort 463355c
Merge branch 'main' of github.com:elastic/kibana into logs-enhanced-e…
weltenwort 6aad1a1
Fix race condition when rendering apm service details
weltenwort 7f1fb14
Improve loading content in hosts
weltenwort 4e95f04
Update XState v5 version
weltenwort ea9dfb2
Merge branch 'main' of github.com:elastic/kibana into logs-enhanced-e…
weltenwort a6ab7e1
Clean up TODOs and comments
weltenwort 5bbda50
Remove temporary test
weltenwort 7843215
[CI] Auto-commit changed files from 'node scripts/notice'
kibanamachine e3f3043
Update license headers
weltenwort 3402293
Remove unused import
weltenwort 542d1f1
Revert term separators back to asterisks
weltenwort eb089a6
Add a logs_overview mock
weltenwort 78b0d4c
Fix telemetry schema
weltenwort 76b9771
Merge branch 'main' of github.com:elastic/kibana into logs-enhanced-e…
weltenwort 2d414c5
Merge branch 'main' of github.com:elastic/kibana into logs-enhanced-e…
weltenwort 14be0af
Merge branch 'main' of github.com:elastic/kibana into logs-enhanced-e…
weltenwort c1962c0
Add link to open all log entries in discover
weltenwort 90282cc
Fix dataview spec passed to locator
weltenwort 6e77e63
Merge branch 'main' of github.com:elastic/kibana into logs-enhanced-e…
weltenwort 1a90cb8
[CI] Auto-commit changed files from 'node scripts/notice'
kibanamachine 70adf04
Merge remote-tracking branch 'upstream/main' into logs-enhanced-embed…
Kerry350 b522494
Merge remote-tracking branch 'upstream/main' into logs-enhanced-embed…
Kerry350 598d9a9
package.json changes from yarn kbn bootstrap
Kerry350 a34ab22
Fix two types
Kerry350 eb3f98c
Merge branch 'main' into logs-enhanced-embeddable-iteration-1
elasticmachine 4940642
Merge remote-tracking branch 'upstream/main' into logs-enhanced-embed…
weltenwort 46f8e9e
Merge branch 'main' into logs-enhanced-embeddable-iteration-1
elasticmachine c6d26db
Use timeField parameter instead of hard-coded field name
weltenwort 7e34bc0
Add a flyout to show category document examples
Kerry350 1efa83b
Merge remote-tracking branch 'upstream/main' into 193450-show-categor…
Kerry350 246b0f7
[CI] Auto-commit changed files from 'node scripts/notice'
kibanamachine 6a5087e
[CI] Auto-commit changed files from 'node scripts/lint_ts_projects --…
kibanamachine 0b08c2c
Remove line from merge conflict
Kerry350 965a477
Merge remote-tracking branch 'upstream/main' into 193450-show-categor…
Kerry350 c290819
Extract out columns / cells to new package
Kerry350 4e55806
[CI] Auto-commit changed files from 'node scripts/generate codeowners'
kibanamachine 8eaaec7
Merge remote-tracking branch 'upstream/main' into 193450-show-categor…
Kerry350 8c8b2a3
Update i18nrc.json
Kerry350 6e60316
More descriptive filter labels
Kerry350 b956a49
Auto-sort imports
Kerry350 d827ac7
Better grouped naming
Kerry350 d31a0f5
Move control columns to separate file
Kerry350 00584d2
Move label
Kerry350 f7cc019
Update x-pack/packages/observability/logs_overview/src/components/log…
Kerry350 4537b8a
Remove spacer
Kerry350 0d12e4d
Inverse dependencies
Kerry350 8de3655
Remove rowHeader as we don't use it
Kerry350 bf14132
Last > latest wording
Kerry350 1fe0ac0
Update packages/kbn-discover-contextual-components/kibana.jsonc
Kerry350 8312715
Merge branch '193450-show-category-docs-in-flyout' of github.com:Kerr…
Kerry350 88e38cc
Move query into filter
Kerry350 537b052
[CI] Auto-commit changed files from 'node scripts/lint_packages --fix'
kibanamachine 601d1f9
Amend config for tests in new package
Kerry350 81e694f
Merge branch '193450-show-category-docs-in-flyout' of github.com:Kerr…
Kerry350 1be56ac
Merge remote-tracking branch 'upstream/main' into 193450-show-categor…
Kerry350 0d8959c
Add a spacer in for some breathing room
Kerry350 83d54fa
Merge branch 'main' into 193450-show-category-docs-in-flyout
Kerry350 73324f2
Merge branch 'main' into 193450-show-category-docs-in-flyout
Kerry350 b4cdf1f
Merge branch 'main' into 193450-show-category-docs-in-flyout
Kerry350 679dd9c
Merge remote-tracking branch 'upstream/main' into 193450-show-categor…
Kerry350 34d9f97
PR feedback
Kerry350 76ead24
[CI] Auto-commit changed files from 'node scripts/notice'
kibanamachine 7b2545e
Don't prebind deps
Kerry350 87869d9
Amend constants imports / exports
Kerry350 0174610
i18n --fix
Kerry350 0b6e2d6
Merge branch '193450-show-category-docs-in-flyout' of github.com:Kerr…
Kerry350 78064e1
[CI] Auto-commit changed files from 'node scripts/notice'
kibanamachine b0a800f
Merge branch 'main' into 193450-show-category-docs-in-flyout
Kerry350 e228f15
Merge remote-tracking branch 'upstream/main' into 193450-show-categor…
Kerry350 4dc459d
PR changes
Kerry350 88bbf99
Amendments from upstream changes
Kerry350 336b0a0
[CI] Auto-commit changed files from 'node scripts/yarn_deduplicate'
kibanamachine File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Validating CODEOWNERS rules …
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
# @kbn/discover-contextual-components | ||
|
||
Houses contextual (e.g. logs) components that are used by Discover. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
14 changes: 14 additions & 0 deletions
14
packages/kbn-discover-contextual-components/jest.config.js
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the "Elastic License | ||
* 2.0", the "GNU Affero General Public License v3.0 only", and the "Server Side | ||
* Public License v 1"; you may not use this file except in compliance with, at | ||
* your election, the "Elastic License 2.0", the "GNU Affero General Public | ||
* License v3.0 only", or the "Server Side Public License, v 1". | ||
*/ | ||
|
||
module.exports = { | ||
preset: '@kbn/test', | ||
rootDir: '../..', | ||
roots: ['<rootDir>/packages/kbn-discover-contextual-components'], | ||
}; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
{ | ||
"type": "shared-browser", | ||
"id": "@kbn/discover-contextual-components", | ||
"owner": ["@elastic/obs-ux-logs-team", "@elastic/kibana-data-discovery"] | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
{ | ||
"name": "@kbn/discover-contextual-components", | ||
"private": true, | ||
"version": "1.0.0", | ||
"license": "Elastic License 2.0 OR AGPL-3.0-only OR SSPL-1.0", | ||
"sideEffects": false | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
11 changes: 11 additions & 0 deletions
11
packages/kbn-discover-contextual-components/src/data_types/logs/components/index.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the "Elastic License | ||
* 2.0", the "GNU Affero General Public License v3.0 only", and the "Server Side | ||
* Public License v 1"; you may not use this file except in compliance with, at | ||
* your election, the "Elastic License 2.0", the "GNU Affero General Public | ||
* License v3.0 only", or the "Server Side Public License, v 1". | ||
*/ | ||
|
||
export * from './summary_column'; | ||
export * from './log_level_badge_cell/log_level_badge_cell'; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
13 changes: 13 additions & 0 deletions
13
...kbn-discover-contextual-components/src/data_types/logs/components/summary_column/index.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the "Elastic License | ||
* 2.0", the "GNU Affero General Public License v3.0 only", and the "Server Side | ||
* Public License v 1"; you may not use this file except in compliance with, at | ||
* your election, the "Elastic License 2.0", the "GNU Affero General Public | ||
* License v3.0 only", or the "Server Side Public License, v 1". | ||
*/ | ||
|
||
export * from './content'; | ||
export * from './resource'; | ||
export * from './summary_column'; | ||
export * from './utils'; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Maybe my confusion about binding the dependencies early below is driven by the fact that I can't seem to figure out why we need those dependencies to be added here and all upstream usage sites. I'm surely missing something obvious, am I not?
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.
We need
share
andcore
for theServiceNameBadgeWithActions
component. Previously this was accessed via context directly (as it lived in Discover). I changed this to accept the dependencies as props as everything was moved to the new package and we can no longer rely on that context.The
SummaryCellPopover
ultimately uses this for the component map creation from this line now.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.
Ah, thanks for helping me understand that. ❤️