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

server:start doesn't work if Che has already been deployed #14465

Closed
benoitf opened this issue Sep 9, 2019 · 3 comments
Closed

server:start doesn't work if Che has already been deployed #14465

benoitf opened this issue Sep 9, 2019 · 3 comments
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/P2 Has a minor but important impact to the usage or development of the system. status/in-progress This issue has been taken by an engineer and is under active development.

Comments

@benoitf
Copy link
Contributor

benoitf commented Sep 9, 2019

From chectl created by l0rd: che-incubator/chectl#77

If Che is already deployed the flow should be different:

a) if Che is deployed and running: returns saying that Che is already running
b) if Che is deployed and stopped: starts Che and return saying that Che was already deployed

There is also #32 that is about these 2 flows but extending them using the flag --if-already-exist flag.

@benoitf benoitf added the bug label Sep 9, 2019
@benoitf
Copy link
Contributor Author

benoitf commented Sep 9, 2019

@l0rd I've already made some changes as well. Thanks for sharing your changes, will take a look and probably reuse some parts.

@benoitf
Copy link
Contributor Author

benoitf commented Sep 9, 2019

@sleshchenko I am perfectly fine if you are work on this issue but I started working on it a few months ago: l0rd/chectl@6682c5c. I wasn't happy with that preliminary work (too complicated) and never actually created a PR. Unassigning it to myself.

@benoitf
Copy link
Contributor Author

benoitf commented Sep 9, 2019

The work is suspended in the flavor of eclipse-che/che-operator#71

@benoitf benoitf added area/chectl Issues related to chectl, the CLI of Che kind/bug Outline of a bug - must adhere to the bug report template. and removed bug labels Sep 9, 2019
@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 Sep 9, 2019
@sleshchenko sleshchenko added status/in-progress This issue has been taken by an engineer and is under active development. team/platform labels Sep 9, 2019
@ibuziuk ibuziuk added severity/P2 Has a minor but important impact to the usage or development of the system. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Sep 9, 2019
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/P2 Has a minor but important impact to the usage or development of the system. status/in-progress This issue has been taken by an engineer and is under active development.
Projects
None yet
Development

No branches or pull requests

4 participants