Skip to content
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

[charts] Fix timezone regression in charts #123663

Merged

Conversation

markov00
Copy link
Member

Summary

We introduced a regression that causes time series charts without a timezone configured to be rendered with UTC by default instead of the local timezone.
We already have fixed the bug in charts here and the dependency is updated already in versions 7.17, 8.0.1, 8.1 but we are missing 8.0 since we are really close to the next RC.

This PR brings back the default behavior marking all time-series with no timezone parameter using the local timezone.

I will forward these changes to 7.17, 8.1, main in a different PR to land this PR sooner as it has a higher priority

Merging the same PR as #123478 fixing the conflict in the Uptime chart that caused the commit to be reverted

@markov00 markov00 added bug Fixes for quality problems that affect the customer experience regression v8.0.0 release_note:skip Skip the PR/issue when compiling release notes labels Jan 24, 2022
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
apm 2.8MB 2.8MB +68.0B
dataVisualizer 536.9KB 536.9KB +17.0B
ml 3.5MB 3.5MB +68.0B
observability 401.0KB 401.1KB +34.0B
securitySolution 4.6MB 4.6MB +17.0B
total +204.0B

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

@markov00 markov00 merged commit 676b061 into elastic:8.0 Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience regression release_note:skip Skip the PR/issue when compiling release notes v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants