fix(xo-server/api): don't log host.stats errors (#5553)

See xoa-support#3323

This avoids flooding the logs with ECONNREFUSED errors when the host's toolstack
is restarted
This commit is contained in:
Rajaa.BARHTAOUI 2021-02-12 11:29:16 +01:00 committed by GitHub
parent 283193e992
commit 9226c6cac1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 4 deletions

View File

@ -17,6 +17,7 @@
- [VM/Snapshot export] Fix `Error: no available place in queue` on canceling an export via browser then starting a new one when the concurrency threshold is reached [#5535](https://github.com/vatesfr/xen-orchestra/issues/5535) (PR [#5538](https://github.com/vatesfr/xen-orchestra/pull/5538))
- [Servers] Hide pool's objects if its master is unreachable [#5475](https://github.com/vatesfr/xen-orchestra/issues/5475) (PR [#5526](https://github.com/vatesfr/xen-orchestra/pull/5526))
- [Host] Restart toolstack: fix `ECONNREFUSED` error (PR [#5553](https://github.com/vatesfr/xen-orchestra/pull/5553))
### Packages to release

View File

@ -342,10 +342,13 @@ export default class Api {
Date.now() - startTime
)}] =!> ${error}`
// 2020-07-10: Work-around: many kinds of error can be triggered by this
// method, which can generates a lot of logs due to the fact that xo-web
// uses 5s active subscriptions to call it
if (name !== 'pool.listMissingPatches') {
// 2020-07-10: Work-around: many kinds of error can be triggered by
// 'pool.listMissingPatches' method, which can generates a lot of logs due to the fact that xo-web
// uses 5s active subscriptions to call it.
// 2021-02-11: Work-around: ECONNREFUSED error can be triggered by
// 'host.stats' method because there is no connection to the host during a
// toolstack restart and xo-web may call it often
if (name !== 'pool.listMissingPatches' || name !== 'host.stats') {
this._logger.error(message, {
...data,
duration: Date.now() - startTime,