{AKS} Fix monitoring consistency and live scenario prerequisites - #10315
Draft
FumingZhang wants to merge 3 commits into
Draft
{AKS} Fix monitoring consistency and live scenario prerequisites#10315FumingZhang wants to merge 3 commits into
FumingZhang wants to merge 3 commits into
Conversation
Use supported ALB arguments, verify persisted state after void reimage actions, handle precise capability gates and operation preemption, and replace the unavailable legacy proxy package repository with Ubuntu 24.04 Squid. Require bounded proxy readiness before creating AKS. Validation: 55 tests and 21 subtests passed; 388 scenarios collected. Ubuntu 24.04 container bootstrap returned HTTP 200 through all four HTTP/HTTPS proxy paths. Syntax, style and diff checks passed. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Apply the canonical Container Insights synchronization fix to aks-preview's separate addon and flow-log implementations. Preserve other Azure Monitor settings and add wire-payload regressions plus a Pending release note. Validation: 69 targeted preview tests and 23 subtests passed across the scenario, custom-command, and flow-log suites; product style and syntax checks passed. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Hi FumingZhang, |
microsoft-github-policy-service
Bot
requested review from
Julie Zhu (yanzhudd) and
Yong Zhang (yonzhan)
September 9, 2026 06:35
Collaborator
|
AKS |
Assert true/false values for legacy string flags without depending on service capitalization. Preserve create, disable, and re-enable checks. Validation: affected test_aks_create_acns_with_flow_logs passed end-to-end LIVE (752.46 seconds). Original True-versus-true failure retained in validation evidence. No unit or mocked run substitutes for this result. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.
This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
az aks enable-addons,az aks disable-addons,az aks update, and affected aks-preview live scenarios.What / Why
Follow-up to #10308:
squid-opensslpackage.Validation
test_aks_applicationloadbalancer_updatewithAZURE_TEST_RUN_LIVE=trueandAZURE_CLI_TEST_RETRY_PROVISIONING_CHECK=true. Local core, aks-preview, and dataprotection source paths were verified; cassette recording was disabled.Standard_D4ds_v4ineastusand permits onlyDCads_v6sizes. The fixed update step was not reached. Resource-group creation and version discovery succeeded; the temporary group was deleted.westus2(516.33 seconds), including the fixed update command. The initial failedeastusattempt remains recorded separately.test_aks_create_with_enable_backupandtest_aks_update_with_enable_backupBOTH PASSED live using the local dataprotection helper and actual AKS/storage/vault/policy/backup-instance resources (3301.37 seconds combined).test_aks_create_with_http_proxy_configPASSED live in 407.50 seconds, including Ubuntu 24.04 cloud-init/Squid bootstrap, both HTTP/HTTPS MCR readiness checks, and AKS creation. The one-hour timeout is unchanged.test_aks_network_isolated_clusterPASSED in live mode (3032.85 seconds), covering all three clusters, the repaired empty node-image action, and follow-up nodepool show.test_aks_disable_and_reenable_http_proxyPASSED live in 2004.06 seconds, including bootstrap/create, disable, and re-enable assertions.test_aks_update_http_proxy_configtimed out at the unchanged 3600-second limit. Proxy bootstrap and initial AKS creation succeeded; the subsequent AKS configuration update remained in LRO polling. It is recorded as a live timeout, not a pass. The timeout hard-exit produced a log but no final JUnit/JSON; service-state capture and scoped cleanup are in progress.test_aks_create_acns_with_flow_logsPASSED its exact live rerun in 752.46 seconds. The original attempt'sTrueversustrueassertion failure is retained separately; the live-validated assertion correction is published inb5bbb71a2.test_aks_azure_service_mesh_enable_disablePASSED live once in 743.976 seconds, covering cluster creation, Istio enable, mesh disable, and deletion. No preemption/error was injected or observed in this successful run.Scope / History
src/index.jsonchange.General Guidelines
azdev style <YOUR_EXT>locally? (pip install azdevrequired) - targeted flake8/syntax checks passed; full repository style is left to CI.python scripts/ci/test_index.py -qlocally? (pip install azdevrequired) - index unchanged.For new extensions:
About Extension Publish
Version remains unchanged; the product release note is under Pending. The index is managed by release automation.