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

STONEBLD-2648: fix OWNERS files #1273

Merged
merged 2 commits into from
Aug 13, 2024
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
approvers:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will it mean that we'll be pinged to all PRs in the repo even the tasks we don't have relation to?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is kind of default.. if some task don't have OWNERS file..and other sub-directories like pipelines or .tekton where OWNERS file are not there.. then it will fall back to OWNERS file in the root for assigning reviews.

- build-team
reviewers:
- build-team
30 changes: 30 additions & 0 deletions OWNERS_ALIASES
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
# See the OWNERS_ALIASES docs: https://git.k8s.io/community/contributors/guide/owners.md#owners_aliases
aliases:
build-team:
- mmorhun
- chmeliik
- mkosiarc
- rcerven
- tkdchen
- tisutisu
- tnevrlka
- MartinBasti
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we also add the gold team as the prefetch task maintainers? At least a few additional members apart from @MartinBasti? E.g. @eskultety @brunoapimentel @taylormadore

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay added them in the build-team alias.

- eskultety
- brunoapimentel
- taylormadore
integration-team:
- dirgim
- jsztuka
- hongweiliu17
- Josh-Everett
- sonam1412
- 14rcole
- dheerajodha
- kasemAlem
ec-team:
- simonbaird
- lcarva
- zregvart
- joejstuart
- robnester-rh
- cuipinghuo
6 changes: 5 additions & 1 deletion task/apply-tags/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Konflux Build team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/build-image-manifest/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/buildah-10gb/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/buildah-20gb/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/buildah-24gb/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/buildah-6gb/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/buildah-8gb/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/buildah-oci-ta/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/buildah-remote/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/buildah-rhtap/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Konflux Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/buildah/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/clair-scan/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Test Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- integration-team
reviewers:
- integration-team
6 changes: 5 additions & 1 deletion task/clamav-scan/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Test Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- integration-team
reviewers:
- integration-team
6 changes: 5 additions & 1 deletion task/deprecated-image-check/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Test Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- integration-team
reviewers:
- integration-team
5 changes: 4 additions & 1 deletion task/ecosystem-cert-preflight-checks/OWNERS
Original file line number Diff line number Diff line change
@@ -1,2 +1,5 @@
approvers:
- skattoju
Copy link
Collaborator

@mmorhun mmorhun Aug 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you should add another devs that are mentioned in the comment

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated now. Thanks.


opdev
reviewers:
- skattoju
6 changes: 5 additions & 1 deletion task/fbc-related-image-check/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Test Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- integration-team
reviewers:
- integration-team
6 changes: 5 additions & 1 deletion task/fbc-validation/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Test Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- integration-team
reviewers:
- integration-team
6 changes: 5 additions & 1 deletion task/git-clone-oci-ta/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/git-clone/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/init/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/inspect-image/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Test Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- integration-team
reviewers:
- integration-team
6 changes: 5 additions & 1 deletion task/oci-copy-oci-ta/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
ralphbean
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- ralphbean
mmorhun marked this conversation as resolved.
Show resolved Hide resolved
reviewers:
- ralphbean
7 changes: 6 additions & 1 deletion task/oci-copy/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,6 @@
ralphbean
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- ralphbean
reviewers:
- ralphbean

1 change: 1 addition & 0 deletions task/operator-sdk-generate-bundle/OWNERS
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- jbpratt
- gurnben
Expand Down
1 change: 1 addition & 0 deletions task/opm-render-bundles/OWNERS
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- jbpratt
- gurnben
Expand Down
6 changes: 5 additions & 1 deletion task/prefetch-dependencies-oci-ta/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/prefetch-dependencies/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/push-dockerfile-oci-ta/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/push-dockerfile/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/s2i-java/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/s2i-nodejs/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/sast-snyk-check-oci-ta/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Test Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- integration-team
reviewers:
- integration-team
6 changes: 5 additions & 1 deletion task/sast-snyk-check/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Test Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- integration-team
reviewers:
- integration-team
6 changes: 5 additions & 1 deletion task/sbom-json-check/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Test Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- integration-team
reviewers:
- integration-team
6 changes: 5 additions & 1 deletion task/show-sbom-rhdh/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/show-sbom/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/slack-webhook-notification/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/source-build-oci-ta/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/source-build/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/summary/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/tkn-bundle/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Enterprise Contract Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- ec-team
reviewers:
- ec-team
6 changes: 5 additions & 1 deletion task/update-deployment/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/update-infra-deployments/OWNERS
tisutisu marked this conversation as resolved.
Show resolved Hide resolved
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stonesoup Build Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- build-team
reviewers:
- build-team
6 changes: 5 additions & 1 deletion task/verify-enterprise-contract/OWNERS
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Enterprise Contract Team
# See the OWNERS docs: https://go.k8s.io/owners
approvers:
- ec-team
reviewers:
- ec-team
Loading