Merge pull request #616 from openziti/fix-cli-config-dir

Don't set envs for all commands, only set for create config commands

ziti/cmd/ziti/cmd/create_config.go

@@ -165,14 +165,9 @@ func NewCmdCreateConfig() *cobra.Command {
 		},
 	}
 
-	// Get env variable data global to all config files
-	templateData := &ConfigTemplateValues{}
-	templateData.populateEnvVars()
-	templateData.populateDefaults()
-
-	cmd.AddCommand(NewCmdCreateConfigController(templateData))
-	cmd.AddCommand(NewCmdCreateConfigRouter(templateData))
-	cmd.AddCommand(NewCmdCreateConfigEnvironment(templateData))
+	cmd.AddCommand(NewCmdCreateConfigController())
+	cmd.AddCommand(NewCmdCreateConfigRouter())
+	cmd.AddCommand(NewCmdCreateConfigEnvironment())
 
 	return cmd
 }

ziti/cmd/ziti/cmd/create_config_controller.go

@@ -67,9 +67,12 @@ type CreateConfigControllerOptions struct {
 }
 
 // NewCmdCreateConfigController creates a command object for the "create" command
-func NewCmdCreateConfigController(data *ConfigTemplateValues) *cobra.Command {
+func NewCmdCreateConfigController() *cobra.Command {
 	controllerOptions := &CreateConfigControllerOptions{}
 
+	// Get env variable data global to all config files
+	data := &ConfigTemplateValues{}
+
 	cmd := &cobra.Command{
 		Use:     "controller",
 		Short:   "Create a controller config",
@@ -90,6 +93,9 @@ func NewCmdCreateConfigController(data *ConfigTemplateValues) *cobra.Command {
 				logrus.SetOutput(logOut)
 			}
 
+			data.populateEnvVars()
+			data.populateDefaults()
+
 			// Update controller specific values with configOptions passed in if the argument was provided or the value is currently blank
 			if data.Controller.Port == "" || controllerOptions.CtrlPort != constants.DefaultZitiControllerPort {
 				data.Controller.Port = controllerOptions.CtrlPort

ziti/cmd/ziti/cmd/create_config_environment.go

@@ -110,7 +110,11 @@ type CreateConfigEnvironmentOptions struct {
 }
 
 // NewCmdCreateConfigEnvironment creates a command object for the "environment" command
-func NewCmdCreateConfigEnvironment(data *ConfigTemplateValues) *cobra.Command {
+func NewCmdCreateConfigEnvironment() *cobra.Command {
+
+	// Get env variable data global to all config files
+	data := &ConfigTemplateValues{}
+
 	environmentOptions := &CreateConfigEnvironmentOptions{
 		ConfigTemplateValues: *data,
 		EnvVariableMetaData: EnvVariableMetaData{
@@ -171,6 +175,9 @@ func NewCmdCreateConfigEnvironment(data *ConfigTemplateValues) *cobra.Command {
 		Long:    createConfigEnvironmentLong,
 		Example: createConfigEnvironmentExample,
 		PreRun: func(cmd *cobra.Command, args []string) {
+			data.populateEnvVars()
+			data.populateDefaults()
+
 			// Setup logging
 			var logOut *os.File
 			environmentOptions.OS = runtime.GOOS

ziti/cmd/ziti/cmd/create_config_router.go

@@ -37,9 +37,12 @@ type CreateConfigRouterOptions struct {
 }
 
 // NewCmdCreateConfigRouter creates a command object for the "router" command
-func NewCmdCreateConfigRouter(data *ConfigTemplateValues) *cobra.Command {
+func NewCmdCreateConfigRouter() *cobra.Command {
 	options := &CreateConfigRouterOptions{}
 
+	// Get env variable data global to all config files
+	data := &ConfigTemplateValues{}
+
 	cmd := &cobra.Command{
 		Use:     "router",
 		Short:   "Creates a config file for specified Router name",
@@ -58,6 +61,9 @@ func NewCmdCreateConfigRouter(data *ConfigTemplateValues) *cobra.Command {
 				logrus.SetOutput(logOut)
 			}
 
+			data.populateEnvVars()
+			data.populateDefaults()
+
 			// Update router data with options passed in
 			data.Router.Name = options.RouterName
 			SetZitiRouterIdentity(&data.Router, data.Router.Name)