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

feat(all charts): improve datadog tagging and correlation #253

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

diranged
Copy link
Contributor

Per the docs at
https://docs.datadoghq.com/getting_started/tagging/unified_service_tagging/?tab=kubernetes - we should not only be labeling our Pods, but the Deployments they're a part of. This PR takes that one step further and just adds the datadogLabels to all of our resource labels. The motivation here is that going forward anything that the Datadog team can use to correlate our resourcs will all be joined together with common labels.

@diranged diranged added the enhancement New feature or request label Dec 11, 2023
@diranged diranged self-assigned this Dec 11, 2023
@diranged diranged requested a review from a team as a code owner December 11, 2023 17:58
@diranged diranged force-pushed the unified_labels branch 3 times, most recently from 16b7591 to 1b1010a Compare December 11, 2023 18:17
Per the docs at
https://docs.datadoghq.com/getting_started/tagging/unified_service_tagging/?tab=kubernetes - we should not only be labeling our Pods, but the Deployments they're a part of. This PR takes that one step further and just adds the datadogLabels to all of our resource labels. The motivation here is that going forward anything that the Datadog team can use to correlate our resourcs will all be joined together with common labels.
@diranged diranged merged commit ed34754 into main Dec 11, 2023
2 checks passed
@diranged diranged deleted the unified_labels branch December 11, 2023 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant