chore: transfer dataplane feature ownership to agent-data-plane team#2929
Conversation
…lane team Transfer /internal/controller/datadogagent/feature/dataplane CODEOWNERS from agent-metric-pipelines to the new agent-data-plane team. Dogstatsd feature ownership remains with agent-metric-pipelines. Also register Agent Data Plane (DADP jira project, team/agent-data-plane label) in .ddqa/config.toml. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 95031deecc
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| /internal/controller/datadogagent/feature/cspm @DataDog/agent-security @DataDog/container-platform | ||
| /internal/controller/datadogagent/feature/cws @DataDog/agent-security @DataDog/container-platform | ||
| /internal/controller/datadogagent/feature/dataplane @DataDog/agent-metric-pipelines @DataDog/container-platform | ||
| /internal/controller/datadogagent/feature/dataplane @DataDog/agent-data-plane @DataDog/container-platform |
There was a problem hiding this comment.
Create the new team label before routing dataplane PRs
For any PR that changes /internal/controller/datadogagent/feature/dataplane after this lands, .github/workflows/pr-codeowners-labeler.yml:156-174 derives team/agent-data-plane from this CODEOWNERS entry and fails the workflow if that label is absent. I checked the repository's current public label list and team/agent-data-plane is not present, while the previous team/agent-metric-pipelines label is, so dataplane PRs will start failing the labeler until the new label exists or this owner maps to an existing label.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Added the missing label.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2929 +/- ##
=======================================
Coverage 40.69% 40.69%
=======================================
Files 321 321
Lines 28413 28413
=======================================
Hits 11563 11563
Misses 16015 16015
Partials 835 835
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
|
🎯 Code Coverage (details) 🔗 Commit SHA: 95031de | Docs | Datadog PR Page | Give us feedback! |
f11b43c
into
main
Summary
/internal/controller/datadogagent/feature/dataplaneCODEOWNERS from@DataDog/agent-metric-pipelinesto the new@DataDog/agent-data-planeteam./internal/controller/datadogagent/feature/dogstatsd) remains owned by@DataDog/agent-metric-pipelines.[teams."Agent Data Plane"]in.ddqa/config.toml(jira projectDADP, labelteam/agent-data-plane).Test plan
@DataDog/agent-data-planeas a reviewer for dataplane changes🤖 Generated with Claude Code