-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Allow an admin to specify Che default editor and containers #21353
Comments
Another option for the new field in the CR is to use (...)
workspaces:
(...)
defaultComponents:
- name: universal-developer-image
container:
image: quay.io/devfile/universal-developer-image:ubi8-latest
memoryRequest: 256M
memoryLimit: 1536M
cpuRequest: 0.1
cpuLimit: 0.5
- name: some-volume
volume: {}
defaultEditor: eclipse/che-theia/next
(...) The benefit of this approach would be that since both the Che Operator and Dashboard depend on devfile/api already, these components could be easily deserialized by both as needed and just added to empty devfiles directly, under |
@olexii4 the che operator PR has been merged and I am wondering why this issue is still open. Is it because you are working on a dashboard PR? I am asking because we would like to include it in the new and noteworthy features of this release. |
PR was merged yesterday in the evening. I waited until the next image has built. I tried to test it. Today I have deployed CHE on minikube but cannot see changes that should be in che-operator. I am investigating the problem. I will do my best to do it ASAP. |
add comment to reset sync date, so we should get a new message here that collects the PRs. |
comemnt |
The beauty of automation 😂 |
Is your enhancement related to a problem? Please describe
If no devfile is present in a git repository the default is to start a workspace with only one component (the Theia container) and Theia as the editor.
That's a bad UX as the Theia container misses many langue tools and because an admin cannot change this behavior.
Describe the solution you'd like
The default container should be a UDI container. Both the default container and the default editor should be configurable in the
CheCluster
CR.Additional Context
This is a #21340 subtask.
Release Notes Text
Che administrators can now configure the default editor (Theia, VS Code, IntelliJ or PyCharm) and the default container. Those defaults are used when the developer doesn't specify an editor or doesn't specify a devfile. The configuration can be done in the
CheCluster
Custom Resource in the theworkspaces
section using the just introduceddefaultComponent
anddefaultEditor
fields.The text was updated successfully, but these errors were encountered: