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

Issues when a devfile is invalid and Che fails gracefully #21793

Closed
l0rd opened this issue Oct 31, 2022 · 1 comment · Fixed by eclipse-che/che-dashboard#664
Closed

Issues when a devfile is invalid and Che fails gracefully #21793

l0rd opened this issue Oct 31, 2022 · 1 comment · Fixed by eclipse-che/che-dashboard#664
Labels
area/dashboard 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. sprint/current

Comments

@l0rd
Copy link
Contributor

l0rd commented Oct 31, 2022

Describe the bug

I have been trying the feature "Handle gracefully when an invalid devfile is found in a git repository" and I have spotted a few problems.

1. If I "continue with the default devfile" then the resulting devworkspace doesn't have an editor nor an UDI container and as a result fails to start

$ kubectl get devworkspace vscode-switch-pt5r -o json | jq .spec.template.components
[
  {
    "attributes": {
      "che.eclipse.org/default-plugin": true
    },
    "name": "default-f00c9bc0a1587654ccb8",
    "plugin": {
      "uri": "https://static.developers.redhat.com/crw/stg/plugins/devworkspace-telemetry-woopra-plugin/0.0.1/devfile.yaml"
    }
  }
]

image

2. The error message is hard to read

image

Che version

7.55@latest

Steps to reproduce

Open https://che-dogfooding.apps.che-dev.x6e0.p1.openshiftapps.com/#https://github.com/l0rd/che-operator/tree/a4f1949c33ddab5f66c19c27a844af1c46aa0820 on your browser.

Expected behavior

  1. The workspace should start when I select "continue with the default devfile".
  2. I expect the default devWorkspce to have the default editor and default container (UDI).
  3. The error message should render well so that I am able to read it.

Additional context

I have been testing on the dogfooding instance.

@l0rd l0rd added the kind/bug Outline of a bug - must adhere to the bug report template. label Oct 31, 2022
@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 Oct 31, 2022
@l0rd l0rd added area/dashboard severity/P1 Has a major impact to 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 Oct 31, 2022
@olexii4 olexii4 self-assigned this Oct 31, 2022
@l0rd
Copy link
Contributor Author

l0rd commented Oct 31, 2022

I am using Firefox

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dashboard 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. sprint/current
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants