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

[xray] Fix for panoramic env indentation #1919

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

3drake
Copy link

@3drake 3drake commented Sep 10, 2024

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.]

  • Chart Version bumped
  • CHANGELOG.md updated
  • Variables and other changes are documented in the README.md
  • Title of the PR starts with chart name (e.g. [artifactory])

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:

Copy link

github-actions bot commented Sep 10, 2024

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

@3drake
Copy link
Author

3drake commented Sep 10, 2024

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

@megha1906
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants