diff options
| author | John Cai <jcai@gitlab.com> | 2020-03-17 00:59:29 +0300 |
|---|---|---|
| committer | John Cai <jcai@gitlab.com> | 2020-03-17 01:05:14 +0300 |
| commit | 1053c49c22cb46f61ecbfbcd2c27ff9b767560b8 (patch) | |
| tree | 54f2f377d2b7f3365ace427eacf876288acb305c | |
| parent | ddbf8c44360bf5cffd9c8b2d7fbbf73be6e17677 (diff) | |
Add storage name to healthcheck error log
| -rw-r--r-- | changelogs/unreleased/jc-add-storage-to-healthcheck-error.yml | 5 | ||||
| -rw-r--r-- | internal/praefect/nodes/manager.go | 2 |
2 files changed, 6 insertions, 1 deletions
diff --git a/changelogs/unreleased/jc-add-storage-to-healthcheck-error.yml b/changelogs/unreleased/jc-add-storage-to-healthcheck-error.yml new file mode 100644 index 000000000..dca916054 --- /dev/null +++ b/changelogs/unreleased/jc-add-storage-to-healthcheck-error.yml @@ -0,0 +1,5 @@ +--- +title: Add storage name to healthcheck error log +merge_request: 1934 +author: +type: added diff --git a/internal/praefect/nodes/manager.go b/internal/praefect/nodes/manager.go index 6aff6c233..9a677327e 100644 --- a/internal/praefect/nodes/manager.go +++ b/internal/praefect/nodes/manager.go @@ -283,7 +283,7 @@ func (n *nodeStatus) check() { resp, err := client.Check(ctx, &healthpb.HealthCheckRequest{Service: ""}) if err != nil { - n.log.WithError(err).WithField("address", n.Address).Warn("error when pinging healthcheck") + n.log.WithError(err).WithField("storage", n.Storage).WithField("address", n.Address).Warn("error when pinging healthcheck") resp = &healthpb.HealthCheckResponse{ Status: healthpb.HealthCheckResponse_UNKNOWN, } |