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

When installing che-operator via helm, the devfile workspace CRDs are not installed #23388

Open
mangelajo opened this issue Mar 18, 2025 · 1 comment
Labels
area/install Issues related to installation, including offline/air gap and initial setup kind/bug Outline of a bug - must adhere to the bug report template. kind/question Questions that haven't been identified as being feature requests or bugs. severity/P1 Has a major impact to usage or development of the system.

Comments

@mangelajo
Copy link

Describe the bug

Installing the che-operator via helm charts fails with:

2025-03-18T10:38:46Z	ERROR	controller-runtime.source	if kind is a CRD, it should be installed before calling Start	{"kind": "DevWorkspaceRouting.controller.devfile.io", "error": "no matches for kind \"DevWorkspaceRouting\" in version \"controller.devfile.io/v1alpha1\""}
sigs.k8s.io/controller-runtime/pkg/source.(*Kind).Start.func1.1
	/che-operator/vendor/sigs.k8s.io/controller-runtime/pkg/source/source.go:143
k8s.io/apimachinery/pkg/util/wait.runConditionWithCrashProtectionWithContext
	/che-operator/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:235
k8s.io/apimachinery/pkg/util/wait.WaitForWithContext
	/che-operator/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:662
k8s.io/apimachinery/pkg/util/wait.poll
	/che-operator/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:596
k8s.io/apimachinery/pkg/util/wait.PollImmediateUntilWithContext
	/che-operator/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:547
sigs.k8s.io/controller-runtime/pkg/source.(*Kind).Start.func1
	/che-operator/vendor/sigs.k8s.io/controller-runtime/pkg/source/source.go:136

The helm operator should offer an option to install those CRDs, there seems to be no helm chart to install the CRDs , that only leaves the option to hack/apply those in place.

Che version

7.10@latest

Steps to reproduce

  1. Install che-operator:
  - name: eclipse-che
    needs:
      - cert-manager/cert-manager
      - dex/dex
    chart: eclipse-che/eclipse-che
    namespace: eclipse-che
    createNamespace: true
    version: 7.100.0
    set:
     - name: networking.domain
       value: che.demo.jumpstarter.dev
     - name: networking.auth.oAuthSecret
       value: eclipse-che
     - name: networking.auth.oAuthClientName
       value: k8s-client
     - name: networking.auth.identityProviderURL
       value: https://keycloak.demo.jumpstarter.dev/realms/che
  1. the che-operator pod stays in crash loop because of the missing CRDs

Expected behavior

The operator works, all necessary CRDs are in place.

Runtime

Kubernetes (vanilla)

Screenshots

No response

Installation method

other (please specify in additional context)

Environment

other (please specify in additional context)

Eclipse Che Logs

Additional context

Using helm charts from https://eclipse-che.github.io/che-operator/charts

on a vanilla OVH/Kubernetes

@mangelajo mangelajo added the kind/bug Outline of a bug - must adhere to the bug report template. label Mar 18, 2025
@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 Mar 18, 2025
@dkwon17 dkwon17 added area/install Issues related to installation, including offline/air gap and initial setup kind/question Questions that haven't been identified as being feature requests or bugs. severity/P2 Has a minor but important impact to the 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 Mar 18, 2025
@tolusha tolusha added the help wanted Community, we are fully engaged on other issues. Feel free to take this one. We'll help you! label Mar 19, 2025
@tolusha
Copy link
Contributor

tolusha commented Mar 19, 2025

Hi @mangelajo
Thank you for reporting.

I don't think adding DevWorkspace operator resources into Eclipse Che helmcharts is a good idea.
From what I can see, there is nothing about DevWorkspace operator dependency in the README file [1]
We have to be clear, that DevWorkspace operator must be installed before deploying Eclipse Che helmcharts

[1] https://artifacthub.io/packages/helm/eclipse-che/eclipse-che#prerequisites

@tolusha tolusha 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. help wanted Community, we are fully engaged on other issues. Feel free to take this one. We'll help you! labels Mar 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/install Issues related to installation, including offline/air gap and initial setup kind/bug Outline of a bug - must adhere to the bug report template. kind/question Questions that haven't been identified as being feature requests or bugs. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

4 participants