From a407d2329d3c4ffad368066a15479d860fa7c6e0 Mon Sep 17 00:00:00 2001
From: John O'Connor <john.oconnor@puppet.com>
Date: Tue, 10 Nov 2020 19:13:24 +0000
Subject: [PATCH] (MAINT) Fix checkout counter allocation

Checkout metric counters were against the template name and not the
actual pool used which prevents us from counting the checkouts in the
pixa4 pools for example.

Note - this is a re-rerun - the last commit on this file over-wrote
the change.
---
 lib/vmpooler/api/v1.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/vmpooler/api/v1.rb b/lib/vmpooler/api/v1.rb
index 25ac1cf..a13ceeb 100644
--- a/lib/vmpooler/api/v1.rb
+++ b/lib/vmpooler/api/v1.rb
@@ -213,7 +213,7 @@ module Vmpooler
           if vmname
             account_for_starting_vm(vmpool, vmname)
             vms << [vmpool, vmname, vmtemplate]
-            metrics.increment("checkout.success.#{vmtemplate}")
+            metrics.increment("checkout.success.#{vmpool}")
           else
             failed = true
             metrics.increment("checkout.empty.#{requested}")