Skip to content

Remove Instrumentation implementation [RUN ALL RSPEC] [RUN AS-IF-FOSS]

Aleksei Lipniagov requested to merge 217978-remove-instrumentation-code into master

What does this MR do and why?

We are no longer loading method Instrumentation: !69662 (merged)
It should be safe to remove:

  • The implementation
  • Tests & mentions in tests
  • Dev docs

How to set up and validate locally

There is no ideal way to check because we don't expect this class and its methods to be called anymore.
Starting the app on local ENV + relying on QA/specs would be the best thing to do.

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 #217978 (closed)

Edited by Aleksei Lipniagov

Merge request reports