Commit b55936bf authored by Bob Van Landuyt's avatar Bob Van Landuyt

Don't instrument `app/services/concerns`

parent 92f82773
...@@ -20,13 +20,17 @@ def instrument_classes(instrumentation) ...@@ -20,13 +20,17 @@ def instrument_classes(instrumentation)
# Path to search => prefix to strip from constant # Path to search => prefix to strip from constant
paths_to_instrument = { paths_to_instrument = {
%w(app finders) => %w(app finders), %w(app finders) => %w(app finders),
%w(app mailers emails) => %w(app mailers), %w(app mailers emails) => %w(app mailers),
%w(app services **) => %w(app services), # Don't instrument `app/services/concerns`
%w(lib gitlab conflicts) => ['lib'], # It contains modules that are included in the services.
%w(lib gitlab diff) => ['lib'], # The services themselves are instrumented so the methods from the modules
%w(lib gitlab email message) => ['lib'], # are included.
%w(lib gitlab checks) => ['lib'] %w(app services [^concerns]**) => %w(app services),
%w(lib gitlab conflicts) => ['lib'],
%w(lib gitlab diff) => ['lib'],
%w(lib gitlab email message) => ['lib'],
%w(lib gitlab checks) => ['lib']
} }
paths_to_instrument.each do |(path, prefix)| paths_to_instrument.each do |(path, prefix)|
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment