Skip to content

Commit 3e3fdb5

Browse files
authored
Merge pull request #474 from QuantStrategyLab/codex/reconciliation-log-filter-20260905
fix: retrieve reconciliation receipts before exact matching
2 parents 0e13264 + eccd1e0 commit 3e3fdb5

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

.github/workflows/collect-reconciliation-evidence.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ jobs:
150150
for _ in $(seq 1 96); do
151151
logging_output=''
152152
logging_read_status=0
153-
logging_output="$(gcloud logging read "resource.type=\"cloud_run_revision\" AND resource.labels.service_name=\"${SERVICE}\" AND timestamp>=\"${REQUESTED_AT}\" AND textPayload:\"reconciliation_receipt_ready request_id=${REQUEST_ID} report_uri=gs://\"" --project "$GCP_PROJECT_ID" --freshness=15m --limit=10 --format=json 2>/dev/null)" || logging_read_status=$?
153+
logging_output="$(gcloud logging read "resource.type=\"cloud_run_revision\" AND resource.labels.service_name=\"${SERVICE}\" AND timestamp>=\"${REQUESTED_AT}\" AND textPayload:\"reconciliation_receipt_ready\"" --project "$GCP_PROJECT_ID" --freshness=15m --limit=10 --format=json 2>/dev/null)" || logging_read_status=$?
154154
if [ "$logging_read_status" -ne 0 ]; then
155155
logging_read_failures=$((logging_read_failures + 1))
156156
sleep 5

tests/test_reconciliation_evidence_workflow.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,11 +92,12 @@ def test_reconciliation_evidence_binds_scheduler_request_to_exact_receipt() -> N
9292
"X-QSL-Reconciliation-Request-Id=${request_id}",
9393
"echo \"request_id=$request_id\"",
9494
"REQUEST_ID: ${{ steps.scheduler.outputs.request_id }}",
95-
"reconciliation_receipt_ready request_id=${REQUEST_ID} report_uri=gs://",
95+
"reconciliation_receipt_ready request_id=\" + $request_id + \" report_uri=gs://",
9696
"--arg request_id \"$REQUEST_ID\"",
9797
".reconciliation_request_id == $request_id",
9898
)
9999
for marker in required_markers:
100100
assert marker in workflow
101101

102-
assert 'textPayload:\"execution_report gs://\"' not in workflow
102+
assert 'textPayload:\\\"reconciliation_receipt_ready\\\"' in workflow
103+
assert 'textPayload:\\\"reconciliation_receipt_ready request_id=${REQUEST_ID}' not in workflow

0 commit comments

Comments
 (0)