Skip to content

Fix Windows container inventory cache refresh races - #1787

Draft
suyadav1 wants to merge 2 commits into
ci_prodfrom
suyadav/fix-windows-container-inventory
Draft

suyadav1 wants to merge 2 commits into
ci_prodfrom
suyadav/fix-windows-container-inventory

Conversation

@suyadav1

Copy link
Copy Markdown
Contributor

Summary

  • Make the Windows-node watcher the sole cache writer and atomically publish validated node lists.
  • Preserve the last complete cache on pagination failures and add watcher regression tests.

Validation

Ruby 3.1 / Fluentd 1.14.2 in Docker: 9 tests, 61 assertions passed. No live-cluster testing.

Draft Follow-up

Pod startup still waits for Windows-node discovery; persistent LIST failures can block Linux pod inventory. Resolve this dependency and verify Linux behavior before marking ready.

@azure-pipelines

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

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.

1 participant