From 3dbbdb76ff2370c0b14dda00e377970d53ddff4d Mon Sep 17 00:00:00 2001
From: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Date: Wed, 13 Nov 2024 07:38:18 +1100
Subject: [PATCH] [8.x] [codeowners] Filter kibanamachine (#199404) (#199864)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[codeowners] Filter kibanamachine
(#199404)](https://github.com/elastic/kibana/pull/199404)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"Jon","email":"jon@elastic.co"},"sourceCommit":{"committedDate":"2024-11-12T18:39:17Z","message":"[codeowners]
Filter kibanamachine (#199404)\n\nCurrently we remove CODEOWNERS on
backport branches to avoid review\r\nassignments: reviews were already
collected on the source pull request.\r\nIf there's a conflict, it will
go through another round of review but\r\nnot require all the original
assignees.\r\n\r\nWe want to re-add the file for our own tooling, and to
avoid CODEOWNERS\r\nmerge conflicts on backports. To do this, we're
going to add a global\r\noverride to code assignments on backport
branches.\r\n\r\nThis updates our CODEOWNERS libraries to ignore
assignments
to\r\n`kibanamachine`.\r\n\r\n---------\r\n\r\nCo-authored-by: Dzmitry
Lemechko
<dzmitry.lemechko@elastic.co>","sha":"a26f5d8b61b19aeeae741a7f750257e692edd430","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Operations","release_note:skip","v9.0.0","backport:all-open"],"title":"[codeowners]
Filter
kibanamachine","number":199404,"url":"https://github.com/elastic/kibana/pull/199404","mergeCommit":{"message":"[codeowners]
Filter kibanamachine (#199404)\n\nCurrently we remove CODEOWNERS on
backport branches to avoid review\r\nassignments: reviews were already
collected on the source pull request.\r\nIf there's a conflict, it will
go through another round of review but\r\nnot require all the original
assignees.\r\n\r\nWe want to re-add the file for our own tooling, and to
avoid CODEOWNERS\r\nmerge conflicts on backports. To do this, we're
going to add a global\r\noverride to code assignments on backport
branches.\r\n\r\nThis updates our CODEOWNERS libraries to ignore
assignments
to\r\n`kibanamachine`.\r\n\r\n---------\r\n\r\nCo-authored-by: Dzmitry
Lemechko
<dzmitry.lemechko@elastic.co>","sha":"a26f5d8b61b19aeeae741a7f750257e692edd430"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/199404","number":199404,"mergeCommit":{"message":"[codeowners]
Filter kibanamachine (#199404)\n\nCurrently we remove CODEOWNERS on
backport branches to avoid review\r\nassignments: reviews were already
collected on the source pull request.\r\nIf there's a conflict, it will
go through another round of review but\r\nnot require all the original
assignees.\r\n\r\nWe want to re-add the file for our own tooling, and to
avoid CODEOWNERS\r\nmerge conflicts on backports. To do this, we're
going to add a global\r\noverride to code assignments on backport
branches.\r\n\r\nThis updates our CODEOWNERS libraries to ignore
assignments
to\r\n`kibanamachine`.\r\n\r\n---------\r\n\r\nCo-authored-by: Dzmitry
Lemechko
<dzmitry.lemechko@elastic.co>","sha":"a26f5d8b61b19aeeae741a7f750257e692edd430"}}]}]
BACKPORT-->

Co-authored-by: Jon <jon@elastic.co>
---
 packages/kbn-code-owners/src/file_code_owner.ts              | 4 +++-
 packages/kbn-lint-packages-cli/migrate_plugins_to_package.ts | 3 ++-
 .../ingest_coverage/team_assignment/parse_owners.js          | 5 +++--
 3 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/packages/kbn-code-owners/src/file_code_owner.ts b/packages/kbn-code-owners/src/file_code_owner.ts
index 34f7970ca0d82..508fac9ce9741 100644
--- a/packages/kbn-code-owners/src/file_code_owner.ts
+++ b/packages/kbn-code-owners/src/file_code_owner.ts
@@ -36,7 +36,9 @@ export function getPathsWithOwnersReversed(): PathWithOwners[] {
   const codeownersLines = codeownersContent.split(/\r?\n/);
   const codeowners = codeownersLines
     .map((line) => line.trim())
-    .filter((line) => line && line[0] !== '#');
+    .filter((line) => line && line[0] !== '#')
+    // kibanamachine is an assignment override on backport branches to avoid review requests
+    .filter((line) => line && !line.includes('@kibanamachine'));
 
   const pathsWithOwners: PathWithOwners[] = codeowners.map((c) => {
     const [path, ...ghTeams] = c.split(/\s+/);
diff --git a/packages/kbn-lint-packages-cli/migrate_plugins_to_package.ts b/packages/kbn-lint-packages-cli/migrate_plugins_to_package.ts
index 8641fa3c324d9..789729993c0bc 100644
--- a/packages/kbn-lint-packages-cli/migrate_plugins_to_package.ts
+++ b/packages/kbn-lint-packages-cli/migrate_plugins_to_package.ts
@@ -78,7 +78,8 @@ export async function migratePluginsToPackages(legacyManifests: RepoPath[]) {
       .split('\n')
       .flatMap((line) => {
         const trim = line.trim();
-        if (!trim || trim.startsWith('#')) {
+        // kibanamachine is an assignment override on backport branches to avoid review requests
+        if (!trim || trim.startsWith('#') || trim.includes('@kibanamachine')) {
           return [];
         }
 
diff --git a/src/dev/code_coverage/ingest_coverage/team_assignment/parse_owners.js b/src/dev/code_coverage/ingest_coverage/team_assignment/parse_owners.js
index c384d4f010dc3..ed027e184ef75 100644
--- a/src/dev/code_coverage/ingest_coverage/team_assignment/parse_owners.js
+++ b/src/dev/code_coverage/ingest_coverage/team_assignment/parse_owners.js
@@ -13,8 +13,9 @@ import { pipe } from '../utils';
 
 const allLines$ = (lineReader) =>
   fromEvent(lineReader, 'line').pipe(
-    filter(function dropEmptiesAndDropComments(x) {
-      return x !== '' && !/^#\s{1,3}/.test(x);
+    filter(function dropEmptiesAndDropCommentsAndDropKibanamachine(x) {
+      // kibanamachine is an assignment override on backport branches to avoid review requests
+      return x !== '' && !/^#\s{1,3}/.test(x) && !x.includes('@kibanamachine');
     }),
     map(pipe(dropCCDelim, pathAndTeams)),
     takeUntil(fromEvent(lineReader, 'close'))