[acl-loader]: acl-loader integration. by oleksandrivantsiv · Pull Request #1000 · sonic-net/sonic-buildimage

oleksandrivantsiv · 2017-09-29T09:12:59Z

acl-loader was inplemented based on translate_acl source code. acl-loader
can't be tested with unittest. Same testcases will be covered with testbed tests.

Remove translate_acl utility and tests.
Remove mirror template.
Do not run ACL rules and mirror configuration generators.
Adopt minigraph parser to work with acl-loader.

stcheng · 2017-09-29T18:12:10Z

Cong! The No. 1000 pull request.

stcheng · 2017-10-02T19:49:56Z

conflicts?

acl-loader was inplemented based on translate_acl source code. acl-loader can't be tested with unittest. Same testcases will be covered with testbed tests. - Remove translate_acl utility and tests. - Remove mirror template. - Do not run ACL rules and mirror configuration generators. - Adopt minigraph parser to work with acl-loader.

Enable ACL dynamic config feature.

taoyl-ms · 2017-11-01T21:26:01Z

src/sonic-config-engine/tests/test_j2files.py


        assert filecmp.cmp(sample_output_file, self.output_file)

-    def test_everflow(self):


Why are we removing this test? We still need the minigraph parser to be able to generate mirror sessions from minigraph.

Will update test

To work with config DB format

lguohan · 2017-11-02T05:13:38Z

src/sonic-config-engine/translate_acl

@@ -1,174 +0,0 @@
-#!/usr/bin/env python


@taoyl-ms , we still need to generate config db acl from the openconfig acl format, right?

@lguohan ACL loader works with files in openconfig acl format.

@lguohan acl-loader input file example:
https://github.com/Azure/sonic-mgmt/pull/299/files#diff-b72b79c3178d5157ef088fe15870f99e

lguohan · 2017-11-02T05:15:26Z

dockers/docker-orchagent/mirror.json.j2

