Skip to content

fix(test): allow all-nodes log generator on Windows 2025 - #1791

Open
zanejohnson-azure wants to merge 2 commits into
ci_prodfrom
zane/test-win25-log-generator
Open

zanejohnson-azure wants to merge 2 commits into
ci_prodfrom
zane/test-win25-log-generator

Conversation

@zanejohnson-azure

Copy link
Copy Markdown
Contributor

Summary

Remove only kubernetes.azure.com/os-sku: Windows2022 from test/scenario/log-gen-all-nodes-daemonset.yaml.

The remaining kubernetes.io/os: windows selector allows the DaemonSet to schedule on both Windows Server 2022 and Windows Server 2025 nodes. The existing servercore:ltsc2022 image is supported on Windows Server 2025 with process isolation (compatibility matrix).

Coverage

The existing Testkube containerstatus-windows workflow is not restricted to a Windows Server version. For ci-logs-dev-aks-all-nodes, the pipeline also enables PerNodeLogCoverage=true, which expects container log ingestion from every cluster node, including a Windows Server 2025 node once added. The tests do not require a Win25 node to exist.

Deployment

After adding the Windows Server 2025 node, reapply this DaemonSet manifest to the cluster. The current build pipeline deploys ama-logs but does not apply this log-generator manifest.

No pipeline, agent image, test implementation, or cluster configuration changes are included.

Remove the Windows2022-only node selector so the Windows log generator also schedules on Windows Server 2025 nodes.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@zanejohnson-azure
zanejohnson-azure requested a review from a team as a code owner September 24, 2026 04:58
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@zanejohnson-azure
zanejohnson-azure enabled auto-merge (squash) September 24, 2026 22:54

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants