Skip to content

Commit e050acf

Browse files
author
jfv
committed
#13 / #14 Support secured private registry and nodeselector
1 parent 4f8bfec commit e050acf

12 files changed

+111
-24
lines changed

README.md

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ helm repo add abcdesktop https://abcdesktopio.github.io/helm/
1616
Install chart
1717

1818
```
19-
helm install my-abcdesktop abcdesktop/abcdesktop --version 4.2.0 --create-namespace -n abcdesktop
19+
helm install my-abcdesktop abcdesktop/abcdesktop --version 4.2.1 --create-namespace -n abcdesktop
2020
```
2121

2222
## To connect
@@ -40,6 +40,7 @@ The following table contains the helm parameters:
4040

4141
| Key | Description | Default Value |
4242
| ------------------------------------- | ----------------------------------------- | ------------------------------------------- |
43+
| `imagePullSecrets` | Secret name to pull images | [] |
4344
| `console.image` | Docker image used for the console service | `ghcr.io/abcdesktopio/console` |
4445
| `console.tag` | Docker image tag | `4.2` |
4546
| `console.replicaCount` | Number of replicas for the console | `1` |
@@ -112,6 +113,7 @@ The following table contains the helm parameters:
112113
| `speedtest.resources.limits.memory` | Memory limit | `128Mi` |
113114
| `speedtest.resources.requests.cpu` | CPU request | `0.1` |
114115
| `speedtest.resources.requests.memory` | Memory request | `32Mi` |
116+
| `nodeSelector` | Node selector | {} |
115117
| `od_config` | configuration file for abcdesktop | *default configuration file* |
116118

117119
Note Secrets and ConfigMap should exists before helm deployment, if not it will be created.
@@ -138,16 +140,16 @@ and build package:
138140

139141
~~~ bash
140142
$ helm package ./abcdesktop/
141-
Successfully packaged chart and saved it to: abcdesktop-4.2.0.tgz
143+
Successfully packaged chart and saved it to: abcdesktop-4.2.1.tgz
142144
~~~
143145

144-
The helm file **abcdesktop-4.2.0.tgz** is created.
146+
The helm file **abcdesktop-4.2.1.tgz** is created.
145147

146148
Let's lint it:
147149

148150
~~~ bash
149-
$ helm lint abcdesktop-4.2.0.tgz
150-
==> Linting abcdesktop-4.2.0.tgz
151+
$ helm lint abcdesktop-4.2.1.tgz
152+
==> Linting abcdesktop-4.2.1.tgz
151153

152154
1 chart(s) linted, 0 chart(s) failed
153155
========================================
@@ -215,7 +217,7 @@ To list the available versions, run the command:
215217
~~~ bash
216218
helm search repo abcdesktop
217219
NAME CHART VERSION APP VERSION DESCRIPTION
218-
abcdesktop/abcdesktop 4.2.0 4.2.0 ABCDesktop helm chart
220+
abcdesktop/abcdesktop 4.2.1 4.2.1 ABCDesktop helm chart
219221
~~~
220222

221223
Then to install:
@@ -227,15 +229,15 @@ helm upgrade --install abcdesktop --create-namespace abcdesktop/abcdesktop -n ab
227229
### From local build
228230

229231
~~~ bash
230-
$ helm upgrade --install abcdesktop --create-namespace ./abcdesktop-4.2.0.tgz -n abcdesktop
232+
$ helm upgrade --install abcdesktop --create-namespace ./abcdesktop-4.2.1.tgz -n abcdesktop
231233
~~~
232234

233235
## Change default values
234236

235237
To disable local embedded openldap, if you are using your own ldap directory service
236238

237239
`
238-
helm install --set openldap.enabled=false my-abcdesktop abcdesktop/abcdesktop --version 4.2.0 --create-namespace -n abcdesktop
240+
helm install --set openldap.enabled=false my-abcdesktop abcdesktop/abcdesktop --version 4.2.1 --create-namespace -n abcdesktop
239241
`
240242

