Skip to content

Commit

Permalink
Fix exchange manager test PVC usage.
Browse files Browse the repository at this point in the history
  • Loading branch information
yardenc2003 authored and nineinchnick committed Sep 20, 2024
1 parent a3209c1 commit baf5762
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 17 deletions.
16 changes: 0 additions & 16 deletions charts/trino/templates/tests/exchange-manager-pvc.yaml

This file was deleted.

12 changes: 11 additions & 1 deletion test-exchange-manager-values.yaml
Original file line number Diff line number Diff line change
@@ -1,11 +1,21 @@
# Exchange Manager values to test.
# This is a YAML-formatted file.

coordinator:
additionalVolumes:
- name: exchange-volume
persistentVolumeClaim:
claimName: exchange-manager-pvc

additionalVolumeMounts:
- name: exchange-volume
mountPath: "/tmp/trino-local-file-system-exchange-manager"

worker:
additionalVolumes:
- name: exchange-volume
persistentVolumeClaim:
claimName: test-exchange-manager-pvc
claimName: exchange-manager-pvc

additionalVolumeMounts:
- name: exchange-volume
Expand Down
12 changes: 12 additions & 0 deletions test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,18 @@ openssl req -new -newkey rsa:4096 -days 365 -nodes -x509 \

kubectl create namespace "$NAMESPACE" --dry-run=client --output yaml | kubectl apply --filename -
kubectl -n "$NAMESPACE" create secret tls certificates --cert=cert.crt --key=cert.key --dry-run=client --output yaml | kubectl apply --filename -
cat <<YAML | kubectl -n "$NAMESPACE" apply -f-
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
name: exchange-manager-pvc
spec:
accessModes:
- ReadWriteOnce
resources:
requests:
storage: 128Mi
YAML

# only install the Prometheus Helm chart when running the `complete_values` test
if printf '%s\0' "${TEST_NAMES[@]}" | grep -qwz complete_values; then
Expand Down

0 comments on commit baf5762

Please sign in to comment.