Commit afba21f6 authored by Ben Kochie's avatar Ben Kochie

Update piplines created metric name

* Use `_total` naming convention.
* Add to metrics documentation.
parent 3453bc32
...@@ -135,7 +135,7 @@ module Ci ...@@ -135,7 +135,7 @@ module Ci
end end
def pipeline_created_counter def pipeline_created_counter
@pipeline_created_counter ||= Gitlab::Metrics.counter(:pipelines_created_count, "Pipelines created count") @pipeline_created_counter ||= Gitlab::Metrics.counter(:pipelines_created_total, "Counter of pipelines created")
end end
end end
end end
...@@ -39,6 +39,7 @@ In this experimental phase, only a few metrics are available: ...@@ -39,6 +39,7 @@ In this experimental phase, only a few metrics are available:
| filesystem_readable | Gauge | Whether or not the filesystem is readable | | filesystem_readable | Gauge | Whether or not the filesystem is readable |
| http_requests_total | Counter | Rack request count | | http_requests_total | Counter | Rack request count |
| http_request_duration_seconds | Histogram | HTTP response time from rack middleware | | http_request_duration_seconds | Histogram | HTTP response time from rack middleware |
| pipelines_created_total | Counter | Counter of pipelines created |
| rack_uncaught_errors_total | Counter | Rack connections handling uncaught errors count | | rack_uncaught_errors_total | Counter | Rack connections handling uncaught errors count |
| redis_ping_timeout | Gauge | Whether or not the last redis ping timed out | | redis_ping_timeout | Gauge | Whether or not the last redis ping timed out |
| redis_ping_success | Gauge | Whether or not the last redis ping succeeded | | redis_ping_success | Gauge | Whether or not the last redis ping succeeded |
......
...@@ -40,7 +40,7 @@ describe Ci::CreatePipelineService, :services do ...@@ -40,7 +40,7 @@ describe Ci::CreatePipelineService, :services do
it 'increments the prometheus counter' do it 'increments the prometheus counter' do
expect(Gitlab::Metrics).to receive(:counter) expect(Gitlab::Metrics).to receive(:counter)
.with(:pipelines_created_count, "Pipelines created count") .with(:pipelines_created_total, "Counter of pipelines created")
.and_call_original .and_call_original
pipeline pipeline
......
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