Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add one-off executor to search list for cancel job #178

Merged
merged 1 commit into from Sep 11, 2014

Conversation

Projects
None yet
2 participants
@rinrinne
Copy link
Member

commented Sep 11, 2014

By "Build Current Pathcset Only" feature, running and queued jobs with
the same change are canceled. These are found in queue items and
executors.

But MatrixJob uses one-off executor. It is not included in executors. So
it cannot be canceled.

This patch adds one-off executors to executor's list.

Fix for JENKINS-24295

Task-Url: https://issues.jenkins-ci.org/browse/JENKINS-24295

Add one-off executor to search list for cancel job
By "Build Current Pathcset Only" feature, running and queued jobs with
the same change are canceled. These are found in queue items and
executors.

But MatrixJob uses one-off executor. It is not included in executors. So
it cannot be canceled.

This patch adds one-off executors to executor's list.

Fix for JENKINS-24295

Task-Url: https://issues.jenkins-ci.org/browse/JENKINS-24295

rsandell added a commit that referenced this pull request Sep 11, 2014

Merge pull request #178 from rinrinne/cancel-jobs
Add one-off executor to search list for cancel job

@rsandell rsandell merged commit 2404c79 into jenkinsci:master Sep 11, 2014

1 check passed

default This pull request looks good
Details

@rinrinne rinrinne deleted the rinrinne:cancel-jobs branch Nov 4, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.