airship/airshipctl
Code Issues Proposed changes

Use env variables only for exec process in KRM toolbox

Browse Source 
Small improvement that allows env variables to be used only for
exec process in the KRM toolbox.

Change-Id: I93436813be883241996ab46154d78452804a8665
This commit is contained in:
Vladislav Kuzmin 2021-04-01 15:21:07 +04:00
parent ea534c379f
commit 532627deed
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
krm-functions/toolbox/main.go
View File

@ -92,12 +92,13 @@ func (c *ScriptRunner) Run() error {
c.ResourceList.Items = nil c.ResourceList.Items = nil
os.Setenv(EnvRenderedBundlePath, bundlePath)
clicmd := exec.Command(scriptPath) clicmd := exec.Command(scriptPath)
clicmd.Stdout = c.OutStream clicmd.Stdout = c.OutStream
clicmd.Stderr = c.ErrStream clicmd.Stderr = c.ErrStream
clicmd.Env = os.Environ()
clicmd.Env = append(clicmd.Env, fmt.Sprintf("%s=%s", EnvRenderedBundlePath, bundlePath))
err = clicmd.Start() err = clicmd.Start()
if err != nil { if err != nil {
return err return err