Skip to content

docs: publish verified tenant rollover acceptance and current target state - #269

Merged
pranems merged 2 commits into
masterfrom
docs/tenant10-tooling-acceptance-20261103
Nov 3, 2026
Merged

pranems merged 2 commits into
masterfrom
docs/tenant10-tooling-acceptance-20261103

Conversation

@pranems

@pranems pranems commented Nov 3, 2026 •

Copy link
Copy Markdown
Owner

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 Image 41696483989, CodeQL 41696338523 and freshness 41696488100 pass. 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

  • RED test written first; observed it fail for the expected reason - N/A for this documentation-only PR; actual behavioral RED/GREEN belongs to merged fix(ops): prepare safe registry-driven tenant rollover #267.
  • GREEN: minimal implementation that turns the test green - N/A; no implementation change.
  • Refactor (if any) keeps the test green - N/A; no refactor.
  • Permanent regression guard added (where applicable - see existing patterns in 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.

  • Unit Tests - service-level and controller-level coverage of the new behavior - N/A; no new runtime behavior.
  • E2E Tests - end-to-end spec under api/test/e2e/ exercising the feature through HTTP - N/A; no HTTP/UI change.
  • Live Integration Tests - new section in scripts/live-test.ps1 runnable against local + Docker + Azure - N/A; no new integration behavior. Actual target acceptance remains a deployment gate, not claimed here.
  • Feature Documentation - dedicated doc in docs/ with architecture, RFC references, Mermaid diagrams, test coverage - existing execution document updated; no new diagrams or code blocks requiring rendering changes.
  • INDEX.md Update - add the new doc reference to docs/INDEX.md - N/A; existing indexed document.
  • CHANGELOG.md Update - [Unreleased] entry with full test counts and feature summary - scoped evidence entry; no fabricated new behavioral test counts.
  • Session & Context Updates - Session_starter.md and docs/CONTEXT_INSTRUCTIONS.md - authoritative current state supersedes historical pending notes.
  • Version Management - bump api/package.json (and any other version refs) when behavior changed - N/A; documentation only, product version and all manifests/lockfiles unchanged.
  • Response Contract Tests - API responses contain ONLY documented fields (key allowlist assertion at unit + E2E + live) - N/A; no response change.

Validation Output

git diff --check
  PASS before documentation commits.
Added-text em-dash check
  PASS before the current-state documentation commit.
Normal default protected git push
  PASS: 26/26 Fast-mode gates at4f674eb3 in643.3 seconds.
  Includes API unit/HTTP, documentation rendering/freshness, builds,
  source/supply-chain checks and migration contracts.
Hosted checks / exact-tip independent review for this documentation PR
  PASS at4f674eb3:
  BuildTestImage41700758314, CodeQL41700758311, freshness41700758112.
  Independent bot review5758076405 reports no open issues and recommends
  approval. Its state is COMMENTED, not formal APPROVED.

Independent behavioral-source evidence, not new documentation-tip results:
Merged master84099293570af5ab06f7d8e2f93e7e6ae2c0236f:
  BuildTestImage41696483989: PASS.
  Actual default PostgreSQL assertions: PASS through2026-11-03T18:44:18Z.
  Container smoke and HIGH/CRITICAL Trivy: PASS.
  CodeQL41696338523: PASS.
  Freshness41696488100: PASS.
Read-only approved-identity target recheck:
  PASS: correct prov10 tenant/subscription/service principal.
  PASS: zero Container Apps in each target resource group.
  PASS: both target PostgreSQL17/B1ms servers Ready.
Release0.55.67 admission / final frozen carry / target browser acceptance:
  NOT ESTABLISHED; no deployment or cutover performed.

Standing Rules Acknowledgment

  • No em-dash (Unicode U+2014) anywhere in the diff. Verified via Select-String -Pattern ([char]0x2014).
  • No git commit --amend on history that has been pushed.
  • No new credentials, tokens, or secrets committed to source. Forbidden patterns regression spec at api/src/security/forbidden-source-patterns.spec.ts will catch known anti-patterns.
  • Migrations are additive-only (expand-contract for column changes). Migration linter at api/src/scripts/lint-migrations.ts enforces this. N/A; no migration change.

Destructive Migration Override (skip if not applicable)

  • This PR contains a NEW destructive Prisma migration (DROP TABLE/COLUMN, ALTER COLUMN ... TYPE, RENAME, INSERT ... SELECT FROM)
  • I have set ALLOW_DESTRUCTIVE_MIGRATION=1 in the CI environment for this PR
  • Justification (required if box above is ticked):
    <!-- Why this destructive change is necessary, what the rollback path is, and which release window it ships in -->
    

N/A: this PR is documentation only and authorizes no restore.

Reviewer Hint

Diff areas worth special attention from the reviewer:

  • Distinguish historical preparation statements from authoritative current-state handoffs.
  • Keep tooling/source evidence separate from release admission, provider guarantees and actual target acceptance.
  • Verify the digest claim is limited to authorized CI push/smoke-pull evidence; direct package REST inventory, permanent retention and off-device backup recovery are not claimed.
  • Self-improvement applied: explicit superseding state and scoped evidence claims prevent fixture success from appearing to authorize a destructive operation.
  • Design disposition accepted: evidence-only update; no new responsibilities, coupling, dependency or abstraction.

Prashant Rane (ALLYIS INC) and others added 2 commits November 3, 2026 12:52
… 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>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 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

@pranems
pranems marked this pull request as ready for review November 3, 2026 19:17
@pranems
pranems merged commit a80fafb into master Nov 3, 2026
6 checks passed
@pranems

pranems commented Nov 3, 2026

Copy link
Copy Markdown
Owner Author

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.

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.

2 participants