metrics: Only count successful syncs

Author: Jorin Vogel

project.clj

@@ -1,4 +1,4 @@
-(defproject googlesheets-sql-sync "0.2.0"
+(defproject googlesheets-sql-sync "0.3.0"
   :description "Keep your SQL database in sync with Google Sheets"
   :url "https://github.com/jorinvo/googlesheets-sql-sync"
   :license {:name "MIT"

src/googlesheets_sql_sync/metrics.clj

@@ -15,7 +15,7 @@
   "Convert metrics to readable string"
   [{:keys [::sync-counter]}]
   (str
-    "# HELP googlesheets_sql_sync_count The total number of syncs completed.\n"
+    "# HELP googlesheets_sql_sync_count The total number of syncs successfully completed.\n"
     "# TYPE googlesheets_sql_sync_count counter\n"
     (format "googlesheets_sql_sync_count %d\n" @sync-counter)))
 

src/googlesheets_sql_sync/worker.clj

@@ -30,7 +30,8 @@
             (->> (:sheets cfg)
                  (map #(sheets/get-rows % token throttler))
                  (run! #(db/update-table cfg %)))
-            (log/info "Sync done"))
+            (log/info "Sync done")
+            (metrics/count-sync ctx))
           (if no-server
             (do
               (log/error "Cannot authenticate when server is disabled")
@@ -39,7 +40,6 @@
         (catch Exception e (log/error (.getMessage e) "\nSync failed")))
       (log/info "Next sync in" (interval/->string (:interval cfg)))
       (async/put! timeout> (:interval cfg)))
-    (metrics/count-sync ctx)
     (catch Exception e (do (log/error "Failed reading config file" (.getMessage e))
                            :not-ok))))