Skip to content

Keep audit overflow observation tests correct across minute boundaries #390

Description

@samovers

Merged and verified: PR #391 merged at eea160152077523c97d877af851b1a2d511c075a on 2026-09-14T17:01:08Z through GitHub's expected-head merge after the original final packet and later exact same-task authorization. Delivery #390 is closed. The fresh pre-merge receipt recheck passed; new exact-head review 5200536975 reported zero findings. No runtime or scope change was added. Next: resume PR #388 with its own fresh applicable evidence.

The accepted final packet below is preserved as the pre-merge record.


PR #391 completes the approved test correction at c430cf8140b43d36f041bb92382c7cfdd1481eeb. This is a separate prerequisite for #387 / PR #388 under #180.

The capability is a correct final overflow observation check across minute rollover. Its sole boundary is test evidence for database-owned audit overflow observation and closure. Only kernel/tests/test_postgresql_audit_operations.py changes, +22/-2. PostgreSQL and the existing isolated fictional fixture retain their authorities.

T01 preserves four rows, exact ordered observations and closures, and wrong-role refusal. T02 brackets the final observation with database time: exact empty result before expiry, exact remaining row after expiry, only those two results across expiry; reversed clocks and wrong/duplicate/extra rows fail. T03 limits edits to that expectation and its two clock reads. No Phase A deviation, new test/helper/fixture/inventory, SQL/role/runtime change, audit restart, retry, skip or discarded probe machinery.

Decision OFARM2-AUDIT-OVERFLOW-CLOCK-TEST-001 version 1 was approved by the exact later task-user message after its complete card naming PR #391. It authorizes this implementation and evidence, with final merge acceptance still pending.

Completed evidence:

  • Focused module: 9 passed in 0.98s. The stable-current branch was measured over 0.7ms with []; crossing/expired branches were source-reviewed, not forced. Local owned PostgreSQL 17.10 resources were removed.
  • Package/architecture/temporal: PASS, zero failures, 8.79s. Ruff, whitespace and lightweight passed. Inventory unchanged at 4,483.
  • Exact-head implementation review: zero Blockers, Follow-ups or Preferences, report preserved in the PR body.
  • Fresh source 34837594371, attempt 1: two baselines of 4,483 passing tests, zero failures/skips, one inventoried warning each; equivalence and both native architectures passed.
  • Fresh publisher 34840469949, attempt 1: PASS. Read-only verification passed for the exact receipt, four source/five published artifact bindings, pinned environment and recomputed equivalence. Receipt artifact 10346046242, ZIP SHA256 2d79bbec31170105644d39c625ee97cb9d1f0fc76ac998290fc99f4acdb186b4.

The automatic ready-for-review workflow also succeeded and posted no new findings. Its hidden model output is not treated as additional inspected content review.

This is provisional development maintenance. No deployment/release authority or production-readiness claim. No workflow rerun or reuse of PR #388's failed source. PR #388 remains unchanged and requires its own fresh applicable evidence after this prerequisite is merged through its own gates.

Next: complete final human acceptance for PR #391 with the exact-head packet. Leave the PR and Delivery open until the later authorized native merge is verified.

Activity

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

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions