Project

General

Profile

Bug #27122

Overhaul bridge support for APIv2

Added by Patrick M. Hausen about 1 year ago. Updated 22 days ago.

Status:
Passed Testing
Priority:
No priority
Assignee:
William Grzybowski
Category:
Middleware
Target version:
11.3
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 RewriteScreened
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

Associated revisions

Revision 3271531c (diff)
Added by William Grzybowski 27 days ago

feat(migration): add bridge model

Ticket: #27122

Revision 6d43cca0 (diff)
Added by William Grzybowski 25 days ago

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

Ticket: #27122

Revision 19e2361f (diff)
Added by William Grzybowski 25 days ago

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

Ticket: #27122

Revision c7645875 (diff)
Added by William Grzybowski 25 days ago

feat(middlewared/network): basic bridge unit tests

Ticket: #27122

Revision 4a0c5ad0 (diff)
Added by William Grzybowski 25 days ago

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

Ticket: #27122

Revision 170a4697 (diff)
Added by William Grzybowski 25 days ago

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

Ticket: #27122

Revision 054a3749 (diff)
Added by William Grzybowski 25 days ago

fix(middlewared/network): remove unused var

Ticket: #27122

Revision 1bd6af83 (diff)
Added by William Grzybowski 25 days ago

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

Ticket: #27122

Revision 0e5e4d85 (diff)
Added by William Grzybowski 25 days ago

fix(middlewared/network): just `raise`

Ticket: #27122

Revision f7b48a54 (diff)
Added by William Grzybowski 25 days ago

fix(middlewared/network): just `raise`

Ticket: #27122

Revision bfd35c8f (diff)
Added by William Grzybowski 22 days ago

feat(migration): add bridge model

Ticket: #27122

Revision c7d52841 (diff)
Added by William Grzybowski 22 days ago

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

Ticket: #27122

Revision f79bd128 (diff)
Added by William Grzybowski 22 days ago

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

Ticket: #27122

Revision 80728470 (diff)
Added by William Grzybowski 22 days ago

feat(middlewared/network): basic bridge unit tests

Ticket: #27122

Revision 254f5d36 (diff)
Added by William Grzybowski 22 days ago

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

Ticket: #27122

Revision 72934655 (diff)
Added by William Grzybowski 22 days ago

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

Ticket: #27122

Revision 651eb312 (diff)
Added by William Grzybowski 22 days ago

fix(middlewared/network): remove unused var

Ticket: #27122

Revision 058465e9 (diff)
Added by William Grzybowski 22 days ago

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

Ticket: #27122

Revision 3f3bdd8f (diff)
Added by William Grzybowski 22 days ago

fix(middlewared/network): just `raise`

Ticket: #27122

History

#1 Updated by Dru Lavigne about 1 year ago

  • Assignee changed from Release Council to Marcelo Araujo

#2 Updated by Dru Lavigne 12 months ago

  • Target version set to 11.2-BETA1

#3 Updated by Marcelo Araujo 12 months ago

  • Status changed from Unscreened to Screened

#4 Updated by Dru Lavigne 11 months ago

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

#5 Updated by Dru Lavigne 9 months ago

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

#6 Updated by Nick Wolff 8 months ago

  • Severity set to Low Medium

#7 Updated by Dru Lavigne 6 months ago

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

#8 Updated by Marcelo Araujo 6 months ago

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

I'm bumping this to 11.4 because it is not something easy to implement and I need to discuss it with @brandon.

#9 Updated by Marcelo Araujo 6 months 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 6 months 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 5 months ago

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

#12 Updated by William Grzybowski 4 months ago

  • Target version changed from Backlog to 11.3

#13 Updated by Dru Lavigne 4 months ago

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

#15 Updated by Dru Lavigne about 2 months ago

#16 Updated by Dru Lavigne about 2 months ago

#17 Updated by Dru Lavigne about 2 months ago

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

#18 Updated by Bug Clerk 25 days ago

  • Status changed from Screened to In Progress

#19 Updated by Bug Clerk 22 days ago

  • Status changed from In Progress to Ready for Testing

#20 Updated by William Grzybowski 22 days 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 22 days 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

Also available in: Atom PDF