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] Add attribute version check feature #1470

Merged
merged 1 commit into from
Nov 22, 2024
Merged

Conversation

kcudnik
Copy link
Collaborator

@kcudnik kcudnik commented Nov 21, 2024

By default disabled. If enabled, it will use SAI metadata and libsai version returned by sai_api_version_query to verify if given attribute during SAI discovery process was introduced in the current libsai version or below that version.

This feature is partial solution to this problem:
sonic-net/sonic-buildimage#20725, more here: opencomputeproject/SAI#2099

By default disabled. If enabled, it will use SAI metadata and libsai
version returned by sai_api_version_query to verify if given attribute
during SAI discovery process was introduced in the current libsai
version or below that version.

This feature is partial solution to this problem:
sonic-net/sonic-buildimage#20725,
more here: opencomputeproject/SAI#2099
@kcudnik kcudnik requested review from lguohan and yxieca November 21, 2024 16:51
@kcudnik kcudnik merged commit e6ec142 into sonic-net:master Nov 22, 2024
18 checks passed
@kcudnik kcudnik deleted the attr2 branch November 22, 2024 13:35
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]
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