-
Notifications
You must be signed in to change notification settings - Fork 192
Open
Labels
enhancementNew feature or requestNew feature or request
Description
New Feature
The current Helm chart has some issues that make it difficult for newcomers to install.
- When ingress.create is true and service.port is not the default 3000, the containerPort in the deployment is incorrectly set to the service port. This should always be 3000 because that's what the application exposes itself as inside the container.
- Required environment variables are not present as examples (e.g. APP_ID, PRIVATE_KEY, and WEBHOOK_SECRET)
- Example validators are present and not commented out, so if a user doesn't want those validators, they have to explicitly set them to null instead of omit them.
- Min replica counts default to 2, but I would argue that 1 would be a more cost effective default.
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request