Replace forecasting_db grants with FORECASTING_API_KEY in reports deploy
Reports no longer queries forecasting_db directly — it calls the forecasting public API. Remove GRANT SELECT and FORECAST_DATABASE_URL; add FORECASTING_URL and FORECASTING_API_KEY (provisioned separately in the forecasting app's API key manager). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
parent
fe0e5f5e71
commit
a40bf2f8cf
1 changed files with 3 additions and 4 deletions
|
|
@ -1446,10 +1446,8 @@ deploy_reports() {
|
||||||
\"CREATE DATABASE reports_db OWNER reports;\" 2>/dev/null || true
|
\"CREATE DATABASE reports_db OWNER reports;\" 2>/dev/null || true
|
||||||
docker exec hotel-manage-postgres psql -U postgres -d reports_db -c \
|
docker exec hotel-manage-postgres psql -U postgres -d reports_db -c \
|
||||||
\"GRANT ALL ON SCHEMA public TO reports;\" 2>/dev/null || true
|
\"GRANT ALL ON SCHEMA public TO reports;\" 2>/dev/null || true
|
||||||
docker exec hotel-manage-postgres psql -U postgres -d forecasting_db -c \
|
|
||||||
\"GRANT SELECT ON newbook_bookings_stats, newbook_net_revenue_data, forecasts, monthly_budgets, daily_budgets TO reports;\" 2>/dev/null || true
|
|
||||||
" &>/dev/null
|
" &>/dev/null
|
||||||
msg_ok "Database reports_db ready + read access to forecasting_db"
|
msg_ok "Database reports_db ready"
|
||||||
|
|
||||||
msg_info "Deploying reports"
|
msg_info "Deploying reports"
|
||||||
deploy_service 122 "reports" "${REPO_ROOT}/reports" /opt/reports
|
deploy_service 122 "reports" "${REPO_ROOT}/reports" /opt/reports
|
||||||
|
|
@ -1458,7 +1456,8 @@ deploy_reports() {
|
||||||
NODE_ENV=production
|
NODE_ENV=production
|
||||||
APP_SLUG=reports
|
APP_SLUG=reports
|
||||||
DATABASE_URL=postgresql://reports:${REPORTS_DB_PASS}@10.10.10.100:5432/reports_db
|
DATABASE_URL=postgresql://reports:${REPORTS_DB_PASS}@10.10.10.100:5432/reports_db
|
||||||
FORECAST_DATABASE_URL=postgresql://reports:${REPORTS_DB_PASS}@10.10.10.100:5432/forecasting_db
|
FORECASTING_URL=http://10.10.10.113:3080
|
||||||
|
FORECASTING_API_KEY=${FORECASTING_API_KEY:-}
|
||||||
CENTRAL_AUTH_SECRET=${CENTRAL_AUTH_SECRET}
|
CENTRAL_AUTH_SECRET=${CENTRAL_AUTH_SECRET}
|
||||||
SETTINGS_URL=http://10.10.10.116:3080
|
SETTINGS_URL=http://10.10.10.116:3080
|
||||||
SETTINGS_SECRET=${SETTINGS_SECRET}
|
SETTINGS_SECRET=${SETTINGS_SECRET}
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue