search

jfrog / charts

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

[jfrog-platform] Typo to get fourth parameter for setupPostgres.sh #1922

Closed neolutin closed 1 month ago

neolutin commented 2 months ago

PR Checklist

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

What this PR does / why we need it:

To be able to use the setup DB script manually, the parameters should be correctly mapped.

Which issue this PR fixes:

In ./stable/jfrog-platform/files/setupPostgres.sh, the CHART_NAME is either the value of the variable if defined, otherwise '4'. It would be better to be the fourth parameter of the script if not defined.

github-actions[bot] commented 2 months ago

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

neolutin commented 2 months ago

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

RobinDuhan commented 2 months ago

Thanks @neolutin, Acknowledging, will take this.