Skip to content

[C1]Added skip for ip/test_forward_ip_packet_with_0xffff_chksum_drop for IXS-C1 platform#25866

Merged
yejianquan merged 1 commit into
sonic-net:masterfrom
harish-kalyanaraman:C1-skip_ip_packet
Jul 7, 2026
Merged

[C1]Added skip for ip/test_forward_ip_packet_with_0xffff_chksum_drop for IXS-C1 platform#25866
yejianquan merged 1 commit into
sonic-net:masterfrom
harish-kalyanaraman:C1-skip_ip_packet

Conversation

@harish-kalyanaraman

Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Skipping ip/test_ip_packet.py::test_forward_ip_packet_with_0xffff_chksum_drop for IXS-7215-C1 platforms as ip packets with 0xffff checksum is tolerated
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202311
  • 202405
  • 202411
  • 202505
  • 202511
  • 202512
  • 202605

Tracking issue/work item for backport/cherry-pick request:
Failure type:

Approach

What is the motivation for this PR?

IXS-7215-C1 platform can tolerate ip packets with 0xffff checksum. So need to skip testcase that expects dropping of such packets

How did you do it?

Added skip condition in test_mark_conditions.yaml for IXS-C1 platforms

How did you verify/test it?

Ran ip_packet.py tests on IXS-C1 platform

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

…as IP packets with 0xffff checksum is tolerated by Nokia-C1 platforms

Signed-off-by: kalyanar <harish.kalyanaraman@nokia.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

@StormLiangMS @wangxin @yxieca A user wants to merge changes to the conditional mark files into master. Please review.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@yejianquan yejianquan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mssonicbld

Copy link
Copy Markdown
Collaborator

Cherry-pick PR to msft-202603: Azure/sonic-mgmt.msft#1288

zypgithub pushed a commit to zypgithub/sonic-mgmt that referenced this pull request Jul 8, 2026
…for IXS-C1 platform (sonic-net#25866)

<!--
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:
Skipping
ip/test_ip_packet.py::test_forward_ip_packet_with_0xffff_chksum_drop for
IXS-7215-C1 platforms as ip packets with 0xffff checksum is tolerated
Fixes # (issue)
<!--
If you request a backport/cherry-pick below, link the GitHub issue or
ADO work
item here (for example, "Fixes #<issue>" or "ADO: <work item URL>").
-->

### 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
    - [x] Skipped for non-supported platforms
- [ ] Test case improvement


### Back port request
<!--
Only check a release or feature branch when the PR links a GitHub issue
or ADO
work item above. The linked tracker should explain the failure in
detail,
including whether it is a day-one issue or a regression, the affected
branch/image/platform/test, and why this branch needs the fix. Backport
or
cherry-pick requests without a linked issue/work item may not be
favored.
-->
- [ ] 202311
- [ ] 202405
- [ ] 202411
- [ ] 202505
- [ ] 202511
- [ ] 202512
- [ ] 202605

Tracking issue/work item for backport/cherry-pick request:
Failure type: <!-- day-one issue / regression / other -->

### Approach
#### What is the motivation for this PR?
IXS-7215-C1 platform can tolerate ip packets with 0xffff checksum. So
need to skip testcase that expects dropping of such packets
#### How did you do it?
Added skip condition in test_mark_conditions.yaml for IXS-C1 platforms
#### How did you verify/test it?
Ran ip_packet.py tests on IXS-C1 platform
#### 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?
-->

Signed-off-by: kalyanar <harish.kalyanaraman@nokia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants