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

Update prefixlimit.feature.textproto #1754

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

sachendras
Copy link
Contributor

Add missing OC paths in the textproto based on the feature requests in the various bugs from the catalog.

Add missing OC paths in the textproto based on the feature requests in the various bugs from the catalog.
@sachendras sachendras requested review from a team as code owners June 14, 2023 04:58
@OpenConfigBot
Copy link

Pull Request Functional Test Report for #1754 / 2b63fb3

No tests identified for validation.

Help

@sachendras sachendras requested a review from bensons June 14, 2023 04:59
@github-actions
Copy link

Pull Request Test Coverage Report for Build 5263215301

  • 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 47.722%

Totals Coverage Status
Change from base Build 5261551616: 0.0%
Covered Lines: 1320
Relevant Lines: 2766

💛 - Coveralls

@dplore
Copy link
Member

dplore commented Jul 25, 2023

/gcbrun

Copy link
Member

@dplore dplore left a comment

Choose a reason for hiding this comment

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

These paths are present in

# max-prefix: IPv4 unicast, IPv6 unicast
.

They should be moved here as you have indicated. Can you also remove them from

# max-prefix: IPv4 unicast, IPv6 unicast
?

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.

3 participants