-
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
doc: Add Azure CNI Powered by cilium as external installer #28286
doc: Add Azure CNI Powered by cilium as external installer #28286
Conversation
d6dd01c
to
ebc6c7f
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.
Thanks for this!
Please find some minor comments, inline below. Contents look good.
Don't hesitate to add the links to your text ( `see relevant doc <https://some.link>`__
) rather than just printing the links themselves (see this long link `<https://some.link>`_
), it's more readable.
Also, please squash your commits together.
Does this documentation apply to the |
it can be backported to 1.12, 1.13 and 1.14 |
0fd0bb4
to
459fc42
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.
Hey @tamilmani1989 before I do a review, is this intended to cover everything from #27289 or only your bits?
EDIT: OK I found the other PRs.
Added a doc to update installation instructions of cilium via Azure CNI Powered by Cilium AKS cluster. Added a page to describe about delegated ipam.
Related Issue: #27289