-
Notifications
You must be signed in to change notification settings - Fork 25.2k
[otel-data] Hide 10m and 60m aggregated metrics generated for the APM UI #114042
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
Changes from 6 commits
9baf9a5
e31588d
4f5b20a
5c2c8c4
32014f1
25bc984
3fda420
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
version: ${xpack.oteldata.template.version} | ||
_meta: | ||
description: | | ||
Default mappings that can't be changed by users for | ||
the OpenTelemetry metrics index template installed by x-pack | ||
managed: true | ||
template: | ||
mappings: | ||
dynamic_templates: | ||
- ecs_ip: | ||
mapping: | ||
type: ip | ||
path_match: [ "ip", "*.ip", "*_ip" ] | ||
match_mapping_type: string | ||
- all_strings_to_keywords: | ||
mapping: | ||
ignore_above: 1024 | ||
type: keyword | ||
match_mapping_type: string |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,40 @@ | ||
--- | ||
version: ${xpack.oteldata.template.version} | ||
index_patterns: ["metrics-service_destination.10m.otel-*"] | ||
priority: 130 | ||
data_stream: | ||
hidden: true | ||
allow_auto_create: true | ||
_meta: | ||
description: aggregated APM metrics template installed by x-pack | ||
managed: true | ||
composed_of: | ||
- metrics@tsdb-settings | ||
- otel@mappings | ||
- metrics-otel@mappings | ||
- semconv-resource-to-ecs@mappings | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-10m.otel@custom | ||
- metrics-otel-fixed@mappings | ||
ignore_missing_component_templates: | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-10m.otel@custom | ||
template: | ||
settings: | ||
index: | ||
mode: time_series | ||
mappings: | ||
properties: | ||
data_stream.type: | ||
type: constant_keyword | ||
value: metrics | ||
metricset: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Originally I wanted to define But as it seems, this works as well - this is also covered by tests. Just calling out to make sure there is no unwanted implication I may missed. |
||
properties: | ||
interval: | ||
type: constant_keyword | ||
value: 10m | ||
name: | ||
type: constant_keyword | ||
value: service_destination | ||
felixbarny marked this conversation as resolved.
Show resolved
Hide resolved
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,39 @@ | ||
--- | ||
version: ${xpack.oteldata.template.version} | ||
index_patterns: ["metrics-service_destination.1m.otel-*"] | ||
priority: 130 | ||
data_stream: {} | ||
allow_auto_create: true | ||
_meta: | ||
description: aggregated APM metrics template installed by x-pack | ||
managed: true | ||
composed_of: | ||
- metrics@tsdb-settings | ||
- otel@mappings | ||
- metrics-otel@mappings | ||
- semconv-resource-to-ecs@mappings | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-1m.otel@custom | ||
- metrics-otel-fixed@mappings | ||
ignore_missing_component_templates: | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-1m.otel@custom | ||
template: | ||
settings: | ||
index: | ||
mode: time_series | ||
mappings: | ||
properties: | ||
data_stream.type: | ||
type: constant_keyword | ||
value: metrics | ||
metricset: | ||
properties: | ||
interval: | ||
type: constant_keyword | ||
value: 1m | ||
name: | ||
type: constant_keyword | ||
value: service_destination |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,40 @@ | ||
--- | ||
version: ${xpack.oteldata.template.version} | ||
index_patterns: ["metrics-service_destination.60m.otel-*"] | ||
priority: 130 | ||
data_stream: | ||
hidden: true | ||
allow_auto_create: true | ||
_meta: | ||
description: aggregated APM metrics template installed by x-pack | ||
managed: true | ||
composed_of: | ||
- metrics@tsdb-settings | ||
- otel@mappings | ||
- metrics-otel@mappings | ||
- semconv-resource-to-ecs@mappings | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-60m.otel@custom | ||
- metrics-otel-fixed@mappings | ||
ignore_missing_component_templates: | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-60m.otel@custom | ||
template: | ||
settings: | ||
index: | ||
mode: time_series | ||
mappings: | ||
properties: | ||
data_stream.type: | ||
type: constant_keyword | ||
value: metrics | ||
metricset: | ||
properties: | ||
interval: | ||
type: constant_keyword | ||
value: 60m | ||
name: | ||
type: constant_keyword | ||
value: service_destination |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,40 @@ | ||
--- | ||
version: ${xpack.oteldata.template.version} | ||
index_patterns: ["metrics-service_summary.10m.otel-*"] | ||
priority: 130 | ||
data_stream: | ||
hidden: true | ||
allow_auto_create: true | ||
_meta: | ||
description: aggregated APM metrics template installed by x-pack | ||
managed: true | ||
composed_of: | ||
- metrics@tsdb-settings | ||
- otel@mappings | ||
- metrics-otel@mappings | ||
- semconv-resource-to-ecs@mappings | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-10m.otel@custom | ||
- metrics-otel-fixed@mappings | ||
ignore_missing_component_templates: | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-10m.otel@custom | ||
template: | ||
settings: | ||
index: | ||
mode: time_series | ||
mappings: | ||
properties: | ||
data_stream.type: | ||
type: constant_keyword | ||
value: metrics | ||
metricset: | ||
properties: | ||
interval: | ||
type: constant_keyword | ||
value: 10m | ||
name: | ||
type: constant_keyword | ||
value: service_summary |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,39 @@ | ||
--- | ||
version: ${xpack.oteldata.template.version} | ||
index_patterns: ["metrics-service_summary.1m.otel-*"] | ||
priority: 130 | ||
data_stream: {} | ||
allow_auto_create: true | ||
_meta: | ||
description: aggregated APM metrics template installed by x-pack | ||
managed: true | ||
composed_of: | ||
- metrics@tsdb-settings | ||
- otel@mappings | ||
- metrics-otel@mappings | ||
- semconv-resource-to-ecs@mappings | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-1m.otel@custom | ||
- metrics-otel-fixed@mappings | ||
ignore_missing_component_templates: | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-1m.otel@custom | ||
template: | ||
settings: | ||
index: | ||
mode: time_series | ||
mappings: | ||
properties: | ||
data_stream.type: | ||
type: constant_keyword | ||
value: metrics | ||
metricset: | ||
properties: | ||
interval: | ||
type: constant_keyword | ||
value: 1m | ||
name: | ||
type: constant_keyword | ||
value: service_summary |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,40 @@ | ||
--- | ||
version: ${xpack.oteldata.template.version} | ||
index_patterns: ["metrics-service_summary.60m.otel-*"] | ||
priority: 130 | ||
data_stream: | ||
hidden: true | ||
allow_auto_create: true | ||
_meta: | ||
description: aggregated APM metrics template installed by x-pack | ||
managed: true | ||
composed_of: | ||
- metrics@tsdb-settings | ||
- otel@mappings | ||
- metrics-otel@mappings | ||
- semconv-resource-to-ecs@mappings | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-60m.otel@custom | ||
- metrics-otel-fixed@mappings | ||
ignore_missing_component_templates: | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-60m.otel@custom | ||
template: | ||
settings: | ||
index: | ||
mode: time_series | ||
mappings: | ||
properties: | ||
data_stream.type: | ||
type: constant_keyword | ||
value: metrics | ||
metricset: | ||
properties: | ||
interval: | ||
type: constant_keyword | ||
value: 60m | ||
name: | ||
type: constant_keyword | ||
value: service_summary |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,40 @@ | ||
--- | ||
version: ${xpack.oteldata.template.version} | ||
index_patterns: ["metrics-service_transaction.10m.otel-*"] | ||
priority: 130 | ||
data_stream: | ||
hidden: true | ||
allow_auto_create: true | ||
_meta: | ||
description: aggregated APM metrics template installed by x-pack | ||
managed: true | ||
composed_of: | ||
- metrics@tsdb-settings | ||
- otel@mappings | ||
- metrics-otel@mappings | ||
- semconv-resource-to-ecs@mappings | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-10m.otel@custom | ||
- metrics-otel-fixed@mappings | ||
ignore_missing_component_templates: | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-10m.otel@custom | ||
template: | ||
settings: | ||
index: | ||
mode: time_series | ||
mappings: | ||
properties: | ||
data_stream.type: | ||
type: constant_keyword | ||
value: metrics | ||
metricset: | ||
properties: | ||
interval: | ||
type: constant_keyword | ||
value: 10m | ||
name: | ||
type: constant_keyword | ||
value: service_transaction |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,39 @@ | ||
--- | ||
version: ${xpack.oteldata.template.version} | ||
index_patterns: ["metrics-service_transaction.1m.otel-*"] | ||
priority: 130 | ||
data_stream: {} | ||
allow_auto_create: true | ||
_meta: | ||
description: aggregated APM metrics template installed by x-pack | ||
managed: true | ||
composed_of: | ||
- metrics@tsdb-settings | ||
- otel@mappings | ||
- metrics-otel@mappings | ||
- semconv-resource-to-ecs@mappings | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-1m.otel@custom | ||
- metrics-otel-fixed@mappings | ||
ignore_missing_component_templates: | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-1m.otel@custom | ||
template: | ||
settings: | ||
index: | ||
mode: time_series | ||
mappings: | ||
properties: | ||
data_stream.type: | ||
type: constant_keyword | ||
value: metrics | ||
metricset: | ||
properties: | ||
interval: | ||
type: constant_keyword | ||
value: 1m | ||
name: | ||
type: constant_keyword | ||
value: service_transaction |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,40 @@ | ||
--- | ||
version: ${xpack.oteldata.template.version} | ||
index_patterns: ["metrics-service_transaction.60m.otel-*"] | ||
priority: 130 | ||
data_stream: | ||
hidden: true | ||
allow_auto_create: true | ||
_meta: | ||
description: aggregated APM metrics template installed by x-pack | ||
managed: true | ||
composed_of: | ||
- metrics@tsdb-settings | ||
- otel@mappings | ||
- metrics-otel@mappings | ||
- semconv-resource-to-ecs@mappings | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-60m.otel@custom | ||
- metrics-otel-fixed@mappings | ||
ignore_missing_component_templates: | ||
- metrics@custom | ||
- metrics-otel@custom | ||
- metrics-60m.otel@custom | ||
template: | ||
settings: | ||
index: | ||
mode: time_series | ||
mappings: | ||
properties: | ||
data_stream.type: | ||
type: constant_keyword | ||
value: metrics | ||
metricset: | ||
properties: | ||
interval: | ||
type: constant_keyword | ||
value: 60m | ||
name: | ||
type: constant_keyword | ||
value: service_transaction |
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's the reverse. Users should be able to customize the dynamic mappings. That's why this component template needs to be imported last. This is because for dynamic templates, the first one that matches wins. Therefore, users need to be able to put a dynamic template before this in order to be able to customize how attributes are mapped dynamically. We have a test for this scenario:
elasticsearch/x-pack/plugin/otel-data/src/yamlRestTest/resources/rest-api-spec/test/20_metrics_tests.yml
Lines 112 to 148 in 3b9150d
So I think all is correctly implemented, but we'll need to change the description, and probably the name of the dynamic template. Suggestion:
ecs-tsdb@mappings
(since the ip path match is coming from ECS).