aerokube / ggr-ui

The missing /status API for Ggr
https://aerokube.com/ggr-ui/latest/
Apache License 2.0
39 stars 15 forks source link

Do not take into consideration not running Selenoid instances behind ggr-ui #5

Closed slipwalker closed 6 years ago

slipwalker commented 6 years ago

Currently, if there are no running Selenoid instances behind ggr (or almost all of them are down), it takes a while until /status request gets resolved with some response. It's a bit inconvenient. Would be good to have some mechanism to sort these not running instances out and do not consider them in order to have more fast ggr-ui request-response logic. Once instance becomes available, it should be considered then. Any other suggestions to improve this would be appreciated.

vania-pooh commented 6 years ago

Seems to be a reasonable idea.

vania-pooh commented 6 years ago

We fixed this by adding -response-time flag and parallel polling. Should not reproduce in release 1.1.0.