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

Allow additional secrets to set environment variables #167

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

pantierra
Copy link
Contributor

Currently, the envVars for eoapi allow setting of environment variables as a string. However these values might be set in some kubernetes secret. Let's extend eoAPI to handle those cases.

@pantierra pantierra force-pushed the feature/handle-secrets branch from d4801cc to 3eece9a Compare December 5, 2024 09:46
@pantierra pantierra marked this pull request as draft December 5, 2024 09:50
@pantierra pantierra force-pushed the feature/handle-secrets branch from 3eece9a to aab936a Compare December 5, 2024 13:34
@pantierra pantierra marked this pull request as ready for review December 5, 2024 13:34
@pantierra
Copy link
Contributor Author

The first approach wasn't working, so I introduced an envSecrets variable, which adds secrets' contents as environment variables, if they are specified.

@pantierra pantierra changed the title Make envVars handle secrets. Allow additonal secrets to set environment variables Dec 5, 2024
@pantierra pantierra changed the title Allow additonal secrets to set environment variables Allow additional secrets to set environment variables Dec 5, 2024
@pantierra pantierra force-pushed the feature/handle-secrets branch from aab936a to c82541a Compare December 5, 2024 13:44
@pantierra pantierra merged commit 100c076 into main Dec 5, 2024
2 of 3 checks passed
@pantierra pantierra deleted the feature/handle-secrets branch December 5, 2024 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant