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

[datadog_logs_custom_pipeline] Add tags and description to logs pipelines #2773

Merged

Conversation

joboccara
Copy link
Contributor

@joboccara joboccara commented Jan 16, 2025

This PR adds the tags and description for pipelines to the Datadog Terraform provider.

The provider interprets the Terraform files and uses the Go SDK to invoke the Datadog public API to change the state of the logs pipelines and make them match the state expressed in the Terraform files. This PR makes it read and writes the tags and description of logs pipelines from the public API.

LOGSC-2078

@joboccara joboccara requested review from a team as code owners January 16, 2025 14:27
@joboccara joboccara marked this pull request as draft January 16, 2025 14:27
@joboccara joboccara marked this pull request as ready for review January 29, 2025 09:53
@joboccara joboccara requested a review from a team as a code owner January 29, 2025 09:53
thibaultkrebs
thibaultkrebs previously approved these changes Jan 29, 2025
janine-c
janine-c previously approved these changes Jan 29, 2025
InesDatadog
InesDatadog previously approved these changes Jan 29, 2025
Copy link

@InesDatadog InesDatadog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good for Logs Access ✅

Copy link

@EdwinSri EdwinSri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please add a PR description? 😄 I am not familiar with this part of the code so don't really understand what change is being made, why we are making this change and why my team needs to review it.

EDIT: edited out the part about linking the jira ticket since apparently github is smart enough to pick it up from the PR title

LorisFriedel
LorisFriedel previously approved these changes Jan 30, 2025
EdwinSri
EdwinSri previously approved these changes Jan 30, 2025
@joboccara joboccara changed the title LOGSC-2078 Add tags and description to logs pipelines [datadog_logs_custom_pipeline] Add tags and description to logs pipelines Jan 30, 2025
Co-authored-by: skarimo <[email protected]>
@joboccara joboccara requested a review from a team as a code owner February 4, 2025 10:15
LorisFriedel
LorisFriedel previously approved these changes Feb 4, 2025
@skarimo skarimo merged commit e85d563 into master Feb 4, 2025
10 checks passed
@skarimo skarimo deleted the jonathan.boccara/add-tags-description-to-logs-pipelines branch February 4, 2025 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants