Project

General

Profile

Actions

Bug #5247

closed
PD PD

NTP Status Widget format

Bug #5247: NTP Status Widget format

Added by Phillip Davis almost 11 years ago. Updated almost 11 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
Start date:
10/03/2015
Due date:
% Done:

100%

Estimated time:

Description

The NTP Status dashboard widget is displaying some stuff a bit weird - see screenshot attached.
First row says "Server Time" 5:30 - what is the "5:30" there? I am not sure. My timezone happens to be Asia/Kathmandu GMT+0545 and often this gets rounded to 0530or 0600 on systems that do not calculate this 0545 timezone correctly.

Next row has "Server Time" again and a date-tike that is displayed in GMT. I guess the NTP server's time is in GMT by definition. But the 2.2.* widget used to show this adjusted for local time.

The screenshot shows a sample NTP Status widget display form another box running 2.2.5.


Files

NTP-Status-Widget.png (7.71 KB) NTP-Status-Widget.png Phillip Davis, 10/03/2015 08:11 AM
NTP-Status-Widget-2.png (4.36 KB) NTP-Status-Widget-2.png Phillip Davis, 10/06/2015 12:24 AM

Updated by Anonymous almost 11 years ago Actions #1

Thanks Phil,

I'll look at the code and figure out what the display represents.

PD Updated by Phillip Davis almost 11 years ago Actions #2

Also the old display used to tick the time, the new display seems static.

KB Updated by Kill Bill almost 11 years ago Actions #3

This obviously should show local time (adding the TZ info there is good idea, the rest - definitely not.)

While here, perhaps you could also look at the broken IPv6 display - #4815

Updated by Anonymous almost 11 years ago Actions #4

Looks like some debugging perhaps. Sure, I'll look at the hIPv6 thing too.

Updated by Anonymous almost 11 years ago Actions #5

  • Status changed from New to Feedback
  • Assignee set to Phillip Davis

Fixed numerous issues in NTP widget.
Time now displays correctly (with offset from GMT) and updates once per second.

Updated by Anonymous almost 11 years ago Actions #6

  • % Done changed from 0 to 100

Applied in changeset pfsense:commit:1e8f28de9b7d482a2f6eeca04d4c894d016ec752.

PD Updated by Phillip Davis almost 11 years ago Actions #7

That looks much better.
It has a settings button, which does not do anything and was not there in 2.2.* - maybe that should be removed? Or are there some settings features to add?

CB Updated by Chris Buechler almost 11 years ago Actions #8

  • Status changed from Feedback to Resolved

settings button was removed. all fixed

Actions

Also available in: Atom