Commit 20d8cc06 authored by Ahmad Sherif's avatar Ahmad Sherif
Browse files

Merge branch 'remove-lua-script-remains' into 'master'

Remove final mentions of the LUA script

See merge request gitlab-org/gitlab-exporter!146
parents a7ecb469 48a2ec81
Pipeline #203969 passed with stage
in 1 minute and 45 seconds
PATH
remote: .
specs:
gitlab-exporter (11.0.0)
gitlab-exporter (11.0.1)
connection_pool (= 2.2.5)
pg (= 1.2.3)
puma (= 5.3.2)
......
......@@ -189,14 +189,9 @@ module GitLab
def connected?
return @connected unless @connected.nil?
# This is also a good "connected check"
Sidekiq.redis do |conn|
# Using administrative commands on conn directly (which is a Redis::Namespace)
# will be removed in redis-namespace 2.0.
conn.redis.script(:load, QUEUE_JOB_STATS_SCRIPT) unless conn.redis.script(:exists, QUEUE_JOB_STATS_SHA)
@connected = (conn.ping == "PONG")
end
@connected = true
rescue Redis::BaseConnectionError => e
@logger&.error "Error connecting to the Redis: #{e}"
@connected = false
......
module GitLab
module Exporter
VERSION = "11.0.0".freeze
VERSION = "11.0.1".freeze
end
end
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment