Skip to content

[Cisco][Srv6] Decap warmboot related fix for Srv6 #1049

Open
itskittycisco wants to merge 1 commit intofacebook:mainfrom
itskittycisco:srv6_decap_trace_fix
Open

[Cisco][Srv6] Decap warmboot related fix for Srv6 #1049
itskittycisco wants to merge 1 commit intofacebook:mainfrom
itskittycisco:srv6_decap_trace_fix

Conversation

@itskittycisco
Copy link
Copy Markdown

@itskittycisco itskittycisco commented Mar 29, 2026

Exclude MY_SID_ENTRY from should_log() so sai_get_object_key tracing does not treat structured My SID keys as object_id (avoids tracer aborts).

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
    git sclang-format.............................................................tPassed
    shellcheck...........................................(no files to check)Skipped
    shfmt................................................(no files to check)Skipped
    trim trailing whitespace.................................................atPassed
    fix end of files.........................................................Passed
    check yaml...........................................(no files to check)Skipped
    check json...........................................(no files to check)Skipped
    check for merge conflicts................................................sPassed
    ruff check...........................................(no files to check)Skipped
    ruff format..........................................(no files to check)Skipped

Summary

Test Plan

…does not treat structured My SID keys as object_id (avoids tracer aborts).
@itskittycisco itskittycisco requested a review from a team as a code owner March 29, 2026 17:33
@meta-cla meta-cla bot added the CLA Signed label Mar 29, 2026
@github-actions github-actions bot added the cisco label Mar 29, 2026
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