Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add cluster name to all v2alpha examples #1172

Merged
merged 15 commits into from
May 16, 2024
Merged

Add cluster name to all v2alpha examples #1172

merged 15 commits into from
May 16, 2024

Conversation

dogeahill
Copy link
Contributor

What does this PR do?

Add clusterName to all v2alpha examples.

Motivation

While implementing on Azure Redhat OpenShift, we used the example and had to add clusterName to properly view the data in our Kubernetes Overview page.

It is a known issue that without the clusterName, the Kubernetes explorer will not be useful as nothing will show besides the container view.

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

@dogeahill dogeahill requested review from a team as code owners May 8, 2024 16:26
@codecov-commenter
Copy link

codecov-commenter commented May 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.55%. Comparing base (6b0c7bd) to head (fda3339).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1172   +/-   ##
=======================================
  Coverage   59.55%   59.55%           
=======================================
  Files         174      174           
  Lines       21559    21559           
=======================================
  Hits        12839    12839           
  Misses       7941     7941           
  Partials      779      779           
Flag Coverage Δ
unittests 59.55% <ø> (ø)

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


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 6b0c7bd...fda3339. Read the comment docs.

@dogeahill dogeahill added the enhancement New feature or request label May 8, 2024
@celenechang
Copy link
Contributor

thank you @dogeahill

@celenechang celenechang merged commit 027c033 into main May 16, 2024
21 of 23 checks passed
@celenechang celenechang deleted the dogeahill-patch-1 branch May 16, 2024 17:00
@celenechang celenechang added documentation Improvements or additions to documentation and removed enhancement New feature or request labels May 16, 2024
mftoure pushed a commit that referenced this pull request Oct 3, 2024
* Update datadog-agent-minimum.yaml

Adding cluster name to all example files, as it is required for our Kubernetes explorer page to function properly

* Update datadog-agent-all.yaml

* Update datadog-agent-on-openshift.yaml

* Update datadog-agent-with-apm-hostport.yaml

* Update datadog-agent-with-clusteragent.yaml

* Update datadog-agent-with-dca-clusterchecksrunner.yaml

* Update datadog-agent-with-extraconfd.yaml

* Update datadog-agent-with-dca-with-orchestrator-explorer.yaml

* Update datadog-agent-with-logs-apm.yaml

* Update datadog-agent-with-secret-backend.yaml

* Update datadog-agent-with-remote-configuration.yaml

* Update datadog-agent-with-unix-socket.yaml

* Update datadog-agent-with-tolerations.yaml

* Update datadog-agent-with-secret-keys.yaml

* Update datadog-agent-with-otlp.yaml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation qa/skip-qa
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants