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

Server process manager now displays updated process information #9685

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nicolasbrossard
Copy link
Contributor

This PR modifies the server process manager so that it properly updates the status of processes before displaying the search result list. This fixes a bug that would cause some processes to be listed with an unknown exit code and/or exit text message.

Fixes #8593

@nicolasbrossard nicolasbrossard added Module: server_processes_manager PR or issue related to the server_processes_manger php Pull requests that update Php code 27.0.0 - Bugs Bugs Found in LORIS 27 testing labels Mar 7, 2025
Copy link
Contributor

@jeffersoncasimir jeffersoncasimir left a comment

Choose a reason for hiding this comment

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

This indeed fixes #8593.

exit_* columns are updated as they should on Failure

Also closes #9666

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
27.0.0 - Bugs Bugs Found in LORIS 27 testing Module: server_processes_manager PR or issue related to the server_processes_manger php Pull requests that update Php code
Projects
None yet
2 participants