Skip to content

Commit

Permalink
Merge pull request #2849 from weaveworks/fix-dashboard-default
Browse files Browse the repository at this point in the history
Actually default-yes for installing dashboard
  • Loading branch information
Robin Sonefors authored Oct 12, 2022
2 parents 6b2c60c + 13fed59 commit 105364c
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions cmd/gitops/beta/run/cmd.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,10 @@ import (
"os"
"os/signal"
"path/filepath"
"strings"
"sync/atomic"
"syscall"
"time"

"github.com/weaveworks/weave-gitops/pkg/kube"

"github.com/fsnotify/fsnotify"
"github.com/manifoldco/promptui"
"github.com/minio/minio-go/v7"
Expand All @@ -23,6 +20,7 @@ import (
clilogger "github.com/weaveworks/weave-gitops/cmd/gitops/logger"
"github.com/weaveworks/weave-gitops/pkg/fluxexec"
"github.com/weaveworks/weave-gitops/pkg/fluxinstall"
"github.com/weaveworks/weave-gitops/pkg/kube"
"github.com/weaveworks/weave-gitops/pkg/run"
"github.com/weaveworks/weave-gitops/pkg/run/bootstrap"
"github.com/weaveworks/weave-gitops/pkg/run/install"
Expand Down Expand Up @@ -346,8 +344,10 @@ func runCommandWithoutSession(cmd *cobra.Command, args []string) error {
Default: "Y",
}

result, err := prompt.Run()
if err == nil && strings.ToUpper(result) == "Y" {
// Answering "n" causes err to not be nil. Hitting enter without typing
// does not return the default.
_, err := prompt.Run()
if err == nil {
password, err := install.ReadPassword(log)
if err != nil {
cancel()
Expand Down

0 comments on commit 105364c

Please sign in to comment.