test: add node name to error messages in RebootAllNodes

This makes troubleshooting easier.

Signed-off-by: Andrey Smirnov <smirnov.andrey@gmail.com>
This commit is contained in:
Andrey Smirnov 2020-05-06 23:22:33 +03:00 committed by Andrew Rynhard
parent 1c20a32ab4
commit 28a6eb207a

View File

@ -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