diff options
author | Z.J. van de Weg <git@zjvandeweg.nl> | 2017-07-24 12:44:33 +0200 |
---|---|---|
committer | Z.J. van de Weg <git@zjvandeweg.nl> | 2017-07-31 11:51:28 +0200 |
commit | 5f86347ee614445d23ec9b21cf77fcbcf790756c (patch) | |
tree | 7e5024748953fa9b77e7fc492afb9ed7d77b6d1f /app/services/projects/update_pages_service.rb | |
parent | 202806c029802dee6924925313d75d476192ea59 (diff) | |
download | gitlab-ce-5f86347ee614445d23ec9b21cf77fcbcf790756c.tar.gz |
Add two more metrics for CI/CDzj-add-ci-metrics
As its hard right now to determine what is a good metric and whats not,
these two are not listed in the docs, nor will they get a CHANGELOG
entry.
Diffstat (limited to 'app/services/projects/update_pages_service.rb')
-rw-r--r-- | app/services/projects/update_pages_service.rb | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/app/services/projects/update_pages_service.rb b/app/services/projects/update_pages_service.rb index 749a1cc56d8..5038155ca31 100644 --- a/app/services/projects/update_pages_service.rb +++ b/app/services/projects/update_pages_service.rb @@ -33,8 +33,10 @@ module Projects success end rescue => e + register_failure error(e.message) ensure + register_attempt build.erase_artifacts! unless build.has_expiring_artifacts? end @@ -168,5 +170,21 @@ module Projects def sha build.sha end + + def register_attempt + pages_deployments_total_counter.increase + end + + def register_failure + pages_deployments_failed_total_counter.increase + end + + def pages_deployments_total_counter + @pages_deployments_total_counter ||= Gitlab::Metrics.counter(:pages_deployments_total, "Counter of GitLab Pages deployments triggered") + end + + def pages_deployments_failed_total_counter + @pages_deployments_failed_total_counter ||= Gitlab::Metrics.counter(:pages_deployments_failed_total, "Counter of GitLab Pages deployments which failed") + end end end |