-
Notifications
You must be signed in to change notification settings - Fork 20
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
[Bug] Not all manifests present in bundle/manifests #979
Comments
I think that's expected behaviour from what you're describing. The bundle is meant to be installed from OperatorHub, and those manifests support the bundle installation. Once the Operator is installed this way it should have its own Deployment. Cryostat Deployments are dynamically created by the Operator, after you as the user create Cryostat CRs. |
You might be able to turn the bundle manifests into a deployment manually (sidestepping the OperatorHub and OLM installation), but that would be a very unsupported configuration. If you want to go that route, the ClusterServiceVersion manifest does actually describe the Operator's own Deployment:
so you could try to extract that description and try to do that manually. But this is really not the way that Operators are intended to be installed and managed. |
Ok, I was under impression this is static manifests set which can be used i.e. in argo or via standalone kubectl, so I was expecing to see https://github.com/cryostatio/cryostat-operator/blob/main/config/manager/manager.yaml
Let's separate the topic how OLM manages operators - not everybody is on openshift using OLM in their cluster, I created an issue earlier - #901 and it seems there was no option proposed for standalop operator installation. Regarding how operators should be installed, recently kubebuilder is introducing helm plugin to generate deployment bundle for operators kubernetes-sigs/kubebuilder#4227 - and there is plenty of operators already shipping helm charts. @andrewazores how do you see |
The simple and unfortunate answer is: we have no support for this. As I said in #901, we don't have a lot of resources (developer hours) to work on implementing yet another way to install Cryostat. Our primary targeted use case is OpenShift with OLM, but as much as possible I try to make sure that we remain generally compatible with other Kubernetes distributions or even non-Kubernetes environments. But, compatibility and supported installation are not the same. I would be very happy to help review anything like a community-developed "Helm chart for installing the Operator", or even "Operator static manifests", but these are not things that the business side prioritizes and so they are not things that I (or the other part-timers on the team) have the time to focus my efforts on. |
got it, are you open to accept contribution in this area ? I believe I can make simple addition to generate static manifest bundle in first place |
Absolutely, I would review and accept a contribution for such a thing :-) Whatever it ends up being, please try to make it such that it can be automatically (re)generated rather than hand-written, or else it's very likely that it will end up falling out of date and out of sync. If it can be updated automatically then I can add it to the upstream release process at least and make sure that it gets updated at release points. Any testing would be on a best-effort basis. |
Current Behavior
I would like to install cryostat by applying manifests from https://github.com/cryostatio/cryostat-operator/tree/cryostat-v3.0/bundle/manifests
but not all manifests are present there, i.e. Deployment
Expected Behavior
After applying manifests from https://github.com/cryostatio/cryostat-operator/tree/cryostat-v3.0/bundle/manifests via kubectl I have up and running cryostat-operator
Steps To Reproduce
No response
Environment
Anything else?
No response
The text was updated successfully, but these errors were encountered: