Skip to content
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

DP-2.4 Police traffic on input interface matching all packets using 1 rate, 2 color marker. #3385

Merged
merged 23 commits into from
Oct 15, 2024

Conversation

vishnureddybadveli
Copy link
Contributor

@vishnureddybadveli vishnureddybadveli commented Aug 19, 2024

Adding test case for one rate, two color policier matching on input interface/sub-interface.

dplore and others added 3 commits August 19, 2024 09:58
Adding test case for vanilla policier per interface/sub-interface support that exists on vendors today
@OpenConfigBot
Copy link

OpenConfigBot commented Aug 19, 2024

Pull Request Functional Test Report for #3385 / 2c58976

No tests identified for validation.

Help

.DS_Store Outdated Show resolved Hide resolved
feature/.DS_Store Outdated Show resolved Hide resolved
feature/qos/.DS_Store Outdated Show resolved Hide resolved
testregistry.textproto Outdated Show resolved Hide resolved
feature/qos/otg_tests/ingress_police_nhg/README.md Outdated Show resolved Hide resolved
@coveralls
Copy link

coveralls commented Aug 20, 2024

Pull Request Test Coverage Report for Build 11282194573

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 55.268%

Totals Coverage Status
Change from base Build 11278339037: 0.0%
Covered Lines: 1983
Relevant Lines: 3588

💛 - Coveralls

@vishnureddybadveli vishnureddybadveli changed the title Test case for vanilla policier per interface/sub-interface support Classify on input MAC and police using 1 rate, 2 color marker. Aug 20, 2024
.DS_Store Outdated Show resolved Hide resolved
feature/qos/otg_tests/ingress_police_nhg/README.md Outdated Show resolved Hide resolved
feature/qos/otg_tests/ingress_police_nhg/README.md Outdated Show resolved Hide resolved
feature/qos/otg_tests/ingress_police_nhg/README.md Outdated Show resolved Hide resolved
feature/qos/otg_tests/ingress_police_nhg/README.md Outdated Show resolved Hide resolved
feature/qos/otg_tests/ingress_police_nhg/README.md Outdated Show resolved Hide resolved
feature/qos/otg_tests/ingress_police_nhg/README.md Outdated Show resolved Hide resolved
feature/qos/otg_tests/ingress_police_nhg/README.md Outdated Show resolved Hide resolved
@dplore dplore changed the title Classify on input MAC and police using 1 rate, 2 color marker. TE-18.4 Classify on input MAC and police using 1 rate, 2 color marker. Aug 22, 2024
@vishnureddybadveli vishnureddybadveli changed the title TE-18.4 Classify on input MAC and police using 1 rate, 2 color marker. TE-18.4 Classify on input matching all and police using 1 rate, 2 color marker. Oct 3, 2024
@dplore dplore self-assigned this Oct 8, 2024
@dplore
Copy link
Member

dplore commented Oct 8, 2024

Please refresh the canonical OC for policer configuration to match the style in DP-2.2

(you can of course remove the classifier terms, but please do use the forwarding-groups and queue configuration)

@vishnureddybadveli vishnureddybadveli changed the title TE-18.4 Classify on input matching all and police using 1 rate, 2 color marker. DP-2.4 Police traffic on input interface matching all packets using 1 rate, 2 color marker. Oct 8, 2024
@vishnureddybadveli
Copy link
Contributor Author

Please refresh the canonical OC for policer configuration to match the style in DP-2.2

(you can of course remove the classifier terms, but please do use the forwarding-groups and queue configuration)

Yes refreshed with the canonical OC to match the style in DP-2.2 and removed classifier terms, in order to avoid any confusion as DP-2.2 is still been in the works and to decouple with DP-2.2 I had refrained from using any queue configurations, forwarding-groups.

feature/qos/otg_tests/ingress_police_default/README.md Outdated Show resolved Hide resolved
feature/qos/otg_tests/ingress_police_default/README.md Outdated Show resolved Hide resolved
testregistry.textproto Outdated Show resolved Hide resolved
Added validation criteria for packets, specific rate.
dplore
dplore previously approved these changes Oct 10, 2024
.DS_Store Outdated Show resolved Hide resolved
@dplore dplore merged commit 9dedc7a into openconfig:main Oct 15, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants