[Alert details page] Fix metric threshold chart time range for bigger lookback windows

[maryam-saeidi]

Fixes #183625
Fixes #183130

Summary

This PR fixes #183130 by removing Last 1 minute subtitle in the alert details pages of the metric and custom threshold rules.

Before	After

Also, it adjusts the chart time range by extending it at least 20 times the rule execution interval to avoid showing no data for bigger intervals such as 1 hour or 1 day.

[elasticmachine]

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

[apmmachine]

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

• /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
• run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

[maryam-saeidi]

In this section, I only added timeRange and removed FormattedMessage related to Last {lookback} {timeLabel}.

[fkanout]

@maryam-saeidi, I have created a rule with 3 hours lookback window. And after 7 minutes of the trigger time, still can't see the alert annotation.

[fkanout]

I have tested many lookback windows with different values and the bucket size in chart was always matching the rule lookback time. 💪🏻 !

[maryam-saeidi]

@maryam-saeidi, I have created a rule with 3 hours lookback window. And after 7 minutes of the trigger time, still can't see the alert annotation.

@fkanout Good catch! It seems it happens due to the gap between last aggregated data and alert annotation:

Aggregated data before alert	Alert annotation

Would you mind creating a ticket for it?
Update: create a separate ticket for it (#184574)

[kibana-ci]

💚 Build Succeeded

• Buildkite Build
• Commit: 20473df
• Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-184409-20473dfa8631
• Observability Deployment

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
observability	491	490	-1

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id	before	after	diff
@kbn/observability-get-padded-alert-time-range-util	3	6	+3

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id	before	after	diff
apm	3.3MB	3.3MB	+70.0B
infra	1.5MB	1.5MB	-79.0B
observability	309.5KB	308.6KB	-953.0B
total			-962.0B

Canvas Sharable Runtime

The Canvas "shareable runtime" is an bundle produced to enable running Canvas workpads outside of Kibana. This bundle is included in third-party webpages that embed canvas and therefor should be as slim as possible.

id	before	after	diff
module count	-	5499	+5499
total size	-	8.9MB	+8.9MB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id	before	after	diff
observability	122.6KB	122.7KB	+70.0B

Unknown metric groups

API count

id	before	after	diff
@kbn/observability-get-padded-alert-time-range-util	3	6	+3

History

• 💚 Build #213277 succeeded 8177bcb
• 💚 Build #212760 succeeded e20bca4

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @maryam-saeidi

[fkanout]

The issue I mentioned -missing alert annotation is already exists and not related to the PR.
So I approve this one and fix the other issue in a separate one. Thanks @maryam-saeidi for creating the issue.