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 stuck in restarting - healthcheck issue #392

Closed
struppinet opened this issue Jan 23, 2025 · 1 comment
Closed

Server stuck in restarting - healthcheck issue #392

struppinet opened this issue Jan 23, 2025 · 1 comment

Comments

@struppinet
Copy link

struppinet commented Jan 23, 2025

Hi,

server was working fine for a while but today I had the case that it didn't survive a restart and was then stuck in a restart loop until I manually changed the healthcheck.sh. Then the server was working fine after it had no timeout limit. It just took longer.

I assume it was some bandwidth issues with steam so the update check/client update took longer but I think there should be a case that handles steam issues as well if the server isn't even running yet.

Docker logs from the restart:

gaming_satisfactory.1.7fjd05mebwop@v2202201165783175348    | [  0%] Checking for available updates...
gaming_satisfactory.1.7fjd05mebwop@v2202201165783175348    | UpdateUI: skip show logo[----] !!! Fatal Error: Steamcmd needs to be online to update. Please confirm your network connection and try again.
gaming_satisfactory.1.7fjd05mebwop@v2202201165783175348    | src/tier0/threadtools.cpp (3601) : Assertion Failed: Illegal termination of worker thread 'Thread(0x0x5852f0c0/0x0xf76c9a'
gaming_satisfactory.1.tfmsbtjq8qoi@v2202201165783175348    | ===== Satisfactory Server 1.9.1 =====

Nevertheless: THANK YOU for your work! Really appreciate it to not have to handle the image myself :)

@wolveix
Copy link
Owner

wolveix commented Jan 23, 2025

Hey @struppinet, yeah this was a known concern when the healthcheck was first introduced. steamcmd. What you're suggesting isn't really possible unfortunately...perhaps we could just bump the initial grace period to 300s from 180s, and increase the timeout interval from 10s to 30s

wolveix added a commit that referenced this issue Jan 23, 2025
…user` directly to the container (#389)

Increase healthcheck timeout (#392)
@wolveix wolveix closed this as completed Jan 23, 2025
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

No branches or pull requests

2 participants