MicrosoftDocs / azure-aks-docs

Public sync with private repo azure-aks-docs-pr

Creative Commons Attribution 4.0 International

3 stars 60 forks source link

Update api-server-vnet-integration.md #87

Closed rahulrai-in closed 2 weeks ago

rahulrai-in commented 2 weeks ago

Added NSG rules required to enable communication between components.

prmerger-automator[bot] commented 2 weeks ago

@rahulrai-in : Thanks for your contribution! The author(s) have been notified to review your proposed change.

rahulrai-in commented 2 weeks ago

Could you please review this change @chasewilson @feiskyer

learn-build-service-prod[bot] commented 2 weeks ago

Learn Build status updates of commit 90f0dc3:

:white_check_mark: Validation status: passed

File	Status	Preview URL	Details
articles/aks/api-server-vnet-integration.md	:white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Try searching the learn.microsoft.com contributor guides
Post your question in the Learn support channel

learn-build-service-prod[bot] commented 2 weeks ago

Learn Build status updates of commit 2aeb190:

:white_check_mark: Validation status: passed

File	Status	Preview URL	Details
articles/aks/api-server-vnet-integration.md	:white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Try searching the learn.microsoft.com contributor guides
Post your question in the Learn support channel

learn-build-service-prod[bot] commented 2 weeks ago

Learn Build status updates of commit 9923b93:

:white_check_mark: Validation status: passed

File	Status	Preview URL	Details
articles/aks/api-server-vnet-integration.md	:white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Try searching the learn.microsoft.com contributor guides
Post your question in the Learn support channel

Court72 commented 2 weeks ago

@asudbring

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

label:"aq-pr-triaged"

@MicrosoftDocs/public-repo-pr-review-team

asudbring commented 2 weeks ago

sign-off

learn-build-service-prod[bot] commented 2 weeks ago

Learn Build status updates of commit 3bc6777:

:white_check_mark: Validation status: passed

File	Status	Preview URL	Details
articles/aks/api-server-vnet-integration.md	:white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Try searching the learn.microsoft.com contributor guides
Post your question in the Learn support channel

Jak-MS commented 2 weeks ago

@rahulrai-in @asudbring - the table was broken so I changed the markdown syntax to fix the issue, but please check that this is what you intended.:

Destination	Source	Protocol	Port	Use
APIServer Subnet CIDR	Cluster Subnet	TCP	443 and 4443	Required to enable communication between Nodes and the API server.
APIServer Subnet CIDR	Azure Load Balancer	TCP	9988	Required to enable communication between Azure Load Balancer and the API server. You can also enable all communicatios between the Azure Load Balancer and the API Server Subnet CIDR.

Jak-MS commented 2 weeks ago

If it was NOT, we can fix in a new PR. thanks.