Azure Load Balancer notes #16456
Azure Load Balancer notes #16456
Conversation
|
Skipping CI for Draft Pull Request. |
istio-testing
added
do-not-merge/work-in-progress
istio-testing
added
size/M
istio-testing
removed
the
do-not-merge/work-in-progress
|
todo, add a section on infra labels |
| @@ -61,6 +61,14 @@ bookinfo-gateway istio bookinfo-gateway-istio.default.svc.cluster.local Tr | |||
|
|
|||
| Wait for the gateway to show as programmed before continuing. | |||
|
|
|||
| {{< tip >}} | |||
There was a problem hiding this comment.
We try very hard to keep the Getting Started guides to be simple, and not full of "if this then that'. Is there any other way we can communicate this to keep it out of the "happy path"?
There was a problem hiding this comment.
That's fair. I was going to justify it by saying "we do similar stuff for GKE", but I think I hallucinated that. We definitely want to add something on in the Azure specific section
There was a problem hiding this comment.
aha! I found it: https://istio.io/latest/docs/examples/bookinfo/#start-the-application-services. Though I agree that this is not be a good precendent to follow
There was a problem hiding this comment.
The same requirement is probably true of AKS, which probably didn't exist when the page was written 🤣
|
@Stevenjin8: The following test failed, say
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Description
Band aid for istio/istio#46769
Reviewers