-
Notifications
You must be signed in to change notification settings - Fork 241
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(datahub-gms): Enable autoscaling via HPA #517
Conversation
{{/* | ||
Create image registry, name and tag for a datahub component | ||
*/}} | ||
{{- define "datahub.image" -}} | ||
{{- $registry := .image.registry | default .imageRegistry -}} | ||
{{ $registry }}/{{ .image.repository }}:{{ required "Global or specific tag is required" (.image.tag | default .version) -}} | ||
{{- end -}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the root chart has a similar template. I had to copy this so I could helm template
from within the subchart folder.
This looks good to me. I would be interested to know how this works in production, can GMS scale quick enough to help with load spikes. I can see this being helpful for long running ingestion runs for sure. Thank you! |
Hi David, first of all, thanks a lot for your time reviewing my contribution. I highly appreciate it. In regards to production, I suppose the default setting isn't the most helpful thing with |
Summary
datahub-project/datahub#11761
The company where I work for, started crashing datahub-gms during Snowflake ingestion, and I thought it would be handy to have an autoscaler for this workload. Hence, this pull request.
How to test it?
Edit the values.yaml of both datahub chart, and datahub-gms subchart. Make sure to enable
datahub-gms.hpa.enabled
andglobal.datahub_standalone_consumers_enabled
. Then, navige to the subchart folder and:Checklist