Do not write operations log to state files on errors by default

Author: oginskis

cmd/hub/cmd/deploy.go

@@ -30,6 +30,7 @@ var (
 	hubSaveStackInstanceOutputs   bool
 	hubSyncStackInstance          bool
 	hubSyncSkipParametersAndOplog bool
+	writeOplogToStateOnError      bool
 )
 
 var deployCmd = &cobra.Command{
@@ -141,6 +142,7 @@ func lifecycleRequest(args []string, verb string) (*lifecycle.Request, error) {
 		Application:                hubApplication,
 		SyncStackInstance:          hubSyncStackInstance,
 		SyncSkipParametersAndOplog: hubSyncSkipParametersAndOplog,
+		WriteOplogToStateOnError:   writeOplogToStateOnError,
 	}
 
 	return request, nil
@@ -174,6 +176,8 @@ func initDeployUndeployFlags(cmd *cobra.Command, verb string) {
 		"Sync Stack Instance state to SuperHub (--hub-stack-instance must be set)")
 	cmd.Flags().BoolVarP(&hubSyncSkipParametersAndOplog, "hub-sync-skip-parameters-and-oplog", "", false,
 		"Sync skip syncing Stack Instance parameters and operation log")
+	cmd.Flags().BoolVar(&writeOplogToStateOnError, "write-oplog-to-state-on-error", false,
+		"Write operations log to state files on error")
 	initCommonLifecycleFlags(cmd, verb)
 	initCommonApiFlags(cmd)
 }

cmd/hub/lifecycle/deploy.go

@@ -386,7 +386,7 @@ NEXT_COMPONENT:
 
 		var rawOutputs parameters.RawOutputs
 		if err != nil {
-			if stateManifest != nil {
+			if stateManifest != nil && request.WriteOplogToStateOnError {
 				stateManifest = state.AppendOperationLog(stateManifest, operationLogId,
 					fmt.Sprintf("%v%s", err, formatStdoutStderr(stdout, stderr)))
 			}

cmd/hub/lifecycle/types.go

@@ -22,4 +22,5 @@ type Request struct {
 	Application                string
 	SyncStackInstance          bool
 	SyncSkipParametersAndOplog bool
+	WriteOplogToStateOnError   bool
 }