Skip to content

Commit b2c34af

Browse files
authored
Merge pull request #4534 from Microsoft/users/jpricket/m117
Fixing bug where JenkinsQueueJob task waits
2 parents d02f72b + 896c772 commit b2c34af

File tree

3 files changed

+8
-2
lines changed

3 files changed

+8
-2
lines changed

Tasks/JenkinsQueueJob/job.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -167,6 +167,12 @@ export class Job {
167167
}
168168

169169
setStreaming(executableNumber: number): void {
170+
// If we aren't waiting for the job to finish then we should end it now
171+
if (!this.queue.taskOptions.captureConsole) { // transition to Finishing
172+
this.changeState(JobState.Streaming);
173+
this.changeState(JobState.Finishing);
174+
return;
175+
}
170176
if (this.state == JobState.New || this.state == JobState.Locating) {
171177
this.executableNumber = executableNumber;
172178
this.executableUrl = Util.addUrlSegment(this.taskUrl, this.executableNumber.toString());

Tasks/JenkinsQueueJob/task.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
"version": {
1515
"Major": 1,
1616
"Minor": 117,
17-
"Patch": 0
17+
"Patch": 2
1818
},
1919
"groups": [
2020
{

Tasks/JenkinsQueueJob/task.loc.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
"version": {
1515
"Major": 1,
1616
"Minor": 117,
17-
"Patch": 0
17+
"Patch": 2
1818
},
1919
"groups": [
2020
{

0 commit comments

Comments
 (0)