From 4db0085c8750555fac5ed5a8e2f6c0c08bec8406 Mon Sep 17 00:00:00 2001 From: fabiankramm Date: Tue, 18 Sep 2018 12:20:06 +0200 Subject: [PATCH 1/2] Resolve #108 --- cmd/up.go | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/cmd/up.go b/cmd/up.go index 5f0e0b422b..7bd7e4b81b 100644 --- a/cmd/up.go +++ b/cmd/up.go @@ -557,7 +557,12 @@ func (cmd *UpCmd) startSync() []*synctool.SyncConfig { labels = append(labels, key+"="+*value) } - pod, err := kubectl.GetFirstRunningPod(cmd.kubectl, strings.Join(labels, ", "), *config.DevSpace.Release.Namespace) + namespace := *config.DevSpace.Release.Namespace + if syncPath.Namespace != nil && *syncPath.Namespace != "" { + namespace = *syncPath.Namespace + } + + pod, err := kubectl.GetFirstRunningPod(cmd.kubectl, strings.Join(labels, ", "), namespace) if err != nil { log.Panicf("Unable to list devspace pods: %s", err.Error()) @@ -608,7 +613,12 @@ func (cmd *UpCmd) startPortForwarding() { labels = append(labels, key+"="+*value) } - pod, err := kubectl.GetFirstRunningPod(cmd.kubectl, strings.Join(labels, ", "), *config.DevSpace.Release.Namespace) + namespace := *config.DevSpace.Release.Namespace + if portForwarding.Namespace != nil && *portForwarding.Namespace != "" { + namespace = *portForwarding.Namespace + } + + pod, err := kubectl.GetFirstRunningPod(cmd.kubectl, strings.Join(labels, ", "), namespace) if err != nil { log.Errorf("Unable to list devspace pods: %s", err.Error()) From ffcb31fbfbd8e80c6b12a1c5eac88f9d2b8b96f2 Mon Sep 17 00:00:00 2001 From: fabiankramm Date: Tue, 18 Sep 2018 12:50:29 +0200 Subject: [PATCH 2/2] Fix reading from values.yaml --- cmd/init.go | 5 ++--- cmd/up.go | 10 +++++++--- pkg/util/yamlutil/yaml.go | 2 +- 3 files changed, 10 insertions(+), 7 deletions(-) diff --git a/cmd/init.go b/cmd/init.go index 0ae645f450..a9a2471ac7 100644 --- a/cmd/init.go +++ b/cmd/init.go @@ -153,12 +153,11 @@ func (cmd *InitCmd) Run(cobraCmd *cobra.Command, args []string) { createChart := cmd.flags.overwrite if !cmd.flags.overwrite { - _, dockerfileNotFound := os.Stat(cmd.workdir + "/Dockerfile") _, chartDirNotFound := os.Stat(cmd.workdir + "/chart") - if dockerfileNotFound == nil || chartDirNotFound == nil { + if chartDirNotFound == nil { overwriteAnswer := stdinutil.GetFromStdin(&stdinutil.GetFromStdinParams{ - Question: "Do you want to overwrite the Dockerfile and the existing files in /chart? (yes | no)", + Question: "Do you want to overwrite the existing files in /chart? (yes | no)", DefaultValue: "no", ValidationRegexPattern: "^(yes)|(no)$", }) diff --git a/cmd/up.go b/cmd/up.go index 7bd7e4b81b..6840186be0 100644 --- a/cmd/up.go +++ b/cmd/up.go @@ -437,7 +437,10 @@ func (cmd *UpCmd) deployChart() { values := map[interface{}]interface{}{} overwriteValues := map[interface{}]interface{}{} - yamlutil.ReadYamlFromFile(chartPath, values) + err := yamlutil.ReadYamlFromFile(chartPath+"values.yaml", values) + if err != nil { + log.Fatalf("Couldn't deploy chart, error reading from chart values %s: %v", chartPath+"values.yaml", err) + } containerValues := map[string]interface{}{} @@ -449,14 +452,15 @@ func (cmd *UpCmd) deployChart() { container["command"] = []string{"sleep"} container["args"] = []string{"99999999"} } + containerValues[imageName] = container } - pullSecrets := []string{} + pullSecrets := []interface{}{} existingPullSecrets, pullSecretsExisting := values["pullSecrets"] if pullSecretsExisting { - pullSecrets = existingPullSecrets.([]string) + pullSecrets = existingPullSecrets.([]interface{}) } for _, registryConf := range *config.Registries { diff --git a/pkg/util/yamlutil/yaml.go b/pkg/util/yamlutil/yaml.go index 01d4b3f5f5..79f3c6f8ff 100644 --- a/pkg/util/yamlutil/yaml.go +++ b/pkg/util/yamlutil/yaml.go @@ -20,9 +20,9 @@ func WriteYamlToFile(yamlData interface{}, filePath string) error { //ReadYamlFromFile reads a yaml file func ReadYamlFromFile(filePath string, yamlTarget interface{}) error { yamlFile, err := ioutil.ReadFile(filePath) - if err != nil { return err } + return yaml.Unmarshal(yamlFile, yamlTarget) }