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

DevPod should send the desired log level to providers #1537

Closed
mrsimonemms opened this issue Jan 10, 2025 · 0 comments · Fixed by #1540
Closed

DevPod should send the desired log level to providers #1537

mrsimonemms opened this issue Jan 10, 2025 · 0 comments · Fixed by #1540
Labels
good first issue Good for newcomers

Comments

@mrsimonemms
Copy link
Contributor

mrsimonemms commented Jan 10, 2025

Is your feature request related to a problem?
I'm a maintainer of a provider and I log things at different levels. By default, the logger is set to Info. If the DevPod application is in debug mode, there should be a way of the application telling the provider to emit messages at the Debug level.

Often, the debug level has useful information for debugging problems (who knew?), but that we don't want to see in normal circumstances.

Which solution do you suggest?
All the communication between the application and the provider is done via environment variables - it would make sense to use that mechanism. My suggestions are either:

  • DEBUG: true|false
  • LOG_LEVEL: debug|info|warn (this would default to info if nothing sent)

It looks like the application only recognises whether it's in debug mode or not, so the top one would make sense. But having a LOG_LEVEL would allow for granularity and future changes.

In truth, I don't really care which is which, so long as something is set.

Which alternative solutions exist?
Do nothing - not a major problem, but does miss out of the ability to see debug messages

Additional context
Slack thread

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants