docs: 12 PR 누적 동기화 (README + HANDOFF + TASKS) - #13
Conversation
본 세션(2026-04-30) 12 PR 머지 결과를 docs에 통합 반영. 변경: - README.md "Quickstart (Helm — P1-4 alpha)" 섹션 신설 — helm install + LibPQExecutor opt-in + multi-cluster env var 패턴 안내 - HANDOFF.md "현재 상태" + "본 세션 머지된 PR 12개" 표 + "권장 완료율" 갱신, "다음 단계 후보" 7개 + 권장 진입(후보 1 e2e cert-manager) 명시 - TASKS.md "P1" 표에 P1-4/P1-6 완료 마킹 + commit hash 매핑 본 세션 최종 누적: - 12 PR 머지 - P0 100% (6/6) — Status reason / SecurityContext / NetworkPolicy / Cascade / AuthPlugin Rotate / LibPQExecutor(phase 1+2a+2b) - P1 33% (2/6) — P1-4 Helm chart, P1-6 BackupOptions.ExecutionMode - 차별화 1 phase 2 완전 완성 (multi-cluster ctx-based DSN lookup) - ADR 0006/0007/0008/0009 (4 신설) - 글로벌 §2 정합 (.github/workflows/ 0개, 로컬 4 계층 활성) 다음 세션 권장 진입: 후보 1 — e2e cert-manager 통합 (PR #1 e2e fail 진짜 fix). Refs: workflow.md §2 종료 의식, plan §14 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b7d8209c81
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| # value: "host=..." | ||
|
|
||
| # 4. Sample PostgresCluster 적용 | ||
| kubectl apply -f config/samples/postgres_v1alpha1_postgrescluster.yaml |
There was a problem hiding this comment.
Point Quickstart to an existing sample manifest
This Quickstart command references config/samples/postgres_v1alpha1_postgrescluster.yaml, but that file is not present in the repo (the available samples are ..._dev.yaml and ..._prod.yaml), so users following the new Helm path will fail at this step with a file-not-found error and cannot complete the first end-to-end apply.
Useful? React with 👍 / 👎.
| helm upgrade my-operator ./charts/postgresql-operator \ | ||
| --reuse-values \ | ||
| --set citusLibPQ.dsn="host=<coord-svc-dns> port=5432 user=postgres dbname=postgres sslmode=require" |
There was a problem hiding this comment.
Keep Helm upgrade in the same release namespace
Step 1 installs the release into postgres-operator-system, but Step 2 runs helm upgrade without --namespace; Helm commands are namespace-scoped (-n, --namespace is the scope for the request, and release lookup follows current context namespace), so this upgrade will fail for users whose active namespace is still default instead of postgres-operator-system.
Useful? React with 👍 / 👎.
본 세션 12 PR 머지 결과를 docs에 통합 반영. workflow.md §2 종료 의식.
변경
README.mdHelm Quickstart 섹션 신설 (P1-4 alpha)HANDOFF.md12 PR 매트릭스 + 권장 완료율 + 다음 단계 후보 7개TASKS.mdP1-4/P1-6 완료 마킹 + commit hash본 세션 최종 누적
다음 세션 권장 진입
후보 1 — e2e cert-manager 통합 (PR #1 e2e fail의 진짜 fix).
🤖 Generated with Claude Code