Add support for Kustomize components

[klausenbusk]

Ref: fluxcd/kustomize-controller#753

Signed-off-by: Kristian Klausen [email protected]

[stefanprodan]

Hey @klausenbusk can you please add here the tests you wrote.

[klausenbusk]

Hey @klausenbusk can you please add here the tests you wrote.

I have added the test back. I'm not sure if we want to use interface{}? If we want to use kustypes.Kustomization we need to do some PR juggling.

[stefanprodan]

We can't use the API types in pkg, that would get us to cyclic dependency between the controller and pkg which would break go.mod.

[klausenbusk]

I'm not sure the fuzz failure is related to this change:

+ cd addimport
/src/build.sh: line 38: cd: addimport: No such file or directory
+ cleanup

[stefanprodan]

@klausenbusk we can ignore the fuzzers, these were broken upstream and we're going to fix them in a different PR.

[souleb]

I have run the tests in flux2 with this PR, and it passes successfully.

[stefanprodan]

Fuzzing has been fixed in main, please rebase.

[klausenbusk]

Any idea why the test is failing on macos-12 https://github.com/fluxcd/pkg/actions/runs/3539040439/jobs/5940762510?

[stefanprodan]

@klausenbusk use:

func testTempDir(t *testing.T) (string, error) {
	tmpDir := t.TempDir()

	tmpDir, err := filepath.EvalSymlinks(tmpDir)
	if err != nil {
		return "", fmt.Errorf("error evaluating symlink: '%w'", err)
	}

	return tmpDir, err
}

instead of t.TempDir()

[souleb]

LGTM. Thanks @klausenbusk for the PR 🙇‍♂️

[klausenbusk]

Do you need more from me, before this can be merged? :)

[pjbgf]

@klausenbusk please rebase so we can go ahead with the merge.

[klausenbusk]

Rebased :)

[pjbgf]

LGTM

Thanks @klausenbusk! 🙇