• Icon: Task Task
    • Resolution: Fixed
    • Icon: Minor Minor
    • evergreen
    • 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
        }
      });
      

          [JENKINS-53286] Enable Sentry user capturing

          Baptiste Mathus created issue -
          Baptiste Mathus made changes -
          Epic Link New: JENKINS-51299 [ 190634 ]
          Baptiste Mathus made changes -
          Attachment New: user count always 0.png [ 43923 ]
          Baptiste Mathus made changes -
          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 JENKINS-53251, I see the "User" field on the top right always at 0.

          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 JENKINS-53251, I see the "User" field on the top right always at 0:
           !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}
          Baptiste Mathus made changes -
          Assignee Original: R. Tyler Croy [ rtyler ] New: Baptiste Mathus [ batmat ]
          Baptiste Mathus made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Baptiste Mathus made changes -
          Remote Link New: This issue links to "evergreen PR (Web Link)" [ 21405 ]
          Baptiste Mathus made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Baptiste Mathus made changes -
          Link New: This issue relates to JENKINS-53251 [ JENKINS-53251 ]
          Baptiste Mathus made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Resolved [ 5 ]

            rtyler R. Tyler Croy
            batmat Baptiste Mathus
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: