From 104f52c091bdc1136261b3fa95c2ac90529c3c70 Mon Sep 17 00:00:00 2001
From: Martin Gruner <mg@zammad.com>
Date: Thu, 28 Nov 2024 10:28:58 +0100
Subject: [PATCH] Patch level image updates, improved tests (#305)

* Patch level image updates, improved tests

* Add explanatory comment
---
 zammad/Chart.yaml                     |  8 ++++----
 zammad/templates/tests/run-tests.yaml | 11 ++++++++---
 zammad/values.yaml                    |  4 +++-
 3 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/zammad/Chart.yaml b/zammad/Chart.yaml
index 207776f5..9bc8ee99 100755
--- a/zammad/Chart.yaml
+++ b/zammad/Chart.yaml
@@ -1,7 +1,7 @@
 apiVersion: v2
 name: zammad
-version: 13.0.2
-appVersion: 6.4.0-27
+version: 13.0.3
+appVersion: 6.4.0-34
 description: Zammad is a web based open source helpdesk/customer support system with many features to manage customer communication via several channels like telephone, facebook, twitter, chat and e-mails.
 home: https://zammad.org
 icon: https://raw.githubusercontent.com/zammad/zammad-documentation/main/images/zammad_logo_600x520.png
@@ -16,7 +16,7 @@ maintainers:
 dependencies:
   - name: elasticsearch
     repository: https://charts.bitnami.com/bitnami
-    version: 21.3.25
+    version: 21.3.26
     condition: zammadConfig.elasticsearch.enabled
   - name: minio
     version: 14.8.5
@@ -27,7 +27,7 @@ dependencies:
     repository: https://charts.bitnami.com/bitnami
     condition: zammadConfig.memcached.enabled
   - name: postgresql
-    version: 16.2.1
+    version: 16.2.2
     repository: https://charts.bitnami.com/bitnami
     condition: zammadConfig.postgresql.enabled
   - name: redis
diff --git a/zammad/templates/tests/run-tests.yaml b/zammad/templates/tests/run-tests.yaml
index a94a3366..d94123ef 100644
--- a/zammad/templates/tests/run-tests.yaml
+++ b/zammad/templates/tests/run-tests.yaml
@@ -13,11 +13,15 @@ data:
         desc 'Runs a set of Helm tests'
         task test: :environment do |_task, args|
 
-          puts 'Checking if temporary file can be created…'
+          puts 'Checking if a Ruby Tempfile can be created…'
           Tempfile.create do |_f|
-            puts '  Temporary file was created successfully.'
+            puts '  Tempfile file was created successfully.'
           end
 
+          puts 'Checking if file can be created directly in /opt/zammad/tmp…'
+          File.write('/opt/zammad/tmp/test.txt', 'test content')
+          puts '  File was created in /tmp successfully.'
+
           {{- if .Values.zammadConfig.storageVolume.enabled }}
           puts 'Checking if storage file can be created…'
           File.write('/opt/zammad/storage/test.txt', 'test content')
@@ -42,7 +46,8 @@ spec:
     {{- include "zammad.podSpec.deployment" . | nindent 4 }}
     containers:
       - name: zammad-run-tests
-        {{- include "zammad.containerSpec" (merge (dict "containerConfig" dict) .) | nindent 8 }}
+        # Use securityContext etc. from railsserver to have identical setup.
+        {{- include "zammad.containerSpec" (merge (dict "containerConfig" .Values.zammadConfig.railsserver) .) | nindent 8 }}
         command: ['bundle']
         args: ['exec', 'rake', 'zammad:helm:test']
         env:
diff --git a/zammad/values.yaml b/zammad/values.yaml
index 8f5f6319..fb477fc4 100755
--- a/zammad/values.yaml
+++ b/zammad/values.yaml
@@ -327,7 +327,9 @@ zammadConfig:
             - ALL
         readOnlyRootFilesystem: true
         privileged: false
-    # volumePermissions will be used by all Zammad Pods
+    # VolumePermissions will be used by all Zammad Pods.
+    # We need it to drop global write permission that comes with EmptyDir, otherwise
+    #   Ruby's Tempfile.create raises an error.
     volumePermissions:
       enabled: true
       image: