Skip to content

Conversation

@RomanPszonka
Copy link
Collaborator

No description provided.

@RomanPszonka
Copy link
Collaborator Author

@hrishiballal would need to be tested online first before merging

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR migrates certificate management from cert-manager with nginx-ingress-controller to Caddy ingress controller, which provides automatic HTTPS certificate management. The PR also includes unrelated database configuration improvements.

  • Removes cert-manager Issuer and Certificate resources
  • Replaces nginx ingress with Caddy ingress controller
  • Standardizes database connection strings to use environment variables

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
templates/issuer-template.yaml Removed - cert-manager issuer configuration no longer needed with Caddy
templates/certificate-wcard-template.yaml Removed - Caddy handles wildcard certificates automatically
templates/certificate-root-template.yaml Removed - Caddy handles root domain certificates automatically
templates/ingress-template.yaml Migrated from nginx to Caddy ingress class, removed cert-manager annotations and TLS configuration
generate-from-templates.sh Updated to only generate ingress configuration, removed cert-manager template processing
README.md Updated installation instructions to use Caddy ingress controller instead of nginx + cert-manager
kustomize/passport/passport.yaml Added POSTGRES_PORT environment variable and updated DATABASE_URL to use it; removed trailing whitespace
kustomize/passport/db-passport.yaml Added POSTGRES_PORT environment variable
kustomize/blender/blender.yaml Added POSTGRES_PORT environment variable and updated DATABASE_URL to use it; removed trailing whitespace
kustomize/blender/worker.yaml Added POSTGRES_PORT environment variable and updated DATABASE_URL to use it; removed trailing whitespace
kustomize/blender/beat.yaml Added POSTGRES_PORT environment variable and updated DATABASE_URL to use it; removed trailing whitespace
kustomize/blender/db-blender.yaml Added POSTGRES_PORT environment variable

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@RomanPszonka RomanPszonka changed the title Switch to Caddy for managing certificates Switch to Caddy for managing certificates and fix env variables passing Dec 29, 2025
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