Replace CartoDB::Logger::RollbarLogger per ResqueFailureLogger

pull/15776/head
Alberto Miedes Garcés 4 years ago
parent 05b7da12c0
commit b62293571c

@ -9,7 +9,7 @@ Dir[Rails.root.join("lib/resque/*.rb")].each {|f| require f}
Resque.redis = RedisFactory.new_connection
Resque::Failure::Multiple.classes = [Resque::Failure::Redis, CartoDB::Logger::RollbarLogger]
Resque::Failure::Multiple.classes = [Resque::Failure::Redis, ResqueFailureLogger]
Resque::Failure.backend = Resque::Failure::Multiple
## Logging

@ -44,19 +44,6 @@ module CartoDB
log('debug', exception: exception, message: message, user: user, **additional_data)
end
class RollbarLogger < Resque::Failure::Base
def save
Logger.error(
message: 'Uncaught Resque failure',
exception: exception,
host: worker.hostname,
pid: worker.pid,
job: payload['class'],
args: payload['args']
)
end
end
# Private
# Creates a Rollbar scope that replaces the auto-detected person with the user passed as parameter

@ -0,0 +1,19 @@
# TODO: move to Carto::Common::JobLogger
class ResqueFailureLogger < Resque::Failure::Base
include ::LoggerHelper
def save
log_error(
message: 'Job failed',
component: 'cartodb.resque',
exception: exception,
worker: { pid: worker.pid, hostname: worker.hostname },
job_class: payload['class'],
args: payload['args'].inspect,
run_at: worker.job["run_at"]
)
end
end
Loading…
Cancel
Save