TEMP

Signed-off-by: Kim Christensen <kimworking@gmail.com>

Author: kichristensen

pkg/cnab/provider/action.go

@@ -14,7 +14,6 @@ import (
 	"github.com/cnabio/cnab-go/driver"
 	"github.com/hashicorp/go-multierror"
 	"go.opentelemetry.io/otel/attribute"
-	"go.uber.org/zap/zapcore"
 )
 
 type HostVolumeMountSpec struct {
@@ -40,7 +39,7 @@ type ActionArguments struct {
 
 	// Params is the fully resolved set of parameters.
 	// TODO(PEP003): This should be removed in https://github.com/getporter/porter/issues/2699
-	Params map[string]interface{}
+	// Params map[string]interface{}
 
 	// Driver is the CNAB-compliant driver used to run bundle actions.
 	Driver string
@@ -108,9 +107,9 @@ func (r *Runtime) AddEnvironment(args ActionArguments) cnabaction.OperationConfi
 		op.Environment[verbosityEnv] = r.Config.GetVerbosity().Level().String()
 
 		// When a bundle is run in debug mode, the verbosity is automatically set to debug
-		if debugMode, _ := args.Params["porter-debug"].(bool); debugMode {
-			op.Environment[verbosityEnv] = zapcore.DebugLevel.String()
-		}
+		// if debugMode, _ := args.Params["porter-debug"].(bool); debugMode {
+		// 	op.Environment[verbosityEnv] = zapcore.DebugLevel.String()
+		// }
 		return nil
 	}
 }

pkg/porter/dependencies.go

@@ -304,7 +304,7 @@ func (e *dependencyExecutioner) prepareDependency(ctx context.Context, dep *queu
 				dep.Parameters = make(map[string]string, 1)
 			}
 			dep.Parameters[paramName] = value
-			delete(e.parentArgs.Params, key)
+			// delete(e.parentArgs.Params, key)
 		}
 	}
 
@@ -426,8 +426,8 @@ func (e *dependencyExecutioner) getActionArgs(ctx context.Context,
 		Run:                   depRun,
 		Driver:                e.parentArgs.Driver,
 		AllowDockerHostAccess: e.parentOpts.AllowDockerHostAccess,
-		Params:                finalParams,
-		PersistLogs:           e.parentArgs.PersistLogs,
+		// Params:                finalParams,
+		PersistLogs: e.parentArgs.PersistLogs,
 	}
 	return nil
 }

pkg/porter/lifecycle.go

@@ -351,10 +351,10 @@ func (p *Porter) BuildActionArgs(ctx context.Context, installation storage.Insta
 	}
 
 	args := cnabprovider.ActionArguments{
-		Run:                   run,
-		Installation:          installation,
-		BundleReference:       bundleRef,
-		Params:                opts.GetParameters(),
+		Run:             run,
+		Installation:    installation,
+		BundleReference: bundleRef,
+		// Params:                opts.GetParameters(),
 		Driver:                opts.Driver,
 		AllowDockerHostAccess: opts.AllowDockerHostAccess,
 		HostVolumeMounts:      opts.GetHostVolumeMounts(),

pkg/porter/lifecycle_test.go

@@ -141,16 +141,16 @@ func TestPorter_BuildActionArgs(t *testing.T) {
 		args, err := p.BuildActionArgs(ctx, existingInstall, opts)
 		require.NoError(t, err, "BuildActionArgs failed")
 
-		expectedParams := map[string]interface{}{
-			"my-first-param":  1,
-			"my-second-param": "VALUE2",
-			"porter-debug":    true,
-			"porter-state":    nil,
-		}
+		// expectedParams := map[string]interface{}{
+		// 	"my-first-param":  1,
+		// 	"my-second-param": "VALUE2",
+		// 	"porter-debug":    true,
+		// 	"porter-state":    nil,
+		// }
 
 		assert.Equal(t, opts.AllowDockerHostAccess, args.AllowDockerHostAccess, "AllowDockerHostAccess not populated correctly")
 		assert.Equal(t, opts.Driver, args.Driver, "Driver not populated correctly")
-		assert.EqualValues(t, expectedParams, args.Params, "Params not populated correctly")
+		// assert.EqualValues(t, expectedParams, args.Params, "Params not populated correctly")
 		assert.NotEmpty(t, args.Installation, "Installation not populated")
 		wantReloMap := relocation.ImageRelocationMap{"gabrtv/microservice@sha256:cca460afa270d4c527981ef9ca4989346c56cf9b20217dcea37df1ece8120687": "my.registry/microservice@sha256:cca460afa270d4c527981ef9ca4989346c56cf9b20217dcea37df1ece8120687"}
 		assert.Equal(t, wantReloMap, args.BundleReference.RelocationMap, "RelocationMapping not populated correctly")