Skip to content

Fix timeout bug for LEAN4 code execution#647

Merged
Kipok merged 3 commits intomainfrom
fchen97/fixlean4
Aug 7, 2025
Merged

Fix timeout bug for LEAN4 code execution#647
Kipok merged 3 commits intomainfrom
fchen97/fixlean4

Conversation

@fchen97
Copy link
Contributor

@fchen97 fchen97 commented Aug 7, 2025

  • The default Nginx timeout is 60 seconds, which is insufficient for long-running processes. Updated the Nginx configuration to set the timeout to 4 hours (14,400 seconds).
  • Bug: After a timeout occurs, the LEAN4 subprocess continued running in the background.
    Fix: On timeout during LEAN4 code execution, the entire process tree is now forcefully terminated.

Docker image needs to be rebuilt to incorporate these changes.

Help from @gwarmstrong.

fchen97 added 2 commits August 6, 2025 17:16
Signed-off-by: Feng Chen <42473790+fchen97@users.noreply.github.com>
@fchen97 fchen97 requested review from Kipok and gwarmstrong August 7, 2025 07:18
Copy link
Collaborator

@Kipok Kipok left a comment

Choose a reason for hiding this comment

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

thanks!

@Kipok
Copy link
Collaborator

Kipok commented Aug 7, 2025

container is updated everywhere

@Kipok Kipok merged commit d75de8e into main Aug 7, 2025
4 checks passed
@fchen97 fchen97 deleted the fchen97/fixlean4 branch August 7, 2025 21:26
i-vainn pushed a commit that referenced this pull request Aug 12, 2025
Signed-off-by: Feng Chen <42473790+fchen97@users.noreply.github.com>
Co-authored-by: Igor Gitman <igitman@nvidia.com>
Signed-off-by: i-vainn <1vanmoshkov@mail.ru>
wasiahmad pushed a commit that referenced this pull request Oct 1, 2025
Signed-off-by: Feng Chen <42473790+fchen97@users.noreply.github.com>
Co-authored-by: Igor Gitman <igitman@nvidia.com>
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.

2 participants