-
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
[Observability] New "No Data" screens #107709
Merged
Merged
Changes from 61 commits
Commits
Show all changes
102 commits
Select commit
Hold shift + click to select a range
202fba7
Initial commit of new NoDataPage and accompanying cards
e6c1290
Adding to Security Solution and forcing view in TemplateWrapper
0ac66b8
Center page
5274c96
Better customizations & sorting on `recommended`
66445f8
Adding custom size to Solution Nav Avatar for reuse and working on pa…
495e905
Put SetupGuideCTA back
4600e9a
Fix TS
80e2063
Added empty state to Infra/Logs
d7c2e29
Added to APM
4563cad
Added to Metrics template
b774ecb
Added to Uptime and fixed layouts
b2ce1c3
Fixing UX layout
5f98d10
Merge remote-tracking branch 'upstream/master' into design/add_data_s…
ab8c4bd
Faked in for UX
c981540
Removing outer template from no data page
549ee0a
Optimizing some prop usages
8c0acc4
Making `noDataConfig` a view in KibanaPageTemplate
67c7b6a
Working on links
6e4e53b
Obs Getting Started page (without solution nav)
7850037
Cleanup
a0128dd
Fix i18n
0f3b96b
Dev docs
b3152c6
Only one link for solutions
snide 23ad9ac
Fix double templates in Uptime
6fe3a39
Move styles to page contents Sass file and only add min width when th…
a5defa0
cleanup
3e6d656
Adding tests to template.
31676e3
Removing a few hard-coded checks to test tests
fbf3267
Actually fix tests for cards
20b0e4e
i18n
d3fdc9e
Merge branch 'master' into design/add_data_screens
cchaos c13cfe3
Merge branch 'elastic:master' into design/add_data_screens
cchaos 76da749
Merge remote-tracking branch 'upstream/master' into design/add_data_s…
64661d7
Rebase fixes
7e3e60a
Mroe rebase fixes, cleanup, and links
57a66d6
Merge branch 'master' into design/add_data_screens
cchaos 81a444b
Fix ES test
da83c6c
Skipping security solution test for now
82d2f40
Hopefully fixing some tests
9bc06ae
Merge remote-tracking branch 'upstream/master' into design/add_data_s…
82e5649
Revert "Skipping security solution test for now"
3be7b0d
Revert "Adding to Security Solution and forcing view in TemplateWrapper"
255acbe
More Security Solution reverts
f957968
Merge branch 'master' into design/add_data_screens
cchaos 33f922f
Update beats card default text to just `Add data`
ab018d7
Updated RUM copy
54d0d49
Updating Uptime copy
5c852fb
Update APM copy
6e9c8ce
Updating Logs copy and reverting changes to `page_no_indices_content`…
25e711a
Updated copy for Metrics
4f0613c
Updated copy for landing and overview pages
7ef38f3
[ElasticBeatsCard] Reuse `title` as default button label
2e9346f
Better SS revert
d041c81
`agent` not `Agent`
bd8d2b1
Merge branch 'master' into design/add_data_screens
cchaos 7cd45fc
Merge branch 'master' into design/add_data_screens
cchaos 683c1f1
logic to show no data screen
cauemarcondes 26fcbbf
Merge pull request #31 from cauemarcondes/apm-new-no-data-screen
cchaos a9f9989
Merge remote-tracking branch upstream/master into design/add_data_scr…
a72d42e
i18n
4f233c1
Merge remote-tracking branch 'upstream/master' into design/add_data_s…
30f86a8
Merge branch 'master' into design/add_data_screens
cchaos 5f39a8f
Fixing bad rebase in APM template
cchaos b92fca9
Delete historical_data.ts
cchaos 33dc8dc
Fix from rebase to Overview page
cchaos 1276d50
Merge branch 'master' into design/add_data_screens
cchaos 604f590
Fix RumHome from rebase
cchaos fdd84aa
Use KibanaPageTemplateProps
cchaos f6cc266
Merge branch 'master' into design/add_data_screens
cchaos 30b7d6f
Add `hasData` prop to `LogsPageTemplate`
edf7e97
Check for data in log pages
8921313
Add `hasData` prop to `MetricsPageTemplate`
89c16eb
Check for data in metrics pages
e4a5a23
Merge branch 'master' into design/add_data_screens
cchaos 6c7d151
Fix bad rebase to routes
cchaos 1493eab
Add test subject to no data screen
bbfc4e5
Fix log entry categories tab test
fa13307
Fix infrastructure_security test
9d6e9af
Remove hard-coded DTS on NoDataPage and ensure one is passed through …
0372ce0
Fix lint error
9aad198
Merge branch 'master' into design/add_data_screens
kibanamachine cc4ae52
Remove DTS test 🤦♀️
bc2b3f4
enable no data view on tls certificate route
shahzad31 45b0070
Fix uptime page layout
d5fc41f
Remove unnecessary check for the metrics settings page
6c5823e
Merge branch 'master' into design/add_data_screens
kibanamachine b6f42aa
Merge branch 'master' into design/add_data_screens
kibanamachine 847fec7
Fix metrics no indices test
d2fbe0f
Fix more tests
e976f05
Fix logEntryRate test
a6507b2
Fix spaces tests
f0142d9
Merge branch 'master' of github.com:elastic/kibana into design/add_da…
shahzad31 6923af0
Merge branch 'master' into design/add_data_screens
kibanamachine 94cc373
update timeout
shahzad31 83b50e0
Fix even more tests
1a9bf58
update test
shahzad31 7ac8c56
Fix test for new index patterns
87d5be8
Merge branch 'master' into design/add_data_screens
cchaos 9461716
Merge branch 'master' into design/add_data_screens
kibanamachine b3164c5
Merge branch 'master' into design/add_data_screens
kibanamachine 69abe1d
Merge branch 'master' into design/add_data_screens
kibanamachine 0707c30
remove uptime flaky test
shahzad31 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
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,25 @@ | ||
/* | ||
* 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; you may not use this file except in compliance with the Elastic License | ||
* 2.0. | ||
*/ | ||
|
||
import { setupRequest } from '../lib/helpers/setup_request'; | ||
import { hasHistoricalAgentData } from '../lib/services/get_services/has_historical_agent_data'; | ||
import { createApmServerRoute } from './create_apm_server_route'; | ||
import { createApmServerRouteRepository } from './create_apm_server_route_repository'; | ||
|
||
const hasDataRoute = createApmServerRoute({ | ||
endpoint: 'GET /api/apm/has_data', | ||
options: { tags: ['access:apm'] }, | ||
handler: async (resources) => { | ||
const setup = await setupRequest(resources); | ||
const hasData = await hasHistoricalAgentData(setup); | ||
return { hasData }; | ||
}, | ||
}); | ||
|
||
export const historicalDataRouteRepository = createApmServerRouteRepository().add( | ||
hasDataRoute | ||
); |
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
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.
are you still planing to make this PR work? if yes i can help with this metric check,
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.
@shahzad31 Yes, this PR is slated for 7.16 and anywhere there are "TODO" or "HELP" comments, are parts that I need Obs folks to jump in on. So all the help is gratefully needed!
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.
Added a PR for UX app has no data screen #111866