Commit f7bbaf9
authored
ci(jobowners): refine @DataDog/multiple ownership for tests_* and new-e2e* (#49826)
### What does this PR do?
Refines the two `@DataDog/multiple` catch-alls in `.gitlab/JOBOWNERS` (`tests_*`, `new-e2e*`) into per-team patterns, and drops `@DataDog/multiple` entirely in favor of `@DataDog/agent-devx` as the fallback for uncategorized jobs.
### Motivation
Done to better involve owning teams in CI incident resolution. Today, `@DataDog/multiple` routes hundreds of `tests_*` / `new-e2e-*` failures to no specific owner even though ownership is already knowable from the job's `TEAM:` variable — this sends incidents directly to the team that owns the code being tested. Any uncategorized job falls back to `agent-devx` (owner of the test framework, runners, and this file) rather than the nobody-owns-it `multiple` alias.
### Judgment calls (please sanity-check for your team's jobs)
- **`tests_*` fallback** → `@DataDog/agent-devx`: generic Linux/macOS unit tests exercise the whole repo; no single product team owns them.
- **`new-e2e*` fallback** → `@DataDog/agent-devx`: same rationale; agent-devx owns the e2e framework and runners.
- **`protobuf_test`** → `@DataDog/agent-runtimes`: owners of `/pkg/proto/` and the `dda inv protobuf.generate` task.
- **`tests_flavor_iot*` / `tests_flavor_heroku*`** → `@DataDog/agent-runtimes`: no dedicated CODEOWNERS entry; best guess.
- **`new-e2e-windows-systemprobe*`** → shared with `ebpf-platform` even though the job's `TEAM:` var is only `windows-products`, because the test exercises system-probe.
- **`new-e2e-windows-fips-compliance-test*`** → shared with `agent-runtimes` to mirror the non-Windows `new-e2e-fips-compliance-test*` owner.
- **`new-e2e-sbom`** (agent-security + container-integrations) and **`new-e2e-cspm`** (agent-cspm + agent-security): shared ownership mirrors CODEOWNERS.
- **`new-e2e-installer-windows*`** → shared fleet + windows-products (installer logic is fleet-owned, Windows specifics are windows-products).
### Describe how you validated your changes
`dda inv -- -e linter.full-gitlab-ci` passes (incl. `jobowners-exist`).
### Additional Notes
_None._
Co-authored-by: nicolas.schweitzer <nicolas.schweitzer@datadoghq.com>1 parent c02df1a commit f7bbaf9
1 file changed
Lines changed: 67 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
22 | | - | |
| 22 | + | |
| 23 | + | |
23 | 24 | | |
| 25 | + | |
| 26 | + | |
24 | 27 | | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
25 | 34 | | |
26 | 35 | | |
27 | 36 | | |
28 | 37 | | |
29 | | - | |
| 38 | + | |
30 | 39 | | |
31 | 40 | | |
32 | 41 | | |
| |||
182 | 191 | | |
183 | 192 | | |
184 | 193 | | |
185 | | - | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
| 199 | + | |
| 200 | + | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
| 204 | + | |
| 205 | + | |
| 206 | + | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
| 210 | + | |
| 211 | + | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
186 | 250 | | |
187 | 251 | | |
188 | 252 | | |
| |||
0 commit comments