Skip to content

Remove no-longer-required metrics in Gitlab::Highlight

Nick Thomas requested to merge 329909-remove-track_highlight_timeouts-ff into master

What does this MR do and why?

In May 2021, we started tracking syntax-highlighting timeouts. This was behind a feature flag, which was rolled out to GitLab.com: #329909 (closed)

We also tracked refusals due to too-large files, which was a similar story: #330374 (closed)

The feature flags were never removed, so this MR does a little bit of cleanup. We don't need the metrics any more, so rather than rolling the feature flags out to self-managed, we remove the (never default_enabled: true) FFs and the code they guard entirely.

MR acceptance checklist

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

Related to #329909 (closed)

Related to #330374 (closed)

Edited by Nick Thomas

Merge request reports