Fix rebase fallout: orphaned ConfigSession includes, stale findFirstEthInterface call

main renamed findFirstEthInterface() to getRandomInterfacePortName()
(virtual-management-port fix); convert the branch-added no-op-commit test
to the new helper. Drop the three includes ConfigSession.cpp no longer
uses directly (misc-include-cleaner runs as errors in CI).

Author: hillol-nexthop

fboss/cli/fboss2/session/ConfigSession.cpp

@@ -41,13 +41,10 @@
 #include "fboss/agent/AgentDirectoryUtil.h"
 #include "fboss/agent/gen-cpp2/agent_config_types.h"
 #include "fboss/agent/gen-cpp2/switch_config_types.h"
-#include "fboss/agent/if/gen-cpp2/FbossCtrl.h"
-#include "fboss/agent/if/gen-cpp2/FbossCtrlAsyncClient.h"
 #include "fboss/cli/fboss2/gen-cpp2/cli_metadata_types.h"
 #include "fboss/cli/fboss2/session/FbossServiceUtil.h"
 #include "fboss/cli/fboss2/session/Git.h"
 #include "fboss/cli/fboss2/utils/CmdClientUtils.h"
-#include "fboss/cli/fboss2/utils/CmdClientUtilsCommon.h"
 #include "fboss/cli/fboss2/utils/HostInfo.h"
 #include "fboss/cli/fboss2/utils/PortMap.h"
 

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

@@ -96,7 +96,7 @@ TEST_F(ConfigInterfaceDescriptionTest, SetAndVerifyDescription) {
 // agent (skip-when-unchanged). Regression test for the agent-side unification.
 TEST_F(ConfigInterfaceDescriptionTest, SetSameDescriptionIsNoOpCommit) {
   XLOG(INFO) << "[Step 1] Finding an interface to test...";
-  Interface interface = findFirstEthInterface();
+  Interface interface = getInterfaceInfo(getRandomInterfacePortName());
   const std::string originalDescription = interface.description;
   XLOG(INFO) << "  Using interface: " << interface.name << " (description: '"
              << originalDescription << "')";