coop-cloud/abra
coop-cloud
/
abra
5
2
Fork 15
Code Issues Pull Requests 4 Packages Releases 36 Activity

Expose all env vars to app container. #263

Merged
moritz merged 4 commits from moritz/abra:expose_envs into main 2023-01-31 14:13:44 +00:00
Conversation 0 Commits 4 Files Changed 4 +6 -3
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 1c9a931143 - Show all commits

9
pkg/config/app.go
View File

@ -454,9 +454,12 @@ func ExposeAllEnv(compose *composetypes.Config, appEnv AppEnv) {
if service.Name == "app" {
logrus.Debug("Add the following environment to the app service config:")
moritz marked this conversation as resolved Outdated
decentral1se commented 2023-01-31 13:04:02 +00:00
Outdated
Review
Copy Link

Perhaps include the app / service / etc. info in the log message? Whatever seems appropriate.

Perhaps include the app / service / etc. info in the log message? Whatever seems appropriate.
for k, v := range appEnv {
value := v
service.Environment[k] = &value
logrus.Debugf("Key: %s Value: %s", k, value)
_, exists := service.Environment[k]
if !exists {
value := v
service.Environment[k] = &value
logrus.Debugf("Add Key: %s Value: %s", k, value)
moritz marked this conversation as resolved Outdated
decentral1se commented 2023-01-31 13:04:22 +00:00
Outdated
Review
Copy Link

Same here, with some marker on the recipe/app/etc. perhaps?

Same here, with some marker on the recipe/app/etc. perhaps?
}
}
}
}