Project

General

Profile

Bug #46436

Change wording for jail creation advanced field

Added by Waqar Ahmed almost 2 years ago. Updated over 1 year ago.

Status:
Done
Priority:
No priority
Assignee:
Aaron St. John
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

The wording for the tooltip in jails->create->advanced->network properties->vnet0_mac should be changed to the following text

Enter two space-delimited MAC addresses for the virtual interface. The first MAC address is the host, the second MAC address is the jail

Basically all fields with names vnetX_mac tool tips should be changed where X is a number from 0-3.

Risk
We risk breaking the tool tip for the mentioned fields

Acceptance Criteria
If the tool tip for the respective fields have been updated, this ticket can be marked as passed testing


Related issues

Related to FreeNAS - Bug #45838: Improve validation for iocage jail CRUD methods Done

History

#2 Updated by Waqar Ahmed almost 2 years ago

  • Related to Bug #45838: Improve validation for iocage jail CRUD methods added

#3 Updated by Warren Block almost 2 years ago

  • Status changed from Unscreened to In Progress
  • Assignee changed from Warren Block to Waqar Ahmed

This brings up questions, like why do we have a single field for two MAC addresses?

The proposed wording has some problems, like assuming the user knows what "space-delimited" means. Also, what does "is the host" mean? Are these MAC addresses being assigned, or do they already exist?

Guessing at what this is trying to say, I think this is clearer:
"Enter the MAC address to be assigned to the host, a space, then the MAC address to be assigned to the jail."

#4 Updated by Waqar Ahmed almost 2 years ago

  • Assignee changed from Waqar Ahmed to Warren Block

So that is how iocage expects the mac addresses, "mac1 mac2". The host is the FreeNAS system and these don't exist before, they default to a none value where we randomly generate them and if the user provides mac addresses, we use those.
Kindly let me know if you have any more queries and we can finalize the text to be used. Many thanks

#5 Updated by Warren Block almost 2 years ago

  • Assignee changed from Warren Block to Waqar Ahmed

Okay, then we can be more explicit.
"Leave this field empty to generate random MAC addresses for the host and jail. To assign fixed MAC addresses, enter the MAC address to be assigned to the host, a space, then the MAC address to be assigned to the jail."

#6 Updated by Dru Lavigne almost 2 years ago

  • Target version changed from 11.2-RC1 to 11.2-RC2

#7 Updated by Dru Lavigne almost 2 years ago

  • Assignee changed from Waqar Ahmed to Warren Block

#8 Updated by Warren Block almost 2 years ago

  • Assignee changed from Warren Block to Aaron St. John

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

  • Status changed from In Progress to Ready for Testing
  • Needs Doc changed from Yes to No

#10 Updated by Dru Lavigne almost 2 years ago

  • Status changed from Ready for Testing to In Progress
  • Target version changed from 11.2-RC2 to 11.2-RC1

#11 Updated by Lola Yang almost 2 years ago

  • Status changed from In Progress to Ready for Testing

#12 Updated by Lola Yang almost 2 years ago

  • Needs Merging changed from Yes to No

#13 Updated by Jeff Ervin almost 2 years ago

  • Status changed from Ready for Testing to Passed Testing
  • Needs QA changed from Yes to No

Test Passed FreeNAS-11.2-INTERNAL21

Unable to take screenshot. The following dialog appeared while hovering over tooltip: "Leave this field empty to generate random MAC addresses for the host and jail. To assign fixed MAC addresses, enter the MAC address to be assigned to the host, a space, then the MAC address to be assigned to the jail."

#14 Updated by Dru Lavigne over 1 year ago

  • Status changed from Passed Testing to In Progress
  • Needs Merging changed from No to Yes

#16 Updated by Aaron St. John over 1 year ago

#17 Updated by Dru Lavigne over 1 year ago

  • Status changed from In Progress to Done
  • Needs Merging changed from Yes to No

Also available in: Atom PDF