Merge lp:~cjohnston/launchpad-work-items-tracker/add-print-statement-debug into lp:launchpad-work-items-tracker

[Chris Johnston]

Adds print statements to functions on all-projects for debugging

Since the move from one DC to another DC last week, the amount of time to process the scripts has more than doubled. IS said that the move shouldn't be causing the increase in time. This merge will print out the start and stop times for the different command allowing us to further track and debug the cause of the extended time.

[James Westby (james-w)]

31 + print "Publish new database started at: ", datetime.datetime.now(), "for", config_file

Should be "ended" I think?

Otherwise this looks good.

Thanks,

James

review: Approve

=== modified file 'all-projects'
--- all-projects	2011-07-08 10:08:00 +0000
+++ all-projects	2012-08-22 16:34:24 +0000
@@ -12,9 +12,11 @@
 import shutil
 import subprocess
 import sys
+import datetime
 
 
 def collect(source_dir, db_file, config_file, extra_args):
+    print "Launchpad collect started at: ", datetime.datetime.now(), "for", config_file
     args = [os.path.join(source_dir, "collect")]
     args.extend(["-d", db_file])
     args.extend(["-c", config_file])
@@ -22,19 +24,22 @@
     proc = subprocess.Popen(args)
     proc.communicate()
     return proc.returncode == 0
-
+    print "Launchpad collect ended at: ", datetime.datetime.now(), "for", config_file
 
 def publish_new_db(project_name, project_output_dir, db_file):
+    print "Publish new database started at: ", datetime.datetime.now(), "for", config_file
     if not os.path.exists(project_output_dir):
         os.makedirs(project_output_dir)
     target_file = os.path.join(project_output_dir, "%s.db" % project_name)
     temp_file = target_file + ".new"
     shutil.copyfile(db_file, temp_file)
     os.rename(temp_file, target_file)
+    print "Publish new database started at: ", datetime.datetime.now(), "for", config_file
 
 
 def generate_reports(project_output_dir, config_file, db_file, source_dir,
         extra_args, debug=False):
+    print "Generate reports started at: ", datetime.datetime.now(), "for", config_file
     stdout = None
     if not debug:
         stdout = subprocess.PIPE
@@ -46,7 +51,8 @@
     proc = subprocess.Popen(args, stdout=stdout)
     proc.communicate()
     assert proc.returncode == 0, "generate-all failed for %s" % config_file
-
+    print "Generate reports ended at: ", datetime.datetime.now(), "for", config_file
+    
 
 def backup_db(project_name, project_output_dir, current_time):
     project_db = os.path.join(project_output_dir, "%s.db" % project_name)