• Evergreen - Milestone 1

      Now JEP 308 has been merged as draft, we should double check the code does comply to the spec.

      At least:

          [JENKINS-51391] Adjust error telemetry API to JEP-308

          Baptiste Mathus created issue -
          Baptiste Mathus made changes -
          Summary Original: Adjust error telemetry API New: Adjust error telemetry API to JEP-XXX
          Baptiste Mathus made changes -
          Link New: This issue is blocked by JENKINS-51140 [ JENKINS-51140 ]
          Baptiste Mathus made changes -
          Description Original: (_Delaying until JEP PR is merged and a number is assigned for reference easiness._)

          At least:

          * Move the {{/telemetry/error}} API behind authentication
          * Test cover the expected http status codes described in the specification
          *
          New: (_Delaying until JEP PR is merged and a number is assigned for reference easiness and avoid implementation's back and forths._)

          At least:

          * Move the {{/telemetry/error}} API behind authentication
          * Test cover the expected http status codes described in the specification
          *
          Baptiste Mathus made changes -
          Description Original: (_Delaying until JEP PR is merged and a number is assigned for reference easiness and avoid implementation's back and forths._)

          At least:

          * Move the {{/telemetry/error}} API behind authentication
          * Test cover the expected http status codes described in the specification
          *
          New: (_Delaying until JEP PR is merged and a number is assigned for reference easiness and avoid implementation's back and forths._)

          At least:

          * Move the {{/telemetry/error}} API behind authentication
          * Test cover the expected http status codes described in the specification
          * Retrieve and log instance identity along with the log for correlation
          Baptiste Mathus made changes -
          Summary Original: Adjust error telemetry API to JEP-XXX New: Adjust error telemetry API to JEP-308
          Baptiste Mathus made changes -
          Rank New: Ranked lower
          Baptiste Mathus made changes -
          Assignee Original: Baptiste Mathus [ batmat ]
          Baptiste Mathus made changes -
          Rank New: Ranked higher
          Baptiste Mathus made changes -
          Assignee New: Mandie Smith [ asmith_cb ]
          Baptiste Mathus made changes -
          Description Original: (_Delaying until JEP PR is merged and a number is assigned for reference easiness and avoid implementation's back and forths._)

          At least:

          * Move the {{/telemetry/error}} API behind authentication
          * Test cover the expected http status codes described in the specification
          * Retrieve and log instance identity along with the log for correlation
          New: Now [JEP 308|https://github.com/jenkinsci/jep/tree/master/jep/308] has been merged as draft, we should double check the code does comply to the spec.

          At least:

          * Test cover the expected http status codes described in the specification
          ** Example: send data too big and {{expect}} the HTTP 413 status code (see https://github.com/jenkinsci/jep/tree/master/jep/308#maximum-message-size)
          * Retrieve and log instance identity along with the log for correlation

            asmith_cb Mandie Smith
            batmat Baptiste Mathus
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: