Skip to content

Add query-log writer deployment manifests#808

Merged
bill-ph merged 1 commit into
mainfrom
codex/query-log-writer-deploy
Jun 25, 2026
Merged

Add query-log writer deployment manifests#808
bill-ph merged 1 commit into
mainfrom
codex/query-log-writer-deploy

Conversation

@bill-ph

@bill-ph bill-ph commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add disabled-by-default query-log writer Deployment, metrics Service, and example Kafka ConfigMap
  • wire optional DUCKGRES_QUERY_LOG_* env from the example ConfigMap into local, kind, and e2e control-plane manifests
  • add writer RBAC and network policy, a basic PrometheusRule example, and README runbook and rollback notes

Tests

  • go test ./tests/manifests
  • just test-unit
  • just lint

@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

Test Impact Plan

Deterministic summary of how this PR changes tests, CI runners, and coverage-risk signals.

Summary

Area Added Changed Deleted
Test files 1 2 0
E2E/journey files 0 1 0
Workflow files 0 0 0

Signals

  • Test cases: +9 / -0
  • Assertions: +16 / -0
  • Skips or known failures added: 0
  • Workflow continue-on-error added: 0
  • Workflow path filters added: 0
  • Test commands removed from justfile: 2
  • E2E/journey retry lines added: 0

Coverage risk: likely reduced

Warnings

  • Test command removed (likely reduced)
    • 2 test command line(s) removed from justfile
  • E2E or journey files changed (needs review)
    • tests/e2e-mw-dev/manifests.tmpl.yaml

@bill-ph bill-ph force-pushed the codex/query-log-writer-deploy branch from 2750d27 to ae2ea38 Compare June 25, 2026 18:03
@bill-ph bill-ph merged commit 0a5e9d9 into main Jun 25, 2026
27 checks passed
@bill-ph bill-ph deleted the codex/query-log-writer-deploy branch June 25, 2026 18:31
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.

1 participant