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

[acl-loader] modify acl-loader with change in STATE DB ACL capability table #1896

Merged
merged 2 commits into from
Nov 23, 2021

Conversation

stepanblyschak
Copy link
Contributor

@stepanblyschak stepanblyschak commented Oct 25, 2021

… table

Signed-off-by: Stepan Blyshchak stepanb@nvidia.com

What I did

HLD: sonic-net/SONiC#867
DEPENDS ON: sonic-net/sonic-swss#1982

Modified acl-loader and acl-loader UT to work with new STATE DB schema.

How I did it

By modifying the acl-loader and acl-loader UT code.

How to verify it

Run togather with dependent PRs in HLD.

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)

… table

Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@bingwang-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms
Copy link
Contributor

Please merge this PR after PR sonic-net/sonic-swss#1982

@stepanblyschak
Copy link
Contributor Author

https://dev.azure.com/mssonic/build/_build/results?buildId=50017&view=logs&j=3f6395b2-1619-5ebe-f305-2aedcf353cb5&t=35460df8-21be-5313-3ee4-50900c6db765&l=559

test_warm_reboot.py::TestWarmReboot::test_routing_WarmRestart FAILED     [ 98%]
test_warm_reboot.py::TestWarmReboot::test_system_warmreboot_neighbor_syncup XFAIL [ 98%]
##[error]The operation was canceled.
Finishing: Run vs tests

@stepanblyschak
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stepanblyschak
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms bingwang-ms merged commit a0bff26 into sonic-net:master Nov 23, 2021
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
901e2efb4f5c6a1179068b43e6a837dfff19d9d9 (HEAD -> 201911, origin/201911) [fpmsyncd] Skip routes to eth0 or docker0 (sonic-net#1606)
6c6f4fbbeb24b6e16a2bffdee760ba6a95e2ceb3 configure extra inc/lib directory for build (sonic-net#1247)
cccb59efa031b817c1936a74db969debbb9b2775 [201911][Cherry-pick] [acl mirror action] Mirror session ref count fix at acl rule attachment (sonic-net#1896)
62dc36d9da988807c7a397fd90ac2859a5a07699 [ci]: Support Azure pipelines on 201911 branch (sonic-net#1806)

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
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.

3 participants