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

[syncd] Introduce VendorSaiOptions class #1472

Merged
merged 2 commits into from
Nov 27, 2024
Merged

Conversation

kcudnik
Copy link
Collaborator

@kcudnik kcudnik commented Nov 22, 2024

And SaiOptions interface, for easy passing parameters over Sai objects

This will simplify passing parameters over different objects that have SaiInterface.

And SaiOptions interface, for easy passing parameters
over Sai objects
@kcudnik kcudnik merged commit 7835026 into sonic-net:master Nov 27, 2024
18 checks passed
@kcudnik kcudnik deleted the options branch November 27, 2024 11:42
yxieca added a commit that referenced this pull request Dec 5, 2024
yxieca added a commit to yxieca/sonic-sairedis that referenced this pull request Dec 5, 2024
yxieca added a commit that referenced this pull request Dec 6, 2024
Need to keep master branch at SAI header version 1.15 until 202411 branch is cut.

2dbaca8 2024-12-05 | Revert "[submodule] Update SAI to lates origin master ([submodule] Update SAI to latest origin master #1466)" (HEAD -> 202411, yingfork/revert-back-to-1-15) [Ying Xie]
064a3a8 2024-12-05 | Revert "[meta] Use metadata to check port notification port_id type ([meta] Use metadata to check port notification port_id type #1469)" [Ying Xie]
0383740 2024-12-05 | Revert "[syncd] Add attribute version check feature ([syncd] Add attribute version check feature #1470)" [Ying Xie]
0901dd7 2024-12-05 | Revert "[syncd] Introduce VendorSaiOptions class ([syncd] Introduce VendorSaiOptions class #1472)" [Ying Xie]
shiraez pushed a commit to Marvell-switching/sonic-sairedis that referenced this pull request Dec 12, 2024
And SaiOptions interface, for easy passing parameters over Sai objects

This will simplify passing parameters over different objects that have SaiInterface.
shiraez pushed a commit to Marvell-switching/sonic-sairedis that referenced this pull request Dec 12, 2024
Need to keep master branch at SAI header version 1.15 until 202411 branch is cut.

2dbaca8 2024-12-05 | Revert "[submodule] Update SAI to lates origin master ([submodule] Update SAI to latest origin master sonic-net#1466)" (HEAD -> 202411, yingfork/revert-back-to-1-15) [Ying Xie]
064a3a8 2024-12-05 | Revert "[meta] Use metadata to check port notification port_id type ([meta] Use metadata to check port notification port_id type sonic-net#1469)" [Ying Xie]
0383740 2024-12-05 | Revert "[syncd] Add attribute version check feature ([syncd] Add attribute version check feature sonic-net#1470)" [Ying Xie]
0901dd7 2024-12-05 | Revert "[syncd] Introduce VendorSaiOptions class ([syncd] Introduce VendorSaiOptions class sonic-net#1472)" [Ying Xie]
shiraez pushed a commit to Marvell-switching/sonic-sairedis that referenced this pull request Dec 12, 2024
And SaiOptions interface, for easy passing parameters over Sai objects

This will simplify passing parameters over different objects that have SaiInterface.
shiraez pushed a commit to Marvell-switching/sonic-sairedis that referenced this pull request Dec 12, 2024
Need to keep master branch at SAI header version 1.15 until 202411 branch is cut.

2dbaca8 2024-12-05 | Revert "[submodule] Update SAI to lates origin master ([submodule] Update SAI to latest origin master sonic-net#1466)" (HEAD -> 202411, yingfork/revert-back-to-1-15) [Ying Xie]
064a3a8 2024-12-05 | Revert "[meta] Use metadata to check port notification port_id type ([meta] Use metadata to check port notification port_id type sonic-net#1469)" [Ying Xie]
0383740 2024-12-05 | Revert "[syncd] Add attribute version check feature ([syncd] Add attribute version check feature sonic-net#1470)" [Ying Xie]
0901dd7 2024-12-05 | Revert "[syncd] Introduce VendorSaiOptions class ([syncd] Introduce VendorSaiOptions class sonic-net#1472)" [Ying Xie]
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.

1 participant