Handle kyaml Filter errors type Result as non-breaking errors and sto…

…re in ResourceList

- Result can only count as error when passed as pointer, this makes easy use of "errors.As"
- Existing Filter() implementations that return Result from the `framework` package won't return an error anymore but modify the ResourceList

kyaml/fn/framework/framework.go

@@ -4,6 +4,7 @@
 package framework
 
 import (
+	goerrors "errors"
 	"os"
 
 	"sigs.k8s.io/kustomize/kyaml/errors"
@@ -140,8 +141,19 @@ func Execute(p ResourceListProcessor, rlSource *kio.ByteReadWriter) error {
 
 // Filter executes the given kio.Filter and replaces the ResourceList's items with the result.
 // This can be used to help implement ResourceListProcessors. See SimpleProcessor for example.
+//
+// Filters that return a Result as error will store the result in the ResourceList instead of
+// and continue processing instead of erroring out.
 func (rl *ResourceList) Filter(api kio.Filter) error {
 	var err error
 	rl.Items, err = api.Filter(rl.Items)
-	return errors.Wrap(err)
+	if err != nil {
+		var r Results
+		if goerrors.As(err, &r) {
+			rl.Results = append(rl.Results, r...)
+			return nil
+		}
+		return errors.Wrap(err)
+	}
+	return nil
 }

kyaml/fn/framework/framework_test.go

@@ -15,8 +15,8 @@ import (
 )
 
 func TestExecute_Result(t *testing.T) {
-	p := framework.ResourceListProcessorFunc(func(rl *framework.ResourceList) error {
-		err := &framework.Results{
+	p := framework.SimpleProcessor{Filter: kio.FilterFunc(func(in []*yaml.RNode) ([]*yaml.RNode, error) {
+		return in, framework.Results{
 			{
 				Message:  "bad value for replicas",
 				Severity: framework.Error,
@@ -40,9 +40,7 @@ func TestExecute_Result(t *testing.T) {
 				Tags:     map[string]string{"foo": "bar"},
 			},
 		}
-		rl.Results = *err
-		return err
-	})
+	})}
 	out := new(bytes.Buffer)
 	source := &kio.ByteReadWriter{Reader: bytes.NewBufferString(`
 kind: ResourceList
@@ -57,10 +55,7 @@ items:
     replicas: 0
 `), Writer: out}
 	err := framework.Execute(p, source)
-	assert.EqualError(t, err, `[error] v1/Deployment/default/tester .spec.Replicas: bad value for replicas
-
-[error]: some error`)
-	assert.Equal(t, 1, err.(*framework.Results).ExitCode())
+	assert.NoError(t, err)
 	assert.Equal(t, `apiVersion: config.kubernetes.io/v1
 kind: ResourceList
 items: