Skip to content

Load MIRROR_DSCP config files at load_minigraph #1838

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

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

Conversation

bingwang-ms
Copy link
Contributor

What I did

This PR is to update load_minigraph to load MIRROR_DSCP related configuration files at config load_minigraph.
Below config files are loaded if present

config file name
policer /etc/sonic/everflow_dscp/policer.json
mirror_session /etc/sonic/everflow_dscp/mirror_session.json
acl_rule /etc/sonic/everflow_dscp/acl_rule.json

How I did it

  1. Read ACL_TABLE from config_db to check if an EVERFLOW ACL table is present with type MIRROR_DSCP
  2. Load 3 config files if MIRROR_DSCP is present and all 3 config files are present

How to verify it

  1. Verified by UT
  2. Verified by running on a device

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

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

@bingwang-ms
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: bingwang <[email protected]>
@bingwang-ms
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant