Skip to content

Commit 7191ce5

Browse files
author
gentele
committed
improve useKubeConfig handlung
1 parent 255a964 commit 7191ce5

3 files changed

Lines changed: 16 additions & 13 deletions

File tree

cmd/init.go

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,11 @@ import (
2121

2222
// InitCmd is a struct that defines a command call for "init"
2323
type InitCmd struct {
24-
flags *InitCmdFlags
25-
workdir string
26-
chartGenerator *generator.ChartGenerator
27-
config *v1.Config
24+
flags *InitCmdFlags
25+
workdir string
26+
chartGenerator *generator.ChartGenerator
27+
config *v1.Config
28+
overwriteConfig *v1.Config
2829
}
2930

3031
// InitCmdFlags are the flags available for the init-command
@@ -119,11 +120,9 @@ func (cmd *InitCmd) Run(cobraCmd *cobra.Command, args []string) {
119120
Image: &v1.ImageConfig{
120121
Name: configutil.String("devspace"),
121122
},
122-
Cluster: &v1.Cluster{
123-
APIServer: configutil.String("https://192.168.99.100:8443"),
124-
User: &v1.User{},
125-
},
126123
})
124+
cmd.overwriteConfig = configutil.GetOverwriteConfig()
125+
127126
cmd.initChartGenerator()
128127

129128
createChart := cmd.flags.overwrite
@@ -279,7 +278,7 @@ func (cmd *InitCmd) addDefaultSyncConfig() {
279278
}
280279

281280
func (cmd *InitCmd) reconfigure() {
282-
clusterConfig := cmd.config.Cluster
281+
clusterConfig := cmd.overwriteConfig.Cluster
283282
tillerConfig := cmd.config.Services.Tiller
284283
tillerRelease := tillerConfig.Release
285284

@@ -318,6 +317,9 @@ func (cmd *InitCmd) reconfigure() {
318317
clusterConfig.UseKubeConfig = configutil.Bool(useKubeConfig)
319318

320319
if !useKubeConfig {
320+
if clusterConfig.APIServer == nil {
321+
clusterConfig.APIServer = configutil.String("https://192.168.99.100:8443")
322+
}
321323
clusterConfig.APIServer = stdinutil.GetFromStdin(&stdinutil.GetFromStdinParams{
322324
Question: "What is your Kubernetes API Server URL? (e.g. https://127.0.0.1:8443)",
323325
DefaultValue: *clusterConfig.APIServer,
@@ -354,7 +356,6 @@ func (cmd *InitCmd) reconfigure() {
354356
}
355357

356358
func (cmd *InitCmd) reconfigureRegistry() {
357-
overwriteConfig := configutil.GetOverwriteConfig()
358359
registryConfig := cmd.config.Services.Registry
359360

360361
enableAutomaticBuilds := stdinutil.GetFromStdin(&stdinutil.GetFromStdinParams{
@@ -389,7 +390,7 @@ func (cmd *InitCmd) reconfigureRegistry() {
389390
if registryConfig.Internal.Release.Namespace == nil {
390391
registryConfig.Internal.Release.Namespace = cmd.config.DevSpace.Release.Namespace
391392
}
392-
registryUser := overwriteConfig.Services.Registry.User
393+
registryUser := cmd.overwriteConfig.Services.Registry.User
393394

394395
if registryUser.Username == nil {
395396
randomUserSuffix, err := randutil.GenerateRandomString(5)

pkg/devspace/config/configutil/get.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ func GetConfig(reload bool) *v1.Config {
5959
merge(config, overwriteConfig, unsafe.Pointer(&config), unsafe.Pointer(overwriteConfig))
6060
}
6161

62-
if config.Cluster.UseKubeConfig != nil && *config.Cluster.UseKubeConfig {
62+
if (config.Cluster.UseKubeConfig != nil && *config.Cluster.UseKubeConfig) || config.Cluster.APIServer == nil {
6363
loadClusterConfig(config.Cluster, false)
6464
}
6565
return config

pkg/devspace/config/configutil/make.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
package configutil
22

3-
import "github.com/covexo/devspace/pkg/devspace/config/v1"
3+
import (
4+
"github.com/covexo/devspace/pkg/devspace/config/v1"
5+
)
46

57
func makeConfig() *v1.Config {
68
return &v1.Config{

0 commit comments

Comments
 (0)