spinkube / documentation

Site and Docs for the SpinKube community project
https://www.spinkube.dev/
Apache License 2.0
15 stars 24 forks source link

docs: in installation steps remove need to clone `spin-operator` repo #77

Closed kate-goldenring closed 5 months ago

kate-goldenring commented 6 months ago

Ideally, to install spinkube, i should not have to clone the repo. Do we need this make install in the installation steps to install the CRDs or is it bundled in the Helm chart, @vdice?

macolso commented 6 months ago

@kate-goldenring what's the status of this issue? See the PR is closed but issue is open.

I was going to open a PR to remove the dependency on make in this quickstart but want to check that isn't dupe work first.

kate-goldenring commented 6 months ago

I believe it is still unresolved. make install inside the spin-operator repo is still required in the quickstart: https://www.spinkube.dev/docs/spin-operator/quickstart/#deploy-the-spin-operator

@vdice mentioned we are moving the spin operator into it's own helm chart which should resolve this

vdice commented 6 months ago

@macolso this still needs updating as you've pointed out. I think there were rumors of going kustomize? There is of course the helm chart as well. Anyways, will be great to get rid of the clone dependency and either option would be great!

ThorstenHans commented 6 months ago

This needs to be addressed for several parts of the documentation

ThorstenHans commented 5 months ago

I think we could close this one @kate-goldenring

All references to git clone https://github.com/spinkube/spin-operator.git are related to inner-loop dev documentation and/or the contributing guide

kate-goldenring commented 5 months ago

closing as completed