From 28a6eb207ab6504ed68531caf4428fc75705ff7c Mon Sep 17 00:00:00 2001 From: Andrey Smirnov Date: Wed, 6 May 2020 23:22:33 +0300 Subject: [PATCH] test: add node name to error messages in RebootAllNodes This makes troubleshooting easier. Signed-off-by: Andrey Smirnov --- internal/integration/api/reboot.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/integration/api/reboot.go b/internal/integration/api/reboot.go index 44f3f65bc..117850719 100644 --- a/internal/integration/api/reboot.go +++ b/internal/integration/api/reboot.go @@ -151,12 +151,12 @@ func (suite *RebootSuite) TestRebootAllNodes() { uptimeAfter, err := suite.ReadUptime(nodeCtx) if err != nil { // API might be unresponsive during reboot - return retry.ExpectedError(err) + return retry.ExpectedError(fmt.Errorf("error reading uptime for node %q: %w", node, err)) } if uptimeAfter >= uptimeBefore { // uptime should go down after reboot - return retry.ExpectedError(fmt.Errorf("uptime didn't go down: before %f, after %f", uptimeBefore, uptimeAfter)) + return retry.ExpectedError(fmt.Errorf("uptime didn't go down for node %q: before %f, after %f", node, uptimeBefore, uptimeAfter)) } return nil