increasing memory allocation

[dehvCurtis]

increased default memory allocation values in the snyk broker, 256Mi is not sufficient.

[CLAassistant]

All committers have signed the CLA.

[aarlaud]

I read this again, delete my previous response. I'm not sure I can push this to 1G without having following complaints for existing users, but I'm curious to know what motivated that PR specifically.
What did you observe specifically? If you can elaborate on your setup, single broker, HA mode, etc

thanks

[dehvCurtis]

Hello @aarlaud thanks so much for your response. I am on a single broker (wasn't aware that Snyk offered HA :) ), I'm experiencing inconsistent restarts with nothing in logs or events. when looking at it and comparing to other similar applications on clusters I manage, I determined 256Mi was a bit low for Enterprise and that could be the cause. After upping my env, I was able to resolve the issue.

[dehvCurtis]

@aarlaud after we spoke, i lowered my specs down to 256MB again and began to experience resource issues once again, such as delayed page loading, delayed cloning, etc. please keep in mind this is a burstable configuration, so existing users are allocating 1GB, they burst to 1GB based on conditions. Currently, the chart is locked as "guaranteed" QoS.

https://github.com/snyk/snyk-broker-helm/blob/main/charts/snyk-broker/values.yaml#L266C1-L273C21

Let me know what you think.