fix: remove required tag from cloud run region

[bskaplan]

This can now be specified via flag so it's no longer required in the YAML.

[codecov]

Codecov Report

Merging #7784 (ad4aa17) into main (290280e) will decrease coverage by 3.83%.
The diff coverage is 53.28%.

@@            Coverage Diff             @@
##             main    #7784      +/-   ##
==========================================
- Coverage   70.48%   66.65%   -3.84%     
==========================================
  Files         515      590      +75     
  Lines       23150    28484    +5334     
==========================================
+ Hits        16317    18985    +2668     
- Misses       5776     8107    +2331     
- Partials     1057     1392     +335     

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 360 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[aaron-prindle]

What happens here if region is not specified and the flag is not specified? In this case would it make sense to fail with a message stating region needs to be specified? We might need to add a validation rule and an actionable error here

[bskaplan]

You're right, we should have validation on this. Is there somewhere in particular the validation should go? If not, I'll add it to cloudrun.NewDeployer.

[aaron-prindle]

validations typically go here:
https://github.com/GoogleContainerTools/skaffold/blob/main/pkg/skaffold/schema/validation/validation.go

[aaron-prindle]

LGTM