Levan m/dependabot/go modules/k8s.io/apimachinery 0.35.0

[levan-m]

What does this PR do?

A brief description of the change being made with this pull request.

Failing due to this change kubernetes-sigs/controller-runtime#3229, related issue kubernetes-sigs/controller-runtime#3302.

Small functional changes
29cd7b5 run go mod tidy before golangci-lint for e2e, removed in last commit. Also move go work sync to the end of update-golang.sh script as it adds different patch versions to go directive in go.mod files, likely due to min version requirement for respective dependency tree.
eac776f store.go change to accomodate fake client, similar to change in #2650.
098eb84 test change not to reuse clientBuilder, broke with dependency bump.

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

Minimum Agent Versions

Are there minimum versions of the Datadog Agent and/or Cluster Agent required?

• Agent: vX.Y.Z
• Cluster Agent: vX.Y.Z

Describe your test plan

Write there any instructions and details you may have to test your PR.

Checklist

• PR has at least one valid label: bug, enhancement, refactoring, documentation, tooling, and/or dependencies
• PR has a milestone or the qa/skip-qa label
• All commits are signed (see: signing commits)

[codecov-commenter]

Codecov Report

❌ Patch coverage is 33.33333% with 12 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (dependabot/go_modules/k8s.io/apimachinery-0.35.0@694b73f). Learn more about missing BASE report.

Files with missing lines	Patch %	Lines
...atadog/autoscaling/cluster/install/k8s/nodepool.go	0.00%	12 Missing ⚠️

❌ Your patch status has failed because the patch coverage (33.33%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@                                 Coverage Diff                                 @@
##             dependabot/go_modules/k8s.io/apimachinery-0.35.0    #2681   +/-   ##
===================================================================================
  Coverage                                                    ?   38.73%           
===================================================================================
  Files                                                       ?      307           
  Lines                                                       ?    26522           
  Branches                                                    ?        0           
===================================================================================
  Hits                                                        ?    10274           
  Misses                                                      ?    15483           
  Partials                                                    ?      765           

Flag	Coverage Δ
unittests	38.73% <33.33%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
internal/controller/datadogagent/store/store.go	68.39% <100.00%> (ø)
...atadog/autoscaling/cluster/install/k8s/nodepool.go	0.00% <0.00%> (ø)

---

Continue to review full report in Codecov by Sentry.

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[tbavelier]

This is "ugly/useless", it's not really introduced by this PR since we have the same for resourceFieldRef and so on. I'll make a PR to add // +doc-gen:truncate to this field and some others:
edit: #2706