jfrog / charts

JFrog official Helm Charts
https://jfrog.com/integration/helm-repository/
Apache License 2.0
249 stars 436 forks source link

[xray] Incorrect formatting for `volumeMounts` for panoramic #1895

Open rhys96 opened 2 weeks ago

rhys96 commented 2 weeks ago

Is this a request for help?:

Yes, I will submit a PR shortly.

Is this a BUG REPORT or FEATURE REQUEST? (choose one):

BUG REPORT.

Version of Helm and Kubernetes:

$ helm version

version.BuildInfo{Version:"v3.11.0", GitCommit:"472c5736ab01133de504a826bd9ee12cbe4e7904", GitTreeState:"clean", GoVersion:"go1.18.10"}

Kubernetes version: 1.29

Which chart:

xray.

Which product license (Enterprise/Pro/oss):

Enterprise.

JFrog support reference (if already raised with support team):

Haven't raised.

What happened:

When attempting to install the xray chart when using custom volumes, I encountered the following error:

Error: INSTALLATION FAILED: YAML parse error on xray/templates/xray-statefulset.yaml: error converting YAML to JSON: yaml: line 475: did not find expected key

What you expected to happen:

I expected the following output:

NAME: xray
LAST DEPLOYED: Mon Jul  1 16:51:42 2024
NAMESPACE: jfrog
STATUS: deployed
REVISION: 1
TEST SUITE: None
NOTES:
Congratulations! JFrog Xray services are deployed!

How to reproduce it (as minimally and precisely as possible):

Create EKS cluster with secrets-store CSI driver.

Create secret containing the trusted certificate for RDS.

NOTE: We use RDS in our deployment, but any additional volume mount will cause the xray deployment to fail.

Create values.yaml file.

---
global:
  customVolumes: |
    - name: rds-cert-bundle
      csi:
        driver: secrets-store.csi.k8s.io
        readOnly: true
        volumeAttributes:
          secretProviderClass: rds-cert-bundle-secret

  customVolumeMounts: |
    - name: rds-cert-bundle
      mountPath: "/mnt/trusted-certs"
      readOnly: true

xray:
  jfrogUrl: https://<URL>
  joinKey: EEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEE

Install xray.

$ helm repo add jfrog https://charts.jfrog.io
$ helm repo update
$ helm install -f values.yaml -f sizing/xray-xsmall.yaml xray jfrog/xray

Anything else we need to know:

I've narrowed down the issue to Line 1039 in xray-statefulset.yaml. The initial volume mount is out of alignment compared to the custom volume mounts.

To identify that formatting error:

Debug the xray chart output.

$ helm template xray jfrog/xray -f values.yaml -f sizing/xray-xsmall.yaml --debug > output.yaml

Run yamllint against the outputted YAML file.

$ yamllint output.yaml
...
780:9     error    syntax error: expected <block end>, but found '-' (syntax)
...

Show the error.

$ cat -n output.yaml | sed `777,782!d`
777          volumeMounts:
778            - name: data-volume
779              mountPath: "/var/opt/jfrog/xray"
780          - name: rds-cert-bundle
781            mountPath: "/mnt/trusted-certs"
782            readOnly: true

Recommended change:

Original

volumeMounts:
  - name: data-volume
    mountPath: "{{ .Values.xray.persistence.mountPath }}"

Fixed

volumeMounts:
- name: data-volume
  mountPath: "{{ .Values.xray.persistence.mountPath }}"
rahulsadanandan commented 2 weeks ago

@rhys96 Thanks for the PR. We will check this internally and will fix this in the upcoming versions of Xray. cc @megha1906