-
Notifications
You must be signed in to change notification settings - Fork 52
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 deprecated gateway annotation in docs. Improve getstarted.md #504
Conversation
48470de
to
93e5f42
Compare
…ce-vpc-association: "true"` - Improve the section "Set up single-cluster/VPC service-to-service communications" in getstarted.md
93e5f42
to
a3a5bf3
Compare
docs/getstarted.md
Outdated
reason: Reconciled | ||
status: "True" | ||
type: Schedules | ||
Verify that `my-hotel` gateway is created with `PROGRAMMED` equals to `True`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: Verify that my-hotel
Gateway is created with a PROGRAMMED
status equal to True
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only requesting small changes for clarity. Overall looks great!
Thank you! sent a new revision base on PR comments, please take a look if you have a time. |
…ice-vpc-association: "true"`
What type of PR is this? documentation
Changges:
application-networking.k8s.aws/lattice-vpc-association: "true"
in docs and yaml examplesBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.