Skip to content

Commit ae06b18

Browse files
authored
Merge pull request #215 from philipz/master
chore: fixed helm install errors
2 parents 1bb1646 + 7a09bbe commit ae06b18

File tree

4 files changed

+29
-24
lines changed

4 files changed

+29
-24
lines changed

helm/README.md

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,14 @@ helm install quorum-monitoring-ingress ingress-nginx/ingress-nginx \
107107
--set defaultBackend.nodeSelector."kubernetes\.io/os"=linux \
108108
--set controller.admissionWebhooks.patch.nodeSelector."kubernetes\.io/os"=linux \
109109
--set controller.service.externalTrafficPolicy=Local
110+
```
110111

112+
Change Configmap of ingress nginx controller
113+
```
114+
kubectl edit -n quorum configmaps quorum-monitoring-ingress-ingress-nginx-controller
115+
```
116+
Then set `allow-snippet-annotations: "true"`
117+
```
111118
kubectl apply -f ../ingress/ingress-rules-monitoring.yml
112119
```
113120

@@ -256,7 +263,14 @@ helm install quorum-network-ingress ingress-nginx/ingress-nginx \
256263
--set defaultBackend.nodeSelector."kubernetes\.io/os"=linux \
257264
--set controller.admissionWebhooks.patch.nodeSelector."kubernetes\.io/os"=linux \
258265
--set controller.service.externalTrafficPolicy=Local
266+
```
259267
268+
Change Configmap of ingress nginx controller
269+
```
270+
kubectl edit -n quorum configmaps quorum-network-ingress-ingress-nginx-controller
271+
```
272+
Then set `allow-snippet-annotations: "true"`
273+
```
260274
kubectl apply -f ../ingress/ingress-rules-goquorum.yml
261275
```
262276

helm/charts/blockscout/templates/blockscout-statefulset.yaml

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -53,20 +53,11 @@ spec:
5353
selector:
5454
matchLabels:
5555
app: {{ template "blockscout.fullname" . }}
56-
release: {{ .Release.Name }}
57-
component: {{ .Release.Name }}
58-
app.kubernetes.io/name: {{ include "blockscout.fullname" . }}
59-
app.kubernetes.io/instance: {{ .Release.Name }}
6056
serviceName: {{ include "blockscout.fullname" . }}
6157
template:
6258
metadata:
6359
labels:
64-
app.kubernetes.io/name: blockscout
65-
app.kubernetes.io/component: statefulset
66-
app.kubernetes.io/part-of: {{ include "blockscout.fullname" . }}
67-
app.kubernetes.io/namespace: {{ .Release.Namespace }}
68-
app.kubernetes.io/release: {{ .Release.Name }}
69-
app.kubernetes.io/managed-by: helm
60+
app: {{ template "blockscout.fullname" . }}
7061
spec:
7162
serviceAccountName: {{ include "blockscout.fullname" . }}-sa
7263
containers:

helm/charts/goquorum-node/templates/node-hooks-service-account.yaml

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ metadata:
88
namespace: {{ .Release.Namespace }}
99
annotations:
1010
"helm.sh/hook-delete-policy": before-hook-creation
11-
"helm.sh/hook": "pre-install,pre-delete,post-delete"
11+
"helm.sh/hook": "pre-install,pre-delete,post-delete"
1212

1313
{{- end}}
1414

@@ -37,18 +37,18 @@ metadata:
3737
namespace: {{ .Release.Namespace }}
3838
annotations:
3939
"helm.sh/hook-delete-policy": before-hook-creation
40-
"helm.sh/hook": "pre-install,pre-delete,post-delete"
40+
"helm.sh/hook": "pre-install,pre-delete,post-delete"
4141
roleRef:
4242
apiGroup: rbac.authorization.k8s.io
4343
kind: Role
4444
name: {{ include "goquorum-node.fullname" . }}-hooks-role
4545
subjects:
46-
- kind: ServiceAccount
47-
namespace: {{ .Release.Namespace }}
48-
{{- if and (eq .Values.cluster.provider "azure") (.Values.cluster.cloudNativeServices) }}
46+
- kind: ServiceAccount
47+
namespace: {{ .Release.Namespace }}
48+
{{- if and (eq .Values.cluster.provider "azure") (.Values.cluster.cloudNativeServices) }}
4949
name: {{ .Values.azure.serviceAccountName }}
50-
{{- else if and (eq .Values.cluster.provider "aws") (.Values.cluster.cloudNativeServices) }}
50+
{{- else if and (eq .Values.cluster.provider "aws") (.Values.cluster.cloudNativeServices) }}
5151
name: {{ .Values.aws.serviceAccountName }}
52-
{{- else }}
52+
{{- else }}
5353
name: {{ include "goquorum-node.fullname" . }}-hooks-sa
54-
{{- end}}
54+
{{- end}}

helm/charts/goquorum-node/templates/node-service-account.yaml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,12 @@ roleRef:
3333
kind: Role
3434
name: {{ include "goquorum-node.fullname" . }}-role
3535
subjects:
36-
- kind: ServiceAccount
37-
namespace: {{ .Release.Namespace }}
38-
{{- if and (eq .Values.cluster.provider "azure") (.Values.cluster.cloudNativeServices) }}
36+
- kind: ServiceAccount
37+
namespace: {{ .Release.Namespace }}
38+
{{- if and (eq .Values.cluster.provider "azure") (.Values.cluster.cloudNativeServices) }}
3939
name: {{ .Values.azure.serviceAccountName }}
40-
{{- else if and (eq .Values.cluster.provider "aws") (.Values.cluster.cloudNativeServices) }}
40+
{{- else if and (eq .Values.cluster.provider "aws") (.Values.cluster.cloudNativeServices) }}
4141
name: {{ .Values.aws.serviceAccountName }}
42-
{{- else }}
42+
{{- else }}
4343
name: {{ include "goquorum-node.fullname" . }}-sa
44-
{{- end}}
44+
{{- end}}

0 commit comments

Comments
 (0)