-
Bug
-
Resolution: Fixed
-
Blocker
-
None
-
-
Evergreen - Milestone 2
We are currently sending Java exceptions to the Sentry backend by using raven.captureException instead of raven.captureMessage:
if (data.log.exception) { this.raven.captureException(new Error(data.log.message), errorData); } else { this.raven.captureMessage(data.log.message, errorData); }
This makes Sentry aggregate litterally all logs that have an exceptions associated into a single event. This is because doing the new Error(... we create an exception from the place it is, and it's always the same. So Sentry somewhat rightly sees those as the same.
And unfortunately, given we had marked as ignored the "SmokeTestException" one, everything got ignored the same way until recently as I was looking for that specific SmokeTestException ignored on purpose. And only by chance I finally saw in the Events tab that many unrelated events were merged there... :facepalm:
Example:
The title of the Sentry issue:
and the Events tab:
Expected behavior
Exceptions should be sent like the rest. Using the message as the title, and the exception can stay as is currently.
- links to
[JENKINS-55842] Exceptions should be sent to Sentry as messages
| Attachment | New: screenshot-1.png [ 45828 ] |
| Attachment | New: issue-title.png [ 45829 ] |
| Attachment | New: last events.png [ 45830 ] |
| Description |
Original:
We are currently sending Java exceptions to the Sentry backend by using {{raven.captureException}} instead of {{raven.captureMessage}}: {code:java} if (data.log.exception) { this.raven.captureException(new Error(data.log.message), errorData); } else { this.raven.captureMessage(data.log.message, errorData); } {code} This makes Sentry aggregate litterally all logs that have an exceptions associated into a single event. And unfortunately, given we had marked as ignored the "SmokeTestException" one, everything got ignored the same way until recently as I |
New:
We are currently sending Java exceptions to the Sentry backend by using {{raven.captureException}} instead of {{raven.captureMessage}}: {code:java} if (data.log.exception) { this.raven.captureException(new Error(data.log.message), errorData); } else { this.raven.captureMessage(data.log.message, errorData); } {code} This makes Sentry aggregate litterally all logs that have an exceptions associated into a single event. This is because doing the {{new Error(...}} we create an exception from the place it is, and it's always the same. So Sentry somewhat rightly sees those as the same. And unfortunately, given we had marked as ignored the "SmokeTestException" one, everything got ignored the same way until recently as I was looking for that specific SmokeTestException ignored on purpose. And only by chance I finally saw in the Events tab that many unrelated events were merged there... :facepalm: Example: The title of the Sentry issue: !issue-title.png|thumbnail! !screenshot-1.png|thumbnail! and the Events tab: !last events.png|thumbnail! h3. Expected behavior Exceptions should be sent like the rest. Using the message as the title, and the exception can stay as is currently. |
| Status | Original: Open [ 1 ] | New: In Progress [ 3 ] |
| Status | Original: In Progress [ 3 ] | New: In Review [ 10005 ] |
| Remote Link | New: This issue links to "PR (Web Link)" [ 22326 ] |
| Sprint | New: Evergreen - Milestone 2 [ 516 ] |
| Resolution | New: Fixed [ 1 ] | |
| Status | Original: In Review [ 10005 ] | New: Resolved [ 5 ] |