Skip to content

Set EpicNotesController#index urgency to low

John Hope requested to merge set_notes_duration_low into master

What does this MR do and why?

This MR reduces the EpicNotesController#index action urgency to low (5s).

There are a few reasons to do this:

  1. The endpoint returns a notes array of arbitrary size. The duration is correlated to the number of notes, and the complexity of notes, being returned.
  2. The endpoint dominates our error budget spend, when we should be (and are) focused on areas of performance for specific, large customers.
  3. default urgency is inconsistent with other notes endpoints, for example Issue notes (groupproject management).
  4. Direct responsibility for the primary component of slowness (Markdown) lies with another group.

Lowering the urgency does not mean we're not thinking about how to improve this endpoint's performance. We simply have other endpoints that are more urgent.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports