Skip to content

Explicitly add erubi to Gemfile and unquarantine elasticsearch_reindexing_spec test

Erick Banks requested to merge UNQUARANTINE197207 into master

What does this MR do?

It seems the error "14:failed to connect to all addresses"],"limit_reached" was caused by gitaly failing to have its dependency erubi installed. I've added it to the Gemfile and Gemfile.lock here.

To verify that this fix works I've examined the gitaly "current" logs before the change and after. The logs after show no errors pertaining to the bootstrap process that were present before the change.

closes #197207 (closed) closes #199911 (closed)

current_gitaly_logs_pre_change_

current_gitaly_logs_post_change_

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Erick Banks

Merge request reports