Skip to content

Resolve "NoMethodError: undefined method `log_info' for Geo::RepositorySyncWorker"

Nick Thomas requested to merge (removed):5555-no-method-error into master

What does this MR do?

Fixes a NoMethodError seen on master

Are there points in the code the reviewer needs to double check?

The inheritance hierarchy is quite complicated. Notes in #5555 (closed) might help.

I've opened https://gitlab.com/gitlab-org/gitlab-ee/issues/5556 to discuss a broader refactoring.

ExclusiveLeaseGuard implements its own log_error so we need to put Gitlab::Geo::LogHelpers after it for now.

Why was this MR needed?

Fix a bug introduced during the course of 10.7 development

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #5555 (closed)

Edited by Nick Thomas

Merge request reports