Implement PR quota workflow #7882

yurishkuro · 2026-01-16T02:15:58Z

Which problem is this PR solving?

Resolves PR Quota Management System #7881

Signed-off-by: Yuri Shkuro <github@ysh.us>

Copilot

Pull request overview

This PR implements a PR quota management system that limits concurrent open pull requests from new contributors based on their merge history. The system automatically applies labels and posts comments to PRs that exceed quota limits, and removes restrictions when quota becomes available.

Changes:

Added GitHub Actions workflow that triggers on PR events (opened, closed, reopened) and manual dispatch
Implemented core quota logic with tiered limits (1 PR for 0 merges, 2 for 1 merge, 3 for 2 merges, unlimited for 3+)
Created comprehensive test suite covering quota calculation, PR fetching, label management, and comment posting

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
.github/workflows/pr-quota-manager.yml	GitHub Actions workflow configuration for automated quota management
.github/scripts/pr-quota-manager.js	Core implementation of quota calculation, PR processing, and GitHub API interactions
.github/scripts/pr-quota-manager.test.js	Unit tests for all quota management functions
.github/scripts/package.json	Node.js package configuration with dependencies and test scripts
.github/scripts/list-open-prs-by-author.js	Utility script for listing and analyzing open PRs by author
.github/scripts/README.md	Documentation for manual execution and troubleshooting
.github/scripts/.gitignore	Git ignore configuration for node_modules

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

.github/scripts/README.md

Signed-off-by: Yuri Shkuro <github@ysh.us>

graphite-app · 2026-01-16T02:20:28Z

.github/workflows/pr-quota-manager.yml

+            const handler = require('./.github/scripts/pr-quota-manager.js')
+            const username = context.payload.pull_request?.user?.login || context.payload.inputs?.username
+            const owner = context.repo.owner
+            const repo = context.repo.repo


GitHub Actions workflow_dispatch boolean inputs are passed as strings, not booleans. When dryRun is set to false via the UI, context.payload.inputs.dryRun will be the string "false", which is truthy in JavaScript. This causes dry-run mode to always be enabled when manually triggered.

Fix:

const dryRun = context.payload.inputs?.dryRun === true || context.payload.inputs?.dryRun === 'true'

Suggested change

const repo = context.repo.repo

const dryRun = context.payload.inputs?.dryRun === true || context.payload.inputs?.dryRun === 'true'

Spotted by Graphite Agent

Is this helpful? React 👍 or 👎 to let us know.

codecov · 2026-01-16T02:29:32Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.52%. Comparing base (1d24304) to head (5fb93f2).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7882      +/-   ##
==========================================
+ Coverage   95.51%   95.52%   +0.01%     
==========================================
  Files         305      305              
  Lines       16212    16212              
==========================================
+ Hits        15485    15487       +2     
+ Misses        569      568       -1     
+ Partials      158      157       -1

Flag	Coverage Δ
badger_v1	`9.15% <ø> (ø)`
badger_v2	`1.93% <ø> (ø)`
cassandra-4.x-v1-manual	`13.53% <ø> (ø)`
cassandra-4.x-v2-auto	`1.92% <ø> (ø)`
cassandra-4.x-v2-manual	`1.92% <ø> (ø)`
cassandra-5.x-v1-manual	`13.53% <ø> (ø)`
cassandra-5.x-v2-auto	`1.92% <ø> (ø)`
cassandra-5.x-v2-manual	`1.92% <ø> (ø)`
clickhouse	`1.97% <ø> (ø)`
elasticsearch-6.x-v1	`17.47% <ø> (ø)`
elasticsearch-7.x-v1	`17.50% <ø> (ø)`
elasticsearch-8.x-v1	`17.66% <ø> (ø)`
elasticsearch-8.x-v2	`1.93% <ø> (ø)`
elasticsearch-9.x-v2	`1.93% <ø> (ø)`
grpc_v1	`8.81% <ø> (ø)`
grpc_v2	`1.93% <ø> (ø)`
kafka-3.x-v2	`1.93% <ø> (ø)`
memory_v2	`1.93% <ø> (ø)`
opensearch-1.x-v1	`17.55% <ø> (ø)`
opensearch-2.x-v1	`17.55% <ø> (ø)`
opensearch-2.x-v2	`1.93% <ø> (ø)`
opensearch-3.x-v2	`1.93% <ø> (ø)`
query	`1.93% <ø> (ø)`
tailsampling-processor	`0.55% <ø> (ø)`
unittests	`94.18% <ø> (+0.01%)`	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

github-actions · 2026-01-16T02:30:11Z

Metrics Comparison Summary

Total changes across all snapshots: 0

Detailed changes per snapshot

summary_metrics_snapshot_cassandra

📊 Metrics Diff Summary

Total Changes: 0

🆕 Added: 0 metrics
❌ Removed: 0 metrics
🔄 Modified: 0 metrics
🚫 Excluded: 106 metrics

