amdgpud icon indicating copy to clipboard operation
amdgpud copied to clipboard

Mistakes in config parsing

Open house-of-vanity opened this issue 3 years ago • 7 comments

Hi, first of all your project is awesome! Wanna report an issue. I've build v1.0.8 tag and found an issue with config parsing. I used default example config from examples and got:

thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Error { inner: ErrorInner { kind: Custom, line: Some(24), col: 0, at: Some(268), message: "missing field `interval`", key: [] } }', /home/ab/repos/amdgpud/amdgpu/src/utils.rs:92:58 note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace Aborted (core dumped)

I've made some research and know the point. Here is a default values but it works if neither interval nor log_level are defined: https://github.com/Eraden/amdgpud/blob/de5f2b77cb9f48f14f86412aa5b3a36c753e8174/amdgpu-config/src/monitor.rs#L22-L29

But if I have config without interval but log_level and vice versa it fails. Looks like need to check unexciting values and define it with default. Here is an error, config.interval doesn't exist: https://github.com/Eraden/amdgpud/blob/de5f2b77cb9f48f14f86412aa5b3a36c753e8174/amdgpu-config/src/monitor.rs#L44-L50 Rust isn't my strong point so I can't fix it in a fancy way.

house-of-vanity avatar Jan 17 '22 16:01 house-of-vanity