feat: add log level to environment variables sent to provider#1540
feat: add log level to environment variables sent to provider#1540pascalbreuninger merged 1 commit intoloft-sh:mainfrom mrsimonemms:main
Conversation
|
No problem @pascalbreuninger I'm not 100% certain, but I don't think the failing test is any change I've made but looks like a Docker failure. |
|
@pascalbreuninger I was just starting to go through the providers to add the Having a look through, it looks like this actually a thing. Question - do you want to keep both Unless you think you'll change to having different log levels, I would suggest reverting this PR. Once you've done that, I'll update all the official providers to use |
Description
This adds a
DEVPOD_LOG_LEVELenvironment variable to the environment variables sent to the provider.Design decisions
DEVPOD_prefix because it seemed to be in that section of options.Discussion
Any data received from the provider will be output at the
infolevel because it's only doing a simple "send everything toinfo" rather than receiving any metadata to say the log level. My opinion would be to get this in today to allow debug logs to be output and then overhaul the log metadata as a future task.Issue
Fixes #1537
How to test
In order to receive the log level in the provider, you should do something like this.
Now run the binary:
This will then tell the provider to set to the log level to
debug.