helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

Document more NFS settings

Open fiskhest opened this issue 1 year ago • 2 comments

Is your feature request related to a problem? Please describe. When migrating to the new helm chart, I had to find and set a whole load of settings for the installation to succeed

Describe the solution you'd like A parameter key which will automatically template in these settings

Describe alternatives you've considered Adding some comments to values.yaml which shows examples on how to enable this

Additional context These are all settings I had to set to get things working in my environment. Anywhere a uid or gid is specified, it should match the NFS directory owner/group id

redis:
  enabled: true
  containerSecurityContext:
    enabled: true
    runAsGroup: 1000
    runAsUser: 1000

  securityContext:
    enabled: true
    fsGroup: 1000
    runAsUser: 1000

  volumePermissions:
    enabled: true

    ## @param redis.master.persistence.subPath Subpath in PVC; necessary if using single_pvc (set it to `/redis`)
    persistence:
      enabled: true
      size: 1Gi
      storageClass: "redis-pv"
      accessModes: [ReadWriteOnce]
      existingClaim: "redis-hdd"
      subPath: "redis"  # <--- I specifically got an error about using relative paths, so I had to remove the leading slash(!?)

dovecot:
  enabled: true

  securityContext:
    capabilities:
      add:
        - SYS_CHROOT
  overrides:
    dovecot.conf: |
      # More info here: https://mailu.io/1.8/kubernetes/mailu/index.html#dovecot  <-- this link is dead
      mail_nfs_index = yes
      mail_nfs_storage = yes
      mail_fsync = always
      mmap_disable = yes
      mail_max_userip_connections=100
      mail_uid = 1000
      mail_gid = 1000

fiskhest avatar Jul 31 '23 18:07 fiskhest

Thank you for the feedback This chart is using Bitnami's Redis chart as a dependency to deploy Redis. You can find configuration options here: https://artifacthub.io/packages/helm/bitnami/redis Could you maybe create a PR with an updated to the values.yaml file with the necessary comments on what needs to be done to make it work with NFS specifically? Or in the README.md

Thanks

fastlorenzo avatar Aug 11 '23 22:08 fastlorenzo

Thanks. I'll see what I can achieve.

fiskhest avatar Aug 11 '23 22:08 fiskhest

Is this issue still relevant?

WebSpider avatar Sep 09 '24 07:09 WebSpider