Skip to content

Commit

Permalink
fix: print etcd member ID in hex
Browse files Browse the repository at this point in the history
It's confusing in base 10, as every other place prints member ID in hex.

Signed-off-by: Andrey Smirnov <andrey.smirnov@talos-systems.com>
  • Loading branch information
smira committed Sep 13, 2021
1 parent 5ca1fb8 commit ef02295
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions internal/pkg/etcd/etcd.go
Original file line number Diff line number Diff line change
Expand Up @@ -119,11 +119,11 @@ func (c *Client) ValidateForUpgrade(ctx context.Context, config config.Provider,
for _, member := range resp.Members {
// If the member is not started, the name will be an empty string.
if len(member.Name) == 0 {
return fmt.Errorf("etcd member %d is not started, all members must be running to perform an upgrade", member.ID)
return fmt.Errorf("etcd member %016x is not started, all members must be running to perform an upgrade", member.ID)
}

if err = validateMemberHealth(ctx, member.GetClientURLs()); err != nil {
return fmt.Errorf("etcd member %d is not healthy; all members must be healthy to perform an upgrade: %w", member.ID, err)
return fmt.Errorf("etcd member %016x is not healthy; all members must be healthy to perform an upgrade: %w", member.ID, err)
}
}

Expand Down

0 comments on commit ef02295

Please sign in to comment.