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

convert ingress/egress of ACL_TABLE in CONFIG_DB to INGRESS/EGRESS #2218

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

Conversation

1002574903
Copy link

What I did

The stage in ACL_TABLE in CONFIG_DB is ingress/egress when using config acl command. However, the leaf stage defined in sonic-acl.yang is EGRESS/INGRESS.
I convert ingress/egress of ACL_TABLE in CONFIG_DB to INGRESS/EGRESS to to make sure stage in CONFIG_DB and sonic-acl.yang are consistent.

How I did it

convert stage field of ACL_TABLE to upper case

How to verify it

use redis-cli to view ACL_TABLE contents in CONFIG_DB

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

127.0.0.1:6379[4]> HGETALL "ACL_TABLE|test"

  1. "policy_desc"
  2. "test"
  3. "ports@"
  4. "Ethernet20"
  5. "stage"
  6. "ingress"
  7. "type"
  8. "L3"

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

127.0.0.1:6379[4]> HGETALL "ACL_TABLE|test"

  1. "policy_desc"
  2. "test"
  3. "ports@"
  4. "Ethernet20"
  5. "stage"
  6. "INGRESS"
  7. "type"
  8. "L3"

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