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

   ## Main Changes (主要改动)
   Close #3072.
   
   - Root cause: Docker startup failures caused by denied JVM thread creation 
leave their useful `pthread_create`/EPERM explanation in crash files or the 
server log, while the entrypoint reports only startup failure.
   - Fix details: inspect crash files created during the current startup and 
newly written server log bytes when init-store or server startup fails. Print 
seccomp and process-limit troubleshooting hints for EPERM, and 
memory/process-limit guidance for generic GC-thread creation failures. Preserve 
exit status, ignore stale diagnostics, and leave runtime security settings 
unchanged. Document the diagnostic flags and normal-operation guidance in the 
Docker README.
   - Tests: extend `test-docker-entrypoint-process.sh` with current/stale crash 
and log cases, generic GC-thread failures, and init-store failure status. The 
entrypoint configuration/process fixture passes, as does 
`HugeSecurityManagerTest` through the distribution Maven reactor.
   
   ## Verifying these changes (验证改动)
   - [x] Added/updated unit tests in `test-docker-entrypoint-process.sh`
   - [x] Verified `mvn checkstyle:check` and `mvn apache-rat:check` pass locally
   
   Java 17 / Maven 3.9.11 validation:
   - `mvn apache-rat:check`
   - `mvn editorconfig:format`
   - `mvn clean compile -Dmaven.javadoc.skip=true -pl 
hugegraph-server/hugegraph-dist -am`
   - `mvn test-compile checkstyle:check -pl hugegraph-server/hugegraph-dist -am 
-Dcheckstyle.config.location=/workspace/style/checkstyle.xml`
   - `mvn test -pl hugegraph-server/hugegraph-dist -am 
-Dtest=HugeSecurityManagerTest -Dsurefire.failIfNoSpecifiedTests=false`
   - `bash hugegraph-server/hugegraph-dist/docker/docker-entrypoint-test.sh` 
(includes process and partition-wait fixtures)
   
   The explicit Checkstyle configuration applies the repository's rules to the 
distributed modules included by the reactor. No Java test class is needed for 
the shell-only change; the new regression cases run in the existing 
shell/Python process fixture.


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