-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Messages from ovos_utils.log.LOG are swallowed and not sent to STDOUT #173
Comments
Which version? |
0.0.23 for sure, could be earlier |
I'm able to see message from the
|
Are those logs coming from ovos_utils.log.LOG? I do see logs, just not the logs I'm trying to emit from the plugin. Previous versions I could see them. |
Got few from
|
https://github.com/OpenVoiceOS/ovos-tts-plugin-espeakNG/blob/443e8ec672b47ab28498cba3307f300dab502384/ovos_tts_plugin_espeakng/__init__.py#L31 This one and the others like it don't display |
You got some git conflict stuff in there: |
Fixed |
Is it possible this started when ovos-utils was updated to read log level from envvars? It may have never read from config properly and simply been using default settings with changed from |
That sounds like a very likely prospect, although I can't pinpoint when it stopped working because for me, it's never worked. :) |
So far, it seems the logs are going to 369d8aa#diff-27b5d7bc9f622d0bca7e34de6b99013a62bc5849ab3b6379e30a6eee9757b65cR12-R13 |
Found a potential cause here; If log config is always read from Configuration then I don't think this is problematic, but it would override envvar config and potentially module overrides for getting the expected configuration paths |
it says
lets remove it then! |
This may not be it; tests are passing locally but failing in GHA for me; might be something from a dependency |
I'm not able to reproduce this issue. Logs from testing:
|
is this still an issue? |
No idea...since no one was able to reproduce we can probably close it. |
On latest alpha, I can't see log messages over STDOUT in a Docker image anymore.
The text was updated successfully, but these errors were encountered: