From 46b9a73f5be2540ee060eb05f8a5318ecdb6b5d0 Mon Sep 17 00:00:00 2001 From: Maggie Lou Date: Tue, 24 Jun 2025 15:37:51 -0500 Subject: [PATCH] Fix UI for Firecracker actions that aren't snapshotted --- .../remote_execution/containers/firecracker/firecracker.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/enterprise/server/remote_execution/containers/firecracker/firecracker.go b/enterprise/server/remote_execution/containers/firecracker/firecracker.go index 49334e27120..a1ff2bba4d1 100644 --- a/enterprise/server/remote_execution/containers/firecracker/firecracker.go +++ b/enterprise/server/remote_execution/containers/firecracker/firecracker.go @@ -2239,8 +2239,10 @@ func (c *FirecrackerContainer) Exec(ctx context.Context, cmd *repb.Command, stdi stage := "init" result := &interfaces.CommandResult{ExitCode: commandutil.NoExitCode} defer func() { - // Attach VM metadata to the result - result.VMMetadata = c.getVMMetadata() + if c.recyclingEnabled { + // Attach snapshot-related metadata to the execution result. + result.VMMetadata = c.getVMMetadata() + } // Attach VM logs to the result if result.AuxiliaryLogs == nil {