eclipse-iceoryx / iceoryx2

Eclipse iceoryx2™ - true zero-copy inter-process-communication in pure Rust
https://iceoryx.io
Apache License 2.0
1.03k stars 40 forks source link

Warning regarding default config is misleading #437

Closed elBoberido closed 1 month ago

elBoberido commented 1 month ago

Required information

Operating system:

any

Rust version:

any

Cargo version:

any

iceoryx2 version: E.g. v1.2.3 or main branch

Detailed log output:

        2 [W] "Config::global_config()"
              | Default config file found but unable to read data, populate config with default values.

Observed result or behaviour:

When running an iceoryx2 application, it looks for config/iceoryx2.toml. When it is not found, it prints the warning from above. This is misleading, since the config is not found.

Expected result or behaviour:

The warning should not be printed when the config file was not found but only when it was found and and could not be read.

Conditions where it occurred / Performed steps:

Create an out-of-tree iceoryx2 application and run it.

elBoberido commented 1 month ago

Furthermore, it looks at a relative path to where the application is started. This is also error prone since the application might behave differently when launched from different locations. I'll open a separate issue for this if you agree that it should be changed.