Skip to content

Commit 40e0687

Browse files
committed
jsonnet: code cleanup
1 parent 96430c9 commit 40e0687

File tree

6 files changed

+26
-18
lines changed

6 files changed

+26
-18
lines changed

jsonnet/kube-prometheus/components/alertmanager.libsonnet

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -137,9 +137,9 @@ function(params) {
137137
{ name: 'web', targetPort: 'web', port: 9093 },
138138
{ name: 'reloader-web', port: am._config.reloaderPort, targetPort: 'reloader-web' },
139139
],
140-
selector: {
140+
selector: am._config.selectorLabels {
141141
alertmanager: am._config.name,
142-
} + am._config.selectorLabels,
142+
},
143143
sessionAffinity: 'ClientIP',
144144
},
145145
},
@@ -150,9 +150,9 @@ function(params) {
150150
metadata: am._metadata,
151151
spec: {
152152
selector: {
153-
matchLabels: {
153+
matchLabels: am._config.selectorLabels {
154154
alertmanager: am._config.name,
155-
} + am._config.selectorLabels,
155+
},
156156
},
157157
endpoints: [
158158
{ port: 'web', interval: '30s' },
@@ -168,9 +168,9 @@ function(params) {
168168
spec: {
169169
maxUnavailable: 1,
170170
selector: {
171-
matchLabels: {
171+
matchLabels: am._config.selectorLabels {
172172
alertmanager: am._config.name,
173-
} + am._config.selectorLabels,
173+
},
174174
},
175175
},
176176
},
@@ -189,11 +189,11 @@ function(params) {
189189
version: am._config.version,
190190
image: am._config.image,
191191
podMetadata: {
192-
labels: am._metadata.labels,
192+
labels: am.alertmanager.metadata.labels,
193193
},
194194
resources: am._config.resources,
195195
nodeSelector: { 'kubernetes.io/os': 'linux' },
196-
serviceAccountName: am._metadata.name,
196+
serviceAccountName: am.serviceAccount.metadata.name,
197197
securityContext: {
198198
runAsUser: 1000,
199199
runAsNonRoot: true,

jsonnet/kube-prometheus/components/blackbox-exporter.libsonnet

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -213,7 +213,9 @@ function(params) {
213213
metadata: bb._metadata,
214214
spec: {
215215
replicas: bb._config.replicas,
216-
selector: { matchLabels: bb._config.selectorLabels },
216+
selector: {
217+
matchLabels: bb._config.selectorLabels
218+
},
217219
template: {
218220
metadata: {
219221
labels: bb._config.commonLabels,

jsonnet/kube-prometheus/components/kube-state-metrics.libsonnet

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,9 @@ function(params) (import 'github.com/kubernetes/kube-state-metrics/jsonnet/kube-
143143
metadata: ksm._metadata,
144144
spec: {
145145
jobLabel: 'app.kubernetes.io/name',
146-
selector: { matchLabels: ksm._config.selectorLabels },
146+
selector: {
147+
matchLabels: ksm._config.selectorLabels
148+
},
147149
endpoints: [
148150
{
149151
port: 'https-main',

jsonnet/kube-prometheus/components/node-exporter.libsonnet

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -209,7 +209,9 @@ function(params) {
209209
kind: 'DaemonSet',
210210
metadata: ne._metadata,
211211
spec: {
212-
selector: { matchLabels: ne._config.selectorLabels },
212+
selector: {
213+
matchLabels: ne._config.selectorLabels
214+
},
213215
updateStrategy: {
214216
type: 'RollingUpdate',
215217
rollingUpdate: { maxUnavailable: '10%' },

jsonnet/kube-prometheus/components/prometheus-adapter.libsonnet

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,9 @@ function(params) {
232232
metadata: pa._metadata,
233233
spec: {
234234
replicas: pa._config.replicas,
235-
selector: { matchLabels: pa._config.selectorLabels },
235+
selector: {
236+
matchLabels: pa._config.selectorLabels
237+
},
236238
strategy: {
237239
rollingUpdate: {
238240
maxSurge: 1,

jsonnet/kube-prometheus/components/prometheus.libsonnet

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -120,11 +120,11 @@ function(params) {
120120
roleRef: {
121121
apiGroup: 'rbac.authorization.k8s.io',
122122
kind: 'Role',
123-
name: 'prometheus-' + p._config.name,
123+
name: p._metadata.name,
124124
},
125125
subjects: [{
126126
kind: 'ServiceAccount',
127-
name: 'prometheus-' + p._config.name,
127+
name: p.serviceAccount.metadata.name,
128128
namespace: p._config.namespace,
129129
}],
130130
};
@@ -236,9 +236,9 @@ function(params) {
236236
spec: {
237237
minAvailable: 1,
238238
selector: {
239-
matchLabels: {
239+
matchLabels: p._config.selectorLabels {
240240
prometheus: p._config.name,
241-
} + p._config.selectorLabels,
241+
},
242242
},
243243
},
244244
},
@@ -255,11 +255,11 @@ function(params) {
255255
version: p._config.version,
256256
image: p._config.image,
257257
podMetadata: {
258-
labels: p._metadata.labels,
258+
labels: p.prometheus.metadata.labels,
259259
},
260260
externalLabels: p._config.externalLabels,
261261
enableFeatures: p._config.enableFeatures,
262-
serviceAccountName: p._metadata.name,
262+
serviceAccountName: p.serviceAccount.metadata.name,
263263
podMonitorSelector: {},
264264
podMonitorNamespaceSelector: {},
265265
probeSelector: {},

0 commit comments

Comments
 (0)