Project

General

Profile

Bug #6194

Frontend monitoring default to GMT

Added by Igor Sfiligoi about 6 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Igor Sfiligoi
Category:
Frontend Monitoring
Target version:
Start date:
05/07/2014
Due date:
% Done:

0%

Estimated time:
First Occurred:
Occurs In:
Stakeholders:
Duration:

Description

The Frontend monitoring defaults to GMT, instead of local timezone.


Related issues

Related to GlideinWMS - Bug #5940: Local timezone in vofrontend monitoring broken for Chrome, IEClosed04/17/2014

History

#1 Updated by Igor Sfiligoi about 6 years ago

  • Status changed from New to Feedback
  • Assignee changed from Igor Sfiligoi to Marco Mambelli

The fix is a one liner;
basically the default timezone initialization was incorrect.

Anyhow, code in v3/6194.

Please review.

#2 Updated by Parag Mhashilkar about 6 years ago

  • Status changed from Feedback to Resolved
  • Assignee changed from Marco Mambelli to Igor Sfiligoi
  • Target version changed from v3_2_6 to v3_2_5

Merged it to branch_v3_2

#3 Updated by Marco Mambelli about 6 years ago

  • Status changed from Resolved to Feedback

Reopening the ticket:
I checked the Flot documentation and I did not see "default" mentioned as value for timezone.
It mentions a numeric offset or "browser":
https://github.com/flot/flot/blob/master/API.md
http://www.flotcharts.org/flot/examples/axes-time-zones/index.html

Why "default" was chosen?

#4 Updated by Parag Mhashilkar about 6 years ago

Also factoryStatus.html has same issue as well(?)

#5 Updated by Igor Sfiligoi about 6 years ago

No... factory works fine.

As for the use of default... any invalid string will work;
i.e. if the timezone passed makes no sense, it autodetects the local one (else just uses what is being passed to it)

BTW: The docs you point to are irrelevant; javascriptrrd does not pass this value as-is to Flot.

#6 Updated by Igor Sfiligoi about 6 years ago

  • Assignee changed from Igor Sfiligoi to Parag Mhashilkar

#7 Updated by Parag Mhashilkar about 6 years ago

  • Assignee changed from Parag Mhashilkar to Igor Sfiligoi

Then document the un conventional use of the default value in the html file and merge it back to respective branches.

#8 Updated by Igor Sfiligoi about 6 years ago

  • Status changed from Feedback to Resolved

Good point... done.

And merged into branch_v3_2 and branch_v3_2_5.

#9 Updated by Parag Mhashilkar about 6 years ago

  • Status changed from Resolved to Closed


Also available in: Atom PDF