summary_metrics_snapshot_cassandra

📊 Metrics Diff Summary

Total Changes: 0

🆕 Added: 0 metrics
❌ Removed: 0 metrics
🔄 Modified: 0 metrics
🚫 Excluded: 106 metrics

summary_metrics_snapshot_cassandra

📊 Metrics Diff Summary

Total Changes: 0

🆕 Added: 0 metrics
❌ Removed: 0 metrics
🔄 Modified: 0 metrics
🚫 Excluded: 53 metrics

➡️ View full metrics file

Signed-off-by: Yuri Shkuro <github@ysh.us>

Copilot

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

.github/scripts/README.md:1

Extra space between '=' and .repeat(80) should be removed.

# PR Quota Manager - Manual Execution Guide

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: Yuri Shkuro <github@ysh.us>

Copilot

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

.github/workflows/pr-quota-manager.yml

.github/scripts/README.md

Signed-off-by: Yuri Shkuro <github@ysh.us>

Copilot

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

.github/scripts/README.md:1

Corrected spacing in 'repeat(80)' - there should be no space before 'repeat'.

# PR Quota Manager - Manual Execution Guide

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

.github/workflows/pr-quota-manager.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

## Which problem is this PR solving? - Resolves jaegertracing#7881 --------- Signed-off-by: Yuri Shkuro <github@ysh.us> Signed-off-by: Yuri Shkuro <yurishkuro@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Implement PR quota workflow

5aee0c9

Signed-off-by: Yuri Shkuro <github@ysh.us>

Copilot AI review requested due to automatic review settings January 16, 2026 02:15

yurishkuro requested a review from a team as a code owner January 16, 2026 02:15

yurishkuro added the changelog:ci Change related to continuous integration / testing label Jan 16, 2026

yurishkuro requested a review from jkowall January 16, 2026 02:16

dosubot bot added the enhancement label Jan 16, 2026

Copilot AI reviewed Jan 16, 2026

View reviewed changes

.github/scripts/README.md Outdated Show resolved Hide resolved

.github/scripts/README.md Outdated Show resolved Hide resolved

fix

47bca19

Signed-off-by: Yuri Shkuro <github@ysh.us>

graphite-app bot reviewed Jan 16, 2026

View reviewed changes

fix

b24aa06

Signed-off-by: Yuri Shkuro <github@ysh.us>

Copilot AI review requested due to automatic review settings January 16, 2026 02:34

Copilot AI reviewed Jan 16, 2026

View reviewed changes

yurishkuro added 2 commits January 15, 2026 21:36

fix

ded188b

Signed-off-by: Yuri Shkuro <github@ysh.us>

fix

3e6e488

Signed-off-by: Yuri Shkuro <github@ysh.us>

Copilot AI review requested due to automatic review settings January 16, 2026 02:39

Copilot AI reviewed Jan 16, 2026

View reviewed changes

.github/workflows/pr-quota-manager.yml Outdated Show resolved Hide resolved

.github/scripts/README.md Show resolved Hide resolved

yurishkuro added 2 commits January 15, 2026 21:40

fix

cde68d7

Signed-off-by: Yuri Shkuro <github@ysh.us>

fix

0845f81

Signed-off-by: Yuri Shkuro <github@ysh.us>

Copilot AI review requested due to automatic review settings January 16, 2026 02:53

Copilot AI reviewed Jan 16, 2026

View reviewed changes

.github/workflows/pr-quota-manager.yml Outdated Show resolved Hide resolved

yurishkuro commented Jan 16, 2026

View reviewed changes

.github/workflows/pr-quota-manager.yml Outdated Show resolved Hide resolved

Apply suggestion from @Copilot

5fb93f2

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 16, 2026 02:54

Copilot AI reviewed Jan 16, 2026

View reviewed changes

jaegertracingbot approved these changes Jan 16, 2026

View reviewed changes

yurishkuro merged commit 2ae733b into jaegertracing:main Jan 16, 2026
59 checks passed

yurishkuro deleted the pr-quota-system branch January 16, 2026 05:01

	const repo = context.repo.repo
	const dryRun = context.payload.inputs?.dryRun === true \|\| context.payload.inputs?.dryRun === 'true'

Implement PR quota workflow #7882

Implement PR quota workflow #7882

Uh oh!

Conversation

yurishkuro commented Jan 16, 2026

Which problem is this PR solving?

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

graphite-app bot Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

codecov bot commented Jan 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

github-actions bot commented Jan 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Metrics Comparison Summary

summary_metrics_snapshot_cassandra

📊 Metrics Diff Summary

summary_metrics_snapshot_cassandra

📊 Metrics Diff Summary

summary_metrics_snapshot_cassandra

📊 Metrics Diff Summary

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

codecov bot commented Jan 16, 2026 •

edited

Loading

github-actions bot commented Jan 16, 2026 •

edited

Loading