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

[202012] Update load minigraph to load backend acl #2235

Merged
merged 5 commits into from
Jul 8, 2022

Conversation

neethajohn
Copy link
Contributor

@neethajohn neethajohn commented Jun 27, 2022

Backport #2236

What I did

Load backend acl template as part of the load minigraph if the device type is a 'BackEndToRRouter' and the device is a storage device

How to verify it

Added unit tests to verify if the backend acl load commands are applied

Signed-off-by: Neetha John <nejo@microsoft.com>
Signed-off-by: Neetha John <nejo@microsoft.com>
Signed-off-by: Neetha John <nejo@microsoft.com>
Signed-off-by: Neetha John <nejo@microsoft.com>
@neethajohn neethajohn marked this pull request as ready for review June 30, 2022 01:45
Signed-off-by: Neetha John <nejo@microsoft.com>
@neethajohn neethajohn merged commit ecca18f into sonic-net:202012 Jul 8, 2022
@neethajohn neethajohn deleted the backend_acl_202012 branch July 8, 2022 15:49
neethajohn added a commit to sonic-net/sonic-buildimage that referenced this pull request Jul 11, 2022
Signed-off-by: Neetha John <nejo@microsoft.com>

This PR contains the following commits
5a54bd7 Added cisco config platform commands (sonic-net/sonic-utilities#2241)
62c1640 [config/load_mgmt_config] Support load IPv6 mgmt IP (sonic-net/sonic-utilities#2206)
c061a18 Fix header for the output table following 'show ipv6 interface' command (sonic-net/sonic-utilities#2219)
ecca18ff [202012] Update load minigraph to load backend acl (sonic-net/sonic-utilities#2235)
neethajohn added a commit that referenced this pull request Mar 14, 2023
yxieca pushed a commit that referenced this pull request Mar 17, 2023
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.

3 participants