Skip to content

added Gotenberg service for gift voucher PDF rendering - #83

Open
TomasLudvik wants to merge 4 commits into
mainfrom
add-gotenberg-service
Open

added Gotenberg service for gift voucher PDF rendering#83
TomasLudvik wants to merge 4 commits into
mainfrom
add-gotenberg-service

Conversation

@TomasLudvik

Copy link
Copy Markdown
Member

Runs gotenberg/gotenberg (headless Chromium) as an internal service so the application can render the gift voucher PDF from HTML via sensiolabs/gotenberg-bundle. Adds the Deployment + Service, a GOTENBERG_VERSION variable, and registers them in the migrate-application kustomizations alongside redis/rabbitmq. The app reaches it at http://gotenberg:3000 (GOTENBERG_DSN).

Runs gotenberg/gotenberg (headless Chromium) as an internal service so the
application can render the gift voucher PDF from HTML via
sensiolabs/gotenberg-bundle. Adds the Deployment + Service, a GOTENBERG_VERSION
variable, and registers them in the migrate-application kustomizations
alongside redis/rabbitmq. The app reaches it at http://gotenberg:3000
(GOTENBERG_DSN).
Copilot AI lite review requested due to automatic review settings September 7, 2026 20:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The new GOTENBERG_VERSION defaulting block uses an unsafe -z test pattern that can error when the variable is unset, breaking deployments where the default should apply.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds a Gotenberg (headless Chromium) Kubernetes workload to the deployment package so the application can render gift voucher PDFs from HTML via the internal service endpoint (http://gotenberg:3000).

Changes:

  • Add Kubernetes Deployment + Service for gotenberg.
  • Register Gotenberg resources in the migrate-application kustomizations (first deploy, first deploy with demo data, continuous deploy).
  • Introduce GOTENBERG_VERSION variable with a default image value for manifest templating.
File summaries
File Description
kubernetes/services/gotenberg.yaml New internal ClusterIP Service exposing Gotenberg on port 3000.
kubernetes/deployments/gotenberg.yaml New Deployment running the Gotenberg container with health probes and resource settings.
kubernetes/kustomize/migrate-application/first-deploy/kustomization.yaml Includes Gotenberg Deployment/Service in first-deploy migration resources.
kubernetes/kustomize/migrate-application/first-deploy-with-demo-data/kustomization.yaml Includes Gotenberg Deployment/Service in demo-data migration resources.
kubernetes/kustomize/migrate-application/continuous-deploy/kustomization.yaml Includes Gotenberg Deployment/Service in continuous-deploy migration resources.
deploy/parts/kubernetes-variables.sh Adds defaulting + templating variable registration for GOTENBERG_VERSION.
Review details
  • Files reviewed: 6/6 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread deploy/parts/kubernetes-variables.sh Outdated
Comment on lines +16 to +18
if [ -z ${GOTENBERG_VERSION} ]; then
GOTENBERG_VERSION='gotenberg/gotenberg:8'
fi
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.

3 participants