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

Configuration to control ACL set/entry counter allocation #1122

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
37 changes: 35 additions & 2 deletions release/models/acl/openconfig-acl.yang
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,14 @@ module openconfig-acl {
packets should be handled. Entries have a type that indicates
the type of match criteria, e.g., MAC layer, IPv4, IPv6, etc.";

oc-ext:openconfig-version "1.3.3";
oc-ext:openconfig-version "1.3.4";

revision "2024-06-03" {
description
"Add configuration control for statistics collection per
ACL-set and per ACL entry.";
reference "1.3.4";
}

revision "2023-02-06" {
description
Expand Down Expand Up @@ -233,6 +240,12 @@ module openconfig-acl {
and reported per ACL entry.";
}

identity NONE {
base ACL_COUNTER_CAPABILITY;
description
"ACL counters are not incremented.";
}

// grouping statements

// input interface
Expand Down Expand Up @@ -405,6 +418,16 @@ module openconfig-acl {
Entry.";
}

leaf counter {
type identityref {
base ACL_COUNTER_CAPABILITY;
}
description
"Specify the counter resources that should be allocated to
the ACL entry. This setting overrides configuration at the
ACL set level.";
}

}

grouping access-list-entries-state {
Expand Down Expand Up @@ -527,6 +550,15 @@ module openconfig-acl {
"Description, or comment, for the ACL set";
}

leaf counter {
Copy link
Member

Choose a reason for hiding this comment

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

this leaf is not needed, as the grouping "access-list-entries-config" is used in both config and state containers.

type identityref {
base ACL_COUNTER_CAPABILITY;
}
description
"Specify the counter resources that should be allocated to
the ACL entries of the ACL set. This setting can be overridden
on a per ACL entry basis.";
}
}

grouping acl-set-state {
Expand Down Expand Up @@ -891,7 +923,8 @@ module openconfig-acl {
}
description
"System reported indication of how ACL counters are reported
by the target";
by the target. If the setting is not the same for all ACL
entries of all ACL sets, no value is returned.";
}
}
grouping acl-top {
Expand Down
Loading