Fix completion percentage calculation in mirror status

We sometimes record a duration even on a failed fetch attempt, such as
if we get an HTTP 404. However, we never record a last_sync value on a
failed fetch. Use this field instead to sum up the total number of
successful checks.

Signed-off-by: Dan McGee <dan@archlinux.org>

1 files changed, 2 insertions, 2 deletions

diff --git a/mirrors/utils.py b/mirrors/utils.py
index bba8e36b..633731a7 100644
--- a/mirrors/utils.py
+++ b/mirrors/utils.py
@@ -35,7 +35,7 @@ def status_data(cutoff_time, mirror_id=None):
         sql = """
 SELECT l.url_id, u.mirror_id,
     COUNT(l.id) AS check_count,
-    COUNT(l.duration) AS success_count,
+    COUNT(l.last_sync) AS success_count,
     MAX(l.last_sync) AS last_sync,
     MAX(l.check_time) AS last_check,
     AVG(l.duration) AS duration_avg,
@@ -51,7 +51,7 @@ GROUP BY l.url_id, u.mirror_id
         sql = """
 SELECT l.url_id, u.mirror_id,
     COUNT(l.id) AS check_count,
-    COUNT(l.duration) AS success_count,
+    COUNT(l.last_sync) AS success_count,
     MAX(l.last_sync) AS last_sync,
     MAX(l.check_time) AS last_check,
     AVG(l.duration) AS duration_avg,