Skip to content

docs: update cilium setup guide with --network-policy requirement and…#4417

Open
sharifnasser wants to merge 1 commit into
masterfrom
docs/update-cilium-setup-guide
Open

docs: update cilium setup guide with --network-policy requirement and…#4417
sharifnasser wants to merge 1 commit into
masterfrom
docs/update-cilium-setup-guide

Conversation

@sharifnasser
Copy link
Copy Markdown
Contributor

… PDB workaround

The az aks update command requires both --network-dataplane cilium and --network-policy cilium flags. Without --network-policy, the API returns a BadRequest error.

Also documents a workaround for single-node pool clusters where PDBs (konnectivity-agent, coredns) block node drain during the upgrade.

Reason for Change:

Issue Fixed:

Requirements:

Notes:

… PDB workaround

The az aks update command requires both --network-dataplane cilium and
--network-policy cilium flags. Without --network-policy, the API returns
a BadRequest error.

Also documents a workaround for single-node pool clusters where PDBs
(konnectivity-agent, coredns) block node drain during the upgrade.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@sharifnasser sharifnasser requested a review from a team as a code owner May 21, 2026 00:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Swiftv2 managed Cilium setup guide to reflect current AKS CLI requirements and to document an upgrade workaround for single-node system pools where PDBs can block drain during upgrade.

Changes:

  • Updates the az aks update example to include --network-policy cilium alongside --network-dataplane cilium.
  • Adds a NOTE explaining that both flags are required (API rejects requests missing --network-policy).
  • Adds a TIP documenting a PDB deletion workaround for single system node pool upgrades that fail with PodDrainFailure.

> [!TIP]
> If your cluster has a single system node pool, the upgrade may fail with a `PodDrainFailure` due to Pod Disruption Budgets (e.g., `konnectivity-agent`). To work around this, delete the blocking PDBs before running the update — AKS addon manager will recreate them after the upgrade:
> ```
> kubectl delete pdb konnectivity-agent -n kube-system --ignore-not-found
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you delete the pdb like this, does aks reconcile and recreate the pdb?

> Both `--network-dataplane cilium` and `--network-policy cilium` are required. The API will reject the request if only `--network-dataplane cilium` is specified.

> [!TIP]
> If your cluster has a single system node pool, the upgrade may fail with a `PodDrainFailure` due to Pod Disruption Budgets (e.g., `konnectivity-agent`). To work around this, delete the blocking PDBs before running the update — AKS addon manager will recreate them after the upgrade:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did konnectivity agent try to come up on the byo node during upgrade by any chance?

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 5, 2026

This pull request is stale because it has been open for 2 weeks with no activity. Remove stale label or comment or this will be closed in 7 days

@github-actions github-actions Bot added the stale Stale due to inactivity. label Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale Stale due to inactivity.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants