Repository navigation
docs(azure): add the Azure provider page - #779
Open
khuyentran1401 wants to merge 2 commits into
Open
khuyentran1401 wants to merge 2 commits into
khuyentran1401 wants to merge 2 commits into
Conversation
khuyentran1401
requested review from
dcmcand,
jbouder,
marcelovilla and
viniciusdc
November 11, 2026 10:34
|
Docs preview for
|
dcmcand
reviewed
Nov 11, 2026
dcmcand
left a comment
Contributor
There was a problem hiding this comment.
Verdict: Comment. I checked the page against nic 82753a6 and aks-cluster v0.3.0, and the claims hold, with the corrections inline. None of them block.
💡 Suggestion: The page marks Azure supported but does not list two limitations that a user will hit. Add a short Limitations section.
- Longhorn is not installed on Azure, so Longhorn backups do not work (
provider.go:392-396;backup-restore.mdalready says this). - The enterprise TLS
trust_bundleis not installed on Azure nodes, andnicgives no warning.
💡 Suggestion: upgrade-kubernetes.md links the AWS and Hetzner pages for provider-specific upgrade constraints. Add this page there too.
Key: 🛑 Blocker (must fix before merge) · 💡 Suggestion (improvement, not blocking) · ❓ Question (needs author input) · ❤️ Kudos
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reference Issues or PRs
Closes #777.
What does this implement/fix?
Testing
Did you test the pull request locally?
Did you add new tests?
Checked fields, defaults, and destroy behavior against the nic source (
447ed13) and theaks-clustermodule (v0.3.0).Sign-in with
az loginplusAZURE_SUBSCRIPTION_IDconfirmed by a successfulnic deploy --dry-run.Not tested with a full deploy, so the load balancer and public IP cost items are still unconfirmed.