Repository navigation
M2.17-D: surface SGCB/GoCB initial MMS errors in Browser and Diagnostics - #608
Draft
masarray wants to merge 3 commits into
Conversation
This branch has not been deployed
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.
M2.17-D — truthful control-state initial Read failure provenance
Stacked draft base: PR #607 at exact SHA
c5171a76906af60d05155516365353492e699704. Do not merge upstream or re-pin engines.Audited defect (reproducible from code)
The existing finite initial MMS hydration in the runtime already records error-only evidence per IED and association. Ordinary Browser Data Model rows can read those errors, but SGCB and GoCB control-state properties only queried the separate manual Read error recorder. When automatic initial attempts failed, the selected control block could incorrectly remain
Not readwithout disclosing its failure reason. This was an evidence-consumer gap, not a request to issue more MMS traffic.Narrow changes
IedBrowserControlStateEvidence.ReadTargets(verified GetNameList-derived FC inventory) to resolve initial OR manual failure for SGCBNumOfSG/ActSG/LActTmand GoCBGoEna/NdsCom. Manual error detail takes precedence when still current;LiveValuessuccess remains the only value authority. No speculative field, control write or GOOSE-frame inference.IedBrowserReadFailureEvidencenow offers a read-only, sorted, maximum-32 diagnostic snapshot; current IED object, association, model/epoch and connected state are checked under the same gate. It never publishes a value or stores a second copy.Not claimed by this PR
This does not fix the physical cause of unsuccessful MMS SG leaf reads, prove analog DataSet values, restore cross-client RCB reservation detection, or establish BCUGE hardware acceptance. It makes those faults more visible for evidence-led diagnosis. All remaining acceptance work is tracked on the stacked milestone chain.
No change to pinned ARIEC61850 or ArdIrec, no main merge. No local .NET compiler in this connector environment; CI exact-HEAD build/regression is the first executable gate. Keep draft until CI and later physical validation.