-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
CODEOWNERS: Split test/ code owners #14244
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, this is a nice change 🚀.
Signed-off-by: Paul Chaignon <paul@cilium.io>
f33c5b2
to
fd1dc9e
Compare
After discussion with André and Maciej, I've reworked several files in I've re-requested all your reviews because you should know better than to review a draft pull request 😝 |
test-nightly |
Signed-off-by: Paul Chaignon <paul@cilium.io>
fd1dc9e
to
62e93db
Compare
PoliciesNightly are still executed and should remain. Signed-off-by: Paul Chaignon <paul@cilium.io>
Suggested-by: Maciej Kwiek <maciej@isovalent.com> Signed-off-by: Paul Chaignon <paul@cilium.io>
The cilium/ci team (renamed in this commit to cilium/ci-structure) is in charge of the CI infra. (e.g., Jenkinsfile, scripts, etc.), including the Go helper functions and the overall structure of our tests, but it doesn't have the charge of the actual testing logic. This commit splits the test/ code owners across team to better represents the actual owners of the testing logic. Signed-off-by: Paul Chaignon <paul@cilium.io>
62e93db
to
8ff2587
Compare
After discussion with Maciej, we decided to remove the Nightly test files (except for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
The @cilium/ci team (renamed in this pull request to @cilium/ci-structure) is in charge of the CI infra. (e.g., Jenkinsfile, scripts, etc.), including the Go helper functions and the overall structure of our tests, but it doesn't have the charge of the actual testing logic.
This pull request splits the
test/
code owners across team to better represents the actual owners of the testing logic.I have been unable to assign the following files to specific teams and they are therefore matched by the @cilium/ci catch-all at the moment: