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

Using chectl with helm 3 #15235

Closed
5 of 7 tasks
kameshsampath opened this issue Nov 20, 2019 · 0 comments · Fixed by che-incubator/chectl#417
Closed
5 of 7 tasks

Using chectl with helm 3 #15235

kameshsampath opened this issue Nov 20, 2019 · 0 comments · Fixed by che-incubator/chectl#417
Labels
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.

Comments

@kameshsampath
Copy link
Contributor

When I try to use chectl with helm 3 , it fails with an error

I understood from @l0rd that Helm 3 does not use tiller

Che version

  • 7.4.0

Steps to reproduce

  1. Install latest helm from https://helm.sh/
  2. Install latest chectl (chectl/7.3.2 linux-x64 node-v10.17.0)
  3. Run chectl server:start --cheimage=eclipse/che-server:7.4.0 --platform=minikube

Expected behavior

Che server should start successfully, but fails with error

chectl server:start --cheimage=eclipse/che-server:7.4.0 --platform=minikube  ✔ Verify Kubernetes API...OK
  ✔ 👀  Looking for an already existing Che instance
    ✔ Verify if Che is deployed into namespace "che"...it is not
  ✔ ✈️  Minikube preflight checklist
    ✔ Verify if kubectl is installed
    ✔ Verify if minikube is installed
    ✔ Verify if minikube is running
    ↓ Start minikube [skipped]
      → Minikube is already running.
    ✔ Verify if minikube ingress addon is enabled
    ↓ Enable minikube ingress addon [skipped]
      → Ingress addon is already enabled.
    ✔ Retrieving minikube IP and domain for ingress URLs...192.168.39.39.nip.io.
  ❯ 🏃‍  Running Helm to install Che
    ✔ Verify if helm is installed
    ✔ Create Tiller Role Binding...it already exists.
    ✔ Create Tiller Service Account...it already exists.
    ✔ Create Tiller RBAC
    ✖ Create Tiller Service
      → error: E_COMMAND_FAILED
      Preparing Che Helm Chart
      Updating Helm Chart dependencies
      Deploying Che Helm Chart
Error: Command "helm init --service-account tiller --wait" failed with return code 1
stderr: Error: unknown flag: --service-account
stdout:
error: E_COMMAND_FAILED
    at HelmTasks.<anonymous> (/usr/local/lib/chectl/lib/tasks/installers/helm.js:213:23)
    at Generator.next (<anonymous>)
    at fulfilled (/usr/local/lib/chectl/node_modules/tslib/tslib.js:107:62)

Runtime

  • kubernetes
Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.0+724e12f93f", GitCommit:"420c3d6212a959907360fb0b6545f2f0ab173271", GitTreeState:"clean", BuildDate:"2019-10-22T23:36:44Z", GoVersion:"go1.12.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.0", GitCommit:"641856db18352033a0d96dbc99153fa3b27298e5", GitTreeState:"clean", BuildDate:"2019-03-25T15:45:25Z", GoVersion:"go1.12.1", Compiler:"gc", Platform:"linux/amd64"}
  • minikube
commit: 792dbf92a1de583fcee76f8791cff12e0c9440ad-dirty

Installation method

  • chectl chectl server:start --cheimage=eclipse/che-server:7.4.0 --platform=minikube

Environment

  • my computer
    • Windows
    • Linux
    • macOS

CC: @l0rd

@kameshsampath kameshsampath added the kind/bug Outline of a bug - must adhere to the bug report template. label Nov 20, 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 Nov 20, 2019
@l0rd l0rd added severity/P1 Has a major impact to usage or development of the system. team/devex and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Nov 20, 2019
benoitf added a commit to che-incubator/chectl that referenced this issue Nov 25, 2019
fix eclipse-che/che#15235

Change-Id: If48ecd53baaf4bdca70cfbbd3175c97f73592afd
Signed-off-by: Florent Benoit <[email protected]>
benoitf added a commit to che-incubator/chectl that referenced this issue Nov 27, 2019
fix eclipse-che/che#15235

Change-Id: If48ecd53baaf4bdca70cfbbd3175c97f73592afd
Signed-off-by: Florent Benoit <[email protected]>
benoitf added a commit to che-incubator/chectl that referenced this issue Nov 27, 2019
fix eclipse-che/che#15235

Change-Id: If48ecd53baaf4bdca70cfbbd3175c97f73592afd
Signed-off-by: Florent Benoit <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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

Successfully merging a pull request may close this issue.

3 participants