Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Patch level image updates, improved tests #305

Merged
merged 2 commits into from
Nov 28, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions zammad/Chart.yaml
Original file line number Diff line number Diff line change
@@ -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
Expand All @@ -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
Expand All @@ -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
Expand Down
11 changes: 8 additions & 3 deletions zammad/templates/tests/run-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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')
Expand All @@ -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:
Expand Down
4 changes: 3 additions & 1 deletion zammad/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down