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

[Event listener] Add a new supervisord event listener to pmon container #28

Closed
wants to merge 2 commits into from
Closed

[Event listener] Add a new supervisord event listener to pmon container #28

wants to merge 2 commits into from

Conversation

keboliu
Copy link
Collaborator

@keboliu keboliu commented May 17, 2019

Event-listener: Add a new supervisord event listener to pmon container to clear the state DB when pmon was shut down.

  • This event listener will monitor the 'SUPERVISOR_STATE_CHANGE_STOPPING' event of supervisord when it is shutting down. On this event, the event listener will call command 'post-syseeprom -c' to clear the state DB.

Signed-off-by: Kebo Liu kebol@mellanox.com

@jleveque
Copy link
Contributor

Since this is specific to the PMON container, why not add the event listener script in https://github.com/Azure/sonic-buildimage/tree/master/dockers/docker-platform-monitor?

@keboliu
Copy link
Collaborator Author

keboliu commented May 20, 2019

Since this is specific to the PMON container, why not add the event listener script in https://github.com/Azure/sonic-buildimage/tree/master/dockers/docker-platform-monitor?

I was thinking to do so but not quite sure whether this is a good practice at that time. Since you all prefer this way, I moved it and updated PR sonic-net/sonic-buildimage#2866

@qiluo-msft qiluo-msft closed this May 20, 2019
@keboliu keboliu deleted the event-listener branch May 24, 2019 06:58
vdahiya12 pushed a commit to vdahiya12/sonic-platform-daemons that referenced this pull request Apr 4, 2022
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.

3 participants