summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGabriel Mazetto <gabriel@gitlab.com>2016-07-12 17:10:03 +0200
committerGabriel Mazetto <gabriel@gitlab.com>2016-08-04 18:55:37 +0200
commit3a93bae25f03a2992401e1de5bfbf52c3921b1a4 (patch)
tree08c2e0190407c708994b56e800425ee6b5006c61
parent67ae8adc72c1b59c440e0bfbde31df4e0b9920ec (diff)
downloadgitlab-ce-3a93bae25f03a2992401e1de5bfbf52c3921b1a4.tar.gz
Few minor fixes to Redis params order and commented out sentinel config
in resque.yml.example Codestyle changes
-rw-r--r--config/resque.yml.example32
-rw-r--r--lib/gitlab/mail_room.rb1
-rw-r--r--lib/gitlab/redis.rb13
3 files changed, 23 insertions, 23 deletions
diff --git a/config/resque.yml.example b/config/resque.yml.example
index 753c3308aa5..20a1c07690f 100644
--- a/config/resque.yml.example
+++ b/config/resque.yml.example
@@ -3,13 +3,13 @@
#
development:
url: redis://localhost:6379
- sentinels:
- -
- host: localhost
- port: 26380 # point to sentinel, not to redis port
- -
- host: slave2
- port: 26381 # point to sentinel, not to redis port
+ # sentinels:
+ # -
+ # host: localhost
+ # port: 26380 # point to sentinel, not to redis port
+ # -
+ # host: slave2
+ # port: 26381 # point to sentinel, not to redis port
test:
url: redis://localhost:6379
production:
@@ -18,17 +18,17 @@ production:
##
# Redis + Sentinel (for HA)
#
- # Please read instructions carefully before using it as you may loose data:
+ # Please read instructions carefully before using it as you may lose data:
# http://redis.io/topics/sentinel
#
# You must specify a list of a few sentinels that will handle client connection
# please read here for more information: https://github.com/redis/redis-rb#sentinel-support
##
- #url: redis://master:6379
- # sentinels:
- # -
- # host: slave1
- # port: 26379 # point to sentinel, not to redis port
- # -
- # host: slave2
- # port: 26379 # point to sentinel, not to redis port
+ # url: redis://master:6379
+ # sentinels:
+ # -
+ # host: slave1
+ # port: 26379 # point to sentinel, not to redis port
+ # -
+ # host: slave2
+ # port: 26379 # point to sentinel, not to redis port
diff --git a/lib/gitlab/mail_room.rb b/lib/gitlab/mail_room.rb
index b49cf1c633b..1f68e09fa2b 100644
--- a/lib/gitlab/mail_room.rb
+++ b/lib/gitlab/mail_room.rb
@@ -4,7 +4,6 @@ require_relative 'redis' unless defined?(Gitlab::Redis)
module Gitlab
module MailRoom
-
class << self
def enabled?
config[:enabled] && config[:address]
diff --git a/lib/gitlab/redis.rb b/lib/gitlab/redis.rb
index 70e333eb29f..17ac15a01dd 100644
--- a/lib/gitlab/redis.rb
+++ b/lib/gitlab/redis.rb
@@ -53,18 +53,19 @@ module Gitlab
def redis_store_options
config = raw_config_hash
+ redis_url = config.delete(:url)
+ redis_uri = URI.parse(redis_url)
- redis_uri = URI.parse(config[:url])
if redis_uri.scheme == 'unix'
# Redis::Store does not handle Unix sockets well, so let's do it for them
config[:path] = redis_uri.path
+ config
else
- redis_hash = ::Redis::Store::Factory.extract_host_options_from_uri(config[:url])
- config.merge!(redis_hash)
+ redis_hash = ::Redis::Store::Factory.extract_host_options_from_uri(redis_url)
+ # order is important here, sentinels must be after the connection keys.
+ # {url: ..., port: ..., sentinels: [...]}
+ redis_hash.merge(config)
end
-
- config.delete(:url)
- config
end
def raw_config_hash