Skip to content

Allow large repo dashboard webhooks#68

Open
trask wants to merge 1 commit into
open-telemetry:mainfrom
trask:dashboard-large-repo-webhooks-backfill-50
Open

Allow large repo dashboard webhooks#68
trask wants to merge 1 commit into
open-telemetry:mainfrom
trask:dashboard-large-repo-webhooks-backfill-50

Conversation

@trask

@trask trask commented Jul 9, 2026

Copy link
Copy Markdown
Member

I've done several things that should help and allow real-time updates. Especially now that hourly runs only do a portion of the PRs each time, since they were the bigger problem with the hourly token rate limit. Although maybe that's only because collector contrib was only running hourly.

Well, we can always revert.

🤞

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Enables webhook-driven dashboard refreshes for large repositories while reducing hourly backfill API usage.

Changes:

  • Removes the large_repo webhook skip.
  • Reduces hourly backfills from 100 to 50 PRs.
  • Updates documentation accordingly.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
pull-request-dashboard/README.md Removes the obsolete webhook restriction.
.github/workflows/pull-request-dashboard.yml Allows targeted webhook runs for large repositories.
.github/scripts/pull-request-dashboard/RATIONALE.md Documents the reduced backfill limit.
.github/scripts/pull-request-dashboard/dashboard.py Sets the backfill maximum to 50 PRs.

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

@trask trask marked this pull request as ready for review July 9, 2026 23:15
@trask trask requested a review from a team as a code owner July 9, 2026 23:15

@mx-psi mx-psi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🤞

@maryliag maryliag left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤞

@maryliag

Copy link
Copy Markdown
Contributor

When I added the collector contrib, it was making hit the limit in the first 10 minutes, this is why I changed to the hourly only. So let's hope it got better 😄

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.

4 participants