Skip to content

Add weekly scheduled CI workflow#2757

Open
marcpopMSFT with Copilot wants to merge 7 commits into
mainfrom
copilot/weekly-ci-job-scheduling
Open

Add weekly scheduled CI workflow#2757
marcpopMSFT with Copilot wants to merge 7 commits into
mainfrom
copilot/weekly-ci-job-scheduling

Conversation

Copilot AI commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Pipeline checks weren't running reliably due to infrequent commits. This adds a scheduled workflow to ensure CI runs regularly regardless of commit activity.

Changes

  • Add a weekly Monday 08:00 UTC scheduled trigger to the Azure Pipelines definition (1es-azure-pipeline.yml) for the main branch.
  • Ensure the scheduled run executes regardless of recent changes (always: true).

Copilot AI and others added 2 commits July 13, 2026 22:41
Co-authored-by: marcpopMSFT <12663534+marcpopMSFT@users.noreply.github.com>
Co-authored-by: marcpopMSFT <12663534+marcpopMSFT@users.noreply.github.com>
Co-authored-by: marcpopMSFT <12663534+marcpopMSFT@users.noreply.github.com>
Co-authored-by: marcpopMSFT <12663534+marcpopMSFT@users.noreply.github.com>
@marcpopMSFT
marcpopMSFT marked this pull request as ready for review July 13, 2026 23:21
Copilot AI review requested due to automatic review settings July 13, 2026 23:21

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 aims to ensure CI runs regularly even when commits are infrequent by introducing a weekly scheduled run. In the current diff, this is implemented as an Azure Pipelines scheduled trigger (not as a new GitHub Actions workflow as stated in the PR description).

Changes:

  • Add a weekly Monday 08:00 UTC scheduled trigger to the Azure Pipelines definition (1es-azure-pipeline.yml) for the main branch.
  • Ensure the scheduled run executes regardless of recent changes (always: true).

Comment thread 1es-azure-pipeline.yml
…hedule change

Co-authored-by: marcpopMSFT <12663534+marcpopMSFT@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot July 14, 2026 22:56
Co-authored-by: marcpopMSFT <12663534+marcpopMSFT@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot July 14, 2026 22:57

@nagilson nagilson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is the code QL flagger upset because of the PR builds, the CI builds, or both? This only builds on the internal CI build; the pr yml is 1pr-azure-pipeline.

… runs regularly on both pipelines

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 21, 2026 20:50
@nagilson

Copy link
Copy Markdown
Member

Codeql.Enabled: true is set on both pipelines: the internal CI build (1es-azure-pipeline.yml, def 656) and the public PR build (1pr-azure-pipeline.yml, def 60), so CodeQL runs in both. To make sure the scan runs on a regular cadence regardless of commit/PR activity, I've now added the same weekly Monday 08:00 UTC schedule to 1pr-azure-pipeline.yml as well, so it fires on both the CI and PR yamls.

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread 1pr-azure-pipeline.yml
@nagilson
nagilson enabled auto-merge July 21, 2026 22:00
@nagilson
nagilson requested a review from a team July 21, 2026 22:00
@marcpopMSFT

Copy link
Copy Markdown
Member

Codeql.Enabled: true is set on both pipelines: the internal CI build (1es-azure-pipeline.yml, def 656) and the public PR build (1pr-azure-pipeline.yml, def 60), so CodeQL runs in both. To make sure the scan runs on a regular cadence regardless of commit/PR activity, I've now added the same weekly Monday 08:00 UTC schedule to 1pr-azure-pipeline.yml as well, so it fires on both the CI and PR yamls.

@nagilson codeql is only required to run once a month on CI. I don't think it hurts us to include it in the PR builds as well but it's not a requirement fyi.

Also, we now need a third person to review this.

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.

4 participants