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

install via helmfile doesn't install CRDs #525

Closed
amilkov3 opened this issue Sep 13, 2023 · 3 comments
Closed

install via helmfile doesn't install CRDs #525

amilkov3 opened this issue Sep 13, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@amilkov3
Copy link

A clear and concise description of what the bug is.

adding keda as a helmfile release and doing helmfile apply doesnt install CRDS but manually doing a helm install keda kedacore/keda --namespace keda --create-namespace and then removing keda from the helmfile followed by a helmfile apply does install CRDs

i've run into similar behavior with vanilla helm3. manually adding a crds dir and overriding the necessary annotations mitigates the issue but its sloppy

Expected Behavior

installs the CRDs

Actual Behavior

unable to build kubernetes objects from release manifest: resource mapping not found for name: "cron-server-scale" namespace: "keda" from "": no matches for kind "ScaledObject" in version "keda.sh/v1alpha1"
  ensure CRDs are installed first

Steps to Reproduce the Problem

repositories:
  - name: kedacore
    url: https://kedacore.github.io/charts

releases:
  - name: keda
    namespace: keda
    chart: kedacore/keda

Specifications

  • KEDA Version: latest
  • Platform & Version: Please elaborate
  • Kubernetes Version: 1.28
  • Scaler(s): Please elaborate
@amilkov3 amilkov3 added the bug Something isn't working label Sep 13, 2023
@JorTurFer
Copy link
Member

do you have any idea about this part @zroubalik ?

@pragmaticivan
Copy link
Contributor

I'm not able to replicate this one, are we good with closing this issue?

❯ helm version
version.BuildInfo{Version:"v3.12.3", GitCommit:"3a31588ad33fe3b89af5a2a54ee1d25bfe6eaa5e", GitTreeState:"clean", GoVersion:"go1.20.7"}

@JorTurFer
Copy link
Member

Yeah.
I guess that we need to add stalebot also in this repo. FYI @tomkerkhove

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants