Skip to content

Commit e707ae4

Browse files
Name fake-SAI agent binaries with -sai_impl suffix
The BUILD_SAI_FAKE arm now builds the agents with the -sai_impl suffix (and is made mutually exclusive with the real-SAI SAI_IMPL path via elseif), and fboss_fake_agent_targets depends on wedge_agent-sai_impl / fboss_hw_agent-sai_impl. This lands fake-SAI builds at the /opt/fboss/bin paths the systemd units and package.py already reference, so the fboss-sim runtime no longer needs the -fake -> -sai_impl symlinks. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent bdf9a08 commit e707ae4

2 files changed

Lines changed: 7 additions & 6 deletions

File tree

CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1341,8 +1341,8 @@ add_dependencies(fboss_fake_agent_targets
13411341
fboss_agent_thrift_libs
13421342
fboss_sw_agent
13431343
sai_replayer-fake
1344-
wedge_agent-fake
1345-
fboss_hw_agent-fake
1344+
wedge_agent-sai_impl
1345+
fboss_hw_agent-sai_impl
13461346
)
13471347

13481348
add_custom_target(fboss_fake_agent_test_targets)

cmake/AgentPlatformsSai.cmake

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -206,10 +206,11 @@ find_library(SAI_IMPL sai_impl)
206206
message(STATUS "SAI_IMPL: ${SAI_IMPL}")
207207

208208
if(BUILD_SAI_FAKE)
209-
BUILD_SAI_WEDGE_AGENT("fake" fake_sai)
210-
endif()
211-
212-
if(SAI_IMPL)
209+
# Name fake-SAI agent binaries with the -sai_impl suffix so they sit at
210+
# the same /opt/fboss/bin/ paths the systemd unit files reference and
211+
# package.py FORWARDING_BINARIES matches without modification.
212+
BUILD_SAI_WEDGE_AGENT("sai_impl" fake_sai)
213+
elseif(SAI_IMPL)
213214
BUILD_SAI_WEDGE_AGENT("sai_impl" ${SAI_IMPL})
214215
install(
215216
TARGETS

0 commit comments

Comments
 (0)