Skip to content

[8.x] [otel-data] Hide 10m and 60m aggregated metrics generated for the APM UI (#114042) #114173

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

Merged

Conversation

gregkalapos
Copy link
Contributor

Backports the following commits to 8.x:

… UI (elastic#114042)

* Hide aggregated metrics generated for the APM UI

* Update 30_aggregated_metrics_tests.yml

* Review feedback

- Introduced templates for 1 minute aggregations
- Moved dynamic templates `ecs_ip` and `all_strings_to_keywords` into a dedicated file and now I pull the file in instead of repeating them
- Introduced `metrics-[x]m.otel@custom`
- Added tests with terms aggregation that assert by default 1 bucket (only 1m) with metricsset.interval, and with allowing hidden indices it's 3 buckets (1m, 10m, 60m)

* Update 30_aggregated_metrics_tests.yml

Simplify - no need to separate tests for the 3bucket queries.

* Rename metrics-otel-fixed@mappings to ecs-tsdb@mappings

---------

Co-authored-by: Elastic Machine <[email protected]>
@gregkalapos gregkalapos added :Data Management/Data streams Data streams and their lifecycles >non-issue auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Data Management Meta label for data/management team labels Oct 4, 2024
@gregkalapos
Copy link
Contributor Author

@elasticmachine test this please

1 similar comment
@gregkalapos
Copy link
Contributor Author

@elasticmachine test this please

@elasticsearchmachine elasticsearchmachine merged commit 19e625e into elastic:8.x Oct 7, 2024
15 checks passed
@gregkalapos gregkalapos deleted the backport/8.x/pr-114042 branch October 7, 2024 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Data Management/Data streams Data streams and their lifecycles external-contributor Pull request authored by a developer outside the Elasticsearch team >non-issue Team:Data Management Meta label for data/management team v8.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants