Skip to content

compile-perf: enable nightly schedule at 05:00 UTC#11702

Merged
jvepsalainen-nv merged 4 commits into
masterfrom
dev/jvepsalainen/compile-perf-enable-nightly
Jun 26, 2026
Merged

compile-perf: enable nightly schedule at 05:00 UTC#11702
jvepsalainen-nv merged 4 commits into
masterfrom
dev/jvepsalainen/compile-perf-enable-nightly

Conversation

@jvepsalainen-nv

@jvepsalainen-nv jvepsalainen-nv commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

The compile performance suite has been validated end-to-end on the nvrgfx-perf runner and the release history is seeded in shader-slang/slang-compile-perf. Enable the daily cron to start collecting tip-of-tree performance data.

Suite validated on runner, release history seeded. Enable the daily
cron to start collecting tip-of-tree performance data.
@jvepsalainen-nv jvepsalainen-nv requested a review from a team as a code owner June 23, 2026 13:19
@jvepsalainen-nv jvepsalainen-nv added the pr: non-breaking PRs without breaking changes label Jun 23, 2026
@jvepsalainen-nv jvepsalainen-nv requested review from bmillsNV and removed request for a team June 23, 2026 13:19
@coderabbitai

coderabbitai Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The nightly compile-perf workflow's on.schedule trigger is re-enabled with a daily cron expression (0 5 * * *), restoring automated runs at 05:00 UTC. The previously disabled or commented-out schedule configuration is replaced by this active cron block.

Changes

Nightly Compile-Perf Schedule Restore

Layer / File(s) Summary
Restore daily cron trigger
.github/workflows/compile-perf-nightly.yml
on.schedule is added back with cron 0 5 * * *, replacing the disabled schedule so the workflow triggers automatically at 05:00 UTC daily rather than only on manual dispatch.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: enabling a nightly schedule for the compile-perf workflow at 05:00 UTC, which matches the workflow update from manual-only to automated daily scheduling.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The description clearly explains the purpose of enabling the daily cron schedule for the compile-perf suite, matching the changeset.

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


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@jvepsalainen-nv jvepsalainen-nv self-assigned this Jun 23, 2026

@jkiviluoto-nv jkiviluoto-nv 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.

LGTM

@jvepsalainen-nv jvepsalainen-nv added this pull request to the merge queue Jun 24, 2026
@jkiviluoto-nv jkiviluoto-nv removed this pull request from the merge queue due to a manual request Jun 24, 2026
@jkiviluoto-nv jkiviluoto-nv enabled auto-merge June 24, 2026 09:52
@jkiviluoto-nv jkiviluoto-nv added this pull request to the merge queue Jun 24, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Jun 25, 2026
@jvepsalainen-nv jvepsalainen-nv added this pull request to the merge queue Jun 25, 2026
Merged via the queue into master with commit b1bdd88 Jun 26, 2026
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: non-breaking PRs without breaking changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants