chore(outputs): update comments (#595)

executor/linux/opts.go

@@ -185,11 +185,12 @@ func WithVersion(version string) Opt {
 	}
 }
 
+// WithOutputCtn sets the outputs container in the executor client for Linux.
 func WithOutputCtn(ctn *pipeline.Container) Opt {
 	return func(c *client) error {
 		c.Logger.Trace("configuring output container in linux executor client")
 
-		// set the version in the client
+		// set the outputs container in the client
 		c.OutputCtn = ctn
 
 		return nil

executor/linux/outputs.go

@@ -60,7 +60,7 @@ func (o *outputSvc) destroy(ctx context.Context, ctn *pipeline.Container) error
 		return nil
 	}
 
-	// update engine logger with secret metadata
+	// update engine logger with outputs metadata
 	//
 	// https://pkg.go.dev/github.com/sirupsen/logrus#Entry.WithField
 	logger := o.client.Logger.WithField("outputs", ctn.Name)

runtime/kubernetes/container.go

@@ -69,6 +69,8 @@ func (c *client) RemoveContainer(ctx context.Context, ctn *pipeline.Container) e
 	return nil
 }
 
+// PollOutputsContainer captures the `cat` response for a given path in the Docker volume.
+// This is a no-op for kubernetes. Pod environments cannot be dynamic.
 func (c *client) PollOutputsContainer(ctx context.Context, ctn *pipeline.Container, path string) ([]byte, error) {
 	c.Logger.Tracef("no-op: removing container %s", ctn.ID)