From 84c01fde3e14bb9b5bd875c0dbbf5743f9e93cdb Mon Sep 17 00:00:00 2001 From: Artem Chernyshev Date: Tue, 14 Jan 2025 19:56:22 +0300 Subject: [PATCH] fix: properly reset `MachineStatus` hostname for deallocated machines Need to treat `nil` hostname as no hostname and wipe the current value in the `MachineStatus`. Signed-off-by: Artem Chernyshev --- .../omni/controllers/omni/internal/task/machine/poll.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/internal/backend/runtime/omni/controllers/omni/internal/task/machine/poll.go b/internal/backend/runtime/omni/controllers/omni/internal/task/machine/poll.go index 6ce30421..198c3968 100644 --- a/internal/backend/runtime/omni/controllers/omni/internal/task/machine/poll.go +++ b/internal/backend/runtime/omni/controllers/omni/internal/task/machine/poll.go @@ -99,6 +99,9 @@ func pollVersion(ctx context.Context, c *client.Client, info *Info) error { } func pollHostname(ctx context.Context, c *client.Client, info *Info) error { + info.Hostname = pointer.To("") + info.Domainname = pointer.To("") + return forEachResource( ctx, c,