From 4de6ffd6b409401fd8225912da15612a13b4e127 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julian=20Peliz=C3=A4us?= <julian.pelizaeus@canonical.com>
Date: Fri, 16 Oct 2026 14:39:40 +0200
Subject: [PATCH] microcluster/app: Ignore returned value (errcheck)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

We only ever send nil into the chan.
Therefore switch to struct{} as its done for ctx.Done().

Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
---
 microcluster/app.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/microcluster/app.go b/microcluster/app.go
index c3c9bfbd..1da5f029 100644
--- a/microcluster/app.go
+++ b/microcluster/app.go
@@ -141,7 +141,7 @@ func (m *MicroCluster) Status(ctx context.Context) (*types.Status, error) {
 func (m *MicroCluster) Ready(ctx context.Context) error {
 	logger := slog.New(m.args.LogHandler)
 
-	finger := make(chan error, 1)
+	finger := make(chan struct{}, 1)
 	var errLast error
 	go func() {
 		for i := 0; ; i++ {
@@ -185,7 +185,7 @@ func (m *MicroCluster) Ready(ctx context.Context) error {
 				continue
 			}
 
-			finger <- nil
+			finger <- struct{}{}
 			return
 		}
 	}()