-
Task
-
Resolution: Fixed
-
Minor
-
None
-
-
Evergreen - Milestone 1
User capturing is not enabled, hence makes it harder to automatically correlate messages from the same user in Sentry UI
When working on JENKINS-53251, I see the "User" field on the top right always at 0:
Expected behavior
Reading https://docs.sentry.io/learn/context/#capturing-the-user, I think that we only have to change our code in two places in sentry.js.
Instead of
Raven.captureMessage(data.log.message, { level: data.log.level.toLowerCase(), logger: data.log.name, extra: { uuid: data.uuid, source: data.log } });
We should use `id` field instead of `uuid`, i.e.:
Raven.captureMessage(data.log.message, { level: data.log.level.toLowerCase(), logger: data.log.name, extra: { id: data.uuid, source: data.log } });
- relates to
-
JENKINS-53251 Review Sentry issues
-
- Resolved
-
- links to
[JENKINS-53286] Enable Sentry user capturing
Epic Link |
New:
|
Attachment | New: user count always 0.png [ 43923 ] |
Description |
Original:
h3. User capturing is not enabled, hence makes it harder to automatically correlate messages from the same user in Sentry UI When working on My understanding is that we should send https://docs.sentry.io/learn/context/#capturing-the-user |
New:
h3. User capturing is not enabled, hence makes it harder to automatically correlate messages from the same user in Sentry UI When working on !user count always 0.png! h3. Expected behavior Reading https://docs.sentry.io/learn/context/#capturing-the-user, I _think_ that we only have to change our code in two places in [sentry.js|https://github.com/jenkins-infra/evergreen/blob/fc4875a6d112f7bca2d1080e301125703de63285/services/src/libs/sentry.js#L47]. Instead of {code:js} Raven.captureMessage(data.log.message, { level: data.log.level.toLowerCase(), logger: data.log.name, extra: { uuid: data.uuid, source: data.log } }); {code} We should use `id` field instead of `uuid`, i.e.: {code:js} Raven.captureMessage(data.log.message, { level: data.log.level.toLowerCase(), logger: data.log.name, extra: { id: data.uuid, source: data.log } }); {code} |
Assignee | Original: R. Tyler Croy [ rtyler ] | New: Baptiste Mathus [ batmat ] |
Status | Original: Open [ 1 ] | New: In Progress [ 3 ] |
Remote Link | New: This issue links to "evergreen PR (Web Link)" [ 21405 ] |
Status | Original: In Progress [ 3 ] | New: In Review [ 10005 ] |
Link |
New:
This issue relates to |
Resolution | New: Fixed [ 1 ] | |
Status | Original: In Review [ 10005 ] | New: Resolved [ 5 ] |