diff options
author | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2014-12-01 18:58:37 +0100 |
---|---|---|
committer | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2014-12-01 19:11:08 +0100 |
commit | 1e033c2ee65d0278876ae79eba6c943513fc7b64 (patch) | |
tree | f7e4e73be70747acfd270d5a2f61e018f81b3b34 | |
parent | 36679b577beb6a2ee181e15c67885c99b27fbdb8 (diff) | |
download | gitlab-ce-1e033c2ee65d0278876ae79eba6c943513fc7b64.tar.gz |
Disable Sidekiq arguments logging by default
Conflicts:
config/initializers/4_sidekiq.rb
-rw-r--r-- | CHANGELOG | 3 | ||||
-rw-r--r-- | config/initializers/4_sidekiq.rb | 3 | ||||
-rw-r--r-- | doc/development/README.md | 1 | ||||
-rw-r--r-- | doc/sidekiq_debugging.md | 14 |
4 files changed, 20 insertions, 1 deletions
diff --git a/CHANGELOG b/CHANGELOG index 3734611e04c..606a0d9dc76 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -1,6 +1,9 @@ v 7.5.1 - Add missing timestamps to 'members' table +v 7.5.2 + - Don't log Sidekiq arguments by default + v 7.5.0 - API: Add support for Hipchat (Kevin Houdebert) - Add time zone configuration on gitlab.yml (Sullivan Senechal) diff --git a/config/initializers/4_sidekiq.rb b/config/initializers/4_sidekiq.rb index 228b14cb526..75c543c0f47 100644 --- a/config/initializers/4_sidekiq.rb +++ b/config/initializers/4_sidekiq.rb @@ -14,7 +14,8 @@ Sidekiq.configure_server do |config| } config.server_middleware do |chain| - chain.add Gitlab::SidekiqMiddleware::ArgumentsLogger + chain.add Gitlab::SidekiqMiddleware::ArgumentsLogger if ENV['SIDEKIQ_LOG_ARGUMENTS'] + chain.add Gitlab::SidekiqMiddleware::MemoryKiller if ENV['SIDEKIQ_MAX_RSS'] end end diff --git a/doc/development/README.md b/doc/development/README.md index 20db6662aca..c31e5d7ae97 100644 --- a/doc/development/README.md +++ b/doc/development/README.md @@ -4,3 +4,4 @@ - [Shell commands](shell_commands.md) in the GitLab codebase - [Rake tasks](rake_tasks.md) for development - [CI setup](ci_setup.md) for testing GitLab +- [Sidekiq debugging](sidekiq_debugging.md) diff --git a/doc/sidekiq_debugging.md b/doc/sidekiq_debugging.md new file mode 100644 index 00000000000..cea11e5f126 --- /dev/null +++ b/doc/sidekiq_debugging.md @@ -0,0 +1,14 @@ +# Sidekiq debugging + +## Log arguments to Sidekiq jobs + +If you want to see what arguments are being passed to Sidekiq jobs you can set +the SIDEKIQ_LOG_ARGUMENTS environment variable. + +``` +SIDEKIQ_LOG_ARGUMENTS=1 bundle exec foreman start +``` + +It is not recommend to enable this setting in production because some Sidekiq +jobs (such as sending a password reset email) take secret arguments (for +example the password reset token). |