rollback keep tmp files

Signed-off-by: Pavel Okhlopkov <[email protected]>

pkg/app/debug.go

@@ -30,11 +30,7 @@ func DefineDebugFlags(kpApp *kingpin.Application, cmd *kingpin.CmdClause) {
 	cmd.Flag("debug-keep-tmp-files", "set to yes to disable cleanup of temporary files").
 		Envar("DEBUG_KEEP_TMP_FILES").
 		Hidden().
-		Default(DebugKeepTmpFilesVar).Action(func(_ *kingpin.ParseContext) error {
-		DebugKeepTmpFiles = DebugKeepTmpFilesVar == "yes"
-
-		return nil
-	}).
+		Default(DebugKeepTmpFilesVar).
 		StringVar(&DebugKeepTmpFilesVar)
 
 	cmd.Flag("debug-kubernetes-api", "enable client-go debug messages").

pkg/hook/hook.go

@@ -13,6 +13,7 @@ import (
 	"github.com/kennygrant/sanitize"
 	"golang.org/x/time/rate"
 
+	"github.com/flant/shell-operator/pkg/app"
 	"github.com/flant/shell-operator/pkg/executor"
 	bctx "github.com/flant/shell-operator/pkg/hook/binding_context"
 	"github.com/flant/shell-operator/pkg/hook/config"
@@ -124,7 +125,7 @@ func (h *Hook) Run(_ htypes.BindingType, context []bctx.BindingContext, logLabel
 
 	// remove tmp file on hook exit
 	defer func() {
-		if h.KeepTemporaryHookFiles {
+		if app.DebugKeepTmpFilesVar != "yes" {
 			_ = os.Remove(contextPath)
 			_ = os.Remove(metricsPath)
 			_ = os.Remove(conversionPath)