Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: Confusing error message reading configuration files #112

Open
ddneilson opened this issue Apr 23, 2024 · 0 comments
Open

Bug: Confusing error message reading configuration files #112

ddneilson opened this issue Apr 23, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@ddneilson
Copy link
Contributor

Expected Behaviour

The message should be more informative about what is going on, and not be so alarming. Is this really a warning-level event?

e.g. In this case, the runtime should simply say that the configuration file doesn't exist or just print nothing and move on -- that adaptor doesn't have a configuration file so maybe that's intentional and is not worth logging.

Current Behaviour

Customers see messages like

2024/04/23 17:05:54+00:00 WARNING: Default configuration file at C:\Program Files\Python39\lib\site-packages\deadline\unreal_adaptor\UnrealAdaptor\UnrealAdaptor.json is not a valid file. Using empty configuration.

when the runtime attempts to read the adaptor-local configuration file, but the file does not exist.

Reproduction Steps

Run an adaptor that does not have a local configuration file.

Code Snippet

N/A

@ddneilson ddneilson added the bug Something isn't working label Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant