-
Notifications
You must be signed in to change notification settings - Fork 550
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
Add helm chart to the release #829
Comments
Anyone from the devs care to weigh in on this? |
We don't intend to continue to use helm as our templating layer forever, so this has not been a high priority for us. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
+1 for this, if only because it makes it super easy to use a tool like Flux to manage the OLM (which then manages everything else :D) |
If folks step up to support this, would the project accept the pr's? It seems like a useful thing to support. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Bump |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Bump again. I am also interested in having a helm chart for managing OLM. |
I'm also interested in this capability, so that we can manage the installation with the terraform helm provider. |
This is a good idea. |
interested in this two please |
Any update on this? |
I've published the chart here: https://gitlab.com/kubitus-project/external-helm-charts/operator-lifecycle-manager It'll be updated automatically when new olm versions appears. EDIT: updated URL |
Above gitlab.com link 404's for me... any update on this? |
@sathieu @chrisdewar Looks like the git repo has been moved to https://gitlab.com/kubitus-project/external-helm-charts/operator-lifecycle-manager. Helm repository address has not been changed and it still is https://gitlab.com/api/v4/projects/29255838/packages/helm/stable as mentioned in README. Also you can check available Helm repository packages in GitLab packages |
@chrisdewar Sorry, I've just updated the URL above. As @sturman said, the repo has moved. |
v0.19.1 was published 3 days ago, automatically, thanks to renovate. |
@sathieu the chart version 0.19.1 is not found in the repository. I can see version 0.19.0 only user@think-t460p:~/projects$ helm search repo lifecycle
NAME CHART VERSION APP VERSION DESCRIPTION
operator-lifecycle-manager/olm v0.19.0 A Helm chart for Kubernetes
user@think-t460p:~/projects$ |
@sturman I don't have the same result:
You are probably using the old repo:
Some context: We needed this chart for Kubitus Installer, to install Keycloak Operator. Kubitus installer installs a full-featured Kubernetes cluster "batteries included" (see readme). It's still alpha, but beta is expected around December 2021. |
We have no intention to release helm chart for OLM. It is not how OLM is being installed at the moment. We simply use helm for templating during release. We don't want to maintain a OLM helm chart either. |
This can be done with argocd, specifiy the git repo and and targetRevision (release-xyz) add the helm |
For those using flux V2, this seems to work: ---
apiVersion: source.toolkit.fluxcd.io/v1beta2
kind: GitRepository
metadata:
name: olm
namespace: flux-system
spec:
interval: 1m0s
ref:
semver: ">= v0.23.1"
url: https://github.com/operator-framework/operator-lifecycle-manager.git
ignore: |
# exclude all
/*
# include chart
!/deploy/chart/
---
apiVersion: helm.toolkit.fluxcd.io/v2beta1
kind: HelmRelease
metadata:
name: olm
namespace: flux-system
spec:
interval: 1m0s
chart:
spec:
chart: ./deploy/chart/
sourceRef:
kind: GitRepository
name: olm
namespace: flux-system
interval: 1m Though I haven't worked out which is the least painful option:
Will go play with either painful option. |
Thank you @sathieu for the chart and for maintaining it you are a ⭐ . As it stands the official install is a bash script. Maybe my understanding of what GitOps is is completely wrong but this looks like a red flag to me signaling that OLM is not GitOps friendly, a shame really given that the industry is heading towards GitOps & Operator direction and this is clearly a mess of an install to makes you choose one or the other... |
@atropos112 running bash script for installation is opposite of declarative approach, so you are right. There should be defined with to use it with GitOps, especially counting the fact it's this project is base for operators doing exactly that :) |
thanks for that @sathieu 🙌🏻 |
@rowi1de I'm currently moving away from OLM (which was only used to install keycloak-operator). I've started a new Helm chart for Keycloak Operator Legacy (it's on artifacthub.io). I'm also working on Keycloak Operator (the new one, without KeycloakClient support) and I'll add it once it's ready. |
I don't know, 5+ years seems like long enough that it would have been nice to save everyone all this trouble 😞 |
OLM is already using helm to generate
olm.yaml
under the hood. It'd be nice to have the chart published as well, so that it could be installed likeThe text was updated successfully, but these errors were encountered: