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

Happy path checks fail for che-theia: "Command server:deploy failed" #20076

Closed
1 of 3 tasks
RomanNikitenko opened this issue Jul 5, 2021 · 6 comments
Closed
1 of 3 tasks
Assignees
Labels
area/ci CI build and releases, PR testing, & whitelabel/productization issues 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

@RomanNikitenko
Copy link
Member

Describe the bug

The issue was detected on che-theia PR's check.

In the logs I see the following:

'[11:47:28] Enable minikube ingress addon [started]\n' +
    '[11:48:28] Enable minikube ingress addon [failed]\n' +
    '[11:48:28] → Command timed out after 60000 milliseconds: minikube addons enable ingress\n' +
    'Timed out\n' +
    '  - Using image k8s.gcr.io/ingress-nginx/controller:v0.44.0\n' +
    '  - Using image docker.io/jettech/kube-webhook-certgen:v1.5.1\n' +
    '  - Using image docker.io/jettech/kube-webhook-certgen:v1.5.1\n' +
    '* Verifying ingress addon...\n' +
    '[11:48:28] ✈️  Minikube preflight checklist [failed]\n' +
    '[11:48:28] → Command timed out after 60000 milliseconds: minikube addons enable ingress\n' +
    'Timed out\n' +
    '  - Using image k8s.gcr.io/ingress-nginx/controller:v0.44.0\n' +
    '  - Using image docker.io/jettech/kube-webhook-certgen:v1.5.1\n' +
    '  - Using image docker.io/jettech/kube-webhook-certgen:v1.5.1\n' +
    '* Verifying ingress addon...',
  stderr: '    Error: Command server:deploy failed. Error log: \n' +
    '    /home/runner/.cache/chectl/error.log. Eclipse Che logs: \n' +
    '    /tmp/chectl-logs/1625485648085.',
  failed: true,
  timedOut: false,
  isCanceled: false,
  killed: false

Che version

  • latest
  • nightly
  • other: please specify

Steps to reproduce

I detected a problem on the che-theia PR's checks.
Please see eclipse-che/che-theia#1157

Screenshots

errors_che-theia

@RomanNikitenko RomanNikitenko added the kind/bug Outline of a bug - must adhere to the bug report template. label Jul 5, 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 Jul 5, 2021
@mmorhun
Copy link
Contributor

mmorhun commented Jul 5, 2021

Command timed out after 60000 milliseconds: minikube addons enable ingress

This is infrastructure issue

@RomanNikitenko
Copy link
Member Author

Restarting jobs few times didn't help, will try one more time + check if it depends somehow on minikube version.

@RomanNikitenko
Copy link
Member Author

It looks like it really depends somehow on the minikube version - switching to 1.21.0 fixes the problem...
I'm going to prepare a PR to che-theia to change the minikube version.

@ibuziuk ibuziuk added severity/P2 Has a minor but important impact to the usage or development of the system. area/ci CI build and releases, PR testing, & whitelabel/productization issues labels Jul 5, 2021
@ibuziuk
Copy link
Member

ibuziuk commented Jul 5, 2021

@RomanNikitenko looks like you are on the right track with the PR with a fix, assigning to you based on the draft PR

@ibuziuk ibuziuk removed the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Jul 5, 2021
@RomanNikitenko RomanNikitenko changed the title Command server:deploy failed Happy path checks fail for che-theia: "Command server:deploy failed" Jul 6, 2021
@RomanNikitenko RomanNikitenko 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 Jul 6, 2021
@RomanNikitenko
Copy link
Member Author

RomanNikitenko commented Jul 6, 2021

I have changed the severity to P1 as at the moment there is no way to merge any PR to che-theia without those happy path checks (the check from che-bot fails due to another issue).

@RomanNikitenko
Copy link
Member Author

should be fixed by eclipse-che/che-theia@9f2aaf9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci CI build and releases, PR testing, & whitelabel/productization issues 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

4 participants