Skip to content

Commit 142e3b2

Browse files
improved values yaml (#202)
* improved values yaml
1 parent dd7f344 commit 142e3b2

20 files changed

+358
-154
lines changed

venonactl/cmd/install-app-proxy.go

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ var installAppProxyCmdOptions struct {
3737
resources map[string]interface{}
3838
host string
3939
ingressClass string
40+
dryRun bool
4041
}
4142

4243
var installAppProxyCmd = &cobra.Command{
@@ -55,10 +56,9 @@ var installAppProxyCmd = &cobra.Command{
5556
mergeValueStr(templateValuesMap, "CodefreshHost", &cfAPIHost)
5657
mergeValueStr(templateValuesMap, "Namespace", &installAppProxyCmdOptions.kube.namespace)
5758
mergeValueStr(templateValuesMap, "Context", &installAppProxyCmdOptions.kube.context)
58-
mergeValueStr(templateValuesMap, "AppProxy.Host", &installAppProxyCmdOptions.host)
59-
mergeValueStr(templateValuesMap, "AppProxy.IngressClass", &installAppProxyCmdOptions.ingressClass)
59+
mergeValueStr(templateValuesMap, "AppProxy.Ingress.Host", &installAppProxyCmdOptions.host)
60+
mergeValueStr(templateValuesMap, "AppProxy.Ingress.IngressClass", &installAppProxyCmdOptions.ingressClass)
6061
mergeValueStr(templateValuesMap, "DockerRegistry", &installAppProxyCmdOptions.dockerRegistry)
61-
6262
mergeValueMSI(templateValuesMap, "AppProxy.resources", &installAppProxyCmdOptions.resources)
6363

6464
s := store.GetStore()
@@ -73,6 +73,7 @@ var installAppProxyCmd = &cobra.Command{
7373
}
7474
builderInstallOpt := &plugins.InstallOptions{
7575
CodefreshHost: cfAPIHost,
76+
DryRun: installAppProxyCmdOptions.dryRun,
7677
}
7778
s.AppProxy.Resources = installAppProxyCmdOptions.resources
7879
s.DockerRegistry = installAppProxyCmdOptions.dockerRegistry
@@ -112,5 +113,7 @@ func init() {
112113
installAppProxyCmd.Flags().StringVar(&installAppProxyCmdOptions.kube.namespace, "kube-namespace", viper.GetString("kube-namespace"), "Name of the namespace on which venona should be installed [$KUBE_NAMESPACE]")
113114
installAppProxyCmd.Flags().StringVar(&installAppProxyCmdOptions.kube.context, "kube-context-name", viper.GetString("kube-context"), "Name of the kubernetes context on which venona should be installed (default is current-context) [$KUBE_CONTEXT]")
114115
installAppProxyCmd.Flags().StringArrayVarP(&installAppProxyCmdOptions.templateValueFiles, "values", "f", []string{}, "specify values in a YAML file")
116+
installAppProxyCmd.Flags().StringArrayVar(&installAppProxyCmdOptions.templateFileValues, "set-file", []string{}, "Set values for templates from file")
115117
installAppProxyCmd.Flags().StringArrayVar(&installAppProxyCmdOptions.templateValues, "set-value", []string{}, "Set values for templates, example: --set-value LocalVolumesDir=/mnt/disks/ssd0/codefresh-volumes")
118+
installAppProxyCmd.Flags().BoolVar(&installAppProxyCmdOptions.dryRun, "dry-run", false, "Set to true to simulate installation")
116119
}

venonactl/cmd/install-monitor.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ var installMonitorAgentCmdOptions struct {
4040
templateFileValues []string
4141
templateValueFiles []string
4242
resources map[string]interface{}
43+
dryRun bool
4344
}
4445

4546
// installK8sAgentCmd represents the install command
@@ -83,6 +84,7 @@ var installMonitorAgentCmd = &cobra.Command{
8384

8485
builderInstallOpt := &plugins.InstallOptions{
8586
ClusterNamespace: s.KubernetesAPI.Namespace,
87+
DryRun: installMonitorAgentCmdOptions.dryRun,
8688
}
8789

8890
builderInstallOpt.KubeBuilder = getKubeClientBuilder(s.KubernetesAPI.ContextName, s.KubernetesAPI.Namespace, s.KubernetesAPI.ConfigPath, s.KubernetesAPI.InCluster)
@@ -137,7 +139,7 @@ func init() {
137139
installMonitorAgentCmd.Flags().StringVar(&installMonitorAgentCmdOptions.dockerRegistry, "docker-registry", "", "The prefix for the container registry that will be used for pulling the required components images. Example: --docker-registry=\"docker.io\"")
138140

139141
installMonitorAgentCmd.Flags().StringVar(&installMonitorAgentCmdOptions.codefreshHost, "codefreshHost", "", "Override codefresh host if you use your own codefresh installation")
140-
142+
installMonitorAgentCmd.Flags().BoolVar(&installMonitorAgentCmdOptions.dryRun, "dry-run", false, "Set to true to simulate installation")
141143
installMonitorAgentCmd.Flags().BoolVar(&installMonitorAgentCmdOptions.helm3, "helm3", false, "Set flag if cluster use helm3")
142144
installMonitorAgentCmd.Flags().StringArrayVar(&installMonitorAgentCmdOptions.templateValues, "set-value", []string{}, "Set values for templates, example: --set-value LocalVolumesDir=/mnt/disks/ssd0/codefresh-volumes")
143145
installMonitorAgentCmd.Flags().StringArrayVar(&installMonitorAgentCmdOptions.templateFileValues, "set-file", []string{}, "Set values for templates from file, example: --set-file Storage.GoogleServiceAccount=/path/to/service-account.json")

0 commit comments

Comments
 (0)