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

Remove references to SPIRE where no longer relevant #137

Merged
merged 5 commits into from
Sep 12, 2023

Conversation

orishoshan
Copy link
Contributor

Previously, SPIRE was deployed by default. It is now no longer deployed by default, and this PR removes references to configuring it in tutorials where it is not used.

@orishoshan orishoshan enabled auto-merge (squash) September 7, 2023 09:03
@usarid
Copy link
Contributor

usarid commented Sep 10, 2023

What about tutorials where we now need to add it, e.g. Kafka tutorials where you'd need it if you're not connecting to Cloud?

For the Istio one, and perhaps others, we still have:

Otterize requires about 200 MBs and 200 mCPU for all components (including a SPIRE deployment) to install and run properly on Minikube and EKS clusters.

What is the right replacement for this?

@orishoshan
Copy link
Contributor Author

@usarid On your comment: In the PR that disabled SPIRE, we added instructions to enable SPIRE in the tutorials that need it.

I'm removing the system requirements since they are incorrect.

…github.com/otterize/docs into orisho/remove_irrelevant_references_to_spire
Copy link
Contributor

@usarid usarid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@orishoshan orishoshan merged commit 8f64910 into main Sep 12, 2023
2 checks passed
@orishoshan orishoshan deleted the orisho/remove_irrelevant_references_to_spire branch September 12, 2023 16:02
@github-actions github-actions bot locked and limited conversation to collaborators Sep 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants