-
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
[Stack Monitoring] support entsearch package #148668
[Stack Monitoring] support entsearch package #148668
Conversation
Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI) |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: cc @klacabane |
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! I've tested this PR while testing elastic/integrations#4926
@@ -190,7 +190,7 @@ export const BEATS_SYSTEM_ID = 'beats'; | |||
* The name of the Enterprise Search System ID used to publish and look up Enterprise Search stats through the Monitoring system. | |||
* @type {string} | |||
*/ | |||
export const ENTERPRISE_SEARCH_SYSTEM_ID = 'enterprise_search'; | |||
export const ENTERPRISE_SEARCH_SYSTEM_ID = 'enterprisesearch'; |
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.
sweet!
@@ -47,7 +47,7 @@ export const errorsQuery = ({ | |||
}, | |||
{ | |||
range: { | |||
timestamp: { | |||
'@timestamp': { |
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.
did timestamp
ever work?
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.
It works except for enterprisesearch
(added in this change) which does not have a timestamp
alias. Since every data streams/metricbeat module exposes the @timestamp
ecs field it made sense to update
### Summary Update stack monitoring to read from data streams created by the [enterprisesearch package](elastic/integrations#4926) ### Testing - build [enterprisesearch package](elastic/integrations#4926) - start stack: `elastic-package stack up -v -d --version 8.7.0-SNAPSHOT`; make sure your image contains [this change](elastic/elastic-agent#2121) - start enterprisesearch service: `elastic-package service up -v` - install elasticsearch integration with variables - `hosts: http://host.docker.internal:9201; username: elastic; password: changeme` - install enterprisesearch integration with variables - `hosts: http://host.docker.internal:3002; username: elastic; password: changeme` - connect [local kibana](https://github.com/elastic/kibana/blob/2d893bf40b47864058065f8fd2998d63c87f65f9/x-pack/plugins/monitoring/dev_docs/how_to/work_with_packages.md#connecting-a-local-kibana) - navigate to Stack Monitoring on the local kibana, enterprise search section shows up with populated graphs --------- Co-authored-by: Kibana Machine <[email protected]>
Summary
Update stack monitoring to read from data streams created by the enterprisesearch package
Testing
elastic-package stack up -v -d --version 8.7.0-SNAPSHOT
; make sure your image contains this changeelastic-package service up -v
hosts: http://host.docker.internal:9201; username: elastic; password: changeme
hosts: http://host.docker.internal:3002; username: elastic; password: changeme