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

[202311] [YANG]: Add Yang model support for adding dom_polling to PORT table (#18277) #18344

Merged
merged 1 commit into from
Mar 21, 2024

Conversation

mihirpat1
Copy link
Contributor

202311 Cherry-pick for #18277

Why I did it

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.

Work item tracking
  • Microsoft ADO (number only):
    27069573

How I did it

Added dom_polling field to CONFIG_DB PORT table.

How to verify it

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

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

…onic-net#18277)

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
@mihirpat1 mihirpat1 requested a review from qiluo-msft as a code owner March 12, 2024 18:21
@mihirpat1 mihirpat1 requested a review from yxieca March 12, 2024 18:22
@yxieca yxieca merged commit 4807c94 into sonic-net:202311 Mar 21, 2024
18 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.

2 participants