From 5a51c9a815e2f75cef95ae024d5a075cea8485ce Mon Sep 17 00:00:00 2001
From: depristo <depristo@348d0f76-0448-11de-a6fe-93d51630548a>
Date: Tue, 15 Feb 2011 20:20:14 +0000
Subject: [PATCH] AWS_S3 logging is now enabled by default.  It first tries to
 log internally at the Broad, and if it can't goes to AWS_S3.  DEV option is
 removed

git-svn-id: file:///humgen/gsa-scr1/gsa-engineering/svn_contents/trunk@5249 348d0f76-0448-11de-a6fe-93d51630548a
---
 .../sting/gatk/phonehome/GATKRunReport.java          | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/java/src/org/broadinstitute/sting/gatk/phonehome/GATKRunReport.java b/java/src/org/broadinstitute/sting/gatk/phonehome/GATKRunReport.java
index 23a8a58aa..3a8c139fc 100644
--- a/java/src/org/broadinstitute/sting/gatk/phonehome/GATKRunReport.java
+++ b/java/src/org/broadinstitute/sting/gatk/phonehome/GATKRunReport.java
@@ -155,16 +155,11 @@ public class GATKRunReport {
     @Element(required = true, name = "reads")
     private long nReads;
 
-    // TODO
-    // todo md5 all filenames
-    // todo size of filenames
-
     public enum PhoneHomeOption {
         NO_ET,
         STANDARD,
-        DEV,
         STDOUT,
-        AWS_S3
+        AWS_S3   // todo -- remove me -- really just for testing purposes
     }
 
     private static final DateFormat dateFormat = new SimpleDateFormat("yyyy/MM/dd HH.mm.ss");
@@ -250,14 +245,15 @@ public class GATKRunReport {
     }
 
     public void postReport(PhoneHomeOption type) {
+        logger.info("Posting report of type " + type);
         switch (type) {
             case NO_ET: // don't do anything
                 break;
-            case STANDARD: case DEV:
+            case STANDARD:
                 if ( repositoryIsOnline() ) {
                     postReportToLocalDisk(REPORT_SUBMIT_DIR);
                 } else {
-                    logger.debug("Not writing report: sentinel " + REPORT_SENTINEL + " doesn't exist");
+                    postReportToAWSS3();
                 }
                 break;
             case STDOUT: