Merge pull request #6 from holysoles/fix_indent_valkey_subchart_values

fixes for the valkey values section

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,6 +187,7 @@ valkey:
     enabled: true
     existingSecret: ""
     existingSecretPasswordKey: "password"
+    
   # TLS settings
   tls:
     enabled: false
@@ -195,6 +196,9 @@ valkey:
 
   # primary (control plane) configuration
   primary:
+    # -- Laravel requires the ability to call FLUSHDB, which is disabled by default
+    disableCommands:
+      - FLUSHALL
     persistence:
       enabled: true
       existingClaim: ""
@@ -212,10 +216,10 @@ valkey:
     whenDeleted: Retain
 
   metrics:
-      # we use a grafana exporter that logs into valkey directly
+    # -- 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
+  # -- 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"