241243

charts/abcdesktop/Chart.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@ apiVersion: v2
22
name: abcdesktop
33
description: ABCDesktop helm chart
44
type: application
5-
version: 4.2.0
6-
appVersion: "4.2.0"
5+
version: 4.2.1
6+
appVersion: "4.2.1"
77
icon: https://www.abcdesktop.io/img/abcdesktop.svg
88

99
annotations:

charts/abcdesktop/README.md

Lines changed: 21 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
# ABCDesktop helm package
22

3+
[![Artifact Hub](https://img.shields.io/endpoint?url=https://artifacthub.io/badge/repository/abcdesktop)](https://artifacthub.io/packages/search?repo=abcdesktop)
4+
35
ABCDesktop is a cloud native desktopless service, and a complete work environment accessible from a simple HTML 5 web browser, without any installation. Have a look on https://www.abcdesktop.io/ to get more informations.
46

57
## To install
@@ -14,7 +16,7 @@ helm repo add abcdesktop https://abcdesktopio.github.io/helm/
1416
Install chart
1517

1618
```
17-
helm install my-abcdesktop abcdesktop/abcdesktop --version 4.2.0 --create-namespace -n abcdesktop
19+
helm install my-abcdesktop abcdesktop/abcdesktop --version 4.2.1 --create-namespace -n abcdesktop
1820
```
1921

2022
## To connect
@@ -38,6 +40,7 @@ The following table contains the helm parameters:
3840

3941
| Key | Description | Default Value |
4042
| ------------------------------------- | ----------------------------------------- | ------------------------------------------- |
43+
| `imagePullSecrets` | Secret name to pull images | [] |
4144
| `console.image` | Docker image used for the console service | `ghcr.io/abcdesktopio/console` |
4245
| `console.tag` | Docker image tag | `4.2` |
4346
| `console.replicaCount` | Number of replicas for the console | `1` |
@@ -71,6 +74,7 @@ The following table contains the helm parameters:
7174
| `website.autoscaling.maxReplicas` | Maximum number of replicas allowed | `100` |
7275
| `website.autoscaling.targetCPUUtilizationPercentage`| CPU utilization percentage to trigger autoscaling | `80` |
7376
| `website.autoscaling.targetMemoryUtilizationPercentage`| Memory utilization percentage to trigger autoscaling (commented, not active by default) | `80` |
77+
| `openldap.enabled` | boolean `true` or `false` to enable embedded openldap | `true` |
7478
| `openldap.image` | Docker image for OpenLDAP | `ghcr.io/abcdesktopio/docker-test-openldap` |
7579
| `openldap.tag` | Docker image tag | `4.2` |
7680
| `openldap.replicaCount` | Number of replicas | `1` |
@@ -109,6 +113,7 @@ The following table contains the helm parameters:
109113
| `speedtest.resources.limits.memory` | Memory limit | `128Mi` |
110114
| `speedtest.resources.requests.cpu` | CPU request | `0.1` |
111115
| `speedtest.resources.requests.memory` | Memory request | `32Mi` |
116+
| `nodeSelector` | Node selector | {} |
112117
| `od_config` | configuration file for abcdesktop | *default configuration file* |
113118

114119
Note Secrets and ConfigMap should exists before helm deployment, if not it will be created.
@@ -135,16 +140,16 @@ and build package:
135140

136141
~~~ bash
137142
$ helm package ./abcdesktop/
138-
Successfully packaged chart and saved it to: abcdesktop-4.2.0.tgz
143+
Successfully packaged chart and saved it to: abcdesktop-4.2.1.tgz
139144
~~~
140145

141-
The helm file **abcdesktop-4.2.0.tgz** is created.
146+
The helm file **abcdesktop-4.2.1.tgz** is created.
142147

143148
Let's lint it:
144149

145150
~~~ bash
146-
$ helm lint abcdesktop-4.2.0.tgz
147-
==> Linting abcdesktop-4.2.0.tgz
151+
$ helm lint abcdesktop-4.2.1.tgz
152+
==> Linting abcdesktop-4.2.1.tgz
148153

149154
1 chart(s) linted, 0 chart(s) failed
150155
========================================
@@ -212,7 +217,7 @@ To list the available versions, run the command:
212217
~~~ bash
213218
helm search repo abcdesktop
214219
NAME CHART VERSION APP VERSION DESCRIPTION
215-
abcdesktop/abcdesktop 4.2.0 4.2.0 ABCDesktop helm chart
220+
abcdesktop/abcdesktop 4.2.1 4.2.1 ABCDesktop helm chart
216221
~~~
217222

218223
Then to install:
@@ -224,9 +229,18 @@ helm upgrade --install abcdesktop --create-namespace abcdesktop/abcdesktop -n ab
224229
### From local build
225230

226231
~~~ bash
227-
$ helm upgrade --install abcdesktop --create-namespace ./abcdesktop-4.2.0.tgz -n abcdesktop
232+
$ helm upgrade --install abcdesktop --create-namespace ./abcdesktop-4.2.1.tgz -n abcdesktop
228233
~~~
229234

235+
## Change default values
236+
237+
To disable local embedded openldap, if you are using your own ldap directory service
238+
239+
`
240+
helm install --set openldap.enabled=false my-abcdesktop abcdesktop/abcdesktop --version 4.2.1 --create-namespace -n abcdesktop
241+
`
242+
243+
230244
## Uninstall
231245

232246
~~~ bash

charts/abcdesktop/templates/console-deployment.yaml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,10 @@ spec:
1717
run: console-od
1818
type: admin-console
1919
spec:
20+
{{- with .Values.imagePullSecrets }}
21+
imagePullSecrets:
22+
{{- toYaml . | nindent 8 }}
23+
{{- end }}
2024
containers:
2125
- name: console
2226
image: {{ .Values.console.image }}:{{ .Values.console.tag }}
@@ -33,4 +37,8 @@ spec:
3337
memory: {{ .Values.console.resources.limits.memory }}
3438
requests:
3539
cpu: {{ .Values.console.resources.requests.cpu }}
36-
memory: {{ .Values.console.resources.requests.memory }}
40+
memory: {{ .Values.console.resources.requests.memory }}
41+
{{- with .Values.nodeSelector }}
42+
nodeSelector:
43+
{{- toYaml . | nindent 8 }}
44+
{{- end }}

charts/abcdesktop/templates/memcached-deployment.yaml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@ spec:
1616
run: memcached-od
1717
type: database
1818
spec:
19+
{{- with .Values.imagePullSecrets }}
20+
imagePullSecrets:
21+
{{- toYaml . | nindent 8 }}
22+
{{- end }}
1923
containers:
2024
- name: memcached
2125
image: {{ .Values.memcached.image }}:{{ .Values.memcached.tag }}
@@ -27,4 +31,8 @@ spec:
2731
cpu: {{ .Values.memcached.resources.requests.cpu }}
2832
memory: {{ .Values.memcached.resources.requests.memory }}
2933
ports:
30-
- containerPort: 11211
34+
- containerPort: 11211
35+
{{- with .Values.nodeSelector }}
36+
nodeSelector:
37+
{{- toYaml . | nindent 8 }}
38+
{{- end }}

charts/abcdesktop/templates/mongo-deployment.yaml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,10 @@ spec:
1919
run: mongodb-od
2020
type: database
2121
spec:
22+
{{- with .Values.imagePullSecrets }}
23+
imagePullSecrets:
24+
{{- toYaml . | nindent 8 }}
25+
{{- end }}
2226
containers:
2327
- name: mongodb
2428
image: {{ .Values.mongo.image }}:{{ .Values.mongo.tag }}
@@ -66,7 +70,11 @@ spec:
6670
mode: 0444
6771
- name: mongodb-scripts
6872
configMap:
69-
name: configmap-mongodb-scripts
73+
name: configmap-mongodb-scripts
7074
items:
7175
- key: ensure-users.js
72-
path: ensure-users.js
76+
path: ensure-users.js
77+
{{- with .Values.nodeSelector }}
78+
nodeSelector:
79+
{{- toYaml . | nindent 8 }}
80+
{{- end }}

charts/abcdesktop/templates/openldap-deployment.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,10 @@ spec:
1818
run: openldap-od
1919
netpol/dns: 'true'
2020
spec:
21+
{{- with .Values.imagePullSecrets }}
22+
imagePullSecrets:
23+
{{- toYaml . | nindent 8 }}
24+
{{- end }}
2125
containers:
2226
- name: openldap
2327
# abcdesktopio/docker-test-openldap:master
@@ -34,4 +38,8 @@ spec:
3438
ports:
3539
- containerPort: 389
3640
- containerPort: 636
41+
{{- with .Values.nodeSelector }}
42+
nodeSelector:
43+
{{- toYaml . | nindent 8 }}
44+
{{- end }}
3745
{{- end }}

charts/abcdesktop/templates/py-od-deployment.yaml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,10 @@ spec:
2929
netpol/router: 'true'
3030
netpol/nginx: 'true'
3131
spec:
32+
{{- with .Values.imagePullSecrets }}
33+
imagePullSecrets:
34+
{{- toYaml . | nindent 8 }}
35+
{{- end }}
3236
serviceAccountName: pyos-serviceaccount
3337
containers:
3438
- name : pyos
@@ -134,4 +138,8 @@ spec:
134138
- name: volume-abcdesktop-gshadow-templatefile
135139
configMap:
136140
name: abcdesktop-gshadow-templatefile
137-
dnsPolicy: ClusterFirst
141+
dnsPolicy: ClusterFirst
142+
{{- with .Values.nodeSelector }}
143+
nodeSelector:
144+
{{- toYaml . | nindent 8 }}
145+
{{- end }}

charts/abcdesktop/templates/router-od-deployment.yaml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,10 @@ spec:
2525
netpol/nginx: 'true'
2626
netpol/console: 'true'
2727
spec:
28+
{{- with .Values.imagePullSecrets }}
29+
imagePullSecrets:
30+
{{- toYaml . | nindent 8 }}
31+
{{- end }}
2832
containers:
2933
- name: nginx-router
3034
imagePullPolicy: Always
@@ -100,4 +104,8 @@ spec:
100104
- name: jwtpayloadkeys
101105
secret:
102106
secretName: abcdesktopjwtdesktoppayload
103-
dnsPolicy: ClusterFirst
107+
dnsPolicy: ClusterFirst
108+
{{- with .Values.nodeSelector }}
109+
nodeSelector:
110+
{{- toYaml . | nindent 8 }}
111+
{{- end }}

charts/abcdesktop/templates/speedtest-deployment.yaml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,12 @@ spec:
1414
metadata:
1515
namespace: {{ .Release.Namespace }}
1616
labels:
17-
run: speedtest-od
17+
run: speedtest-od
1818
spec:
19+
{{- with .Values.imagePullSecrets }}
20+
imagePullSecrets:
21+
{{- toYaml . | nindent 8 }}
22+
{{- end }}
1923
containers:
2024
- name: speedtest
2125
resources:
@@ -28,3 +32,7 @@ spec:
2832
image: {{ .Values.speedtest.image }}:{{ .Values.speedtest.tag }}
2933
ports:
3034
- containerPort: 8080
35+
{{- with .Values.nodeSelector }}
36+
nodeSelector:
37+
{{- toYaml . | nindent 8 }}
38+
{{- end }}

0 commit comments

Comments
 (0)