feat: fix render and deploy when using kustomize and helm deployer.

[tejal29]

In this PR,

Completely addressing the issue #7675 to support backward compatibility.

Changes in this PR

• If Raw manifests are defined, even if kubectl deployer is not set add it. e.g.

manifests:
  rawYaml:
  - frontend/k8s/*.yaml
deploy:
      helm:
        releases:
        - name: backend
          chartPath: backend

In the above example, skaffold was configured to deploy helm charts. The frontend k8s manifests were rendered but never deployed.

• When running skaffold render with legacy helm deploy stanza, force helm rendering.
  In the above same config, only kubectl manifests were rendered with skaffold render. This behavior is backward incompatible.

=== TODO

1. follow up if removing duplicates render.helm.releases and deploy.helm.release makes sense. @marlon-gamez discussed before it could be confusing to remove these duplicated fields.

[codecov]

Codecov Report

Merging #7723 (20c382e) into main (290280e) will decrease coverage by 3.87%.
The diff coverage is 53.59%.

@@            Coverage Diff             @@
##             main    #7723      +/-   ##
==========================================
- Coverage   70.48%   66.61%   -3.88%     
==========================================
  Files         515      587      +72     
  Lines       23150    28239    +5089     
==========================================
+ Hits        16317    18811    +2494     
- Misses       5776     8051    +2275     
- Partials     1057     1377     +320     

Impacted Files	Coverage Δ
cmd/skaffold/app/cmd/credits/export.go	0.00% <0.00%> (ø)
cmd/skaffold/app/cmd/deploy.go	40.90% <0.00%> (-12.94%)	⬇️
cmd/skaffold/app/cmd/test.go	44.44% <0.00%> (ø)
cmd/skaffold/skaffold.go	0.00% <0.00%> (ø)
cmd/skaffold/app/cmd/inspect_tests.go	62.50% <14.28%> (-1.14%)	⬇️
cmd/skaffold/app/cmd/render.go	35.48% <18.18%> (-5.90%)	⬇️
cmd/skaffold/app/cmd/lsp.go	28.12% <28.12%> (ø)
cmd/skaffold/app/cmd/run.go	64.28% <33.33%> (-9.63%)	⬇️
cmd/skaffold/app/cmd/fix.go	56.41% <37.50%> (-20.07%)	⬇️
cmd/skaffold/app/cmd/verify.go	41.17% <41.17%> (ø)
... and 344 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

[aaron-prindle]

Seems there is a test error atm:

    --- FAIL: TestRun/multiple_renderers_mixed_in#01 (3.37s)

also a small lint error related to a newline

[tejal29]

fixed it. Some bad rebasing.

[aaron-prindle]

LGTM!