Skip to content

[Cisco][G200] SRv6 validation fixes#1002

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

[Cisco][G200] SRv6 validation fixes#1002
itskittycisco wants to merge 1 commit intofacebook:mainfrom
itskittycisco:srv6_test

Conversation

@itskittycisco
Copy link

@itskittycisco itskittycisco commented Mar 14, 2026

This change updates SRv6 encap tests to:
TTL mode: Use UNIFORM instead of PIPE in makeSrv6TunnelConfig() so the outer IPv6 hop limit is decremented by 1 (uniform as the inner pkt).
DSCP check: Add a check that the outer header traffic class copies the inner DSCP (upper 6 bits), using EXPECT_EQ(v6Hdr.trafficClass >> 2, kTc).

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.............................................................Passed
    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

Summary

Test Plan

This change updates SRv6 encap tests to:
TTL mode: Use UNIFORM instead of PIPE in makeSrv6TunnelConfig() so the outer IPv6 hop limit is decremented by 1 (uniform as the inner pkt).
DSCP check: Add a check that the outer header traffic class copies the inner DSCP (upper 6 bits), using EXPECT_EQ(v6Hdr.trafficClass >> 2, kTc).

Signed-off-by: Khiruthigai (Kitty) Balasubramanian <khbalasu@khbalasu-dev1.cisco.com>
@itskittycisco itskittycisco requested a review from a team as a code owner March 14, 2026 20:12
@meta-cla meta-cla bot added the CLA Signed label Mar 14, 2026
@itskittycisco itskittycisco changed the title SRv6 validation fixes [G200] [Cisco] SRv6 validation fixes Mar 14, 2026
@itskittycisco itskittycisco changed the title [G200] [Cisco] SRv6 validation fixes [Cisco][G200] SRv6 validation fixes Mar 20, 2026
@github-actions github-actions bot added the cisco label Mar 20, 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