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

Add ACL_TABLE in show runningconfiguration acl #2339

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

1002574903
Copy link

What I did

The show runningconfiguration ACL output does not contain the ACL_TABLE configuration

How I did it

Add ACL_TABLE in show runningconfiguration acl

How to verify it

config acl table and then run command show runningconfiguration acl

Previous command output (if the output of a command-line utility has changed)

root@sonic:/home/admin# config acl add table test L3 -p Ethernet10
root@sonic:/home/admin# show runningconfiguration acl
root@sonic:/home/admin#
root@sonic:/home/admin#

New command output (if the output of a command-line utility has changed)

root@sonic:/home/admin# config acl add table test L3 -p Ethernet10
root@sonic:/home/admin# show runningconfiguration acl
{
"test": {
"policy_desc": "test",
"ports": [
"Ethernet10"
],
"stage": "ingress",
"type": "L3"
}
}
root@sonic:/home/admin#

@1002574903
Copy link
Author

@prsunny Could you please review?

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.

1 participant