diff --git a/edgelet/iotedge/src/check/checks/container_engine_logrotate.rs b/edgelet/iotedge/src/check/checks/container_engine_logrotate.rs index d4fdf4e037f..665446b77f3 100644 --- a/edgelet/iotedge/src/check/checks/container_engine_logrotate.rs +++ b/edgelet/iotedge/src/check/checks/container_engine_logrotate.rs @@ -55,22 +55,21 @@ impl ContainerEngineLogrotate { .context(MESSAGE)?; self.daemon_config = Some(daemon_config.clone()); - match daemon_config.log_driver.as_deref() { - Some("journald") => return Ok(CheckResult::Ok), - None => return Ok(CheckResult::Warning(anyhow!(MESSAGE))), - _ => (), - } - - if let Some(log_opts) = &daemon_config.log_opts { - if log_opts.max_file.is_none() { - return Ok(CheckResult::Warning(anyhow!(MESSAGE))); - } + if matches!( + daemon_config.log_driver.as_deref(), + Some("json-file") | None + ) { + if let Some(log_opts) = &daemon_config.log_opts { + if log_opts.max_file.is_none() { + return Ok(CheckResult::Warning(anyhow!(MESSAGE))); + } - if log_opts.max_size.is_none() { + if log_opts.max_size.is_none() { + return Ok(CheckResult::Warning(anyhow!(MESSAGE))); + } + } else { return Ok(CheckResult::Warning(anyhow!(MESSAGE))); } - } else { - return Ok(CheckResult::Warning(anyhow!(MESSAGE))); } Ok(CheckResult::Ok)