feat: add kind-disable-load and k3d-disable-load config values

[shaxbee]

Fixes: #3461

Description
Add two new configuration options kind-disable-load and k3d-disable-load to allow disabling image loading for local kind and k3d clusters.

This significantly improves deployment times to Kind.

Configuration

skaffold config set kind-disable-load true
skaffold config set default-repo localhost:5000
skaffold config set insecure-registries http://localhost:5000

Demo

• Clone github.com/shaxbee/todo-app-skaffold.
• make bootstrap
• skaffold run --force (job is being deployed, therefore force flag is required)

[codecov]

Codecov Report

Merging #5012 (b9bfaed) into master (bbcfdb0) will increase coverage by 0.12%.
The diff coverage is 86.48%.

@@            Coverage Diff             @@
##           master    #5012      +/-   ##
==========================================
+ Coverage   72.28%   72.40%   +0.12%     
==========================================
  Files         368      368              
  Lines       12927    12934       +7     
==========================================
+ Hits         9344     9365      +21     
+ Misses       2891     2883       -8     
+ Partials      692      686       -6     

Impacted Files	Coverage Δ
pkg/skaffold/config/types.go	91.66% <ø> (ø)
pkg/skaffold/runner/deploy.go	50.00% <0.00%> (+6.06%)	⬆️
pkg/skaffold/runner/runcontext/context.go	83.33% <60.00%> (-1.81%)	⬇️
pkg/skaffold/config/util.go	70.34% <92.59%> (+5.07%)	⬆️
pkg/skaffold/build/local/types.go	82.35% <100.00%> (+3.10%)	⬆️
pkg/skaffold/docker/image.go	80.93% <0.00%> (+1.39%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bbcfdb0...b9bfaed. Read the comment docs.

[shaxbee]

Push is enabled automatically if load is disabled for either kind/k3d.

[shaxbee]

Build failure for one of the jobs seems intermittent as it shows maven download error.
Could you rerun?

[gsquared94]

Thanks for your contribution! PR looks great, but I think we shouldn't make isDefaultLocal function more ambiguous by checking for IsImageLoadingRequired inside it since it might be used in some other context later.

[shaxbee]

Build seems to be failing on gci lint check in unchanged file.
Fixed it up.

[shaxbee]

BTW this could be extended in future PR to support: https://github.com/kubernetes/enhancements/tree/master/keps/sig-cluster-lifecycle/generic/1755-communicating-a-local-registry.

Edit: build seems to be failing on flaky java tests downloading maven packages again.

[shaxbee]

@gsquared94 care to review? Review comments were addressed 9 days ago.

[gsquared94]

@gsquared94 care to review? Review comments were addressed 9 days ago.

Sorry for the delay. I was out on vacation. Will take a look first thing in the morning.

[gsquared94]

lgtm!

[zanitete]

any plan to backport this feature to 0.16.x for people still stuck to Helm 2?

[nkubala]

@zanitete unfortunately not :( Helm 2 has been EOL'd so we're not putting work into supporting it. sorry about that