dgrove-oss commented on a change in pull request #4570: KCF: propagate
cf_ca_extraArgs_env_i into user containers
URL:
https://github.com/apache/incubator-openwhisk/pull/4570#discussion_r309316353
##########
File path: core/invoker/src/main/resources/application.conf
##########
@@ -93,6 +93,7 @@ whisk {
dns-servers: []
dns-search: []
dns-options: []
+ extra-env-vars: [] # sequence of `key` and/or `key=value` bindings to
add to all user action container environments
Review comment:
I prototyped using a map instead of a seq[k=v] and found an unfortunate
interaction with `transformEnvironment.sh` that makes a map cumbersome to use.
The problem is `transformEnvirovnment.sh` rewrites `_` to `.` which makes it
quite cumbersome to define environment variables whose name contains an `_`
without adding an escape sequence.
For example, consider trying to set `__OW_ALLOW_CONCURRENT=false` via a
`CONFIG_` envar. With a Seq, it is:
```
CONFIG_whisk_containerFactory_containerArgs_extraArgs_env_0=__OW_ALLOW_CONCURRENT=false
```
With a Map, it should be
```
CONFIG_whisk_containerFactory_containerArgs_extraArgs_env___OW_ALLOW_CONCURRENT=false
```
but this doesn't work because we also need to somehow escape all the `_`
after `env_` to prevent `transformEnvironment.sh` from converting them to `.`
(which would take an extension to `transformEnvironment.sh` and proper
documentation of it).
So, I think using Seq to avoid needing to escape `_` is a better trade-off
overall.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
