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

Deprecate deploy_che.sh in favor of chectl / operatorhub #13888

Closed
6 tasks done
benoitf opened this issue Jul 18, 2019 · 5 comments
Closed
6 tasks done

Deprecate deploy_che.sh in favor of chectl / operatorhub #13888

benoitf opened this issue Jul 18, 2019 · 5 comments
Assignees
Labels
area/install Issues related to installation, including offline/air gap and initial setup kind/epic A long-lived, PM-driven feature request. Must include a checklist of items that must be completed. severity/P1 Has a major impact to usage or development of the system.

Comments

@benoitf
Copy link
Contributor

benoitf commented Jul 18, 2019

Is your task related to a problem? Please describe.

There is a deploy_che.sh script used to deploy che before chectl CLI was added.
Now that there are operatorhub and chectl this script should be deprecated and removed

Describe the solution you'd like

  • Gather all usage of deploy_che.sh script and
  • check if chectl is allowing the same kind of options
  • create issues if it's not the case

Describe alternatives you've considered

Open to suggestions

Additional context

chectl is supporting a widely range of deployments than deploy_che.sh

Subtasks

@benoitf benoitf added kind/task Internal things, technical debt, and to-do tasks to be performed. area/chectl Issues related to chectl, the CLI of Che labels Jul 18, 2019
@skabashnyuk
Copy link
Contributor

@benoitf wdyt can we extend this task to all content of https://github.com/eclipse/che/tree/master/deploy/openshift ? I believe that chectl + che-operator is the replace of this folder.

@benoitf
Copy link
Contributor Author

benoitf commented Jul 18, 2019

@skabashnyuk yes correct, IMHO it includes the whole folder

@skabashnyuk
Copy link
Contributor

@rhopp @vparfonov can you please check if these scripts/yamls are used in CI/CD or tests.
@davidfestal wdyt should we create any issue to implement some functionality that exists in /deploy/openshift but not exists in che-operator. For example, now it's including tracing/monitoring but afaik there is no such functionality in che-operator.

@l0rd l0rd added kind/epic A long-lived, PM-driven feature request. Must include a checklist of items that must be completed. area/install Issues related to installation, including offline/air gap and initial setup and removed kind/task Internal things, technical debt, and to-do tasks to be performed. labels Dec 12, 2019
@tolusha tolusha added this to the Backlog - Deploy milestone Dec 17, 2019
@l0rd
Copy link
Contributor

l0rd commented Dec 17, 2019

@tolusha I have added subtasks here although I think that instead of adding the support for every single che property here we should think about a generic solution. For example using a convention as looking for environment variables starting with CHE_ and using them as config maps.

@AndrienkoAleksandr AndrienkoAleksandr self-assigned this Dec 18, 2019
@tolusha tolusha added severity/P1 Has a major impact to usage or development of the system. team/deploy and removed area/chectl Issues related to chectl, the CLI of Che labels Dec 18, 2019
@tolusha
Copy link
Contributor

tolusha commented Dec 19, 2019

@rhopp
What are minimal requirements to use chectl instead of deploy.sh for CI jobs ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/install Issues related to installation, including offline/air gap and initial setup kind/epic A long-lived, PM-driven feature request. Must include a checklist of items that must be completed. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

5 participants