if_bridge_test: fix MTU test
ClosedPublic
Actions

Authored by ivy on Jul 11 2025, 6:49 AM.

Details

Reviewers

des
kevans
kp
zlei

Group Reviewers

network

Commits

rG44a5649bc3a2: tests/sys/net/if_bridge_test: fix MTU test
rGe6ef675de261: tests/sys/net/if_bridge_test: fix MTU test

Summary

"ifconfig gif0 ether" doesn't return any output, so this wasn't
correctly checking the MTU on the gif interface. Remove "ether".

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 65337
Build 62220: arc lint + arc unit

Event Timeline

ivy created this revision.Jul 11 2025, 6:49 AM

Herald added subscribers: glebius, melifaro, ae, imp. · View Herald TranscriptJul 11 2025, 6:49 AM

ivy requested review of this revision.Jul 11 2025, 6:49 AM

Harbormaster completed remote builds in B65337: Diff 158305.Jul 11 2025, 6:49 AM

ivy added a parent revision: D51244: ifconfig/ifbridge.c: add get_vlan_id() [old].Jul 11 2025, 6:49 AM

ivy added a child revision: D51252: ifconfig: don't print bridge member ifmaxaddr if it's 0.Jul 11 2025, 7:07 AM

"Whoops"

This revision is now accepted and ready to land.Jul 12 2025, 12:13 PM

Looks good to me.

Closed by commit rGe6ef675de261: tests/sys/net/if_bridge_test: fix MTU test (authored by ivy). · Explain WhyJul 25 2025, 11:10 PM

This revision was automatically updated to reflect the committed changes.

ivy added a commit: rGe6ef675de261: tests/sys/net/if_bridge_test: fix MTU test.

ivy added a commit: rG44a5649bc3a2: tests/sys/net/if_bridge_test: fix MTU test.Jul 28 2025, 12:43 PM

Revision Contents
Changeset List

Path

Size

tests/

sys/

net/

if_bridge_test.sh

4 lines

Diff 158305

View Options

tests/sys/net/if_bridge_test.sh

tests/sys/net/if_bridge_test: fix MTU testClosedPublicActions