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

roachtest,CODEOWNERS: remove primary assignments to the server team #82992

Merged
merged 1 commit into from
Jun 24, 2022

Conversation

knz
Copy link
Contributor

@knz knz commented Jun 16, 2022

As discussed with @lunevalex and @jtsiros .

As the server team is currently unstaffed, we don't want issues to
be automatically assigned to it. This commit redirects the
issues to either the appropriate team, or to the virtual "noowner"
team.

The cockroach/server-prs notification target remains in CODEOWNERS
so that folk can use it to register to changes to particular code areas.

Release note: None

@knz knz requested a review from a team as a code owner June 16, 2022 14:26
@knz knz requested review from renatolabs and removed request for a team June 16, 2022 14:26
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@knz knz force-pushed the 20220616-owners branch from 8e8a518 to a4d31c4 Compare June 16, 2022 14:27
Copy link
Collaborator

@celiala celiala left a comment

Choose a reason for hiding this comment

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

code owner areas assigned to Dev Infra team LGTM!

As the server team is currently unstaffed, we don't want issues to
be automatically assigned to it. This commit redirects the
issues to either the appropriate team, or to the virtual "noowner"
team.

The `cockroach/server-prs` notification target remains in CODEOWNERS
so that folk can use it to register to changes to particular code areas.

Release note: None
@knz knz force-pushed the 20220616-owners branch from a4d31c4 to ba6b0a7 Compare June 23, 2022 17:34
@knz
Copy link
Contributor Author

knz commented Jun 23, 2022

bors r=celiala

@craig
Copy link
Contributor

craig bot commented Jun 23, 2022

This PR was included in a batch that was canceled, it will be automatically retried

@craig
Copy link
Contributor

craig bot commented Jun 23, 2022

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Jun 24, 2022

Build succeeded:

@craig craig bot merged commit 7af6c0d into cockroachdb:master Jun 24, 2022
@knz knz deleted the 20220616-owners branch July 18, 2022 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants