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

Timeout wait on chectl when deploying Che with Minishift addon #14136

Closed
5 of 23 tasks
benoitf opened this issue Aug 6, 2019 · 2 comments · Fixed by che-incubator/chectl#242
Closed
5 of 23 tasks

Timeout wait on chectl when deploying Che with Minishift addon #14136

benoitf opened this issue Aug 6, 2019 · 2 comments · Fixed by che-incubator/chectl#242
Assignees
@benoitf
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. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community.
Milestone
7.0.0

Comments

@benoitf
Copy link
Contributor

benoitf commented Aug 6, 2019

Describe the bug

Chectl is waiting for a Devfile registry pod but there is no such pod with minishift-addon.

Che version

  • latest
  • nightly
  • other: please specify

Steps to reproduce

$ chectl server:start --installer=minishift-addon --platform=minishift
  ✔ ✈️  Minishift preflight checklist
    ✔ Verify if oc is installed...done.
    ✔ Verify if minishift is installed...done.
    ✔ Verify if minishift is running...done.
    ✔ Retrieving minishift IP and domain for routes URLs...192.168.99.130.nip.io.
  ❯ 🏃‍  Running the Che minishift-addon
    ✔ Check minishift version...done.
  ✔ 🏃‍  Running the Che minishift-addon
    ✔ Check minishift version...done.
    ✔ Check logged...done.
    ✔ Check che addon is available
    ✔ Apply Che addon...done.
  ❯ ✅  Post installation checklist
    ❯ Devfile registry pod bootstrap
      ⠏ scheduling
        downloading images
        starting
      Plugin registry pod bootstrap
      Che pod bootstrap
      Retrieving Che Server URL
      Che status check

Expected behavior

startup should be fine

Runtime

  • kubernetes (include output of kubectl version)
  • Openshift (include output of oc version)
  • minikube (include output of minikube version and kubectl version)
  • minishift 1.34.1
  • docker-desktop + K8S (include output of docker version and kubectl version)
  • other: (please specify)

Screenshots

Installation method

  • chectl
  • che-operator
  • minishift-addon
  • I don't know

Environment

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

Additional context
@benoitf benoitf added severity/P1 Has a major impact to usage or development of the system. area/chectl Issues related to chectl, the CLI of Che labels Aug 6, 2019
@benoitf
Copy link
Contributor Author

benoitf commented Aug 6, 2019

Introduced by che-incubator/chectl#228 and che-incubator/chectl#189

@benoitf benoitf added the kind/bug Outline of a bug - must adhere to the bug report template. label Aug 6, 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 Aug 6, 2019
benoitf added a commit to che-incubator/chectl that referenced this issue Aug 6, 2019
@benoitf
fix(minishift-addon): do not wait for local registries
71707d0 
fix eclipse-che/che#14136
Chectl was waiting for ever the pod

Change-Id: I4a1b61104842e9433350fe08444a7fe5165e68de
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@benoitf benoitf mentioned this issue Aug 6, 2019
Merged
@slemeur slemeur added this to the 7.0.0 milestone Aug 6, 2019
@slemeur
Copy link
Contributor

slemeur commented Aug 6, 2019

needed for #13889 -> milestone 7.0.0

@slemeur slemeur removed the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Aug 6, 2019
@benoitf benoitf self-assigned this Aug 7, 2019
@benoitf benoitf added the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Aug 7, 2019
benoitf added a commit to che-incubator/chectl that referenced this issue Aug 9, 2019
@benoitf
fix(minishift-addon): do not wait for local registries
3bed253 
fix eclipse-che/che#14136
Chectl was waiting for ever the pod

Change-Id: I4a1b61104842e9433350fe08444a7fe5165e68de
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@benoitf benoitf

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. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community.
Projects
None yet
Milestone
7.0.0
Development

Successfully merging a pull request may close this issue.

feat(minishift-addon): Embed minishift addon che-incubator/chectl
3 participants
@benoitf @slemeur @che-bot