Skip to content

README for PF-1.4: Static GUEv1 Decapsulation over decap subnet range, decap TTL and DSCP behavior test #4178

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 32 commits into from
Jun 12, 2025

Conversation

nupkanoi
Copy link
Contributor

@nupkanoi nupkanoi commented May 2, 2025

No description provided.

@nupkanoi nupkanoi requested a review from a team as a code owner May 2, 2025 19:35
@OpenConfigBot
Copy link

OpenConfigBot commented May 2, 2025

Pull Request Functional Test Report for #4178 / 02051c5

No tests identified for validation.

Help

@nupkanoi nupkanoi requested a review from sachendras May 2, 2025 19:35
@coveralls
Copy link

coveralls commented May 2, 2025

Pull Request Test Coverage Report for Build 15600398896

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 17.685%

Totals Coverage Status
Change from base Build 15576610815: 0.0%
Covered Lines: 2481
Relevant Lines: 14029

💛 - Coveralls

sachendras
sachendras previously approved these changes May 3, 2025
@dplore dplore changed the title Static GUE decapsulation README README for PF-1.4: Static GUEv1 Decapsulation to IPv4 or IPv6 payload May 5, 2025
@nupkanoi nupkanoi requested review from sachendras and dplore May 6, 2025 16:32
sachendras
sachendras previously approved these changes May 6, 2025
@sachendras
Copy link
Contributor

/gcbrun

- Initiate traffic flow type#1
Verification:
- The outer header destination IP of the traffic falls within the configured decap range (DECAP-DST-SUBNET/30)
- The outer header destination port of the traffic (UDP 6080) matches the default UDP decap port criteria
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as noted above, there is no default in the OC model for this, the configuration must specify the destination port used

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nupkanoi nupkanoi requested a review from dplore May 7, 2025 21:35
Copy link
Contributor Author

@nupkanoi nupkanoi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replying to comments

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@nupkanoi nupkanoi requested a review from dplore May 31, 2025 01:06
nupkanoi and others added 10 commits June 10, 2025 17:57
…tic_decap_subnet_range/README.md

Co-authored-by: Rob Shakir <[email protected]>
…tic_decap_subnet_range/README.md

Co-authored-by: Rob Shakir <[email protected]>
…tic_decap_subnet_range/README.md

Co-authored-by: Rob Shakir <[email protected]>
…tic_decap_subnet_range/README.md

Co-authored-by: Rob Shakir <[email protected]>
…tic_decap_subnet_range/README.md

Co-authored-by: Rob Shakir <[email protected]>
…tic_decap_subnet_range/README.md

Co-authored-by: Rob Shakir <[email protected]>
…tic_decap_subnet_range/README.md

Co-authored-by: Rob Shakir <[email protected]>
…tic_decap_subnet_range/README.md

Co-authored-by: Rob Shakir <[email protected]>
@nupkanoi nupkanoi requested a review from robshakir June 11, 2025 00:21
@robshakir
Copy link
Contributor

/fptest run

@nupkanoi nupkanoi dismissed dplore’s stale review June 12, 2025 17:42

Darren is OOO and Rob Shakir reviewed and approved on behalf of openconfig/featureprofiles-approvers

@nupkanoi nupkanoi merged commit 4c478e4 into main Jun 12, 2025
16 checks passed
@nupkanoi nupkanoi deleted the decap_1 branch June 12, 2025 17:44
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.

7 participants