Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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 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

@jeffersoncasimir jeffersoncasimir linked an issue Mar 7, 2025 that may be closed by this pull request
@driusan driusan merged commit 4611fed into aces:main Mar 10, 2025
19 checks passed
driusan pushed a commit that referenced this pull request Apr 14, 2025
This adds a condition for which processes to monitor before loading
the data table, significantly improving page load time.

It will only monitor `server_processes` that have either `end_time` or
`exit_code` equal to `NULL` (default)

Feature introduced by #9685.
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

Projects

None yet

3 participants