Skip to content
This repository was archived by the owner on Jul 30, 2021. It is now read-only.

Commit 2de5df2

Browse files
committed
e2e: Change from jsonPatch to MergePatch (former does not apply without error)
1 parent c61ca0c commit 2de5df2

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

e2e/checkpointer_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -222,8 +222,8 @@ func TestCheckpointerUnscheduleCheckpointer(t *testing.T) {
222222
}
223223

224224
// Delete the pod checkpointer on the worker node by updating the daemonset.
225-
patch = `[{"op": "replace", "path": "/spec/template/spec/nodeSelector", "value": {"node-role.kubernetes.io/master":""}}]`
226-
_, err = client.ExtensionsV1beta1().DaemonSets("kube-system").Patch("pod-checkpointer", types.JSONPatchType, []byte(patch))
225+
patch = `{"spec":{"template":{"spec":{"nodeSelector":{"node-role.kubernetes.io/master":""}}}}}`
226+
_, err = client.ExtensionsV1beta1().DaemonSets("kube-system").Patch("pod-checkpointer", types.MergePatchType, []byte(patch))
227227
if err != nil {
228228
t.Fatalf("Failed to patch checkpointer: %v", err)
229229
}
@@ -352,8 +352,8 @@ func TestCheckpointerUnscheduleParent(t *testing.T) {
352352
}
353353

354354
// Delete test pod on the workers.
355-
patch = `[{"op": "replace", "path": "/spec/template/spec/nodeSelector", "value": {"node-role.kubernetes.io/master":""}}]`
356-
_, err = client.ExtensionsV1beta1().DaemonSets(testNS).Patch("nginx-daemonset", types.JSONPatchType, []byte(patch))
355+
patch = `{"spec":{"template":{"spec":{"nodeSelector":{"node-role.kubernetes.io/master":""}}}}}`
356+
_, err = client.ExtensionsV1beta1().DaemonSets(testNS).Patch("nginx-daemonset", types.MergePatchType, []byte(patch))
357357
if err != nil {
358358
t.Fatalf("unable to patch daemonset: %v", err)
359359
}

0 commit comments

Comments
 (0)