-
Notifications
You must be signed in to change notification settings - Fork 24.9k
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
Add "synthetics-*-*" templates for synthetics fleet data #62193
Conversation
For the Elastic Agent we currently have `logs` and `metrics`, however, synthetic data doesn't belong with those and thus we should have a place for it to live. This would be data reported from heartbeat and under the 'monitoring' category. This commit adds a composable index template for `synthetics-*-*` indices similar to the work in elastic#56709 and elastic#57629. Resolves elastic#61665
Pinging @elastic/es-core-features (:Core/Features/Indices APIs) |
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, thanks Lee (it's only the docs tests that need some adjustments)
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.
Overall LGTM but I want some others to chime in too.
"hot": { | ||
"actions": { | ||
"rollover": { | ||
"max_size": "50gb", |
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.
@andrewvc I assume this also works for synthetics for 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.
TBH we haven't done enough benchmarking to know, but it seems reasonable
"priority": 100, | ||
"data_stream": {}, | ||
"composed_of": [ | ||
"synthetics-mappings", |
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.
@jpountz I remember you wanted to "not" use components here. Perhaps it is a good time to adjust?
@ph @neptunian As soon as this lands, we also probably need to add a fake doc for the index pattern in Kibana. |
There is no "message" field for this to be the default for
@ruflin if we expect to create an index pattern, like |
@elasticmachine update branch |
@elasticmachine run elasticsearch-ci/2 |
* Add "synthetics-*-*" templates for synthetics fleet data For the Elastic Agent we currently have `logs` and `metrics`, however, synthetic data doesn't belong with those and thus we should have a place for it to live. This would be data reported from heartbeat and under the 'monitoring' category. This commit adds a composable index template for `synthetics-*-*` indices similar to the work in elastic#56709 and elastic#57629. Resolves elastic#61665
… (#62346) * Add "synthetics-*-*" templates for synthetics fleet data For the Elastic Agent we currently have `logs` and `metrics`, however, synthetic data doesn't belong with those and thus we should have a place for it to live. This would be data reported from heartbeat and under the 'monitoring' category. This commit adds a composable index template for `synthetics-*-*` indices similar to the work in #56709 and #57629. Resolves #61665
@neptunian SGTM |
We need to ensure these are mapped as 'ip' instead of a keyword, even if they do end up not being used. Relates to elastic#62193
…astic#62412) We need to ensure these are mapped as 'ip' instead of a keyword, even if they do end up not being used. Relates to elastic#62193
For the Elastic Agent we currently have
logs
andmetrics
, however, synthetic data doesn't belongwith those and thus we should have a place for it to live. This would be data reported from
heartbeat and under the 'monitoring' category.
This commit adds a composable index template for
synthetics-*-*
indices similar to the work in#56709 and #57629.
Resolves #61665