Skip to content

Commit

Permalink
Add PRIORITY_GROUP to MultipleEventIDs test
Browse files Browse the repository at this point in the history
Summary: Add PRIORITY_GROUP to MultipleEventIDs test to reproduce CS00012388701

Reviewed By: nivinl

Differential Revision: D68540802

fbshipit-source-id: 8f8f1687d2f4d9070df0f6398373249b08b6e491
  • Loading branch information
maxwindiff authored and facebook-github-bot committed Jan 23, 2025
1 parent 05d71c8 commit d4d23fe
Showing 1 changed file with 14 additions and 2 deletions.
16 changes: 14 additions & 2 deletions fboss/agent/test/agent_hw_tests/AgentMirrorOnDropTests.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -440,7 +440,9 @@ TEST_F(AgentMirrorOnDropTest, MultipleEventIDs) {
&config,
mirrorPortId,
kCollectorIp_,
// Sandwich PP drops between other reasons and see if it still works
// Sandwich PP drops between other reasons and see if it still works.
// Ensure all aging group tests are tested.
// Also make sure to test multiple reason aggregations in one event.
{
{0,
makeEventConfig(
Expand All @@ -451,12 +453,22 @@ TEST_F(AgentMirrorOnDropTest, MultipleEventIDs) {
makeEventConfig(
cfg::MirrorOnDropAgingGroup::PORT,
{cfg::MirrorOnDropReasonAggregation::
INGRESS_PACKET_PROCESSING_DISCARDS})},
INGRESS_PACKET_PROCESSING_DISCARDS,
cfg::MirrorOnDropReasonAggregation::
INGRESS_QUEUE_RESOLUTION_DISCARDS})},
{2,
makeEventConfig(
cfg::MirrorOnDropAgingGroup::VOQ,
{cfg::MirrorOnDropReasonAggregation::
INGRESS_DESTINATION_CONGESTION_DISCARDS})},
{3,
makeEventConfig(
cfg::MirrorOnDropAgingGroup::PRIORITY_GROUP,
{
cfg::MirrorOnDropReasonAggregation::INGRESS_MISC_DISCARDS,
cfg::MirrorOnDropReasonAggregation::
UNEXPECTED_REASON_DISCARDS,
})},
});
utility::addTrapPacketAcl(&config, kCollectorNextHopMac_);
applyNewConfig(config);
Expand Down

0 comments on commit d4d23fe

Please sign in to comment.