search

livepeer / catalyst-api

MIT License
4 stars 2 forks source link

Add timeout for fetching Mist metrics from catalyst-api #1368

Closed leszko closed 2 months ago

leszko commented 2 months ago

During the catalyst deployment, we sometimes lost the catalyst-api metrics, because we had no timeout and the /metrics endpoint was waiting forever when the Mist was down.

This PR adds a simple timeout

pwilczynskiclearcode commented 2 months ago

Shouldn't we have a timeout on all the queries to mist?

leszko commented 2 months ago

Shouldn't we have a timeout on all the queries to mist?

I think we should, though the usual (POST) request to Mist didn't cause any issues. Wrt metrics, for some reason, if Mist is down, then catalyst-api becomes blocked and stops returning metrics.