Skip to content

Require loaded configuration revision before controller attach - #5518

Draft
mgazza wants to merge 1 commit into
fix/control-service-debug-exportfrom
fix/require-loaded-controller-revision
Draft

mgazza wants to merge 1 commit into
fix/control-service-debug-exportfrom
fix/require-loaded-controller-revision

Conversation

@mgazza

@mgazza mgazza commented Oct 12, 2026

Copy link
Copy Markdown
Collaborator

An explicitly enrolled engine could attach to the service's current revision while running stale local arguments. Require the emitted saas_config_version before making an attach request, then compare it with the service revision before starting the worker or constructing components. Missing, malformed and mismatched revisions leave admission closed.

Stacks on #5516. Paired operator and legacy image-wrapper changes emit the response revision for explicit enrollment only. Enrollment remains disabled by default; this does not prove a configuration content hash, exact physical inverter identity or target readiness.

Validation: 32 control/service tests pass, including actual PredBat.initialize refusing stale configuration before component construction. Pre-commit passes. Staged GitNexus analysis reports low risk across four expected files.

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