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

Add pull request preferences backoffice #1744

Closed
wants to merge 11 commits into from

Conversation

apiraino
Copy link
Contributor

@apiraino apiraino commented Nov 8, 2023

No description provided.

This CLI tool insert or sync all members defined in
$NEW_PR_ASSIGNMENT_TEAMS into the review preferences DB table.
This is the implementation of a new workflow for assigning pull requests
to the Rust project contributors.

This new workflow will assign a pull request to be reviewed based on the
number of assigned pull requests to team members.

Everytime a pull request assignment is invoked with `r? <team>` or `r?
<team_member>` the Triagebot will check the current workload of the
candidates and assign the pull request to the team member less busy.

The new workflow is DISABLED by default. It can be enabled by setting
the env variable `USE_NEW_PR_ASSIGNMENT` (any value) and restarting the
Triagebot.

Teams that are subject to the new PR review assignment are listed as a
comma separated list in the env variable `NEW_PR_ASSIGNMENT_TEAMS`. See
`.env.sample` for a usage example.

Team members workload is tracked in a new DB table `review_capacity`.

Both the initial population and the synchronization of this table are
handled by a command line tool that will speak to the Triagebot through
a number of HTTP endpoints. These HTTP endpoints must be PRIVATE and
accessible ONLY to the sync tool. This should be handled at the
infrastructure level, the endpoints are not authenticated.
@apiraino apiraino closed this Nov 8, 2023
@apiraino apiraino deleted the new-pr-assignment branch November 8, 2023 21:41
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.

1 participant