Skip to content

Commit 207faee

Browse files
reevejdreevejd
committed
style(helm-chart): add well-known annotations and labels
1 parent 2e95f8d commit 207faee

File tree

6 files changed

+18
-17
lines changed

6 files changed

+18
-17
lines changed

‎deploy/cert-manager-ibm-cis-webhook/templates/apiservice.yaml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ kind: APIService
33
metadata:
44
name: v1alpha1.{{ .Values.groupName }}
55
labels:
6-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
6+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
77
chart: {{ include "cert-manager-ibm-cis-webhook.chart" . }}
88
release: {{ .Release.Name }}
99
heritage: {{ .Release.Service }}

‎deploy/cert-manager-ibm-cis-webhook/templates/deployment.yaml‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,22 +4,23 @@ metadata:
44
name: {{ include "cert-manager-ibm-cis-webhook.fullname" . }}
55
namespace: {{ .Release.Namespace | quote }}
66
labels:
7-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
7+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
88
chart: {{ include "cert-manager-ibm-cis-webhook.chart" . }}
99
release: {{ .Release.Name }}
1010
heritage: {{ .Release.Service }}
1111
spec:
1212
replicas: {{ .Values.replicaCount }}
1313
selector:
1414
matchLabels:
15-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
15+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
1616
release: {{ .Release.Name }}
1717
template:
1818
metadata:
1919
labels:
20-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
20+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
2121
release: {{ .Release.Name }}
2222
annotations:
23+
kubectl.kubernetes.io/default-container: {{ .Chart.Name }}
2324
checksum/secrets: {{ include (print $.Template.BasePath "/secret.yaml") $ | sha256sum }}
2425

2526
spec:

‎deploy/cert-manager-ibm-cis-webhook/templates/networkpolicies.yaml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,15 +7,15 @@ metadata:
77
spec:
88
podSelector:
99
matchLabels:
10-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
10+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
1111
policyTypes:
1212
- Egress
1313
egress:
1414
- to:
1515
- namespaceSelector: {}
1616
podSelector:
1717
matchLabels:
18-
k8s-app: kube-dns
18+
k8s-app.kubernetes.io/name: kube-dns
1919
ports:
2020
- protocol: UDP
2121
port: 53

‎deploy/cert-manager-ibm-cis-webhook/templates/pki.yaml‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ metadata:
77
name: {{ include "cert-manager-ibm-cis-webhook.selfSignedIssuer" . }}
88
namespace: {{ .Release.Namespace | quote }}
99
labels:
10-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
10+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
1111
chart: {{ include "cert-manager-ibm-cis-webhook.chart" . }}
1212
release: {{ .Release.Name }}
1313
heritage: {{ .Release.Service }}
@@ -23,7 +23,7 @@ metadata:
2323
name: {{ include "cert-manager-ibm-cis-webhook.rootCACertificate" . }}
2424
namespace: {{ .Release.Namespace | quote }}
2525
labels:
26-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
26+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
2727
chart: {{ include "cert-manager-ibm-cis-webhook.chart" . }}
2828
release: {{ .Release.Name }}
2929
heritage: {{ .Release.Service }}
@@ -44,7 +44,7 @@ metadata:
4444
name: {{ include "cert-manager-ibm-cis-webhook.rootCAIssuer" . }}
4545
namespace: {{ .Release.Namespace | quote }}
4646
labels:
47-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
47+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
4848
chart: {{ include "cert-manager-ibm-cis-webhook.chart" . }}
4949
release: {{ .Release.Name }}
5050
heritage: {{ .Release.Service }}
@@ -61,7 +61,7 @@ metadata:
6161
name: {{ include "cert-manager-ibm-cis-webhook.servingCertificate" . }}
6262
namespace: {{ .Release.Namespace | quote }}
6363
labels:
64-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
64+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
6565
chart: {{ include "cert-manager-ibm-cis-webhook.chart" . }}
6666
release: {{ .Release.Name }}
6767
heritage: {{ .Release.Service }}

‎deploy/cert-manager-ibm-cis-webhook/templates/rbac.yaml‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ metadata:
44
name: {{ include "cert-manager-ibm-cis-webhook.fullname" . }}
55
namespace: {{ .Release.Namespace | quote }}
66
labels:
7-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
7+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
88
chart: {{ include "cert-manager-ibm-cis-webhook.chart" . }}
99
release: {{ .Release.Name }}
1010
heritage: {{ .Release.Service }}
@@ -18,7 +18,7 @@ metadata:
1818
name: {{ include "cert-manager-ibm-cis-webhook.fullname" . }}:webhook-authentication-reader
1919
namespace: kube-system
2020
labels:
21-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
21+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
2222
chart: {{ include "cert-manager-ibm-cis-webhook.chart" . }}
2323
release: {{ .Release.Name }}
2424
heritage: {{ .Release.Service }}
@@ -39,7 +39,7 @@ kind: ClusterRoleBinding
3939
metadata:
4040
name: {{ include "cert-manager-ibm-cis-webhook.fullname" . }}:auth-delegator
4141
labels:
42-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
42+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
4343
chart: {{ include "cert-manager-ibm-cis-webhook.chart" . }}
4444
release: {{ .Release.Name }}
4545
heritage: {{ .Release.Service }}
@@ -59,7 +59,7 @@ kind: ClusterRole
5959
metadata:
6060
name: {{ include "cert-manager-ibm-cis-webhook.fullname" . }}:domain-solver
6161
labels:
62-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
62+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
6363
chart: {{ include "cert-manager-ibm-cis-webhook.chart" . }}
6464
release: {{ .Release.Name }}
6565
heritage: {{ .Release.Service }}
@@ -76,7 +76,7 @@ kind: ClusterRoleBinding
7676
metadata:
7777
name: {{ include "cert-manager-ibm-cis-webhook.fullname" . }}:domain-solver
7878
labels:
79-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
79+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
8080
chart: {{ include "cert-manager-ibm-cis-webhook.chart" . }}
8181
release: {{ .Release.Name }}
8282
heritage: {{ .Release.Service }}

‎deploy/cert-manager-ibm-cis-webhook/templates/service.yaml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ metadata:
44
name: {{ include "cert-manager-ibm-cis-webhook.fullname" . }}
55
namespace: {{ .Release.Namespace | quote }}
66
labels:
7-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
7+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
88
chart: {{ include "cert-manager-ibm-cis-webhook.chart" . }}
99
release: {{ .Release.Name }}
1010
heritage: {{ .Release.Service }}
@@ -16,5 +16,5 @@ spec:
1616
protocol: TCP
1717
name: https
1818
selector:
19-
app: {{ include "cert-manager-ibm-cis-webhook.name" . }}
19+
app.kubernetes.io/name: {{ include "cert-manager-ibm-cis-webhook.name" . }}
2020
release: {{ .Release.Name }}

0 commit comments

Comments
 (0)