fix: skaffold inspect commands should have non-zero exit-code on error

[briandealwis]

Fixes: #6801

Description

• ensure actual skaffold inspect errors are bubbled back up for exit-code reporting
• add ability for certain commands to suppress error reporting
• mark inspect commands as suppressing error reporting
• add top-level tests to ensure errors are reported or not reported as required

cc: @j-windsor @sujit-kamireddy @ivanporty

[gsquared94]

changes look good, but the related tests are failing

[briandealwis]

oops 😊 tests should be fixed

[codecov]

Codecov Report

Merging #6807 (d8ad121) into main (290280e) will decrease coverage by 0.97%.
The diff coverage is 69.89%.

@@            Coverage Diff             @@
##             main    #6807      +/-   ##
==========================================
- Coverage   70.48%   69.51%   -0.98%     
==========================================
  Files         515      542      +27     
  Lines       23150    24628    +1478     
==========================================
+ Hits        16317    17119     +802     
- Misses       5776     6379     +603     
- Partials     1057     1130      +73     

Impacted Files	Coverage Δ
cmd/skaffold/app/cmd/deploy.go	52.00% <ø> (-1.85%)	⬇️
cmd/skaffold/app/cmd/dev.go	84.61% <0.00%> (ø)
cmd/skaffold/app/cmd/flags.go	91.00% <0.00%> (+0.18%)	⬆️
cmd/skaffold/app/cmd/render.go	36.66% <0.00%> (-4.72%)	⬇️
cmd/skaffold/skaffold.go	0.00% <0.00%> (ø)
cmd/skaffold/app/cmd/find_configs.go	48.88% <50.00%> (+0.24%)	⬆️
cmd/skaffold/app/cmd/lint.go	52.94% <52.94%> (ø)
cmd/skaffold/app/skaffold.go	76.19% <70.00%> (-8.43%)	⬇️
cmd/skaffold/app/cmd/delete.go	69.23% <80.00%> (+13.67%)	⬆️
cmd/skaffold/app/cmd/cmd.go	72.13% <100.00%> (+1.07%)	⬆️
... and 148 more

---

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 008e267...d8ad121. Read the comment docs.

[tejal29]

nice catch.