2
92%
master: %

Ran 23 Apr 2026 07:46PM UTC

Files 132

Run time 7s

Badge

Embed ▾

Committed 23 Apr 2026 07:28PM UTC coverage: 88.277%. Remained the same

Job # 24854619598.2

Build Type

push

github

Committed by

web-flow

Commit Message

fix #3515: run register_commands off the ioloop (#3516)

ReloadPluginAPItestHandler.post() was calling register_commands()
synchronously, which shells out via Popen.communicate() and blocks the
master Tornado ioloop for the duration of the plugin register
subprocess. That subprocess calls back into qiita through nginx; nginx
round-robins across master and workers, so roughly one in four callback
POSTs land on the blocked master and time out (504), breaking CI for
plugins that register more than one command (e.g. qtp-biom).

Move the blocking call off the ioloop via IOLoop.run_in_executor so
master can continue serving /activate/ callbacks while register_commands
runs in the default thread pool. _system_call and register_commands are
unchanged; all their other callers run outside handler context already.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Coverage Stats

9955 of 11277 relevant lines covered (88.28%)

0.88 hits per line

qiita-spots / qiita / 24854619598 / 2
92%
master: %

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job qiita_pet qiita_core qiita_ware - 24854619598.2

qiita-spots / qiita / 24854619598 / 2 92% master: %

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job qiita_pet qiita_core qiita_ware - 24854619598.2

qiita-spots / qiita / 24854619598 / 2
92%
master: %

README BADGES
x