Monitor only final statesrack-timeout-rename

There is no reason to monitor transition states so we ignore
ready and active states. We can get ratio of completed vs failed
requests from final states.

1 files changed, 4 insertions, 1 deletions

diff --git a/lib/gitlab/cluster/rack_timeout_observer.rb b/lib/gitlab/cluster/rack_timeout_observer.rb
index 2bc006b8011..5182b2be148 100644
--- a/lib/gitlab/cluster/rack_timeout_observer.rb
+++ b/lib/gitlab/cluster/rack_timeout_observer.rb
@@ -3,8 +3,10 @@
 module Gitlab
   module Cluster
     class RackTimeoutObserver
+      TRANSITION_STATES = %i(ready active).freeze
+
       def initialize
-        @counter = Gitlab::Metrics.counter(:rack_state_total, 'Number of requests in a given rack state')
+        @counter = Gitlab::Metrics.counter(:rack_requests_total, 'Number of requests in a given rack state')
       end
 
       # returns the Proc to be used as the observer callback block
@@ -17,6 +19,7 @@ module Gitlab
       def log_timeout_exception(env)
         info = env[::Rack::Timeout::ENV_INFO_KEY]
         return unless info
+        return if TRANSITION_STATES.include?(info.state)
 
         @counter.increment(labels(info, env))
       end