Skip to content

ci: increase comment-pr-artifacts polling timeout#466

Merged
shantur merged 1 commit into
NeuralNomadsAI:devfrom
omercnet:fix/comment-workflow-timeout
May 16, 2026
Merged

ci: increase comment-pr-artifacts polling timeout#466
shantur merged 1 commit into
NeuralNomadsAI:devfrom
omercnet:fix/comment-workflow-timeout

Conversation

@omercnet
Copy link
Copy Markdown
Contributor

Problem

The Comment PR Artifacts workflow consistently times out before the PR Build Validation run can complete. The build pipeline typically takes 14–25 minutes (especially the Tauri macOS build), but the comment workflow only polled for ~12 minutes (30 attempts × 10-second intervals plus API overhead).

This has been causing the comment check to fail on every PR — see PR #463 where it failed 3 consecutive times.

Fix

  • Increase polling attempts from 30 → 90
  • Increase sleep interval from 10s → 20s
  • Effective maximum wait: ~30 minutes of sleep + API overhead ≈ 45+ minutes total

This gives ample headroom for the full build matrix to complete, including slower runners like build-tauri-macos.

Why this needs to merge first

The comment-pr-artifacts.yml workflow uses pull_request_target, which means it runs from the base branch (dev), not the PR branch. Changes to this file in PR #463 cannot take effect until this fix lands on dev. Once merged, the comment workflow will stop timing out on PR #463 and all future PRs.


This PR was created by an AI agent (OpenHands) on behalf of the user to unblock PR #463.

The Comment PR Artifacts workflow was timing out before the PR Build
Validation run could complete. The build typically takes ~14 minutes,
but the comment workflow only polled for ~12 minutes (30 attempts with
10-second intervals plus API overhead).

Increase the polling budget to 90 attempts at 20-second intervals
(~30 minutes effective wait) so the comment job reliably waits for
the full build pipeline to finish.

Co-authored-by: openhands <openhands@all-hands.dev>
@shantur shantur merged commit 00bfe52 into NeuralNomadsAI:dev May 16, 2026
10 of 11 checks passed
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.

2 participants