fixup: log levels/wording (#532)

installers/binstall/src/install.rs

@@ -71,8 +71,11 @@ impl Installer {
 
     fn write_bin_to_fs(&self) -> Result<(), InstallerError> {
         let bin_path = self.get_bin_path()?;
-        tracing::debug!("copying from: {}", &self.executable_location);
-        tracing::debug!("copying to: {}", &bin_path);
+        tracing::debug!(
+            "copying \"{}\" to \"{}\"",
+            &self.executable_location,
+            &bin_path
+        );
         // attempt to remove the old binary
         // but do not error if it doesn't exist.
         let _ = fs::remove_file(&bin_path);

src/command/graph/publish.rs

@@ -47,7 +47,7 @@ impl Publish {
 
         let schema_document = load_schema_from_flag(&self.schema, std::io::stdin())?;
 
-        tracing::debug!(?schema_document);
+        tracing::debug!("Publishing \n{}", &schema_document);
 
         let publish_response = publish::run(
             publish::publish_schema_mutation::Variables {

src/command/subgraph/publish.rs

@@ -72,7 +72,7 @@ impl Publish {
 
         let schema_document = load_schema_from_flag(&self.schema, std::io::stdin())?;
 
-        tracing::debug!("Schema Document to publish:\n{}", &schema_document);
+        tracing::debug!("Publishing \n{}", &schema_document);
 
         // This response is used to check whether or not the current graph is federated.
         let federated_response = is_federated::run(

src/utils/env.rs

@@ -33,7 +33,7 @@ impl RoverEnv {
     /// returns the value of the environment variable if it exists
     pub fn get(&self, key: RoverEnvKey) -> io::Result<Option<String>> {
         let key_str = key.to_string();
-        tracing::debug!("Checking for ${}", &key_str);
+        tracing::trace!("Checking for ${}", &key_str);
         let result = match &self.mock_store {
             Some(mock_store) => Ok(mock_store.get(&key_str).map(|v| v.to_owned())),
             None => match env::var(&key_str) {
@@ -54,7 +54,7 @@ impl RoverEnv {
         if let Some(result) = &result {
             tracing::debug!("read {}", self.get_debug_value(key, result));
         } else {
-            tracing::debug!("could not find #{}", &key_str);
+            tracing::trace!("could not find ${}", &key_str);
         }
 
         Ok(result)

src/utils/telemetry.rs

@@ -75,14 +75,21 @@ impl Report for Rover {
         let json_args = serde_json::to_string(&self)?;
         let mut value_args = serde_json::from_str(&json_args)?;
         let serialized_command = get_command_from_args(&mut value_args);
-        tracing::debug!(serialized_command = ?serialized_command);
+        tracing::debug!(?serialized_command);
         Ok(serialized_command)
     }
 
     fn is_telemetry_enabled(&self) -> Result<bool, SputnikError> {
         let value = self.env_store.get(RoverEnvKey::TelemetryDisabled)?;
         let is_telemetry_disabled = value.is_some();
-        tracing::debug!(is_telemetry_disabled);
+        if is_telemetry_disabled {
+            tracing::info!("Telemetry has been disabled.");
+        } else {
+            tracing::info!(
+                "Telemetry is enabled. To disable, set ${}=1",
+                RoverEnvKey::TelemetryDisabled.to_string()
+            )
+        }
         Ok(!is_telemetry_disabled)
     }
 

src/utils/version.rs

@@ -31,20 +31,18 @@ pub fn check_for_update(config: config::Config, force: bool) -> Result<()> {
         do_update_check(&mut checked, force)?;
     } else if let Some(last_checked_time) = last_checked_time {
         let time_since_check = current_time.duration_since(last_checked_time)?.as_secs();
-        tracing::debug!(
+        tracing::trace!(
             "Time since last update check: {:?}h",
             time_since_check / ONE_HOUR
         );
 
         if time_since_check > ONE_DAY {
             do_update_check(&mut checked, force)?;
-        } else {
-            tracing::debug!("No need to check for updates. Automatic checks happen once per day");
         }
     }
 
     if checked {
-        tracing::debug!("Checked for available updates. Writing current time to disk");
+        tracing::trace!("Checked for available updates. Writing current time to disk");
         fs::write(&version_file, toml::to_string(&current_time)?)?;
     }