chore: updating google api and opentelemetry version

[bskaplan]

Description

Update the Google Cloud API Client and OpenTelemetry to the latest versions.

User facing changes (remove if N/A)

The latest Cloud Run features will now be supported, notably built-in support for mounting gcsfuse and NFS volumes. See https://cloud.google.com/run/docs/configuring/services/cloud-storage-volume-mounts#yaml and https://cloud.google.com/run/docs/configuring/services/nfs-volume-mounts#yaml for example YAMLs.

[codecov]

Codecov Report

Attention: Patch coverage is 80.00000% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 63.58%. Comparing base (290280e) to head (96ea662).
Report is 1118 commits behind head on main.

Files	Patch %	Lines
pkg/skaffold/instrumentation/export.go	82.60%	4 Missing ⚠️
pkg/skaffold/instrumentation/firelog/exporter.go	33.33%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9352      +/-   ##
==========================================
- Coverage   70.48%   63.58%   -6.91%     
==========================================
  Files         515      635     +120     
  Lines       23150    32790    +9640     
==========================================
+ Hits        16317    20848    +4531     
- Misses       5776    10334    +4558     
- Partials     1057     1608     +551     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[idsulik]

@bskaplan hi! why you added fs/assets/credits_generated/github.com/hashicorp/hcl/go.mod? Any go command changes this file now according to this golang/go#34842.

[bskaplan]

I didn't have any particular reason. I was updating the dependent modules, I guess that file got picked up when I updated them and I was mostly focused on making sure everything still worked with the updates. If that file isn't necessary, you can send another PR deleting it.

[idsulik]

@bskaplan have you deleted the line by yourself? because golang adds this line by itself whenever you use any go command.
@ericzzzzzzz , what do you think about this #9352 (comment) ? should we delete the go.mod file or should we specify go version there?

[ericzzzzzzz]

Hi @idsulik, I think we may have to keep this file. The file was auto-generated by go-license, go-license is supposed to just extract LICENSES from dependencies of skaffold without other files, but somehow it also includes this file. We have license check in our ci which also compares the output from go-license run on our ci and the generated_credits folder from pr branch, if there is discrepancy, ci will fail. Although it's a little annoying, it doesn't affect skaffold functionalities.

[idsulik]

@ericzzzzzzz Thank you for the details, added the go version here https://github.com/GoogleContainerTools/skaffold/pull/9390/files#diff-8939f41b03be02a3877b5d557f6b6adb21c33906b5c3093f6e7f626ed9f07044R3 , otherwise go cli updates it automatically and you see changed file, so you have to manually exclude it from commit