Merged
Conversation
- Make it work against the NG testbed - Make it work against t1 and t1-lag testbed_type Signed-off-by: Shuotian Cheng <shuche@microsoft.com>
stcheng
requested review from
abgupta and
lguohan
and removed request for
abgupta
lguohan
pushed a commit
that referenced
this pull request
Aug 3, 2017
- Make it work against the NG testbed - Make it work against t1 and t1-lag testbed_type Signed-off-by: Shuotian Cheng <shuche@microsoft.com>
nhe-NV
pushed a commit
to nhe-NV/sonic-mgmt
that referenced
this pull request
May 12, 2025
…ailed to announce route from t0 VM (sonic-net#134) <!-- 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 Add "tornum" for t0 VM definition in ansible/vars/topo_t1-isolated-d28u1.yml to trigger exabgp announce route during topology deployment. The "tornum" is necessary for generating routes from T0 VM: https://github.com/sonic-net/sonic-mgmt/blob/221012beaac677677139fed1ab92cd0d5adad789/ansible/library/announce_routes.py#L417 > suffix = ((podset failure_prs.log skip_prs.log tor_number failure_prs.log skip_prs.log max_tor_subnet_number failure_prs.log skip_prs.log tor_subnet_size) + > (tor failure_prs.log skip_prs.log max_tor_subnet_number failure_prs.log skip_prs.log tor_subnet_size) + > (subnet failure_prs.log skip_prs.log tor_subnet_size)) > octet2 = (168 + int(suffix / (256 failure_prs.log skip_prs.log 2))) > octet1 = (192 + int(octet2 / 256)) > octet2 = (octet2 % 256) > octet3 = (int(suffix / 256) % 256) > octet4 = (suffix % 256) > prefixlen_v4 = (32 - int(math.log(tor_subnet_size, 2))) > > prefix = "{}.{}.{}.{}/{}".format(octet1, > octet2, octet3, octet4, prefixlen_v4) > prefix_v6 = ipv6_address_pattern % ( > octet1, octet2, octet3, octet4) Summary: Fixes # (issue) Exabgp of T0 VM failed to announce route during topology deployment ### Type of change <!-- - Fill x for your type of change. - e.g. - [x] Bug fix --> - [x] 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? Bug fix for the definition of ansible/vars/topo_t1-isolated-d28u1.yml Exabgp of T0 VM failed to announce route during topology deployment #### How did you do it? "tornum" for t0 VM #### How did you verify/test it? Run it in internal testbed #### 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
…e submodule head (sonic-net#12149) linkmgrd: * 05e5f4c 2022-09-20 | [Active-Active] flaky LinkmgrdBootupSequence unit tests (sonic-net#134) (HEAD -> 202205) [Jing Zhang] * 16fcadf 2022-09-13 | [active-standby] update warmboot reconciliation logic (sonic-net#129) [Jing Zhang] * e656a87 2022-09-09 | [active-active] shutdown link prober when starting as isolated (sonic-net#130) [Jing Zhang] uttilities: yinxi@ying-dev-vm-01:~/src/sonic-202205/src/sonic-utilities$ git hist github/202205..HEAD * 562188f 2022-09-14 | Use 'default' VRF when VRF name is not provided (sonic-net#2368) (HEAD -> 202205) [Sumukha Tumkur Vani] * c50ba4f 2022-09-20 | [minigraph] add option to specify golden path in load_minigraph (sonic-net#2350) [jingwenxie] * cec5ab2 2022-09-20 | [GCU]Remove GCU unique lane check for duplicate lanes platforms (sonic-net#2343) [jingwenxie] * 8d20771 2022-09-15 | Vnet_route_check Vxlan tunnel route update. (sonic-net#2281) [siqbal1986] swss: * 88371f7 2022-09-21 | [ci] Only when test stage succeeded or succeededwithissues, PR run Gcov (sonic-net#2460) (HEAD -> 202205) [Liu Shilong] * c11dbd7 2022-09-15 | [QoS] Enforce drop probability only for colors whose WRED are enabled (sonic-net#2422) [Stephen Sun] sairedis: * 80928dd 2022-09-06 | [lgtm] Add uuid library (sonic-net#1119) (HEAD -> 202205, github/202205) [Kamil Cudnik] * c147dd0 2022-09-16 | [202205][vslib]: Add SAI_PORT_ATTR_OPER_SPEED get sonic-net#1123 [Ze Gan] platform-daemon: * 9cf8adf 2022-09-21 | [ycabled] add notification for gRPC connection state transitions to IDLE/TRANSIENT_FAILURE (sonic-net#295) (HEAD -> 202205) [vdahiya12] * 1e07ae3 2022-09-20 | Use get() to fetch default value from dictionary for port admin_status sonic-net#286 [anamehra] * 157f483 2022-09-15 | [Xcvrd] Soak duplicate events and process only updated interested events (sonic-net#285) [Prince George] Signed-off-by: Ying Xie <ying.xie@microsoft.com>
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
…ically (#25742) #### Why I did it src/sonic-dash-ha ``` * 8f9893d - (HEAD -> 202511, origin/master, origin/HEAD, origin/202511, master) Create bfd sessions only to NPU participating ha-set (sonic-net#143) (9 days ago) [yue-fred-gao] * a6cf697 - update dash-api submodule (10 days ago) [Jing Zhang] * 64022eb - Change convert_pb_to_json to parse proto encoded value from binary input (sonic-net#142) (10 days ago) [yue-fred-gao] * 53fb250 - [ci] fix build error and save binaries (sonic-net#144) (11 days ago) [Jing Zhang] * d01ed94 - Add .github/copilot-instructions.md for AI-assisted development (sonic-net#140) (2 weeks ago) [rustiqly] * 2b6b37c - Write DASH_DPU_RESET_INFO_TABLE when dpu midplane or control plane down (sonic-net#137) (4 weeks ago) [yue-fred-gao] * 9b3c0bf - Add bfd rewrite on pmon change. (sonic-net#136) (4 weeks ago) [dypet] * af44396 - [build] Disable debian helper auto install for cargo project. (sonic-net#135) (5 weeks ago) [Liu Shilong] * 17e2e0b - Implement bfd pinned state (sonic-net#134) (5 weeks ago) [yue-fred-gao] * c04969e - switch to using libboost1.83 (sonic-net#133) (6 weeks ago) [yijingyan2] * b38d8fb - Change to DBConnector::clone_timeout_async (sonic-net#132) (3 months ago) [yue-fred-gao] ``` #### How I did it #### How to verify it #### Description for the changelog
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
following PR's : Implement cbgpPeer2State in CiscoBgp4MIB (sonic-net#119) Fix index nodes in LLDP tables whose access right is not-accessible. (sonic-net#112) Fix quagga/FRR parser on IPv6 BGP sessions (sonic-net#122) [lint] Fix some syntax errors or warnings (sonic-net#127) Update README.md: Add lgtm badges (sonic-net#128) [Multi-asic]: Support multi-asic platform (sonic-net#126) Simplify test code (sonic-net#132) [Multi-asic]: Namespace support for LLDP and Sensor tables (sonic-net#131) Fix undefined variable and warning message (sonic-net#134) Fix SNMP AgentX socket connection timeout when using Namespace.get_all() (sonic-net#140) [Namespace] Fix interfaces counters in InterfacesMIB RFC 2863 (sonic-net#141) Fix LGTM reported alert of PR#141 (sonic-net#142)
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
Why I did it [Submodule][202211] Advance sonic-restapi pointer The branch 202012 has already updated to commit 47e4b53. 4f6f979 Fix the redis security issue CVE-2023-28858 and CVE-2023-28859 (sonic-net#139) 47e4b53 Fix adv_pfx len for ipv6 (sonic-net#135) 44121be Support ipv6 prefix lenght greater than 64 and check for adv_prefix (sonic-net#134) 99c467d Add API support for adv prefix and custom monitoring (sonic-net#133) 347684a Use github code scanning instead of LGTM (sonic-net#132) 86543d0 Updates to route PATCH API (sonic-net#129) a1af82c Install libyang to azure pipeline (sonic-net#128) 2007c4c Increase coverage threshold (sonic-net#126) Work item tracking Microsoft ADO (number only): 17705422 How I did it How to verify it
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
2 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.
Signed-off-by: Shuotian Cheng shuche@microsoft.com