Skip to content

Commit 5f67dd7

Browse files
Fix stale findFirstEthInterface call in ConfigBgpSessionTest
The base renamed findFirstEthInterface() to getRandomInterfacePortName() (virtual-management-port fix); convert the agent-only-commit step of the BGP rollback test to the new helper, same as ConfigInterfaceDescriptionTest.
1 parent d88e6a7 commit 5f67dd7

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

fboss/cli/fboss2/test/integration_test/ConfigBgpSessionTest.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ TEST_F(ConfigBgpSessionTest, AgentCommitSnapshotsBgpAndRollbackPreservesIt) {
175175
const folly::dynamic defaultBgp = readSystemBgpConfig();
176176

177177
// Step 1: an agent-only commit (interface description). No BGP is staged.
178-
Interface intf = findFirstEthInterface();
178+
Interface intf = getInterfaceInfo(getRandomInterfacePortName());
179179
auto setRes =
180180
runCli({"config", "interface", intf.name, "description", "bgp-rb-test"});
181181
ASSERT_EQ(setRes.exitCode, 0) << "stderr=" << setRes.stderr;

0 commit comments

Comments
 (0)