-
Notifications
You must be signed in to change notification settings - Fork 527
docs: add doc and example api model for IPv6 #3777
Conversation
5e47b1b
to
0113b9d
Compare
Codecov Report
@@ Coverage Diff @@
## master #3777 +/- ##
=======================================
Coverage 73.19% 73.19%
=======================================
Files 148 148
Lines 25378 25378
=======================================
Hits 18576 18576
Misses 5666 5666
Partials 1136 1136 Continue to review full report at Codecov.
|
0113b9d
to
ff7ca3e
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aramase, jackfrancis The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
- IPv6 support is available for Kubernetes version 1.18.0 and later on Azure. | ||
- IPv6 support is in beta as of Kubernetes version 1.18 in Kubernetes community. | ||
|
||
In order to create IPv6 enabled Azure virtual networks and use standard loadbalancer with IPv6 you must first configure your subscription [as follows](https://docs.microsoft.com/en-us/azure/virtual-network/virtual-network-ipv4-ipv6-dual-stack-cli#prerequisites). |
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.
there is no #prerequisites paragraph on that page
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.
also nit: as follows
implies it's described below, should be as described [here](link)
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.
The feature is now GA, so they have removed the prerequisites
. I'll update the doc to reflect that change.
Reason for Change:
Issue Fixed:
Requirements:
Notes: