Skip to content

perf(azure): skip redundant worker tag writes - #397

Merged
ruby-dlee merged 1 commit into
mainfrom
codex/nm-skip-redundant-tags
Aug 28, 2026
Merged

perf(azure): skip redundant worker tag writes#397
ruby-dlee merged 1 commit into
mainfrom
codex/nm-skip-redundant-tags

Conversation

@ruby-dlee

Copy link
Copy Markdown
Owner

Summary

  • avoid rewriting Azure worker tags when every exact assignment tag is already present
  • retain the existing repair path for missing or differing tags
  • cover both no-op convergence and selective repair with an executable regression

Why

Real admission tracing showed exact resources being tagged again after the deployment had already produced the correct identity. Those writes make Azure reconverge the VM extension and managed Run Commands, adding control-plane latency without changing the worker boundary.

Verification

  • python3 -m py_compile bin/fm-azure-worker-provider.py
  • bash tests/fm-worker-lifecycle.test.sh
  • bash tests/fm-azure-pilot.test.sh
  • bin/fm-lint.sh
  • git diff --check

Safety boundary

One-review/one-VM isolation, exact assignment tags, exact-head binding, credential custody, cleanup proof, and fail-closed behavior are unchanged. Any missing or differing required tag still takes the existing mutation and convergence path.

@ruby-dlee
ruby-dlee merged commit 38b8d72 into main Aug 28, 2026
7 checks passed
@ruby-dlee
ruby-dlee deleted the codex/nm-skip-redundant-tags branch August 29, 2026 20:42
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