Skip to content

CI(workflows): Increase job timeouts and improve nightly workflow control#2451

Merged
soumeh01 merged 6 commits into
mainfrom
nightly-workflow-steer
Apr 24, 2026
Merged

CI(workflows): Increase job timeouts and improve nightly workflow control#2451
soumeh01 merged 6 commits into
mainfrom
nightly-workflow-steer

Conversation

@soumeh01

@soumeh01 soumeh01 commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Timeout increase across tool workflows in multiple jobs from 15 to 25 minutes, as GH runner takes more and more time to update their dependencies.
  • Improved workflow steer by running them in parallel
  • Nightly workflow orchestration updates
    • If one workflow fails, the other workflow still runs
    • 0.1% threshold prevents flaky gate failures on non-functional changes
    • One publish results job for nightly

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@github-actions

github-actions Bot commented Apr 22, 2026

Copy link
Copy Markdown

Test Results

   58 files    176 suites   22m 33s ⏱️
  910 tests   893 ✅ 17 💤 0 ❌
2 867 runs  2 799 ✅ 68 💤 0 ❌

Results for commit 2ce0263.

♻️ This comment has been updated with latest results.

@soumeh01 soumeh01 force-pushed the nightly-workflow-steer branch 7 times, most recently from 681cccd to 5241402 Compare April 22, 2026 11:03
@soumeh01 soumeh01 marked this pull request as ready for review April 22, 2026 11:16
@soumeh01 soumeh01 force-pushed the nightly-workflow-steer branch from 5241402 to 9b57878 Compare April 22, 2026 11:32
@soumeh01 soumeh01 force-pushed the nightly-workflow-steer branch from 9b57878 to 9050197 Compare April 22, 2026 12:07

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

This PR updates CI configuration to reduce flaky failures and improve nightly orchestration for the devtools tool workflows by increasing job timeouts, relaxing Codecov gate sensitivity, and centralizing nightly test result publishing.

Changes:

  • Increased multiple workflow job timeouts to 25 minutes and adjusted concurrency group naming per tool workflow.
  • Relaxed Codecov project status sensitivity by setting a 0.1% threshold per tool flag.
  • Updated nightly workflow orchestration to run tool workflows independently and added a single consolidated test-results publishing job.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
codecov.yml Adds per-flag 0.1% thresholds to reduce flaky coverage gate failures.
.github/workflows/unit_test_results.yml Improves job naming to include the source workflow name.
.github/workflows/test_libs.yml Updates concurrency group, increases timeout, and ensures test-results prep runs with always().
.github/workflows/svdconv.yml Updates concurrency group, increases timeouts, and ensures test-results prep runs with always().
.github/workflows/projmgr.yml Updates concurrency group, increases timeouts, and ensures test-results prep runs with always().
.github/workflows/packgen.yml Updates concurrency group, increases timeouts, and ensures test-results prep runs with always().
.github/workflows/packchk.yml Updates concurrency group, increases timeouts, and ensures test-results prep runs with always().
.github/workflows/nightly.yml Adds workflow-level concurrency, removes strict chaining between tool workflows, consolidates coverage merge, and adds a single nightly publish-test-results job.
.github/workflows/buildmgr.yml Updates concurrency group, increases timeouts, and ensures test-results prep runs with always().

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

Comment thread .github/workflows/nightly.yml Outdated
Comment thread .github/workflows/nightly.yml Outdated
Comment thread .github/workflows/nightly.yml Outdated
@soumeh01 soumeh01 requested a review from jkrech April 24, 2026 13:43
@soumeh01 soumeh01 self-assigned this Apr 24, 2026
@soumeh01 soumeh01 requested a review from brondani April 24, 2026 13:48

@brondani brondani left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@soumeh01 soumeh01 merged commit 79b66b5 into main Apr 24, 2026
225 checks passed
@soumeh01 soumeh01 deleted the nightly-workflow-steer branch April 24, 2026 16:38
@codecov

codecov Bot commented Apr 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.93%. Comparing base (3b20142) to head (2ce0263).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2451   +/-   ##
=======================================
  Coverage   64.93%   64.93%           
=======================================
  Files         145      145           
  Lines       26209    26209           
  Branches    15828    15828           
=======================================
  Hits        17019    17019           
- Misses       7036     7042    +6     
+ Partials     2154     2148    -6     
Flag Coverage Δ
buildmgr-cov 79.87% <ø> (ø)
packchk-cov 69.75% <ø> (ø)
packgen-cov 82.02% <ø> (ø)
projmgr-cov 88.21% <ø> (ø)
svdconv-cov 40.28% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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