Skip to content

Update recommended redis HA gitlab.rb

Ian Baum requested to merge update-redis-ha-docs into master

What does this MR do?

Updates HA documentation to disable gitlab-rails recipe entirely on redis nodes.

reconfigure was attempting to run gitlab-rake cache:clear. This was failing since the recommended configuration sets a password for the redis server, but does not tell the gitlab-rails cookbook to setup resque.yml with the password.

It does not appear that a redis node needs a gitlab-rails configuration, so I'm opting to remove it entirely, rather than update resque.yml

Related issues

Closes omnibus-gitlab#4706 (closed)

Author's checklist

Review checklist

All reviewers can help ensure accuracy, clarity, completeness, and adherence to the Documentation Guidelines and Style Guide.

1. Primary Reviewer

  • Review by a code reviewer or other selected colleague to confirm accuracy, clarity, and completeness. This can be skipped for minor fixes without substantive content changes.

2. Technical Writer

  • Optional: Technical writer review. If not requested for this MR, must be scheduled post-merge. To request for this MR, assign the writer listed for the applicable DevOps stage.

3. Maintainer

  1. Review by assigned maintainer, who can always request/require the above reviews. Maintainer's review can occur before or after a technical writer review.
  2. Ensure a release milestone is set.
  3. If there has not been a technical writer review, create an issue for one using the Doc Review template.
Edited by DJ Mountney

Merge request reports