weiguoqiang/cert-manager
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Merge pull request #983 from munnerz/e2e-bazel-helm

Browse Source 
Fix using Bazel built helm during e2e tests
This commit is contained in:
jetstack-bot 2018-10-23 01:52:48 +01:00 committed by GitHub
commit 4d4cb4d379
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
test/e2e/framework/addon/chart/addon.go
View File

@ -84,6 +84,10 @@ type Details struct {
func (c *Chart) Setup(cfg *config.Config) error {
var err error
c.config = cfg
if c.config.Addons.Helm.Path == "" {
return fmt.Errorf("--helm-binary-path must be set")
}
if c.Tiller == nil {
return fmt.Errorf("tiller base addon must be provided")
}
@ -168,7 +172,7 @@ func (c *Chart) buildHelmCmd(args ...string) *exec.Cmd {
"--kube-context", c.tillerDetails.KubeContext,
"--tiller-namespace", c.tillerDetails.Namespace,
}, args...)
cmd := exec.Command("helm", args...)
cmd := exec.Command(c.config.Addons.Helm.Path, args...)
cmd.Stdout = os.Stdout
cmd.Stderr = os.Stderr
return cmd