jfrog / charts

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

[xray] Fix for panoramic env indentation #1919

Open 3drake opened 2 months ago

3drake commented 2 months ago

De-indents the ENV section for Panoramic from 10 to 8 to be in-line with the other containers

Without it, it generates the following invalid YAML:

image

And throws the following error:

Error: YAML parse error on jfrog-platform/charts/xray/templates/xray-statefulset.yaml: error converting YAML to JSON: yaml: line 508: did not find expected key

PR Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

What this PR does / why we need it:

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

github-actions[bot] commented 2 months ago

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

3drake commented 2 months ago

I have read the CLA Document and I hereby sign the CLA

megha1906 commented 2 months ago

@3drake Thanks for the PR. We will check this internally and will fix this in the upcoming versions of Xray. cc @rahulsadanandan