Instrument all service classes

This will help us see where (mostly) Sidekiq code is spending time.

@yorickpeterse Thanks! I don't see a changelog entry, nor tests for that (but regarding tests, I suppose if the test suite still passes, we're good)?

@rymai There's no way of testing if things work other than booting up with metrics enabled (which in turn will mess up some other tests). I'll add the changelog entry.

There's no way of testing if things work other than booting up with metrics enabled (which in turn will mess up some other tests).

Yeah that's what I thought.

I'll add the changelog entry.

Thanks!

Added 1 commit:

• 435892ed - Changelog entry for service class instrumentation

@rymai I added the changelog entry.

mentioned in commit 0b8852d3

Status changed to merged

mentioned in merge request gitlab-com/www-gitlab-com!1886 (merged)