-
-
Notifications
You must be signed in to change notification settings - Fork 0
chore: Update Druid product version to 31.0.1 and adjust coordinator memory limit #685
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
Conversation
735c50a
to
15698d8
Compare
This comment was marked as outdated.
This comment was marked as outdated.
Oh, right. And since this PR, or rather all Druid 31.0.1 PRs, are blocked by this decision and it might take a while, I removed the OPA bump from this PR (was not really necessary anyway, but I thought this PR would be a quick one). I'll open a separate PR for the OPA bump. |
285aa14
to
2daed10
Compare
7c70e1f
to
911d483
Compare
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.
Thanks! Please also update https://docs.stackable.tech/home/stable/druid/usage-guide/resources-and-storage/
Fun fact: @NickLarsenNZ also just chatted me about increasing the memory default ;)
Co-authored-by: Sebastian Bernauer <sebastian.bernauer@stackable.de>
6f0d84b
to
cb4d110
Compare
Done 🙂 |
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.
Otherwise LGTM. Let's get this in before #693
Co-authored-by: Sebastian Bernauer <sebastian.bernauer@stackable.de>
Description
Part of stackabletech/docker-images#965
I had to increase the memory limit of the coordinator and middlemanager, otherwise the authorization integration test fails because the Pods get OOMKilled (happens on my local machine and on Replicated).
I also had to adapt the authorization integration test to support druid-opa-authorizer 0.6.0 (introduced by stackabletech/docker-images#984).
Definition of Done Checklist