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

quickstart guide should have paths to install all components with helm or kubectl #122

Open
michelleN opened this issue Mar 18, 2024 · 1 comment
Labels
area: installation Content related to installation guides. enhancement New feature request.

Comments

@michelleN
Copy link
Contributor

In the quickstart guide, we use kubectl for most install steps and helm for one step. It'd be nice if there was a path for an all helm (or mostly helm) installation path and another one that just uses kubectl.

One argument for not using helm for all or most of the steps is that we are installing CRDs which is a cluster wide resource and operators may not want to use Helm for installing CRDs and also upgrades are tricky. Considering this is a quickstart, we should favor ease of installation and point to a diff doc that talks about what to consider when installing in prod environments. There is some guidance on how to deal with CRDs on this helm doc . Here is what cert-manager does with CRDs in their helm install path.

@bacongobbler
Copy link
Contributor

bacongobbler commented Jul 8, 2024

To recap, here are the list of steps necessary to install SpinKube onto a cluster:

  1. Install cert-manager
  2. Apply a RuntimeClass
  3. Apply Spin-operator CRDs
  4. helm install spin-operator
  5. Apply the shim executor

Some open questions to consider here:

Could we add cert-manager as a dependency of the spin-operator chart? That is, is there a way to determine if cert-manager needs to be installed or not at the time we are installing spin-operator?

Can we install a default RuntimeClass? Could we make it configurable through Helm values?

Can we apply the spin-operator CRDs via the crds/ directory?

I recall there being a discussion around whether we wanted to include a default SpinAppExecutor in the chart or not. Is there a conversation going on in the spin-operator repo tracking this discussion? I think part of that conversation exists in spinkube/spin-operator#187. I'm not sure what else may be missing from that piece.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: installation Content related to installation guides. enhancement New feature request.
Projects
None yet
Development

No branches or pull requests

2 participants