Skip to content

Match health check failure messages between the electors

Sami Hiltunen requested to merge smh-match-health-check-message into master

per_repository elector's health check failure log message differs from what is outputted by the NodeManager. This is failing the QA test pipeline which relies on the log message to identify when Praefect has failed health checks from a secondary node. As a quick fix, let's just match the log message so the pipeline passes.

Failing wait:

  1. https://gitlab.com/gitlab-org/gitlab-foss/-/blob/ede2fbdc8734f095d371614d362b5ae373d6a243/qa/qa/specs/features/api/3_create/gitaly/distributed_reads_spec.rb#L46
  2. https://gitlab.com/gitlab-org/gitlab-foss/-/blob/ede2fbdc8734f095d371614d362b5ae373d6a243/qa/qa/service/praefect_manager.rb#L305

Merge request reports