Skip to content

Recover service-link profiles and preserve orchestration correlation - #129

Merged
Proxicon merged 2 commits into
mainfrom
fix/service-link-aborted-transactions
Oct 5, 2026
Merged

Proxicon merged 2 commits into
mainfrom
fix/service-link-aborted-transactions

Conversation

@Proxicon

@Proxicon Proxicon commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Two failures blocked actual reciprocal orchestration: a PostgreSQL serialization abort during outbound-profile staging escaped as HTTP 500, and an ingest request omitted its correlation header, causing NetRatel to reject its mismatched body/header correlation with HTTP 400.

Recover verified serialization/deadlock aborts by discarding rolled-back tracked state and reloading the same durable attempt within the existing four-attempt limit; exhaustion returns the existing HTTP 409 conflict. Forward the original ingest correlation in X-Correlation-Id using normal header validation. Preserve protected exchanges, operation identities, admission checks and legacy empty-correlation behavior.

Retain the test fixture's bound socket until real Kestrel takes ownership, keeping its configured endpoint identity unchanged across restart. Add actual PostgreSQL recovery/restart/worker tests and correlation-header regressions. The central companion version is 0.1.1-beta.12. Refs #128; companion continuation: BostonTechnologies/netratel#164.

Validation

  • Tests added or updated where appropriate.
  • Whole Release build passes: 0 errors, 21 warnings; emitted warning lines match the prior build exactly.
  • Focused suite: 15 passed, 0 failed, 0 skipped; three actual PostgreSQL cases, six classification guards, two header regressions and four scope-reordering cases.
  • Unfiltered dotnet test Helpdesk.sln --configuration Release --no-build: 1,989 passed, 0 failed, 6 unchanged baseline skips (1,995 total).
  • All 297 relevant lifecycle, authority, contract, network, provider and orchestration-client cases pass with 0 failures or skips; all eleven new regressions pass.
  • Documentation/configuration examples reviewed; this correction introduces no configuration setting.

Disclosure check

  • No credentials, customer data, employee data, private URLs, internal hostnames, or deployment details are included.

@Proxicon Proxicon changed the title Recover aborted service-link profile transactions Recover service-link profiles and preserve orchestration correlation Oct 5, 2026

Proxicon commented Oct 5, 2026

Copy link
Copy Markdown
Contributor Author

Current-source validation receipt for edde0117270e12a564c9add90c213410a9ee30dd (base 372d7b59c1c648e147fd1cb6b2724db6c803d6ae, actual checkout 3eddbf0ebf2142b1c0255539a3d9f5c80aec5dfb, tree 054f7a51efc11cd61c8515e483aa5e65bca98bde):

  • Hosted .NET: 1,989 passed, zero failed, six unchanged baseline skips; 297 relevant cases and all 11 new cases passed.
  • Hosted provider acceptance: 202 passed, zero failed/skipped, including all three new PostgreSQL cases.
  • Mailbox: eight embedded reports passed with no unexpected, flaky or skipped outcomes.
  • UI logs report 74 passed invocations with no failed/flaky/skipped summaries or explicit retries. The 54 MB artifact payload was not examined, so this is not a distinct-case/archive-content claim.

The full 17-check gate remains rejected: nine required jobs succeeded, five were cancelled before any runner or test step was assigned, and three dependent archive RID jobs never ran. GitHub annotations say “The job was not acquired by Runner of type hosted even after multiple attempts”; GitHub currently reports an Actions major outage. The affected-job recoveries retained the successful executions. No further retry is being requested during the active outage.

Runs: normal validation, managed PostgreSQL. Sealed audit SHA-256: 3ece00f9a4f376057d30fdf62fd69525dfcacd89516356e79d3708184a2ae217. Original failed attempts and the unchanged gate rejection are retained. Merge/tag/release are pending. NetRatel's requested release remains 0.1.1-beta.1; this companion fixes source is 0.1.1-beta.12.

@Proxicon
Proxicon merged commit 27543c0 into main Oct 5, 2026
31 of 47 checks passed
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