[CECO-1441] Use fully-qualified domain name finalizer

[tbavelier]

What does this PR do?

• Uses fully-qualified domain name on dashboard CRD per k8s convention : https://kubernetes.io/docs/tasks/extend-kubernetes/custom-resources/custom-resource-definitions/#finalizers

Identifiers of custom finalizers consist of a domain name, a forward slash and the name of the finalizer. Any controller can add a finalizer to any object's list of finalizers.

Motivation

• https://datadoghq.atlassian.net/browse/CECO-1441

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

• Covered by tests, but apply dashboard CR, ensure there are no logs from KubeAPIWarningLogger in the operator, then delete the CR with kubectl delete and ensure the CR is properly deleted

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

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.40%. Comparing base (4c40ffd) to head (9f5e30c).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1397   +/-   ##
=======================================
  Coverage   47.40%   47.40%           
=======================================
  Files         223      223           
  Lines       19244    19244           
=======================================
  Hits         9123     9123           
  Misses       9635     9635           
  Partials      486      486           

Flag	Coverage Δ
unittests	47.40% <ø> (ø)

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

Files with missing lines	Coverage Δ
internal/controller/datadogdashboard/finalizer.go	61.53% <ø> (ø)

---

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 4c40ffd...9f5e30c. Read the comment docs.

[tbavelier]

/merge

[dd-devflow]

🚂 MergeQueue: pull request added to the queue

The median merge time in main is 15m.

Use /merge -c to cancel this operation!