31e3f47280
This reduces some polling overhead and reduces the need to aggregate empty metrics from the Sidekiq worker.
212 lines
7.2 KiB
Ruby
212 lines
7.2 KiB
Ruby
# Autoload all classes that we want to instrument, and instrument the methods we
|
|
# need. This takes the Gitlab::Metrics::Instrumentation module as an argument so
|
|
# that we can stub it for testing, as it is only called when metrics are
|
|
# enabled.
|
|
#
|
|
# rubocop:disable Metrics/AbcSize
|
|
def instrument_classes(instrumentation)
|
|
instrumentation.instrument_instance_methods(Gitlab::Shell)
|
|
|
|
instrumentation.instrument_methods(Gitlab::Git)
|
|
|
|
Gitlab::Git.constants.each do |name|
|
|
const = Gitlab::Git.const_get(name)
|
|
|
|
next unless const.is_a?(Module)
|
|
|
|
instrumentation.instrument_methods(const)
|
|
instrumentation.instrument_instance_methods(const)
|
|
end
|
|
|
|
# Path to search => prefix to strip from constant
|
|
paths_to_instrument = {
|
|
%w(app finders) => %w(app finders),
|
|
%w(app mailers emails) => %w(app mailers),
|
|
# Don't instrument `app/services/concerns`
|
|
# It contains modules that are included in the services.
|
|
# The services themselves are instrumented so the methods from the modules
|
|
# are included.
|
|
%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)|
|
|
prefix = Rails.root.join(*prefix)
|
|
|
|
Dir[Rails.root.join(*path + ['*.rb'])].each do |file_path|
|
|
path = Pathname.new(file_path).relative_path_from(prefix)
|
|
const = path.to_s.sub('.rb', '').camelize.constantize
|
|
|
|
instrumentation.instrument_methods(const)
|
|
instrumentation.instrument_instance_methods(const)
|
|
end
|
|
end
|
|
|
|
instrumentation.instrument_methods(Premailer::Adapter::Nokogiri)
|
|
instrumentation.instrument_instance_methods(Premailer::Adapter::Nokogiri)
|
|
|
|
[
|
|
:Blame, :Branch, :BranchCollection, :Blob, :Commit, :Diff, :Repository,
|
|
:Tag, :TagCollection, :Tree
|
|
].each do |name|
|
|
const = Rugged.const_get(name)
|
|
|
|
instrumentation.instrument_methods(const)
|
|
instrumentation.instrument_instance_methods(const)
|
|
end
|
|
|
|
# Instruments all Banzai filters and reference parsers
|
|
{
|
|
Filter: Rails.root.join('lib', 'banzai', 'filter', '*.rb'),
|
|
ReferenceParser: Rails.root.join('lib', 'banzai', 'reference_parser', '*.rb')
|
|
}.each do |const_name, path|
|
|
Dir[path].each do |file|
|
|
klass = File.basename(file, File.extname(file)).camelize
|
|
const = Banzai.const_get(const_name).const_get(klass)
|
|
|
|
instrumentation.instrument_methods(const)
|
|
instrumentation.instrument_instance_methods(const)
|
|
end
|
|
end
|
|
|
|
instrumentation.instrument_methods(Banzai::Renderer)
|
|
instrumentation.instrument_methods(Banzai::Querying)
|
|
|
|
instrumentation.instrument_instance_methods(Banzai::ObjectRenderer)
|
|
instrumentation.instrument_instance_methods(Banzai::Redactor)
|
|
instrumentation.instrument_methods(Banzai::NoteRenderer)
|
|
|
|
[Issuable, Mentionable, Participable].each do |klass|
|
|
instrumentation.instrument_instance_methods(klass)
|
|
instrumentation.instrument_instance_methods(klass::ClassMethods)
|
|
end
|
|
|
|
instrumentation.instrument_methods(Gitlab::ReferenceExtractor)
|
|
instrumentation.instrument_instance_methods(Gitlab::ReferenceExtractor)
|
|
|
|
# Instrument the classes used for checking if somebody has push access.
|
|
instrumentation.instrument_instance_methods(Gitlab::GitAccess)
|
|
instrumentation.instrument_instance_methods(Gitlab::GitAccessWiki)
|
|
|
|
instrumentation.instrument_instance_methods(API::Helpers)
|
|
|
|
instrumentation.instrument_instance_methods(RepositoryCheck::SingleRepositoryWorker)
|
|
|
|
instrumentation.instrument_instance_methods(Rouge::Plugins::Redcarpet)
|
|
instrumentation.instrument_instance_methods(Rouge::Formatters::HTMLGitlab)
|
|
|
|
[:XML, :HTML].each do |namespace|
|
|
namespace_mod = Nokogiri.const_get(namespace)
|
|
|
|
instrumentation.instrument_methods(namespace_mod)
|
|
instrumentation.instrument_methods(namespace_mod::Document)
|
|
end
|
|
|
|
instrumentation.instrument_methods(Rinku)
|
|
instrumentation.instrument_instance_methods(Repository)
|
|
|
|
instrumentation.instrument_methods(Gitlab::Highlight)
|
|
instrumentation.instrument_instance_methods(Gitlab::Highlight)
|
|
|
|
# This is a Rails scope so we have to instrument it manually.
|
|
instrumentation.instrument_method(Project, :visible_to_user)
|
|
|
|
# Needed for https://gitlab.com/gitlab-org/gitlab-ce/issues/30224#note_32306159
|
|
instrumentation.instrument_instance_method(MergeRequestDiff, :load_commits)
|
|
|
|
# Needed for https://gitlab.com/gitlab-org/gitlab-ce/issues/36061
|
|
instrumentation.instrument_instance_method(MergeRequest, :ensure_ref_fetched)
|
|
instrumentation.instrument_instance_method(MergeRequest, :fetch_ref)
|
|
end
|
|
# rubocop:enable Metrics/AbcSize
|
|
|
|
unless Sidekiq.server?
|
|
Gitlab::Metrics::UnicornSampler.initialize_instance(Settings.monitoring.unicorn_sampler_interval).start
|
|
end
|
|
|
|
Gitlab::Application.configure do |config|
|
|
# 0 should be Sentry to catch errors in this middleware
|
|
config.middleware.insert(1, Gitlab::Metrics::RequestsRackMiddleware)
|
|
end
|
|
|
|
if Gitlab::Metrics.enabled?
|
|
require 'pathname'
|
|
require 'influxdb'
|
|
require 'connection_pool'
|
|
require 'method_source'
|
|
|
|
# These are manually require'd so the classes are registered properly with
|
|
# ActiveSupport.
|
|
require_dependency 'gitlab/metrics/subscribers/action_view'
|
|
require_dependency 'gitlab/metrics/subscribers/active_record'
|
|
require_dependency 'gitlab/metrics/subscribers/rails_cache'
|
|
|
|
Gitlab::Application.configure do |config|
|
|
config.middleware.use(Gitlab::Metrics::RackMiddleware)
|
|
config.middleware.use(Gitlab::Middleware::RailsQueueDuration)
|
|
end
|
|
|
|
Sidekiq.configure_server do |config|
|
|
config.server_middleware do |chain|
|
|
chain.add Gitlab::Metrics::SidekiqMiddleware
|
|
end
|
|
end
|
|
|
|
# This instruments all methods residing in app/models that (appear to) use any
|
|
# of the ActiveRecord methods. This has to take place _after_ initializing as
|
|
# for some unknown reason calling eager_load! earlier breaks Devise.
|
|
Gitlab::Application.config.after_initialize do
|
|
Rails.application.eager_load!
|
|
|
|
models = Rails.root.join('app', 'models').to_s
|
|
|
|
regex = Regexp.union(
|
|
ActiveRecord::Querying.public_instance_methods(false).map(&:to_s)
|
|
)
|
|
|
|
Gitlab::Metrics::Instrumentation
|
|
.instrument_class_hierarchy(ActiveRecord::Base) do |klass, method|
|
|
# Instrumenting the ApplicationSetting class can lead to an infinite
|
|
# loop. Since the data is cached any way we don't really need to
|
|
# instrument it.
|
|
if klass == ApplicationSetting
|
|
false
|
|
else
|
|
loc = method.source_location
|
|
|
|
loc && loc[0].start_with?(models) && method.source =~ regex
|
|
end
|
|
end
|
|
|
|
# Ability is in app/models, is not an ActiveRecord model, but should still
|
|
# be instrumented.
|
|
Gitlab::Metrics::Instrumentation.instrument_methods(Ability)
|
|
end
|
|
|
|
Gitlab::Metrics::Instrumentation.configure do |config|
|
|
instrument_classes(config)
|
|
end
|
|
|
|
GC::Profiler.enable
|
|
|
|
Gitlab::Metrics::InfluxSampler.initialize_instance.start
|
|
|
|
module TrackNewRedisConnections
|
|
def connect(*args)
|
|
val = super
|
|
|
|
if current_transaction = Gitlab::Metrics::Transaction.current
|
|
current_transaction.increment(:new_redis_connections, 1)
|
|
end
|
|
|
|
val
|
|
end
|
|
end
|
|
|
|
class ::Redis::Client
|
|
prepend TrackNewRedisConnections
|
|
end
|
|
end
|