[MCLAG] the change of port mac would trigger teamd to send LACP pdu instantly by shine4chen · Pull Request #3764 · sonic-net/sonic-buildimage

shine4chen · 2019-11-15T10:40:03Z

Signed-off-by: shine.chen shine.chen@mediatek.com

- What I did
In MCLAG scenario when keep-alive connection between active and standby node is down , standby node port mac will be changed back to his original mac. After host node receives LACP PDU from standby node, it will remove the local port connected to standby node from port-channel and switch all traffic to active node. But In existed teamd implementation when the port src mac is changed teamd doesn't send LACP PDU instantly , but wait for timer event expiry to send it. It would cause host still forward half of the traffic to standby node for quite a long period( maybe up to 30 seconds). During this period these traffic would be dropped by standby node. So we add a patch for teamd here. The change of port mac would trigger teamd to send LACP pdu instantly.

- How I did it
The change of port mac would trigger teamd to send LACP pdu instantly.

- How to verify it
After apply this patch the disruptive time decrease to less than 20ms.

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: shine.chen <shine.chen@mediatek.com>

shine4chen · 2019-11-15T13:44:23Z

retest vs please

shine4chen · 2019-11-16T01:26:40Z

please help to review it @rlhui

Signed-off-by: shine.chen <shine.chen@mediatek.com>

merge master branch

Signed-off-by: shine.chen <shine.chen@mediatek.com>

shine4chen · 2020-02-21T07:51:06Z

@pavel-shirshov Could you please to review and approve this PR?

shine4chen · 2020-02-21T12:48:04Z

retest vsimage please

madhukar-kamarapu · 2020-02-22T12:09:08Z

