Closed mathieu-benoit closed 1 year ago
Hi @mathieu-benoit I would like to look into this issue. Can you please assign me to it?
Hi @gitsan13, please see https://github.com/GoogleCloudPlatform/microservices-demo/issues/1428#issuecomment-1400628355.
@NimJay I'll draft a PR for this per our conversation today. Feel free to assign this issue to me and I'll ping you when the PR is ready for review.
The current Service Mesh doc https://github.com/GoogleCloudPlatform/microservices-demo/blob/main/docs/service-mesh.md is outdated, incomplete and misleading.
Could we have something up-to-date and more prescriptive?
For example:
istio-manifests
For 1., maybe showing this snippet:
For 2., maybe showing this snippet:
With that, OB users won't have to search everywhere to find the actual sequence of commands.
For 4., maybe a little cleanup in that folder too? For example, the
allow-egress-googleapis.yaml
file is not needed anymore (need to be confirmed, but someone reported me few days ago that by applying them as-is it generates errors today?).