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.

Sign up for GitHub

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

Jump to bottom

[Platform] Health check time is always in the Pacific timezone #5172

Closed
iSignal opened this issue Jul 21, 2020 · 1 comment
Closed

[Platform] Health check time is always in the Pacific timezone #5172

iSignal opened this issue Jul 21, 2020 · 1 comment
Assignees
@jaydeepkumara
Labels
area/platform Yugabyte Platform team/platform-hi Tickets handled by HashedIn team
Milestone
2.5.x

Comments

@iSignal
Copy link
Contributor

iSignal commented Jul 21, 2020 •
edited by chirag-yb
Loading

Repro steps:

Change timezone on client browser device. Health check timestamps do not change though other timestamps like metrics / backups do.

It seems to be hardcoded here -

yugabyte-db/managed/devops/bin/cluster_health.py

Line 620 in 1403b2b

return datetime.utcnow().replace(tzinfo=tz.tzutc()).astimezone(tz.gettz('America/Los_Angeles'))

Aha! Link: https://yugabyte-test.aha.io/features/PLATFORM-107
@iSignal iSignal added the area/platform Yugabyte Platform label Jul 21, 2020
@streddy-yb streddy-yb added this to the v2.5 milestone Oct 23, 2020
@streddy-yb streddy-yb added the team/platform-hi Tickets handled by HashedIn team label Jan 13, 2021
@jaydeepkumara jaydeepkumara self-assigned this Jan 19, 2021
@jaydeepkumara
Copy link
Contributor

PR:- #6933

jaydeepkumara added a commit that referenced this issue Jan 22, 2021
@jaydeepkumara
[#5172] Health check time is always in the Pacific timezone (#6933)
edfe17b 
* Fix timezone issue

* Changed moment timezone to moment
polarweasel pushed a commit to lizayugabyte/yugabyte-db that referenced this issue Mar 9, 2021
@jaydeepkumara
[yugabyte#5172] Health check time is always in the Pacific timezone (y…
22f9559 
…ugabyte#6933)

* Fix timezone issue

* Changed moment timezone to moment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jaydeepkumara jaydeepkumara

Labels
area/platform Yugabyte Platform team/platform-hi Tickets handled by HashedIn team
Projects
None yet
Milestone
2.5.x
Development

No branches or pull requests
5 participants
@iSignal @Arnav15 @jaydeepkumara @andrewc-dev @streddy-yb