Repository navigation
docs: publish verified tenant rollover acceptance and current target state - #269
Conversation
… hold Retain exact merged-master CI, immutable test-image identity, real default PostgreSQL assertions, package inventory permission limitation and pending source/target operational gates. This evidence-only handoff does not admit0.55.67 or authorize deployment. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Supersede preparation-time pending statements with reviewed merged-source evidence. Verify the approved prov10 identity, both Ready PostgreSQL servers and actual target application absence. Record tool-discovery friction without changing shared configuration or deploying an unadmitted release. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues remain for this documentation-only update to the migration evidence and handoff records.
No open issues 🎉
What changed in this PR
This documentation-only PR records the postmerge evidence for tenant rollover hardening in #267 and updates SCIMServer’s handoff documents with verified prov10 foundation observations. It changes no runtime behavior or cloud resources.
Changes:
- Records merged-source validation, review results and the tooling test image digest.
- Documents both target PostgreSQL servers as Ready and zero target Container Apps, explicitly superseding historical pending notes.
- Keeps release 0.55.67 admission, final frozen data carry, target acceptance and cutover pending, with recovery limitations documented.
| File | Description |
|---|---|
Session_starter.md |
Adds the current tooling handoff and remaining release/deployment holds. |
docs/TENANT_10_MIGRATION_EXECUTION_2026_10_30.md |
Records detailed validation evidence, target observations and recovery limitations. |
docs/DEPLOYMENT_INFRASTRUCTURE_AND_FORM_FACTORS.md |
Updates tooling validation and target infrastructure status. |
docs/CONTEXT_INSTRUCTIONS.md |
Aligns agent context with the current handoff and outstanding gates. |
CHANGELOG.md |
Summarizes tooling acceptance separately from release admission and deployment. |
Review effort: Balanced
|
Postmerge documentation acceptance: mastera80fafb219c07628c7cd0b4fb67bed983570e7cf passes authoritative freshness41701994315 and automatic CodeQL41701918252. The documentation worktree tree matches merged master and is clean. This merge changes documentation only; the already validated behavioral artifact/source84099293570a and its full postmerge matrix41696483989 remain the runtime evidence. No redundant runtime rebuild, performance campaign, deployment, restore, source-writer change or tenant cutover was performed. Fresh verified-identity prov10 provider reads confirm both PostgreSQL17/B1ms servers Ready and zero target Container Apps. Requested0.55.67 still lacks admission; this evidence update does not substitute another release or waive any operational gate. |
Summary
Publish the evidence-only handoff for merged tenant rollover hardening PR #267, and replace stale current-state impressions with verified prov10 foundation observations.
The behavioral source is merged master
84099293570af5ab06f7d8e2f93e7e6ae2c0236f. Authoritative postmerge Build Test Image41696483989, CodeQL41696338523and freshness41696488100pass. The default fixture proves literal YAML/password/worker transport followed by real PostgreSQL identity, snapshot/signal/fidelity/isolation, image smoke and HIGH/CRITICAL Trivy.A subsequent read-only check verifies the approved prov10 service principal, exact tenant/subscription, zero Container Apps in both target resource groups, and both PostgreSQL17/B1ms servers Ready in Central US. No cloud resource is changed.
The recorded immutable tooling test image is not an admitted0.55.67 release or an authorized alternative. Requested0.55.67 admission, source writer freeze/final carry, actual target acceptance and cutover remain pending. Customer production is untouched.
Linked Defect / Plan Reference
TDD Process
api/src/security/) - N/A; existing default fixture and offline guards are already merged.Feature / Bug-Fix Commit Checklist (standing rule)
Every behavioral change must include all that apply. Tick or strike through with N/A + reason.
api/test/e2e/exercising the feature through HTTP - N/A; no HTTP/UI change.scripts/live-test.ps1runnable against local + Docker + Azure - N/A; no new integration behavior. Actual target acceptance remains a deployment gate, not claimed here.docs/with architecture, RFC references, Mermaid diagrams, test coverage - existing execution document updated; no new diagrams or code blocks requiring rendering changes.docs/INDEX.md- N/A; existing indexed document.[Unreleased]entry with full test counts and feature summary - scoped evidence entry; no fabricated new behavioral test counts.Session_starter.mdanddocs/CONTEXT_INSTRUCTIONS.md- authoritative current state supersedes historical pending notes.api/package.json(and any other version refs) when behavior changed - N/A; documentation only, product version and all manifests/lockfiles unchanged.Validation Output
Standing Rules Acknowledgment
U+2014) anywhere in the diff. Verified viaSelect-String -Pattern ([char]0x2014).git commit --amendon history that has been pushed.api/src/security/forbidden-source-patterns.spec.tswill catch known anti-patterns.api/src/scripts/lint-migrations.tsenforces this. N/A; no migration change.Destructive Migration Override (skip if not applicable)
ALLOW_DESTRUCTIVE_MIGRATION=1in the CI environment for this PRN/A: this PR is documentation only and authorizes no restore.
Reviewer Hint
Diff areas worth special attention from the reviewer: