From 817244eadf9da24d70eab4cce0af8abcfa7d46f9 Mon Sep 17 00:00:00 2001 From: Andrew Karpow Date: Wed, 8 Apr 2026 15:05:28 -0400 Subject: [PATCH] EvictionController: Fix unused logger context return value Assign the return value of logger.IntoContext() back to ctx so that subsequent calls using ctx will have the enriched logger with additional fields (server, server_status). --- internal/controller/eviction_controller.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/controller/eviction_controller.go b/internal/controller/eviction_controller.go index b8fc678..ddc68f7 100644 --- a/internal/controller/eviction_controller.go +++ b/internal/controller/eviction_controller.go @@ -241,7 +241,7 @@ func (r *EvictionReconciler) evictNext(ctx context.Context, eviction *kvmv1.Evic instances := &eviction.Status.OutstandingInstances uuid := (*instances)[len(*instances)-1] log := logger.FromContext(ctx).WithName("Evict").WithValues("server", uuid) - logger.IntoContext(ctx, log) + ctx = logger.IntoContext(ctx, log) res := servers.Get(ctx, r.computeClient, uuid) vm, err := res.Extract() @@ -255,7 +255,7 @@ func (r *EvictionReconciler) evictNext(ctx context.Context, eviction *kvmv1.Evic } log = log.WithValues("server_status", vm.Status) - logger.IntoContext(ctx, log) + ctx = logger.IntoContext(ctx, log) // First, check the transient statuses switch vm.Status {