Skip to content

Commit 89880f9

Browse files
authored
Merge pull request #5509 from taskcluster/matt-boris/provisionFailedCount
feat(logs): make provision failed log searchable
2 parents 6b1ae27 + 18a64c8 commit 89880f9

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
audience: general
2+
level: patch
3+
---
4+
This patch adds a new field to be logged out on a failed provision call. This field will be used to measure the provisioning failed count.

services/worker-manager/src/provisioner.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,12 @@ class Provisioner {
159159
};
160160
await provider.provision({ workerPool, workerInfo });
161161
} catch (err) {
162-
this.monitor.reportError(err, { providerId: workerPool.providerId }); // Just report this and move on
162+
this.monitor.reportError(err,
163+
{
164+
providerId: workerPool.providerId,
165+
type: 'provisioning-failed',
166+
},
167+
); // Just report this and move on
163168
}
164169

165170
await Promise.all(previousProviderIds.map(async pId => {

0 commit comments

Comments
 (0)