@shine4chen - This code change is not required in teamd to send the LACPDUs instantly when MCLAG on the Standby device is DOWN (the port-channel MAC address is reverted back to the device's MAC address).

Your commit (https://github.com/shine4chen/sonic-buildimage/blob/b697da2fb436cf0b430d47378acce8abfb2a4562/src/iccpd/src/iccp_netlink.c#L675
) to flap admin status of the port-channel will restart the LACP SM in teamd. When the LACP SM is restarted for all the member ports, the LACPDUs are transmitted immediately with the new MAC address.

The above changes are from your PR (Iccpd support ipv6-nd) -
shine4chen@b697da2#diff-5b542ccf50497fd7faa6e5c5dfa20597R675

But there is a bug in teamd to handle the above change (restart the LACP SM when admin status is flapped):
During MCLAG test scenarios, when a portchannel is removed/added from an operational MCLAG, the ICCPd component issues a MAC change, followed by a IFF_UP clear & IFF_UP set operations in quick succession. Quite often (depending on the load on the system and the timings of various events in the system) teamd does not get to process the IFF_UP clear operation. This results in teamd not syncing-up immediately with the partner upon MAC change; and it can take a max of 30 seconds (LACP timeout) to complete the synchronization.

Teamd relies on RTM_NEWLINK messages to be notified of the IFF_UP clear and set operations from the kernel. When teamd receives an RTM_NEWLINK message from the kernel, it turns around and enquires the kernel for a full set of information regarding the link event. The kernel populates all the contents of the netlink message, including updating/overwriting the flags field (that carries the IFF_UP status) with its current state in the kernel. Depending on the timing of the call into the kernel, a previous IFF_UP state could have been changed in the kernel, hence overwriting the previous flags as it returns. If this happens, the teamd application would end-up missing a IFF_UP event (clear or set), thereby not effecting state-machine changes properly.

Here is the patch for above mentioned problem, I'll be submitting a PR for the fix in teamd:


diff --git a/libteam/ifinfo.c b/libteam/ifinfo.c
index a7a83ac..7a8d733 100644
--- a/libteam/ifinfo.c
+++ b/libteam/ifinfo.c
@@ -273,6 +273,7 @@ static void obj_input_newlink(struct nl_object *obj, void *arg, bool event)
        struct rtnl_link *link;
        struct team_ifinfo *ifinfo;
        uint32_t ifindex;
+       uint32_t flags_cache = 0;
        int err;
        char *ifname;

@@ -280,6 +281,14 @@ static void obj_input_newlink(struct nl_object *obj, void *arg, bool event)

        link = (struct rtnl_link *) obj;

+       /* Cache the flags received in the new-link notification. Without this, the subsequent
+        * calls to the kernel to get complete details would override the received flags,
+        * overwriting with kernel's current state. This could/does result in application
+        * missing some of the state changes - especially the IFF_UP flag changes when there
+        * are back to back IFF_UP clear and set operations.
+        */
+       flags_cache = rtnl_link_get_flags(link);
+
        ifname = rtnl_link_get_name(link);
        if (ifname)
        {
@@ -310,6 +319,14 @@ static void obj_input_newlink(struct nl_object *obj, void *arg, bool event)
                return;

        clear_changed(ifinfo);
+
+       /* Restore the IFF_UP flag from the cache */
+       if (flags_cache & IFF_UP) {
+               rtnl_link_set_flags(link, IFF_UP);
+       } else {
+               rtnl_link_unset_flags(link, IFF_UP);
+       }
+
        ifinfo_update(ifinfo, link);

        if (event)

Note: The above fix would resolve few timing issues in teamd to honor the port-channel admin flap triggered by iccpd.

I strongly recommend not to merge the changes in PR#3764.

Rgds,
Madhukar

shine4chen · 2020-02-24T09:15:37Z

@madhukar-kamarapu Thanks you for the feedback. I study your reply and summary it as the follows:

Portchannel status flapping would trigger lacp-sm restart which can sync updated mac address to peer.
The existing teamd implementation has some issue so that it can't handle very fast port status flapping properly.
You are ready for a patch to resolve it.

I admit your solution make some sense. But it depends on some factors.

It depend on my outgoing PR (Iccpd support ipv6-nd).
Linux netlink message could be dropped in kernel if much netlink events is generated or system load is very high.

#3764 can simply solve the converge time issue without iccpd-support-nd PR. And I don't see any side effort.
How about your comment?

madhukar-kamarapu · 2020-02-24T15:50:16Z

@madhukar-kamarapu Thanks you for the feedback. I study your reply and summary it as the follows:

Portchannel status flapping would trigger lacp-sm restart which can sync updated mac address to peer.

The existing teamd implementation has some issue so that it can't handle very fast port status flapping properly.

You are ready for a patch to resolve it.

I admit your solution make some sense. But it depends on some factors.

It depend on my outgoing PR (Iccpd support ipv6-nd).

Linux netlink message could be dropped in kernel if much netlink events is generated or system load is very high.

#3764 can simply solve the converge time issue without iccpd-support-nd PR. And I don't see any side effort.
How about your comment?

@shine4chen - Actor_System (in LACPDU) is defined as the MAC address of the system. In a typical scenario Actor_System for a given port never changes. Reason - Actor_System is the MAC address of the system; MAC address does not change on the fly.
MCLAG is a special scenario where we change the MAC address of the port-channel netdevice.

The IEEE standards 802.3AD(old) or 802.1AX(new) do not talk about what needs to be done when MAC address change happens.

With the current fix (PR#3764), we'd be transmitting LACPDUs when the MAC address change happens. This behavior is not defined in the standard.

Since MCLAG is special case scenario of MAC address change, it would be better to stick with the port-channel admin state flap which would restart the LACP SM with new MAC address instantaneously. We'd not be deviating from the standard.

Note: This fix mentioned by me (retain IFF_UP flag) is anyways required in teamd; if this fix is taken (along with port-channel admin state flap), the changes done in PR#3764 would be redundant. We'd unnecessarily send more LACPDUs from the port.

shine4chen · 2020-02-25T05:53:49Z

@madhukar-kamarapu Sure, I will close this PR after you submit your teamd patch PR.

shine4chen · 2020-02-26T07:07:59Z

@madhukar-kamarapu Could you please to send me your patch file for libteam? Then we can test it locally. shine.chen@mediatek.com

…lly (#24534) #### Why I did it src/sonic-swss ``` * dabbd57c - (HEAD -> master, origin/master, origin/HEAD) Disable test_Srv6MySidUNTunnelDscpMode and test_Srv6MySidUNTunnelDscpModeAmbiguity (#4033) (31 hours ago) [Changrong Wu] * a7198d1a - Revert "Orchagent changes needed to support single ASIC VOQ Fixed-System (#3847)" (#4035) (2 days ago) [Ying Xie] * 83adbd96 - Revert "Avoid nhgroup update when mux state changes (#3822)" (#4030) (2 days ago) [Ying Xie] * f34f624b - Respect Cargo.lock for dependencies version (#4028) (3 days ago) [Qi Luo] * d644d2e4 - [bulker] Add support for bulk set object attributes (#3703) (4 days ago) [Nikola Dancejic] * eae91a23 - [Dash] Update ENI Based Forwarding Orchagent (#3905) (9 days ago) [Vivek] * 48e28b64 - Populate the Voq system Port information for the local port when the Port is removed and created when the Speed is changed dynamically via GCU (#3976) (9 days ago) [saksarav-nokia] * 4d397121 - [DPB]: Fix stale queue counter maps in COUNTERS_DB after port breakout (#3982) (9 days ago) [Ravi Minnikanti(Marvell)] * e2cc8ce0 - Add support for platform based on Clounix asic (#3846) (9 days ago) [clounix-sw] * 10df75b4 - Change DB that DPU orchagents listens to for all orchs (#3827) (9 days ago) [prabhataravind] * a2decc5a - Support SAI_PORT_SERDES_ATTR_CUSTOM_COLLECTION (#3764) (10 days ago) [longhuan-cisco] * c5caf506 - [SmartSwitch-HA] Support peer_ip update in ha set. (#3964) (11 days ago) [dypet] * 7119c2b5 - Enable output queue for HFT (#3962) (11 days ago) [Ze Gan] * 4c6457e8 - [SmartSwitch-HA] Set pending flags back to false. (#3997) (12 days ago) [dypet] * 2ed250d4 - Set egress mirror headroom to 0 on SN6600 platform (#4005) (12 days ago) [Stephen Sun] * 1c7ab034 - [HFT OTEL]: OTEL conversion init (#3920) (12 days ago) [Janet Cui] * 7c9315a3 - [buffermgrd] Optimize fast-reboot startup (#3952) (12 days ago) [Jianyue Wu] * 7d540cb7 - [fpmsyncd]: Fix uA SID programming for link-local adjacencies (#3958) (12 days ago) [Carmine Scarpitta] * 85412005 - [vnetorch] missing handling of rx and tx interval of monitoring session (#3878) (12 days ago) [Jing Zhang] * 46daad09 - [syncd] Fix the error log while running lua plugin (#3974) (12 days ago) [Vivek] * 5671e08b - Orchagent changes needed to support single ASIC VOQ Fixed-System (#3847) (2 weeks ago) [lakshmi-nexthop] * b017bd38 - Permanent isolate a fabric port if it repeatedly flapping. (#3933) (2 weeks ago) [jfeng-arista] * b426b2bb - Support checking capabilities of the mirror (#3934) (2 weeks ago) [Stephen Sun] * 25647cdf - [fpmsyncd]: Add Support for SRv6 VPN Route and PIC Context Processing (#3605) (2 weeks ago) [Yuqing Zhao(Alibaba Inc)] * 820eb74a - Allow state db to take modified entries made to the tunnel decap table (#3960) (2 weeks ago) [Dev Ojha] * 56856536 - Temporarily skip failing port tests to unblock pipeline runs (#4010) (2 weeks ago) [prabhataravind] * a4ed9594 - Avoid nhgroup update when mux state changes (#3822) (3 weeks ago) [manamand2020] * 42929d82 - dot3 Stats collection (#3615) (3 weeks ago) [Brad House - NextHop] * ffea522b - [portsorch] fix crash when number of PGs returned 0 (#3966) (3 weeks ago) [Stepan Blyshchak] * ea54ff84 - [ci] Migrate agent pool from sonicbld-1es to sonicso1ES-amd64 (#3987) (3 weeks ago) [Liu Shilong] * 0adab60c - [fpmsyncd] skip routes for eth1-midplane (#3724) (3 weeks ago) [arista-nwolfe] ``` #### How I did it #### How to verify it #### Description for the changelog

…lly (sonic-net#24534) #### Why I did it src/sonic-swss ``` * dabbd57 - (HEAD -> master, origin/master, origin/HEAD) Disable test_Srv6MySidUNTunnelDscpMode and test_Srv6MySidUNTunnelDscpModeAmbiguity (sonic-net#4033) (31 hours ago) [Changrong Wu] * a7198d1 - Revert "Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847)" (sonic-net#4035) (2 days ago) [Ying Xie] * 83adbd9 - Revert "Avoid nhgroup update when mux state changes (sonic-net#3822)" (sonic-net#4030) (2 days ago) [Ying Xie] * f34f624 - Respect Cargo.lock for dependencies version (sonic-net#4028) (3 days ago) [Qi Luo] * d644d2e - [bulker] Add support for bulk set object attributes (sonic-net#3703) (4 days ago) [Nikola Dancejic] * eae91a2 - [Dash] Update ENI Based Forwarding Orchagent (sonic-net#3905) (9 days ago) [Vivek] * 48e28b6 - Populate the Voq system Port information for the local port when the Port is removed and created when the Speed is changed dynamically via GCU (sonic-net#3976) (9 days ago) [saksarav-nokia] * 4d39712 - [DPB]: Fix stale queue counter maps in COUNTERS_DB after port breakout (sonic-net#3982) (9 days ago) [Ravi Minnikanti(Marvell)] * e2cc8ce - Add support for platform based on Clounix asic (sonic-net#3846) (9 days ago) [clounix-sw] * 10df75b - Change DB that DPU orchagents listens to for all orchs (sonic-net#3827) (9 days ago) [prabhataravind] * a2decc5 - Support SAI_PORT_SERDES_ATTR_CUSTOM_COLLECTION (sonic-net#3764) (10 days ago) [longhuan-cisco] * c5caf50 - [SmartSwitch-HA] Support peer_ip update in ha set. (sonic-net#3964) (11 days ago) [dypet] * 7119c2b - Enable output queue for HFT (sonic-net#3962) (11 days ago) [Ze Gan] * 4c6457e - [SmartSwitch-HA] Set pending flags back to false. (sonic-net#3997) (12 days ago) [dypet] * 2ed250d - Set egress mirror headroom to 0 on SN6600 platform (sonic-net#4005) (12 days ago) [Stephen Sun] * 1c7ab03 - [HFT OTEL]: OTEL conversion init (sonic-net#3920) (12 days ago) [Janet Cui] * 7c9315a - [buffermgrd] Optimize fast-reboot startup (sonic-net#3952) (12 days ago) [Jianyue Wu] * 7d540cb - [fpmsyncd]: Fix uA SID programming for link-local adjacencies (sonic-net#3958) (12 days ago) [Carmine Scarpitta] * 8541200 - [vnetorch] missing handling of rx and tx interval of monitoring session (sonic-net#3878) (12 days ago) [Jing Zhang] * 46daad0 - [syncd] Fix the error log while running lua plugin (sonic-net#3974) (12 days ago) [Vivek] * 5671e08 - Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847) (2 weeks ago) [lakshmi-nexthop] * b017bd3 - Permanent isolate a fabric port if it repeatedly flapping. (sonic-net#3933) (2 weeks ago) [jfeng-arista] * b426b2b - Support checking capabilities of the mirror (sonic-net#3934) (2 weeks ago) [Stephen Sun] * 25647cd - [fpmsyncd]: Add Support for SRv6 VPN Route and PIC Context Processing (sonic-net#3605) (2 weeks ago) [Yuqing Zhao(Alibaba Inc)] * 820eb74 - Allow state db to take modified entries made to the tunnel decap table (sonic-net#3960) (2 weeks ago) [Dev Ojha] * 5685653 - Temporarily skip failing port tests to unblock pipeline runs (sonic-net#4010) (2 weeks ago) [prabhataravind] * a4ed959 - Avoid nhgroup update when mux state changes (sonic-net#3822) (3 weeks ago) [manamand2020] * 42929d8 - dot3 Stats collection (sonic-net#3615) (3 weeks ago) [Brad House - NextHop] * ffea522 - [portsorch] fix crash when number of PGs returned 0 (sonic-net#3966) (3 weeks ago) [Stepan Blyshchak] * ea54ff8 - [ci] Migrate agent pool from sonicbld-1es to sonicso1ES-amd64 (sonic-net#3987) (3 weeks ago) [Liu Shilong] * 0adab60 - [fpmsyncd] skip routes for eth1-midplane (sonic-net#3724) (3 weeks ago) [arista-nwolfe] ``` #### How I did it #### How to verify it #### Description for the changelog

…lly (sonic-net#24534) #### Why I did it src/sonic-swss ``` * dabbd57 - (HEAD -> master, origin/master, origin/HEAD) Disable test_Srv6MySidUNTunnelDscpMode and test_Srv6MySidUNTunnelDscpModeAmbiguity (sonic-net#4033) (31 hours ago) [Changrong Wu] * a7198d1 - Revert "Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847)" (sonic-net#4035) (2 days ago) [Ying Xie] * 83adbd9 - Revert "Avoid nhgroup update when mux state changes (sonic-net#3822)" (sonic-net#4030) (2 days ago) [Ying Xie] * f34f624 - Respect Cargo.lock for dependencies version (sonic-net#4028) (3 days ago) [Qi Luo] * d644d2e - [bulker] Add support for bulk set object attributes (sonic-net#3703) (4 days ago) [Nikola Dancejic] * eae91a2 - [Dash] Update ENI Based Forwarding Orchagent (sonic-net#3905) (9 days ago) [Vivek] * 48e28b6 - Populate the Voq system Port information for the local port when the Port is removed and created when the Speed is changed dynamically via GCU (sonic-net#3976) (9 days ago) [saksarav-nokia] * 4d39712 - [DPB]: Fix stale queue counter maps in COUNTERS_DB after port breakout (sonic-net#3982) (9 days ago) [Ravi Minnikanti(Marvell)] * e2cc8ce - Add support for platform based on Clounix asic (sonic-net#3846) (9 days ago) [clounix-sw] * 10df75b - Change DB that DPU orchagents listens to for all orchs (sonic-net#3827) (9 days ago) [prabhataravind] * a2decc5 - Support SAI_PORT_SERDES_ATTR_CUSTOM_COLLECTION (sonic-net#3764) (10 days ago) [longhuan-cisco] * c5caf50 - [SmartSwitch-HA] Support peer_ip update in ha set. (sonic-net#3964) (11 days ago) [dypet] * 7119c2b - Enable output queue for HFT (sonic-net#3962) (11 days ago) [Ze Gan] * 4c6457e - [SmartSwitch-HA] Set pending flags back to false. (sonic-net#3997) (12 days ago) [dypet] * 2ed250d - Set egress mirror headroom to 0 on SN6600 platform (sonic-net#4005) (12 days ago) [Stephen Sun] * 1c7ab03 - [HFT OTEL]: OTEL conversion init (sonic-net#3920) (12 days ago) [Janet Cui] * 7c9315a - [buffermgrd] Optimize fast-reboot startup (sonic-net#3952) (12 days ago) [Jianyue Wu] * 7d540cb - [fpmsyncd]: Fix uA SID programming for link-local adjacencies (sonic-net#3958) (12 days ago) [Carmine Scarpitta] * 8541200 - [vnetorch] missing handling of rx and tx interval of monitoring session (sonic-net#3878) (12 days ago) [Jing Zhang] * 46daad0 - [syncd] Fix the error log while running lua plugin (sonic-net#3974) (12 days ago) [Vivek] * 5671e08 - Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847) (2 weeks ago) [lakshmi-nexthop] * b017bd3 - Permanent isolate a fabric port if it repeatedly flapping. (sonic-net#3933) (2 weeks ago) [jfeng-arista] * b426b2b - Support checking capabilities of the mirror (sonic-net#3934) (2 weeks ago) [Stephen Sun] * 25647cd - [fpmsyncd]: Add Support for SRv6 VPN Route and PIC Context Processing (sonic-net#3605) (2 weeks ago) [Yuqing Zhao(Alibaba Inc)] * 820eb74 - Allow state db to take modified entries made to the tunnel decap table (sonic-net#3960) (2 weeks ago) [Dev Ojha] * 5685653 - Temporarily skip failing port tests to unblock pipeline runs (sonic-net#4010) (2 weeks ago) [prabhataravind] * a4ed959 - Avoid nhgroup update when mux state changes (sonic-net#3822) (3 weeks ago) [manamand2020] * 42929d8 - dot3 Stats collection (sonic-net#3615) (3 weeks ago) [Brad House - NextHop] * ffea522 - [portsorch] fix crash when number of PGs returned 0 (sonic-net#3966) (3 weeks ago) [Stepan Blyshchak] * ea54ff8 - [ci] Migrate agent pool from sonicbld-1es to sonicso1ES-amd64 (sonic-net#3987) (3 weeks ago) [Liu Shilong] * 0adab60 - [fpmsyncd] skip routes for eth1-midplane (sonic-net#3724) (3 weeks ago) [arista-nwolfe] ``` #### How I did it #### How to verify it #### Description for the changelog Signed-off-by: Dawei Huang <daweihuang@microsoft.com>

…lly (sonic-net#24534) #### Why I did it src/sonic-swss ``` * dabbd57 - (HEAD -> master, origin/master, origin/HEAD) Disable test_Srv6MySidUNTunnelDscpMode and test_Srv6MySidUNTunnelDscpModeAmbiguity (sonic-net#4033) (31 hours ago) [Changrong Wu] * a7198d1 - Revert "Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847)" (sonic-net#4035) (2 days ago) [Ying Xie] * 83adbd9 - Revert "Avoid nhgroup update when mux state changes (sonic-net#3822)" (sonic-net#4030) (2 days ago) [Ying Xie] * f34f624 - Respect Cargo.lock for dependencies version (sonic-net#4028) (3 days ago) [Qi Luo] * d644d2e - [bulker] Add support for bulk set object attributes (sonic-net#3703) (4 days ago) [Nikola Dancejic] * eae91a2 - [Dash] Update ENI Based Forwarding Orchagent (sonic-net#3905) (9 days ago) [Vivek] * 48e28b6 - Populate the Voq system Port information for the local port when the Port is removed and created when the Speed is changed dynamically via GCU (sonic-net#3976) (9 days ago) [saksarav-nokia] * 4d39712 - [DPB]: Fix stale queue counter maps in COUNTERS_DB after port breakout (sonic-net#3982) (9 days ago) [Ravi Minnikanti(Marvell)] * e2cc8ce - Add support for platform based on Clounix asic (sonic-net#3846) (9 days ago) [clounix-sw] * 10df75b - Change DB that DPU orchagents listens to for all orchs (sonic-net#3827) (9 days ago) [prabhataravind] * a2decc5 - Support SAI_PORT_SERDES_ATTR_CUSTOM_COLLECTION (sonic-net#3764) (10 days ago) [longhuan-cisco] * c5caf50 - [SmartSwitch-HA] Support peer_ip update in ha set. (sonic-net#3964) (11 days ago) [dypet] * 7119c2b - Enable output queue for HFT (sonic-net#3962) (11 days ago) [Ze Gan] * 4c6457e - [SmartSwitch-HA] Set pending flags back to false. (sonic-net#3997) (12 days ago) [dypet] * 2ed250d - Set egress mirror headroom to 0 on SN6600 platform (sonic-net#4005) (12 days ago) [Stephen Sun] * 1c7ab03 - [HFT OTEL]: OTEL conversion init (sonic-net#3920) (12 days ago) [Janet Cui] * 7c9315a - [buffermgrd] Optimize fast-reboot startup (sonic-net#3952) (12 days ago) [Jianyue Wu] * 7d540cb - [fpmsyncd]: Fix uA SID programming for link-local adjacencies (sonic-net#3958) (12 days ago) [Carmine Scarpitta] * 8541200 - [vnetorch] missing handling of rx and tx interval of monitoring session (sonic-net#3878) (12 days ago) [Jing Zhang] * 46daad0 - [syncd] Fix the error log while running lua plugin (sonic-net#3974) (12 days ago) [Vivek] * 5671e08 - Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847) (2 weeks ago) [lakshmi-nexthop] * b017bd3 - Permanent isolate a fabric port if it repeatedly flapping. (sonic-net#3933) (2 weeks ago) [jfeng-arista] * b426b2b - Support checking capabilities of the mirror (sonic-net#3934) (2 weeks ago) [Stephen Sun] * 25647cd - [fpmsyncd]: Add Support for SRv6 VPN Route and PIC Context Processing (sonic-net#3605) (2 weeks ago) [Yuqing Zhao(Alibaba Inc)] * 820eb74 - Allow state db to take modified entries made to the tunnel decap table (sonic-net#3960) (2 weeks ago) [Dev Ojha] * 5685653 - Temporarily skip failing port tests to unblock pipeline runs (sonic-net#4010) (2 weeks ago) [prabhataravind] * a4ed959 - Avoid nhgroup update when mux state changes (sonic-net#3822) (3 weeks ago) [manamand2020] * 42929d8 - dot3 Stats collection (sonic-net#3615) (3 weeks ago) [Brad House - NextHop] * ffea522 - [portsorch] fix crash when number of PGs returned 0 (sonic-net#3966) (3 weeks ago) [Stepan Blyshchak] * ea54ff8 - [ci] Migrate agent pool from sonicbld-1es to sonicso1ES-amd64 (sonic-net#3987) (3 weeks ago) [Liu Shilong] * 0adab60 - [fpmsyncd] skip routes for eth1-midplane (sonic-net#3724) (3 weeks ago) [arista-nwolfe] ``` #### How I did it #### How to verify it #### Description for the changelog Signed-off-by: xiaweijiang <xiaweijiang@microsoft.com>

…lly (sonic-net#24534) #### Why I did it src/sonic-swss ``` * dabbd57 - (HEAD -> master, origin/master, origin/HEAD) Disable test_Srv6MySidUNTunnelDscpMode and test_Srv6MySidUNTunnelDscpModeAmbiguity (sonic-net#4033) (31 hours ago) [Changrong Wu] * a7198d1 - Revert "Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847)" (sonic-net#4035) (2 days ago) [Ying Xie] * 83adbd9 - Revert "Avoid nhgroup update when mux state changes (sonic-net#3822)" (sonic-net#4030) (2 days ago) [Ying Xie] * f34f624 - Respect Cargo.lock for dependencies version (sonic-net#4028) (3 days ago) [Qi Luo] * d644d2e - [bulker] Add support for bulk set object attributes (sonic-net#3703) (4 days ago) [Nikola Dancejic] * eae91a2 - [Dash] Update ENI Based Forwarding Orchagent (sonic-net#3905) (9 days ago) [Vivek] * 48e28b6 - Populate the Voq system Port information for the local port when the Port is removed and created when the Speed is changed dynamically via GCU (sonic-net#3976) (9 days ago) [saksarav-nokia] * 4d39712 - [DPB]: Fix stale queue counter maps in COUNTERS_DB after port breakout (sonic-net#3982) (9 days ago) [Ravi Minnikanti(Marvell)] * e2cc8ce - Add support for platform based on Clounix asic (sonic-net#3846) (9 days ago) [clounix-sw] * 10df75b - Change DB that DPU orchagents listens to for all orchs (sonic-net#3827) (9 days ago) [prabhataravind] * a2decc5 - Support SAI_PORT_SERDES_ATTR_CUSTOM_COLLECTION (sonic-net#3764) (10 days ago) [longhuan-cisco] * c5caf50 - [SmartSwitch-HA] Support peer_ip update in ha set. (sonic-net#3964) (11 days ago) [dypet] * 7119c2b - Enable output queue for HFT (sonic-net#3962) (11 days ago) [Ze Gan] * 4c6457e - [SmartSwitch-HA] Set pending flags back to false. (sonic-net#3997) (12 days ago) [dypet] * 2ed250d - Set egress mirror headroom to 0 on SN6600 platform (sonic-net#4005) (12 days ago) [Stephen Sun] * 1c7ab03 - [HFT OTEL]: OTEL conversion init (sonic-net#3920) (12 days ago) [Janet Cui] * 7c9315a - [buffermgrd] Optimize fast-reboot startup (sonic-net#3952) (12 days ago) [Jianyue Wu] * 7d540cb - [fpmsyncd]: Fix uA SID programming for link-local adjacencies (sonic-net#3958) (12 days ago) [Carmine Scarpitta] * 8541200 - [vnetorch] missing handling of rx and tx interval of monitoring session (sonic-net#3878) (12 days ago) [Jing Zhang] * 46daad0 - [syncd] Fix the error log while running lua plugin (sonic-net#3974) (12 days ago) [Vivek] * 5671e08 - Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847) (2 weeks ago) [lakshmi-nexthop] * b017bd3 - Permanent isolate a fabric port if it repeatedly flapping. (sonic-net#3933) (2 weeks ago) [jfeng-arista] * b426b2b - Support checking capabilities of the mirror (sonic-net#3934) (2 weeks ago) [Stephen Sun] * 25647cd - [fpmsyncd]: Add Support for SRv6 VPN Route and PIC Context Processing (sonic-net#3605) (2 weeks ago) [Yuqing Zhao(Alibaba Inc)] * 820eb74 - Allow state db to take modified entries made to the tunnel decap table (sonic-net#3960) (2 weeks ago) [Dev Ojha] * 5685653 - Temporarily skip failing port tests to unblock pipeline runs (sonic-net#4010) (2 weeks ago) [prabhataravind] * a4ed959 - Avoid nhgroup update when mux state changes (sonic-net#3822) (3 weeks ago) [manamand2020] * 42929d8 - dot3 Stats collection (sonic-net#3615) (3 weeks ago) [Brad House - NextHop] * ffea522 - [portsorch] fix crash when number of PGs returned 0 (sonic-net#3966) (3 weeks ago) [Stepan Blyshchak] * ea54ff8 - [ci] Migrate agent pool from sonicbld-1es to sonicso1ES-amd64 (sonic-net#3987) (3 weeks ago) [Liu Shilong] * 0adab60 - [fpmsyncd] skip routes for eth1-midplane (sonic-net#3724) (3 weeks ago) [arista-nwolfe] ``` #### How I did it #### How to verify it #### Description for the changelog

…lly (sonic-net#24534) #### Why I did it src/sonic-swss ``` * dabbd57 - (HEAD -> master, origin/master, origin/HEAD) Disable test_Srv6MySidUNTunnelDscpMode and test_Srv6MySidUNTunnelDscpModeAmbiguity (sonic-net#4033) (31 hours ago) [Changrong Wu] * a7198d1 - Revert "Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847)" (sonic-net#4035) (2 days ago) [Ying Xie] * 83adbd9 - Revert "Avoid nhgroup update when mux state changes (sonic-net#3822)" (sonic-net#4030) (2 days ago) [Ying Xie] * f34f624 - Respect Cargo.lock for dependencies version (sonic-net#4028) (3 days ago) [Qi Luo] * d644d2e - [bulker] Add support for bulk set object attributes (sonic-net#3703) (4 days ago) [Nikola Dancejic] * eae91a2 - [Dash] Update ENI Based Forwarding Orchagent (sonic-net#3905) (9 days ago) [Vivek] * 48e28b6 - Populate the Voq system Port information for the local port when the Port is removed and created when the Speed is changed dynamically via GCU (sonic-net#3976) (9 days ago) [saksarav-nokia] * 4d39712 - [DPB]: Fix stale queue counter maps in COUNTERS_DB after port breakout (sonic-net#3982) (9 days ago) [Ravi Minnikanti(Marvell)] * e2cc8ce - Add support for platform based on Clounix asic (sonic-net#3846) (9 days ago) [clounix-sw] * 10df75b - Change DB that DPU orchagents listens to for all orchs (sonic-net#3827) (9 days ago) [prabhataravind] * a2decc5 - Support SAI_PORT_SERDES_ATTR_CUSTOM_COLLECTION (sonic-net#3764) (10 days ago) [longhuan-cisco] * c5caf50 - [SmartSwitch-HA] Support peer_ip update in ha set. (sonic-net#3964) (11 days ago) [dypet] * 7119c2b - Enable output queue for HFT (sonic-net#3962) (11 days ago) [Ze Gan] * 4c6457e - [SmartSwitch-HA] Set pending flags back to false. (sonic-net#3997) (12 days ago) [dypet] * 2ed250d - Set egress mirror headroom to 0 on SN6600 platform (sonic-net#4005) (12 days ago) [Stephen Sun] * 1c7ab03 - [HFT OTEL]: OTEL conversion init (sonic-net#3920) (12 days ago) [Janet Cui] * 7c9315a - [buffermgrd] Optimize fast-reboot startup (sonic-net#3952) (12 days ago) [Jianyue Wu] * 7d540cb - [fpmsyncd]: Fix uA SID programming for link-local adjacencies (sonic-net#3958) (12 days ago) [Carmine Scarpitta] * 8541200 - [vnetorch] missing handling of rx and tx interval of monitoring session (sonic-net#3878) (12 days ago) [Jing Zhang] * 46daad0 - [syncd] Fix the error log while running lua plugin (sonic-net#3974) (12 days ago) [Vivek] * 5671e08 - Orchagent changes needed to support single ASIC VOQ Fixed-System (sonic-net#3847) (2 weeks ago) [lakshmi-nexthop] * b017bd3 - Permanent isolate a fabric port if it repeatedly flapping. (sonic-net#3933) (2 weeks ago) [jfeng-arista] * b426b2b - Support checking capabilities of the mirror (sonic-net#3934) (2 weeks ago) [Stephen Sun] * 25647cd - [fpmsyncd]: Add Support for SRv6 VPN Route and PIC Context Processing (sonic-net#3605) (2 weeks ago) [Yuqing Zhao(Alibaba Inc)] * 820eb74 - Allow state db to take modified entries made to the tunnel decap table (sonic-net#3960) (2 weeks ago) [Dev Ojha] * 5685653 - Temporarily skip failing port tests to unblock pipeline runs (sonic-net#4010) (2 weeks ago) [prabhataravind] * a4ed959 - Avoid nhgroup update when mux state changes (sonic-net#3822) (3 weeks ago) [manamand2020] * 42929d8 - dot3 Stats collection (sonic-net#3615) (3 weeks ago) [Brad House - NextHop] * ffea522 - [portsorch] fix crash when number of PGs returned 0 (sonic-net#3966) (3 weeks ago) [Stepan Blyshchak] * ea54ff8 - [ci] Migrate agent pool from sonicbld-1es to sonicso1ES-amd64 (sonic-net#3987) (3 weeks ago) [Liu Shilong] * 0adab60 - [fpmsyncd] skip routes for eth1-midplane (sonic-net#3724) (3 weeks ago) [arista-nwolfe] ``` #### How I did it #### How to verify it #### Description for the changelog Signed-off-by: Feng Pan <fenpan@microsoft.com>

The change of port mac would trigger teamd to send LACP pdu instantly

764b145

Signed-off-by: shine.chen <shine.chen@mediatek.com>

shine4chen changed the title ~~The change of port mac would trigger teamd to send LACP pdu instantly~~ [MCLAG] the change of port mac would trigger teamd to send LACP pdu instantly Nov 15, 2019

Merge branch 'master' into libteamd

fb1ed39

shine4chen requested a review from pavel-shirshov February 12, 2020 03:24

shine.chen and others added 3 commits February 20, 2020 20:28

rename patch serial number

af83fa4

Signed-off-by: shine.chen <shine.chen@mediatek.com>

Merge pull request #1 from Azure/master

b0e2c7c

merge master branch

rename patch name

bdc370c

Signed-off-by: shine.chen <shine.chen@mediatek.com>

prsunny approved these changes Feb 21, 2020

View reviewed changes

lguohan force-pushed the master branch from 3690c1a to 512eb6b Compare December 25, 2020 18:34

shine4chen requested a review from lguohan as a code owner February 6, 2021 20:29

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[MCLAG] the change of port mac would trigger teamd to send LACP pdu instantly#3764

[MCLAG] the change of port mac would trigger teamd to send LACP pdu instantly#3764
shine4chen wants to merge 5 commits intosonic-net:masterfrom
shine4chen:libteamd

shine4chen commented Nov 15, 2019 •

edited

Loading

Uh oh!

shine4chen commented Nov 15, 2019

Uh oh!

shine4chen commented Nov 16, 2019

Uh oh!

shine4chen commented Feb 21, 2020

Uh oh!

shine4chen commented Feb 21, 2020

Uh oh!

madhukar-kamarapu commented Feb 22, 2020 •

edited

Loading

Uh oh!

shine4chen commented Feb 24, 2020

Uh oh!

madhukar-kamarapu commented Feb 24, 2020

Uh oh!

shine4chen commented Feb 25, 2020

Uh oh!

shine4chen commented Feb 26, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

shine4chen commented Nov 15, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

shine4chen commented Nov 15, 2019

Uh oh!

shine4chen commented Nov 16, 2019

Uh oh!

shine4chen commented Feb 21, 2020

Uh oh!

shine4chen commented Feb 21, 2020

Uh oh!

madhukar-kamarapu commented Feb 22, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

shine4chen commented Feb 24, 2020

Uh oh!

madhukar-kamarapu commented Feb 24, 2020

Uh oh!

shine4chen commented Feb 25, 2020

Uh oh!

shine4chen commented Feb 26, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

shine4chen commented Nov 15, 2019 •

edited

Loading

madhukar-kamarapu commented Feb 22, 2020 •

edited

Loading