Project

General

Profile

Bug #40076

Update API to properly handle vnet devices

Added by Vaibhav Chauhan about 2 years ago. Updated about 2 years ago.

Status:
Done
Priority:
No priority
Assignee:
Waqar Ahmed
Category:
Middleware
Target version:
Severity:
Medium
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

as of right now when we query a FreeNAS mini
freenas# midclt call notifier.choices 'NICChoices'
[["em0", "em0"], ["vnet0:2", "vnet0:2"]]

we get `vnet0:2`, these virtual interfaces are not suppose to be shown up at all.

Associated revisions

Revision 64b4abce (diff)
Added by Waqar Ahmed about 2 years ago

Nic choices updated to support vnet This commit adds support where vnet nic's are treated similar to epairs and hence removed when epairs are removed from nic choices. Ticket: #40076

History

#1 Updated by William Grzybowski about 2 years ago

  • Status changed from Unscreened to Not Started
  • Target version changed from Backlog to 11.2-BETA3
  • Severity changed from New to Medium

#2 Updated by William Grzybowski about 2 years ago

  • Assignee changed from William Grzybowski to Waqar Ahmed

Waqar, this should be an easy one, we need to treat vnet* same as epair* in NICChoices

#3 Updated by Waqar Ahmed about 2 years ago

  • Status changed from Not Started to In Progress

#4 Updated by Waqar Ahmed about 2 years ago

  • Status changed from In Progress to Ready for Testing

#5 Updated by Dru Lavigne about 2 years ago

  • Subject changed from please exclude vnet interfaces from showing up in the results for `midclt call notifier.choices 'NICChoices'` to Update API to properly handle vnet devices
  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#6 Updated by Bonnie Follweiler about 2 years ago

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

Test Passed in FreeNAS-11.2-MASTER-201808300858

#8 Updated by Dru Lavigne about 2 years ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF