-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
docs: Add BGP GSG #15519
docs: Add BGP GSG #15519
Conversation
025fdc5
to
e13ff45
Compare
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.
Formatting nits: Could you please make sure you use the same level of indentation for all your blocks? Ideally left-align on the name of the directive (code-block
), like you did for the parsed-literal
with 3 spaces.
Looks all good otherwise.
Signed-off-by: Chris Tarazi <chris@isovalent.com>
e13ff45
to
819df2c
Compare
spec: | ||
containers: | ||
- name: web | ||
image: nginx |
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.
It looks like nginx is only used as example app in other page, do we have a demo app that is good for this? If not, I think podinfo could be a good candidate. Just a thought, and I guess here it doesn't matter all that much as we are not doing blue/green deployments or anything like that, but may a simple app that has Cilium logo would be nice.
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.
This looks good overall, just added a comment with some thoughts regarding the demo app.
Signed-off-by: Chris Tarazi chris@isovalent.com