Add support for templated env entries #472
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I've encountered a number of people (including myself) that would like templated environment variable entries in kernelspecs, so I thought I'd implement it add see if this is something useful to the community.
With this change, kernelspec env entries (or even those clients that set
extra_env
on theKernelManager
instance (along withkernel_cmd
)) can reference existing env variables using${ENV_VAR}
within variousenv
entries and expect those substitutions to be performed prior to the kernel's launch.Here's an example kernelspec env entry:
Following the kernel's invocation, it's environment will include the following variable/values:
If the templated env variable is not in the launching environment, the templated value will remain in place - just as it shows in the kernelspec's
kernel.json
file.Kernelspec administrators should be cognizant of potential to share environments. However, given that's case currently, this shouldn't introduce any new issues.