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

Update Istio VirualService documentation #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alexchronopoulos
Copy link

  • Fix spacing to avoid errors when attempting to apply.
  • The resource needs to be deployed into the 'istio-system' namespace to function.
  • Update gateway name to match that of previously created gateway or else the VirtualService does not function as expected.

Thank you for sharing this project! I followed your instructions and everything worked well except for this Istio VirutalService resource. The changes here allowed me to test Nginx successfully. I hope these changes can help someone else in the future.

…eds to be deployed into the 'istio-system' namespace to function. Update gateway name to match that of previously created gateway or else the VirtualService does not function as expected.
@s-raza
Copy link

s-raza commented Jun 9, 2022

I was about to post an exact same PR, good I checked on here before I did it. I was stumped why this was not working in the final step. I am just learning about Virtual services through istio and this took me sometime to figure out. Thanks @alexchronopoulos

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.

2 participants