Allow redis function without needing sentinel

Bug #1876185 reported by Robert Lyon
26
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Mahara
Wishlist
Anupama Dharmajan

Bug Description

When redis was set up in mahara it was envisioned to be used with redis sentinel in a cluster situation

But we should also allow redis to be usable without sentinel

Looking at the code we probably need to adjust the get_redis_master() and get_redis_servers() functions in htdocs/auth/lib.php

Robert Lyon (robertl-9)
Changed in mahara:
milestone: none → 20.10.0
Changed in mahara:
assignee: nobody → Anupama Dharmajan (anupamadharmajan)
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/11074

Changed in mahara:
status: Confirmed → In Progress
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/11074
Committed: https://git.mahara.org/mahara/mahara/commit/6c60ace121d9a7674b07233b8d9025a2c549ea9c
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 6c60ace121d9a7674b07233b8d9025a2c549ea9c
Author: anupamadharmajan <email address hidden>
Date: Mon Jul 20 13:03:18 2020 +1000

Bug 1876185: Allow redis function without needing sentinel

behatnotneeded

Change-Id: Ie3bdf87e55eba107d57eaade2e4d5c9c2d98ca37

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
tags: added: nominatedfeature
tags: added: newfeature
removed: nominatedfeature
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers