bridge tests: Test that we also forward on some interfaces
ClosedPublic
Actions

Authored by kp on Feb 24 2021, 5:15 PM.

Details

Reviewers

None

Group Reviewers

Commits

rG9c744d299243: bridge tests: Test that we also forward on some interfaces
rGeb4221b9fb91: bridge tests: Test that we also forward on some interfaces
rG7a4dbffa4205: bridge tests: Test that we also forward on some interfaces

Summary

Ensure that we not only block on some interfaces, but also forward on
some. Without the previous commit we wound up discarding on all ports,
rather than only on the ports needed to break the loop.

MFC after: 1 week
Sponsored by: Orange Business Services

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kp created this revision.Feb 24 2021, 5:15 PM

Herald added subscribers: melifaro, ae, imp. · View Herald TranscriptFeb 24 2021, 5:15 PM

kp requested review of this revision.Feb 24 2021, 5:15 PM

Harbormaster completed remote builds in B37349: Diff 84617.Feb 24 2021, 5:15 PM

kp added a parent revision: D28916: bridgestp: Ensure we send STP on VLAN interfaces.Feb 24 2021, 5:16 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 25 2021, 12:33 PM

This revision was landed with ongoing or failed builds.

Closed by commit rG7a4dbffa4205: bridge tests: Test that we also forward on some interfaces (authored by kp). · Explain Why

This revision was automatically updated to reflect the committed changes.

kp added a commit: rG7a4dbffa4205: bridge tests: Test that we also forward on some interfaces.

kp added a commit: rGeb4221b9fb91: bridge tests: Test that we also forward on some interfaces.Mar 4 2021, 9:33 AM

kp added a commit: rG9c744d299243: bridge tests: Test that we also forward on some interfaces.

Revision Contents
Changeset List

Path

Size

tests/

sys/

net/

if_bridge_test.sh

18 lines

Diff 84670

View Options

tests/sys/net/if_bridge_test.sh

bridge tests: Test that we also forward on some interfacesClosedPublicActions