fix(uptime): don't report monitor_task_ended as an error (#492)

This isn't an error for canary checkers, as they are in readonly mode
and aren't supposed to monitor tasks, and so exit immediately. For the
regular checkers, this is okay too (since the monitor task is looking at
the shutdown signal as well.)

Author: klochek

src/check_config_provider/redis_config_provider.rs

@@ -321,6 +321,8 @@ pub fn run_config_provider(
                 .await
         });
 
+        // The monitor task also responds to the shutdown event, but it's possible it could be
+        // stuck doing other things, so we race it here, too.
         tokio::select! {
             _ = shutdown.cancelled() => {
                 tracing::info!("redis_config_provider.shutdown_requested");
@@ -329,7 +331,7 @@ pub fn run_config_provider(
                 manager.update_partitions(&HashSet::default());
             }
             _ = monitor_task => {
-                tracing::error!("redis_config_provider.monitor_task_ended");
+                tracing::info!("redis_config_provider.monitor_task_ended");
             }
         }
     })