Skip to content

Support ASIC/SDK health event#3129

Merged
qiluo-msft merged 11 commits intosonic-net:masterfrom
stephenxs:asic-sdk-health-event
May 13, 2024
Merged

Support ASIC/SDK health event#3129
qiluo-msft merged 11 commits intosonic-net:masterfrom
stephenxs:asic-sdk-health-event

Conversation

@stephenxs
Copy link
Collaborator

@stephenxs stephenxs commented Jan 23, 2024

What I did

Support ASIC/SDK health event

  1. config asic-sdk-health-event suppress
  2. show asic-sdk-health-event [received|suppress]
  3. sonic-clear asic-sdk-health-event

Depends on sonic-net/sonic-buildimage#17879

How I did it

How to verify it

Unit test

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

Signed-off-by: Stephen Sun <stephens@nvidia.com>
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jan 23, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Eliminate oldest events if the number of events exceed the threshold

Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
@stephenxs stephenxs force-pushed the asic-sdk-health-event branch from 8cf1def to d9b2425 Compare January 23, 2024 02:22
@stephenxs stephenxs changed the title Asic sdk health event Support Asic sdk health event Jan 23, 2024
@stephenxs stephenxs changed the title Support Asic sdk health event Support ASIC/SDK health event Feb 14, 2024
@stephenxs stephenxs marked this pull request as ready for review March 14, 2024 09:45
@stephenxs
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@zhangyanzhao
Copy link
Collaborator

reviewers, can you please help to complete the review and merge the code by 5/30? Thanks.

@stephenxs
Copy link
Collaborator Author

@kperumalbfn @prabhataravind Could you please review this PR? We need to finish it by the end of May. Thank you very much!

Copy link
Contributor

@prabhataravind prabhataravind left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a minor comment.

@qiluo-msft qiluo-msft merged commit 8629b68 into sonic-net:master May 13, 2024
@stephenxs stephenxs deleted the asic-sdk-health-event branch May 13, 2024 21:52
arfeigin pushed a commit to arfeigin/sonic-utilities that referenced this pull request Jun 16, 2024
#### What I did

Support ASIC/SDK health event
1. config asic-sdk-health-event suppress
2. show asic-sdk-health-event [received|suppress]
3. sonic-clear asic-sdk-health-event

Depends on sonic-net/sonic-buildimage#17879

#### How to verify it

Unit test
nmoray pushed a commit to nmoray/sonic-utilities that referenced this pull request Jun 25, 2025
#### What I did

Support ASIC/SDK health event
1. config asic-sdk-health-event suppress
2. show asic-sdk-health-event [received|suppress]
3. sonic-clear asic-sdk-health-event

Depends on sonic-net/sonic-buildimage#17879

#### How to verify it

Unit test
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.

6 participants