Skip to content

fix: increase copilot-pr-merged-report timeout from 10 to 15 minutes#30439

Closed
Copilot wants to merge 3 commits into
mainfrom
copilot/fix-daily-copilot-pr-merged-report
Closed

fix: increase copilot-pr-merged-report timeout from 10 to 15 minutes#30439
Copilot wants to merge 3 commits into
mainfrom
copilot/fix-daily-copilot-pr-merged-report

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 5, 2026

The "Execute GitHub Copilot CLI" step was hitting the 10-minute timeout-minutes limit at exactly the wrong moment — the Copilot agent itself completed successfully (processed 50 PRs, created discussion), but awf container startup (~13s) + agent execution (~9m 44s) + container cleanup pushed the total step wall time just past 10 minutes, causing GitHub Actions to kill the step and mark the job as failure.

Evidence from the failed run:

Changes

  • copilot-pr-merged-report.md: timeout-minutes 10 → 15; update success criterion text to match
  • copilot-pr-merged-report.lock.yml: recompiled — step-level timeout-minutes updated to 15

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/github/gh-aw/actions/jobs/74449615662
    • Triggering command: /usr/bin/curl curl -s -H Authorization: ****** REDACTED (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI linked an issue May 5, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix daily Copilot PR merged report failure fix: increase copilot-pr-merged-report timeout from 10 to 15 minutes May 5, 2026
Copilot AI requested a review from pelikhan May 5, 2026 20:39
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

Hey @copilot-swe-agent 👋 — great detective work tracking down the timeout race condition in the copilot-pr-merged-report workflow! The evidence trail (exact 600s wall-clock match, clean agent exit code, confirmed discussion creation) makes a compelling case for the 10→15 minute bump.

One thing worth addressing before merge:

  • No tests present — the lock file regeneration touches ~99 workflows across the repo. Even a lightweight smoke-test or validation step (e.g. a CI job that compiles/lints the changed .md workflow source and verifies the emitted .lock.yml matches) would give reviewers confidence the bulk regeneration is correct and no lock file drifted unexpectedly.

If you would like a hand, you can assign this prompt to your coding agent:

Add a CI validation step for the workflow lock file compilation.
Specifically:
1. Check whether the repo already has a Makefile target or script (e.g. `make compile` or `scripts/compile-workflows.sh`) that regenerates `.lock.yml` files from `.md` sources.
2. If such a target exists, add or update a GitHub Actions CI workflow (e.g. `.github/workflows/ci.yml`) with a job that runs the compile step and then verifies there are no uncommitted changes to any `.lock.yml` file (using `git diff --exit-code`).
3. If no compile script exists, document in the PR body how the lock files were regenerated so reviewers can reproduce the output.

Generated by Contribution Check · ● 758.5K ·

@pelikhan
Copy link
Copy Markdown
Collaborator

pelikhan commented May 5, 2026

@copilot merge main and recompile

@pelikhan pelikhan closed this May 5, 2026
Copilot stopped work on behalf of pelikhan due to an error May 5, 2026 21:44
@github-actions github-actions Bot deleted the copilot/fix-daily-copilot-pr-merged-report branch May 14, 2026 03:01
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.

[aw] Daily Copilot PR Merged Report failed

2 participants