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
FLEXMEASURES_PLUGINS configurable from env #660
Conversation
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
…parated string; document wich settings are also read as env vars Signed-off-by: Nicolas Höning <nicolas@seita.nl>
…RES_PLUGINS setting Signed-off-by: Nicolas Höning <nicolas@seita.nl>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still needs an actual changelog entry, I think. And we might need to update some plugin documentation if anything still uses the sunset way. I also noticed the deprecation notice was set to "debug" level, which may have been too subtle to notice. I'd recommend not sunsetting right now and increasing the logging level instead first.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this new add-on!
I haven't tested this procedure yet, but it looks clean and I don't think it will give any problems (I'll try it this week, though).
@Flix6x I agree in principle that we should pre-announce removal of config settings. In this case, it lived only a few weeks about 18 months ago. We did, however, mention it in the Readmes of the entsoe- and zinfo-plugins. The former might have been implemented by somebody, so I will upgrade the log level and warn in the release notes. |
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 😄
The FLEXMEASURES_PLUGINS settings should also be possible as an env var. This is handy for devs, who simply want to add their own logic, but are not busy with how FlexMeasures works or is deployed. For example, they use a docker-compose or Kubernetes setup and just add their logic.