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

Enhance SPIRE Quickstart Tutorial for kubeadm based Kubernetes #135

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

saiakhil2012
Copy link

@saiakhil2012 saiakhil2012 commented Sep 4, 2024

Creating this PR to add couple of files to the k8s/quickstart directory to enhance the SPIFFE Documentation related to K8S Quick start for users using kubeadm based K8S Clusters.

When I was trying to utilize SPIFFE Documentation https://spiffe.io/docs/latest/try/getting-started-k8s/ using kubeadm based K8S cluster, it was involved to setup other storage classes and made following the tutorial less streamlined.

Adding local storage based storage class and persistent volume related yaml files.

Once this PR is merged, will raise another PR in https:/spiffe/spiffe.io to modify the corresponding getting started guide (i.e. https:/spiffe/spiffe.io/blob/master/content/docs/latest/try/getting-started-k8s.md) to add a section, Considerations When Using Kubeadm similar to the one for Minikube.

There is an associated issue as well, "Tutorial fails when installing spire server" (#91), raised couple of years back which is still Open. With the change in this PR and a subsequent PR in spiffie.io repo as mentioned above, the #91 issue can be resolved as well.

@saiakhil2012 saiakhil2012 force-pushed the akhil/ehance-spire-k8s-kubeadm-quickstart branch 3 times, most recently from fca1e24 to 91e97fb Compare September 4, 2024 16:45
@saiakhil2012 saiakhil2012 force-pushed the akhil/ehance-spire-k8s-kubeadm-quickstart branch from 91e97fb to b8bebe4 Compare September 4, 2024 16:50
@saiakhil2012
Copy link
Author

@rturner3 greetings, I saw you had worked on this repo before and reviewed past PRs. Requesting you to review this PR or suggest anyone else who can review this PR, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant