Re-add errors (#402)

* Re-add errors

Re-adds errors that were warnified in b2c1733. Specifically
targeting failures when connecting to web services, and did not errorify
messages about parsing codeowners, waiting for a test subcommand to
complete, and gathering pre-test context.

* Wrong one

Author: cmillar-trunk

bundle/src/bundler.rs

@@ -83,7 +83,7 @@ impl BundlerUtil {
                     .iter()
                     .fold(tempfile::tempfile()?, |f, event| {
                         if let Err(e) = serde_json::to_writer(&f, event) {
-                            tracing::warn!("Failed to write BEP event: {}", e);
+                            tracing::error!("Failed to write BEP event: {}", e);
                         }
                         f
                     });

cli/src/context_quarantine.rs

@@ -182,7 +182,7 @@ pub async fn gather_quarantine_context(
         let result = api_client.get_quarantining_config(request).await;
 
         if let Err(ref err) = result {
-            tracing::warn!("{}", err);
+            tracing::error!("{}", err);
         }
 
         result.unwrap_or_default()

cli/src/main.rs

@@ -76,7 +76,7 @@ fn main() -> anyhow::Result<()> {
                         std::process::exit(exitcode::OK);
                     }
                     _ => {
-                        tracing::warn!("Error: {:?}", e);
+                        tracing::error!("Error: {:?}", e);
                         std::process::exit(exitcode::SOFTWARE);
                     }
                 },

cli/src/quarantine_command.rs

@@ -23,7 +23,7 @@ pub async fn run_quarantine(QuarantineArgs { upload_args }: QuarantineArgs) -> a
              upload_bundle_error,
          }| {
             if let Some(e) = upload_bundle_error {
-                tracing::warn!("Error uploading test results: {:?}", e);
+                tracing::error!("Error uploading test results: {:?}", e);
             }
             exit_code
         },

cli/src/test_command.rs

@@ -72,7 +72,7 @@ pub async fn run_test(
             },
         )
         .or_else(|e| {
-            tracing::warn!("Error uploading test results: {:?}", e);
+            tracing::error!("Error uploading test results: {:?}", e);
             Ok(test_run_result_exit_code)
         })
 }

test_report/src/report.rs

@@ -169,7 +169,7 @@ impl MutTestReport {
                 )) {
                 Ok(_) => true,
                 Err(e) => {
-                    tracing::warn!("Error uploading: {:?}", e);
+                    tracing::error!("Error uploading: {:?}", e);
                     false
                 }
             }