Compare commits

..

No commits in common. "0b1d14afe82666b6fb5b4e0ddab663541a5c0d97" and "95ee4228bd29dfda0ef3916a792816fac84bbdc8" have entirely different histories.

2 changed files with 3 additions and 10 deletions

View File

@ -1,11 +1,4 @@
# Revision history for Difluoroborane # Revision history for Difluoroborane
A changelog following the [keepachangelog.com/en/1.0.0](https://keepachangelog.com/en/1.0.0/) standard
## 0.4.3 -- unreleased
### CHANGED Changed
* errors in actors are now logged as errors ([#84](https://gitlab.com/fabinfra/fabaccess/bffh/-/issues/84))
## 0.4.1 -- 2022-04-24 ## 0.4.1 -- 2022-04-24

View File

@ -71,17 +71,17 @@ impl Actor for Process {
} }
} }
Ok(retv) => { Ok(retv) => {
tracing::error!(%name, ?state, code=?retv.status, tracing::warn!(%name, ?state, code=?retv.status,
"Actor returned nonzero exitcode" "Actor returned nonzero exitcode"
); );
if !retv.stderr.is_empty() { if !retv.stderr.is_empty() {
let errstr = String::from_utf8_lossy(&retv.stderr); let errstr = String::from_utf8_lossy(&retv.stderr);
for line in errstr.lines() { for line in errstr.lines() {
tracing::error!(%name, %line, "actor stderr"); tracing::warn!(%name, %line, "actor stderr");
} }
} }
} }
Err(error) => tracing::error!(%name, ?error, "process actor failed to run cmd"), Err(error) => tracing::warn!(%name, ?error, "process actor failed to run cmd"),
} }
}) })
} }