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.

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

Strange interaction with chectl #19128

Closed
3 of 14 tasks
tsmaeder opened this issue Feb 23, 2021 · 2 comments
Closed
3 of 14 tasks

Strange interaction with chectl #19128

tsmaeder opened this issue Feb 23, 2021 · 2 comments
Assignees
Labels
area/chectl Issues related to chectl, the CLI of Che kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Milestone

Comments

@tsmaeder
Copy link
Contributor

Describe the bug

I just had a really strange interaction with chectl:

C:\Users\thomas\code\chedev\che>chectl server:update
› Current Kubernetes context: 'docker-desktop'
A newer version of chectl is available.
To deploy the latest version of Eclipse Che you have to update chectl first [y/n]: y
Please run "chectl update" and then repeat "server:deploy" command.

There are a two strange things here:

  1. If you ask me to update chectl and ask for [y/n], I would expect the update to be executed instead of giving me instructions on how to do it manually
  2. I did not run "server:deploy", so the instruction is wrong.

Che version

chectl/7.26.0 win32-x64 node-v12.20.1

Expected behavior

Runtime

  • docker-desktop + K8S (include output of docker version and kubectl version)

Installation method

  • chectl
    • provide a full command that was used to deploy Eclipse Che (including the output)
    • provide an output of chectl version command
  • OperatorHub
  • I don't know

Environment

  • my computer
    • Windows
    • Linux
    • macOS
  • Cloud
    • Amazon
    • Azure
    • GCE
    • other (please specify)
  • other: please specify

Eclipse Che Logs

Additional context

@tsmaeder tsmaeder added kind/bug Outline of a bug - must adhere to the bug report template. area/chectl Issues related to chectl, the CLI of Che labels Feb 23, 2021
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Feb 23, 2021
@tolusha tolusha added the severity/P2 Has a minor but important impact to the usage or development of the system. label Feb 23, 2021
@RomanNikitenko RomanNikitenko removed the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Feb 23, 2021
@tolusha tolusha added this to the 7.28 milestone Feb 24, 2021
@tolusha tolusha added severity/P1 Has a major impact to usage or development of the system. and removed severity/P2 Has a minor but important impact to the usage or development of the system. labels Feb 24, 2021
@tolusha tolusha mentioned this issue Feb 25, 2021
70 tasks
@mmorhun mmorhun self-assigned this Mar 2, 2021
@tolusha
Copy link
Contributor

tolusha commented Mar 3, 2021

We are not going to update chectl automatically.
This issue is about make message more clear for user.

@tolusha
Copy link
Contributor

tolusha commented Mar 19, 2021

After some discussion we are going to provide a fix which updates chectl when user answered yes to a question.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/chectl Issues related to chectl, the CLI of Che kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

5 participants