Project

General

Profile

Bug #31476

System -> Advanced -> Enable Serial Console: Do not check this box if the serial port is disabled

Added by Michael Reynolds over 2 years ago. Updated over 2 years ago.

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

Description

The Current tooltip says:
Do not check this box if the serial port is disabled.

This tooltip has Do Not in bold which makes it seem pretty important.

This indicates serious issues if checked and there isn't a Serial Console available.
Is there a way to check if there is a Serial Console available? If this is the case can we only show this checkbox if there is actually a serial port enabled?

If there is not a way to check for a serial port programmatically, would it be better to hide the checkbox under a secondary "Serial Console Settings" dropdown?

Actually it seems like all Serial Console settings should be unavailable unless there is in fact a Serial Console available to avoid unintended/accidental issues.

serial console checked.png (42.1 KB) serial console checked.png Vaibhav Chauhan, 04/27/2018 04:23 PM
serial console unchecked.png (50 KB) serial console unchecked.png Vaibhav Chauhan, 04/27/2018 04:23 PM
17031
17034

History

#1 Updated by Erin Clark over 2 years ago

  • Assignee changed from Erin Clark to Xin Tan
  • Priority changed from No priority to Important

#2 Updated by Dru Lavigne over 2 years ago

  • Assignee changed from Xin Tan to Erin Clark

#3 Updated by Erin Clark over 2 years ago

  • Assignee changed from Erin Clark to Vaibhav Chauhan

#4 Updated by Vaibhav Chauhan over 2 years ago

17031
17034

serial port choices were not being shown(due to an existing bug, which I have a fix for along with this fix). so as long as we have a valid serial port choice being shown in the GUI after my fix, does this resolve this issue ? attaching a screenshots with my changes as a reference.

#5 Updated by Dru Lavigne over 2 years ago

  • Status changed from In Progress to Done
  • Target version changed from 11.2-RC2 to Master - FreeNAS Nightlies
  • Needs Merging changed from Yes to No

#6 Updated by Vaibhav Chauhan over 2 years ago

  • Status changed from Done to In Progress
  • Assignee changed from Michael Reynolds to Erin Clark
  • Severity set to New

#7 Updated by Dru Lavigne over 2 years ago

  • Status changed from In Progress to Done
  • Assignee changed from Erin Clark to Vaibhav Chauhan

Also available in: Atom PDF