pravega / zookeeper-operator

Kubernetes Operator for Zookeeper
Apache License 2.0
368 stars 207 forks source link

default heap value for java #512

Open Thedin87 opened 2 years ago

Thedin87 commented 2 years ago

Description

The default heap size for java is 1 gb and if you set a higher ram limit in the cr the data on the pv will be corrupted. (Describe the feature, bug, question, proposal that you are requesting)

Importance

must-have (Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have))

Location

The env variable ZK_SERVERHEAP in the container. (Where is the piece of code, package, or document affected by this issue?)_

Suggestions for an improvement

The ZK_SERVERHEAP should be coordinated by the cr. (How do you suggest to fix or proceed with this issue?)_

lutartur commented 10 months ago

use env var ZK_SERVER_HEAP, it works

spec:
  image:
    repository: pravega/zookeeper
    tag: 0.2.13
    pullPolicy: IfNotPresent
  pod:
    env:
      - name: ZK_SERVER_HEAP
        value: "2000"
    resources:
      requests:
        cpu: 1
        memory: 6Gi
      limits:
        cpu: 2
        memory: 8Gi