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

README for Flow_matching_using_ACL_and_Portmirror #3535

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

vvlakshmanamurthy
Copy link
Contributor

@vvlakshmanamurthy vvlakshmanamurthy commented Oct 22, 2024

This test is to verify flow matching using ACL and to port mirror or redirect the traffic.

This test is to verify flow matching using CL and to port mirror or redirect the traffic.
@OpenConfigBot
Copy link

OpenConfigBot commented Oct 22, 2024

Pull Request Functional Test Report for #3535 / 17147f4

No tests identified for validation.

Help

Updated the test case with few more details.
@dplore
Copy link
Member

dplore commented Oct 25, 2024

Recommend moving this to the /feature/qos folder and @sezhang2 review this.

@coveralls
Copy link

coveralls commented Oct 25, 2024

Pull Request Test Coverage Report for Build 11577397807

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

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

💛 - Coveralls

@vvlakshmanamurthy vvlakshmanamurthy changed the title Create README.md README for Interface based ARP policer Oct 29, 2024
Copy link
Contributor Author

@vvlakshmanamurthy vvlakshmanamurthy left a comment

Choose a reason for hiding this comment

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

updated the README and addressed the comments.

@vvlakshmanamurthy vvlakshmanamurthy changed the title README for Interface based ARP policer README for Flow_matching_using_ACL_and_Portmirror Oct 29, 2024
@sezhang2
Copy link
Contributor

sezhang2 commented Dec 4, 2024

Please fix the check error

Run go install ./tools/validate_readme_spec
########## READMEs in changed directories to be validated (including ones to be exempted):
feature/acl/Flow_matching_using_ACL_and_Portmirror/README.md
########## Validating READMEs in changed directories:
I1029 21:14:54.484407 4339 validate_readme_spec.go:149] Validating "feature/acl/Flow_matching_using_ACL_and_Portmirror/README.md"
E1029 21:14:54.485008 4339 validate_readme_spec.go:156] file feature/acl/Flow_matching_using_ACL_and_Portmirror/README.md: mdocspec: error parsing YAML: yaml: line 4: mapping values are not allowed in this context
F1029 21:14:54.485308 4339 validate_readme_spec.go:178] The following files have errors:
feature/acl/Flow_matching_using_ACL_and_Portmirror/README.md
Error: Process completed with exit code 1.

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