Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-85145

update redis session handler to allow clustermode to be explicitly set

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Minor Minor
    • None
    • 4.5.3
    • Caching
    • None
    • MOODLE_405_STABLE

      The session handler sets the clustermode based on whether there are multiple hosts provided:

              if (isset($CFG->session_redis_host))

      Unknown macro: {             // If there is only one host, use the single Redis connection.             // If there are multiple hosts (separated by a comma), use the Redis cluster connection.             $this->host = array_filter(array_map('trim', explode(',', $CFG->session_redis_host)));             $this->clustermode = count($this->host) > 1;         }

      https://github.com/moodle/moodle/blob/main/lib/classes/session/redis.php

       

      Some clusters provide a single IP and then use MOVED to direct clients to the correct endpoint.

      Please can we allow for clustermode to be explicitly set, even if there only is one IP.

      I believe this is also the ask from this comment:

      [MDL-59196] Add support for Redis Cluster - Moodle Tracker

       

      From a cursory glance I'm thinking I can specify the same host twice.. if that works I will report back.

            Unassigned Unassigned
            kewalaka Stu
            Stu
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.