Skip to content

Conversation

@liyonghua0910
Copy link
Collaborator

@liyonghua0910 liyonghua0910 commented Aug 8, 2025

问题描述
当后台worker进程挂掉时,前台 api server 没有感知,看起来一切正常,此时发请求是收不到响应的,也收不到报错。

解决方法
在api server模块新增一个线程,每 5 秒探测一次worker进程是否存活。如果worker进程已挂,主动发送SIGINT信号关闭服务。

@paddle-bot
Copy link

paddle-bot bot commented Aug 8, 2025

Thanks for your contribution!

@paddle-bot paddle-bot bot added the contributor External developers label Aug 8, 2025
@codecov-commenter
Copy link

codecov-commenter commented Aug 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (develop@ce9c091). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff            @@
##             develop   #3271   +/-   ##
=========================================
  Coverage           ?   0.00%           
=========================================
  Files              ?       2           
  Lines              ?      14           
  Branches           ?       1           
=========================================
  Hits               ?       0           
  Misses             ?      14           
  Partials           ?       0           
Flag Coverage Δ
diff 0.00% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@liyonghua0910 liyonghua0910 changed the title [fix] fix terminal hangs when worker process is dead [Others] api server exits when worker process is dead Oct 23, 2025
@Jiang-Jia-Jun Jiang-Jia-Jun merged commit cdc40cd into PaddlePaddle:develop Oct 27, 2025
23 of 27 checks passed
kevincheng2 pushed a commit to kevincheng2/FastDeploy that referenced this pull request Oct 27, 2025
)

* [fix] fix terminal hangs when worker process is dead

* [chore] change sleep time of monitor

* [chore] remove redundant comments
Jiang-Jia-Jun added a commit that referenced this pull request Oct 27, 2025
* support mm prefix caching

* update code

* fix mm_hashes

* support encoder cache

* add encoder cache

* update code

* update encoder cache

* fix features bug

* fix worker bug

* support processor cache, need to optimize yet

* refactor multimodal data cache

* update code

* update code

* update v1 scheduler

* update code

* update code

* update codestyle

* support turn off processor cache and encoder cache

* update pre-commit

* fix code

* solve review

* update code

* update code

* update test case

* set processor cache in GiB

* update test case

* support mm prefix caching for qwen model

* fix code style check

* update pre-commit

* fix unit test

* fix unit test

* add ci test case

* fix rescheduled bug

* change text_after_process to prompt_tokens

* fix unit test

* fix chat template

* change model path

* [EP] fix adapter bugs (#4572)

* Update expert_service.py

* Update common_engine.py

* Update expert_service.py

* fix v1 hang bug (#4573)

* fix import image_ops error on some platforms (#4559)

* [CLI]Update parameters in bench latecy cli tool and fix collect-env cli tool (#4558)

* add collect-env

* del files

* [Graph Optimization] Add dy_runnable and introduce cudagraph_switch_threshold for cudagraph mode switching (#4578)

* add new branch for sot

* reorder

* fix batch bug

* [XPU]Moe uses a new operator (#4585)

* [XPU]Moe uses a new operator

* [XPU]Moe uses a new operator

* update response

* [Feature] Support Paddle-OCR (#4396)

* init

* update code

* fix code style & disable thinking

* adapt for common_engine.update_mm_requests_chunk_size

* use 3d rope

* use flash_attn_unpadded

* opt siglip

* update to be compatible with the latest codebase

* fix typo

* optim OCR performance

* fix bug

* fix bug

* fix bug

* fix bug

* normlize name

* modify xpu rope

* revert logger

* fix bug

* fix bug

* fix bug

* support default_v1

* optim performance

* fix bug

---------

Co-authored-by: root <[email protected]>
Co-authored-by: zhangyue66 <[email protected]>

* [DataProcessor] add reasoning_tokens into usage info (#4520)

* add reasoning_tokens into usage info initial commit

* add unit tests

* modify unit test

* modify and add unit tests

* fix unit test

* move steam usage to processor

* modify processor

* modify test_logprobs

* modify test_logprobs.py

* modify stream reasoning tokens accumulation

* fix unit test

* perf: Optimize task queue communication from engine to worker (#4531)

* perf: Optimize task queue communication from engine to worker

* perf: get_tasks to numpy

* perf: get_tasks remove to_numpy

* fix: request & replace ENV

* remove test_e2w_perf.py

* fix code style

---------

Co-authored-by: Jiang-Jia-Jun <[email protected]>

* Clean up ports after processing results (#4587)

* [CI] Add /re-run command in PR comments to restart failed CI workflows (#4593)

* [Others] api server exits when worker process is dead (#3271)

* [fix] fix terminal hangs when worker process is dead

* [chore] change sleep time of monitor

* [chore] remove redundant comments

* update docs

---------

Co-authored-by: ApplEOFDiscord <[email protected]>
Co-authored-by: ApplEOFDiscord <[email protected]>
Co-authored-by: ltd0924 <[email protected]>
Co-authored-by: yinwei <[email protected]>
Co-authored-by: JYChen <[email protected]>
Co-authored-by: qwes5s5 <[email protected]>
Co-authored-by: Ryan <[email protected]>
Co-authored-by: yyssys <[email protected]>
Co-authored-by: ming1753 <[email protected]>
Co-authored-by: root <[email protected]>
Co-authored-by: zhangyue66 <[email protected]>
Co-authored-by: kxz2002 <[email protected]>
Co-authored-by: SunLei <[email protected]>
Co-authored-by: Jiang-Jia-Jun <[email protected]>
Co-authored-by: Zhang Yulong <[email protected]>
Co-authored-by: YuBaoku <[email protected]>
Co-authored-by: 李泳桦 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor External developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants