Shopify / shipit-engine

Deployment coordination
https://shopify.engineering/introducing-shipit
MIT License
1.41k stars 144 forks source link

Revert "Make RefreshCheckRunsJob and RefreshStatusesJob enqueue one of themselves per commit" #1343

Closed kwboyd-shopify closed 4 months ago

kwboyd-shopify commented 4 months ago

Reverts Shopify/shipit-engine#1341

Enqueuing these jobs separately led to too many GitHub requests being made at once, leading to hitting secondary rate limits frequently. I'll return to this issue when I have more bandwidth to debug, unless someone else has an idea of a quick fix for that issue.

@erik-shopify has an incoming PR to handle retries when we do hit GitHub secondary rate limits as well.