zhuxiangyi opened a new pull request, #18312:
URL: https://github.com/apache/dolphinscheduler/pull/18312

   ## Was this PR generated or assisted by AI?
   
   YES. Codex/ChatGPT assisted with debugging, code changes, tests, issue 
drafting, and PR draft preparation. I reviewed the changes and verified them 
locally.
   
   ## Purpose of the pull request
   
   Fix task kill status checking when `kill -0` returns `Operation not 
permitted` for an unkillable sudo parent process or when tenant login shell 
emits stderr with exit code 0.
   
   Closes #18311.
   
   ## Brief change log
   
   - Add exit code accessor to `ExitCodeException`.
   - Distinguish alive, not-alive, and no-permission process states in 
`ProcessUtils`.
   - Continue killing permitted child processes instead of reporting success 
when only the sudo parent check is denied.
   - Add unit coverage for sudo parent denied plus killable child processes.
   
   ## Verify this pull request
   
   This change added tests and can be verified as follows:
   
   - `./mvnw -pl 
dolphinscheduler-common,dolphinscheduler-task-plugin/dolphinscheduler-task-api 
-DskipTests -Djacoco.skip=true spotless:check`
   - `./mvnw -pl dolphinscheduler-task-plugin/dolphinscheduler-task-api -am 
-Dspotless.check.skip=true -Djacoco.skip=true 
-Dsurefire.failIfNoSpecifiedTests=false -Dtest=ProcessUtilsTest test`
   
   ## Pull Request Notice
   
   [Pull Request 
Notice](https://github.com/apache/dolphinscheduler/blob/dev/docs/docs/en/contribute/join/pull-request.md)
   
   If your pull request contains incompatible change, you should also add it to 
`docs/docs/en/guide/upgrade/incompatible.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]

Reply via email to