Skip to content

Commit 1632c6e

Browse files
fix(nx-cloud): adjust string templating
1 parent a5d46a4 commit 1632c6e

File tree

5 files changed

+29
-13
lines changed

5 files changed

+29
-13
lines changed

README.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,13 @@ This repo contains:
77
* Nx Cloud Helm Chart
88
* Nx Agents Helm Chart
99

10+
> Important: Breaking changes in nx-cloud chart version 1.0.0
11+
>
12+
> Starting with chart version 1.0.0, there are breaking changes to values and templates. Please review the migration guide before upgrading and plan for required adjustments.
13+
>
14+
> See charts/nx-cloud/MIGRATION.md for details: [MIGRATION.md](charts/nx-cloud/MIGRATION.md)
15+
16+
1017
### Compatibility Matrix
1118

1219
| Chart Version | Compatible Images |

charts/nx-cloud/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ apiVersion: v2
22
name: nx-cloud
33
description: Nx Cloud Helm Chart
44
type: application
5-
version: 1.0.0
5+
version: 1.0.1-rc1
66
maintainers:
77
- name: nx
88
url: "https://nx.app/"

charts/nx-cloud/README.md

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,16 @@ Controller service for Nx Agents).
77
>
88
> Starting with chart version 1.0.0, there are breaking changes to values and templates. Please review the migration guide before upgrading and plan for required adjustments.
99
>
10-
> See MIGRATION.md for details: ./MIGRATION.md
10+
> See MIGRATION.md for details: [MIGRATION.md](MIGRATION.md)
11+
12+
## Prerequisites
13+
* MongoDB 7+
14+
* Valkey 7.2+
15+
* S3-compatible storage / Google Cloud Storage / Azure Blob Storage (recommended) **OR** built-in file server (not recommended for production)
16+
17+
## Object storage
18+
We recommend using a S3-compatible storage / Google Cloud Storage / Azure Blob Storage for production deployments.
19+
However, if you do not have a storage solution available, you can enable the built-in file server by setting `fileServer.enabled` to `true`.
1120

1221
## Values
1322

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
{{ range $name, $manifest := .Values.extraObjects }}
22
---
3-
{{ if typeIs "string" $manifest }}
4-
{{- tpl $manifest $ }}
3+
{{- if typeIs "string" $manifest }}
4+
{{- $manifest }}
55
{{- else }}
6-
{{- tpl (toYaml $manifest) $ }}
6+
{{- toYaml $manifest }}
77
{{- end }}
88
{{ end }}

charts/nx-cloud/templates/configs/valkey-configmap.yaml

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,11 @@ metadata:
77
{{- include "nxCloud.mergedLabels" (dict "resourceLabels" .Values.api.deployment.labels "context" .) | nindent 4 }}
88
data:
99
USE_VALKEY: "true"
10-
VALKEY_CLIENT_PROVIDER: "{{ .Values.api.valkey.clientProvider }}"
11-
VALKEY_USE_SENTINEL: "{{ ternary "true" "false" .Values.api.valkey.useSentinel }}"
12-
VALKEY_SENTINEL_MASTER_NAME: "{{ .Values.api.valkey.sentinelMasterName }}"
13-
VALKEY_PRIMARY_ADDRESS: "{{ .Values.api.valkey.primaryAddress }}"
14-
VALKEY_REPLICA_ADDRESSES: "{{ .Values.api.valkey.replicaAddresses }}"
15-
VALKEY_PORT: "{{ .Values.api.valkey.port }}"
16-
VALKEY_SENTINEL_PORT: "{{ .Values.api.valkey.sentinelPort }}"
17-
VALKEY_USERNAME: "{{ .Values.api.valkey.username }}"
10+
VALKEY_CLIENT_PROVIDER: {{ .Values.api.valkey.clientProvider | quote }}
11+
VALKEY_USE_SENTINEL: {{ ternary "true" "false" .Values.api.valkey.useSentinel | quote }}
12+
VALKEY_SENTINEL_MASTER_NAME: {{ .Values.api.valkey.sentinelMasterName | quote }}
13+
VALKEY_PRIMARY_ADDRESS: {{ .Values.api.valkey.primaryAddress | quote }}
14+
VALKEY_REPLICA_ADDRESSES: {{ .Values.api.valkey.replicaAddresses | quote }}
15+
VALKEY_PORT: {{ .Values.api.valkey.port | quote }}
16+
VALKEY_SENTINEL_PORT: {{ .Values.api.valkey.sentinelPort | quote }}
17+
VALKEY_USERNAME: {{ .Values.api.valkey.username | quote }}

0 commit comments

Comments
 (0)