Major Now 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
- is blocked by
-
JENKINS-51140 Design (JEP) what the Error Telemetry API will be
-
- Resolved
-
- links to
[JENKINS-51391] Adjust error telemetry API to JEP-308
| Summary | Original: Adjust error telemetry API | New: Adjust error telemetry API to JEP-XXX |
| Link |
New:
This issue is blocked by |
| 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 * |
| 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 |
| Summary | Original: Adjust error telemetry API to JEP-XXX | New: Adjust error telemetry API to JEP-308 |
| Rank | New: Ranked lower |
| Assignee | Original: Baptiste Mathus [ batmat ] |
| Rank | New: Ranked higher |
| Assignee | New: Mandie Smith [ asmith_cb ] |