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

[Observability] All the times on the Elastic Charts are using UTC instead of the browser timezone on the overview page #122684

Closed
simianhacker opened this issue Jan 11, 2022 · 4 comments
Labels
bug Fixes for quality problems that affect the customer experience Feature:Observability Overview Relaunch of the Observability Overview Page Team:Observability Team label for Observability Team (for things that are handled across all of observability)

Comments

@simianhacker
Copy link
Member

simianhacker commented Jan 11, 2022

Kibana version:

7.16.2+

Elasticsearch version:

7.16.2+

Original install method (e.g. download page, yum, from source, etc.):

any

Describe the bug:

All the Elastic Charts in the overview page are displaying UTC on the X-Axis. This issue was introduced in elastic/elastic-charts#1383 and appearded in 7.16.2. Anything that is using the niceTimeFormatter and relying on the default behavior is affected by this bug. The solution is to add the browser's timezone param to the BarChart component.

x-pack/plugins/observability/public/components/app/section/apm/index.tsx:  const formatter = niceTimeFormatter([min, max]);
x-pack/plugins/observability/public/components/app/section/logs/index.tsx:  const formatter = niceTimeFormatter([min, max]);
x-pack/plugins/observability/public/components/app/section/uptime/index.tsx:  const formatter = niceTimeFormatter([min, max]);

Steps to reproduce:

  1. Index some APM, Logs or Uptime Data
  2. Visit the Observability Overview page

Expected behavior:

All times should be local to the browser.

Related Issues

@simianhacker simianhacker added bug Fixes for quality problems that affect the customer experience Feature:Observability Overview Relaunch of the Observability Overview Page labels Jan 11, 2022
@botelastic botelastic bot added the needs-team Issues missing a team label label Jan 11, 2022
@mshustov mshustov added the Team:Observability Team label for Observability Team (for things that are handled across all of observability) label Jan 13, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/unified-observability (Team:Observability)

@hendry-lim
Copy link

hendry-lim commented Feb 11, 2022

This was fixed in 7.17.0, but seems to either return or not fixed in 8.0.0.
image

Looks fine in 8.1.0-SNAPSHOT
image

@markov00
Copy link
Member

@hendry-lim the PR to fix that in 8.0.1 was just merged #123265

@formgeist
Copy link
Contributor

Looks like this issue has been resolved, so closing. If I'm wrong, feel free to open again 👍

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 Feature:Observability Overview Relaunch of the Observability Overview Page Team:Observability Team label for Observability Team (for things that are handled across all of observability)
Projects
None yet
Development

No branches or pull requests

6 participants