Document controller-backed E2E and cluster CNI#235
Merged
Conversation
This was referenced Jul 10, 2026
bcho
force-pushed
the
hbc/controller-cni-docs
branch
from
c80db43 to
0551518
Compare
bcho
force-pushed
the
hbc/unbounded-net-cni-e2e
branch
from
31865b3 to
0a1628b
Compare
bcho
force-pushed
the
hbc/controller-cni-docs
branch
2 times, most recently
from
6b3cf23 to
76a3094
Compare
bcho
force-pushed
the
hbc/unbounded-net-cni-e2e
branch
from
ea69964 to
ee57ed1
Compare
bcho
force-pushed
the
hbc/controller-cni-docs
branch
from
76a3094 to
bd2ecd8
Compare
bcho
force-pushed
the
hbc/unbounded-net-cni-e2e
branch
from
ee57ed1 to
c28c0e7
Compare
bcho
force-pushed
the
hbc/controller-cni-docs
branch
from
bd2ecd8 to
fce78ac
Compare
bcho
force-pushed
the
hbc/unbounded-net-cni-e2e
branch
from
c28c0e7 to
48c3648
Compare
bcho
force-pushed
the
hbc/controller-cni-docs
branch
from
fce78ac to
5d1f03a
Compare
bcho
force-pushed
the
hbc/unbounded-net-cni-e2e
branch
from
48c3648 to
b2d9588
Compare
bcho
force-pushed
the
hbc/controller-cni-docs
branch
from
5d1f03a to
fd7c62b
Compare
# Conflicts: # .env.example # hack/e2e/lib/cleanup.sh # hack/e2e/lib/common.sh # hack/e2e/run.sh
wenxuan0923
approved these changes
Jul 14, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates AKS Flex Node documentation and demo assets to reflect the controller-backed (ConfigMap + service-proxy) machine endpoint and the no-CNI AKS + Unbounded-Net E2E/cluster networking model, and adds demo manifests/scripts for validating connectivity and drain behavior.
Changes:
- Refresh E2E docs to describe no-CNI AKS provisioning, Unbounded-Net installation, and controller-backed machine goal/repave validation.
- Update configuration and design docs for the in-cluster machine client, ARM-compatible machine response shape (ETag drift key), and status patch model.
- Add demo Site + sample app manifests and a connectivity/latency validation script for demo-site nodes.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| hack/e2e/README.md | Updates E2E flow/prereqs/env vars to include Unbounded-Net + in-cluster controller/registry. |
| hack/demo/sample-application.yaml | Adds a multi-replica demo workload for drain/placement demonstrations. |
| hack/demo/demo-site.yaml | Adds a demo Unbounded-Net Site + gateway pool assignment manifest. |
| hack/demo/aks-flex-validate-demo-connectivity.sh | Adds a demo connectivity/latency validation script for demo-site nodes. |
| docs/usages/configuration.md | Documents agent.machineClient.* configuration for ARM vs in-cluster mode. |
| docs/development.md | Updates the project tree to include aks-flex-controller. |
| docs/design/agent-and-aks.md | Updates design narrative for controller-backed machine goals and ETag-based drift. |
| docs/design.md | Adjusts high-level design wording for cluster-CNI wiring. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Related PRs
This PR is intentionally based on
hbc/unbounded-net-cni-e2e; review only the documentation and demo-asset diff from that branch.Summary
Draft follow-up
docs/design.mdwith the current configuration/authentication model and remove remaining local-file/test-mode wordingValidation
git diff --checkbash -n hack/demo/aks-flex-validate-demo-connectivity.shThis PR contains documentation and demo assets relative to #234.