Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[202311][Mellanox]Enable forwarding of packets with link local SIP/DIP on specific platforms #18692

Merged
merged 2 commits into from
Jun 7, 2024

Conversation

dgsudharsan
Copy link
Collaborator

@dgsudharsan dgsudharsan commented Apr 16, 2024

Cherry-pick of #18487.

Must be merged after #18570

Why I did it

Enable forwarding of packets with link local SIP/DIP on specific platforms and SKUs.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Setting SAI key value pair SAI_NOT_DROP_SIP_DIP_LINK_LOCAL=1

How to verify it

Manual test to verify that packets with link local SIP/DIP are not dropped

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

…cific platforms (sonic-net#18487)

- Why I did it
Enable forwarding of packets with link local SIP/DIP on specific platforms and SKUs.

- How I did it
Setting SAI key value pair SAI_NOT_DROP_SIP_DIP_LINK_LOCAL=1

- How to verify it
Manual test to verify that packets with link local SIP/DIP are not dropped
@dgsudharsan
Copy link
Collaborator Author

dgsudharsan commented Apr 16, 2024

@yxieca Can you please help to review and merge this cherry-pick once this is merged #18570?

@dgsudharsan dgsudharsan marked this pull request as draft April 16, 2024 22:43
@yxieca
Copy link
Contributor

yxieca commented Apr 18, 2024

@dgsudharsan is this PR still in draft?

@dgsudharsan
Copy link
Collaborator Author

@yxieca It depends on merge of #18570

@dgsudharsan
Copy link
Collaborator Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dgsudharsan
Copy link
Collaborator Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dgsudharsan
Copy link
Collaborator Author

@yxieca Can you please help to merge this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants