fix(ddb): grant universal schema permissions to all roles
Adds SQL to postInitApplicationSQL granting schema permissions to PUBLIC. Allows any role (authentik, temporal, etc) to create tables in databases. For existing cluster: run SQL manually (done). For future bootstrap: automatic via initdb. Pattern for apps: Database CR + app-specific init Job optional (co-located).
This commit is contained in:
@@ -26,6 +26,11 @@ spec:
|
|||||||
- CREATE EXTENSION IF NOT EXISTS vector;
|
- CREATE EXTENSION IF NOT EXISTS vector;
|
||||||
- CREATE EXTENSION IF NOT EXISTS pgcrypto;
|
- CREATE EXTENSION IF NOT EXISTS pgcrypto;
|
||||||
- CREATE EXTENSION IF NOT EXISTS pg_trgm;
|
- CREATE EXTENSION IF NOT EXISTS pg_trgm;
|
||||||
|
# Universal fix: grant schema permissions to all roles
|
||||||
|
# Allows any role to create tables in public schema of any database
|
||||||
|
- GRANT ALL ON SCHEMA public TO PUBLIC;
|
||||||
|
- ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON TABLES TO PUBLIC;
|
||||||
|
- ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON SEQUENCES TO PUBLIC;
|
||||||
|
|
||||||
# Per-app login roles, passwords sourced from secrets (CNPG reconciles the
|
# Per-app login roles, passwords sourced from secrets (CNPG reconciles the
|
||||||
# role password to match the secret). Their databases are separate Database
|
# role password to match the secret). Their databases are separate Database
|
||||||
|
|||||||
@@ -1,99 +0,0 @@
|
|||||||
# PostSync Job to grant schema permissions after Database CRs reconcile
|
|
||||||
# CNPG Database CR creates DBs but doesn't set schema permissions correctly
|
|
||||||
apiVersion: v1
|
|
||||||
kind: ServiceAccount
|
|
||||||
metadata:
|
|
||||||
name: db-permissions
|
|
||||||
namespace: ddb
|
|
||||||
---
|
|
||||||
apiVersion: rbac.authorization.k8s.io/v1
|
|
||||||
kind: Role
|
|
||||||
metadata:
|
|
||||||
name: db-permissions
|
|
||||||
namespace: ddb
|
|
||||||
rules:
|
|
||||||
- apiGroups: [""]
|
|
||||||
resources: ["secrets"]
|
|
||||||
verbs: ["get"]
|
|
||||||
---
|
|
||||||
apiVersion: rbac.authorization.k8s.io/v1
|
|
||||||
kind: RoleBinding
|
|
||||||
metadata:
|
|
||||||
name: db-permissions
|
|
||||||
namespace: ddb
|
|
||||||
subjects:
|
|
||||||
- kind: ServiceAccount
|
|
||||||
name: db-permissions
|
|
||||||
namespace: ddb
|
|
||||||
roleRef:
|
|
||||||
kind: Role
|
|
||||||
name: db-permissions
|
|
||||||
apiGroup: rbac.authorization.k8s.io
|
|
||||||
---
|
|
||||||
apiVersion: batch/v1
|
|
||||||
kind: Job
|
|
||||||
metadata:
|
|
||||||
name: db-permissions
|
|
||||||
namespace: ddb
|
|
||||||
annotations:
|
|
||||||
argocd.argoproj.io/hook: PostSync
|
|
||||||
argocd.argoproj.io/hook-delete-policy: BeforeHookCreation
|
|
||||||
spec:
|
|
||||||
backoffLimit: 5
|
|
||||||
template:
|
|
||||||
spec:
|
|
||||||
serviceAccountName: db-permissions
|
|
||||||
restartPolicy: Never
|
|
||||||
containers:
|
|
||||||
- name: grant-permissions
|
|
||||||
image: postgres:16-alpine
|
|
||||||
command:
|
|
||||||
- /bin/sh
|
|
||||||
- -c
|
|
||||||
- |
|
|
||||||
set -e
|
|
||||||
|
|
||||||
echo "Granting schema permissions to app users..."
|
|
||||||
|
|
||||||
# Get app user password
|
|
||||||
export PGPASSWORD=$(cat /postgres-secret/password)
|
|
||||||
|
|
||||||
# Grant for authentik (as app user, owner of the DB)
|
|
||||||
psql -h ddb-cluster-rw -U app -d authentik << 'SQL'
|
|
||||||
GRANT ALL ON SCHEMA public TO app;
|
|
||||||
GRANT ALL ON SCHEMA public TO authentik;
|
|
||||||
ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON TABLES TO app;
|
|
||||||
ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON SEQUENCES TO app;
|
|
||||||
ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON TABLES TO authentik;
|
|
||||||
ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON SEQUENCES TO authentik;
|
|
||||||
SQL
|
|
||||||
|
|
||||||
# Grant for temporal
|
|
||||||
psql -h ddb-cluster-rw -U app -d temporal << 'SQL'
|
|
||||||
GRANT ALL ON SCHEMA public TO app;
|
|
||||||
GRANT ALL ON SCHEMA public TO temporal;
|
|
||||||
ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON TABLES TO app;
|
|
||||||
ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON SEQUENCES TO app;
|
|
||||||
ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON TABLES TO temporal;
|
|
||||||
ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON SEQUENCES TO temporal;
|
|
||||||
SQL
|
|
||||||
|
|
||||||
# Grant for temporal_visibility
|
|
||||||
psql -h ddb-cluster-rw -U app -d temporal_visibility << 'SQL'
|
|
||||||
GRANT ALL ON SCHEMA public TO app;
|
|
||||||
GRANT ALL ON SCHEMA public TO temporal;
|
|
||||||
ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON TABLES TO app;
|
|
||||||
ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON SEQUENCES TO app;
|
|
||||||
ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON TABLES TO temporal;
|
|
||||||
ALTER DEFAULT PRIVILEGES IN SCHEMA public GRANT ALL ON SEQUENCES TO temporal;
|
|
||||||
SQL
|
|
||||||
|
|
||||||
echo "Permissions granted successfully"
|
|
||||||
volumeMounts:
|
|
||||||
- name: postgres-secret
|
|
||||||
mountPath: /postgres-secret
|
|
||||||
readOnly: true
|
|
||||||
volumes:
|
|
||||||
- name: postgres-secret
|
|
||||||
secret:
|
|
||||||
secretName: ddb-cluster-app
|
|
||||||
@@ -8,7 +8,6 @@ namespace: ddb
|
|||||||
# GitOps-managed database schemas (ArgoCD wave 6).
|
# GitOps-managed database schemas (ArgoCD wave 6).
|
||||||
# These depend on ddb-cluster existing (bootstrap wave 0).
|
# These depend on ddb-cluster existing (bootstrap wave 0).
|
||||||
resources:
|
resources:
|
||||||
- db-permissions-job.yaml
|
|
||||||
- authentik-database.yaml
|
- authentik-database.yaml
|
||||||
- temporal-database.yaml
|
- temporal-database.yaml
|
||||||
- temporal-visibility-database.yaml
|
- temporal-visibility-database.yaml
|
||||||
|
|||||||
Reference in New Issue
Block a user