From f9148b6b831158ab24b3b89ca431f789e0b2ca4e Mon Sep 17 00:00:00 2001 From: Grigory Panov Date: Thu, 6 Aug 2026 10:19:39 +0200 Subject: [PATCH 1/2] OWNERS: own the environments integration tests Add @rugpanov, @rclarey, @anton-107 and @misha-db as owners of /integration/cmd/environments/, matching the existing ownership of /libs/localenv/, /cmd/environments/ and /acceptance/localenv/ so the DB Connect / setup-local tests are reviewed by the same people as the code they exercise. The rest of /integration/ stays with team:platform. The entry has to sit below the broader /integration/ rule: findOwners is last-match-wins, so placing it next to the other localenv lines earlier in the file would let team:platform override it. Verified both ways with findOwners before settling on this position. --- .github/OWNERS | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/OWNERS b/.github/OWNERS index 76bc081f0eb..04fe48cf6df 100644 --- a/.github/OWNERS +++ b/.github/OWNERS @@ -64,6 +64,10 @@ # Integration tests /integration/ team:platform +# Must stay below /integration/: rules are last-match-wins, so the broader +# team:platform rule above would otherwise take precedence for these paths. +/integration/cmd/environments/ @rugpanov @rclarey @anton-107 @misha-db + # Internal /internal/ team:platform From 722c947cf9832d5079a77aecf62e7196b487da7a Mon Sep 17 00:00:00 2001 From: Grigory Panov Date: Thu, 6 Aug 2026 10:33:48 +0200 Subject: [PATCH 2/2] OWNERS: introduce team:ide for the localenv / DB Connect paths The four localenv paths each repeated the same list of four handles, so a roster change meant four edits that could silently drift apart. Define team:ide in OWNERTEAMS once and reference it from all of them, including the new /integration/cmd/environments/ entry. Ownership is unchanged: all four paths still resolve to @rugpanov, @rclarey, @anton-107 and @misha-db, verified with findOwners. No GitHub org team page yet, so the validator emits its non-blocking "no GitHub team-page URL" warning, same as team:ai-training. Creating the org team needs org-owner access; the alias works standalone in the meantime, which is how team:bundle and team:platform already operate (neither cli-maintainers nor cli-platform exists as an org team today). --- .github/OWNERS | 8 ++++---- .github/OWNERTEAMS | 1 + 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/.github/OWNERS b/.github/OWNERS index 04fe48cf6df..46a78a1a8c4 100644 --- a/.github/OWNERS +++ b/.github/OWNERS @@ -15,9 +15,9 @@ /cmd/labs/ @alexott @asnare # Local environments / DB Connect -/libs/localenv/ @rugpanov @rclarey @anton-107 @misha-db -/cmd/environments/ @rugpanov @rclarey @anton-107 @misha-db -/acceptance/localenv/ @rugpanov @rclarey @anton-107 @misha-db +/libs/localenv/ team:ide +/cmd/environments/ team:ide +/acceptance/localenv/ team:ide # Apps /cmd/apps/ team:eng-apps-devex @@ -66,7 +66,7 @@ # Must stay below /integration/: rules are last-match-wins, so the broader # team:platform rule above would otherwise take precedence for these paths. -/integration/cmd/environments/ @rugpanov @rclarey @anton-107 @misha-db +/integration/cmd/environments/ team:ide # Internal /internal/ team:platform diff --git a/.github/OWNERTEAMS b/.github/OWNERTEAMS index 73dfc5cdc62..50df8a368ad 100644 --- a/.github/OWNERTEAMS +++ b/.github/OWNERTEAMS @@ -16,4 +16,5 @@ team:bundle @andrewnester @anton-107 @denik @janniklasrose @lennartkats-db @ team:platform @simonfaltum @renaudhartert-db @hectorcast-db @parthban-db @tanmay-db @Divyansh-db @tejaskochar-db @mihaimitrea-db @chrisst @rauchy team:eng-apps-devex @fjakobs @Shridhad @atilafassina @keugenek @igrekun @pkosiec @MarioCadenas @pffigueiredo @ditadi @calvarjorge team:eng-sandbox @pietern @shuochen0311 @akshaysingla-db @anwell-db @samhuan-db +team:ide @rugpanov @rclarey @anton-107 @misha-db team:ai-training @apeforest @bfontain @lu-wang-dl @panchalhp-db @vinchenzo-db @maggiewang-db @riddhibhagwat-db @ben-hansen-db @pardis-beikzadeh-db