Skip to content

Update condition logic in competitive-test.yml#1158

Open
wolfgang-desalvador wants to merge 2 commits into
mainfrom
wdesalvador/review-cancellation-condition
Open

Update condition logic in competitive-test.yml#1158
wolfgang-desalvador wants to merge 2 commits into
mainfrom
wdesalvador/review-cancellation-condition

Conversation

@wolfgang-desalvador
Copy link
Copy Markdown
Collaborator

This pull request updates the jobs/competitive-test.yml workflow to improve job cancellation handling and make job conditions more robust. The main changes introduce a configurable cancellation timeout and ensure jobs do not run if already canceled.

Job execution and cancellation improvements:

  • Added a new cancellation_timeout_in_minutes parameter to allow configuring how long to wait before force-canceling a job, and set its default value.
  • Updated the job definition to use the new cancelTimeoutInMinutes parameter and changed the job's condition to ensure it does not run if already canceled.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the jobs/competitive-test.yml Azure Pipelines job template to improve cancellation behavior and prevent the job from starting when the run is already canceled.

Changes:

  • Added a cancellation_timeout_in_minutes parameter and wired it to the job’s cancelTimeoutInMinutes.
  • Updated the job condition to include not(canceled()) in addition to existing manual/scheduled-on-main gating.

Comment thread jobs/competitive-test.yml Outdated
Comment thread jobs/competitive-test.yml
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