Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Log better error messages in CLI #549

Closed
mxyng opened this issue Oct 28, 2021 · 0 comments
Closed

Log better error messages in CLI #549

mxyng opened this issue Oct 28, 2021 · 0 comments
Assignees
@BruceMacD
Labels
area/cli Issue or PR related to the command-line interface. kind/bug A report or a fix for a problem with functional correctness.

Comments

@mxyng
Copy link
Collaborator

mxyng commented Oct 28, 2021 •
edited by BruceMacD
Loading

Add increased debug logging to the CLI and log suggested fixes to common issues.

Known issues:

  • When the Infra CLI and server are out of sync, and the server response doesn't recognize a changed message body:
...Could not verify certificate for host localhost
? Are you sure you want to continue? Yes
Error: undefined response type
  • Fix is to update the CLI version.
@mxyng mxyng mentioned this issue Oct 28, 2021
Merged
@jmorganca jmorganca added the area/cli Issue or PR related to the command-line interface. label Oct 29, 2021
@mxyng mxyng mentioned this issue Nov 1, 2021
Closed
@jmorganca jmorganca changed the title Log better debug messages in CLI Log better error messages in CLI Nov 25, 2021
@jmorganca jmorganca added the kind/bug A report or a fix for a problem with functional correctness. label Nov 25, 2021
@BruceMacD BruceMacD self-assigned this Nov 29, 2021
BruceMacD added a commit that referenced this issue Nov 30, 2021
@BruceMacD
Log better error messages in CLI (#549)
16e8241 
- Add context to error messages in the CLI so that the user has actions to take
- Wrap error outputs in red formatting to make them stand out
- Capitalize user facing errors
BruceMacD added a commit that referenced this issue Nov 30, 2021
@BruceMacD
Log better error messages in CLI (#549)
0a121d5 
- Add context to error messages in the CLI so that the user has actions to take
- Wrap error outputs in red formatting to make them stand out
- Capitalize user facing errors
@BruceMacD BruceMacD mentioned this issue Nov 30, 2021
Merged
6 tasks
BruceMacD added a commit that referenced this issue Dec 1, 2021
@BruceMacD
Log better error messages in CLI (#549)
b25c3f0 
- Add context to error messages in the CLI so that the user has actions to take
- Wrap error outputs in red formatting to make them stand out
- Capitalize user facing errors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BruceMacD BruceMacD

Labels
area/cli Issue or PR related to the command-line interface. kind/bug A report or a fix for a problem with functional correctness.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jmorganca @mxyng @BruceMacD