Skip to content

Load the devise initializer before metrics

Bob Van Landuyt requested to merge reprazent/gitlab-ce:bvl-fix-start-with-metrics-enabled into master

What does this MR do?

Metrics tries to initialize the EventsFinder to instrumnt it. The EventsFinder tries to load the Event model, which in turn loads the User model.

Loading the User model requires Devise to be initialized. So to avoid problems around this, we load devise before loading metrics.

This was surfaced by https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/23746

What are the relevant issue numbers?

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/55513

Does this MR meet the acceptance criteria?

Merge request reports