Disable the gitaly OperationService apdex completely

Igorrequested to merge
disable-gitaly-opservice-apdex into master

The OperationService apdex is disabled primarily to deal with very slow operations producing a lot of alerts.

Excluding those RPCs is not a very effective strategy, as OperationService is very low-traffic to begin with, and with every excluded RPC, we increase the sensitivity of the alerts. Thus, we opt to remove this SLO completely for the time being.

Former attempts to exclude RPCs:

Increased alert volume:

Infradev issues addressing sources of latency:

The consequence of this change is that we may not get alerted for slow RPCs in the OperationService. Most other unary gitaly RPCs are still in scope for the goserver apdex.

We can re-evaluate this once some progress has been made on gitlab-com/gl-infra/production#5311 (closed).

cc @jarv @andrewn

Edited by Igor

Merge request reports