Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-55842

Exceptions should be sent to Sentry as messages

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Blocker
    • Resolution: Fixed
    • Component/s: evergreen
    • Labels:
      None
    • Sprint:
      Evergreen - Milestone 2
    • Similar Issues:

      Description

      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.

        Attachments

        1. issue-title.png
          issue-title.png
          18 kB
        2. last events.png
          last events.png
          116 kB
        3. screenshot-1.png
          screenshot-1.png
          166 kB

          Issue Links

            Activity

            There are no comments yet on this issue.

              People

              • Assignee:
                batmat Baptiste Mathus
                Reporter:
                batmat Baptiste Mathus
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: