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.

Sign up for GitHub

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

Allow override of validator image nitro #5

Merged
merged 3 commits into from
Jan 27, 2024
Merged

Allow override of validator image nitro #5

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
ccecaa5
Allow override of validator image nitro
a-thomas-22 Jan 27, 2024
0dcd787
tag metadata
a-thomas-22 Jan 27, 2024
f74f68e
Update README and values.schema.json for modified charts
a-thomas-22 Jan 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion charts/nitro/Chart.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,6 @@ maintainers:

type: application

version: 0.1.12
version: 0.1.13

appVersion: "v2.2.2-8f33fea"
2 changes: 2 additions & 0 deletions charts/nitro/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,8 @@

| Name | Description | Value |
| ---------------------------------------------------------- | -------------------------------------------------------- | -------------------- |
| `validator.image.repository` | Docker image repository | `""` |
| `validator.image.tag` | Docker image tag | `""` |
| `validator.enabled` | Enable the stateless validator | `false` |
| `validator.configmap.data.auth.addr` | Address to bind auth service to | `0.0.0.0` |
| `validator.configmap.data.auth.port` | Port to bind auth service to | `8549` |
Expand Down
4 changes: 2 additions & 2 deletions charts/nitro/templates/validator/statefulset.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ spec:
{{- end }}
containers:
- name: validator
image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
image: "{{ .Values.validator.image.repository | default .Values.image.repository }}:{{ .Values.validator.image.tag | default .Values.image.tag | default .Chart.AppVersion }}"
command: ["/usr/local/bin/nitro-val"]
args:
- --conf.env-prefix=NITRO
Expand Down Expand Up @@ -154,4 +154,4 @@ spec:
{{- if .Values.validator.statefulset.extraVolumes}}
{{- toYaml .Values.validator.statefulset.extraVolumes | nindent 6 }}
{{- end }}
{{- end }}
{{- end }}
15 changes: 15 additions & 0 deletions charts/nitro/values.schema.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -441,6 +441,21 @@
"validator": {
"type": "object",
"properties": {
"image": {
"type": "object",
"properties": {
"repository": {
"type": "string",
"description": "Docker image repository",
"default": ""
},
"tag": {
"type": "string",
"description": "Docker image tag",
"default": ""
}
}
},
"enabled": {
"type": "boolean",
"description": "Enable the stateless validator",
Expand Down
5 changes: 5 additions & 0 deletions charts/nitro/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -179,6 +179,11 @@ configmap:
## @section Stateless Validator

validator:
image:
## @param validator.image.repository Docker image repository
repository: ""
## @param validator.image.tag Docker image tag
tag: ""
## @param validator.enabled Enable the stateless validator
enabled: false
configmap:
Expand Down