@@ -1,24 +0,0 @@
-[


how are we going to create mirror session in the first place?

It will be created by sonic-cfggen utility. In the same way as ACL table.

Include fix for acl-loader incremental update command.

taoyl-ms

LGTM.
I still think we might need some unit test for acl-loader to make sure the parsing logic is correct. Do you think that will be doable in sonic-utilities repo?

oleksandrivantsiv · 2017-11-03T08:10:38Z

@taoyl-ms Yes, it is doable. But we need to bring up redis server in sonic-slave for this. If this is ok I can add unit tests.

taoyl-ms · 2017-11-03T18:27:18Z

Is it possible to mock it?

9f6efa0 [port/buffer] introduce a sync mechanism to protect port PG/queue from changes under PFC storm (sonic-net#1143) 823e426 [aclorch] Enable DSCP rules on IPv6 mirror tables (sonic-net#1146) b8745f8 [bitmap_vnet]: Fix removal flow for tunnel route (sonic-net#1139) 03be983 Increase ip2me CIR/CBR for faster in-band file transfers (sonic-net#1000) a4a1d3b [vnet]: Update VNET route table size to 40K for BITMAP implementation (sonic-net#1132) efe142a Fix bug: Wrong condition for mac address (sonic-net#1142) 7bf63a0 [teammgrd]during warm-reboot teamd need to recover system-id from saved lacp-pdu (sonic-net#1003) 8b4cfb6 Cleanup configure.ac from BFN specific code (sonic-net#1133) b931751 [teamsyncd]: Add retry logic in teamsyncd to avoid team handler init failure (sonic-net#854) Signed-off-by: Stepan Blyschak <[email protected]>

9f6efa0 [port/buffer] introduce a sync mechanism to protect port PG/queue from changes under PFC storm (#1143) 823e426 [aclorch] Enable DSCP rules on IPv6 mirror tables (#1146) b8745f8 [bitmap_vnet]: Fix removal flow for tunnel route (#1139) 03be983 Increase ip2me CIR/CBR for faster in-band file transfers (#1000) a4a1d3b [vnet]: Update VNET route table size to 40K for BITMAP implementation (#1132) efe142a Fix bug: Wrong condition for mac address (#1142) 7bf63a0 [teammgrd]during warm-reboot teamd need to recover system-id from saved lacp-pdu (#1003) 8b4cfb6 Cleanup configure.ac from BFN specific code (#1133) b931751 [teamsyncd]: Add retry logic in teamsyncd to avoid team handler init failure (#854) Signed-off-by: Stepan Blyschak <[email protected]>

Enable m_isCombinedMirrorV6Table for BFN platform (#1212) [vnet]: Update VNET route table size to 40K for BITMAP implementation (#1132) Default action for Egress ACL Table not poulated. (#1208) Add/Del lag_name_map item according to lag adding and removing (#1124) Increase ip2me CIR/CBR for faster in-band file transfers (#1000)

Enable m_isCombinedMirrorV6Table for BFN platform (sonic-net#1212) [vnet]: Update VNET route table size to 40K for BITMAP implementation (sonic-net#1132) Default action for Egress ACL Table not poulated. (sonic-net#1208) Add/Del lag_name_map item according to lag adding and removing (sonic-net#1124) Increase ip2me CIR/CBR for faster in-band file transfers (sonic-net#1000)

) Increase incoming packet rate on in-band interfaces to support faster download of large files. SONiC firmware image download over in-band can take a lot of time if the incoming packet rate is limited to 600pps. This, change increases it to 6000pps. Especially when used by Zero Touch Provisioning or by sonic_installer for firmware upgrade over in-band interfaces. Signed-off-by: Rajendra Dendukuri <[email protected]>

Enable m_isCombinedMirrorV6Table for BFN platform (sonic-net#1212) [vnet]: Update VNET route table size to 40K for BITMAP implementation (sonic-net#1132) Default action for Egress ACL Table not poulated. (sonic-net#1208) Add/Del lag_name_map item according to lag adding and removing (sonic-net#1124) Increase ip2me CIR/CBR for faster in-band file transfers (sonic-net#1000)

) Increase incoming packet rate on in-band interfaces to support faster download of large files. SONiC firmware image download over in-band can take a lot of time if the incoming packet rate is limited to 600pps. This, change increases it to 6000pps. Especially when used by Zero Touch Provisioning or by sonic_installer for firmware upgrade over in-band interfaces. Signed-off-by: Rajendra Dendukuri <[email protected]>

``` 57ed180 [configure.ac] implement SAI API version check (sonic-net#1000) 8894dc7 vslib: add support for read-only port capabilities (sonic-net#1038) 42af975 [vslib]: Update packet number of MACsec SA at runtime (sonic-net#1007) ``` Signed-off-by: Stepan Blyschak <[email protected]>

* 48cccb4 2022-06-13 | do not use sai_query_api_version if vendor sai does not support in VendorSai.cpp (sonic-net#1064) (HEAD, origin/master, origin/HEAD) [Guohan Lu] * 9b0f773 2022-06-13 | [vslib]: Fixbug in cleanup MACsec device (sonic-net#1059) [Ze Gan] * cdf9427 2022-06-11 | No sai api version check if vendor sai does not support (sonic-net#1063) (HEAD, origin/master, origin/HEAD) [Guohan Lu] * 3964cf1 2022-06-09 | [counter] Fix port flex counter (sonic-net#1052) [Junhua Zhai] * 2231b7a 2022-06-03 | Purge package sonic-db-cli which depends on libswsscommon (sonic-net#1057) [Qi Luo] * 7aa09b9 2022-06-01 | Set PR diff code coverage threshold to 80% (sonic-net#1039) [Kamil Cudnik] * 66a29bc 2022-05-18 | [syncd] Use vendor SAI instead of direct SAI api (sonic-net#1042) [Kamil Cudnik] * 564bea7 2022-05-18 | [ci] Paralize azure pipeline (sonic-net#1040) [Shilong Liu] * 57ed180 2022-05-17 | [configure.ac] implement SAI API version check (sonic-net#1000) [Stepan Blyshchak] * 8894dc7 2022-05-17 | vslib: add support for read-only port capabilities (sonic-net#1038) [Dante (Kuo-Jung) Su] * 42af975 2022-04-29 | [vslib]: Update packet number of MACsec SA at runtime (sonic-net#1007) [Ze Gan] Signed-off-by: Guohan Lu <[email protected]>

b13d7d2 [debian/rules] Fail on script error during package build (sonic-net#1050) 48cccb4 do not use sai_query_api_version if vendor sai does not support in VendorSai.cpp (sonic-net#1064) 9b0f773 [vslib]: Fixbug in cleanup MACsec device (sonic-net#1059) cdf9427 No sai api version check if vendor sai does not support (sonic-net#1063) 3964cf1 [counter] Fix port flex counter (sonic-net#1052) 2231b7a Purge package sonic-db-cli which depends on libswsscommon (sonic-net#1057) 7aa09b9 Set PR diff code coverage threshold to 80% (sonic-net#1039) 66a29bc [syncd] Use vendor SAI instead of direct SAI api (sonic-net#1042) 564bea7 [ci] Paralize azure pipeline (sonic-net#1040) 57ed180 [configure.ac] implement SAI API version check (sonic-net#1000) 8894dc7 vslib: add support for read-only port capabilities (sonic-net#1038) 42af975 [vslib]: Update packet number of MACsec SA at runtime (sonic-net#1007) Signed-off-by: Ze Gan <[email protected]>

* 48cccb4 2022-06-13 | do not use sai_query_api_version if vendor sai does not support in VendorSai.cpp (#1064) (HEAD, origin/master, origin/HEAD) [Guohan Lu] * 9b0f773 2022-06-13 | [vslib]: Fixbug in cleanup MACsec device (#1059) [Ze Gan] * cdf9427 2022-06-11 | No sai api version check if vendor sai does not support (#1063) (HEAD, origin/master, origin/HEAD) [Guohan Lu] * 3964cf1 2022-06-09 | [counter] Fix port flex counter (#1052) [Junhua Zhai] * 2231b7a 2022-06-03 | Purge package sonic-db-cli which depends on libswsscommon (#1057) [Qi Luo] * 7aa09b9 2022-06-01 | Set PR diff code coverage threshold to 80% (#1039) [Kamil Cudnik] * 66a29bc 2022-05-18 | [syncd] Use vendor SAI instead of direct SAI api (#1042) [Kamil Cudnik] * 564bea7 2022-05-18 | [ci] Paralize azure pipeline (#1040) [Shilong Liu] * 57ed180 2022-05-17 | [configure.ac] implement SAI API version check (#1000) [Stepan Blyshchak] * 8894dc7 2022-05-17 | vslib: add support for read-only port capabilities (#1038) [Dante (Kuo-Jung) Su] * 42af975 2022-04-29 | [vslib]: Update packet number of MACsec SA at runtime (#1007) [Ze Gan] Signed-off-by: Guohan Lu <[email protected]>

Signed-off-by: Stepan Blyschak [email protected] The motivation for this change is described in the proposal sonic-net/SONiC#935 and proposal in SAI opencomputeproject/SAI#1404 NOTE: Requires to update SAI once opencomputeproject/SAI#1404 is in.

…omatically (#22269) #### Why I did it src/sonic-swss-common ``` * d3abf15 - (HEAD -> master, origin/master, origin/HEAD) [consumer_table_pops]: Support Stats ST Capability query (#1000) (24 hours ago) [Ze Gan] ``` #### How I did it #### How to verify it #### Description for the changelog

…tomatically (sonic-net#1000) #### Why I did it src/sonic-swss ``` * 9e9442d - (HEAD -> 202412, origin/202412) [orchagent] Fix issue: typo in high BER (#68) (3 hours ago) [mssonicbld] ``` #### How I did it #### How to verify it #### Description for the changelog

oleksandrivantsiv changed the title ~~[acl-loader]: Remove translate_acl utility and tests.~~ [acl-loader]: acl-loader integration. Sep 29, 2017

stcheng added the Enhancement ➕ label Oct 2, 2017

oleksandrivantsiv added 2 commits November 1, 2017 22:15

Update sonic-swss-common and sonic-swss submodules.

20459ac

Enable ACL dynamic config feature.

taoyl-ms reviewed Nov 1, 2017

View reviewed changes

lguohan reviewed Nov 2, 2017

View reviewed changes

oleksandrivantsiv added 2 commits November 2, 2017 18:33

Update sonic-utilities submodule.

23c4c42

Include fix for acl-loader incremental update command.

[sonic-cfggen]: Add test for everflow cfggen configuration.

d8c65d5

taoyl-ms approved these changes Nov 2, 2017

View reviewed changes

lguohan merged commit 29f67eb into sonic-net:master Nov 2, 2017

stepanblyschak mentioned this pull request Dec 20, 2019

[swss] update swss submodule #3925

Merged

lguohan mentioned this pull request Jun 11, 2022

[submodule]: update sonic-sairedis #11112

Merged

6 tasks


		assert filecmp.cmp(sample_output_file, self.output_file)

		def test_everflow(self):

Conversation

oleksandrivantsiv commented Sep 29, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

stcheng commented Sep 29, 2017

Uh oh!

stcheng commented Oct 2, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

taoyl-ms left a comment

Choose a reason for hiding this comment

Uh oh!

oleksandrivantsiv commented Nov 3, 2017

Uh oh!

taoyl-ms commented Nov 3, 2017

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

oleksandrivantsiv commented Sep 29, 2017 •

edited

Loading