Add ACL ranges test#168
Merged
oleksandrivantsiv merged 2 commits intosonic-net:masterfrom Apr 28, 2017
Merged
Conversation
pavel-shirshov
approved these changes
Apr 25, 2017
| - name: Read port reverse alias mapping | ||
| set_fact: | ||
| alias_reverse_map: "{{ lookup('file', 'roles/sonicv2/files/ssw/{{ sonic_hwsku }}/alias_reverse_map.json') | from_json }}" | ||
|
|
Contributor
There was a problem hiding this comment.
Is it possible to get these values from minigraph facts?
Contributor
Author
There was a problem hiding this comment.
not sure...
I just used the solution used in fib, lag, everflow and other tests
| - name: Change JSON configs to delete (table) | ||
| shell: sed 's/SET/DEL/g' /tmp/{{ item }}.json >/tmp/del_{{ item }}.json | ||
| register: res | ||
| failed_when: res.rc != 0 |
Contributor
There was a problem hiding this comment.
will it fail itself, when rc != 0?
Contributor
Author
There was a problem hiding this comment.
probably it will
I will remove extra checks
lguohan
pushed a commit
that referenced
this pull request
Aug 3, 2017
* Add ACL ranges test * Removed extra return code cheks
nhe-NV
pushed a commit
to nhe-NV/sonic-mgmt
that referenced
this pull request
May 12, 2025
<!-- Please make sure you've read and understood our contributing guidelines; https://github.com/sonic-net/SONiC/blob/gh-pages/CONTRIBUTING.md Please provide following information to help code review process a bit easier: --> ### Description of PR <!-- - Please include a summary of the change and which issue is fixed. - Please also include relevant motivation and context. Where should reviewer start? background context? - List any dependencies that are required for this change. --> Summary: add the following topology: ansible/vars/topo_t0-isolated-d256u256s2.yml ansible/vars/topo_t0-isolated-d32u32s2.yml ansible/vars/topo_t1-isolated-d448u16.yml ansible/vars/topo_t1-isolated-d56u2.yml ### Type of change <!-- - Fill x for your type of change. - e.g. - [x] Bug fix --> - [ ] Bug fix - [ ] Testbed and Framework(new/improvement) - [ ] New Test case - [ ] Skipped for non-supported platforms - [ ] Test case improvement ### Back port request - [ ] 202012 - [ ] 202205 - [ ] 202305 - [ ] 202311 - [ ] 202405 - [ ] 202411 ### Approach #### What is the motivation for this PR? Add TH-512 topo #### How did you do it? #### How did you verify/test it? #### Any platform specific information? #### Supported testbed topology if it's a new test case? ### Documentation <!-- (If it's a new feature, new test case) Did you update documentation/Wiki relevant to your implementation? Link to the wiki page? -->
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
Includes below sonic-snmpagent commits ``` dfde06e 2021-01-13 | Revert "[rfc1213] Interface MIB add l3 vlan interfaces & aggregate rif counters (sonic-net#169)" (sonic-net#191) [Stepan Blyshchak] 45edd7e 2021-01-04 | [snmpagent] Fix hardcoded qsfp lane count by reading sensor status from DB (sonic-net#184) [Junchao-Mellanox] 3b72a6f 2021-01-02 | Fix: handle empty LOC_CHASSIS_TABLE (sonic-net#190) [Qi Luo] 4aad821 2020-12-29 | [sysName]: Implement sysName OID (sonic-net#185) [SuvarnaMeenakshi] 8efb4bb 2020-12-29 | [rfc1213] fix counter value type (sonic-net#189) [Stepan Blyshchak] 025483a 2020-12-23 | [RouteUpdater]: Fix multi_asic mock function implementation and multi_asic variable name (sonic-net#186) [SuvarnaMeenakshi] 381ae47 2020-12-10 | [mibs] b'VLAN_TABLE:' -> 'VLAN_TABLE' (sonic-net#181) [Stepan Blyshchak] e54036c 2020-12-09 | [rfc1213] Interface MIB add l3 vlan interfaces & aggregate rif counters (sonic-net#169) [Stepan Blyshchak] fd1eae7 2020-11-24 | Set swsscommon logging level (sonic-net#178) [Qi Luo] 706d504 2020-11-23 | Improve MockRedis _encode(): so it will work on all types of value (sonic-net#179) [Qi Luo] 64c93a1 2020-11-16 | [RFC4292][Namespace]: Fix implementation of RouteUpdater for multi-asic platform (sonic-net#176) [SuvarnaMeenakshi] b8f19ee 2020-11-12 | [sonic-snmpagent] SONiC physical entity mib extension (sonic-net#168) [Junchao-Mellanox] 6b94ec3 2020-11-05 | Replace swsssdk.SonicV2Connector with swsscommon.SonicV2Connector (SWIG wrapper of C++ implementation) in production code (sonic-net#162) [Qi Luo] ``` Signed-off-by: Stepan Blyshchak <[email protected]> Co-authored-by: Qi Luo <[email protected]>
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
This PR updates the following commits c6b642b [ci]: download from sonic-buildimage.vs artifact (sonic-net#168) e76ecc6 [sonic_y_cable] add support for retrieving firmware info for Y cable, internal and nic temperature and voltage (sonic-net#162) f9cf8c9 [GitHub] Add pull request template (sonic-net#167) c31636e [ci] Call pip2/3 using sudo (sonic-net#166) 5521f67 [ci] Test and build packages using Azure Pipelines (sonic-net#164) faca35c [ci]: Set up CI with Azure Pipelines Signed-off-by: vaibhav-dahiya <[email protected]>
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
c5be3ca4 [psud] Increase unit test coverage; Refactor mock platform (sonic-net#154) 450b7d78 Bug fix: the fields that are not supported by vendor should be "N/A" in STATE_DB (sonic-net#168) Signed-off-by: Stephen Sun <[email protected]>
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
…atically (sonic-net#17016) src/sonic-telemetry * 1a70b50 - (HEAD -> 202205, origin/202205) Merge pull request sonic-net#168 from zbud-msft/cherry-pick-fix-panic-202205 (4 hours ago) [Ying Xie] * 2eb9275 - Recover from potential panic when doing map to JSON serialization (sonic-net#161) (7 days ago) [Zain Budhwani]
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Usage example:
ansible-playbook test_sonic.yml -i inventory --limit arc-switch1028 --tags acltb_ranges_test -b -e ranges_number=3 -e ptf_host=ptf-fib
Variable "ranges_number" defines how many ACL range objects the test will attempt to create