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

Make list of environment variable names available in child process #199

Open
doodlesbykumbi opened this issue Feb 2, 2021 · 1 comment

Comments

@doodlesbykumbi
Copy link
Contributor

doodlesbykumbi commented Feb 2, 2021

Is your feature request related to a problem? Please describe

If we strip both @SUMMONENVFILE and the proposed @SUMMONDOCKERARGSwe see that at the core they use the list of summon injected envvar names. If you know that the envvars that summon injected are A, B and C, then the rest can be done in bash or otherwise with relative ease.

This allows us to not build special cases into Summon. Instead we provide primitives to accommodate such cases and beyond.

Describe the solution you would like

Perhaps Summon injects a special environment variable called SUMMONVARNAMES or has a substitution variable similar to @SUMMONENVFILE called @SUMMONVARNAMES.

Describe alternatives you have considered

N/A

Additional context

N/A

@mcanevet
Copy link
Contributor

Maybe having a json output using a magic string (@SUMMONENVJSON or anything else) would do the trick.
This output could be easily formatted with any external tool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants