Document controller-backed E2E and cluster CNI#235
Merged
Conversation
This was referenced Jul 10, 2026
c80db43 to
0551518
Compare
31865b3 to
0a1628b
Compare
6b3cf23 to
76a3094
Compare
ea69964 to
ee57ed1
Compare
76a3094 to
bd2ecd8
Compare
ee57ed1 to
c28c0e7
Compare
bd2ecd8 to
fce78ac
Compare
c28c0e7 to
48c3648
Compare
fce78ac to
5d1f03a
Compare
48c3648 to
b2d9588
Compare
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.