Skip to content

epic: enterprise hardening — trustworthy v0.4 release #305

Closed as not planned

Description

@m-khan-97

Goal

Make the next OpenShield release trustworthy, secure and operable for an enterprise deployment without adding product features for their own sake.

The current rule inventory is 95 unique rules with 95 matching playbooks. Rule count is not the release KPI: verified evidence coverage, explicit uncertainty, false-positive control and safe operation are.

Product boundary for this release

Recommended default: ship and document a securely enforced single-tenant enterprise deployment first. Do not claim multi-tenant SaaS until identity, ownership, tenant-scoped persistence and cross-tenant isolation tests exist.

Do not claim certification or "full compliance." Reports should describe versioned technical evidence coverage.

Phase 0 — containment and trust blockers

These block restoring or calling the current deployment enterprise-ready:

Phase 1 — durable enterprise core

Phase 2 — validate, then expand scanner coverage

Existing rule-pack work should remain reviewable, but it must consume the Phase 0 evaluation and collector contracts before merge:

Every repaired or new rule must include:

  • explicit applicability and PASS/FAIL/UNKNOWN/NOT_APPLICABLE behavior;
  • required permissions and 403/429/5xx/partial-collection tests;
  • real Azure SDK model contract tests and pagination coverage;
  • authoritative evidence source, version, timestamp and fingerprint;
  • false-positive analysis, exceptions and suppression expiry;
  • versioned framework mapping rationale;
  • preview-first, target-verified remediation with rollback and validation.

Enterprise release gates

The release stays blocked until all of the following are demonstrated:

  • No reusable credential is present in public frontend or website assets.
  • Authentication, RBAC and authorized subscription boundaries are integration-tested.
  • Failed or incomplete evidence cannot produce PASS, a clean scan, or score 100.
  • CRITICAL is consistent in scanner, database, APIs, reports and frontend.
  • The published image starts, reaches database-aware readiness, and completes a worker-backed real scan.
  • Required GitHub checks and two-person promotion are effective, not just documented.
  • Framework editions/mappings are versioned, reviewed and presented as technical evidence coverage.
  • SLOs, alerts, on-call ownership, backup restore and rollback drills have recorded evidence.
  • Existing 95 rules have an evidence/permission/SDK validation matrix.
  • No unresolved P0; every accepted P1 has named owner, rationale and expiry.

review model

  • Programme/risk/claim gate: named lead plus cross-stream reviewer.
  • Platform/release gate: named lead plus independent backup.
  • Scanner/evidence gate: named lead plus cross-stream reviewer.
  • Auth, scoring, compliance, infrastructure and release changes require review from two leads; authors do not self-approve.

Delivery order

  1. Contain credentials and unsafe public surfaces.
  2. Restore enforced merge controls.
  3. Repair scanner truth and severity semantics.
  4. Prove the runtime/deployment path end-to-end.
  5. Harden persistence, operations and evidence provenance.
  6. Validate all 95 rules.
  7. Resume rule expansion in small quality-gated batches.

Activity

  1. added
    coreCore team ownership not for students
    priority: criticalMust be fixed immediately, breaks core functionality
    on Aug 21, 2026
  2. m-khan-97 commented on Aug 21, 2026

    @m-khan-97
    CollaboratorAuthor

    First enterprise-hardening implementation batch is ready for lead review. Nothing has been merged yet.

    • fix(api): return readiness connections to the pool #306 fixes the public /ready connection-pool leak. Draft, DCO-signed, mergeable, and all 20 checks are green.
    • fix(core): enforce severity contract v1 #308 introduces severity contract v1 and repairs CRITICAL scoring across scanner, database, APIs, Sentinel, and dashboard. Draft, DCO-signed, mergeable, and all 20 checks are green. This needs a production-sized staging migration run before merge.
    • fix(platform): enforce container runtime gate #309 makes the image migration-aware and bootable, adds a local-only API/worker/frontend/PostgreSQL Compose contract, and turns container boot/readiness/non-root/Trivy into a required CI gate. Draft, DCO-signed, mergeable, and all 21 checks are green. Runtime evidence: /ready passed, runtime UID was 999, and Trivy found no blocking CRITICAL/HIGH result on the tested tag.

    Integration order:

    1. Review and merge fix(api): return readiness connections to the pool #306 first.
    2. Run fix(core): enforce severity contract v1 #308's migration against a production-sized staging copy, complete two-lead review, then merge it.
    3. Rebase fix(platform): enforce container runtime gate #309 onto the resulting dev, rerun the container gate so it exercises the combined migration chain, then review and merge it.

    Do not merge #309 before #306: its recurring Compose readiness probe depends on #306 returning probe connections to the pool.

    This is progress on Phase 0, not an enterprise-ready declaration. Public auth/token containment (#294), website/PAT containment (#297), effective branch enforcement (#298), evidence semantics (#263), and the remaining #299 deployment/worker gates are still release blockers.

  3. m-khan-97 commented on Aug 28, 2026

    @m-khan-97
    CollaboratorAuthor

    Enterprise-hardening status update — 28 August 2026

    Completed and merged into dev:

    Active release-hardening queue:

    Unowned release blockers still requiring lead assignment:

    The current priority remains stabilization of existing behavior and evidence correctness. Agent-automation issues #311–#315 should not displace these release blockers.

  4. m-khan-97 commented on Aug 31, 2026

    @m-khan-97
    CollaboratorAuthor

    Enterprise-hardening lead update — 31 August 2026

    Merged into dev since the previous update:

    Lead reviews completed today:

    Architecture coordination:

    Still blocked before review:

    Next lead focus after developer updates: rereview #319/#318 merge readiness if dev moves, review the reconciled #321/#325 stack, then return to #310 and #280 when their focused fixes land.

  5. github-actions commented on Oct 1, 2026

    @github-actions
    Contributor

    This issue has been inactive for 30 days and will be closed in 7 days unless there is activity.

  6. github-actions commented on Oct 9, 2026

    @github-actions
    Contributor

    Closing due to inactivity. Feel free to reopen if still relevant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    coreCore team ownership not for studentspriority: criticalMust be fixed immediately, breaks core functionalitystale

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions