Skip to content

[Accton][minipack3bta] Fix minipack3bta agent config: add sai_eapol_trap_use_bcast_mac#1349

Open
brandonchen-accton wants to merge 1 commit into
facebook:mainfrom
brandonchen-accton:fix-minipack3bta-agent-config-eapol
Open

[Accton][minipack3bta] Fix minipack3bta agent config: add sai_eapol_trap_use_bcast_mac#1349
brandonchen-accton wants to merge 1 commit into
facebook:mainfrom
brandonchen-accton:fix-minipack3bta-agent-config-eapol

Conversation

@brandonchen-accton

@brandonchen-accton brandonchen-accton commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Summary

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
clang-format.........................................(no files to check)Skipped
shellcheck...........................................(no files to check)Skipped
shfmt................................................(no files to check)Skipped
trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check yaml...........................................(no files to check)Skipped
check json...........................................(no files to check)Skipped
check for merge conflicts................................................Passed
ruff check...........................................(no files to check)Skipped
ruff format..........................................(no files to check)Skipped
Prevent sai_impl in fboss manifest.......................................Passed

Test Plan

In the test AgentCoppEapolTest/0.EapolToHighPriQ, EAPOL packets were not being trapped to the CPU. To resolve this, we enabled sai_eapol_trap_use_bcast_mac=1 in the SDK to support trapping broadcast MAC EAPOL packets to the CPU.

@brandonchen-accton brandonchen-accton requested a review from a team as a code owner June 30, 2026 10:48
@meta-cla meta-cla Bot added the CLA Signed label Jun 30, 2026
@meta-codesync

meta-codesync Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

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

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

1 participant