Skip to content

fix: increase timeout-minutes to 40 for daily-performance-summary workflow - #49005

Merged
pelikhan merged 2 commits into
mainfrom
copilot/aw-fix-performance-summary-timeout
Jul 30, 2026
Merged

fix: increase timeout-minutes to 40 for daily-performance-summary workflow#49005
pelikhan merged 2 commits into
mainfrom
copilot/aw-fix-performance-summary-timeout

Conversation

Copilot AI commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

The daily-performance-summary workflow was hitting its 30-minute agent timeout, causing runs to fail before completing the full data gather → analysis → chart generation → discussion creation pipeline.

Changes

  • .github/workflows/daily-performance-summary.md: bumped timeout-minutes from 30 to 40 in frontmatter
  • .github/workflows/daily-performance-summary.lock.yml: recompiled to reflect the updated timeout

…kflow

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix timeout issue in daily project performance summary generator fix: increase timeout-minutes to 40 for daily-performance-summary workflow Jul 29, 2026
Copilot AI requested a review from pelikhan July 29, 2026 22:50
@github-actions

Copy link
Copy Markdown
Contributor

Triage: fix: increase timeout-minutes to 40 for daily-performance-summary

Generated by 🔧 PR Triage Agent · aut00 · 52.7 AIC · ⌖ 9.26 AIC · ⊞ 7.9K ·

@github-actions

Copy link
Copy Markdown
Contributor

Warning

threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding.

Details

The threat detection engine failed to produce results.

Review the workflow run logs for details.

Great work, Copilot SWE Agent! 🚀 This fix is spot-on — the daily-performance-summary workflow was timing out at 30 minutes, and bumping it to 40 gives the full data gather → analysis → chart generation → discussion creation pipeline room to breathe.

The change is focused, minimal, and directly addresses issue #48981. Both the workflow definition and lock file are properly updated.

This PR looks ready for review and merge!

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • patchdiff.githubusercontent.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "patchdiff.githubusercontent.com"

See Network Configuration for more information.

Generated by ✅ Contribution Check · aut00 · 61.8 AIC · ⊞ 8.7K ·

@github-actions

Copy link
Copy Markdown
Contributor

Triage Summary

  • Category: chore
  • Risk: low
  • Priority Score: 24/100 (impact 10 + urgency 8 + quality 6)
  • Recommended Action: defer

Tiny timeout bump for daily-performance-summary workflow. Still in draft; low urgency.

Generated by 🔧 PR Triage Agent · auto · 54.9 AIC · ⌖ 5.64 AIC · ⊞ 7.9K ·

@pelikhan
pelikhan marked this pull request as ready for review July 30, 2026 07:43
Copilot AI review requested due to automatic review settings July 30, 2026 07:43
@pelikhan
pelikhan enabled auto-merge (squash) July 30, 2026 07:43

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

Raises the workflow timeout from 30 to 40 minutes to prevent premature termination, resolving #48981.

Changes:

  • Increased the agent timeout to 40 minutes.
  • Recompiled the generated workflow lock file.
Show a summary per file
File Description
.github/workflows/daily-performance-summary.md Sets the workflow timeout to 40 minutes.
.github/workflows/daily-performance-summary.lock.yml Propagates the timeout into the generated workflow.

Review details

Tip

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

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Medium

@pelikhan
pelikhan merged commit 49cb12d into main Jul 30, 2026
1 check passed
@pelikhan
pelikhan deleted the copilot/aw-fix-performance-summary-timeout branch July 30, 2026 07:46
@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot please run the pr-finisher skill, address unresolved review feedback if any, and rerun checks once the branch is up to date.

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 4.89 AIC · ⌖ 7.33 AIC · ⊞ 8.7K ·
Comment /souschef to run again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[aw] Daily Project Performance Summary Generator (Using MCP Scripts) timed out

4 participants