Skip to content

Commit 881db22

Browse files
authored
refactor: remove method-use (#273)
Removes ksonnet-libs uses of the form `<instance>.withFoo` and replaces them with `<instance> + <type>.withFoo`, as this is required for migrating to jsonnet-libs/k8s
1 parent 3c07292 commit 881db22

File tree

2 files changed

+13
-16
lines changed

2 files changed

+13
-16
lines changed

k8s-node-termination-handler/handler.libsonnet

Lines changed: 12 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -38,33 +38,30 @@
3838
daemonset:
3939
daemonSet.new('node-termination-handler', [self.container]) +
4040
daemonSet.mixin.spec.template.spec.withServiceAccount(self.service_account.metadata.name) +
41-
daemonSet.mixin.spec.template.spec.affinity.nodeAffinity.requiredDuringSchedulingIgnoredDuringExecution.mixinInstance(
42-
nodeSelector.withNodeSelectorTerms([
41+
daemonSet.mixin.spec.template.spec.affinity.nodeAffinity.requiredDuringSchedulingIgnoredDuringExecution.withNodeSelectorTerms(
42+
[
4343
nodeSelector.nodeSelectorTermsType.new() +
4444
nodeSelector.nodeSelectorTermsType.withMatchExpressions([
45-
nodeSelector.nodeSelectorTermsType.matchFieldsType
46-
.withKey('cloud.google.com/gke-accelerator')
47-
.withOperator('Exists'),
45+
nodeSelector.nodeSelectorTermsType.matchFieldsType.withKey('cloud.google.com/gke-accelerator')
46+
+ nodeSelector.nodeSelectorTermsType.matchFieldsType.withOperator('Exists'),
4847
]),
4948
nodeSelector.nodeSelectorTermsType.new() +
5049
nodeSelector.nodeSelectorTermsType.withMatchExpressions([
51-
nodeSelector.nodeSelectorTermsType.matchFieldsType
52-
.withKey('cloud.google.com/gke-preemptible')
53-
.withOperator('Exists'),
50+
nodeSelector.nodeSelectorTermsType.matchFieldsType.withKey('cloud.google.com/gke-preemptible')
51+
+ nodeSelector.nodeSelectorTermsType.matchFieldsType.withOperator('Exists'),
5452
]),
55-
])
53+
]
5654
) +
5755
daemonSet.mixin.metadata.withNamespace(_config.namespace) +
5856
daemonSet.mixin.spec.template.spec.withHostPid(true) +
5957
daemonSet.mixin.spec.template.spec.withTolerations([
6058
tolerations.new() +
61-
tolerations
62-
.withOperator('Exists')
63-
.withEffect('NoSchedule'),
59+
tolerations.withOperator('Exists') +
60+
tolerations.withEffect('NoSchedule'),
61+
6462
tolerations.new() +
65-
tolerations
66-
.withOperator('Exists')
67-
.withEffect('NoExecute'),
63+
tolerations.withOperator('Exists') +
64+
tolerations.withEffect('NoExecute'),
6865
]),
6966

7067
local serviceAccount = k.core.v1.serviceAccount,

prometheus-ksonnet/lib/prometheus.libsonnet

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@
8181
local _config = self._config;
8282

8383
statefulset.new(self.name, 1, [
84-
self.prometheus_container.withVolumeMountsMixin(
84+
self.prometheus_container + container.withVolumeMountsMixin(
8585
volumeMount.new('%s-data' % self.name, '/prometheus')
8686
),
8787
self.prometheus_watch_container,

0 commit comments

Comments
 (0)