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

   ## Purpose
   
   The Server container watched a daemon JVM with `tail --pid` and returned 1 
whenever it stopped. A normal JVM exit 0 and an application exit 42 therefore 
looked identical. The entrypoint now owns the existing foreground launcher and 
returns its status after readiness, including 137 for SIGKILL.
   
   ## Changes
   
   - Preserve the existing REST startup deadline and HStore partition check; a 
JVM that exits normally before readiness still fails startup.
   - Keep readiness probes and partition checks interruptible, and 
terminate/reap their owned processes during shutdown.
   - Ignore duplicate group/forwarded signals while the launcher reaps its JVM. 
TERM and INT request cooperative shutdown and return 143 and 130.
   - Document timeout and shutdown behavior, and run the new native-process 
regression from the existing entrypoint suite.
   
   ## Validation
   
   The native lifecycle cases passed, covering normal and failing exits, 
SIGKILL, direct and process-group signals, early exits, startup deadlines, 
pending partition checks and real stalled HTTP readiness probes. They also 
check recorded child processes are gone after cooperative shutdown. The 
existing configuration, encoding and foreground launcher signal suites passed. 
Formatting and a clean Java 17 reactor compile passed; three independent 
reviews and re-review found no remaining actionable issue.
   
   The process tests exercise the actual shell scripts with a small JVM 
substitute. Real Linux container/OOM and Kubernetes acceptance remain separate 
follow-up checks. A startup timeout requests termination and returns 1 without 
adding an indefinite wait; an unresponsive JVM still depends on container 
termination for final cleanup.
   
   The same org branch passed its selected CI, including startup contracts, 
Server/RocksDB/HStore and cluster tests, container/deployment checks, platform 
compatibility and CodeQL.
   
   Org validation: https://github.com/hugegraph/hugegraph/pull/289
   
   ## Documentation
   
   Doc - Done: `docker/README.md`.
   


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