HTTP Pool monitoring should be enabled by the thread-pool monitoring module, not by the http-service module

[dmatej]

Description

Expectation: when I enable thread-pool monitoring, some data will be seen.

Relevant classes:

• ThreadPoolStatsProvider

Reproducer

asadmin set configs.config.server-config.monitoring-service.module-monitoring-levels.thread-pool=HIGH
# Command set executed successfully.
asadmin get 'configs.config.server-config.monitoring-service.module-monitoring-levels.thread-pool'
# configs.config.server-config.monitoring-service.module-monitoring-levels.thread-pool=HIGH
curl http://localhost:8080
# welcome page printed
asadmin get -m '*'
# Command get executed successfully.

The last command should print some data.

[OndroMih]

I’m not sure but isn’t http thread pool monitoring under the http monitoring component and not thread-pool monitoring?

[dmatej]

I would expect any thread pool in the UI listed under thread pools should be printed here.

[OndroMih]

Sure, makes sense. I just vaguely remember that HTTP thread pools were under HTTP monitoring. If that’s true, the functionality works, it just needs to be linked to thread pool monotoring, not HTTP.