Skip to content

[action] [PR:277] Fix ProcessStatsST column name issue and add test case to cover check#286

Merged
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/277
Jul 10, 2025
Merged

[action] [PR:277] Fix ProcessStatsST column name issue and add test case to cover check#286
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/277

Conversation

@mssonicbld
Copy link

What I did
Fix ProcessStatsST column issue

Why I did it
#176, when did optimization of the state_db update into batch as dict, not realized "%" is missed, which is a regression issue.
image

How I verified it
Correct typo and added corresponding unit test to cover column name check.

What I did
Fix ProcessStatsST column issue

Why I did it
sonic-net#176, when did optimization of the state_db update into batch as dict, not realized "%" is missed, which is a regression issue.
![image](https://github.com/user-attachments/assets/b544ce77-9670-479b-822f-5b7ca74192ba)

How I verified it
Correct typo and added corresponding unit test to cover column name check.
@mssonicbld
Copy link
Author

Original PR: #277

@mssonicbld
Copy link
Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit d86b612 into sonic-net:202505 Jul 10, 2025
5 checks passed
gpunathilell pushed a commit to gpunathilell/sonic-host-services that referenced this pull request Sep 24, 2025
```<br>* 5f79c8f - (HEAD -> 202506, origin/202505) Improve LC reboot cause for supervisor heartbeat loss (sonic-net#291) (2025-07-16) [mssonicbld]
* d86b612 - Fix ProcessStatsST column name issue and add test case to cover check (sonic-net#286) (2025-07-10) [mssonicbld]<br>```
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