-
Notifications
You must be signed in to change notification settings - Fork 15
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
Add GITBASE_READONLY env var #59
Conversation
@@ -142,6 +142,8 @@ spec: | |||
value: "{{ .Values.gitbaseServer.repositoriesDir }}" | |||
- name: GITBASE_UNSTABLE_SQUASH_ENABLE | |||
value: "{{ .Values.gitbaseServer.squashEnable }}" | |||
- name: GITBASE_READONLY |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please enclose this in a conditional to avoid adding an empty env variable if gitbaseServer.readonly
is empty
My bad for not having enforced this in your previous PR around GITBASE_UNSTABLE_SQUASH_ENABLE
gitbase-playground/values.yaml
Outdated
@@ -77,6 +77,7 @@ gitbaseServer: | |||
size: 10Gi | |||
logLevel: debug | |||
squashEnable: "" | |||
readonly: "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment this out, just signaling it is a possibility to set it. It does not make sense to set env variables to empty values
Signed-off-by: Carlos Martín <carlos.martin.sanchez@gmail.com>
c46e585
to
a08de30
Compare
I've made the requested changes, rebasing to have 1 commit only. |
Thx for fixing the squash variable too 😍 |
@@ -140,8 +140,14 @@ spec: | |||
value: "{{ .Values.gitbaseServer.logLevel }}" | |||
- name: GITBASE_REPOS | |||
value: "{{ .Values.gitbaseServer.repositoriesDir }}" | |||
{{- if ..Values.gitbaseServer.squashEnable }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it work? Here is 2 dots...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It shouldn't... Well spotted! 😅
@carlosms Can you please send a fix?
Good catch. Fix here: #61 |
Fix #58.