From a79905d2d137f86bbef0811c8ad6aa9a1bf71127 Mon Sep 17 00:00:00 2001 From: B R GOVIND Date: Mon, 22 Jun 2026 07:17:23 +0000 Subject: [PATCH] fix(unikernels/linux): remove always-true conditional in firecracker block In MonitorBlockCli, the firecracker switch case contained an inner check `if l.Monitor == "firecracker"` which is always true since we are already inside the "firecracker" case. Remove the redundant conditional and directly prepend the "FC" prefix to the block ID. Signed-off-by: B R GOVIND --- pkg/unikontainers/unikernels/linux.go | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/pkg/unikontainers/unikernels/linux.go b/pkg/unikontainers/unikernels/linux.go index d6bb8095..3ccea8ad 100644 --- a/pkg/unikontainers/unikernels/linux.go +++ b/pkg/unikontainers/unikernels/linux.go @@ -170,12 +170,8 @@ func (l *Linux) MonitorBlockCli() []types.MonitorBlockArgs { } case "firecracker": for _, aBlock := range l.Blk { - id := aBlock.ID - if l.Monitor == "firecracker" { - id = "FC" + aBlock.ID - } blkArgs = append(blkArgs, types.MonitorBlockArgs{ - ID: id, + ID: "FC" + aBlock.ID, Path: aBlock.Source, }) }