Merge pull request #6 from holysoles/fix_indent_valkey_subchart_values

fixes for the valkey values section

Author: jessebot

charts/pixelfed/Chart.yaml

@@ -15,7 +15,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.3.0
+version: 0.3.1
 
 # This is the version number of the application being deployed.
 # renovate:image=ghcr.io/mattlqx/docker-pixelfed

charts/pixelfed/README.md

@@ -1,6 +1,6 @@
 # pixelfed
 
-![Version: 0.3.0](https://img.shields.io/badge/Version-0.3.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v0.12.3-nginx](https://img.shields.io/badge/AppVersion-v0.12.3--nginx-informational?style=flat-square)
+![Version: 0.3.1](https://img.shields.io/badge/Version-0.3.1-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v0.12.3-nginx](https://img.shields.io/badge/AppVersion-v0.12.3--nginx-informational?style=flat-square)
 
 A Helm chart for deploying Pixelfed on Kubernetes
 
@@ -172,21 +172,22 @@ A Helm chart for deploying Pixelfed on Kubernetes
 | valkey.auth.enabled | bool | `true` |  |
 | valkey.auth.existingSecret | string | `""` |  |
 | valkey.auth.existingSecretPasswordKey | string | `"password"` |  |
-| valkey.auth.metrics.enabled | bool | `false` |  |
-| valkey.auth.persistentVolumeClaimRetentionPolicy.enabled | bool | `true` |  |
-| valkey.auth.persistentVolumeClaimRetentionPolicy.whenDeleted | string | `"Retain"` |  |
-| valkey.auth.persistentVolumeClaimRetentionPolicy.whenScaled | string | `"Retain"` |  |
-| valkey.auth.primary.persistence.enabled | bool | `true` |  |
-| valkey.auth.primary.persistence.existingClaim | string | `""` |  |
-| valkey.auth.replica.persistence.enabled | bool | `true` |  |
-| valkey.auth.replica.persistence.existingClaim | string | `""` |  |
-| valkey.auth.resourcesPreset | string | `"small"` |  |
-| valkey.auth.tls.authClients | bool | `true` |  |
-| valkey.auth.tls.autoGenerated | bool | `false` |  |
-| valkey.auth.tls.enabled | bool | `false` |  |
 | valkey.enabled | bool | `true` | enable the bundled valkey sub chart from Bitnami. Must set to true if externalValkey.enabled=false |
 | valkey.fullnameOverride | string | `"valkey"` |  |
 | valkey.global.storageClass | string | `""` |  |
+| valkey.metrics.enabled | bool | `false` | we use a grafana exporter that logs into valkey directly, but you can enable this if you don't use that |
+| valkey.persistentVolumeClaimRetentionPolicy.enabled | bool | `true` |  |
+| valkey.persistentVolumeClaimRetentionPolicy.whenDeleted | string | `"Retain"` |  |
+| valkey.persistentVolumeClaimRetentionPolicy.whenScaled | string | `"Retain"` |  |
+| valkey.primary.disableCommands | list | `["FLUSHALL"]` | Laravel requires the ability to call FLUSHDB, which is disabled by default |
+| valkey.primary.persistence.enabled | bool | `true` |  |
+| valkey.primary.persistence.existingClaim | string | `""` |  |
+| valkey.replica.persistence.enabled | bool | `true` |  |
+| valkey.replica.persistence.existingClaim | string | `""` |  |
+| valkey.resourcesPreset | string | `"small"` | definitions: https://github.com/bitnami/charts/blob/main/bitnami/common/templates/_resources.tpl#L15 Options: nano, micro, small, medium, large, xlarge, 2xlarge default: nano |
+| valkey.tls.authClients | bool | `true` |  |
+| valkey.tls.autoGenerated | bool | `false` |  |
+| valkey.tls.enabled | bool | `false` |  |
 | volumeMounts | list | `[]` | Additional volumeMounts on the output Deployment definition. |
 | volumes | list | `[]` | Additional volumes on the output Deployment definition. |
 

charts/pixelfed/values.yaml

@@ -187,38 +187,42 @@ valkey:
     enabled: true
     existingSecret: ""
     existingSecretPasswordKey: "password"
-    # TLS settings
-    tls:
-      enabled: false
-      authClients: true
-      autoGenerated: false
-
-    # primary (control plane) configuration
-    primary:
-      persistence:
-        enabled: true
-        existingClaim: ""
-
-    # valkey replica configuration
-    replica:
-      persistence:
-        enabled: true
-        existingClaim: ""
-
-    # persistnent volume retention policy for the StatefulSet
-    persistentVolumeClaimRetentionPolicy:
+    
+  # TLS settings
+  tls:
+    enabled: false
+    authClients: true
+    autoGenerated: false
+
+  # primary (control plane) configuration
+  primary:
+    # -- Laravel requires the ability to call FLUSHDB, which is disabled by default
+    disableCommands:
+      - FLUSHALL
+    persistence:
+      enabled: true
+      existingClaim: ""
+
+  # valkey replica configuration
+  replica:
+    persistence:
       enabled: true
-      whenScaled: Retain
-      whenDeleted: Retain
+      existingClaim: ""
 
-    metrics:
-      # we use a grafana exporter that logs into valkey directly
-      enabled: false
+  # persistnent volume retention policy for the StatefulSet
+  persistentVolumeClaimRetentionPolicy:
+    enabled: true
+    whenScaled: Retain
+    whenDeleted: Retain
+
+  metrics:
+    # -- we use a grafana exporter that logs into valkey directly, but you can enable this if you don't use that
+    enabled: false
 
-    # definitions: https://github.com/bitnami/charts/blob/main/bitnami/common/templates/_resources.tpl#L15
-    # Options: nano, micro, small, medium, large, xlarge, 2xlarge
-    # default: nano
-    resourcesPreset: "small"
+  # -- definitions: https://github.com/bitnami/charts/blob/main/bitnami/common/templates/_resources.tpl#L15
+  # Options: nano, micro, small, medium, large, xlarge, 2xlarge
+  # default: nano
+  resourcesPreset: "small"
 
 
 postgresql: