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

Harmonize handling of environment variables #1223

Closed
dominikriemer opened this issue Feb 3, 2023 · 0 comments · Fixed by #1224
Closed

Harmonize handling of environment variables #1223

dominikriemer opened this issue Feb 3, 2023 · 0 comments · Fixed by #1224
Assignees
Labels
backend Everything that is related to the StreamPipes backend refactoring Indicates when a pull request or issue contains larger refactoring aspects.
Milestone

Comments

@dominikriemer
Copy link
Member

Body

Move env variables referring to host and port that are currently stored in Consul and provide them directly via the Environment interface.
Harmonize usage of env variables in the whole system.

StreamPipes Committer

I acknowledge that I am a maintainer/committer of the Apache StreamPipes project.

@dominikriemer dominikriemer self-assigned this Feb 3, 2023
dominikriemer added a commit that referenced this issue Feb 16, 2023
* Provide environment variables via Environment provider (#1223)

* Clean up consul configuration and env variable provider (#1223)

* Fix bug in environment variable provider (#1223)

* Receive database name from environment (#1223)
dominikriemer added a commit that referenced this issue Feb 17, 2023
* Provide environment variables via Environment provider (#1223)

* Clean up consul configuration and env variable provider (#1223)

* Fix bug in environment variable provider (#1223)

* Receive database name from environment (#1223)
@bossenti bossenti added this to the 0.91.0 milestone Feb 17, 2023
@bossenti bossenti added refactoring Indicates when a pull request or issue contains larger refactoring aspects. backend Everything that is related to the StreamPipes backend labels Feb 17, 2023
flomickl pushed a commit that referenced this issue Feb 18, 2023
* Provide environment variables via Environment provider (#1223)

* Clean up consul configuration and env variable provider (#1223)

* Fix bug in environment variable provider (#1223)

* Receive database name from environment (#1223)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Everything that is related to the StreamPipes backend refactoring Indicates when a pull request or issue contains larger refactoring aspects.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants