-
Notifications
You must be signed in to change notification settings - Fork 20
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
The README is not up to date regarding the instruction #4 #25
Comments
@Razzorback225 have you managed to make it work? |
@alexszilagyi Yeap, I after I added the 2 missing parameters I got it to work. But otherwise the quickstart script does not do anything without them. |
@Razzorback225 would you mind submitting a PR with the change that fixed the issue? |
@dcaputo-harmoni Sure ! |
@Razzorback225 Hey what steps did you take to get this to work? I am able to run the quickstart-helm after adding the missing parameters and then when I try helm install I am getting the error below
If you have a step by step guide that would be awesome. I'm still learning helm and kubernetes. |
In the following instruction on the ReadMe :
The minimum parameters to include in the quickstart command is not correct as the user is supposed to add also the 'S3_ACCESS_KEY' and 'S3_SECRET_KEY' as asked by the following condition in the quickstart script (line 54 to 57):
if [ -z "${VERSION}" ] || [ -z "${JWT_SECRET}" ]|| [ -z "${S3_ACCESS_KEY}" ]|| [ -z "${S3_SECRET_KEY}" ]; then usage exit 1 fi
The helper is also not up to date.
The text was updated successfully, but these errors were encountered: