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

[System ready] Report ready state to Sysmonitor daemon #3126

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

fastiuk
Copy link

@fastiuk fastiuk commented Apr 28, 2024

depends-on: sonic-net/sonic-buildimage#18817

What I did
This change implements swss's ability to report it's readines to sysmonitor

Why I did it
The motivation is to report PortInitDone to sysmonitor and not just the startup of swss

How I verified it

  • Run NOS on the box
  • Make sure swss reports good readiness state and you can see it in sysmonitor

Details if related

@liat-grozovik
Copy link
Collaborator

@fastiuk please refer to the comment provided. also please update the PR description of what are you are trying to fix and the scope. if this is a bug to whihc branches it should go?

portsyncd/portsyncd.cpp Show resolved Hide resolved
portsyncd/portsyncd.cpp Show resolved Hide resolved
Signed-off-by: Yevhen Fastiuk <yfastiuk@nvidia.com>
@fastiuk
Copy link
Author

fastiuk commented Aug 14, 2024

@prsunny any more comments?

@fastiuk
Copy link
Author

fastiuk commented Aug 14, 2024

/azpw run Azure.sonic-swss

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-swss

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

4 participants