fnproject / fn-helm

Helm Chart for Fn
Apache License 2.0
56 stars 24 forks source link

api versions out of date #56

Open CC007 opened 4 years ago

CC007 commented 4 years ago

The apiVersion for deployments and the ingress are not working for kubernetes 1.18. Even when manually upgrading them gives me issues:

Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "Deployment" in version "extensions/v1beta1"

Even though I have apiVersion: apps/v1 in my deployments.

I am using:

tiberiucorbu commented 4 years ago

I have the same problem using helm v3.3, linting the chart shows a copuple of errors :

[tiber@linux fn-helm]$ helm lint fn
 ==> Linting fn
[ERROR] Chart.yaml: apiVersion is required. The value must be either "v1" or "v2"
[ERROR] templates/flow-deployment.yaml: the kind "apps/v1beta1 Deployment" is deprecated in favor of "apps/v1 Deployment"
 [ERROR] templates/flow-deployment.yaml: a Deployment must contain matchLabels or matchExpressions, and "test-release-fn-flow-depl" does not
 [ERROR] templates/fn-deployment.yaml: the kind "apps/v1beta1 Deployment" is deprecated in favor of "apps/v1 Deployment"
 [ERROR] templates/fn-deployment.yaml: a Deployment must contain matchLabels or matchExpressions, and "test-release-fn" does not
 [ERROR] templates/fn-ingress.yaml: the kind "extensions/v1beta1 Ingress" is deprecated in favor of "networking.k8s.io/v1beta1 Ingress"
 [ERROR] templates/fn-runner-node-deployment.yaml: the kind "apps/v1beta1 Deployment" is deprecated in favor of "apps/v1 Deployment"
 [ERROR] templates/fn-runner-node-deployment.yaml: a Deployment must contain matchLabels or matchExpressions, and "test-release-fn-runner" does not
 [ERROR] templates/ui-deployment.yaml: the kind "apps/v1beta1 Deployment" is deprecated in favor of "apps/v1 Deployment"
 [ERROR] templates/ui-deployment.yaml: a Deployment must contain matchLabels or matchExpressions, and "test-release-fn-ui" does not

On the other thought helm added some breaking changes to version 3. for example: init, install --name doesn't exist anymore 🙄

jslay77 commented 2 years ago

Are there any plans to update this chart for recent versions of k8s APIs and Helm 3?

metamemelord commented 2 years ago

Are there any plans to update this chart for recent versions of k8s APIs and Helm 3?

I don't think this project has active development..

CC007 commented 2 years ago

Are there any plans to update this chart for recent versions of k8s APIs and Helm 3?

I don't think this project has active development..

Yup, I swapped to OpenFaaS