Commit 0c361852 authored by GitLab Bot's avatar GitLab Bot

Automatic merge of gitlab-org/gitlab-ce master

parents 070d54b1 2fd73269
...@@ -308,7 +308,6 @@ gem 'batch-loader', '~> 1.4.0' ...@@ -308,7 +308,6 @@ gem 'batch-loader', '~> 1.4.0'
# Perf bar # Perf bar
gem 'peek', '~> 1.0.1' gem 'peek', '~> 1.0.1'
gem 'peek-gc', '~> 0.0.2'
# Snowplow events tracking # Snowplow events tracking
gem 'snowplow-tracker', '~> 0.6.1' gem 'snowplow-tracker', '~> 0.6.1'
......
...@@ -672,8 +672,6 @@ GEM ...@@ -672,8 +672,6 @@ GEM
concurrent-ruby (>= 0.9.0) concurrent-ruby (>= 0.9.0)
concurrent-ruby-ext (>= 0.9.0) concurrent-ruby-ext (>= 0.9.0)
railties (>= 4.0.0) railties (>= 4.0.0)
peek-gc (0.0.2)
peek
pg (1.1.4) pg (1.1.4)
po_to_json (1.0.1) po_to_json (1.0.1)
json (>= 1.6.0) json (>= 1.6.0)
...@@ -1215,7 +1213,6 @@ DEPENDENCIES ...@@ -1215,7 +1213,6 @@ DEPENDENCIES
omniauth_openid_connect (~> 0.3.1) omniauth_openid_connect (~> 0.3.1)
org-ruby (~> 0.9.12) org-ruby (~> 0.9.12)
peek (~> 1.0.1) peek (~> 1.0.1)
peek-gc (~> 0.0.2)
pg (~> 1.1) pg (~> 1.1)
premailer-rails (~> 1.9.7) premailer-rails (~> 1.9.7)
prometheus-client-mmap (~> 0.9.8) prometheus-client-mmap (~> 0.9.8)
......
...@@ -112,12 +112,6 @@ export default { ...@@ -112,12 +112,6 @@ export default {
:header="metric.header" :header="metric.header"
:keys="metric.keys" :keys="metric.keys"
/> />
<div id="peek-view-gc" class="view">
<span v-if="currentRequest.details" class="bold">
<span title="Invoke Time">{{ currentRequest.details.gc.gc_time }}</span
>ms / <span title="Invoke Count">{{ currentRequest.details.gc.invokes }}</span> gc
</span>
</div>
<div <div
v-if="currentRequest.details && currentRequest.details.tracing" v-if="currentRequest.details && currentRequest.details.tracing"
id="peek-view-trace" id="peek-view-trace"
......
---
title: Remove GC metrics from performance bar
merge_request:
author:
type: removed
...@@ -10,9 +10,4 @@ Peek.into Peek::Views::Gitaly ...@@ -10,9 +10,4 @@ Peek.into Peek::Views::Gitaly
Peek.into Peek::Views::RedisDetailed Peek.into Peek::Views::RedisDetailed
Peek.into Peek::Views::Rugged Peek.into Peek::Views::Rugged
# `Peek::Views::GC` is currently disabled in production, as it runs with every request
# even if PerformanceBar is inactive and clears `GC::Profiler` reports we need for metrics.
# Check https://gitlab.com/gitlab-org/gitlab-ce/issues/65455
Peek.into Peek::Views::GC if Rails.env.development?
Peek.into Peek::Views::Tracing if Labkit::Tracing.tracing_url_enabled? Peek.into Peek::Views::Tracing if Labkit::Tracing.tracing_url_enabled?
...@@ -16,7 +16,6 @@ It allows you to see (from left to right): ...@@ -16,7 +16,6 @@ It allows you to see (from left to right):
![Rugged profiling using the Performance Bar](img/performance_bar_rugged_calls.png) ![Rugged profiling using the Performance Bar](img/performance_bar_rugged_calls.png)
- time taken and number of Redis calls, click through for details of these calls - time taken and number of Redis calls, click through for details of these calls
![Redis profiling using the Performance Bar](img/performance_bar_redis_calls.png) ![Redis profiling using the Performance Bar](img/performance_bar_redis_calls.png)
- time taken and number of Ruby GC calls
On the far right is a request selector that allows you to view the same metrics On the far right is a request selector that allows you to view the same metrics
(excluding the page timing and line profiler) for any requests made while the (excluding the page timing and line profiler) for any requests made while the
......
...@@ -184,7 +184,7 @@ module QA ...@@ -184,7 +184,7 @@ module QA
end end
def gcloud_num_nodes def gcloud_num_nodes
ENV.fetch('GCLOUD_NUM_NODES', 3) ENV.fetch('GCLOUD_NUM_NODES', 1)
end end
def has_gcloud_credentials? def has_gcloud_credentials?
......
Markdown is supported
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