diff options
author | Douwe Maan <douwe@gitlab.com> | 2017-08-23 08:55:15 +0000 |
---|---|---|
committer | Douwe Maan <douwe@gitlab.com> | 2017-08-23 08:55:15 +0000 |
commit | 72b5d95489040649348b678c4aca1a7a0ba1d8a5 (patch) | |
tree | 326f0b006317892abd06652d1014615382d1fce7 /app | |
parent | 08c07ad145eeac455536a91ca35f183dbc67c9a6 (diff) | |
parent | e288afec9ccd85fb444ec11d5359a17f8c657f06 (diff) | |
download | gitlab-ce-72b5d95489040649348b678c4aca1a7a0ba1d8a5.tar.gz |
Merge branch 'rs-fix-admin-logs-timeout' into 'master'
Fix a potential timeout in `Gitlab::Logger.read_latest`
See merge request !13751
Diffstat (limited to 'app')
-rw-r--r-- | app/controllers/admin/logs_controller.rb | 9 | ||||
-rw-r--r-- | app/views/admin/logs/show.html.haml | 16 |
2 files changed, 15 insertions, 10 deletions
diff --git a/app/controllers/admin/logs_controller.rb b/app/controllers/admin/logs_controller.rb index b999018dde4..bdc4332ae69 100644 --- a/app/controllers/admin/logs_controller.rb +++ b/app/controllers/admin/logs_controller.rb @@ -1,2 +1,11 @@ class Admin::LogsController < Admin::ApplicationController + def show + @loggers = [ + Gitlab::AppLogger, + Gitlab::GitLogger, + Gitlab::EnvironmentLogger, + Gitlab::SidekiqLogger, + Gitlab::RepositoryCheckLogger + ] + end end diff --git a/app/views/admin/logs/show.html.haml b/app/views/admin/logs/show.html.haml index 487f1cf5c4f..ee87f25a225 100644 --- a/app/views/admin/logs/show.html.haml +++ b/app/views/admin/logs/show.html.haml @@ -1,25 +1,21 @@ - @no_container = true - page_title "Logs" -- loggers = [Gitlab::GitLogger, Gitlab::AppLogger, - Gitlab::EnvironmentLogger, Gitlab::SidekiqLogger, - Gitlab::RepositoryCheckLogger] = render 'admin/monitoring/head' %div{ class: container_class } %ul.nav-links.log-tabs - - loggers.each do |klass| - %li{ class: active_when(klass == Gitlab::GitLogger) }> - = link_to klass::file_name, "##{klass::file_name_noext}", - 'data-toggle' => 'tab' + - @loggers.each do |klass| + %li{ class: active_when(klass == @loggers.first) }> + = link_to klass.file_name, "##{klass.file_name_noext}", data: { toggle: 'tab' } .row-content-block To prevent performance issues admin logs output the last 2000 lines .tab-content - - loggers.each do |klass| - .tab-pane{ class: active_when(klass == Gitlab::GitLogger), id: klass::file_name_noext } + - @loggers.each do |klass| + .tab-pane{ class: active_when(klass == @loggers.first), id: klass.file_name_noext } .file-holder#README .js-file-title.file-title %i.fa.fa-file - = klass::file_name + = klass.file_name .pull-right = link_to '#', class: 'log-bottom' do %i.fa.fa-arrow-down |