Closed paulliwog closed 1 month ago
LGTM! This shouldn't affect functionality, but it does keep consistent naming across Deepgram resources.
Could you add a line to the CHANGELOG.md under a new header called "Unreleased" describing the change in the specified format there?
I'll get that added. This was impacting functionality in our cluster as we are using Karpenter Autoscaler; the name mismatch prevented karpenter from scheduling the engine pod, so it wouldn't spin up a new node for it.
That's good to know! This hasn't been explicitly tested with Karpenter, so I wasn't aware of that. Thanks for the heads up!
In that case, if you have autoscaling enabled to include the HPA manifests, make sure to explicitly disable the cluster-autoscaler dependency.
Thank you! I'll cut a release for this later today.
Thanks for your patience on the Changelog request. The other open PR in this report will include a few more CI checks to help guide PR submissions to include that 😀
This change fixes a template issue when using AWS EFS as the storage for self-hosted deployments