Project

General

Profile

Bug #39493

Show warning if Swap size is set to 0

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

Status:
Done
Priority:
No priority
Assignee:
Erin Clark
Category:
GUI (new)
Target version:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

System:Advanced -> Swap size in GiB

Setting to 0 disables swap creation completely. This is STRONGLY DISCOURAGED (caps is from the previous in-field text)

If swap size is set to 0 an error dialog should open that the user has to acknowledge that they are aware doing so is strongly discouraged.
Perhaps red warning text under the field that persists as long as the swap size is set to 0?

The tooltip will be updated soon.

Can we also shorten the field size to only accept 2 digits?
Unless people are using > 99GiB of swap?

SwapSizeFieldLength-2018-07-24.png (22.5 KB) SwapSizeFieldLength-2018-07-24.png Michael Reynolds, 07/24/2018 02:43 PM
swap_size_msg.png (165 KB) swap_size_msg.png Aaron St. John, 08/07/2018 06:16 AM
22508
23840

Associated revisions

Revision 6750b59b (diff)
Added by Erin Clark about 2 years ago

Add 'warnings' to entity-form components Ticket: #39493

Revision 67493f53 (diff)
Added by Erin Clark about 2 years ago

Add warning for 0 swap size and swap size validators to system advanced Ticket: #39493

History

#1 Updated by Dru Lavigne about 2 years ago

  • Target version changed from Backlog to 11.2-BETA3

#2 Updated by Erin Clark about 2 years ago

  • Status changed from Unscreened to In Progress

#3 Updated by Erin Clark about 2 years ago

  • Status changed from In Progress to Ready for Testing

#4 Updated by Dru Lavigne about 2 years ago

  • Needs Merging changed from Yes to No

#5 Updated by Aaron St. John about 2 years ago

23840

#6 Updated by Dru Lavigne about 2 years ago

  • Needs Doc changed from No to Yes

#8 Updated by Aaron St. John about 2 years ago

angulargui PR: https://github.com/freenas/freenas-docs/pull/281

Leaving "Needs Doc" set until PR merged.

#9 Updated by Aaron St. John about 2 years ago

  • Needs Doc changed from Yes to No

#10 Updated by Dru Lavigne about 2 years ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF