Skip to content
This repository has been archived by the owner on Apr 13, 2019. It is now read-only.

Commit

Permalink
Merge pull request #20 from gregarndt/dont_report_deadline_exceeded_runs
Browse files Browse the repository at this point in the history
Bug 1148965 - Check to see if run was created because of deadline-exceeded
  • Loading branch information
gregarndt committed Aug 5, 2015
2 parents a4c326a + d05f2e9 commit 5b23872
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions src/treeherder/job_handler.js
Expand Up @@ -362,6 +362,10 @@ class Handler {
});
}

shouldReportExceptionRun(run) {
return run.reasonCreated !== 'exception';
}

async handleTaskRunning(project, revisionHash, task, payload) {
let taskId = payload.status.taskId;
let run = payload.status.runs[payload.runId];
Expand All @@ -381,6 +385,15 @@ class Handler {
async handleTaskException(project, revisionHash, task, payload) {
let taskId = payload.status.taskId;
let run = payload.status.runs[payload.runId];

if (!this.shouldReportExceptionRun(run)) {
debug('ignoring task exception for task %s. Reason Resolved: %s',
taskId,
run.reasonResolved
);
return
}

await this.addPush({
revision_hash: revisionHash,
project,
Expand Down

0 comments on commit 5b23872

Please sign in to comment.