Skip to content

Adjust polling timeout for Kitmaker release with exponential backoff#256

Merged
aristarkhovNV merged 1 commit into
mainfrom
aaristarkhov/kitmaker-poll
Mar 13, 2026
Merged

Adjust polling timeout for Kitmaker release with exponential backoff#256
aristarkhovNV merged 1 commit into
mainfrom
aaristarkhov/kitmaker-poll

Conversation

@aristarkhovNV
Copy link
Copy Markdown
Collaborator

@aristarkhovNV aristarkhovNV commented Mar 12, 2026

Summary by CodeRabbit

  • Chores
    • Enhanced build system efficiency by optimizing status polling behavior with exponential backoff strategy and reduced retry attempts. The updated approach minimizes unnecessary wait times during the build process, reduces overall build duration, and maintains reliability for continuous integration workflows while improving resource utilization across build iterations.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 5bee4cda-e22f-44bb-a41b-6040a4a6baa5

📥 Commits

Reviewing files that changed from the base of the PR and between b05c7ee and 9acfd53.

📒 Files selected for processing (1)
  • .github/workflows/build-ubuntu.yml

📝 Walkthrough

Walkthrough

A GitHub Actions workflow's Kitmaker status polling logic was optimized by reducing maximum poll attempts from 20 to 15 and implementing exponential backoff timing (1.25x multiplier) instead of fixed 30-second intervals, with updated comments reflecting an hourly cap.

Changes

Cohort / File(s) Summary
Workflow Polling Optimization
.github/workflows/build-ubuntu.yml
Reduced maximum poll attempts to 15, replaced fixed 30s wait with exponential backoff (multiplying sleep by 1.25 after each iteration), and updated polling cap comment from ~10 minutes to hourly reference. Changes applied to both initial status check and Kitmaker status monitor loops.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Hop, hop, backoff we go,
From fixed waits to exponential flow,
Fewer attempts, smarter delay,
Workflows run in a quicker way! 🚀

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly describes the main change: adjusting polling timeout and implementing exponential backoff for Kitmaker release status checks.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch aaristarkhov/kitmaker-poll
📝 Coding Plan
  • Generate coding plan for human review comments

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Kitmaker release status polling in the Ubuntu build workflow to allow longer waits using an exponential backoff strategy, reducing the chance of timing out on slower releases.

Changes:

  • Increase the maximum polling attempts (20 → 15) while switching from fixed-interval polling to exponential backoff.
  • Add a 1.25x sleep backoff multiplier starting from 30 seconds between polls.

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

You can also share your feedback on Copilot code review. Take the survey.

@aristarkhovNV aristarkhovNV merged commit 3e467d8 into main Mar 13, 2026
37 checks passed
@aristarkhovNV aristarkhovNV deleted the aaristarkhov/kitmaker-poll branch March 13, 2026 15:30
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