Project

General

Profile

Bug #25954

System uptime graph label

Added by Logan Willmert about 3 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority:
Important
Assignee:
ChenTao Wang
Category:
GUI (new)
Seen in:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

The system uptime graph label reads "Days" while the numerical scale is set for minutes (0-600 in my case).

uptime graph.PNG (76.2 KB) uptime graph.PNG screenshot of uptime graph Logan Willmert, 09/22/2017 08:22 AM
12521

History

#1 Updated by Dru Lavigne about 3 years ago

  • Assignee changed from Release Council to William Grzybowski

William: please punt to Erin if this is better suited for the new UI.

#2 Updated by Stefan Schroepfer about 3 years ago

Logan Willmert wrote:

The system uptime graph label reads "Days" while the numerical scale is set for minutes (0-600 in my case).

I'm pretty sure that the meaning of e.g. "600 m" is 600 millidays (1 milliday being 0.001 days) in this graph label, not 600 minutes.

#3 Updated by William Grzybowski about 3 years ago

  • Category changed from 2 to GUI (new)
  • Assignee changed from William Grzybowski to Erin Clark

Stefan is right. This is millidays, thats what the "m" stands for. While I agree this is subptimal, there isnt much we can do in the old UI.

Re-targeting for new UI.

#4 Avatar?id=14398&size=24x24 Updated by Kris Moore almost 3 years ago

  • Assignee changed from Erin Clark to ChenTao Wang
  • Priority changed from No priority to Important
  • Target version set to 11.2-BETA1

ChenTao - Lets get started with this one. Please add a system-uptime graph to the dashboard view.

#5 Updated by ChenTao Wang almost 3 years ago

  • Status changed from Unscreened to Needs Developer Review

#6 Updated by Dru Lavigne almost 3 years ago

  • Status changed from Needs Developer Review to 46

Chen: what is the URL to the PR and who would you like to perform the review?

#7 Updated by ChenTao Wang almost 3 years ago

Hi Dru,

PR: https://github.com/freenas/webui/pull/57
I don't know who can perform the review. Please ask Kris.

#8 Updated by Dru Lavigne almost 3 years ago

  • Status changed from 46 to Needs Developer Review
  • Assignee changed from ChenTao Wang to Lola Yang

#9 Updated by Lola Yang almost 3 years ago

this PR already merged to master.

But I think maybe the Y-axis label need more details. What do you think about this? kris Payne @chentao

#10 Updated by Lola Yang almost 3 years ago

  • Status changed from Needs Developer Review to Reviewed by Developer
  • Assignee changed from Lola Yang to ChenTao Wang

Lola Yang wrote:

this PR already merged to master.

But I think maybe the Y-axis label need more details. What do you think about this? kris Payne @chentao

okay. Damian will figure out the chart label things when he starts to do the dashboard. so this ticket can be closed. thanks.

#11 Updated by Dru Lavigne almost 3 years ago

  • Status changed from Reviewed by Developer to Resolved
  • Target version changed from 11.2-BETA1 to Master - FreeNAS Nightlies

Also available in: Atom PDF