airship/airshipctl
Code Issues Proposed changes

Redirect all stdout of the toolbox containers to stderr

Browse Source 
Initially we did not want this because we had a feeling that
at some point we want to perform render action inside the script.
Rendering inside the script would facilitate better debugging
functionality. Where each script would be able to render, validate
and run. However, at this point solution seems like overengineering
and seems like this should be simply a limitation of the toolbox image.

Change-Id: I99871c1c79934612d9a1ac06a5141e57e5497054
This commit is contained in:
Kostiantyn Kalynovskyi 2021-05-19 13:49:32 +00:00
parent 281e99aa3d
commit c4598ba4cb
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -61,7 +61,7 @@ func main() {
ResourceList: resourceList, ResourceList: resourceList,
ConfigMap: cfg, ConfigMap: cfg,
ErrStream: os.Stderr, ErrStream: os.Stderr,
OutStream: os.Stdout, OutStream: os.Stderr,
} }
cmd := framework.Command(resourceList, runner.Run) cmd := framework.Command(resourceList, runner.Run)
if err := cmd.Execute(); err != nil { if err := cmd.Execute(); err != nil {