chore: remove schedule triggers from smoke-* workflows#26260
Conversation
Agent-Logs-Url: https://github.com/github/gh-aw/sessions/07a1ed1b-f2b8-442e-a937-20f980142008 Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Removes scheduled execution from the active smoke-* agentic workflows so they no longer run on a cron schedule, and updates the generated .lock.yml artifacts accordingly.
Changes:
- Removed
on.scheduletriggers from 9smoke-*workflow source files (.md). - Regenerated the corresponding 9 compiled workflow lockfiles (
*.lock.yml) to removeschedule/cronblocks and update generated sections.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/smoke-update-cross-repo-pr.md | Removes the schedule: trigger from the workflow source. |
| .github/workflows/smoke-update-cross-repo-pr.lock.yml | Regenerated compiled workflow; removes on.schedule cron stanza. |
| .github/workflows/smoke-test-tools.md | Removes the schedule: trigger from the workflow source. |
| .github/workflows/smoke-test-tools.lock.yml | Regenerated compiled workflow; removes on.schedule cron stanza. |
| .github/workflows/smoke-service-ports.md | Removes the schedule: trigger from the workflow source. |
| .github/workflows/smoke-service-ports.lock.yml | Regenerated compiled workflow; removes on.schedule cron stanza (and other regenerated content). |
| .github/workflows/smoke-multi-pr.md | Removes the schedule: trigger from the workflow source. |
| .github/workflows/smoke-multi-pr.lock.yml | Regenerated compiled workflow; removes on.schedule cron stanza. |
| .github/workflows/smoke-gemini.md | Removes the schedule: trigger from the workflow source. |
| .github/workflows/smoke-gemini.lock.yml | Regenerated compiled workflow; removes on.schedule cron stanza. |
| .github/workflows/smoke-create-cross-repo-pr.md | Removes the schedule: trigger from the workflow source. |
| .github/workflows/smoke-create-cross-repo-pr.lock.yml | Regenerated compiled workflow; removes on.schedule cron stanza. |
| .github/workflows/smoke-copilot.md | Removes the schedule: trigger from the workflow source. |
| .github/workflows/smoke-copilot.lock.yml | Regenerated compiled workflow; removes on.schedule cron stanza. |
| .github/workflows/smoke-codex.md | Removes the schedule: trigger from the workflow source. |
| .github/workflows/smoke-codex.lock.yml | Regenerated compiled workflow; removes on.schedule cron stanza. |
| .github/workflows/smoke-claude.md | Removes the schedule: trigger from the workflow source. |
| .github/workflows/smoke-claude.lock.yml | Regenerated compiled workflow; removes on.schedule cron stanza. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 18/18 changed files
- Comments generated: 1
| permissions: | ||
| contents: read | ||
| issues: read | ||
| pull-requests: read | ||
| concurrency: | ||
| group: "gh-aw-copilot-${{ github.workflow }}" | ||
| env: |
There was a problem hiding this comment.
The lockfile recompilation removed the concurrency block from the agent job (it used to set group: "gh-aw-copilot-${{ github.workflow }}"). This is a behavior change beyond removing schedule triggers; please confirm it’s intentional. If intentional, update the PR description to mention the concurrency change; if not, recompile/pin the generator to preserve the previous job-level concurrency behavior.
Removes
schedule:triggers from all active smoke-* agentic workflow files.Changes
Removed
schedule:lines from 9 workflow files:smoke-claude.md—every 12hsmoke-codex.md—every 12hsmoke-copilot.md—every 12hsmoke-create-cross-repo-pr.md—every 12hsmoke-gemini.md—every 12hsmoke-multi-pr.md—every 24hsmoke-service-ports.md—dailysmoke-test-tools.md—every 12hsmoke-update-cross-repo-pr.md—every 12hCorresponding
.lock.ymlfiles recompiled. All 20 smoke-* workflows compiled successfully.Note:
smoke-project.mdandsmoke-temporary-id.mdalready had their schedule triggers commented out (#schedule:) and were left unchanged.