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

Update usage leaf in sonic-events-host yang models #15805

Merged
merged 5 commits into from
Jul 18, 2023

Conversation

zbud-msft
Copy link
Contributor

@zbud-msft zbud-msft commented Jul 12, 2023

Why I did it

event yang models for usage currently use int as type for usage leaf, needs to be of type decimal64

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

How I did it

Update yang models and UT

How to verify it

UT

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)

@zbud-msft zbud-msft changed the title Update host yang models Update usage leaf in sonic-events-host yang models Jul 12, 2023
@zbud-msft zbud-msft marked this pull request as ready for review July 12, 2023 22:59
@zbud-msft zbud-msft requested a review from qiluo-msft as a code owner July 12, 2023 22:59
@zbud-msft zbud-msft requested review from qiluo-msft and removed request for qiluo-msft July 12, 2023 22:59
@qiluo-msft qiluo-msft requested a review from ganglyu July 13, 2023 23:03
@qiluo-msft qiluo-msft merged commit e2a58ac into sonic-net:master Jul 18, 2023
@qiluo-msft qiluo-msft added the YANG YANG model related changes label Jul 18, 2023
@zbud-msft zbud-msft requested a review from StormLiangMS July 18, 2023 17:32
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Aug 14, 2023
#### Why I did it

event yang models for usage currently use int as type for usage leaf, needs to be of type decimal64

##### Work item tracking
- Microsoft ADO **(number only)**:17747466

#### How I did it

Update yang models and UT

#### How to verify it

UT
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202305: #16136

StormLiangMS pushed a commit that referenced this pull request Aug 14, 2023
#### Why I did it

event yang models for usage currently use int as type for usage leaf, needs to be of type decimal64

##### Work item tracking
- Microsoft ADO **(number only)**:17747466

#### How I did it

Update yang models and UT

#### How to verify it

UT

Co-authored-by: Zain Budhwani <99770260+zbud-msft@users.noreply.github.com>
zbud-msft added a commit to zbud-msft/sonic-buildimage that referenced this pull request Aug 15, 2023
#### Why I did it

event yang models for usage currently use int as type for usage leaf, needs to be of type decimal64

##### Work item tracking
- Microsoft ADO **(number only)**:17747466

#### How I did it

Update yang models and UT

#### How to verify it

UT
StormLiangMS pushed a commit that referenced this pull request Aug 15, 2023
* [yang] Change swss-event, dhcp-relay-event leafref to string (#13326)

Why I did it
Do not require leafref as part of yang. Only need string to compare whether string received from event matches what is possible for ifname.

How I did it
How to verify it
Run UT

* Add fix to monit_regex.json for catching mem_usage and cpu_usage (#14954)

Why I did it
Current regex not able to capture logs, modify regex to capture syslog messages

Work item tracking
Microsoft ADO (number only): 13366345
How I did it
Code change

How to verify it
sonic-mgmt test case

* Update usage leaf in sonic-events-host yang models (#15805)

#### Why I did it

event yang models for usage currently use int as type for usage leaf, needs to be of type decimal64

##### Work item tracking
- Microsoft ADO **(number only)**:17747466

#### How I did it

Update yang models and UT

#### How to verify it

UT
sonic-otn pushed a commit to sonic-otn/sonic-buildimage that referenced this pull request Sep 20, 2023
#### Why I did it

event yang models for usage currently use int as type for usage leaf, needs to be of type decimal64

##### Work item tracking
- Microsoft ADO **(number only)**:17747466

#### How I did it

Update yang models and UT

#### How to verify it

UT
@zbud-msft zbud-msft deleted the update_host_yang_models branch February 12, 2025 18:10
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.

5 participants