Skip to content

[hash] update ECMP/LAG hash VS lib with SAI_NATIVE_HASH_FIELD_IPV6_FLOW_LABEL#1508

Merged
kcudnik merged 1 commit intosonic-net:masterfrom
ayurkiv-nvda:master_ipv6_flow_label_upstream
Feb 21, 2025
Merged

[hash] update ECMP/LAG hash VS lib with SAI_NATIVE_HASH_FIELD_IPV6_FLOW_LABEL#1508
kcudnik merged 1 commit intosonic-net:masterfrom
ayurkiv-nvda:master_ipv6_flow_label_upstream

Conversation

@ayurkiv-nvda
Copy link
Contributor

@ayurkiv-nvda ayurkiv-nvda commented Jan 24, 2025

Generic Hash feature:
update ECMP/LAG hash VS lib with SAI_NATIVE_HASH_FIELD_IPV6_FLOW_LABEL
Updated VS unittest tests

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ayurkiv-nvda ayurkiv-nvda force-pushed the master_ipv6_flow_label_upstream branch from 7a1b5fa to 4a1211a Compare January 24, 2025 10:24
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

kcudnik
kcudnik previously approved these changes Jan 24, 2025
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ayurkiv-nvda ayurkiv-nvda force-pushed the master_ipv6_flow_label_upstream branch from 882419d to 488934f Compare February 3, 2025 14:30
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ayurkiv-nvda ayurkiv-nvda marked this pull request as draft February 4, 2025 18:55
@ayurkiv-nvda ayurkiv-nvda force-pushed the master_ipv6_flow_label_upstream branch from 488934f to 80616c6 Compare February 10, 2025 19:22
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ayurkiv-nvda ayurkiv-nvda marked this pull request as ready for review February 11, 2025 08:40
@prsunny
Copy link
Contributor

prsunny commented Feb 20, 2025

@kcudnik , please merge

@kcudnik kcudnik merged commit 2564354 into sonic-net:master Feb 21, 2025
15 checks passed
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to msft-202412: Azure/sonic-sairedis.msft#17

@dgsudharsan
Copy link
Collaborator

@r12f This was reverted by you in 202412 branch. Can you please update labels so that it doesn't mislead stating the changes are included in 202412 branch?

qiluo-msft pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Mar 24, 2025
Related PRs:
sonic-net/sonic-utilities#3730
sonic-net/sonic-sairedis#1508
sonic-net/sonic-swss#3481

Why I did it
Need to add SAI_NATIVE_HASH_FIELD_IPV6_FLOW_LABEL for hash calculation. Need to align Yang model.

Work item tracking
Microsoft ADO (number only):
How I did it
Added IPV6_FLOW_LABEL to Generic Hash Yang model (both ecmp_hash and lag_hash). path: SWITCH_HASH/GLOBAL
Updated existing UT.
Updated Configuration.md

How to verify it
1)make configure PLATFORM=mellanox
2)make target/sonic-mellanox.bin
3)check model
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.

7 participants