Set default memory resources for the ZooKeeper quorum#2463
Conversation
Hello delthas,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Request integration branchesWaiting for integration branch creation to be requested by the user. To request integration branches, please comment on this pull request with the following command: Alternatively, the |
|
is this really the issue we had in production? If Zookeeper were actually "easting" the memory, it would have affected the system already (i.e. prevented others from using memory); or is there something else causing this to happen now ? |
e2e9eb6 to
89a1365
Compare
|
Requested @DarkIsDude in place of Maha Benzekri, who is currently on PTO. |
this is the information which was missing! thanks! |
89a1365 to
a4ff9db
Compare
|
The |
Brings support for the ZenkoVersion defaults.zookeeperResources field. Issue: ZENKO-5316
Without these, the quorum pods run BestEffort and the zookeeper image sizes its JVM heap as a percentage of the host memory instead of the container limit. Requires the zenko-operator supporting defaults.zookeeperResources. Issue: ZENKO-5316
a4ff9db to
4d12cfb
Compare
What does this PR do, and why do we need it?
Bumps zenko-operator to v1.8.14 and sets
defaults.zookeeperResourcesinsolution/zenkoversion.yaml, alongside the existingdefaults.kafkaResources. Without it the quorum pods run BestEffort and the zookeeper image sizes its JVM heap as a percentage of the host memory instead of the container limit (RD-1962 finding, confirmed fleet-wide on ~240 sosreports).Which issue does this PR fix?
Fixes ZENKO-5316.
Special notes for your reviewers:
zenko-operator v1.8.14 (scality/zenko-operator#624) is what applies the field to the ZookeeperCluster; the bump commit lands before the field is set. Sizing rationale and fleet data: see the review thread.
Issue: ZENKO-5316