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

[8.x] Override all codeowners #197710

Closed
wants to merge 4 commits into from
Closed

[8.x] Override all codeowners #197710

wants to merge 4 commits into from

Conversation

jbudz
Copy link
Member

@jbudz jbudz commented Oct 24, 2024

On older release branches we want to preserve CODEOWNERS for other automation, and to reduce the number of merge conflicts on backports.

We also don't want reviewer assigned and the subsequent notifications sent when these reviews are not a requirement.

This assigns kibanamachine as the owner of all files.

We want to preserve CODEOWNERS for other automation, and to reduce
the number of merge conflicts on backports, but avoid
notifications on backports where these reviews aren't required.

This assigns kibanamachine as the owner of all files.
@jbudz jbudz added Team:Operations Team label for Operations Team release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting labels Oct 24, 2024
@jbudz jbudz marked this pull request as ready for review October 24, 2024 18:18
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@jbudz jbudz marked this pull request as draft October 24, 2024 18:24
@jbudz
Copy link
Member Author

jbudz commented Oct 24, 2024

Converting to draft for the moment, I need to run some tests on assignment when kibanamachine is the PR author.

.github/CODEOWNERS Outdated Show resolved Hide resolved
@jbudz jbudz marked this pull request as ready for review October 29, 2024 19:23
@jbudz jbudz requested a review from a team as a code owner October 29, 2024 19:23
@elasticmachine
Copy link
Contributor

⏳ Build in-progress, with failures

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #2 / dev/mocha/junit report generation reports on failed setup hooks
  • [job] [logs] Jest Tests #2 / dev/mocha/junit report generation reports on failed setup hooks

@jbudz
Copy link
Member Author

jbudz commented Oct 29, 2024

Glad there was a test. I see at least 4 different implementations of CODEOWNERS parsing that will need to be updated to skip the last entry. I'll revisit this starting from main.

@jbudz jbudz marked this pull request as draft October 29, 2024 22:44
@elasticmachine
Copy link
Contributor

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!

Ikuni17 added a commit that referenced this pull request Oct 30, 2024
Temporary removal of `CODEOWNERS` to stop review notifications until
#197710 issues are resolved.
@jbudz
Copy link
Member Author

jbudz commented Nov 12, 2024

Closing in favor of #199686

@jbudz jbudz closed this Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants