Meshery: Issue running meshery installation script in quick start guide

[pranav-bhatt]

Description

The Meshery installation bash script ends with the line mesheryctl system start. When run for the first time, the output generated is as shown below:

Extracting mesheryctl-v0.5.0-beta-1...
Archive:  /usr/bin/meshery.zip
  inflating: LICENSE                 
  inflating: README.md               
  inflating: mesheryctl              

Installing mesheryctl in /usr/local/bin.
mesheryctl installed.
permissions moved to pranav
Removing installation files and opening Meshery...Missing Meshery config file.
Create default config now [y/n]? EOF

Expected Behavior

Instead of waiting for user input (y/N), the script reads EOF, and exits.

Line 99 should be moved out of the script, and the tutorials/docs should explicitly tell the user to run the mesheryctl system start command.

Environment:

• Host OS: Ubuntu 20.04
• Meshery Version: 0.5.0-beta-1
• Kubernetes Version: 1.20.2
• Browser: Firefox 84.0.2

To Reproduce

Steps to reproduce the behavior:

1. Follow steps in the Meshery quick start guide. Issue arises on the "Install Meshery" step.

[leecalcote]

Fixed by https://github.com/layer5io/meshery/pull/2298

[navendu-pottekkat]

@pranav-bhatt The current flow is that when you run the bash script for installation, it would configure everything and start meshery server on your machine.

But, when system start is first executed, there will not be any config.yaml file in your ~/home/.meshery folder. So, it shows a prompt and the user should respond "y". This will create a new default config.yaml file and then meshery will proceed to start normally.

We have recently "silenced" this user confirmation through this PR from @hexxdump

So, after this PR is merged, when installing through the bash script, it will not ask confirmation from the user and would proceed to create the config.yaml file.

[leecalcote]

2298 should fix the defect described here. The change described in Expected Behavior, though, is slightly askew from our goal: short time to value; get users using Meshery quickly.

This installer is aimed at users looking to run Meshery. We want to make that happen fast. If we can delight users along the way, all the better. We really want to delight users with a single step download, install, deploy, run Meshery experience. See the Preface and Guiding Principles of Command Design in the Meshery CLI & Documentation.

As an option for advanced users, consideration for a skip-run (or similar) option in the bash script to forego starting mesheryctl could be made.

[navendu-pottekkat]

@pranav-bhatt Any updates on this issue? Were you able to work it out?

[pranav-bhatt]

Still facing the EOF issue, despite using the code in the mentioned PR. @leecalcote mentioned a PR by "The Piscean" (not sure what their GH username is) that fixes this though.

[navendu-pottekkat]

Hmm, this might be an issue. Although I have almost the exact same setup as you apart from Kubernetes version 1.20.1. We will need to look into it.

@hexxdump Could you please check this?

[hexxdump]

Still facing the EOF issue, despite using the code in the mentioned PR. @leecalcote mentioned a PR by "The Piscean" (not sure what their GH username is) that fixes this though.

@pranav-bhatt, Even after #2298 fix, you are hitting this issue?

[pranav-bhatt]

Sorry for the late reply. I tested this on various other platforms, and ignoring the fact that the release pulled doesn't have silent flag, it does the trick!