increment TOTAL_BUILDS after a build finishes

...instead of when it's automatically queued

Author: QuietMisdreavus

src/docbuilder/queue.rs

@@ -62,6 +62,7 @@ impl DocBuilder {
         match builder.build_package(self, &name, &version) {
             Ok(_) => {
                 let _ = conn.execute("DELETE FROM queue WHERE id = $1", &[&id]);
+                ::web::metrics::TOTAL_BUILDS.inc();
             }
             Err(e) => {
                 // Increase attempt count
@@ -70,6 +71,7 @@ impl DocBuilder {
                 let attempt: i32 = rows.get(0).get(0);
                 if attempt >= 5 {
                     ::web::metrics::FAILED_BUILDS.inc();
+                    ::web::metrics::TOTAL_BUILDS.inc();
                 }
                 error!("Failed to build package {}-{} from queue: {}",
                        name,

src/utils/queue.rs

@@ -4,7 +4,6 @@ use postgres::Connection;
 use error::Result;
 
 pub fn add_crate_to_queue(conn: &Connection, name: &str, version: &str, priority: i32) -> Result<()> {
-    ::web::metrics::TOTAL_BUILDS.inc();
     try!(conn.execute("INSERT INTO queue (name, version, priority) VALUES ($1, $2, $3)",
                       &[&name, &version, &priority]));
     Ok(())