Skip to content

Ladakh800bcls: fix exception 'Update : update fib application to HW f…#913

Open
gang-tao wants to merge 1 commit intofacebook:mainfrom
gang-tao:ladakh800bcls_agent_fix4
Open

Ladakh800bcls: fix exception 'Update : update fib application to HW f…#913
gang-tao wants to merge 1 commit intofacebook:mainfrom
gang-tao:ladakh800bcls_agent_fix4

Conversation

@gang-tao
Copy link
Contributor

@gang-tao gang-tao commented Feb 8, 2026

…ailed' in test case
Pre-submission checklist

  • [✓ ] I've ran the linters locally and fixed lint errors related to the files I modified in this PR. You can install the linters by running pip install -r requirements-dev.txt && pre-commit install
  • [✓ ] pre-commit run

Summary

test case failed and report:
C++ exception with description "Update : update fib application to HW failed" thrown in the test body.

Motivation

The ports/interfaces generated by the test case are assigned to a specific NPU, but the routing is global. This means there is a route on one NPU that is not associated with the specific interface. To avoid abnormal exits, the route must be dropped—even for non-connected routes.

Test Plan

command

for filter in AgentL3ForwardingTest.linkLocalNeighborAndNextHop AgentL3ForwardingTest.ttl255 AgentCoppTest/0.UnresolvedRoutesToLowPriQueue AgentPacketSendTest.PortTxEnableTest AgentDscpQueueMappingTest.VerifyDscpQueueMapping AgentDscpQueueMappingTest.VerifyDscpQueueMappingChangePostWarmboot AgentDscpQueueMappingTest.VerifyDscpQueueMappingCanaryOff AgentAclAndDscpQueueMappingTest.VerifyAclAndQosMap AgentAclConflictAndDscpQueueMappingTest.VerifyAclAndQosMapConflict AgentOlympicQosTests.VerifyDscpQueueMapping AgentNetworkAIQosTests.VerifyDscpQueueMapping AgentPortBandwidthPpsTest.VerifyPps AgentPortBandwidthPpsTest.VerifyPpsDynamicChanges AgentSendPacketToQueueTest.SendPacketSwitchedToDefaultUCQueue ; do
  LD_LIBRARY_PATH=/opt/fboss/lib/ /root/gangtao/img/multi_switch_agent_hw_test \
  --gtest_filter=${filter} \
  --config /root/gangtao/cfg/link_test_config  \
  --platform_mapping_override_path /root/gangtao/cfg/ladakh800bc_platform_mapping_is_multi_npu.json-no-65 \
  --multi_npu_platform_mapping \
  --switch_id_for_testing 1 \
  --logging=DBG5 \
  2>&1 | tee /root/gangtao/log/multi_switch_agent-${TS}.log
done

result

[       OK ] AgentL3ForwardingTest.linkLocalNeighborAndNextHop (557381 ms)
[       OK ] AgentL3ForwardingTest.ttl255 (813313 ms)
[       OK ] AgentCoppTest/0.UnresolvedRoutesToLowPriQueue (546300 ms)
[       OK ] AgentPacketSendTest.PortTxEnableTest (483655 ms)
[       OK ] AgentDscpQueueMappingTest.VerifyDscpQueueMapping (596757 ms)
[       OK ] AgentDscpQueueMappingTest.VerifyDscpQueueMappingChangePostWarmboot (568970 ms)
[       OK ] AgentDscpQueueMappingTest.VerifyDscpQueueMappingCanaryOff (700448 ms)
[       OK ] AgentAclAndDscpQueueMappingTest.VerifyAclAndQosMap (1012629 ms)
[       OK ] AgentAclConflictAndDscpQueueMappingTest.VerifyAclAndQosMapConflict (696069 ms)
[       OK ] AgentOlympicQosTests.VerifyDscpQueueMapping (949254 ms)
[       OK ] AgentNetworkAIQosTests.VerifyDscpQueueMapping (514485 ms)
[       OK ] AgentPortBandwidthPpsTest.VerifyPps (1018090 ms)
[       OK ] AgentSendPacketToQueueTest.SendPacketSwitchedToDefaultUCQueue (847839 ms)
[       OK ] AgentPortBandwidthPpsTest.VerifyPpsDynamicChanges (785661 ms)

@meta-cla meta-cla bot added the CLA Signed label Feb 8, 2026
@meta-codesync
Copy link

meta-codesync bot commented Feb 10, 2026

@arunsarat has imported this pull request. If you are a Meta employee, you can view this in D92861125.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant