fixes 'Add kustomize support to kfctl'

[kkasravi]

fixes

• Add kustomize support to kfctl #2445 Add kustomize support to kfctl
• Move ksonnet, kustomize implementations out of KfApps.Children #2561 Move ksonnet, kustomize implementations out of KfApps.Children
• kfctl: Add a show subcommand  #2560 Add a show command
• kfctl generated yaml fails validation #2653 kfctl generated yaml fails validation
• Error in application component #2642 kfctl error in application component

---

This change is 

[kkasravi]

additional functionality will follow

[googlebot]

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers: ironpan, kunmingg

If they are not already assigned, you can assign the PR to them by writing /assign @ironpan @kunmingg in a comment when ready.

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• bootstrap/OWNERS
• kubeflow/argo/OWNERS
• kubeflow/common/OWNERS
• kubeflow/pipeline/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[kkasravi]

@jlewi high number of files is because I had to rewrite author on an early commit I did so it needs to do it from one commit before to HEAD. I think that caused git to think merge from master files are part of this commit.

[jlewi]

What if you rebase on master and squash all the commit? Does that get rid of unnecessary changes?

[jlewi]

Reviewed 2 of 102 files at r2, 2 of 79 files at r3, 1 of 14 files at r6.
Reviewable status: 3 of 51 files reviewed, 3 unresolved discussions (waiting on @ashahba, @ellis-bigelow, and @kkasravi)

---

bootstrap/Makefile, line 30 at r6 (raw file):

%.so:
	cd cmd/plugins/$* && \

Should we start thinking about adding a presubmit test?

---

bootstrap/Makefile, line 113 at r6 (raw file):

	echo created $(IMG):latest

install: build-kfctl dockerfordesktop.so kustomize.so ksonnet.so

Are we not building a single statically linked go binary?

---

bootstrap/cmd/bootstrap/app/gcpUtils_test.go, line 24 at r4 (raw file):

)

func TestGetUpdatedPolicy(t *testing.T) {

Did you actually modify this file?

If this is a spurious change maybe try rebasing on master?

[kkasravi]

This has too many spurious changes. See #2673