Rename platform from WEDGE800BA to WEDGE800BACT (#616)

jcyu-accton · meta-codesync[bot] · commit eae1e7f2fabb · 2025-11-03T17:27:52.000-08:00
Summary: Description: This PR updates all relevant class names and file names to use the new platform name WEDGE800BACT. Changes include: Platform enums: WEDGE800B_ACT to WEDGE800BACT Class names and file names: Updated from Wedge800ba... to Wedge800BACT... CSV files: Renamed from wedge800ba to wedge800bact Pull Request resolved: #616 Reviewed By: shiva-menta Differential Revision: D85779579 Pulled By: Protick666 fbshipit-source-id: c3c48ccadad18ad03e99fd69dc53d465d0def62a
diff --git a/cmake/AgentPlatformsCommon.cmake b/cmake/AgentPlatformsCommon.cmake
@@ -49,7 +49,7 @@ target_link_libraries(platform_mapping_utils
   janga800bic_platform_mapping
   tahan800bc_platform_mapping
   icecube800bc_platform_mapping
-  wedge800ba_platform_mapping
+  wedge800bact_platform_mapping
   icetea800bc_platform_mapping
   tahansb800bc_platform_mapping
   wedge800ca_platform_mapping
diff --git a/cmake/AgentPlatformsCommonWedge800BACT.cmake b/cmake/AgentPlatformsCommonWedge800BACT.cmake
@@ -0,0 +1,12 @@
+# CMake to build libraries and binaries in fboss/agent/platforms/common/wedge800bact
+
+# In general, libraries and binaries in fboss/foo/bar are built by
+# cmake/FooBar.cmake
+
+add_library(wedge800bact_platform_mapping
+  fboss/agent/platforms/common/wedge800bact/Wedge800BACTPlatformMapping.cpp
+)
+
+target_link_libraries(wedge800bact_platform_mapping
+  platform_mapping
+)
diff --git a/cmake/AgentPlatformsCommonWedge800ba.cmake b/cmake/AgentPlatformsCommonWedge800ba.cmake
diff --git a/cmake/AgentPlatformsSai.cmake b/cmake/AgentPlatformsSai.cmake
@@ -43,8 +43,8 @@ add_library(sai_platform
   fboss/agent/platforms/sai/SaiBcmIcetea800bcPlatform.cpp
   fboss/agent/platforms/sai/SaiBcmTahansb800bcPlatform.cpp
   fboss/agent/platforms/sai/SaiBcmLadakh800bclsPlatform.cpp
-  fboss/agent/platforms/sai/SaiBcmWedge800baPlatform.cpp
-  fboss/agent/platforms/sai/SaiBcmWedge800baPlatformPort.cpp
+  fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatform.cpp
+  fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatformPort.cpp
   fboss/agent/platforms/sai/oss/SaiBcmMinipackPlatform.cpp
   fboss/agent/platforms/sai/oss/SaiTajoPlatform.cpp
   fboss/agent/platforms/sai/oss/SaiBcmMinipackPlatformPort.cpp
@@ -116,7 +116,7 @@ target_link_libraries(sai_platform
   icetea800bc_platform_mapping
   led_structs_types_cpp2
   led_mapping_cpp2
-  wedge800ba_platform_mapping
+  wedge800bact_platform_mapping
   wedge800ca_platform_mapping
 )
 
diff --git a/fboss/agent/platforms/common/BUCK b/fboss/agent/platforms/common/BUCK
@@ -59,7 +59,7 @@ cpp_library(
         "//fboss/agent/platforms/common/wedge400:wedge400_platform_mapping",
         "//fboss/agent/platforms/common/wedge400:wedge400_platform_utils",
         "//fboss/agent/platforms/common/wedge400c:wedge400c_platform_mapping",
-        "//fboss/agent/platforms/common/wedge800ba:wedge800ba_platform_mapping",
+        "//fboss/agent/platforms/common/wedge800bact:wedge800bact_platform_mapping",
         "//fboss/agent/platforms/common/wedge800ca:wedge800ca_platform_mapping",
         "//fboss/agent/platforms/common/yamp:yamp_platform_mapping",
         "//fboss/agent/platforms/common/yangra:yangra_platform_mapping",
diff --git a/fboss/agent/platforms/common/PlatformMappingUtils.cpp b/fboss/agent/platforms/common/PlatformMappingUtils.cpp
@@ -44,7 +44,7 @@
 #include "fboss/agent/platforms/common/wedge400c/Wedge400CGrandTetonPlatformMapping.h"
 #include "fboss/agent/platforms/common/wedge400c/Wedge400CPlatformMapping.h"
 #include "fboss/agent/platforms/common/wedge400c/Wedge400CPlatformUtil.h"
-#include "fboss/agent/platforms/common/wedge800ba/Wedge800baPlatformMapping.h"
+#include "fboss/agent/platforms/common/wedge800bact/Wedge800BACTPlatformMapping.h"
 #include "fboss/agent/platforms/common/wedge800ca/Wedge800caPlatformMapping.h"
 #include "fboss/agent/platforms/common/yamp/YampPlatformMapping.h"
 #include "fboss/agent/platforms/common/yangra/YangraPlatformMapping.h"
@@ -189,10 +189,10 @@ std::unique_ptr<PlatformMapping> initPlatformMapping(PlatformType type) {
       return platformMappingStr.empty()
           ? std::make_unique<Icecube800bcPlatformMapping>()
           : std::make_unique<Icecube800bcPlatformMapping>(platformMappingStr);
-    case PlatformType::PLATFORM_WEDGE800B_ACT:
+    case PlatformType::PLATFORM_WEDGE800BACT:
       return platformMappingStr.empty()
-          ? std::make_unique<Wedge800baPlatformMapping>()
-          : std::make_unique<Wedge800baPlatformMapping>(platformMappingStr);
+          ? std::make_unique<Wedge800BACTPlatformMapping>()
+          : std::make_unique<Wedge800BACTPlatformMapping>(platformMappingStr);
     case PlatformType::PLATFORM_ICETEA800BC:
       return platformMappingStr.empty()
           ? std::make_unique<Icetea800bcPlatformMapping>()
diff --git a/fboss/agent/platforms/common/wedge800bact/BUCK b/fboss/agent/platforms/common/wedge800bact/BUCK
@@ -3,12 +3,12 @@ load("@fbcode_macros//build_defs:cpp_library.bzl", "cpp_library")
 oncall("fboss_agent_push")
 
 cpp_library(
-    name = "wedge800ba_platform_mapping",
+    name = "wedge800bact_platform_mapping",
     srcs = [
-        "Wedge800baPlatformMapping.cpp",
+        "Wedge800BACTPlatformMapping.cpp",
     ],
     headers = [
-        "Wedge800baPlatformMapping.h",
+        "Wedge800BACTPlatformMapping.h",
     ],
     exported_deps = [
         "//fboss/agent/platforms/common:platform_mapping",
diff --git a/fboss/agent/platforms/common/wedge800bact/Wedge800BACTPlatformMapping.cpp b/fboss/agent/platforms/common/wedge800bact/Wedge800BACTPlatformMapping.cpp
@@ -8,17 +8,17 @@
  *
  */
 
-#include "fboss/agent/platforms/common/wedge800ba/Wedge800baPlatformMapping.h"
+#include "fboss/agent/platforms/common/wedge800bact/Wedge800BACTPlatformMapping.h"
 
 namespace {
 constexpr auto kJsonPlatformMappingStr = R"()";
 } // namespace
 
 namespace facebook::fboss {
-Wedge800baPlatformMapping::Wedge800baPlatformMapping()
+Wedge800BACTPlatformMapping::Wedge800BACTPlatformMapping()
     : PlatformMapping(kJsonPlatformMappingStr) {}
 
-Wedge800baPlatformMapping::Wedge800baPlatformMapping(
+Wedge800BACTPlatformMapping::Wedge800BACTPlatformMapping(
     const std::string& platformMappingStr)
     : PlatformMapping(platformMappingStr) {}
 
diff --git a/fboss/agent/platforms/common/wedge800bact/Wedge800BACTPlatformMapping.h b/fboss/agent/platforms/common/wedge800bact/Wedge800BACTPlatformMapping.h
@@ -13,15 +13,15 @@
 
 namespace facebook::fboss {
 
-class Wedge800baPlatformMapping : public PlatformMapping {
+class Wedge800BACTPlatformMapping : public PlatformMapping {
  public:
-  Wedge800baPlatformMapping();
-  explicit Wedge800baPlatformMapping(const std::string& platformMappingStr);
+  Wedge800BACTPlatformMapping();
+  explicit Wedge800BACTPlatformMapping(const std::string& platformMappingStr);
 
  private:
   // Forbidden copy constructor and assignment operator
-  Wedge800baPlatformMapping(Wedge800baPlatformMapping const&) = delete;
-  Wedge800baPlatformMapping& operator=(Wedge800baPlatformMapping const&) =
+  Wedge800BACTPlatformMapping(Wedge800BACTPlatformMapping const&) = delete;
+  Wedge800BACTPlatformMapping& operator=(Wedge800BACTPlatformMapping const&) =
       delete;
 };
 } // namespace facebook::fboss
diff --git a/fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatform.cpp b/fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatform.cpp
@@ -8,37 +8,38 @@
  *
  */
 
-#include "fboss/agent/platforms/sai/SaiBcmWedge800baPlatform.h"
+#include "fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatform.h"
 
 #include "fboss/agent/hw/switch_asics/Tomahawk5Asic.h"
-#include "fboss/agent/platforms/common/wedge800ba/Wedge800baPlatformMapping.h"
+#include "fboss/agent/platforms/common/wedge800bact/Wedge800BACTPlatformMapping.h"
 
 #include <cstring>
 namespace facebook::fboss {
 
-SaiBcmWedge800baPlatform::SaiBcmWedge800baPlatform(
+SaiBcmWedge800BACTPlatform::SaiBcmWedge800BACTPlatform(
     std::unique_ptr<PlatformProductInfo> productInfo,
     folly::MacAddress localMac,
     const std::string& platformMappingStr)
     : SaiBcmPlatform(
           std::move(productInfo),
           platformMappingStr.empty()
-              ? std::make_unique<Wedge800baPlatformMapping>()
-              : std::make_unique<Wedge800baPlatformMapping>(platformMappingStr),
+              ? std::make_unique<Wedge800BACTPlatformMapping>()
+              : std::make_unique<Wedge800BACTPlatformMapping>(
+                    platformMappingStr),
           localMac) {}
 
-void SaiBcmWedge800baPlatform::setupAsic(
+void SaiBcmWedge800BACTPlatform::setupAsic(
     std::optional<int64_t> switchId,
     const cfg::SwitchInfo& switchInfo,
     std::optional<HwAsic::FabricNodeRole> fabricNodeRole) {
   CHECK(!fabricNodeRole.has_value());
   asic_ = std::make_unique<Tomahawk5Asic>(switchId, switchInfo);
 }
 
-HwAsic* SaiBcmWedge800baPlatform::getAsic() const {
+HwAsic* SaiBcmWedge800BACTPlatform::getAsic() const {
   return asic_.get();
 }
 
-SaiBcmWedge800baPlatform::~SaiBcmWedge800baPlatform() = default;
+SaiBcmWedge800BACTPlatform::~SaiBcmWedge800BACTPlatform() = default;
 
 } // namespace facebook::fboss
diff --git a/fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatform.h b/fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatform.h
@@ -15,13 +15,13 @@ namespace facebook::fboss {
 
 class Tomahawk5Asic;
 
-class SaiBcmWedge800baPlatform : public SaiBcmPlatform {
+class SaiBcmWedge800BACTPlatform : public SaiBcmPlatform {
  public:
-  SaiBcmWedge800baPlatform(
+  SaiBcmWedge800BACTPlatform(
       std::unique_ptr<PlatformProductInfo> productInfo,
       folly::MacAddress localMac,
       const std::string& platformMappingStr);
-  ~SaiBcmWedge800baPlatform() override;
+  ~SaiBcmWedge800BACTPlatform() override;
   HwAsic* getAsic() const override;
   uint32_t numLanesPerCore() const override {
     return 8;
diff --git a/fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatformPort.cpp b/fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatformPort.cpp
@@ -7,17 +7,18 @@
  *  of patent rights can be found in the PATENTS file in the same directory.
  *
  */
-#include "fboss/agent/platforms/sai/SaiBcmWedge800baPlatformPort.h"
+#include "fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatformPort.h"
 
 namespace facebook::fboss {
 
-void SaiBcmWedge800baPlatformPort::linkStatusChanged(
+void SaiBcmWedge800BACTPlatformPort::linkStatusChanged(
     bool /*up*/,
     bool /*adminUp*/) {
   // TODO: set led color
 }
 
-void SaiBcmWedge800baPlatformPort::externalState(PortLedExternalState /*lfs*/) {
+void SaiBcmWedge800BACTPlatformPort::externalState(
+    PortLedExternalState /*lfs*/) {
   // TODO: set led color
 }
 
diff --git a/fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatformPort.h b/fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatformPort.h
@@ -13,9 +13,9 @@
 
 namespace facebook::fboss {
 
-class SaiBcmWedge800baPlatformPort : public SaiBcmPlatformPort {
+class SaiBcmWedge800BACTPlatformPort : public SaiBcmPlatformPort {
  public:
-  SaiBcmWedge800baPlatformPort(PortID id, SaiPlatform* platform)
+  SaiBcmWedge800BACTPlatformPort(PortID id, SaiPlatform* platform)
       : SaiBcmPlatformPort(id, platform) {}
   void linkStatusChanged(bool up, bool adminUp) override;
   void externalState(PortLedExternalState lfs) override;
diff --git a/fboss/agent/platforms/sai/SaiPlatform.cpp b/fboss/agent/platforms/sai/SaiPlatform.cpp
@@ -26,7 +26,7 @@
 #include "fboss/agent/platforms/sai/SaiBcmTahansb800bcPlatformPort.h"
 #include "fboss/agent/platforms/sai/SaiBcmWedge100PlatformPort.h"
 #include "fboss/agent/platforms/sai/SaiBcmWedge400PlatformPort.h"
-#include "fboss/agent/platforms/sai/SaiBcmWedge800baPlatformPort.h"
+#include "fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatformPort.h"
 #include "fboss/agent/platforms/sai/SaiBcmYampPlatformPort.h"
 #include "fboss/agent/platforms/sai/SaiElbert8DDPhyPlatformPort.h"
 #include "fboss/agent/platforms/sai/SaiFakePlatformPort.h"
@@ -400,8 +400,8 @@ void SaiPlatform::initPorts() {
       saiPort = std::make_unique<SaiBcmIcetea800bcPlatformPort>(portId, this);
     } else if (platformMode == PlatformType::PLATFORM_LADAKH800BCLS) {
       saiPort = std::make_unique<SaiBcmLadakh800bclsPlatformPort>(portId, this);
-    } else if (platformMode == PlatformType::PLATFORM_WEDGE800B_ACT) {
-      saiPort = std::make_unique<SaiBcmWedge800baPlatformPort>(portId, this);
+    } else if (platformMode == PlatformType::PLATFORM_WEDGE800BACT) {
+      saiPort = std::make_unique<SaiBcmWedge800BACTPlatformPort>(portId, this);
     } else if (platformMode == PlatformType::PLATFORM_WEDGE800C_ACT) {
       saiPort = std::make_unique<SaiWedge800caPlatformPort>(portId, this);
     } else if (platformMode == PlatformType::PLATFORM_TAHANSB800BC) {
diff --git a/fboss/agent/platforms/sai/SaiPlatformInit.cpp b/fboss/agent/platforms/sai/SaiPlatformInit.cpp
@@ -26,7 +26,7 @@
 #include "fboss/agent/platforms/sai/SaiBcmTahansb800bcPlatform.h"
 #include "fboss/agent/platforms/sai/SaiBcmWedge100Platform.h"
 #include "fboss/agent/platforms/sai/SaiBcmWedge400Platform.h"
-#include "fboss/agent/platforms/sai/SaiBcmWedge800baPlatform.h"
+#include "fboss/agent/platforms/sai/SaiBcmWedge800BACTPlatform.h"
 #include "fboss/agent/platforms/sai/SaiBcmYampPlatform.h"
 #include "fboss/agent/platforms/sai/SaiFakePlatform.h"
 #include "fboss/agent/platforms/sai/SaiJanga800bicPlatform.h"
@@ -125,8 +125,8 @@ std::unique_ptr<SaiPlatform> chooseSaiPlatform(
   } else if (productInfo->getType() == PlatformType::PLATFORM_LADAKH800BCLS) {
     return std::make_unique<SaiBcmLadakh800bclsPlatform>(
         std::move(productInfo), localMac, platformMappingStr);
-  } else if (productInfo->getType() == PlatformType::PLATFORM_WEDGE800B_ACT) {
-    return std::make_unique<SaiBcmWedge800baPlatform>(
+  } else if (productInfo->getType() == PlatformType::PLATFORM_WEDGE800BACT) {
+    return std::make_unique<SaiBcmWedge800BACTPlatform>(
         std::move(productInfo), localMac, platformMappingStr);
   } else if (productInfo->getType() == PlatformType::PLATFORM_WEDGE800C_ACT) {
     return std::make_unique<SaiWedge800caPlatform>(
diff --git a/fboss/lib/if/fboss_common.thrift b/fboss/lib/if/fboss_common.thrift
@@ -46,7 +46,7 @@ enum PlatformType {
   PLATFORM_MINIPACK3N = 38,
   PLATFORM_ICECUBE800BC = 39,
   PLATFORM_MERU800BIAC = 40,
-  PLATFORM_WEDGE800B_ACT = 41,
+  PLATFORM_WEDGE800BACT = 41,
   PLATFORM_ICETEA800BC = 42,
   PLATFORM_TAHANSB800BC = 43,
   PLATFORM_WEDGE800C_ACT = 44,
diff --git a/fboss/lib/platform_mapping_v2/generated_platform_mappings/wedge800bact_platform_mapping.json b/fboss/lib/platform_mapping_v2/generated_platform_mappings/wedge800bact_platform_mapping.json
diff --git a/fboss/lib/platform_mapping_v2/platforms/wedge800bact/wedge800bact_port_profile_mapping.csv b/fboss/lib/platform_mapping_v2/platforms/wedge800bact/wedge800bact_port_profile_mapping.csv
diff --git a/fboss/lib/platform_mapping_v2/platforms/wedge800bact/wedge800bact_profile_settings.csv b/fboss/lib/platform_mapping_v2/platforms/wedge800bact/wedge800bact_profile_settings.csv
diff --git a/fboss/lib/platform_mapping_v2/platforms/wedge800bact/wedge800bact_si_settings.csv b/fboss/lib/platform_mapping_v2/platforms/wedge800bact/wedge800bact_si_settings.csv
diff --git a/fboss/lib/platform_mapping_v2/platforms/wedge800bact/wedge800bact_static_mapping.csv b/fboss/lib/platform_mapping_v2/platforms/wedge800bact/wedge800bact_static_mapping.csv
diff --git a/fboss/lib/platform_mapping_v2/platforms/wedge800bact/wedge800bact_vendor_config.json b/fboss/lib/platform_mapping_v2/platforms/wedge800bact/wedge800bact_vendor_config.json
diff --git a/fboss/lib/platform_mapping_v2/test/verify_generated_files.py b/fboss/lib/platform_mapping_v2/test/verify_generated_files.py
@@ -44,7 +44,7 @@ class TestVerifyPlatformMappingGeneratedFiles(unittest.TestCase):
             "icecube800bc",
             "icetea800bc",
             "tahansb800bc",
-            "wedge800ba",
+            "wedge800bact",
         ],
         True: [
             "meru800bfa",
diff --git a/fboss/lib/platforms/PlatformMode.h b/fboss/lib/platforms/PlatformMode.h
@@ -100,8 +100,8 @@ inline std::string toString(PlatformType mode) {
       return "MINIPACK3N";
     case PlatformType::PLATFORM_ICECUBE800BC:
       return "ICECUBE";
-    case PlatformType::PLATFORM_WEDGE800B_ACT:
-      return "WEDGE800BA";
+    case PlatformType::PLATFORM_WEDGE800BACT:
+      return "WEDGE800BACT";
     case PlatformType::PLATFORM_ICETEA800BC:
       return "ICETEA";
     case PlatformType::PLATFORM_TAHANSB800BC:
diff --git a/fboss/lib/platforms/PlatformProductInfo.cpp b/fboss/lib/platforms/PlatformProductInfo.cpp
@@ -79,6 +79,10 @@ void PlatformProductInfo::initMode() {
     } else if (
         modelName.find("Wedge400") == 0 || modelName.find("WEDGE400") == 0) {
       type_ = PlatformType::PLATFORM_WEDGE400;
+    } else if (
+        modelName.find("Wedge800BACT") == 0 ||
+        modelName.find("WEDGE800BACT") == 0) {
+      type_ = PlatformType::PLATFORM_WEDGE800BACT;
     } else if (modelName.find("DARWIN48V") == 0) {
       type_ = PlatformType::PLATFORM_DARWIN48V;
     } else if (
@@ -190,8 +194,6 @@ void PlatformProductInfo::initMode() {
       type_ = PlatformType::PLATFORM_TAHAN800BC;
     } else if (modelName.find("ICECUBE") == 0) {
       type_ = PlatformType::PLATFORM_ICECUBE800BC;
-    } else if (modelName.find("WEDGE800B_ACT") == 0) {
-      type_ = PlatformType::PLATFORM_WEDGE800B_ACT;
     } else if (modelName.find("ICETEA") == 0) {
       type_ = PlatformType::PLATFORM_ICETEA800BC;
     } else if (modelName.find("WEDGE800C_ACT") == 0) {
@@ -266,8 +268,8 @@ void PlatformProductInfo::initMode() {
       type_ = PlatformType::PLATFORM_YANGRA;
     } else if (FLAGS_mode == "minipack3n") {
       type_ = PlatformType::PLATFORM_MINIPACK3N;
-    } else if (FLAGS_mode == "wedge800ba") {
-      type_ = PlatformType::PLATFORM_WEDGE800B_ACT;
+    } else if (FLAGS_mode == "wedge800bact") {
+      type_ = PlatformType::PLATFORM_WEDGE800BACT;
     } else if (FLAGS_mode == "tahansb800bc") {
       type_ = PlatformType::PLATFORM_TAHANSB800BC;
     } else if (FLAGS_mode == "wedge800ca") {
