[tor-commits] [ooni-probe/master] The last runtime should be determined based on the time we started the task

[art at torproject.org]

commit 5c0fd074630b7fdd4e066c0b783242880d0838a3
Author: Arturo Filastò <arturo at filasto.net>
Date:   Mon Sep 12 14:53:32 2016 +0200

    The last runtime should be determined based on the time we started the task
---
 ooni/agent/scheduler.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/ooni/agent/scheduler.py b/ooni/agent/scheduler.py
index 4002369..a7713e0 100644
--- a/ooni/agent/scheduler.py
+++ b/ooni/agent/scheduler.py
@@ -93,10 +93,9 @@ class ScheduledTask(object):
             date_str = in_file.read()
         return datetime.strptime(date_str, self._time_format)
 
-    def _update_last_run(self):
+    def _update_last_run(self, last_run_time):
         with self._last_run.open('w') as out_file:
-            current_time = datetime.utcnow()
-            out_file.write(current_time.strftime(self._time_format))
+            out_file.write(last_run_time.strftime(self._time_format))
 
     def task(self):
         raise NotImplementedError
@@ -117,8 +116,9 @@ class ScheduledTask(object):
         try:
             if self.last_run == CANARY_DATE:
                 yield defer.maybeDeferred(self.first_run)
+            last_run_time = datetime.utcnow()
             yield self.task()
-            self._update_last_run()
+            self._update_last_run(last_run_time)
         except:
             raise
         finally: