diff --git a/internal/executors/sandbox/executor.go b/internal/executors/sandbox/executor.go index ceac6a5..d8cbd08 100644 --- a/internal/executors/sandbox/executor.go +++ b/internal/executors/sandbox/executor.go @@ -43,9 +43,12 @@ func (e *Sandbox) Run(cmd stage.Cmd) (*stage.Result, error) { func (e *Sandbox) Cleanup() error { slog.Info("sandbox cleanup") for _, fileID := range e.cachedMap { - e.execClient.FileDelete(context.TODO(), &pb.FileID{ + _, err := e.execClient.FileDelete(context.TODO(), &pb.FileID{ FileID: fileID, }) + if err != nil { + slog.Error("sandbox cleanup", "error", err) + } } return nil } diff --git a/internal/stage/model.go b/internal/stage/model.go index 1a0b5ce..78997b3 100644 --- a/internal/stage/model.go +++ b/internal/stage/model.go @@ -115,7 +115,6 @@ type Result struct { FileIDs map[string]string `json:"fileIds,omitempty"` FileError []FileError `json:"fileError,omitempty"` - files []string Buffs map[string][]byte `json:"-"` } diff --git a/internal/stage/run.go b/internal/stage/run.go index fa9dd6d..886aa30 100644 --- a/internal/stage/run.go +++ b/internal/stage/run.go @@ -55,6 +55,9 @@ func Run(stages []Stage) []StageResult { func Cleanup() { for _, executor := range executorMap { - executor.Cleanup() + err := executor.Cleanup() + if err != nil { + slog.Error("executor cleanup error", "error", err) + } } }