weaveworks / launcher

Weave Cloud Launcher
Apache License 2.0
10 stars 13 forks source link

don't prompt #86

Closed rade closed 6 years ago

rade commented 6 years ago

ATM we present the user with a wall of text and a prompt.

$ curl -Ls https://get.weave.works | sh -s -- --token=XXXXXXXX                                
Downloading the Weave Cloud installer...

This will install Weave Cloud on the following cluster:
    Name: launcher-tests
    Server: https://192.168.99.100:8443

Please run 'kubectl config use-context' or pass '--kubeconfig' if you would like to change this.

Would you like to continue? [y/n]: y

Yes, I can see why we want to be careful here, but it introduces a friction point. Let's get rid of the prompt and simply tell the user what cluster we are installing things on. And cut down the text. A simple

Installing Weave Cloud on launcher-tests

suffices.

rade commented 6 years ago

perhaps Installing Weave Cloud agents on blah - we don't want to confuse users into thinking this is some sort of on-prem install of Weave Cloud.