[everflow_test]: Use separated everflow session for tests.#177
Merged
oleksandrivantsiv merged 2 commits intosonic-net:masterfrom May 16, 2017
Merged
[everflow_test]: Use separated everflow session for tests.#177oleksandrivantsiv merged 2 commits intosonic-net:masterfrom
oleksandrivantsiv merged 2 commits intosonic-net:masterfrom
Conversation
- Create and use separated everflow session for tests. - Fix issue with L4 port range in PTF test. - Handle difference in mirrored packets format on different platforms. - Do not compare mirrored packeе CHKSUM. - Remove checking Ethertype in separated testcase as it redundant. Ethertype is checked in each testcase as well as other L2 fields.
taoyl-ms
reviewed
May 10, 2017
|
|
||
| return dataplane.match_exp_pkt(pkt2send, inner_pkt) | ||
| masked_inner_pkt = Mask(inner_pkt) | ||
| masked_inner_pkt.set_do_not_care_scapy(scapy.IP, "chksum") |
Contributor
There was a problem hiding this comment.
In case that test packet is a TCP packer, we need to ignore TCP checksum as well.
taoyl-ms
approved these changes
May 15, 2017
lguohan
pushed a commit
that referenced
this pull request
Aug 3, 2017
[everflow_test]: Use separated everflow session for tests. - Create and use separated everflow session for tests. - Fix issue with L4 port range in PTF test. - Handle difference in mirrored packets format on different platforms. - Do not compare mirrored packeе CHKSUM. - Remove checking Ethertype in separated testcase as it redundant. Ethertype is checked in each testcase as well as other L2 fields.
nhe-NV
pushed a commit
to nhe-NV/sonic-mgmt
that referenced
this pull request
May 12, 2025
…olated-d28u1 and t0-isolated-d16u16s1 (sonic-net#177) <!-- 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 Handle non portchannel in the upstream ports for generic hash test script at topology t1-isolated-d28u1 and t0-isolated-d16u16s1 Summary: Fixes # (issue) ### 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 - [x] Test case improvement ### Back port request - [ ] 202012 - [ ] 202205 - [ ] 202305 - [ ] 202311 - [ ] 202405 - [ ] 202411 ### Approach #### What is the motivation for this PR? No portchannels in topology t1-isolated-d28u1 and t0-isolated-d16u16s1, and generic hash test would be failed in this scenario. #### How did you do it? Handle non portchannel in the upstream ports #### How did you verify/test it? Run in it local 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
872f0a3 [Y-Cable] refactor get_firmware_version to comply with all vendors (sonic-net#182) cc162d6 [sonic_y_cable]: Decorate all method for mux simulator (sonic-net#181) fa02416 Change import order in Ycable helper and EEPROM read bytearray change in SFP plugin (sonic-net#177) 0b60982 [thermal_base] Add setter functions for critical thresholds (sonic-net#180) 10dc16f [y_cable] add support for enable/disable autoswitch feature on Y cable (sonic-net#176) c6c81a8 [fan_drawer_base.py] Fix FanDrawer get_status_led interface (sonic-net#175) Signed-off-by: vaibhav-dahiya <vdahiya@microsoft.com>
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
…nic-net#7563) * [202012][swss/swss-common/utilities/platform-daemons] Update submodule sonic-swss - [flex-counters] Delay flex counters stats init for faster boot time [202012] (sonic-net#1736) sonic-swss-common - [swig] allow threads (sonic-net#477) sonic-utilities - [sfpshow] Gracefully handle improper 'specification_compliance' field (sonic-net#1594) sonic-platform-daemons - [xcvrd] Change the y_cable presence logic to use "mux_cable" table as identifier from Config DB (sonic-net#176) - [xcvrd] Enhance Media Settings (sonic-net#177) Signed-off-by: Danny Allen <daall@microsoft.com>
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
Include commit: ``` 6ea1f03 Jing Zhang Tue Mar 28 08:42:44 2023 -0700 [202012] remove chatty log message for peer link event (sonic-net#192) 198292d Jing Zhang Tue Mar 21 17:53:11 2023 -0700 [active-standby] avoid unnecessary mux state probe after configuring to `auto` (sonic-net#183) 47de88e Jing Zhang Mon Mar 20 18:14:25 2023 -0700 [202012] Avoid unnecessary error logs from `handleGetServerMacAddressNotification` sonic-net#96 (sonic-net#185) 8a33319 Jing Zhang Mon Mar 6 11:53:27 2023 -0800 loose link down swithcover condition (sonic-net#178) c2bf08d Jing Zhang Thu Mar 16 18:59:10 2023 -0700 fix ActiveStandbyStateMachine referrence (sonic-net#186) 99d26af Jing Zhang Thu Mar 16 18:58:48 2023 -0700 [ci] Fix apt-get install unable locate package issue. (sonic-net#177) (sonic-net#187) d893be9 Longxiang Lyu Wed Feb 22 12:55:44 2023 +0800 [active-standby] Toggle to standby if link down and config auto (sonic-net#173) ```
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
…net#14412) Why I did it src/linkmgrd * 3403160 - (HEAD -> 202205, origin/202205) [ci] fix azure pipeline (sonic-net#190) (33 hours ago) [Jing Zhang] * 95abb4c - [active-standby] avoid unnecessary mux state probe after configuring to `auto` (sonic-net#183) (33 hours ago) [Jing Zhang] * 9277650 - [ci] Fix apt-get install unable locate package issue. (sonic-net#177) (33 hours ago) [Liu Shilong] src/sonic-linux-kernel * 836aedf - (HEAD -> 202205, origin/202205) [202205] Add Secure Boot Kernel configuration (sonic-net#310) (35 hours ago) [Sachin Naik] src/sonic-platform-common * 9b367e4 - (HEAD -> 202205, origin/202205) Modify get_host_lane_assignment_option to return value based on application id (sonic-net#352) (33 hours ago) [mihirpat1] src/sonic-platform-daemons * a6015f2 - (HEAD -> 202205, origin/202205) chassisd: Fix crash on exit on linecard (sonic-net#347) (33 hours ago) [Patrick MacArthur] * dcc0079 - [Supervisor][fabric] Modify the deinit function not to remove CHASSIS_FABRIC_ASIC_TABLE when supervisorctl stop chassisd (sonic-net#340) (33 hours ago) [prem-nokia] src/sonic-swss * a2f1428 - (HEAD -> 202205, origin/202205) Run individule pytests instead of running all pytests at once. (sonic-net#2350) (33 hours ago) [mint570] src/sonic-swss-common * 643132e - (HEAD -> 202205, origin/202205) Fix mod_config with empty config will delete table bug. (sonic-net#767) (34 hours ago) [Hua Liu] * 4c7ee30 - Revamp module build script to make it work for 5.15 on Ubuntu 20.04 (sonic-net#720) (34 hours ago) [Saikrishna Arcot]
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
…ically (#25039) #### Why I did it src/sonic-restapi ``` * df38186 - (HEAD -> 202511, origin/202511) [Pfx_len] Extend prefix length for routes and advertisement (sonic-net#182) (9 days ago) [mssonicbld] * c80e7a8 - Changing the wildcard matching logic for client cert common names (sonic-net#179) (2 weeks ago) [mssonicbld] * 6770863 - Adding support for wildcard CN matching during client cert authentication (sonic-net#177) (2 weeks ago) [mssonicbld] * a521a1b - [202511] upgrade debian to bookworm (sonic-net#174) (2 weeks ago) [yijingyan2] ``` #### 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
[RFC4292][Namespace][201911]: Fix implementation of RouteUpdater for multi-asic platform (sonic-net#177) Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
sonic-net#15576) #### Why I did it src/linkmgrd ``` * fc6e299 - (HEAD -> 202211, origin/202211) [active-standby] Enforce switchover based on heartbeats when mux probe keeps failing (sonic-net#184) (2 hours ago) [Jing Zhang] * 5c6332d - Re-handle pre-init mux config notification after state machine init (sonic-net#202) (2 hours ago) [Longxiang Lyu] * c49a5ef - [active-standby][bsl] fix no mux probe issue (sonic-net#201) (2 hours ago) [Jing Zhang] * 2d62ef1 - Enable debug symbols (sonic-net#199) (2 hours ago) [Longxiang Lyu] * f952b64 - [active-active] Add mux server state correction mechanism (sonic-net#191) (2 hours ago) [Longxiang Lyu] * e6b1bfc - [ci] fix azure pipeline (sonic-net#190) (2 hours ago) [Jing Zhang] * 615ed21 - [active-standby] Enforce switchover based on heartbeats when mux probe keeps failing (sonic-net#184) (2 hours ago) [Jing Zhang] * 3adfc69 - [active-active] add state transition handler for (LinkProber: Unknown, MuxState: Active, LinkState: Down) (sonic-net#179) (2 hours ago) [Jing Zhang] * 6fb9a0b - loose link down swithcover condition (sonic-net#178) (2 hours ago) [Jing Zhang] * 8ffbd8a - [ci] Fix apt-get install unable locate package issue. (sonic-net#177) (2 hours ago) [Liu Shilong] * 3dfbdd7 - [active-active] link operational down didn't trigger toggle to standby if `MuxUnknown` event arrives first. (sonic-net#175) (2 hours ago) [Jing Zhang] * 68c06d9 - [active-active] Toggle to standby if default route is missing (sonic-net#171) (2 hours ago) [Longxiang Lyu] * 1592cd3 - [active-active] fix issue that interfaces get stuck in `active` if service starts up with link state down (sonic-net#169) (2 hours ago) [Jing Zhang] * a96105f - Fix the warning of unused variables (sonic-net#167) (2 hours ago) [Longxiang Lyu] * 984ffe3 - Fix mux config when gRPC connection is lost (sonic-net#166) (2 hours ago) [Longxiang Lyu] * 4d0b457 - [active-active] force switch when config == active && target state == active (sonic-net#164) (2 hours ago) [Jing Zhang] * 32bd2bb - Use github code scanning instead of LGTM (sonic-net#157) (2 hours ago) [Liu Shilong] * 6f23fe2 - [active-active] update linkmgr health label definition (sonic-net#163) (3 hours ago) [Jing Zhang] ``` #### How I did it #### How to verify it #### Description for the changelog
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
3 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.
Ethertype is checked in each testcase as well as other L2 fields.