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

[code sync] Merge code from sonic-net/sonic-buildimage:202305 to 202305 #304

Merged
merged 3 commits into from
Mar 18, 2024

Conversation

mssonicbld
Copy link
Collaborator

* 12dcaf453 - (head/202305) [YANG]: Add Yang model support for adding dom_polling to PORT table (#18277) (#18343) (2024-03-18) [mihirpat1]
* 913fbce91 - Update cisco-8000.ini (#18381) (2024-03-18) [James An]<br>```

jamesan47 and others added 3 commits March 18, 2024 09:54
Why I did it
Release notes for Cisco 8102-64H-O, 8101-32FH-O, and 8111-32EH-O
• Fix to address Netscan packet drop issue
• Fix for show platform npu router route-table CLI
• Addressed testcase errors for test_drop_counter syslog
• Addressed testcase failures for test_thermal.TestThermalApi for 8111
• Double commit of fix for ECC errors for 202305

How I did it
Update platform version to 202305.1.0.9
…#18277) (#18343)

Added YANG related changes for adding `dom_polling` field in PORT table of CONFIG_DB. This field can be set with `config interface transceiver dom PORT_NAME (enable|disable)` CLI.

The `dom_polling` field was added through sonic-net/sonic-utilities#3187. Please refer to this PR for the details on the reason for adding `dom_polling` field.

Added `dom_polling` field to CONFIG_DB PORT table.

Added unit tests for both valid and invalid options for controlling `dom_polling`.
Valid values for for `dom_polling` are `enabled` and `disabled`
Any other value is treated as an invalid value
@mssonicbld mssonicbld requested a review from qiluo-msft as a code owner March 18, 2024 03:02
@mssonicbld mssonicbld merged commit 22ca1b0 into Azure:202305 Mar 18, 2024
3 checks passed
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