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

[YangModel][asic_sensors] Add ASIC_SENSORS yang models #21218

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

mlok-nokia
Copy link
Contributor

@mlok-nokia mlok-nokia commented Dec 18, 2024

Why I did it

Add Yang Models support for ASIC_SENSORS configuration. Fixes #20633

Work item tracking
  • Microsoft ADO (number only):

How I did it

Add Yang Models support for the following ASIC_SENSORs configuration

{
   "ASIC_SENSORS": {
        "ASIC_SENSORS_POLLER_INTERVAL": {
            "interval": "10"
        },
        "ASIC_SENSORS_POLLER_STATUS": {
            "admin_status": "enable"
        }
    }
}

How to verify it

  1. Image Build should be passed.

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

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

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)

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xwjiang-ms
Copy link
Contributor

/azpw ms_conflict

@wangxin wangxin merged commit cb2286e into sonic-net:master Dec 19, 2024
22 checks passed
@mlok-nokia
Copy link
Contributor Author

@judyjoseph and @arlakshm This PR needs cherrypick to 202405 branch. Thanks

@judyjoseph
Copy link
Contributor

Related to : #20826

@bingwang-ms
Copy link
Contributor

@mlok-nokia Can you raise a PR to 202405 to address conflict?

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.

Need YANG model for ASIC_SENSORS
7 participants