Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants