Project

General

Profile

Bug #27122

Add overhauled bridge support to API

Added by Patrick M. Hausen over 2 years ago. Updated about 1 year ago.

Status:
Done
Priority:
No priority
Assignee:
William Grzybowski
Category:
Middleware
Target version:
11.3-ALPHA1
Seen in:
11.1-RC3
Severity:
Medium
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
Yes
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

Dear developers,

FreeNAS already uses if_bridge for jails and bhyve VMs. But the resulting setup violates what is clearly and explicitly stated in the FreeBSD documentation:
https://www.freebsd.org/doc/en_US.ISO8859-1/books/handbook/network-bridging.html

The IP address belongs on the bridge, not on one (igb0 in my case) of the member interfaces.

Second, it would be great if I could add the other 3 ports of my system to that very same bridge to have an additional switch for my home network. I already do this with a post-init script, but bridging configuration should be available via the UI.

Third, it seems like the automagic bridge configuration assigns bhyve VMs to one and iocage jails to a second bridge IF. This should be either one and the same or even better configurable via the UI and per jail/VM.

Apart from the fact that Corral did not support jails, all of this was in place and working flawlessly in Corral. Put 4 physical interfaces into one bridge, configure IPv4 and IPv6 for the bridge instead for one of the members, pick which bridge to use for a VM ...

Kind regards,
Patrick

P.S. A "networking" category is missing in Redmine ;)

Related issues

Related to FreeNAS - Bug #27780: Network Interface configuration seems to be broken when using VM's and iocageClosed
Related to FreeNAS - Bug #42269: Can't select tap interface to a VM (using classic UI)Closed
Related to FreeNAS - Umbrella #50841: Network Interfaces Management RewriteDone
Has duplicate FreeNAS - Bug #29068: Can't bridge tap interface to a virtual machineClosed2018-03-04
Has duplicate FreeNAS - Feature #34288: Would like be able to bridge network interfacesClosed
Has duplicate FreeNAS - Feature #24347: Add bridge supportClosed
Copied to FreeNAS - Bug #67735: Implement bridge support in new UIClosed

Associated revisions

Revision 3271531c (diff)
Added by William Grzybowski over 1 year ago

feat(migration): add bridge model Ticket: #27122

Revision 6d43cca0 (diff)
Added by William Grzybowski over 1 year ago

feat(middlewared/network): initial support for bridge interfaces Ticket: #27122

Revision 19e2361f (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): do not allow same interface in different bridge Ticket: #27122

Revision c7645875 (diff)
Added by William Grzybowski over 1 year ago

feat(middlewared/network): basic bridge unit tests Ticket: #27122

Revision 4a0c5ad0 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): start bridge name on index 2 Ticket: #27122

Revision 170a4697 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): allow bridge[2-9] to be destroyed Ticket: #27122

Revision 054a3749 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): remove unused var Ticket: #27122

Revision 1bd6af83 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): do not define inline get_next Ticket: #27122

Revision 0e5e4d85 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): just `raise` Ticket: #27122

Revision f7b48a54 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): just `raise` Ticket: #27122

Revision bfd35c8f (diff)
Added by William Grzybowski over 1 year ago

feat(migration): add bridge model Ticket: #27122

Revision c7d52841 (diff)
Added by William Grzybowski over 1 year ago

feat(middlewared/network): initial support for bridge interfaces Ticket: #27122

Revision f79bd128 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): do not allow same interface in different bridge Ticket: #27122

Revision 80728470 (diff)
Added by William Grzybowski over 1 year ago

feat(middlewared/network): basic bridge unit tests Ticket: #27122

Revision 254f5d36 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): start bridge name on index 2 Ticket: #27122

Revision 72934655 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): allow bridge[2-9] to be destroyed Ticket: #27122

Revision 651eb312 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): remove unused var Ticket: #27122

Revision 058465e9 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): do not define inline get_next Ticket: #27122

Revision 3f3bdd8f (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/network): just `raise` Ticket: #27122

History

#1 Updated by Dru Lavigne over 2 years ago

  • Assignee changed from Release Council to Marcelo Araujo

#2 Updated by Dru Lavigne over 2 years ago

  • Target version set to 11.2-BETA1

#3 Updated by Marcelo Araujo over 2 years ago

  • Status changed from Unscreened to Screened

#4 Updated by Dru Lavigne about 2 years ago

  • Status changed from Screened to Not Started
  • Target version changed from 11.2-BETA1 to 11.2-RC2

#5 Updated by Dru Lavigne about 2 years ago

  • Has duplicate Bug #29068: Can't bridge tap interface to a virtual machine added

#6 Updated by Nick Wolff almost 2 years ago

  • Severity set to Low Medium

#7 Updated by Dru Lavigne almost 2 years ago

  • Has duplicate Feature #34288: Would like be able to bridge network interfaces added

#9 Updated by Marcelo Araujo almost 2 years ago

  • Category changed from OS to Middleware
  • Status changed from Screened to Unscreened
  • Assignee changed from Marcelo Araujo to William Grzybowski
  • Target version changed from 11.4 to N/A

Forward to middleware team.

#10 Updated by William Grzybowski almost 2 years ago

  • Status changed from Unscreened to Screened
  • Target version changed from N/A to Backlog
  • Severity changed from Low Medium to Medium

#11 Updated by Dru Lavigne almost 2 years ago

  • Related to Bug #27780: Network Interface configuration seems to be broken when using VM's and iocage added

#12 Updated by William Grzybowski over 1 year ago

  • Target version changed from Backlog to 11.3

#13 Updated by Dru Lavigne over 1 year ago

  • Related to Bug #42269: Can't select tap interface to a VM (using classic UI) added

#15 Updated by Dru Lavigne over 1 year ago

#16 Updated by Dru Lavigne over 1 year ago

#17 Updated by Dru Lavigne over 1 year ago

  • Subject changed from Bridging setup is in desperate need of an overhaul to Overhaul bridge support

#18 Updated by Bug Clerk over 1 year ago

  • Status changed from Screened to In Progress

#19 Updated by Bug Clerk over 1 year ago

  • Status changed from In Progress to Ready for Testing

#20 Updated by William Grzybowski over 1 year ago

  • Status changed from Ready for Testing to Passed Testing

Testing will be made on related ticket when new UI part is implemented.

#21 Updated by Dru Lavigne over 1 year ago

  • Subject changed from Overhaul bridge support to Overhaul bridge support for APIv2
  • Needs QA changed from Yes to No
  • Needs Merging changed from Yes to No

#22 Updated by Dru Lavigne about 1 year ago

  • Status changed from Passed Testing to Done
  • Needs Doc changed from Yes to No

#24 Updated by Dru Lavigne about 1 year ago

  • Copied to Bug #67735: Implement bridge support in new UI added

#25 Updated by Dru Lavigne about 1 year ago

  • Target version changed from 11.3 to 11.3-BETA1

#26 Updated by Dru Lavigne about 1 year ago

  • Subject changed from Overhaul bridge support for APIv2 to Add overhauled bridge support to legacy UI
  • Status changed from Done to Ready for Testing
  • Needs QA changed from No to Yes
  • Needs Doc changed from No to Yes

#27 Updated by William Grzybowski about 1 year ago

  • Subject changed from Add overhauled bridge support to legacy UI to Add overhauled bridge support to API
  • Status changed from Ready for Testing to Done
  • Needs QA changed from Yes to No

#29 Updated by Dru Lavigne about 1 year ago

  • Target version changed from 11.3-BETA1 to 11.3-ALPHA1

Also available in: Atom PDF