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

SRv6: update for uSID endpoint behavior mapping #3414

Merged
merged 11 commits into from
Feb 4, 2025

Conversation

Yakiv-Huryk
Copy link
Contributor

What I did
Updated the mapping for uSID-related endpoint behavior to use new SAI attributes.

Why I did it
To correctly support a set of uSID behaviors.

How I verified it
Run vstest (test_srv6.py)

Details if related

Signed-off-by: Yakiv Huryk <yhuryk@nvidia.com>
@kperumalbfn
Copy link
Contributor

@Yakiv-Huryk Recently sairedis SAI refpoint moved to 1.15.1 - sonic-net/sonic-sairedis#1481

We could merge this after SAI refpoint update to 1.15.2 or later.

@dgsudharsan
Copy link
Collaborator

@kperumalbfn Can you please clarify when SAI will be updated in master branch give we have now 202411 branched out.

@dgsudharsan
Copy link
Collaborator

PLEASE DON'T MERGE

Copy link
Collaborator

@dgsudharsan dgsudharsan left a comment

Choose a reason for hiding this comment

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

Please do not merge until SAI is updated

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kperumalbfn
Copy link
Contributor

@dgsudharsan SAIA v1.15.3 has been created with the changes.

Copy link
Contributor

@kperumalbfn kperumalbfn left a comment

Choose a reason for hiding this comment

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

LGTM

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@BYGX-wcr
Copy link

LGTM and tested in VS

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@abdosi
Copy link
Contributor

abdosi commented Jan 29, 2025

@dgsudharsan : please approve this PR

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@abdosi
Copy link
Contributor

abdosi commented Jan 31, 2025

/arp run Azure.sonic-swss

@mssonicbld
Copy link
Collaborator

/azp run

1 similar comment
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Yakiv-Huryk
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to msft-202412: Azure/sonic-swss.msft#26

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.

10 participants