bitflicker64 opened a new pull request, #3187:
URL: https://github.com/apache/hugegraph/pull/3187

   ## Purpose of the PR
   
   - close #3186
   
   `docker-entrypoint.sh` runs `start-hugegraph.sh` with a literal `-t 120`, so 
a Server that needs longer than 120 seconds to answer on its REST port is ended 
by its own container. Under an orchestrator the startup budget belongs to the 
startup probe, and a probe cannot extend a process that terminates itself 
first: with the 450-second probe budget the Helm chart in #3132 configures, the 
process is gone at 120 seconds. Measured on 2026-09-01, 2 of 6 Server starts 
across two independent installs died this way and recovered only on container 
restart (log excerpts and the measurement setup are in the issue).
   
   ## Main Changes
   
   - `docker-entrypoint.sh` reads the timeout from 
`HG_SERVER_STARTUP_TIMEOUT_S`, defaulting to the current 120, so nothing 
changes for deployments that do not set it. Both Server images share this 
entrypoint, so one change covers the RocksDB and HStore images.
   - The value is validated as a positive whole number before init-store runs. 
A bad value fails the container immediately with a clear message instead of 
surfacing later as an arithmetic error inside `wait_for_startup`.
   - `docker-entrypoint-test.sh` now records the arguments the 
`start-hugegraph.sh` stub receives and asserts the default `-t 120`, an 
explicit `-t 450` override, and rejection of `2m` before the server would have 
been started.
   - Documented as a new section in 
`hugegraph-server/hugegraph-dist/docker/README.md`. The issue proposed the 
environment table in `docker/README.md`, but #3149 removed that table, so the 
Server docker README is the current home for entrypoint variables.
   
   ## Verifying these changes
   
   - [x] Need tests and can be verified as follows:
       - `bash 
hugegraph-server/hugegraph-dist/docker/docker-entrypoint-test.sh` (already 
wired into `server-ci.yml`) covers the default, an override, and an invalid 
value.
   
   ## Does this PR potentially affect the following parts?
   
   - [x]  Modify configurations
   
   ## Documentation Status
   
   - [x]  `Doc - Done`
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to